Repository navigation
chore(deps): bump the actions-minor-patch group across 1 directory with 3 updates - #1013
dependabot[bot] wants to merge 1 commit into
Conversation
|
984bad7 to
9d944ce
Compare
freshtonic
left a comment
There was a problem hiding this comment.
Run Tests (Node 22) and Run Tests (Node 24) fail because of this PR. Dependabot cannot fix this on its own, so a person must push one more commit to this branch.
Blocker
scripts/__tests__/claude-review-workflow.test.mjs:6 pins ACTION_SHA = 'bf38e86e58df9ebf3420326d019f955bb3be64dd' (v1.0.225). The test pins the reviewed Claude action release and authenticates only with OIDC compares the workflow against that constant, so the bump to 756cc22… (v1.0.235) fails it:
Expected: "anthropics/claude-code-action@bf38e86e58df9ebf3420326d019f955bb3be64dd"
Received: "anthropics/claude-code-action@756cc22e19660d20e8cc9496b4f242475a7f7790"
The pin is a review gate, not an accident. Before you change the constant, read the claude-code-action changes from v1.0.225 to v1.0.235. Make sure that this job still gets the agent mode and the job github_token that the workflow comments describe. Then set ACTION_SHA to 756cc22e19660d20e8cc9496b4f242475a7f7790 in the same PR.
Verified
anthropics/claude-code-actiontagv1.0.235resolves to756cc22e19660d20e8cc9496b4f242475a7f7790. The comment on the pin is correct.release-plz/actiontagv0.5resolves tob8d6b54b02889ff2ae2bb82e8b57c3a8fc1683a5. Both release-plz pins are correct.- The CodeQL bump from
v4.38.0tov4.38.2is a patch release.
Not from this PR (no action needed here)
In .github/workflows/codeql.yml, the step Perform CodeQL Analysis uses github/codeql-action/init, not github/codeql-action/analyze. Thus the job runs init two times and does not analyze or upload results. The job shows green, but it does no scan. This problem was on main before this PR. A separate PR must fix it.
No changeset is necessary. This PR changes only CI workflows, and they are not part of a published package.
| - name: Review pull request | ||
| id: claude-review | ||
| uses: anthropics/claude-code-action@bf38e86e58df9ebf3420326d019f955bb3be64dd # v1.0.225 | ||
| uses: anthropics/claude-code-action@756cc22e19660d20e8cc9496b4f242475a7f7790 # v1.0.235 |
There was a problem hiding this comment.
This bump fails scripts/__tests__/claude-review-workflow.test.mjs, because line 6 of that file still pins ACTION_SHA to bf38e86e58df9ebf3420326d019f955bb3be64dd. Read the action changes from v1.0.225 to v1.0.235. Then set ACTION_SHA to 756cc22e19660d20e8cc9496b4f242475a7f7790 in this PR.
9d944ce to
e8acc6d
Compare
freshtonic
left a comment
There was a problem hiding this comment.
Re-review of e8acc6dc. This commit is a Dependabot rebase of 9d944cec. The diff is the same: the three workflow pins and no other change.
The blocker from my previous review is not fixed. On this head, scripts/__tests__/claude-review-workflow.test.mjs:6 still sets ACTION_SHA = 'bf38e86e58df9ebf3420326d019f955bb3be64dd' (v1.0.225). .github/workflows/claude-review.yml:100 now uses 756cc22e19660d20e8cc9496b4f242475a7f7790 (v1.0.235). Thus the test pins the reviewed Claude action release and authenticates only with OIDC fails, and Run Tests (Node 22) is red again.
Dependabot cannot fix this. A person must read the claude-code-action changes from v1.0.225 to v1.0.235, and then push a commit to this branch that sets ACTION_SHA to 756cc22e19660d20e8cc9496b4f242475a7f7790. The open inline thread on claude-review.yml has the same request.
My other notes from the previous review do not change. The release-plz and CodeQL pins are correct, and this PR needs no changeset.
e8acc6d to
6960b48
Compare
freshtonic
left a comment
There was a problem hiding this comment.
Re-review of 6960b48e. This commit is one more Dependabot rebase. The diff is the same as e8acc6dc: the three workflow pins and no other change.
The blocker is not fixed. On this head, scripts/__tests__/claude-review-workflow.test.mjs:6 still sets ACTION_SHA = 'bf38e86e58df9ebf3420326d019f955bb3be64dd' (v1.0.225), but .github/workflows/claude-review.yml:100 uses 756cc22e19660d20e8cc9496b4f242475a7f7790 (v1.0.235). Run Tests (Node 22) and Run Tests (Node 24) fail again on this head.
A rebase cannot fix this. A person must read the claude-code-action changes from v1.0.225 to v1.0.235, and then push a commit to this branch that sets ACTION_SHA to 756cc22e19660d20e8cc9496b4f242475a7f7790. The open inline thread on claude-review.yml has the same request.
My other notes do not change. The release-plz and CodeQL pins are correct, and this PR needs no changeset.
558dac2 to
260e774
Compare
|
Important Review skippedBot user detected. To trigger a single review, invoke the ⚙️ Run configuration
You can disable this status message by setting the Use the checkbox below for a quick retry:
Comment |
6feea72 to
cf6723a
Compare
freshtonic
left a comment
There was a problem hiding this comment.
The blocker in the earlier reviews is still not fixed, and the SHA that the test must use has changed.
Head cf6723ae pins anthropics/claude-code-action@6c690188da61cf16ee7debb28383ba67f02abb4c # v1.0.242, not v1.0.235. Run Tests (Node 22/24) fail with the same assertion, and the test receives 6c690188…. Thus the commit that a person pushes must set ACTION_SHA to 6c690188da61cf16ee7debb28383ba67f02abb4c, not to 756cc22….
To help with the v1.0.225 → v1.0.242 review
- I verified that tag
v1.0.242points to commit6c690188…. compare/bf38e86…6c690188has 19 commits. 17 of them only bump Claude Code and the Agent SDK. The other two change only upstream CI workflows (#1853, #1867).- The runtime change is one line, in two places:
CLAUDE_CODE_VERSION2.1.272 → 2.1.290 inbase-action/action.ymlandsrc/entrypoints/run.ts. The top-levelaction.ymldoes not change. Thus the inputs, the OIDC token exchange,github_tokenand the mode selection are the same as in v1.0.225.
Suggested follow-up PR
- The release-plz pin
b8d6b54b…isv0.5.139, not thev0.5that the comment says. The floatingv0.5tag now points tov0.5.142(89212866…). Write the exact tag in the comment, as the other pins do, so that a reviewer can verify it.
…th 3 updates Bumps the actions-minor-patch group with 3 updates in the / directory: [anthropics/claude-code-action](https://github.com/anthropics/claude-code-action), [github/codeql-action](https://github.com/github/codeql-action) and [release-plz/action](https://github.com/release-plz/action). Updates `anthropics/claude-code-action` from 1.0.225 to 1.0.247 - [Release notes](https://github.com/anthropics/claude-code-action/releases) - [Commits](anthropics/claude-code-action@bf38e86...2dca132) Updates `github/codeql-action` from 4.38.0 to 4.38.3 - [Release notes](https://github.com/github/codeql-action/releases) - [Changelog](https://github.com/github/codeql-action/blob/main/CHANGELOG.md) - [Commits](github/codeql-action@v4.38.0...v4.38.3) Updates `release-plz/action` from 0.5.137 to 0.5.140 - [Release notes](https://github.com/release-plz/action/releases) - [Commits](release-plz/action@8e61445...18641b6) --- updated-dependencies: - dependency-name: anthropics/claude-code-action dependency-version: 1.0.235 dependency-type: direct:production update-type: version-update:semver-patch dependency-group: actions-minor-patch - dependency-name: github/codeql-action dependency-version: 4.38.2 dependency-type: direct:production update-type: version-update:semver-patch dependency-group: actions-minor-patch - dependency-name: release-plz/action dependency-version: 0.5.139 dependency-type: direct:production update-type: version-update:semver-patch dependency-group: actions-minor-patch ... Signed-off-by: dependabot[bot] <support@github.com>
cf6723a to
d79eead
Compare
Bumps the actions-minor-patch group with 3 updates in the / directory: anthropics/claude-code-action, github/codeql-action and release-plz/action.
Updates
anthropics/claude-code-actionfrom 1.0.225 to 1.0.247Release notes
Sourced from anthropics/claude-code-action's releases.
... (truncated)
Commits
2dca132chore: bump Claude Code to 2.1.295 and Agent SDK to 0.3.29538c80c1chore: bump Claude Code to 2.1.294 and Agent SDK to 0.3.2946fed3cachore: bump Claude Code to 2.1.293 and Agent SDK to 0.3.2935898584chore: bump Claude Code to 2.1.292 and Agent SDK to 0.3.29286d88e6chore: bump Claude Code to 2.1.291 and Agent SDK to 0.3.2916c69018chore: bump Claude Code to 2.1.290 and Agent SDK to 0.3.290cab360fchore: bump Claude Code to 2.1.289 and Agent SDK to 0.3.289ed670b4chore: bump Claude Code to 2.1.288 and Agent SDK to 0.3.28897c5347chore: bump Claude Code to 2.1.287 and Agent SDK to 0.3.28712dd8d7chore: bump Claude Code to 2.1.286 and Agent SDK to 0.3.286Updates
github/codeql-actionfrom 4.38.0 to 4.38.3Release notes
Sourced from github/codeql-action's releases.
Changelog
Sourced from github/codeql-action's changelog.
Commits
24c5418Merge pull request #4214 from github/update-v4.38.3-e0b52dbdc69e87c5Add changelog note for #418423b6828Update changelog for v4.38.3e0b52dbMerge pull request #4210 from github/henrymercer/pin-python-for-older-clis-in-ci6b99698Pin Python 3.13.15 for older CLI versions in multi-language check63d3d63Merge pull request #4203 from github/update-bundle/codeql-bundle-v2.27.27377a10Add changelog notec1a30cdUpdate default bundle to codeql-bundle-v2.27.2d208ec7Merge pull request #4200 from github/mbg/esm/migrate-pr-checksa71b0c6Replace__dirnameinsync.tsUpdates
release-plz/actionfrom 0.5.137 to 0.5.140Release notes
Sourced from release-plz/action's releases.
Commits
18641b6Update to 0.3.170 (#561)ff20e36chore(deps): update dependency taiki-e/install-action to v2.87.26 (#560)7a48fcfchore(deps): update dependency cargo-bins/cargo-binstall to v1.25.2 (#559)3491eb1chore(deps): update dependency taiki-e/install-action to v2.87.25 (#558)b143befchore(deps): lock file maintenance (#557)11586b6update github actions weekly (#556)8402dc2chore(deps): update dependency taiki-e/install-action to v2.87.24 (#555)f9a4b7echore(deps): update dependency obi1kenobi/cargo-semver-checks to v0.51 (#554)eeb8829chore(deps): update dependency taiki-e/install-action to v2.87.23 (#553)3999d03chore(deps): update dependency cargo-bins/cargo-binstall to v1.25.1 (#552)