jun-claude-code 0.6.0 → 0.6.2

package/dist/copy.js

@@ -40,25 +40,10 @@ exports.mergeSettingsJson = mergeSettingsJson;
 exports.copyClaudeFiles = copyClaudeFiles;
 const fs = __importStar(require("fs"));
 const path = __importStar(require("path"));
-const readline = __importStar(require("readline"));
 const crypto = __importStar(require("crypto"));
 const chalk_1 = __importDefault(require("chalk"));
-/**
- * Prompt user for confirmation using readline
- */
-function askConfirmation(question) {
-    const rl = readline.createInterface({
-        input: process.stdin,
-        output: process.stdout,
-    });
-    return new Promise((resolve) => {
-        rl.question(question, (answer) => {
-            rl.close();
-            const normalized = answer.toLowerCase().trim();
-            resolve(normalized === 'y' || normalized === 'yes');
-        });
-    });
-}
+// eslint-disable-next-line @typescript-eslint/no-var-requires
+const { MultiSelect } = require('enquirer');
 /**
  * Calculate SHA-256 hash of a file
  */
@@ -119,6 +104,137 @@ function getDestClaudeDir() {
     }
     return path.join(homeDir, '.claude');
 }
+/**
+ * Categorize files into agents, skills (top-level dirs), and others
+ */
+function categorizeFiles(files) {
+    const agents = [];
+    const skillDirs = new Set();
+    const others = [];
+    for (const file of files) {
+        if (file.startsWith('agents/')) {
+            agents.push(file);
+        }
+        else if (file.startsWith('skills/')) {
+            const parts = file.split('/');
+            if (parts.length >= 2 && parts[1]) {
+                skillDirs.add(parts[1]);
+            }
+        }
+        else {
+            others.push(file);
+        }
+    }
+    return {
+        agents: agents.sort(),
+        skills: Array.from(skillDirs).sort(),
+        others: others.sort(),
+    };
+}
+/**
+ * Get file status (new/changed/unchanged)
+ */
+function getFileStatus(sourcePath, destPath) {
+    if (!fs.existsSync(destPath))
+        return 'new';
+    return getFileHash(sourcePath) === getFileHash(destPath) ? 'unchanged' : 'changed';
+}
+/**
+ * Get skill directory status by checking all files within
+ */
+function getSkillStatus(skillName, sourceDir, destDir) {
+    const sourceSkillDir = path.join(sourceDir, 'skills', skillName);
+    const destSkillDir = path.join(destDir, 'skills', skillName);
+    if (!fs.existsSync(destSkillDir))
+        return 'new';
+    const sourceFiles = getAllFiles(sourceSkillDir, sourceSkillDir);
+    for (const file of sourceFiles) {
+        const src = path.join(sourceSkillDir, file);
+        const dst = path.join(destSkillDir, file);
+        if (!fs.existsSync(dst) || getFileHash(src) !== getFileHash(dst)) {
+            return 'changed';
+        }
+    }
+    return 'unchanged';
+}
+/**
+ * Format status label for display
+ */
+function statusLabel(status) {
+    switch (status) {
+        case 'new': return chalk_1.default.green('new');
+        case 'changed': return chalk_1.default.yellow('changed');
+        case 'unchanged': return chalk_1.default.gray('unchanged');
+    }
+}
+/**
+ * Format status label with brackets for log output
+ */
+function statusBracket(status) {
+    switch (status) {
+        case 'new': return chalk_1.default.green('[new]');
+        case 'changed': return chalk_1.default.yellow('[changed]');
+        case 'unchanged': return chalk_1.default.gray('[unchanged]');
+    }
+}
+/**
+ * Show MultiSelect prompt for a category
+ */
+async function selectItems(category, items) {
+    if (items.length === 0)
+        return [];
+    const choices = items.map(item => ({
+        name: item.name,
+        message: item.displayName,
+        hint: statusLabel(item.status),
+        enabled: item.status !== 'unchanged',
+    }));
+    const prompt = new MultiSelect({
+        name: category,
+        message: `Select ${category} to install`,
+        choices,
+        hint: '(↑↓ navigate, <space> toggle, <a> select all, <enter> confirm)',
+    });
+    try {
+        return await prompt.run();
+    }
+    catch {
+        console.log(chalk_1.default.yellow('\nInstallation cancelled.'));
+        process.exit(0);
+    }
+}
+/**
+ * Show MultiSelect prompt for skill sub-files across multiple skills.
+ * Groups files by skill with separators.
+ */
+async function selectSkillSubFiles(skills, sourceDir, destDir) {
+    const choices = [];
+    for (const { skillName, subFiles } of skills) {
+        choices.push({ role: 'separator', message: chalk_1.default.cyan(`── ${skillName} ──`) });
+        for (const file of subFiles) {
+            const status = getFileStatus(path.join(sourceDir, file), path.join(destDir, file));
+            choices.push({
+                name: file,
+                message: `  ${path.basename(file)}`,
+                hint: statusLabel(status),
+                enabled: status !== 'unchanged',
+            });
+        }
+    }
+    const prompt = new MultiSelect({
+        name: 'skill-files',
+        message: 'Select skill files to install',
+        choices,
+        hint: '(↑↓ navigate, <space> toggle, <a> select all, <enter> confirm)',
+    });
+    try {
+        return await prompt.run();
+    }
+    catch {
+        console.log(chalk_1.default.yellow('\nInstallation cancelled.'));
+        process.exit(0);
+    }
+}
 /**
  * Merge settings.json from source into destination.
  * Hooks are merged per event key; duplicate hook entries (by deep equality) are skipped.
@@ -293,27 +409,37 @@ async function copyClaudeFiles(options = {}) {
         console.log(chalk_1.default.yellow('No files found in templates/global directory'));
         return;
     }
-    console.log(chalk_1.default.cyan(`Found ${files.length} files to copy:`));
+    const categorized = categorizeFiles(files);
+    console.log(chalk_1.default.cyan(`Found ${files.length} files (${categorized.agents.length} agents, ${categorized.skills.length} skills, ${categorized.others.length} others)`));
     console.log();
-    // Dry run mode - just show what would be copied
+    // Dry run mode - show categorized status
     if (dryRun) {
         console.log(chalk_1.default.yellow('[DRY RUN] Files that would be copied:'));
         console.log();
-        for (const file of files) {
-            const sourcePath = path.join(sourceDir, file);
-            const destPath = path.join(destDir, file);
-            const exists = fs.existsSync(destPath);
-            if (exists) {
-                const sourceHash = getFileHash(sourcePath);
-                const destHash = getFileHash(destPath);
-                const status = sourceHash === destHash ? chalk_1.default.gray('[unchanged]') : chalk_1.default.yellow('[overwrite]');
-                console.log(`  ${status} ${file}`);
+        if (categorized.agents.length > 0) {
+            console.log(chalk_1.default.cyan('  Agents:'));
+            for (const file of categorized.agents) {
+                const status = getFileStatus(path.join(sourceDir, file), path.join(destDir, file));
+                console.log(`    ${statusBracket(status)} ${path.basename(file, '.md')}`);
             }
-            else {
-                console.log(`  ${chalk_1.default.green('[new]')} ${file}`);
+            console.log();
+        }
+        if (categorized.skills.length > 0) {
+            console.log(chalk_1.default.cyan('  Skills:'));
+            for (const skill of categorized.skills) {
+                const status = getSkillStatus(skill, sourceDir, destDir);
+                console.log(`    ${statusBracket(status)} ${skill}`);
+            }
+            console.log();
+        }
+        if (categorized.others.length > 0) {
+            console.log(chalk_1.default.cyan('  Others (auto-install):'));
+            for (const file of categorized.others) {
+                const status = getFileStatus(path.join(sourceDir, file), path.join(destDir, file));
+                console.log(`    ${statusBracket(status)} ${file}`);
             }
+            console.log();
         }
-        // settings.json merge indicator
         const sourceSettingsExists = fs.existsSync(path.join(sourceDir, 'settings.json'));
         if (sourceSettingsExists) {
             console.log(`  ${chalk_1.default.blue('[merge]')} settings.json`);
@@ -322,34 +448,80 @@ async function copyClaudeFiles(options = {}) {
         console.log(chalk_1.default.yellow('No files were copied (dry run mode)'));
         return;
     }
-    // Copy files
+    // Determine files to copy per category
+    let agentFiles = [];
+    let skillFiles = [];
+    let otherFiles = [];
+    if (force) {
+        agentFiles = categorized.agents;
+        skillFiles = files.filter(f => f.startsWith('skills/'));
+        otherFiles = categorized.others;
+    }
+    else {
+        // Others: auto-copy new/changed files
+        for (const file of categorized.others) {
+            const status = getFileStatus(path.join(sourceDir, file), path.join(destDir, file));
+            if (status !== 'unchanged') {
+                otherFiles.push(file);
+            }
+        }
+        // Agents: MultiSelect
+        if (categorized.agents.length > 0) {
+            const agentItems = categorized.agents.map(file => ({
+                name: file,
+                displayName: path.basename(file, '.md'),
+                status: getFileStatus(path.join(sourceDir, file), path.join(destDir, file)),
+            }));
+            agentFiles = await selectItems('Agents', agentItems);
+        }
+        // Skills: MultiSelect (2-step)
+        if (categorized.skills.length > 0) {
+            const skillItems = categorized.skills.map(skill => ({
+                name: skill,
+                displayName: skill,
+                status: getSkillStatus(skill, sourceDir, destDir),
+            }));
+            const selectedSkills = await selectItems('Skills', skillItems);
+            // Step 2: 선택된 스킬의 하위 파일 선택
+            const multiFileSkills = [];
+            for (const skillName of selectedSkills) {
+                const skillSubFiles = files.filter(f => f.startsWith(`skills/${skillName}/`));
+                if (skillSubFiles.length > 1) {
+                    multiFileSkills.push({ skillName, subFiles: skillSubFiles });
+                }
+                else {
+                    // 파일이 1개뿐이면 자동 포함
+                    skillFiles.push(...skillSubFiles);
+                }
+            }
+            // 하위 파일 선택이 필요한 스킬이 있으면 한 번의 프롬프트로 표시
+            if (multiFileSkills.length > 0) {
+                const selectedSubFiles = await selectSkillSubFiles(multiFileSkills, sourceDir, destDir);
+                skillFiles.push(...selectedSubFiles);
+            }
+        }
+    }
+    // Copy all selected files
+    const allFilesToCopy = [...otherFiles, ...agentFiles, ...skillFiles];
     let copiedCount = 0;
-    let skippedCount = 0;
-    for (const file of files) {
-        const sourcePath = path.join(sourceDir, file);
-        const destPath = path.join(destDir, file);
-        const exists = fs.existsSync(destPath);
-        if (exists && !force) {
-            const sourceHash = getFileHash(sourcePath);
-            const destHash = getFileHash(destPath);
-            if (sourceHash === destHash) {
+    if (otherFiles.length > 0 || categorized.others.length > 0) {
+        // Show unchanged others for context
+        for (const file of categorized.others) {
+            if (!otherFiles.includes(file)) {
                 console.log(`  ${chalk_1.default.gray('[unchanged]')} ${file}`);
-                skippedCount++;
-                continue;
-            }
-            // Hash differs - ask for confirmation
-            const shouldOverwrite = await askConfirmation(chalk_1.default.yellow(`File changed: ${file}. Overwrite? (y/N): `));
-            if (!shouldOverwrite) {
-                console.log(chalk_1.default.gray(`  Skipped: ${file}`));
-                skippedCount++;
-                continue;
             }
         }
+    }
+    for (const file of allFilesToCopy) {
+        const sourcePath = path.join(sourceDir, file);
+        const destPath = path.join(destDir, file);
+        const exists = fs.existsSync(destPath);
         copyFile(sourcePath, destPath);
-        const status = exists ? chalk_1.default.yellow('[overwritten]') : chalk_1.default.green('[created]');
-        console.log(`  ${status} ${file}`);
+        const label = exists ? chalk_1.default.yellow('[overwritten]') : chalk_1.default.green('[created]');
+        console.log(`  ${label} ${file}`);
         copiedCount++;
     }
+    const skippedCount = files.length - copiedCount;
     // Merge settings.json (hooks are merged, not overwritten)
     mergeSettingsJson(sourceDir, destDir, { project });
     console.log();

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "jun-claude-code",
-  "version": "0.6.0",
+  "version": "0.6.2",
   "description": "Claude Code configuration template - copy .claude settings to your project",
   "main": "dist/index.js",
   "bin": "dist/cli.js",
@@ -16,7 +16,8 @@
   "packageManager": "yarn@3.8.7",
   "dependencies": {
     "chalk": "^4.1.2",
-    "commander": "^11.1.0"
+    "commander": "^11.1.0",
+    "enquirer": "^2.4.1"
   },
   "devDependencies": {
     "@types/node": "^20.10.0",

package/templates/global/agents/code-reviewer.md

@@ -1,7 +1,7 @@
 ---
 name: code-reviewer
-description: 코드 작성 완료 후 품질 검토 시 호출. CLAUDE.md/Skills 규칙 준수 확인, Critical/Warning 분류, lint 실행 및 수정 제안.
-keywords: [코드리뷰, 체크리스트, lint, 규칙검증, 품질검사, Critical, Warning, 수정제안]
+description: 코드 품질 검토 및 GitHub PR line-level comment 게시. CLAUDE.md/Skills 규칙 준수 확인, Critical/Warning 분류, lint 실행, PR 리뷰 코멘트 작성.
+keywords: [코드리뷰, 체크리스트, lint, 규칙검증, 품질검사, Critical, Warning, 수정제안, PR리뷰, GitHub, comment, 라인코멘트]
 model: opus
 color: yellow
 disallowedTools: [Edit, Write, NotebookEdit]
@@ -13,12 +13,13 @@ memory: project
 
 <role>
 
-작성된 코드가 프로젝트 규칙을 준수하는지 검토하는 전문 Agent입니다.
+작성된 코드가 프로젝트 규칙을 준수하는지 검토하고, PR 리뷰 시 GitHub에 line-level comment를 게시하는 전문 Agent입니다.
 
 1. **규칙 준수 확인**: CLAUDE.md, 프로젝트 체크리스트 기준 검토
 2. **코드 품질 검사**: 가독성, 유지보수성, 일관성
 3. **Lint 실행**: lint 실행 및 결과 확인
 4. **개선 제안**: 발견된 문제에 대한 수정 방안 제시
+5. **PR 리뷰 코멘트**: GitHub PR에 파일/라인별 review comment 게시
 
 </role>
 
@@ -28,7 +29,16 @@ memory: project
 
 ## 검토 프로세스
 
-### Step 1: 공통 체크리스트
+### Step 1: 리뷰 모드 판별
+
+호출 시 전달받은 정보로 리뷰 모드를 결정합니다.
+
+| 조건 | 모드 | 동작 |
+|------|------|------|
+| PR 번호가 주어짐 | **PR 리뷰 모드** | diff 분석 → 코드 검토 → GitHub PR에 line-level comment 게시 |
+| PR 번호 없음 | **로컬 리뷰 모드** | 코드 검토 → 터미널에 결과 출력 |
+
+### Step 2: 공통 체크리스트
 
 | 항목 | 확인 |
 |------|------|
@@ -39,11 +49,11 @@ memory: project
 | 불필요한 코드 없음 | ☐ |
 | 네이밍 컨벤션 준수 | ☐ |
 
-### Step 2: 프로젝트별 체크리스트
+### Step 3: 프로젝트별 체크리스트
 
 프로젝트의 `.claude/skills/` 에 정의된 체크리스트 확인
 
-### Step 3: Lint 실행
+### Step 4: Lint 실행
 
 ```bash
 # 프로젝트에 맞는 lint 명령어 실행
@@ -52,6 +62,130 @@ npm run lint
 yarn lint
 ```
 
+### Step 5: PR 리뷰 코멘트 게시 (PR 리뷰 모드 전용)
+
+PR 리뷰 모드에서는 발견된 이슈를 GitHub PR에 line-level review comment로 게시합니다.
+
+#### 5-1. PR diff 수집
+
+```bash
+# PR의 변경된 파일과 diff 확인
+gh pr diff {PR_NUMBER}
+```
+
+#### 5-2. 이슈별 comment 데이터 구성
+
+각 이슈를 아래 형식으로 수집합니다:
+
+```json
+{
+  "path": "src/example.ts",
+  "line": 42,
+  "body": "**[Critical]** 설명\n\n수정 방안: ..."
+}
+```
+
+- `path`: 리포지토리 루트 기준 상대 경로
+- `line`: diff에서 변경된 라인 번호 (새 파일 기준)
+- `body`: Markdown 형식의 코멘트 본문
+
+**여러 라인에 걸친 이슈**는 `start_line`과 `line`을 함께 사용합니다:
+
+```json
+{
+  "path": "src/example.ts",
+  "start_line": 10,
+  "line": 15,
+  "body": "**[Warning]** 설명"
+}
+```
+
+#### 5-3. comment body 작성 규칙
+
+심각도에 따라 접두사를 붙입니다:
+
+| 심각도 | 접두사 | 예시 |
+|--------|--------|------|
+| Critical | `🔴 **[Critical]**` | `🔴 **[Critical]** any 타입 사용 — 구체적 타입으로 변경 필요` |
+| Warning | `🟡 **[Warning]**` | `🟡 **[Warning]** 매직 넘버 사용 — 상수로 추출 권장` |
+| Info | `🔵 **[Info]**` | `🔵 **[Info]** Optional chaining으로 간소화 가능` |
+
+본문 구성:
+
+```markdown
+{접두사} 이슈 제목
+
+**문제**: 구체적인 문제 설명
+**수정 방안**:
+\`\`\`typescript
+// 수정 예시 코드
+\`\`\`
+```
+
+#### 5-4. GitHub PR Review Comment 게시
+
+수집한 comment들을 개별 review comment로 게시합니다.
+
+```bash
+# owner/repo 확인
+gh repo view --json nameWithOwner -q '.nameWithOwner'
+
+# 최신 commit SHA 확인
+gh pr view {PR_NUMBER} --json headRefOid -q '.headRefOid'
+```
+
+**단일 라인 comment 게시:**
+
+```bash
+gh api repos/{owner}/{repo}/pulls/{PR_NUMBER}/comments \
+  --method POST \
+  -f path="src/example.ts" \
+  -F line=42 \
+  -f side="RIGHT" \
+  -f commit_id="{COMMIT_SHA}" \
+  -f body="코멘트 내용"
+```
+
+**여러 라인 comment 게시:**
+
+```bash
+gh api repos/{owner}/{repo}/pulls/{PR_NUMBER}/comments \
+  --method POST \
+  -f path="src/example.ts" \
+  -F start_line=10 \
+  -F line=15 \
+  -f start_side="RIGHT" \
+  -f side="RIGHT" \
+  -f commit_id="{COMMIT_SHA}" \
+  -f body="코멘트 내용"
+```
+
+#### 5-5. 전체 요약 comment 게시
+
+모든 line comment 게시 후, PR에 전체 요약을 일반 comment로 남깁니다.
+
+```bash
+gh pr comment {PR_NUMBER} --body "$(cat <<'EOF'
+# Code Review 결과 요약
+
+- **검토 파일**: N개
+- **발견 이슈**: Critical N개, Warning N개, Info N개
+- **전체 평가**: 통과 / 수정 필요 / 재작업 필요
+
+## diff 범위 밖 이슈
+- `path/to/file.ts:100` - 설명 (해당되는 경우만)
+
+## 다음 단계
+- ...
+EOF
+)"
+```
+
+#### 5-6. 주의사항
+
+- **diff에 포함된 라인만 comment 가능**: 변경되지 않은 라인에는 comment를 달 수 없음 → diff 범위 밖 이슈는 요약 comment에 포함
+- **이슈가 없으면 요약 comment만 게시**: line comment 없이 "이슈 없음" 요약만 남김
+
 </instructions>
 
 ---
@@ -70,6 +204,8 @@ yarn lint
 
 <output_format>
 
+## 로컬 리뷰 모드 출력
+
 ```markdown
 # Code Review 결과
 
@@ -116,6 +252,32 @@ yarn lint
 - 이슈 없음: 최종 검증 진행
 ```
 
+## PR 리뷰 모드 출력
+
+```markdown
+# PR Review 완료
+
+## 1. 요약
+- **PR**: #{PR_NUMBER}
+- **검토 파일**: N개
+- **게시된 comment**: N개 (Critical N, Warning N, Info N)
+
+## 2. 게시된 comment 목록
+
+| 파일 | 라인 | 심각도 | 이슈 |
+|------|------|--------|------|
+| `src/example.ts` | L42 | Critical | 설명 |
+| `src/util.ts` | L10-15 | Warning | 설명 |
+
+## 3. diff 범위 밖 이슈 (요약 comment에 포함)
+- `path/to/file.ts:100` - 설명
+
+## 4. Lint 결과
+```
+[lint 출력]
+```
+```
+
 </output_format>
 
 ---
@@ -125,5 +287,6 @@ yarn lint
 - **실제 문제만 지적**: 코드에 실질적 영향이 있는 부분만 리뷰
 - **대안 제시 필수**: 문제 지적 시 해결책도 함께 제시
 - **컨텍스트 고려**: 프로젝트 상황에 맞게 유연하게 판단
+- **diff 범위 준수**: PR comment는 diff에 포함된 라인에만 게시 (범위 밖 이슈는 요약 comment에 기재)
 
 </constraints>

package/templates/global/skills/Git/git.md

@@ -144,13 +144,15 @@ grep -r "import.*변경된함수명" --include="*.ts" --include="*.tsx"
 
 ## 🔄 주요 변경사항
 
-### [변경 제목 1]
-**파일:** `path/to/file.ts`
+> 파일별이 아닌 **기능/목적 단위**로 묶어서 설명합니다.
+
+### [기능/변경 목적 1]
 - 변경 내용 설명
+- 관련 파일: `file1.ts`, `file2.ts`
 
-### [변경 제목 2]
-**파일:** `path/to/file.ts`
+### [기능/변경 목적 2]
 - 변경 내용 설명
+- 관련 파일: `file3.ts`
 
 ## ⚠️ 사이드 이펙트