Skip to content

ci: switch npm publish to OIDC Trusted Publishing#100

Merged
miya0001 merged 2 commits into
mainfrom
feat/trusted-publishing
Apr 5, 2026
Merged

ci: switch npm publish to OIDC Trusted Publishing#100
miya0001 merged 2 commits into
mainfrom
feat/trusted-publishing

Conversation

@miya0001

@miya0001 miya0001 commented Apr 5, 2026

Copy link
Copy Markdown
Member

Summary

  • npm publish をレガシートークン認証から OIDC Trusted Publishing に移行
  • NODE_AUTH_TOKEN を明示的に空にし、npm が OIDC で認証するよう変更
  • npmjs.com 側で geolonia/geonicdb-cli + publish.yml の Trusted Publisher 設定済み

Test plan

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

Summary by CodeRabbit

  • Chores
    • npm パッケージの公開プロセスを OIDC Trusted Publishing に移行し、従来のトークン認証を廃止しました。

miya0001 added 2 commits April 6, 2026 01:00
Override NODE_AUTH_TOKEN to empty so npm uses OIDC instead of
legacy token authentication. Trusted Publisher is configured
on npmjs.com for geolonia/geonicdb-cli + publish.yml.
@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: fa73f8a8-7e60-43f5-b001-03cf2d39a29d

📥 Commits

Reviewing files that changed from the base of the PR and between 3e62c2a and 1e6eaaa.

📒 Files selected for processing (2)
  • .github/workflows/publish.yml
  • CHANGELOG.md

📝 Walkthrough

Walkthrough

GitHub Actions の npm パブリッシュワークフロー設定を更新し、レガシートークン認証から OIDC Trusted Publishing への移行を実施。NODE_AUTH_TOKEN 環境変数を空文字列で明示的にオーバーライドし、ステップに命名を追加しました。

Changes

Cohort / File(s) Summary
CI/CD ワークフロー設定
.github/workflows/publish.yml
npm publish ステップを「Publish to npm (OIDC Trusted Publishing)」と命名し、環境変数 NODE_AUTH_TOKEN を空文字列に設定。OIDC Trusted Publishing への認証方式変更。
CHANGELOG
CHANGELOG.md
バージョン 0.10.1 に npm publish の OIDC Trusted Publishing 移行とレガシートークン認証の廃止についてのエントリを追加。

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~10 minutes

Possibly related PRs

Poem

🐰 トークンは昔の話、OIDC の道へ
信頼と安全が舞い降りて
npm の園で花開く 🌸
認証の新時代、ウサギも応援!

🚥 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 プルリクエストのタイトルはCI設定の主な変更であるnpm publishをOIDC Trusted Publishingに切り替えることを正確に反映しており、明確で具体的である。
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 feat/trusted-publishing

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

@miya0001
miya0001 merged commit 337cb38 into main Apr 5, 2026
7 checks passed
@miya0001
miya0001 deleted the feat/trusted-publishing branch April 5, 2026 16:07
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