Skip to content

fix(ci): release checkout는 기본 GITHUB_TOKEN 사용 + fetch-depth/token 제거#151

Merged
noahluftyang merged 2 commits into
mainfrom
fix/release-checkout-token
Jun 19, 2026
Merged

fix(ci): release checkout는 기본 GITHUB_TOKEN 사용 + fetch-depth/token 제거#151
noahluftyang merged 2 commits into
mainfrom
fix/release-checkout-token

Conversation

@noahluftyang

Copy link
Copy Markdown
Contributor

문제

PR #150 머지 후 Release 워크플로우가 checkout에서 실패:

fatal: could not read Username for 'https://github.com': terminal prompts disabled  (exit 128)

actions/checkout에 봇 PAT(GH_TOSSPAYMENTS_BOT_TOKEN)를 token으로 주입했는데, 그 PAT로 git fetch가 거부됨.

수정

  • checkout은 레포 read만 필요 → 기본 GITHUB_TOKEN 사용 (커스텀 token 입력 제거)
  • 봇 PAT는 PR 생성이 필요한 changesets/action에만 유지
  • fetch-depth: 0 제거 — changesets는 changeset 파일 + npm 레지스트리 비교로 동작하므로 전체 git 히스토리 불필요 (기본 depth로 충분)

효과

  • checkout 통과 → Release 워크플로우 정상화
  • pending changeset이 없으면 changeset publish는 no-op이라 green
  • 실제 Version PR 생성 시 changesets/action이 봇 PAT로 PR을 만들어 CI를 트리거 — 이때 PAT는 Contents: RW + Pull requests: RW (+ org SSO 승인) 가 필요

🤖 Generated with Claude Code

noahluftyang and others added 2 commits June 19, 2026 13:07
…n에만 주입

checkout이 봇 PAT로 git fetch를 시도하다 'could not read Username'(exit 128)으로
실패. checkout은 레포 read만 필요하므로 기본 GITHUB_TOKEN으로 충분하고,
봇 PAT(GH_TOSSPAYMENTS_BOT_TOKEN)는 PR 생성이 필요한 changesets/action에만 둔다.

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
changesets/action은 changeset 파일과 npm 레지스트리 비교로 동작해 전체 git
히스토리가 불필요. 기본 depth로 충분.

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

changeset-bot Bot commented Jun 19, 2026

Copy link
Copy Markdown

⚠️ No Changeset found

Latest commit: 002da64

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

@noahluftyang noahluftyang merged commit 9883c9c into main Jun 19, 2026
1 check passed
@noahluftyang noahluftyang deleted the fix/release-checkout-token branch June 19, 2026 04:11
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