Skip to content

ci: resync actions.lock after #6 and #7 - #8

Merged
hyperpolymath merged 1 commit into
mainfrom
claude/kcut-ladder-i0-b5zcm8
Oct 14, 2026
Merged

hyperpolymath merged 1 commit into
mainfrom
claude/kcut-ladder-i0-b5zcm8

Conversation

@hyperpolymath

Copy link
Copy Markdown
Member

Summary

The secret scanner already runs and passes here; it has been green on every push and PR since #7. What has been red since #6 is Lock Sync Gate and Governance / Actions lockfile verify: the Dependabot bump in #6 and the scanner's move to standards 571cc734 in #7 both changed uses: references without regenerating .github/workflows/actions.lock. This PR resyncs the lock so both gates can pass.

Changes

  • actions.lock, workflows:: the four drifted workflows now list the references they use.
    • push-email-notify.yml: smtp-notify-action v0.5.0
    • release.yml: upload-artifact v7.0.2
    • static-analysis-gate.yml: upload-artifact v7.0.2 and download-artifact v8.0.2
    • secret-scanner.yml: standards 571cc734
  • actions.lock, dependencies::
    • Records for the three new tags, with commits checked by git ls-remote (c1c9fa07, cf430e03, 9000827c).
    • A record for hyperpolymath/standards@571cc734, whose nested uses: is the one action secret-scanner-reusable.yml uses at that commit: actions/checkout@3d3c42e5, which is already recorded. The lock stays transitively closed.
    • The three records nothing references any more are removed. The standards@da2c748a record stays, because five workflows still use it.
  • push-email-notify.yml: the pin comment named the v0.3.0 commit; it now names v0.5.0's.

RSR Quality Checklist

Required

  • Tests pass: scripts/check-lock-sync.sh reproduced the CI failure on main, then passed on this branch.
  • Code is formatted (lock keys in LC_ALL=C order, as the checker requires)
  • Linter is clean: scripts/check-action-pinning.js reports all action refs pinned.
  • No banned language patterns
  • No unsafe blocks without // SAFETY: comments (none touched)
  • No banned functions (none touched)
  • SPDX license headers present on all new/modified source files (headers unchanged)
  • No secrets, credentials, or .env files included

As Applicable

  • .machine_readable/descriptiles/STATE.a2ml updated (not applicable: CI configuration only)
  • Documentation updated for user-facing changes (not applicable)
  • CHANGELOG or release notes updated (not applicable)

Testing

scripts/check-lock-sync.sh, run locally with gawk 5.2.1:

  • on main: the same eight FAIL lines as the Lock Sync Gate run on 7a4a833;
  • on this branch: "actions.lock is in sync and transitively closed … 0 dangling edges", exit 0.

The decisive check is this PR's own CI. The lock is enforced when a workflow starts, so every workflow here should create jobs, with Lock Sync Gate and Governance green.

Branch name: this session's work branch, reused because it is the only branch the session is cleared to push to. The commit is SSH-signed (this repository's ruleset requires signatures).

🤖 Generated with Claude Code

https://claude.ai/code/session_01MF5GeeBBDF6sUzr6pZTjWV


Generated by Claude Code

Lock Sync Gate and the Governance lockfile verify have failed on every
run since the Dependabot bump in #6 (smtp-notify-action v0.3.0 -> v0.5.0,
upload-artifact v7.0.1 -> v7.0.2, download-artifact v8.0.1 -> v8.0.2),
and #7 moved the secret scanner's reusable workflow from standards
da2c748a to 571cc734; neither regenerated the lockfile.

- workflows: the four drifted workflows now list the refs they use.
- dependencies: records for the three new tags (commits checked with
  git ls-remote: c1c9fa07, cf430e03, 9000827c) and for
  metadatastician/standards@571cc734, whose nested uses is the one action
  secret-scanner-reusable.yml uses at that commit (actions/checkout
  3d3c42e5, already recorded), so the lock stays transitively closed.
  The three records nothing references any more are removed; the
  standards@da2c748a record stays, five workflows still use it.
- push-email-notify.yml: its pin comment named the v0.3.0 commit; it
  now names v0.5.0's.

scripts/check-lock-sync.sh, run locally with gawk: in sync and
transitively closed, 0 dangling edges (it reproduced the CI failure on
main first). scripts/check-action-pinning.js: all refs pinned.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01MF5GeeBBDF6sUzr6pZTjWV
@coderabbitai

coderabbitai Bot commented Oct 14, 2026 •

Copy link
Copy Markdown

Review in Change Stack →

Note

Currently processing new changes in this PR. This may take a few minutes, please wait...

⚙️ Run configuration
  • Configuration used: Organization UI
  • Review profile: ASSERTIVE
  • Plan: Free
  • Run ID: adc7f559-7ce1-401a-8fa1-2ecbcab7a5c5
📥 Commits

Reviewing files that changed from the base of the PR and between 7a4a833 and 7486b23.

⛔ Files ignored due to path filters: 1
  • .github/workflows/actions.lock is excluded by !**/*.lock
📒 Files selected for processing: 1
  • .github/workflows/push-email-notify.yml
 ___________________________________________________________________________________________________________________________________
< Keep knowledge in plain text. Plain text won't become obsolete. It helps leverage your work and simplifies debugging and testing. >
 -----------------------------------------------------------------------------------------------------------------------------------
  \
   \   (\__/)
       (•ㅅ•)
       /   づ
  • Autofix · Keep fixing CodeRabbit findings and required CI, and resolving merge conflicts

Note

🎁 Summarized by CodeRabbit Free

Your organization is on the Free plan. CodeRabbit will generate a high-level summary and a walkthrough for each pull request. For a comprehensive line-by-line review, please upgrade your subscription to CodeRabbit Essentials by visiting https://app.coderabbit.ai/settings/billing.

Comment @coderabbitai help to get the list of available commands.

Copy link
Copy Markdown
Member Author

Results on this head. All seven workflows started. Lock Sync Gate is now green; it was red on main since #6. The Secret Scanner, Rust CI, SonarQube, DEED and K9 checks all pass.

Governance / Actions lockfile verify is still red, and it is not this PR's. It has failed on every push since c500cae / 66fd6f2, before #6. On this head, the verifier (standards update-actions-lock.sh at the helper pin 874ffe58) accepted all seven job-level reusable-workflow entries ("Accepted reusable-workflow lock coverage"). One finding is left, and that pinned helper counts every unaccepted finding as a failure:

.github/workflows/codeql.yml  sha-as-ref (warning)
github/codeql-action@b96794f015dfd88f77b49b1c93e0fa7110f94c63
"pinned to a bare SHA without a symbolic ref"

codeql.yml is untouched here. Proposed fix, as a separate change: reference the action by tag, as the other workflows here do, and lock the tag to the same commit:

uses: github/codeql-action/init@v4.38.0      # and .../analyze@v4.38.0
# actions.lock: workflows['.github/workflows/codeql.yml'] lists 'github/codeql-action@v4.38.0', plus a record
'github/codeql-action@v4.38.0':
    ref: 'v4.38.0'
    commit: 'sha1-b96794f015dfd88f77b49b1c93e0fa7110f94c63'
    owner_id: 9919
    repo_id: 259445878

Two notes on that file. The current uses: lines name the repository root, github/codeql-action@…, without the /init and /analyze subpaths, and I found no CodeQL Security Analysis run in this repository's recent run history. That workflow needs the subpaths anyway. Standards' newer verifier (192389bf) also accepts more of the tool's false positives, so bumping the Governance reusable pin is the other route. Both are outside this PR's scope.


Generated by Claude Code

@hyperpolymath
hyperpolymath marked this pull request as ready for review October 14, 2026 13:18
@hyperpolymath
hyperpolymath merged commit 759b651 into main Oct 14, 2026
29 of 30 checks passed
@hyperpolymath
hyperpolymath deleted the claude/kcut-ladder-i0-b5zcm8 branch October 14, 2026 13:19
hyperpolymath added a commit that referenced this pull request Oct 14, 2026
## Summary

Clears the last failure in **Governance / Actions lockfile verify**,
which has been red on `main` since before #8. `codeql.yml` called
`github/codeql-action` at a bare commit SHA. The gate flags that as
`sha-as-ref`, the one finding it does not accept. The same reference was
also broken at runtime: with no subpath it runs the repository root of
`codeql-action`, which is a stub that prints "This is a stub" and exits
1. That is why both runs this workflow ever made (5 Oct) failed.

## Changes

- `codeql.yml`: the two steps call `github/codeql-action/init@v4.38.0`
and `github/codeql-action/analyze@v4.38.0`. This is the same release as
before. 4.38.1 stays held, per `dependabot.yml` and nexia-list#100.
- `actions.lock`:
  - `codeql.yml` now lists `github/codeql-action@v4.38.0`.
- A new dependency record pins that tag to commit `b96794f0` (owner
9919, repo 259445878), so a moved tag is refused at startup. `git
ls-remote` confirms `refs/tags/v4.38.0^{}` is
`b96794f015dfd88f77b49b1c93e0fa7110f94c63`.
- The record keyed by the bare SHA is removed, because nothing else
references it. The `cdf488f5` record stays, because a standards
dependency still reaches it.
- `dependabot.yml`: the hold comment said "SHA-pinned"; it now says
where the commit is held.

## RSR Quality Checklist

### Required

- [x] Tests pass: `scripts/check-lock-sync.sh` reports the lock in sync
and transitively closed, exit 0.
- [x] Code is formatted (lock keys stay in `LC_ALL=C` order)
- [x] Linter is clean: `scripts/check-action-pinning.js` reports all
action refs pinned; actionlint is clean on `codeql.yml`.
- [x] No banned language patterns
- [x] No `unsafe` blocks without `// SAFETY:` comments (none touched)
- [x] No banned functions (none touched)
- [x] SPDX license headers present on all new/modified source files
(headers unchanged)
- [x] No secrets, credentials, or `.env` files included

### As Applicable

- [ ] `.machine_readable/descriptiles/STATE.a2ml` updated (not
applicable: CI configuration only)
- [ ] Documentation updated for user-facing changes (not applicable)
- [ ] `CHANGELOG` or release notes updated (not applicable)

## Testing

- Locally: the three checks above.
- On this PR: Governance / Actions lockfile verify should go green, with
only the "Accepted reusable-workflow lock coverage" lines left in its
log. Every other workflow should start as it did on `main` at `759b651`.
- The CodeQL workflow itself is `disabled_manually` in this repository's
Actions settings, along with about 25 others since 6 Oct. So this PR
does not run it. Re-enable it in the Actions tab if you want the scan;
it should then run instead of hitting the stub.

The commit is SSH-signed. The branch name is this session's work branch,
restarted from `main` after #8 merged.

🤖 Generated with [Claude Code](https://claude.com/claude-code)

https://claude.ai/code/session_01MF5GeeBBDF6sUzr6pZTjWV

---
_Generated by [Claude
Code](https://claude.ai/code/session_01MF5GeeBBDF6sUzr6pZTjWV)_

Co-authored-by: Claude <noreply@anthropic.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants