fix: add repository field for npm provenance#24
Conversation
npm publish --provenance rejects packages whose package.json lacks a repository.url matching the building repo (E422: "repository.url" is "", expected to match https://github.com/pleaseai/code-style). The eslint-config and prettier-config packages had no repository field, so the publish_only run failed at provenance verification. Add it to both (cli already had one).
There was a problem hiding this comment.
Code Review
This pull request adds repository metadata to the package.json files for both the ESLint and Prettier configuration packages. A review comment highlights a typo in the directory name ('perttier-config' instead of 'prettier-config') within the Prettier package configuration, which should be addressed to maintain consistency.
Important
The consumer version of Gemini Code Assist on GitHub is being sunset. Starting June 18, 2026, new organization installations will be blocked, and all code review activity will officially cease on July 17, 2026.
For more details on the timeline and next steps, please review the Help Documentation.
| "repository": { | ||
| "type": "git", | ||
| "url": "https://github.com/pleaseai/code-style.git", | ||
| "directory": "packages/perttier-config" |
There was a problem hiding this comment.
디렉터리 이름에 오타(perttier-config -> prettier-config)가 있습니다. 패키지 이름은 @pleaseai/prettier-config로 올바르게 설정되어 있으나, 실제 디렉터리 경로명에 오타가 존재하여 일관성이 떨어집니다.\n\n현재 실제 디렉터리명이 perttier-config로 되어 있어 npm provenance 검증을 통과하려면 이 값을 유지해야 할 수 있습니다. 하지만 혼선을 방지하기 위해 추후 디렉터리 이름을 prettier-config로 변경하고 이 필드도 함께 수정하는 리팩토링을 진행하는 것을 권장합니다.
문제
publish_only디스패치 발행이 provenance 검증에서 실패했습니다 (run). OIDC 인증·provenance 서명까지는 성공했지만:npm publish --provenance는 package.json의repository.url이 빌드 repo와 일치해야 합니다.@pleaseai/eslint-config와@pleaseai/prettier-config엔repository필드가 아예 없어서 거부됐습니다. (cli는 이미 보유 → 첫 스텝 실패로 뒤 패키지는 skip됨.)변경
두 패키지 package.json에
repository필드 추가 (cli와 동일 패턴,directory포함):머지 후
→ eslint 0.0.4 / code-style 0.0.3 / prettier 0.0.1이 provenance와 함께 발행됩니다.