Skip to content

feat(ai): claude code init settings#16

Merged
citron03 merged 1 commit intomainfrom
feat/claude-md
Mar 17, 2026
Merged

feat(ai): claude code init settings#16
citron03 merged 1 commit intomainfrom
feat/claude-md

Conversation

@citron03
Copy link
Owner

No description provided.

@github-actions
Copy link

PR Reviewer Guide 🔍

Here are some key observations to aid the review process:

⏱️ Estimated effort to review: 2 🔵🔵⚪⚪⚪
🧪 No relevant tests
🔒 No security concerns identified
⚡ Recommended focus areas for review

명령어 가용성

pnpm web:lint 명령어의 현재 가용성에 대한 명확한 설명이 있으면 좋을 것 같아요. 현재 사용 가능한 명령어인지, 아니면 향후 추가될 예정인지 명시하면 Claude가 더 정확하게 작업을 수행할 수 있을 것 같습니다! 🤖✨

- 앱 단위 실행: `pnpm web:dev`, `pnpm tanstack:dev`, `pnpm ds:dev`

@github-actions
Copy link

PR Code Suggestions ✨

Explore these optional code suggestions:

CategorySuggestion                                                                                                                                    Impact
General
린트 스크립트 가용성 지침 명확화

CLAUDE.md 파일에는 pnpm web:lint 스크립트가 명시적으로 나열되어 있지 않아, AI가 해당 스크립트의 가용성을 판단하기 어렵습니다.
혼란을 방지하고 일관성을 유지하기 위해, CLAUDE.md에 명시된 루트 pnpm lint 스크립트를 사용하도록 지침을 단순화하는 것이 좋습니다.

.claude/skills/lint-fix/SKILL.md [11]

-3. If only web app changed, use `pnpm web:lint` when available; otherwise root.
+3. If only web app changed, use `pnpm lint` from the repo root.
Suggestion importance[1-10]: 7

__

Why: CLAUDE.mdpnpm web:lint 스크립트가 명시되어 있지 않아 AI가 가용성을 판단하기 어렵습니다. pnpm lint로 변경하여 AI 지침의 일관성과 정확성을 높입니다.

Medium
테스트 설정 확인 지침 명확화

"ensure"라는 표현은 AI가 능동적으로 설정을 확인하거나 변경해야 한다는 의미로 해석될 수 있습니다. AI의 역할을 명확히 하기 위해, 이 지침을
AI가 확인하고 보고해야 하는 사항으로 재구성하는 것이 좋습니다.

.claude/skills/test-run/SKILL.md [12]

-4. For web unit tests, ensure Vitest + jsdom config is used.
+4. For web unit tests, verify that Vitest + jsdom config is used.
Suggestion importance[1-10]: 6

__

Why: "ensure"를 "verify"로 변경하여 AI의 역할을 명확히 합니다. 이는 AI가 능동적으로 설정을 변경하는 대신 확인하고 보고해야 함을 강조합니다.

Low

@citron03 citron03 merged commit a97f885 into main Mar 17, 2026
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant