build(deps-dev): bump @anthropic-ai/claude-code from 2.1.259 to 2.1.268 - #77
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.268. - [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.268) --- updated-dependencies: - dependency-name: "@anthropic-ai/claude-code" dependency-version: 2.1.268 dependency-type: direct:development update-type: version-update:semver-patch ... Signed-off-by: dependabot[bot] <support@github.com>
There was a problem hiding this comment.
Routine Dependabot npm devDependency bump (@anthropic-ai/claude-code 2.1.259 → 2.1.268), not an upstream anthropics/claude-code-action SHA bump — package-lock.json is updated consistently everywhere (main package plus all 8 optional platform packages), so no inconsistency there.
Title and description (Dependabot's auto-generated changelog summary) are accurate and complete for this change — no update needed.
No draft-conversion signal present (mechanical bot-authored bump, no WIP/incomplete markers) — not applicable regardless, since mark_draft_if_unready is off for this run.
Findings
- 🟠 Should fix
[policy]— this bump orphans theallowScriptsallowlist entry inpackage.json(still pinned to2.1.259), which will cause npm 12's lifecycle-script gate to block@anthropic-ai/claude-code's postinstall script again after merge — silently undoing the fix commit2d35b3fmade for exactly this CLI. See inline comments onpackage.json.
Requesting changes for the one Should-fix; everything else here is fine.
| }, | ||
| "devDependencies": { | ||
| "@anthropic-ai/claude-code": "2.1.259", | ||
| "@anthropic-ai/claude-code": "2.1.268", |
There was a problem hiding this comment.
🟠 Should fix [policy]
This bump orphans the allowScripts entry at line 43, which pins the postinstall-script allowlist to the old exact version:
"allowScripts": {
"@anthropic-ai/claude-code@2.1.259": true
}npm 12's lifecycle-script gate (this repo's own packageManager: npm@12.0.2...) matches allowScripts keys by exact name@version. After this bump, 2.1.268 no longer matches the allowlisted 2.1.259, so @anthropic-ai/claude-code's postinstall script will be silently blocked again on any npm ci/npm install — including in .github/workflows/dependabot.yml's investigate job and every npm ci step in ci.yml.
This field was added deliberately in 2d35b3f specifically so this CLI's postinstall script keeps working under the new npm 12 lifecycle-script gate ("the dependabot investigate job invokes this CLI directly via npx, so the script genuinely needs to run"). This PR's version bump silently undoes that fix.
| "@anthropic-ai/claude-code": "2.1.268", | |
| "@anthropic-ai/claude-code": "2.1.268", |
Please also update line 43:
| "@anthropic-ai/claude-code": "2.1.268", | |
| "@anthropic-ai/claude-code@2.1.268": true |
There was a problem hiding this comment.
Correction to my comment on line 20 above: please disregard both suggestion blocks in that comment — a GitHub suggestion always replaces the exact line(s) the comment is anchored to (line 20), so the second block (showing the allowScripts fix) would incorrectly overwrite the devDependencies version line if applied, and the first is a no-op since line 20 already reads 2.1.268. The allowScripts fix still needs to be applied by hand, since it's outside this diff's hunk and can't be anchored directly:
"allowScripts": {
- "@anthropic-ai/claude-code@2.1.259": true
+ "@anthropic-ai/claude-code@2.1.268": true
}There was a problem hiding this comment.
Re-reviewed at the same head commit (588e4ad) as the existing github-actions[bot] review, so this isn't a stale-thread situation — no new inline comment added to avoid duplicating the one already on package.json.
Independently re-verified the prior Should-fix finding, and it still stands [policy]: the bump updates devDependencies["@anthropic-ai/claude-code"] to 2.1.268 but leaves allowScripts pinned to the old version key:
"allowScripts": {
"@anthropic-ai/claude-code@2.1.259": true
}npm 12's allowScripts gate (introduced in this repo by 2d35b3f, specifically to approve this package's postinstall script) matches on the exact name@version string. Since the key wasn't bumped alongside the dependency, npm ci/npm install will block @anthropic-ai/claude-code@2.1.268's postinstall script — which the Dependabot investigate job relies on (it invokes this CLI directly via npx claude). This needs:
"allowScripts": {
"@anthropic-ai/claude-code@2.1.268": true
}Everything else checks out:
package-lock.jsonbumps the main package and all 8 optional platform packages (darwin-arm64/x64,linux-arm64,linux-arm64-musl,linux-x64,linux-x64-musl,win32-arm64/x64) consistently to2.1.268— no version drift.- Title/description are Dependabot's standard auto-generated summary and accurately describe the change; no edit needed.
- No draft-conversion signal present (this is a routine, complete dependency bump, not a WIP or stub).
Requesting changes only because the allowScripts key is a hard install-time gate, not a style nit — this diff will make CI's Dependabot investigation step fail on a blocked postinstall script.
|
🗜️ Headroom context compression
|
Bumps @anthropic-ai/claude-code from 2.1.259 to 2.1.268.
Release notes
Sourced from @anthropic-ai/claude-code's releases.
... (truncated)
Changelog
Sourced from @anthropic-ai/claude-code's changelog.
... (truncated)
Commits
536a2e2chore: Update CHANGELOG.md and feed.xmlfb63c4aMerge pull request #93244 from anthropics/poteat/mods-telemetry-fixes1297731mods/diff: backgrounds summary in two lines, as the internal copy has it573ada6mods/diff: context lines draw on no background, not the cyan background-task ...8939decmods/diff: the backend probes comment names where a host adds backends2142ecemods/diff: a backend seam, git as the built-in backend1a7c76emods: API renames in diff, telemetry rows queue and third-party providers sen...e62465dMerge pull request #93215 from anthropics/poteat/modsd9c456dAdd mods: sec-default, diff and telemetry, the hooks-module plugins built int...9cdc2a4chore: Update CHANGELOG.md and feed.xmlDependabot 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)