Skip to content

fix(ci): sync interception fixture lockfile - #3119

Merged
james-elicx merged 1 commit into
mainfrom
fix/bigbonk-lockfile-20260827
Aug 27, 2026
Merged

fix(ci): sync interception fixture lockfile#3119
james-elicx merged 1 commit into
mainfrom
fix/bigbonk-lockfile-20260827

Conversation

@james-elicx

Copy link
Copy Markdown
Member

Summary

  • add the tests/fixtures/interception-source-authz workspace importer generated by pnpm
  • keep repository setup from dirtying pnpm-lock.yaml before Big Bonk checks out a pull-request branch

The fixture was added in #3078, but its empty workspace importer was not committed. Because Big Bonk runs vp install on main before its action switches to the requested PR branch, that drift made the checkout abort with pnpm-lock.yaml local changes.

Validation

  • vp install from a clean current origin/main reproduced the two-line lockfile drift
  • after this commit, vp install reports Already up to date and leaves the worktree clean
  • git diff --check

@james-elicx
james-elicx enabled auto-merge (squash) August 27, 2026 15:54
@pkg-pr-new

pkg-pr-new Bot commented Aug 27, 2026

Copy link
Copy Markdown

Open in StackBlitz

npm i https://pkg.pr.new/@vinext/cloudflare@3119
npm i https://pkg.pr.new/create-vinext-app@3119
npm i https://pkg.pr.new/@vinext/types@3119
npm i https://pkg.pr.new/vinext@3119

commit: 9d88dca

@github-actions

Copy link
Copy Markdown
Contributor
Example Preview Production Original
app-router-cloudflare preview production
pages-router-cloudflare preview production
app-router-playground preview production original
realworld-api-rest preview production
nextra-docs-template preview production
benchmarks preview production
hackernews preview production original
workers-cache preview production
static-export preview production
web preview production

@james-elicx
james-elicx merged commit 5d0b530 into main Aug 27, 2026
73 checks passed
@james-elicx
james-elicx deleted the fix/bigbonk-lockfile-20260827 branch August 27, 2026 15:58
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