Skip to content

[CP-1323] ci: 워크플로우의 죽은 npm 인증 설정 제거 - #155

Merged
tosspayments-bot merged 2 commits into
mainfrom
ci/release-workflow-cleanup
Aug 19, 2026
Merged

[CP-1323] ci: 워크플로우의 죽은 npm 인증 설정 제거#155
tosspayments-bot merged 2 commits into
mainfrom
ci/release-workflow-cleanup

Conversation

@noahluftyang

@noahluftyang noahluftyang commented Aug 19, 2026

Copy link
Copy Markdown
Contributor

배경

두 가지 변화로 워크플로우의 npm 인증 설정이 전부 불필요해졌습니다.

  1. 퍼블리시 인증이 토큰에서 trusted publishing(OIDC)으로 전환release.yml의 publish 경로에 토큰이 필요 없습니다.
  2. 레지스트리 인증이 필요한 의존성이 사라짐standard-public-interfaces를 소스로 전환한 chore: standard-public-interfaces 타입을 2.17.1 로 갱신 #153 이후 pnpm install이 인증 없이 완결됩니다.

변경

ci.ymlrelease.yml에서 아래 설정을 제거합니다.

제거 대상 근거
NODE_AUTH_TOKEN 양쪽 install 인증이 필요한 의존성이 없습니다 (#153)
registry-url 양쪽 setup-node NODE_AUTH_TOKEN 주입 전용 설정입니다. 게다가 .npmrc에 빈 _authToken=을 써서 npm이 인증이 설정된 것으로 오인하고 OIDC 교환을 건너뛰게 만드는 알려진 함정이라(actions/setup-node#1551), 남겨두면 publish 경로에 해롭습니다
NPM_CONFIG_PROVENANCE release.yml trusted publishing이 provenance를 자동 생성합니다

permissions 3종(contents·pull-requests·id-token)과 cache: 'pnpm', version/publish 커맨드, build/typecheck 스텝은 그대로 둡니다.

검증

ci.yml 변경은 이 PR의 CI가 그대로 검증합니다 — install이 인증 없이 통과하는 것이 곧 실증입니다. install 스텝 구조가 같으므로 release.yml의 install도 함께 뒷받침됩니다.

다만 publish 경로는 changeset이 실제로 쌓인 릴리즈에서만 검증됩니다. changeset이 없으면 publish를 타지 않아 no-op으로 통과하므로, 다음 퍼블리시에서 아래 두 가지를 확인해야 합니다.

  • 런 로그에 using npm trusted publishing + published successfully
  • provenance 유지:
    curl -sS "https://registry.npmjs.org/-/npm/v1/attestations/@tosspayments%2ftosspayments-sdk@<버전>" | jq -e '.attestations|length>0'
    

🤖 Generated with Claude Code

npm trusted publishing(OIDC)으로 퍼블리시하므로 토큰 기반 설정이 더 이상 필요 없다.

- registry-url: OIDC에 불필요하며, .npmrc 에 빈 _authToken 을 써서 npm 이 인증이
  설정된 것으로 오인하고 OIDC 교환을 건너뛰게 만드는 알려진 함정
  (actions/setup-node#1551)
- NODE_AUTH_TOKEN: 의존성이 모두 public 이라 install 에 registry 인증이 필요 없다
- NPM_CONFIG_PROVENANCE: trusted publishing 이 provenance 를 자동 생성한다

CP-1323

Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>
@changeset-bot

changeset-bot Bot commented Aug 19, 2026

Copy link
Copy Markdown

⚠️ No Changeset found

Latest commit: 1d5f338

Merging this PR will not cause a version bump for any packages. If these changes should not result in a new version, you're good to go. If these changes should result in a version bump, you need to add a changeset.

This PR includes no changesets

When changesets are added to this PR, you'll see the packages that this PR includes changesets for and the associated semver types

Click here to learn what changesets are, and how to add one.

Click here if you're a maintainer who wants to add a changeset to this PR

standard-public-interfaces 를 소스로 전환(#153)하면서 레지스트리 인증이 필요한
의존성이 사라졌다. registry-url 은 NODE_AUTH_TOKEN 주입 전용이므로 함께 제거한다.

CP-1323

Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>
@noahluftyang noahluftyang changed the title [CP-1323] ci: trusted publishing 전환 후 불필요해진 release 워크플로우 설정 제거 [CP-1323] ci: 워크플로우의 죽은 npm 인증 설정 제거 Aug 19, 2026
@tosspayments-bot
tosspayments-bot merged commit 6c595a3 into main Aug 19, 2026
1 check passed
@tosspayments-bot
tosspayments-bot deleted the ci/release-workflow-cleanup branch August 19, 2026 03:58
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