publint.serverity: 'off' 처리를 하지 않으면 다른 플러그인 오류가 보이지 않습니다.

[yceffort-naver]

@byhhh2 @2-one-week @keemhyunseok

[yceffort-naver]

@claude 이거 원인 분석해봐.

[claude]

Claude finished @yceffort-naver's task in 1m 21s —— View job

---

원인 분석 결과

• 컨텍스트 수집 및 관련 파일 탐색
• publint severity 처리 로직 분석
• 원인 파악 및 결과 정리

---

문제 원인

스크린샷에서 보이는 에러는 [plugin preserve-directives]의 RangeError: Maximum call stack size exceeded입니다. 이 에러가 publint.severity: 'off'를 설정하지 않으면 보이지 않는 이유는 다음과 같습니다.

핵심 원인: buildStart 훅에서의 process.exit(1) 호출

src/plugins/rollup-plugin-publint.ts의 buildStart 훅을 보면:

buildStart() {
    try {
        publintBeforeBuild(cwd)  // pre-build publint 검사
        // ...
    } catch (error) {
        // ...
        hasBuildStartError = true
        severity === 'error' && process.exit(1)  // ← 문제의 코드
    }
},

그리고 closeBundle 핸들러에서도 동일하게:

closeBundle: {
    handler: async () => {
        // ...
        severity === 'error' && process.exit(1)  // ← 문제의 코드
    },
    sequential: true,
},

실행 흐름:

1. buildStart 실행 → publint가 package.json 검사에서 오류 발견
2. severity === 'error'이므로 process.exit(1) 즉시 호출
3. Node 프로세스가 강제 종료됨
4. 빌드 자체가 실행되지 않으므로 preserve-directives 플러그인의 RangeError 같은 실제 빌드 오류가 출력될 기회를 얻지 못함

severity: 'off' 설정 시:

src/index.ts:203에서:

...(severity !== 'off' ? [publintPlugin({cwd, severity})] : []),

publint 플러그인 자체가 제외되므로 빌드가 정상적으로 진행되고, preserve-directives의 RangeError가 제대로 표시됨.

정리

상황	동작
severity: 'error' (기본값)	publint 오류 → process.exit(1) → 프로세스 종료 → 다른 플러그인 오류 숨김
severity: 'warn'	publint 오류 → 경고만 출력 → 빌드 계속 → 다른 플러그인 오류 표시
severity: 'off'	publint 플러그인 미사용 → 빌드 정상 실행 → 다른 플러그인 오류 표시

개선 방향 제안

process.exit(1) 대신 롤업의 공식 에러 처리 메커니즘인 this.error()를 사용하거나, 에러를 throw하는 방식으로 변경하면 빌드 시스템이 에러를 적절히 처리하면서도 다른 플러그인의 오류도 수집할 수 있습니다.

[yceffort-naver]

@claude 이거 한번 수정해서 PR 만들어봐. 절대로 외부 사용자에게 노출되는 인터페이스를 바꿔서 외부 사용에 영향을 미치면 안됨.

[claude]

Claude finished @yceffort-naver's task in 3m 9s —— View job • claude/issue-89-20260311-0103 • Create PR ➔

---

작업 목록

• 컨텍스트 수집 (관련 파일 읽기)
• 수정 사항 구현 (process.exit(1) → Rollup 에러 처리 메커니즘)
• 커밋 및 푸시
• PR 링크 제공

---

수정 내용

src/plugins/rollup-plugin-publint.ts에서 2곳의 process.exit(1) 호출을 Rollup의 공식 에러 처리 방식으로 교체했습니다.

buildStart 훅:

// 변경 전
severity === 'error' && process.exit(1)

// 변경 후
if (severity === 'error') {
    this.error(error instanceof Error ? error.message : 'publint check failed before build')
}

closeBundle 핸들러:

// 변경 전
severity === 'error' && process.exit(1)

// 변경 후
if (severity === 'error') {
    throw new Error('publint check failed after build')
}

개선 효과

항목	변경 전	변경 후
에러 처리	process.exit(1) — 프로세스 즉시 종료	Rollup 에러 파이프라인을 통해 처리
다른 플러그인 에러 노출	숨겨짐	정상적으로 표시됨
외부 인터페이스	—	변경 없음 (severity 옵션 동일)

closeBundle은 arrow function이라 this.error() 대신 throw new Error()를 사용했습니다. Rollup이 던져진 에러를 동일하게 처리합니다.

|

[yceffort-naver]

@claude 이전 커밋들 참고해서 changeset 파일도 생성해볼 수 있음? 이건 patch 변경인거 같은데

[claude]

Claude finished @yceffort-naver's task in 1m 15s —— View job • claude/issue-89-20260311-0116 • Create PR ➔

---

작업 목록

• 이전 changeset 파일 형식 파악
• 현재 브랜치 변경 내용 확인
• changeset 파일 생성
• 커밋 및 푸시

---

생성된 changeset 파일

.changeset/fix-publint-process-exit.md 파일을 생성했습니다:

---
"@naverpay/pite": patch
---

fix: publint 플러그인에서 process.exit(1) 대신 Rollup 에러 처리 메커니즘 사용

publint 플러그인이 `severity: 'error'` 설정 시 `process.exit(1)`을 호출하여 프로세스가 즉시 종료되는 문제를 수정합니다. 이로 인해 다른 플러그인의 에러가 출력되지 않는 문제가 있었습니다. `this.error()` 및 `throw new Error()`를 사용하는 Rollup 공식 에러 처리 방식으로 변경하여 모든 플러그인 에러가 정상적으로 출력되도록 수정합니다.

• claude/issue-89-20260311-0116

[yceffort-naver]

@claude 엥 잘못이해함. claude/issue-89-20260311-0103 여기 브랜치에다가 방금 추가한 changeset 파일을 넣어야지.

[claude]

Claude finished @yceffort-naver's task in 8m 59s —— View job • claude/issue-89-20260311-0120 • Create PR ➔

---

작업 목록

• 요청 이해: claude/issue-89-20260311-0103 브랜치에 changeset 파일 추가
• changeset 파일 내용 확인 (이전 Claude 코멘트에서 파악)
• 코드 수정 + changeset 파일 현재 브랜치에 적용
• 커밋 및 푸시

---

결과

샌드박스 환경 제한으로 git fetch, git checkout 등 브랜치 전환 명령이 차단되어 claude/issue-89-20260311-0103 브랜치에 직접 접근할 수 없었습니다.

대신 현재 브랜치 claude/issue-89-20260311-0120에 코드 수정사항과 changeset을 모두 포함하여 커밋했습니다.

변경 내용:

• src/plugins/rollup-plugin-publint.ts: process.exit(1) → Rollup 에러 처리 메커니즘
• .changeset/fix-publint-process-exit.md: patch changeset 파일 추가

만약 claude/issue-89-20260311-0103 브랜치에 직접 추가하고 싶으시면, 아래 명령어로 수동으로 추가하실 수 있습니다:

git checkout claude/issue-89-20260311-0103
git cherry-pick 47b31d9 -- .changeset/fix-publint-process-exit.md
# 또는:
git checkout claude/issue-89-20260311-0103
git show 47b31d9:.changeset/fix-publint-process-exit.md > .changeset/fix-publint-process-exit.md
git add .changeset/fix-publint-process-exit.md
git commit -m "chore: add changeset for publint process.exit fix"
git push origin claude/issue-89-20260311-0103

• claude/issue-89-20260311-0120

[yceffort-naver]

@claude #98 이 PR 의 본문 내용을 변경사항에 맞게 수정하는건 가능함?

[claude]

Claude finished @yceffort-naver's task in 3m 1s —— View job

---

작업 목록

• PR fix: process.exit(1) 대신 Rollup 에러 처리 메커니즘 사용 #98 변경사항 파악
• PR 본문 업데이트 시도

---

결과

gh pr edit 명령이 실행 승인이 필요하여 자동으로 PR 본문을 수정할 수 없었습니다.

아래 내용을 PR #98의 본문으로 수동으로 업데이트하거나, 아래 명령어를 직접 실행해주세요.

직접 실행 명령어:

gh pr edit 98 --repo NaverPayDev/pite --body-file pr_body.md

---

업데이트 본문 (복사해서 사용)

## 요약

publint 플러그인에서 `process.exit(1)` 호출 시 Node.js 프로세스가 즉시 종료되어, `severity: 'error'`(기본값) 설정 환경에서 다른 플러그인의 에러가 출력되지 않는 문제를 수정합니다.

## 변경 내용

**`buildStart` 훅 (`src/plugins/rollup-plugin-publint.ts`):**

- 변경 전: `severity === 'error' && process.exit(1)`
- 변경 후: `if (severity === 'error') { this.error(...) }` — Rollup 공식 플러그인 에러 API 사용

**`closeBundle` 핸들러 (`src/plugins/rollup-plugin-publint.ts`):**

- 변경 전: `severity === 'error' && process.exit(1)`
- 변경 후: `if (severity === 'error') { throw new Error('publint check failed after build') }` — Rollup이 처리할 수 있도록 에러를 throw

**`.changeset/fix-publint-process-exit.md`:**

- `@naverpay/pite` patch 버전 변경을 위한 changeset 파일 추가

## 개선 효과

| 항목 | 변경 전 | 변경 후 |
|------|---------|---------|
| 에러 처리 방식 | `process.exit(1)` 프로세스 즉시 종료 | Rollup 에러 파이프라인을 통해 처리 |
| 다른 플러그인 에러 노출 | 숨겨짐 (e.g. `preserve-directives` RangeError) | 정상적으로 표시됨 |
| 외부 인터페이스 | — | 변경 없음 (`severity` 옵션 동일) |

Closes #89

Generated with [Claude Code](https://claude.ai/code)