Skip to content

Fail bundle check when frozen mode needs lockfile changes#9715

Open
hsbt wants to merge 1 commit into
masterfrom
claude/rubygems-issue-9546-9c100e
Open

Fail bundle check when frozen mode needs lockfile changes#9715
hsbt wants to merge 1 commit into
masterfrom
claude/rubygems-issue-9546-9c100e

Conversation

@hsbt

@hsbt hsbt commented Jul 22, 2026

Copy link
Copy Markdown
Member

BUNDLE_FROZEN=1 bundle check printed "The Gemfile's dependencies are satisfied" and exited 0 even when the lockfile was missing a CHECKSUMS entry, because the check command never ran the frozen validation and write_lock only warns in frozen mode. This makes bundle check call ensure_equivalent_gemfile_and_lockfile like bundle install does, so it now fails with the same message and exit status 16. The dedicated branch for frozen mode without a lockfile is removed since validate_platforms! raised first and made it unreachable. Running the frozen validation before validate_runtime! also improves that case to report "Frozen mode is set, but there's no lockfile".

Fixes #9546

Copilot AI review requested due to automatic review settings July 22, 2026 06:46

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Copilot was unable to review this pull request because the user who requested the review has reached their quota limit.

`BUNDLE_FROZEN=1 bundle check` reported that dependencies are satisfied and exited 0 even when the lockfile was missing CHECKSUMS entries, because `write_lock` only warns in frozen mode. Run the same frozen validation as `bundle install` so `bundle check` fails consistently.

#9546

Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
@hsbt
hsbt force-pushed the claude/rubygems-issue-9546-9c100e branch from 5e232ce to 07337df Compare July 23, 2026 01:55
Copilot AI review requested due to automatic review settings July 23, 2026 01:55

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Copilot was unable to review this pull request because the user who requested the review has reached their quota limit.

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

Projects

None yet

Development

Successfully merging this pull request may close these issues.

BUNDLE_FROZEN=1 bundle check exits 0 when some CHECKSUMS entry is missing

2 participants