Skip to content

ci: SOTA-ify all 3 workflows (perms, paths-ignore, npm cache, timeouts, concurrency)#10

Merged
acebot712 merged 1 commit intomainfrom
ci/sota-improvements
Apr 18, 2026
Merged

ci: SOTA-ify all 3 workflows (perms, paths-ignore, npm cache, timeouts, concurrency)#10
acebot712 merged 1 commit intomainfrom
ci/sota-improvements

Conversation

@acebot712
Copy link
Copy Markdown
Owner

Aligns with the audit pattern being rolled out across all 6 repos. No functional change.

Same audit pattern being applied across all repos:

ci.yml
* paths-ignore for docs/markdown/license/.gitignore
* explicit ``permissions: contents: read`` at workflow level
* ``timeout-minutes`` on every job
* ``cache: 'npm'`` in every setup-node call -- ~3-5x faster install
  on warm cache. Drop the ``npm ci || npm install`` fallback because
  with ``cache: 'npm'`` keyed on package-lock.json, ``npm ci`` is
  the only correct command (lockfile must exist + match)

release.yml
* default permissions to ``contents: read``, escalate ``id-token: write``
  per-job only on the npm publish step that needs provenance signing
* concurrency group: only one publish at a time, do NOT cancel in
  progress (a half-rolled-back publish is worse than two queued)
* ``timeout-minutes`` + ``cache: 'npm'`` on every job

sync-from-api.yml
* default permissions to read, escalate write per-job
* concurrency group prevents racing two sync runs on the branch/PR
* ``timeout-minutes`` + ``cache: 'npm'``

No functional change. Same checks, same publish behavior.
@acebot712 acebot712 merged commit 475d89d into main Apr 18, 2026
6 checks passed
@acebot712 acebot712 deleted the ci/sota-improvements branch April 18, 2026 20:51
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.

1 participant