Skip to content

fix: process.exit(1) 대신 Rollup 에러 처리 메커니즘 사용#96

Closed
yceffort-naver wants to merge 1 commit into
mainfrom
claude/issue-89-20260311-0103
Closed

fix: process.exit(1) 대신 Rollup 에러 처리 메커니즘 사용#96
yceffort-naver wants to merge 1 commit into
mainfrom
claude/issue-89-20260311-0103

Conversation

@yceffort-naver

Copy link
Copy Markdown
Contributor

publint 플러그인에서 process.exit(1) 호출 시 프로세스가 즉시 종료되어 다른 플러그인의 에러가 출력되지 않는 문제를 수정합니다.

Closes #89

Generated with Claude Code

publint 플러그인에서 process.exit(1) 호출 시 프로세스가 즉시 종료되어
다른 플러그인(e.g. preserve-directives)의 에러가 출력되지 않는 문제를 수정합니다.

- buildStart: this.error()를 사용하여 Rollup의 공식 에러 처리 파이프라인을 활용
- closeBundle: throw new Error()를 사용하여 Rollup이 에러를 적절히 처리하도록 변경

외부 인터페이스(publint.severity 옵션)는 변경 없음.

Fixes #89

Co-authored-by: yceffort_naver <yceffort-naver@users.noreply.github.com>
@yceffort-naver yceffort-naver self-assigned this Mar 11, 2026
@yceffort-naver yceffort-naver requested a review from a team as a code owner March 11, 2026 01:14
@claude

claude Bot commented Mar 11, 2026

Copy link
Copy Markdown

Claude Code Review

Code review was not run because your organization is out of credits.

An admin can add credits from the billing page in the Anthropic Console. Once credits are available, reopen this pull request to trigger a review.

@npayfebot

Copy link
Copy Markdown
Contributor

⚠️ No Changeset found

Latest commit: 4fd0ef5

@naverpay/pite package have detected changes.

If no version change is needed, please add skip-detect-change to the label.

If you want to add changes to .changeset, please select one of the following options or use the Claude Code Plugin provided by NaverPayDev.

💥 X.0.0 major bump
✨ 0.X.0 minor bump
🩹 0.0.X patch bump

Plugin Usage

Add marketplace & install changeset plugin

/plugin marketplace add NaverPayDev/naverpay-plugins
/plugin install naverpay-changeset@naverpay-plugins

Usage

/naverpay-changeset: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

powered by: naverpay changeset detect-add actions

@npayfebot

Copy link
Copy Markdown
Contributor

NPM Packages

📦 @naverpay/pite

Total Sizes: 10.3 kB

Total Changes: +78 B (+1%) (Size Increased)

File Status Previous Size Updated Size Changed
/dist/plugins/rollup-plugin-publint.mjs 🛠️ 704 B 742 B +38 B (+5%)
/dist/plugins/rollup-plugin-publint.js 🛠️ 709 B 749 B +40 B (+6%)
ℹ️ View Unchanged Files
File Status Previous Size Updated Size Changed
/dist/vite-tsup-plugin.mjs - 678 B 678 B -
/dist/vite-tsup-plugin.js - 681 B 681 B -
/dist/util.mjs - 295 B 295 B -
/dist/util.js - 352 B 352 B -
/dist/polyfill.mjs - 520 B 520 B -
/dist/polyfill.js - 576 B 576 B -
/dist/index.mjs - 1.48 kB 1.48 kB -
/dist/index.js - 1.56 kB 1.56 kB -
/dist/get-vite-entry.mjs - 637 B 637 B -
/dist/get-vite-entry.js - 695 B 695 B -
/dist/dependencies.mjs - 337 B 337 B -
/dist/dependencies.js - 389 B 389 B -
/dist/browserslist.mjs - 264 B 264 B -
/dist/browserslist.js - 323 B 323 B -

powered by: naverpay size-action

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.

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

2 participants