Skip to content

fix(ci): enable npm OIDC Trusted Publishing#101

Merged
miya0001 merged 1 commit into
mainfrom
fix/oidc-publish
Apr 5, 2026
Merged

fix(ci): enable npm OIDC Trusted Publishing#101
miya0001 merged 1 commit into
mainfrom
fix/oidc-publish

Conversation

@miya0001

@miya0001 miya0001 commented Apr 5, 2026

Copy link
Copy Markdown
Member

Summary

  • actions/setup-noderegistry-url を削除 — .npmrc_authToken=${NODE_AUTH_TOKEN} が書き込まれ、npm のネイティブ OIDC トークン交換が無効化されていた
  • NODE_AUTH_TOKEN: "" の明示的な空設定も削除
  • npm が GitHub OIDC トークンを自動取得・交換して Trusted Publishing で認証する

経緯

Test plan

  • CI が通ることを確認
  • マージ後、v0.10.1 タグを作成して publish が成功することを確認

Summary by CodeRabbit

リリースノート

  • Chores
    • 公開ワークフロー設定を簡略化しました。レジストリURL設定とトークン環境変数を削除し、OIDC認証ベースの公開プロセスへ統一しました。

@coderabbitai

coderabbitai Bot commented Apr 5, 2026

Copy link
Copy Markdown

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro

Run ID: cb7be89b-f162-49a9-8fd3-53cd4780c772

📥 Commits

Reviewing files that changed from the base of the PR and between 337cb38 and abf43c2.

📒 Files selected for processing (1)
  • .github/workflows/publish.yml
💤 Files with no reviewable changes (1)
  • .github/workflows/publish.yml

📝 Walkthrough

Walkthrough

.github/workflows/publish.ymlワークフロー内で3行が削除されました。削除された内容は、actions/setup-node@v4からのnpmレジストリURL設定と、npmパブリッシュステップの環境変数設定ブロックです。他のワークフロー処理、OIDCベースのパブリッシュコマンド、および検証ステップは変更されていません。

Changes

Cohort / File(s) Summary
Publish Workflow Configuration
.github/workflows/publish.yml
actions/setup-node@v4のレジストリURL設定を削除し、npm publish --provenanceステップのNODE_AUTH_TOKEN環境変数ブロックを削除。

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~3 minutes

Possibly related PRs

Poem

🐰 プロビナンスの新しい道へ、
設定を削除して身軽に、
レジストリは不要、
OIDC認証で信頼を築く!
シンプルが最高さ、ぴょん!

🚥 Pre-merge checks | ✅ 3
✅ Passed checks (3 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title clearly and specifically describes the main change: enabling npm OIDC Trusted Publishing by removing configurations that were blocking the OIDC token exchange.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch fix/oidc-publish

Comment @coderabbitai help to get the list of available commands and usage tips.

@miya0001
miya0001 merged commit 12ce2f7 into main Apr 5, 2026
7 checks passed
@miya0001
miya0001 deleted the fix/oidc-publish branch April 5, 2026 16:22
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.

1 participant