Skip to content

ci(claude-review): Claude Code 사전 설치 step 추가#26

Merged
laegel123 merged 1 commit into
mainfrom
fix/claude-review-binary-fallback
May 6, 2026
Merged

ci(claude-review): Claude Code 사전 설치 step 추가#26
laegel123 merged 1 commit into
mainfrom
fix/claude-review-binary-fallback

Conversation

@laegel123
Copy link
Copy Markdown
Owner

Summary

claude-review 워크플로의 업스트림 회귀 워크어라운드. anthropics/claude-code-action@v1 (v1.0.100+) 가 내부에서 호출하는 curl claude.ai/install.sh | bash 이 network 실패 시 silent fail (no pipefail) 하여 SDK 가 ~/.local/bin/claude 부재로 ReferenceError: Claude Code native binary not found 에러로 죽는 문제. 액션 실행 전에 strict 에러 처리로 명시적 설치 (1차 install.sh, 2차 npm) → 액션 내부 install 재시도가 실패해도 우리 바이너리가 살아남아 SDK 가 정상 동작.

Changes

  • .github/workflows/claude-review.yml — checkout 직후, action 호출 직전에 Pre-install Claude Code step 추가. 28라인.
    • 1차: curl -fsSL https://claude.ai/install.sh -o /tmp/... + bash (strict pipefail)
    • 2차: npm install -g @anthropic-ai/claude-code + ~/.local/bin/claude 심볼릭 링크
    • 둘 다 실패 시 명시적으로 exit 1

Background

  • 업스트림 이슈: #1242
  • 업스트림 fix PR: #1260 (SDK bundled binary fallback) — 미머지
  • 본 워크어라운드는 #1260 머지 + 새 액션 릴리스 후 제거 가능 (커밋 메시지에 명시)

Test plan

🤖 Generated with Claude Code

…action#1242 워크어라운드)

업스트림 v1.0.100+ 회귀 — 액션 내부의 `curl claude.ai/install.sh | bash` 가
network 실패 시 silent fail (no pipefail) 하여 SDK 가 `~/.local/bin/claude`
부재로 `ReferenceError: Claude Code native binary not found` 로 죽는 이슈
(GitHub-hosted runner 풀 환경에 따라 간헐적 / 일관적 실패). 액션 step 전에
명시적으로 strict 에러 처리로 install 시도, 실패 시 npm 으로 fallback.
업스트림 #1260 (SDK bundled binary fallback) 머지되면 본 step 제거.

Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
@laegel123 laegel123 merged commit 33e294b into main May 6, 2026
1 check failed
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