build(deps): bump actions/github-script from 8.0.0 to 9.0.0 - #105
Conversation
Bumps [actions/github-script](https://github.com/actions/github-script) from 8.0.0 to 9.0.0. - [Release notes](https://github.com/actions/github-script/releases) - [Commits](actions/github-script@ed59741...3a2844b) --- updated-dependencies: - dependency-name: actions/github-script dependency-version: 9.0.0 dependency-type: direct:production update-type: version-update:semver-major ... Signed-off-by: dependabot[bot] <support@github.com>
|
🦞👀 Pull request received. I will update this pull request when review starts. |
|
Codex review: needs maintainer review before merge. Reviewed September 4, 2026, 4:52 AM ET / 08:52 UTC. ClawSweeper reviewWhat this changesUpdates the pinned GitHub Script action to v9.0.0 for the release workflow’s unused-version check. Merge readiness✅ Ready for maintainer review The update remains useful: reviewed main and v0.14.8 still use v8. No actionable patch defect was found, and the updated discussion addresses the earlier compatibility concern with reported bundle-level validation. Priority: P3 Review scores
Verification
How this fits togetherCrawlkit’s release preflight receives a requested version and checks GitHub for an existing tag or release. Only a successful check allows the shared signing and publication workflow to run. flowchart TD
A[Requested release version] --> B[Validate and normalize version]
B --> C[GitHub Script action]
C --> D[Read tag and release state]
D --> E{Version unused?}
E -->|Yes| F[Shared release pipeline]
E -->|No or API failure| G[Stop release]
Before mergeNone. Agent review detailsSecurityNone. Review metricsNone. Technical reviewBest possible solution: Keep the dependency current while preserving the read-only preflight and existing publication gate. Do we have a high-confidence way to reproduce the issue? Not applicable: this dependency update does not report a broken existing behavior, and source review found no introduced defect. Is this the best way to solve the issue? Yes: changing the existing immutable action pin is the narrowest update, and the unchanged caller avoids the v9 incompatibilities described in the captured PR body. AGENTS.md: found and applied where relevant. Codex review notes: model internal, reasoning high; reviewed against c778575cf46a. LabelsLabel justifications:
EvidenceWhat I checked:
Likely related people:
Rating scale
Overall follows the weaker of proof and patch quality. Workflow
HistoryReview history (1 earlier review cycle)
|
Update the pinned GitHub Script action from v8 to v9.0.0 for the release version preflight. The upstream annotated tag peels to
3a2844b7e9c422d3c10d287c895573f7108da1b3, matching this PR.The existing script loads the local CommonJS preflight helper and uses the injected
githubandcontextobjects. It does not import@actions/githubor redeclaregetOctokit, so it avoids the documented v9 breaking changes. The action continues to use Node 24.Validation:
dist/index.jsbundle on Node 24 with the unchanged crawlkit preflight and a synthetic local HTTP server. An unused version passed; existing tag, existing release, HTTP 500 and HTTP 401 each failed closed. All five cases made only the expected read requests. No release workflow was dispatched.make check, uncached full Go tests and a built temporary-configcrawlctlsmoke also passed on the same main code.53edbd471b924d13f5f1fde4d2b06677b48d7dc9.Credit: @dependabot. Prepared for maintainer review.