Skip to content

Supply chain improvements#38

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

Supply chain improvements#38
dgreif merged 1 commit into
masterfrom
supply-chain-improvements

Conversation

@dgreif
Copy link
Copy Markdown
Contributor

@dgreif dgreif commented Jun 3, 2026

Summary

  • Added project npm config with min-release-age=3.
  • Updated Node CI to use Node 26, npm ci, npm caching, least-privilege permissions, and pinned GitHub Actions SHAs.
  • Updated publish workflows to use Node 26 and pinned GitHub Actions SHAs while keeping OIDC/provenance publishing.
  • Refreshed the package lockfile with the package version from package.json.

Ecosystems detected

  • npm package
  • GitHub Actions CI and publish workflows

Recommendations applied

  • npm minimum release age configured.
  • Third-party actions pinned to full commit SHAs using current known releases.
  • CI install changed from npm it/install behavior to explicit npm ci plus npm test.
  • Release workflows updated from Node 24 to Node 26.

Not applied

  • No Vitest or Playwright configuration was present.
  • No Node version requirement files or package engine fields were added.

Human review notes

  • npm trusted publishing may need to be configured for the package before OIDC publishing succeeds.

Validation

  • npm install --no-audit --no-fund
  • npm audit fix
  • npm ci
  • npm test
  • npm audit

Update CI and publish workflows to use Node 26, npm ci, and pinned GitHub Actions SHAs. Add npm min-release-age configuration and refresh the lockfile after npm install/audit fix.\n\nCo-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
@dgreif dgreif marked this pull request as ready for review June 3, 2026 17:26
@dgreif dgreif requested a review from a team as a code owner June 3, 2026 17:26
Copilot AI review requested due to automatic review settings June 3, 2026 17:26
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 strengthens the repo’s npm and GitHub Actions supply-chain posture by introducing an npm minimum release age gate, pinning third-party actions to full commit SHAs, modernizing CI/release workflows (Node 26 + npm ci + caching + least-privilege permissions), and syncing the lockfile metadata to the package version.

Changes:

  • Added .npmrc configuration to enforce a minimum release age for installed packages.
  • Updated CI and publish workflows to Node 26, npm ci/npm test, npm caching, and pinned action SHAs.
  • Refreshed package-lock.json to match the package.json version.
Show a summary per file
File Description
package-lock.json Synces lockfile metadata (version) with package.json.
.npmrc Adds an npm config gate (min-release-age=3) to reduce risk from newly published deps.
.github/workflows/test.yml Updates CI to Node 26, npm ci, npm caching, least-privilege permissions, and pinned action SHAs.
.github/workflows/publish.yml Updates release publishing jobs to Node 26 and pinned action SHAs while keeping OIDC/provenance publishing.

Copilot's findings

Tip

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

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

Comment on lines 11 to 15
steps:
- uses: actions/checkout@v2
- name: Use Node.js 24.x
uses: actions/setup-node@v5
- uses: actions/checkout@df4cb1c069e1874edd31b4311f1884172cec0e10
- name: Use Node.js 26.x
uses: actions/setup-node@48b55a011bda9f5d6aeb4c2d9c7362e8dae4041e
with:
@dgreif dgreif merged commit fbee030 into master Jun 3, 2026
5 checks passed
@dgreif dgreif deleted the supply-chain-improvements branch June 3, 2026 17:56
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