Skip to content

chore(deps): bump go-libs to pick up migrator index name fix#139

Closed
Dav-14 wants to merge 1 commit into
mainfrom
fix/bump-go-libs-migrator-index-fix
Closed

chore(deps): bump go-libs to pick up migrator index name fix#139
Dav-14 wants to merge 1 commit into
mainfrom
fix/bump-go-libs-migrator-index-fix

Conversation

@Dav-14
Copy link
Copy Markdown
Contributor

@Dav-14 Dav-14 commented Apr 29, 2026

Summary

  • Bumps github.com/formancehq/go-libs/v3 to commit 95ea697f from PR formancehq/go-libs#596 (backport of #594 to release/v3.6).
  • Fixes a hardcoded idx_version_id index name collision between migrators sharing a Postgres schema (SQLSTATE 42P10 crash at startup). Affected DBs self-heal on next deploy.
  • Moves the pin from v3.0.x to v3.6.x; this should be re-pinned to a real v3.6.x tag once #596 is merged and tagged.

Test plan

  • go build ./... passes
  • just pre-commit passes (generate, tidy, lint)
  • CI green

🤖 Generated with Claude Code

Summary by CodeRabbit

  • Chores
    • Updated a module dependency to the latest patch version.

@coderabbitai
Copy link
Copy Markdown

coderabbitai Bot commented Apr 29, 2026

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Pro

Run ID: 43cbdb9a-d119-4a81-9cde-80d9860047f4

📥 Commits

Reviewing files that changed from the base of the PR and between eb9ce86 and f8212d5.

⛔ Files ignored due to path filters (1)
  • go.sum is excluded by !**/*.sum
📒 Files selected for processing (1)
  • go.mod

📝 Walkthrough

Walkthrough

The pull request updates a single direct module dependency in go.mod. The github.com/formancehq/go-libs/v3 dependency is incremented from version v3.6.1 to v3.6.2, with no other changes to the module requirements or replace directives.

Changes

Cohort / File(s) Summary
Dependency Version Bump
go.mod
Updated github.com/formancehq/go-libs/v3 from v3.6.1 to v3.6.2.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~2 minutes

Poem

🐰 A tiny hop, a version's leap,
From six-point-one to six-point-two,
Dependencies fresh and oh so neat,
The libraries grow, the code stays true! ✨

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title directly matches the main change: bumping the go-libs dependency to fix a migrator index name issue, which is exactly what the changeset accomplishes.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch fix/bump-go-libs-migrator-index-fix

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share
Review rate limit: 0/1 reviews remaining, refill in 60 minutes.

Comment @coderabbitai help to get the list of available commands and usage tips.

@Dav-14 Dav-14 force-pushed the fix/bump-go-libs-migrator-index-fix branch from bd0b493 to 1275e8e Compare April 29, 2026 15:49
Pulls in formancehq/go-libs#596 (backport of #594 to release/v3.6)
which fixes a hardcoded `idx_version_id` name collision between
migrators sharing a schema. Affected DBs self-heal on next deploy.

This bump also moves from v3.0.x to v3.6.x; should be re-pinned to a
real v3.6.x tag once #596 is merged.

Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
@Dav-14 Dav-14 force-pushed the fix/bump-go-libs-migrator-index-fix branch from 1275e8e to f8212d5 Compare April 29, 2026 16:31
@Dav-14 Dav-14 enabled auto-merge (squash) April 29, 2026 16:43
@Dav-14 Dav-14 closed this Apr 29, 2026
auto-merge was automatically disabled April 29, 2026 16:44

Pull request was closed

@Dav-14
Copy link
Copy Markdown
Contributor Author

Dav-14 commented Apr 29, 2026

Closing — this service does not use the go-libs publish/messaging package, so it never instantiates the CircuitBreakerStorageModule and isn't affected by the migrator index name collision fixed in formancehq/go-libs#594. The bump is cosmetic only; closing to keep the PR queue focused on impacted services.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant