build(deps-dev): bump @anthropic-ai/claude-code from 2.1.259 to 2.1.266 - #71
dependabot[bot] wants to merge 1 commit into
Conversation
Bumps [@anthropic-ai/claude-code](https://github.com/anthropics/claude-code) from 2.1.259 to 2.1.266. - [Release notes](https://github.com/anthropics/claude-code/releases) - [Changelog](https://github.com/anthropics/claude-code/blob/main/CHANGELOG.md) - [Commits](anthropics/claude-code@v2.1.259...v2.1.266) --- updated-dependencies: - dependency-name: "@anthropic-ai/claude-code" dependency-version: 2.1.266 dependency-type: direct:development update-type: version-update:semver-patch ... Signed-off-by: dependabot[bot] <support@github.com>
|
❌ Claude's run did not complete (outcome: failure). View job run |
| }, | ||
| "devDependencies": { | ||
| "@anthropic-ai/claude-code": "2.1.259", | ||
| "@anthropic-ai/claude-code": "2.1.266", |
There was a problem hiding this comment.
🟠 Should fix [policy]
This bumps @anthropic-ai/claude-code to 2.1.266, but the version-pinned allowScripts entry below (line 43) still names the old version:
"allowScripts": {
"@anthropic-ai/claude-code@2.1.259": true
}npm 12's lifecycle-script gate keys allowScripts approvals by exact name@version. Since that key no longer matches the installed version, @anthropic-ai/claude-code@2.1.266's postinstall script will be silently blocked on the next npm ci/npm install — non-fatally (a log.warn, not a failed install), so CI stays green while the script quietly stops running.
This isn't a hypothetical stack default: the allowScripts entry was deliberately added in 2d35b3f specifically because .github/workflows/dependabot.yml's investigate job invokes this CLI directly via npx claude, so the postinstall script "genuinely needs to run." This PR reintroduces the exact gap that commit closed.
Suggested fix — update line 43 to:
| "@anthropic-ai/claude-code": "2.1.266", | |
| "@anthropic-ai/claude-code": "2.1.266", |
(and separately bump "@anthropic-ai/claude-code@2.1.259": true → "@anthropic-ai/claude-code@2.1.266": true on line 43, outside this diff's hunk)
Since the automatic-fix pass is off for this run: @claude please update the allowScriptskey on line 43 of package.json from"@anthropic-ai/claude-code@2.1.259": trueto"@anthropic-ai/claude-code@2.1.266": true to match this bump.
Superseded by re-review on the latest commit.
There was a problem hiding this comment.
Verdict: requesting changes. This is a clean, routine Dependabot patch bump (@anthropic-ai/claude-code 2.1.259 → 2.1.266). Verified the diff line by line: package.json:20 and every one of the 8 platform-specific optional dependencies in package-lock.json (darwin-arm64, darwin-x64, linux-arm64, linux-arm64-musl, linux-x64, linux-x64-musl, win32-arm64, win32-x64) bump consistently to 2.1.266 with matching integrity hashes, and no line in the diff is unaccounted for. Title/description are accurate as-is — left untouched.
One finding carries over from my prior review on this exact commit. Restating it here rather than duplicating the still-open inline thread on package.json:20, which I'm leaving unresolved since it still applies:
🟠 Should fix [policy] — package.json:43: the allowScripts entry is still pinned to the old version:
"allowScripts": {
"@anthropic-ai/claude-code@2.1.259": true
}npm 12's lifecycle-script gate keys approval by exact name@version, so 2.1.266's postinstall script will now silently no-op (a warning, not a failed npm ci, so this won't be caught by CI). This gate was added deliberately in 2d35b3f, and its own commit message explains why: .github/workflows/dependabot.yml's investigate job invokes this CLI directly via npx claude, which depends on that postinstall step having run. Fix:
"allowScripts": {
"@anthropic-ai/claude-code@2.1.266": true
}Note the suggestion block on the existing open thread doesn't actually apply this fix — it shows line 20 unchanged rather than editing line 43 — so don't apply it as-is; the corrected key above is the actual fix needed.
No other issues found.
Housekeeping: cleared and dismissed my own prior review on this same commit (superseded by this one) before submitting.
|
🗜️ Headroom context compression
|
|
Superseded by #74. |
Bumps @anthropic-ai/claude-code from 2.1.259 to 2.1.266.
Release notes
Sourced from @anthropic-ai/claude-code's releases.
... (truncated)
Changelog
Sourced from @anthropic-ai/claude-code's changelog.
... (truncated)
Commits
347b38echore: Update CHANGELOG.md and feed.xml8e02f6dchore: Update CHANGELOG.md and feed.xmlab9b2cfchore: Update CHANGELOG.md and feed.xmld7dbd9achore: Update CHANGELOG.md and feed.xmlb3f0e50chore: Update CHANGELOG.md and feed.xmlee2a058Merge pull request #91894 from williamqian12/frontend-design-skill-updatedbdd79cUpdate /frontend-design SKILL.mdDependabot will resolve any conflicts with this PR as long as you don't alter it yourself. You can also trigger a rebase manually by commenting
@dependabot rebase.Dependabot commands and options
You can trigger Dependabot actions by commenting on this PR:
@dependabot rebasewill rebase this PR@dependabot recreatewill recreate this PR, overwriting any edits that have been made to it@dependabot show <dependency name> ignore conditionswill show all of the ignore conditions of the specified dependency@dependabot ignore this major versionwill close this PR and stop Dependabot creating any more for this major version (unless you reopen the PR or upgrade to it yourself)@dependabot ignore this minor versionwill close this PR and stop Dependabot creating any more for this minor version (unless you reopen the PR or upgrade to it yourself)@dependabot ignore this dependencywill close this PR and stop Dependabot creating any more for this dependency (unless you reopen the PR or upgrade to it yourself)