Repository navigation
fix(ci): pin pnpm/action-setup version in BetterDependabot workflow - #26
Merged
Merged
Conversation
The BetterDependabot scheduled workflow used version: latest for pnpm/action-setup. On 2026-07-13 that resolved to pnpm 11.12.0, whose self-installer crashes with "Cannot use 'in' operator to search for 'integrity' in undefined" while switching pnpm versions, failing the job before any dependency install or build step runs (run 29232206286). Pin to the last known-good release (11.11.0) instead of trusting "latest", matching how originGUI-build.yml already pins its pnpm version.
There was a problem hiding this comment.
Pull request overview
This PR addresses a CI failure in the scheduled BetterDependabot workflow by avoiding pnpm “latest” auto-resolution (which pulled a broken upstream release) and restores the workflow’s ability to run dependency updates end-to-end.
Changes:
- Pin
pnpm/action-setup’s installed pnpm version inBetterDependabot.ymlfromlatestto11.11.0. - Update
originGUI’spostcssdevDependency from^8.5.16to^8.5.17.
Reviewed changes
Copilot reviewed 2 out of 2 changed files in this pull request and generated 2 comments.
| File | Description |
|---|---|
.github/workflows/BetterDependabot.yml |
Pins pnpm to a known-good version to prevent CI breakage from upstream latest installs. |
originGUI/package.json |
Includes a patch-level postcss devDependency bump alongside the CI change. |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
Comment on lines
+29
to
+31
| - uses: pnpm/action-setup@v6 | ||
| with: | ||
| version: latest | ||
| version: 11.11.0 |
Comment on lines
29
to
32
| "eslint-plugin-n": "^18.2.2", | ||
| "eslint-plugin-promise": "^7.3.0", | ||
| "postcss": "^8.5.16", | ||
| "postcss": "^8.5.17", | ||
| "sass": "^1.101.0", |
Address Copilot review feedback on PR #26 by quoting the pnpm version pin so it's unambiguously treated as a string, consistent with the node-version quoting style already used in this workflow.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Root cause
CI was red because the
BetterDependabotscheduled workflow (.github/workflows/BetterDependabot.yml) usedpnpm/action-setup@v6withversion: latest. On 2026-07-13 that resolved to the freshly-published pnpm11.12.0, whose self-installer crashes while switching pnpm versions:This happens inside pnpm's own self-install/version-switch logic, before any repo dependency install, typecheck, or build step ever runs (see failed run 29232206286, and the failing
update-dependenciescheck on themasterHEAD commit).Hypotheses considered
playorig.inuses Vercel Analytics, unrelated to this repo's build). Dependabot did bumptypescript6.0.3 → 7.0.2across all 4 packages (PRs [typings]: Bump typescript from 6.0.3 to 7.0.2 in /typings #22–[extension/firefox]: Bump typescript from 6.0.3 to 7.0.2 in /extension/firefox #25) sincedependabot.ymlhas no major-version ignore rule — that part of the theory is real — but the actual build workflow (Autobuild originGUI) ran and succeeded on 2026-07-10, after that bump landed. TypeScript v7 is not what's failing CI here.BetterDependabot.yml's use ofversion: latestforpnpm/action-setuplet a brand-new, broken pnpm release (11.12.0) into CI. This is the actual failure.originGUI-build.yml,dependency-review.yml) is failing;gh api .../check-runson HEAD shows exactly one failing check,update-dependencies, matching hypothesis 2.latestwill hit it until pnpm patches it), not transient.Fix
Pin
pnpm/action-setupto the last known-good release,11.11.0, instead oflatest— the same patternoriginGUI-build.ymlalready uses (version: 9) for its own pnpm setup.Verification
masterHEAD (update-dependencies→failure, same "integrity" stack trace).BetterDependabot.ymlon this branch with the fix applied (gh workflow run→ run 29267895663): all steps passed, includingRun pnpm/action-setup@v6andCommit and push, which produced this branch'schore: update dependenciescommit — proof the pipeline runs end-to-end again.Test plan
BetterDependabot.ymlruns green on this branch (see run 29267895663 above)Autobuild originGUIalready succeeds withtypescript@7.0.2