Skip to content

chore(deps): backport go-libs migrator index name fix to release/v2.4#1334

Open
Dav-14 wants to merge 1 commit into
release/v2.4from
hotfix/v2.4/go-libs-migrator-index
Open

chore(deps): backport go-libs migrator index name fix to release/v2.4#1334
Dav-14 wants to merge 1 commit into
release/v2.4from
hotfix/v2.4/go-libs-migrator-index

Conversation

@Dav-14

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

Copy link
Copy Markdown
Contributor

Summary

Backport of the go-libs migrator index name fix to ledger release/v2.4. Bumps github.com/formancehq/go-libs/v4 from v4.0.0 to v4.1.4.

Pulls in formancehq/go-libs#597 (backport of #594 to release/v4) which fixes a hardcoded idx_version_id name collision between migrators sharing a schema. The bug crashes services with SQLSTATE 42P10 at startup. Affected DBs self-heal on next deploy — no manual SQL required.

Companion to #1331 (main), #1332 (release/v2.2), #1333 (release/v2.3).

Test plan

  • go mod tidy clean
  • go build ./... succeeds
  • CI green

🤖 Generated with Claude Code

Pulls in formancehq/go-libs#597 (backport to release/v4, tagged as
v4.1.4) which fixes a hardcoded `idx_version_id` name collision between
migrators sharing a schema. The bug crashes services with
`SQLSTATE 42P10` at startup. Affected DBs self-heal on next deploy -
no manual SQL required.

Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
@Dav-14 Dav-14 requested a review from a team as a code owner April 30, 2026 09:52
@coderabbitai

coderabbitai Bot commented Apr 30, 2026

Copy link
Copy Markdown
Contributor

Caution

Review failed

An error occurred during the review process. Please try again later.

✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch hotfix/v2.4/go-libs-migrator-index

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

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

@codecov

codecov Bot commented Apr 30, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 81.10%. Comparing base (14db440) to head (d62fd56).

Additional details and impacted files
@@               Coverage Diff                @@
##           release/v2.4    #1334      +/-   ##
================================================
+ Coverage         80.15%   81.10%   +0.94%     
================================================
  Files               205      205              
  Lines             11087    11071      -16     
================================================
+ Hits               8887     8979      +92     
+ Misses             1574     1551      -23     
+ Partials            626      541      -85     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

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.

2 participants