Skip to content

feat: switch npm publishing to OIDC trusted publishers#18

Open
originalix wants to merge 4 commits into
masterfrom
feat/oidc-trusted-publishing
Open

feat: switch npm publishing to OIDC trusted publishers#18
originalix wants to merge 4 commits into
masterfrom
feat/oidc-trusted-publishing

Conversation

@originalix

Copy link
Copy Markdown

Summary

  • Replace NPM_TOKEN secret with OIDC trusted publishing (id-token: write permission)
  • Update repository, homepage, bugs URLs from bitcoinjs/coinselect to OneKeyHQ/coinselect
  • Bump version to 3.1.18

Action required after merge

  1. Go to npmjs.com/@onekeyfe/coinselect settings and configure Trusted Publisher:
    • Organization: OneKeyHQ
    • Repository: coinselect
    • Workflow filename: package-publish.yml
  2. After verifying OIDC works, remove the NPM_TOKEN secret from repo settings

Test plan

  • Configure trusted publisher on npmjs.com before first publish
  • Trigger a release and verify OIDC-based publish succeeds
  • Verify provenance attestation is automatically generated

🤖 Generated with Claude Code

originalix and others added 3 commits March 5, 2026 11:24
Allow callers to control output ordering by passing sortingStrategy
('bip69', 'none', or 'random') to the witness coinSelect function.
Defaults to 'random' for backward compatibility.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
Replace NPM_TOKEN-based authentication with OIDC trusted publishing
for improved security. Update repository URLs to OneKeyHQ/coinselect.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
@revan-zhang

revan-zhang commented Mar 26, 2026

Copy link
Copy Markdown

Snyk checks have passed. No issues have been found so far.

Status Scan Engine Critical High Medium Low Total (0)
Open Source Security 0 0 0 0 0 issues
Licenses 0 0 0 0 0 issues

💻 Catch issues earlier using the plugins for VS Code, JetBrains IDEs, Visual Studio, and Eclipse.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
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