Skip to content

Conversation

@pja9362
Copy link
Contributor

@pja9362 pja9362 commented Jan 6, 2026

Related Issue

Describe your changes

배경

@naverpay/code-style-cli 도입으로 패키지 설치와 설정 파일 생성을 CLI에서 통합 관리할 수 있게 되었습니다.

기존에 postinstall 스크립트를 사용하던 패키지들은 pnpm v10+ 환경에서 정상 동작하지 않는 문제가 있었고, 이제 CLI를 통해 이 문제가 해결되었으므로 postinstall 스크립트를 제거합니다.

작업 내용

  • @naverpay/editorconfig에서 postinstall 스크립트 및 관련 파일(createConfigFile.js, index.js) 제거
  • @naverpay/markdown-lint에서 postinstall 스크립트 제거 및 npm 배포 대상에서 postInstall 제외
  • 각 패키지 README 업데이트 (CLI 사용 안내 추가)

Breaking Changes

  • postinstall 스크립트 제거로 자동 설정 파일 생성 기능이 제거됨
  • @naverpay/code-style-cli 사용을 권장하는 방식으로 설치 가이드 변경

- package.json에서 postinstall 스크립트 제거
- index.js, createConfigFile.js 파일 삭제
- README.md 업데이트 (CLI 사용 안내 추가)

pnpm v10+에서 postinstall이 기본 차단되는 문제로 인해
code-style-cli를 통한 설정 파일 생성 방식으로 변경
@npayfebot
Copy link
Contributor

npayfebot commented Jan 6, 2026

✅ Changeset detected

Latest commit: e1e0d91

@naverpay/editorconfig, @naverpay/markdown-lint packages have detected changes.

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

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 2 packages
Name Type
@naverpay/markdown-lint 💥 Major
@naverpay/editorconfig 💥 Major
powered by: naverpay changeset detect-add actions

@pja9362
Copy link
Contributor Author

pja9362 commented Jan 6, 2026

/canary-publish

@pja9362
Copy link
Contributor Author

pja9362 commented Jan 8, 2026

/rc-publish

@pja9362
Copy link
Contributor Author

pja9362 commented Jan 12, 2026

/canary-publish

@pja9362
Copy link
Contributor Author

pja9362 commented Jan 12, 2026

/rc-publish

@pja9362
Copy link
Contributor Author

pja9362 commented Jan 13, 2026

/canary-publish

@npayfebot
Copy link
Contributor

Published Canary Packages

@pja9362
Copy link
Contributor Author

pja9362 commented Jan 13, 2026

/rc-publish

@npayfebot
Copy link
Contributor

Published Canary Packages

@pja9362 pja9362 self-assigned this Jan 13, 2026
@pja9362 pja9362 marked this pull request as ready for review January 13, 2026 06:13
@pja9362 pja9362 requested a review from a team as a code owner January 13, 2026 06:13
@yceffort-naver yceffort-naver requested a review from byhhh2 January 13, 2026 06:21
Copy link
Contributor

@yceffort-naver yceffort-naver left a comment

Choose a reason for hiding this comment

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

LGTM

@pja9362 pja9362 merged commit d8f1315 into main Jan 13, 2026
3 checks passed
@pja9362 pja9362 deleted the feature/141 branch January 13, 2026 06:28
@pja9362
Copy link
Contributor Author

pja9362 commented Jan 13, 2026

@Ohjaeyeop 님, 클로드 플러그인 관련하여 제보할 것이 있습니다!! 🎤

재엽님께서 만들어주신 클로드 플러그인 > /changeset 커맨드 활용하여 위 작업의 changeset 파일을 생성하였는데요!

현재 changeset → CHANGELOG 변환 과정에서 마크다운 형식이 깨지는 이슈가 있어서, 플러그인이 생성하는 changeset 형식에 일부 수정이 필요할 것 같습니다 🥹 저희가 관리하는 대부분의 패키지들이 changeset 기반으로 CHANGELOG를 관리하고 있어서, 플러그인에서 생성하는 형식도 이에 맞춰주면 좋을 것 같아서요!

📝 CHANGELOG 생성 과정 : changeset 라이브러리 활용

.changeset/*.md 파일
       ↓
changeset version 실행 (by changeset-actions)
       ↓
@changesets/apply-release-plan이 CHANGELOG 구조 생성
  - 버전 헤딩: ## 2.0.0
  - 섹션 헤딩: ### Major Changes, ### Minor Changes 등
       ↓
@changesets/changelog-git이 각 changeset 내용을 포맷팅 🌟
  - 첫 줄: `- {커밋해시}: {첫 줄 내용}` ← 제목/요약 역할
  - 이후 줄: 2칸 들여쓰기 추가
       ↓
CHANGELOG.md 생성

⚠️ 이슈1: 제목 줄 누락

@changesets/changelog-git은 changeset 내용의 첫 줄을 제목/요약으로 사용합니다.

현재 /changeset 플러그인은 제목 역할을 하는 줄이 없습니다.

⚠️ 이슈 2: 헤딩 들여쓰기로 인한 lint 오류

@changesets/changelog-git이 첫 줄 이후 모든 라인에 들여쓰기를 추가합니다.

이로 인해 ### 헤딩이 들여쓰기되면 마크다운 형식이 깨지고, markdownlint MD023 오류가 발생합니다.

📌 실제 변환 예시

  • .changeset/brave-tigers-fly.md
스크린샷 2026-01-13 오후 4 52 43
  • 생성된 CHANGELOG
스크린샷 2026-01-13 오후 4 54 47 스크린샷 2026-01-13 오후 4 55 12

✅ 수정 제안

1. 첫 줄에 제목/요약 추가

2. ### 대신 **굵은 텍스트** 사용

  • 적용 예시
---
"@naverpay/markdown-lint": major
---

postinstall 스크립트 제거

**Breaking Changes**

- `postinstall` 스크립트 제거로 패키지 설치 시 자동 설정 파일 생성 기능이 제거됨
  - 기존: `npm install` 시 `.markdownlint.json` 파일이 프로젝트 루트에 자동 생성
  - 변경: 수동으로 설정 파일 생성 필요 또는 `@naverpay/code-style-cli` 사용 권장

**코드 정리**

- `package.json`에서 `postinstall` 스크립트 제거
- npm 배포 대상(`files`)에서 `postInstall` 디렉토리 제외

**문서 업데이트**

- README에 새로운 설치 가이드 추가 (`@naverpay/code-style-cli` 사용 또는 수동 설정 방법 안내)

  • 변환 예시
스크린샷 2026-01-13 오후 5 10 59

시간 되실 때 확인 한 번 부탁드리겠습니다! 🙇‍♀️

@Ohjaeyeop
Copy link
Contributor

흐억 감사합니다

@pja9362
Copy link
Contributor Author

pja9362 commented Jan 13, 2026

테스트할 때 CHANGELOG 파일까지는 생각을 못했었네용 .. ㅠㅅㅠ
잘 부탁드립니다 클로드의 마술사님 ~~~ 🪄✨

pja9362 added a commit that referenced this pull request Jan 14, 2026
This reverts commit d8f1315, reversing
changes made to 926454a.
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.

5 participants