Skip to content

ci: fix satisfy npm trusted publishing requirements for evaluation ts - #2758

Merged
cre8ivejp merged 1 commit into
mainfrom
fix-evaluation-npm-oidc
Aug 7, 2026
Merged

ci: fix satisfy npm trusted publishing requirements for evaluation ts#2758
cre8ivejp merged 1 commit into
mainfrom
fix-evaluation-npm-oidc

Conversation

@cre8ivejp

Copy link
Copy Markdown
Member

Summary

  • upgrade the evaluation publishing workflow from Node.js 20 to 24
  • add the package repository metadata required by npm trusted publishing
  • fix the OIDC authorization failure that produced an npm E404

Test plan

  • Validate package.json repository metadata
  • Check workflow and package files for lint errors
  • Merge and retry publishing @bucketeer/evaluation@0.0.9

Use a supported Node runtime and declare the package's source repository so npm can authorize the GitHub OIDC publisher.

Co-authored-by: Cursor <cursoragent@cursor.com>
Copilot AI review requested due to automatic review settings August 7, 2026 02:59
@cre8ivejp
cre8ivejp marked this pull request as ready for review August 7, 2026 03:00
@cre8ivejp cre8ivejp changed the title fix(evaluation): satisfy npm trusted publishing requirements ci: fix satisfy npm trusted publishing requirements for evaluation ts Aug 7, 2026
@cre8ivejp
cre8ivejp merged commit 9421927 into main Aug 7, 2026
18 checks passed
@cre8ivejp
cre8ivejp deleted the fix-evaluation-npm-oidc branch August 7, 2026 03:01

Copilot AI left a comment

Copy link
Copy Markdown

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 updates the TypeScript evaluation package’s publishing setup to meet npm Trusted Publishing expectations by adding required package metadata and aligning the publishing workflow’s Node.js runtime with the desired version.

Changes:

  • Add repository metadata (including subdirectory) to @bucketeer/evaluation’s package.json.
  • Bump the publishing workflow runtime from Node.js 20 to Node.js 24.

Reviewed changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated no comments.

File Description
evaluation/typescript/package.json Adds repository metadata (type/url/directory) for npm trusted publishing/provenance expectations.
.github/workflows/publish-evaluation-ts.yaml Updates NODE_VERSION to 24 for the npm publish workflow while keeping OIDC permissions/provenance publishing intact.

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

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.

2 participants