Skip to content

Supply chain improvements#42

Merged
dgreif merged 1 commit into
mainfrom
supply-chain-improvements-20260603101902
Jun 3, 2026
Merged

Supply chain improvements#42
dgreif merged 1 commit into
mainfrom
supply-chain-improvements-20260603101902

Conversation

@dgreif
Copy link
Copy Markdown
Contributor

@dgreif dgreif commented Jun 3, 2026

Summary

  • Added project npm policy with min-release-age=3.
  • Updated Node CI and publish workflows to use Node 26 and npm ci.
  • Updated third-party GitHub Actions to current releases pinned by full commit SHA.
  • Updated Vitest to 4.1.7 and refreshed package-lock.json with npm audit fix / install.
  • Removed token-based npm publish flow in favor of OIDC trusted publishing with provenance.

Ecosystems detected

  • npm package with GitHub Actions CI and npm publish workflow.

Recommendations applied

  • Node 26 in CI/publish workflows.
  • Full-SHA pins for external Actions.
  • npm ci in workflow installs.
  • npm publish provenance and id-token: write.

Not applied

  • No Playwright usage found.
  • No Node version files or package engine requirements added.

Human review notes

  • npm trusted publishing may need to be configured for @github/memoize before the release workflow can publish without NODE_AUTH_TOKEN.
  • Local npm 11.6.0 warned that min-release-age is an unknown config, but commands completed successfully.

Validation

  • npm audit fix
  • npm install
  • npm ci
  • npm run build --if-present
  • npm test
  • npm audit (0 vulnerabilities)

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
@dgreif dgreif marked this pull request as ready for review June 3, 2026 17:28
@dgreif dgreif requested a review from a team as a code owner June 3, 2026 17:28
Copilot AI review requested due to automatic review settings June 3, 2026 17:28
Copy link
Copy Markdown

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

This PR tightens the project’s supply-chain posture for the npm package by updating the test toolchain/lockfile, introducing an npm install “cooldown” policy, and hardening GitHub Actions CI/release workflows (newer Node, npm ci, SHA-pinned actions, and OIDC/provenance publishing).

Changes:

  • Bump Vitest to 4.1.7 and refresh package-lock.json after audit/install updates.
  • Add .npmrc policy (min-release-age=3) intended to reduce exposure to newly-published dependency versions.
  • Update CI and publish workflows to Node 26, npm ci, SHA-pinned third-party actions, and npm provenance publishing with id-token: write.
Show a summary per file
File Description
package.json Pins Vitest to 4.1.7 to move the test runner/toolchain forward.
package-lock.json Updates the dependency graph/lockfile to match the new Vitest/tooling versions and audit results.
.npmrc Adds a minimum release-age policy for dependency installs.
.github/workflows/publish.yml Moves publishing to Node 26, uses npm ci, SHA-pins actions, and publishes with provenance via OIDC.
.github/workflows/nodejs.yml Moves CI to Node 26, uses npm ci, and SHA-pins actions.

Copilot's findings

Tip

Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

  • Files reviewed: 4/5 changed files
  • Comments generated: 3

Comment thread .npmrc
@@ -0,0 +1 @@
min-release-age=3
Comment on lines +15 to +16
- uses: actions/checkout@df4cb1c069e1874edd31b4311f1884172cec0e10
- uses: actions/setup-node@48b55a011bda9f5d6aeb4c2d9c7362e8dae4041e
Comment on lines +12 to +13
- uses: actions/checkout@df4cb1c069e1874edd31b4311f1884172cec0e10
- uses: actions/setup-node@48b55a011bda9f5d6aeb4c2d9c7362e8dae4041e
@dgreif dgreif merged commit e09afb8 into main Jun 3, 2026
5 checks 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.

3 participants