Skip to content

ci: use npm ci for reproducible workflow installs - #16

Merged
rogerchappel merged 1 commit into
mainfrom
agent/oss-9b3024889f56-npm-ci
Oct 13, 2026
Merged

rogerchappel merged 1 commit into
mainfrom
agent/oss-9b3024889f56-npm-ci

Conversation

@rogerchappel

Copy link
Copy Markdown
Owner

Summary

  • Add the missing npm lockfile and use npm ci in CI for deterministic installs.

Verification

  • npm ci (Node 24)
  • npm run check
  • npm test (17 passing)
  • npm run build
  • bash scripts/validate.sh
  • git diff --check
  • Manual review of CI workflow triggers: push and pull_request validation only; no release, deployment, or package publishing.

Risk Level

  • Low

Notes: Adds a lockfile for the existing workspace dependencies and makes CI enforce it.

Rollback Plan

  • Revert this PR's commit to restore the previous install command and remove the lockfile.

Human Decision Needed

  • Maintainer review

@rogerchappel
rogerchappel marked this pull request as ready for review October 10, 2026 02:00
@rogerchappel
rogerchappel enabled auto-merge (rebase) October 10, 2026 02:00
@rogerchappel
rogerchappel disabled auto-merge October 13, 2026 21:32
@rogerchappel
rogerchappel merged commit 583d4b8 into main Oct 13, 2026
1 check passed
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