Skip to content

Enable trusted publishing for npm releases#723

Merged
NoopDog merged 5 commits intomainfrom
copilot/enable-trusted-publishing
Dec 11, 2025
Merged

Enable trusted publishing for npm releases#723
NoopDog merged 5 commits intomainfrom
copilot/enable-trusted-publishing

Conversation

Copy link
Contributor

Copilot AI commented Dec 11, 2025

Plan: Enable Trusted Publishing for Package Releases via GitHub

  • Update .github/workflows/release-please.yml to add id-token: write permission for trusted publishing
  • Update actions/setup-node@v4 to use always-auth: true for trusted publishing
  • Remove the NODE_AUTH_TOKEN secret usage and replace with trusted publishing
  • Create documentation file explaining the trusted publishing setup
  • Verify the workflow configuration matches npm trusted publisher requirements
  • Run code review to ensure changes are correct
  • Run security checks with CodeQL
  • Address review feedback and add safety notes
  • Fix prettier formatting with correct version (2.8.3)
Original prompt

This section details on the original issue you should resolve

<issue_title>Enable Trusted Publishing for Package Releases via GitHub</issue_title>
<issue_description>## Goal
Adopt GitHub's trusted publishing workflow to securely publish packages, updating the release automation to leverage GitHub's built-in infrastructure.

Tasks

Context

Switching to trusted publishing improves supply chain security and aligns with GitHub best practices for automated package releases.

Acceptance Criteria

  • release-please.yml is updated with trusted publishing permissions
  • Setup instructions are documented
  • Release flow is tested and confirmed

If additional repository secrets or GitHub workflows are impacted, please note and review for security.</issue_description>

<agent_instructions>see instructions here https://docs.npmjs.com/trusted-publishers</agent_instructions>

Comments on the Issue (you are @copilot in this section)


✨ Let Copilot coding agent set things up for you — coding agent works faster and does higher quality work when set up for your repo.

Copilot AI and others added 2 commits December 11, 2025 04:26
- Add id-token: write permission for OIDC authentication
- Configure setup-node with always-auth for trusted publishing
- Remove NODE_AUTH_TOKEN secret usage
- Add provenance and access flags to npm publish command
- Add comprehensive documentation in docs/TRUSTED_PUBLISHING.md

Co-authored-by: NoopDog <3239697+NoopDog@users.noreply.github.com>
Co-authored-by: NoopDog <3239697+NoopDog@users.noreply.github.com>
Copilot AI changed the title [WIP] Enable trusted publishing for package releases via GitHub Enable trusted publishing for npm releases Dec 11, 2025
Copilot AI requested a review from NoopDog December 11, 2025 04:30
@NoopDog NoopDog marked this pull request as ready for review December 11, 2025 04:34
@NoopDog
Copy link
Collaborator

NoopDog commented Dec 11, 2025

@copilot run prettier on the markdown file

Co-authored-by: NoopDog <3239697+NoopDog@users.noreply.github.com>
Copy link
Contributor Author

Copilot AI commented Dec 11, 2025

@copilot run prettier on the markdown file

Done in commit 65c1f17. Prettier adjusted the spacing in inline comments within the YAML code blocks.

Copilot AI requested a review from NoopDog December 11, 2025 05:17
@NoopDog
Copy link
Collaborator

NoopDog commented Dec 11, 2025

@copilot [warn] docs/TRUSTED_PUBLISHING.md has prettier errors

Use project's prettier version (2.8.3) instead of npx version (3.7.4)
to ensure consistent formatting with CI checks.
Copy link
Contributor Author

Copilot AI commented Dec 11, 2025

@copilot [warn] docs/TRUSTED_PUBLISHING.md has prettier errors

Fixed in commit 2d6b548. The issue was using npx prettier (v3.7.4) instead of the project's version (v2.8.3). The project's prettier version requires a blank line between numbered list items and nested bullet lists.

Copy link
Contributor

@frano-m frano-m left a comment

Choose a reason for hiding this comment

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

LGTM! 🌮

@NoopDog NoopDog merged commit ea28376 into main Dec 11, 2025
2 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.

Enable Trusted Publishing for Package Releases via GitHub

3 participants