Skip to content

fix: CI 에서 pnpm v11 build script 차단으로 인한 docker build 실패 해소#139

Merged
youngminss merged 1 commit into
developfrom
fix/ci-pnpm-build-scripts
May 7, 2026
Merged

fix: CI 에서 pnpm v11 build script 차단으로 인한 docker build 실패 해소#139
youngminss merged 1 commit into
developfrom
fix/ci-pnpm-build-scripts

Conversation

@youngminss
Copy link
Copy Markdown
Member

🎯 PR 제목

fix: CI 에서 pnpm v11 build script 차단으로 인한 docker build 실패 해소

📑 작업 상세 내역

  • 버그 수정
    • package.jsonpnpm.onlyBuiltDependencies 화이트리스트를 추가합니다 (sharp, unrs-resolver)
    • corepack 이 CI 환경에서 받는 pnpm v11 이 ERR_PNPM_IGNORED_BUILDS 로 install 을 실패시키던 문제를 해소합니다 — 두 패키지 모두 잘 알려진 native 빌드 의존성(이미지 최적화 / ESLint resolver)이라 명시적 허용

🙏 리뷰 요청 사항

  • 추가된 화이트리스트 두 패키지(sharp, unrs-resolver) 외에 다른 빌드 스크립트가 추가로 막히지 않는지 (CI 통과 확인) 부탁드립니다
  • 향후 새 의존성 추가 시 동일 에러 재발 가능성이 있으니 follow-up 으로 packageManager 필드 핀(예: "packageManager": "pnpm@11.0.8") 도 검토 가치 있음 — 본 PR 비범위

📃 참고 자료

🖼️ 작업 결과물

pnpm v11 (resolved by corepack on the CI image) refuses to run native
install scripts that aren't explicitly approved and exits non-zero. Adds a
pnpm.onlyBuiltDependencies allowlist so sharp (Next.js image optimization)
and unrs-resolver (ESLint module resolver) execute their build steps and
the docker build of the client image succeeds again.

Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
@youngminss youngminss added the 🐞 Fix 버그 수정 및 이슈 해결 label May 7, 2026
@youngminss youngminss self-assigned this May 7, 2026
@youngminss youngminss requested a review from RookieAND May 7, 2026 16:27
@github-actions github-actions Bot added 🐞 Fix 버그 수정 및 이슈 해결 and removed 🐞 Fix 버그 수정 및 이슈 해결 labels May 7, 2026
@youngminss youngminss merged commit 6399273 into develop May 7, 2026
3 checks passed
github-actions Bot pushed a commit that referenced this pull request May 7, 2026
## [2.7.0-beta.6](v2.7.0-beta.5...v2.7.0-beta.6) (2026-05-07)

### Bug Fixes

* **ci:** allowlist sharp and unrs-resolver build scripts ([#139](#139)) ([6399273](6399273))
Copy link
Copy Markdown
Contributor

@gemini-code-assist gemini-code-assist Bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Code Review

package.json 파일에 pnpm.onlyBuiltDependencies 설정을 추가하여 sharp와 unrs-resolver를 빌드 허용 목록에 포함했습니다. 리뷰 과정에서 unrs-resolver 패키지가 npm에 존재하지 않는 것으로 확인되어, 이를 oxc-resolver로 수정할 것을 제안했습니다.

Comment thread package.json
"pnpm": {
"onlyBuiltDependencies": [
"sharp",
"unrs-resolver"
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

high

unrs-resolver라는 패키지는 npm에 존재하지 않는 것으로 보입니다. PR 설명에서 "잘 알려진 native 빌드 의존성(ESLint resolver)"이라고 언급하신 점으로 미루어 보아, Rust 기반의 oxc-resolver를 의도하신 것이 아닌지 확인이 필요합니다. 만약 패키지명이 잘못되었다면 pnpm의 빌드 차단 문제가 해결되지 않을 수 있습니다.

Suggested change
"unrs-resolver"
"oxc-resolver"

youngminss added a commit that referenced this pull request May 7, 2026
The previous attempts (#139, #140) tried to allowlist sharp and
unrs-resolver under pnpm v11's stricter ignored-builds policy, but
v11 still fails the install with ERR_PNPM_IGNORED_BUILDS even when
onlyBuiltDependencies is configured. Corepack picked v11.0.8 because
the project never declared a packageManager field, so a single
release of pnpm flipped CI from green to red.

Pins pnpm to 10.13.1 (the version that produced the last green
build) via packageManager so corepack resolves the same version on
CI as on local dev. The pnpm-workspace.yaml allowlist is kept so a
future migration to v11 stays straightforward.

Co-authored-by: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
github-actions Bot pushed a commit that referenced this pull request May 12, 2026
## [2.7.0](v2.6.0...v2.7.0) (2026-05-12)

### Features

* RegionStep에 지역 요청 Banner 및 Dialog 추가 ([#133](#133)) ([b61ba13](b61ba13))
* 식당 상세 시안 정렬 및 전화하기 CTA 추가 ([#138](#138)) ([b675fc4](b675fc4))
* 식당 상세 페이지 신규 구현 ([#134](#134)) ([055841c](055841c))
* 약속 장소 섹션 캐러셀 UI 개편 ([#137](#137)) ([c77b04a](c77b04a))

### Bug Fixes

* certbot 컨테이너 제거 후 호스트 certbot 방식으로 전환 ([a104bfb](a104bfb))
* **ci:** allowlist sharp and unrs-resolver build scripts ([#139](#139)) ([6399273](6399273))
* **ci:** move pnpm build allowlist to pnpm-workspace.yaml ([#140](#140)) ([17ecd51](17ecd51))
* **ci:** pin pnpm to 10.13.1 via packageManager ([#141](#141)) ([e020a3c](e020a3c)), closes [#139](#139) [#140](#140)
* **ci:** pnpm 버전 충돌 해결을 위해 워크플로우의 version 명시 제거 ([a5000af](a5000af))

### Code Refactoring

* VoteSummarySection 거리 블록 숨김 및 디렉터리 분리 ([#136](#136)) ([31f9d13](31f9d13)), closes [#132](#132)
github-actions Bot pushed a commit that referenced this pull request May 12, 2026
## [2.7.0-beta.7](v2.7.0-beta.6...v2.7.0-beta.7) (2026-05-12)

### Bug Fixes

* certbot 컨테이너 제거 후 호스트 certbot 방식으로 전환 ([a104bfb](a104bfb))
* **ci:** move pnpm build allowlist to pnpm-workspace.yaml ([#140](#140)) ([17ecd51](17ecd51))
* **ci:** pin pnpm to 10.13.1 via packageManager ([#141](#141)) ([e020a3c](e020a3c)), closes [#139](#139) [#140](#140)
* **ci:** pnpm 버전 충돌 해결을 위해 워크플로우의 version 명시 제거 ([a5000af](a5000af))
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

🐞 Fix 버그 수정 및 이슈 해결

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant