Skip to content

fix(ci): pin GRDB fork back to 3.0.1 and stop dependabot downgrades - #8

Merged
asmuelle merged 1 commit into
mainfrom
fix/ci-grdb-pin
Jul 5, 2026
Merged

fix(ci): pin GRDB fork back to 3.0.1 and stop dependabot downgrades#8
asmuelle merged 1 commit into
mainfrom
fix/ci-grdb-pin

Conversation

@asmuelle

@asmuelle asmuelle commented Jul 5, 2026

Copy link
Copy Markdown
Owner

Summary

  • Revert the GRDB.swift fork pin from 6.6.0 back to 3.0.1 — the fork's tags are not comparable semver: 3.0.1 vendors GRDB 7.4.1 + SQLCipher 4.7.0, while 6.6.0 vendors the older upstream GRDB 6.6.0, whose strcmp calls no longer compile on Swift 6.2.
  • Dependabot's tag-order bump (build(deps): bump github.com/thebrowsercompany/grdb.swift from 3.0.1 to 6.6.0 #6) therefore silently downgraded the at-rest crypto layer and broke both CI and CodeQL autobuild on main (cannot find 'strcmp' in scope), while PR Feat/spoken summary confirm loop #7's head — still on 3.0.1 — built green on the same runner image.
  • Add a dependabot ignore for this dependency plus comments in Package.swift so the pin is only ever bumped manually after a local build.

Test plan

  • swift package resolve re-pins revision e4135b5 (3.0.1)
  • swift test: 249 tests in 41 suites pass locally
  • CI green on this PR (the real proof — the PR merge ref now builds with 3.0.1)

🤖 Generated with Claude Code

The thebrowsercompany/GRDB.swift fork's tags are not comparable semver:
"3.0.1" vendors GRDB 7.4.1 + SQLCipher 4.7.0, while "6.6.0" vendors the
older upstream GRDB 6.6.0, whose C-string calls (strcmp) no longer compile
on Swift 6.2. Dependabot's tag-order "bump" (#6) therefore downgraded the
at-rest crypto layer and broke CI and CodeQL autobuild on main, while the
PR #7 head (still on 3.0.1) built green on the same runner.

Revert the pin and ignore this dependency in dependabot; bump it manually
after a local build only.
@asmuelle
asmuelle merged commit dd8018c into main Jul 5, 2026
4 checks passed
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