Skip to content

Supply chain improvements#209

Merged
smockle merged 6 commits into
mainfrom
copilot/supply-chain-improvements
Jun 3, 2026
Merged

Supply chain improvements#209
smockle merged 6 commits into
mainfrom
copilot/supply-chain-improvements

Conversation

@dgreif
Copy link
Copy Markdown
Contributor

@dgreif dgreif commented Jun 3, 2026

Summary

  • Added project npm release-age policy in .npmrc.
  • Updated CI and publish workflows to use Node 26.
  • Pinned third-party GitHub Actions to full commit SHAs.
  • Switched npm publish from token-based auth to OIDC provenance publishing.
  • Applied npm audit fix updates to transitive lockfile entries.

Ecosystems detected

  • npm package with package-lock.json.
  • GitHub Actions workflows for CI, publishing, and accessibility alt text checks.

Recommendations applied

  • npm ci is already used in CI and publish workflows.
  • Existing .nvmrc updated to match the Node 26 workflow runtime.
  • No Vitest or Playwright setup detected.

Not automatically applied

  • npm trusted publishing may need to be verified or configured for this package in npm before release publishing succeeds.

Human review notes

  • Review the OIDC publishing change to confirm the npm package has trusted publishing configured for this repository and workflow.

Validation

  • npm install
  • npm ci
  • npm test
  • npm audit --audit-level=low

Add the npm minimum release age policy, move CI and publish workflows to Node 26, pin third-party Actions to commit SHAs, switch npm publishing to OIDC provenance, and apply npm audit fixes.

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
@dgreif dgreif marked this pull request as ready for review June 3, 2026 17:29
@dgreif dgreif requested a review from a team as a code owner June 3, 2026 17:29
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 hardens the project’s supply-chain posture by tightening dependency and release workflows: it updates lockfile transitive dependencies, pins GitHub Actions by commit SHA, moves CI/publish to Node 26, and switches npm publishing toward OIDC-based provenance.

Changes:

  • Add an npm “minimum release age” policy via .npmrc.
  • Update CI/publish workflows to Node 26 and pin referenced Actions to full commit SHAs.
  • Refresh transitive dependencies in package-lock.json (via npm audit fix).
Show a summary per file
File Description
package-lock.json Updates transitive dependency versions/integrity metadata.
.nvmrc Aligns local dev Node version with CI/publish runtime.
.npmrc Introduces an npm minimum release-age policy.
.github/workflows/publish.yml Pins Actions, updates Node runtime, and removes token-based publish in favor of provenance/OIDC.
.github/workflows/ci.yml Pins Actions and updates Node runtime for CI.
.github/workflows/accessibility-alt-text-bot.yml Pins the third-party action to a full commit SHA.

Copilot's findings

Tip

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

  • Files reviewed: 5/6 changed files
  • Comments generated: 1

Comment thread .npmrc
Comment thread .github/workflows/accessibility-alt-text-bot.yml Outdated
Co-authored-by: Clay Miller <clay@smockle.com>
Comment thread .nvmrc
Comment thread .github/workflows/ci.yml Outdated
Comment thread .github/workflows/publish.yml Outdated
Comment thread .github/workflows/ci.yml Outdated
Comment thread .github/workflows/publish.yml Outdated
Comment thread .github/workflows/publish.yml
@smockle smockle merged commit 6b94203 into main Jun 3, 2026
4 checks passed
@smockle smockle deleted the copilot/supply-chain-improvements branch June 3, 2026 19:35
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.

5 participants