jun-claude-code 0.1.0 → 0.1.1

package/dist/__tests__/merge-settings.test.js

@@ -149,7 +149,7 @@ function writeJson(filePath, data) {
         writeJson(path.join(sourceDir, 'settings.json'), sourceSettings);
         (0, copy_1.mergeSettingsJson)(sourceDir, destDir);
         const result = readJson(path.join(destDir, 'settings.json'));
-        (0, vitest_1.expect)(result.statusLine).toEqual(sourceSettings.statusLine);
+        (0, vitest_1.expect)(result.statusLine).toBeUndefined();
         (0, vitest_1.expect)(result.hooks.UserPromptSubmit).toHaveLength(1);
         (0, vitest_1.expect)(result.hooks.UserPromptSubmit[0].hooks[0].command).toBe('skill-forced.sh');
     });
@@ -268,6 +268,73 @@ function writeJson(filePath, data) {
         const result = readJson(path.join(destDir, 'settings.json'));
         (0, vitest_1.expect)(result).toEqual(destSettings);
     });
+    (0, vitest_1.it)('should exclude statusLine from source even on fresh install', () => {
+        const sourceSettings = {
+            statusLine: { type: 'command', command: '~/.claude/statusline-command.sh' },
+            hooks: {},
+        };
+        writeJson(path.join(sourceDir, 'settings.json'), sourceSettings);
+        (0, copy_1.mergeSettingsJson)(sourceDir, destDir);
+        const result = readJson(path.join(destDir, 'settings.json'));
+        (0, vitest_1.expect)(result.statusLine).toBeUndefined();
+    });
+    (0, vitest_1.it)('should preserve existing statusLine in dest when source also has statusLine', () => {
+        const sourceSettings = {
+            statusLine: { type: 'command', command: 'source-status' },
+            hooks: {},
+        };
+        writeJson(path.join(sourceDir, 'settings.json'), sourceSettings);
+        const destSettings = {
+            statusLine: { type: 'command', command: 'dest-status' },
+        };
+        writeJson(path.join(destDir, 'settings.json'), destSettings);
+        (0, copy_1.mergeSettingsJson)(sourceDir, destDir);
+        const result = readJson(path.join(destDir, 'settings.json'));
+        (0, vitest_1.expect)(result.statusLine.command).toBe('dest-status');
+    });
+    (0, vitest_1.it)('should convert ~/.claude/ to ./.claude/ in command fields when project=true', () => {
+        const sourceSettings = {
+            hooks: {
+                UserPromptSubmit: [
+                    {
+                        hooks: [
+                            { type: 'command', command: '~/.claude/hooks/skill-forced.sh' },
+                        ],
+                    },
+                ],
+                PreToolUse: [
+                    {
+                        matcher: 'Bash',
+                        hooks: [
+                            { type: 'command', command: '~/.claude/hooks/blocker.sh' },
+                        ],
+                    },
+                ],
+            },
+        };
+        writeJson(path.join(sourceDir, 'settings.json'), sourceSettings);
+        (0, copy_1.mergeSettingsJson)(sourceDir, destDir, { project: true });
+        const result = readJson(path.join(destDir, 'settings.json'));
+        (0, vitest_1.expect)(result.hooks.UserPromptSubmit[0].hooks[0].command).toBe('./.claude/hooks/skill-forced.sh');
+        (0, vitest_1.expect)(result.hooks.PreToolUse[0].hooks[0].command).toBe('./.claude/hooks/blocker.sh');
+    });
+    (0, vitest_1.it)('should keep ~/.claude/ paths unchanged when project option is not set', () => {
+        const sourceSettings = {
+            hooks: {
+                UserPromptSubmit: [
+                    {
+                        hooks: [
+                            { type: 'command', command: '~/.claude/hooks/skill-forced.sh' },
+                        ],
+                    },
+                ],
+            },
+        };
+        writeJson(path.join(sourceDir, 'settings.json'), sourceSettings);
+        (0, copy_1.mergeSettingsJson)(sourceDir, destDir);
+        const result = readJson(path.join(destDir, 'settings.json'));
+        (0, vitest_1.expect)(result.hooks.UserPromptSubmit[0].hooks[0].command).toBe('~/.claude/hooks/skill-forced.sh');
+    });
 });
 // ─── init-project.ts mergeSettingsJson (project) ───
 (0, vitest_1.describe)('init-project.ts mergeSettingsJson', () => {

package/dist/copy.d.ts

@@ -7,8 +7,12 @@ export interface CopyOptions {
  * Merge settings.json from source into destination.
  * Hooks are merged per event key; duplicate hook entries (by deep equality) are skipped.
  * Non-hook keys are shallow-merged (source wins for new keys, dest preserved for existing).
+ * statusLine is always excluded from merge (personal environment setting).
+ * When project=true, ~/.claude/ paths in command fields are converted to ./.claude/.
  */
-export declare function mergeSettingsJson(sourceDir: string, destDir: string): void;
+export declare function mergeSettingsJson(sourceDir: string, destDir: string, options?: {
+    project?: boolean;
+}): void;
 /**
  * Copy .claude files to user's home directory
  */

package/dist/copy.js

@@ -125,7 +125,35 @@ function getDestClaudeDir() {
  * Hooks are merged per event key; duplicate hook entries (by deep equality) are skipped.
  * Non-hook keys are shallow-merged (source wins for new keys, dest preserved for existing).
  */
-function mergeSettingsJson(sourceDir, destDir) {
+/**
+ * Replace ~/.claude/ paths with ./.claude/ in all command fields (deep traverse).
+ */
+function replaceClaudePaths(obj) {
+    const result = {};
+    for (const [key, value] of Object.entries(obj)) {
+        if (key === 'command' && typeof value === 'string') {
+            result[key] = value.replace(/~\/\.claude\//g, './.claude/');
+        }
+        else if (Array.isArray(value)) {
+            result[key] = value.map((item) => typeof item === 'object' && item !== null ? replaceClaudePaths(item) : item);
+        }
+        else if (typeof value === 'object' && value !== null) {
+            result[key] = replaceClaudePaths(value);
+        }
+        else {
+            result[key] = value;
+        }
+    }
+    return result;
+}
+/**
+ * Merge settings.json from source into destination.
+ * Hooks are merged per event key; duplicate hook entries (by deep equality) are skipped.
+ * Non-hook keys are shallow-merged (source wins for new keys, dest preserved for existing).
+ * statusLine is always excluded from merge (personal environment setting).
+ * When project=true, ~/.claude/ paths in command fields are converted to ./.claude/.
+ */
+function mergeSettingsJson(sourceDir, destDir, options) {
     const sourcePath = path.join(sourceDir, 'settings.json');
     const destPath = path.join(destDir, 'settings.json');
     if (!fs.existsSync(sourcePath)) {
@@ -151,8 +179,8 @@ function mergeSettingsJson(sourceDir, destDir) {
     }
     // Merge top-level keys (source fills in missing keys, dest's existing keys preserved)
     for (const key of Object.keys(sourceSettings)) {
-        if (key === 'hooks') {
-            continue; // hooks are merged separately below
+        if (key === 'hooks' || key === 'statusLine') {
+            continue; // hooks are merged separately below; statusLine is excluded
         }
         if (!(key in destSettings)) {
             destSettings[key] = sourceSettings[key];
@@ -180,6 +208,10 @@ function mergeSettingsJson(sourceDir, destDir) {
             }
         }
     }
+    // Convert ~/.claude/ → ./.claude/ paths for project mode
+    if (options?.project) {
+        destSettings = replaceClaudePaths(destSettings);
+    }
     ensureDir(destDir);
     fs.writeFileSync(destPath, JSON.stringify(destSettings, null, 2) + '\n', 'utf-8');
     console.log(`  ${chalk_1.default.blue('[merged]')} settings.json`);
@@ -203,6 +235,7 @@ async function copyClaudeFiles(options = {}) {
     // Files to exclude from global copy (merge-handled separately)
     const EXCLUDE_FROM_GLOBAL = [
         'settings.json',
+        'statusline-command.sh',
     ];
     // Get all files to copy
     const allFiles = getAllFiles(sourceDir);
@@ -271,7 +304,7 @@ async function copyClaudeFiles(options = {}) {
         copiedCount++;
     }
     // Merge settings.json (hooks are merged, not overwritten)
-    mergeSettingsJson(sourceDir, destDir);
+    mergeSettingsJson(sourceDir, destDir, { project });
     console.log();
     console.log(chalk_1.default.green(`Done! Copied ${copiedCount} files, skipped ${skippedCount} files.`));
 }

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "jun-claude-code",
-  "version": "0.1.0",
+  "version": "0.1.1",
   "description": "Claude Code configuration template - copy .claude settings to your project",
   "main": "dist/index.js",
   "bin": "dist/cli.js",

package/templates/global/agents/git-manager.md

@@ -100,25 +100,63 @@ git log --oneline -10
 git diff main...HEAD
 ```
 
-### Step 2: 변경사항 분석
+### Step 2: PR 템플릿 확인
+
+```bash
+# 프로젝트에 PR 템플릿이 있는지 확인
+# 확인 경로 (우선순위 순):
+#   1. .github/pull_request_template.md
+#   2. .github/PULL_REQUEST_TEMPLATE.md
+#   3. docs/pull_request_template.md
+#   4. pull_request_template.md
+```
+
+- **템플릿이 있으면**: 해당 템플릿의 섹션 구조를 그대로 따라 PR 본문 작성
+- **템플릿이 없으면**: 아래 Step 4의 기본 포맷 사용
+
+### Step 3: 변경사항 분석
 
 모든 커밋을 분석하여 PR 내용 구성:
 - 변경된 파일 목록
 - 각 파일의 변경 내용
 - 전체 변경의 목적
 
-### Step 3: PR 생성
+### PR 본문 필수 포함 사항
+
+**템플릿 사용 여부와 관계없이**, PR 본문에는 반드시 다음 3가지를 포함:
+
+| 항목 | 설명 | 예시 |
+|------|------|------|
+| **의도 (Why)** | 이 변경을 하는 이유/배경 | "사용자 로그인 실패 시 에러 메시지가 표시되지 않는 문제" |
+| **문제 (What)** | 해결하려는 구체적 문제 | "catch 블록에서 에러를 무시하고 있었음" |
+| **해결 방법 (How)** | 어떻게 해결했는지 | "에러 핸들러를 추가하고 toast 알림으로 사용자에게 표시" |
+
+### Step 4: PR 생성
 
 ```bash
 # 원격에 push (필요시)
 git push -u origin <branch-name>
 
-# PR 생성
+# PR 템플릿이 있는 경우: 템플릿 구조를 따르되 Why/What/How 포함
+# PR 템플릿이 없는 경우: 아래 기본 포맷 사용
+
 gh pr create --title "<간결한 제목>" --body "$(cat <<'EOF'
 ## 📋 Summary
 
 > 이 PR이 해결하는 문제와 접근 방식을 1-2문장으로 설명
 
+## 🎯 Why (의도)
+
+> 왜 이 변경이 필요한가? 배경과 동기를 설명
+
+## 🐛 What (문제)
+
+> 어떤 문제가 있었는가? 구체적인 증상이나 요구사항
+
+## 🔧 How (해결 방법)
+
+> 어떻게 해결했는가? 접근 방식과 핵심 변경 내용
+
 ## 🔄 주요 변경사항
 
 ### 변경 1: [제목]

package/templates/global/skills/Backend/SKILL.md

@@ -1,7 +1,7 @@
 ---
 name: Backend
-description: NestJS/TypeORM 백엔드 개발 시 사용. Controller-Service-Repository 레이어 객체 변환, find vs queryBuilder 선택 기준 제공.
-keywords: [Backend, 백엔드, 레이어, DTO, Entity, Service, Controller, TypeORM, find, queryBuilder]
+description: NestJS/TypeORM 백엔드 개발 시 사용. 레이어 객체 변환, find vs queryBuilder 선택 기준, BDD 테스트 작성 규칙 제공.
+keywords: [Backend, 백엔드, 레이어, DTO, Entity, Service, Controller, TypeORM, find, queryBuilder, test, BDD, 테스트, Jest]
 estimated_tokens: ~400
 ---
 
@@ -174,3 +174,13 @@ const user = await this.userRepository.findOneBy({ id });
 - [ ] QueryBuilder는 groupBy, getRawMany 등 필요한 경우에만 사용하는가?
 
 </checklist>
+
+<reference>
+
+## 관련 문서
+
+| 주제 | 위치 | 설명 |
+|-----|------|------|
+| BDD 테스트 | `bdd-testing.md` | NestJS + Jest BDD 스타일 테스트 작성 규칙 |
+
+</reference>

package/templates/global/skills/Backend/bdd-testing.md

@@ -0,0 +1,211 @@
+---
+name: bdd-testing
+description: NestJS + Jest BDD 스타일 테스트 작성 시 사용. describe 네스팅 구조, Given/When/Then 패턴, Testing Module Mock 설정 규칙 제공.
+keywords: [BDD, test, 테스트, Jest, NestJS, describe, it, Given, When, Then, mock, TestingModule]
+estimated_tokens: ~500
+---
+
+# BDD 테스트 작성 규칙
+
+<rules>
+
+## describe 네스팅 구조
+
+> **Service 클래스명 > 메서드명 > 시나리오** 구조로 describe를 네스팅한다.
+
+| 레벨 | 대상 | 예시 |
+|------|------|------|
+| 1 | Service 클래스 | `describe('UserService', ...)` |
+| 2 | 메서드명 | `describe('create', ...)` |
+| 3 (선택) | 조건 그룹 | `describe('when role is admin', ...)` |
+
+- 3레벨은 동일 메서드에서 조건 분기가 많을 때만 사용한다
+- 3레벨까지만 사용한다
+
+```typescript
+describe('UserService', () => {
+  // 공통 setup (TestingModule, mock 등)
+
+  describe('create', () => {
+    it('should create user when valid dto given', () => {});
+    it('should throw ConflictException when email duplicated', () => {});
+  });
+
+  describe('findById', () => {
+    it('should return user when id exists', () => {});
+    it('should throw NotFoundException when id not found', () => {});
+  });
+});
+```
+
+## it 문구 및 Given/When/Then
+
+### it 문구 형식
+
+```
+"should [결과] when [조건]"
+```
+
+| 요소 | 설명 | 예시 |
+|------|------|------|
+| 결과 | 기대하는 동작 | `create user`, `throw NotFoundException` |
+| 조건 | 입력/상태 | `valid dto given`, `id not found` |
+
+### 블록 내부 구조
+
+각 `it` 블록 내부에 `// Given` / `// When` / `// Then` 주석으로 구분한다.
+
+```typescript
+it('should create user when valid dto given', async () => {
+  // Given
+  const dto: CreateUserDto = { name: 'John', email: 'john@test.com' };
+  jest.spyOn(userRepository, 'findOneBy').mockResolvedValue(null);
+  jest.spyOn(userRepository, 'save').mockResolvedValue({ id: 1, ...dto } as User);
+
+  // When
+  const result = await service.create(dto);
+
+  // Then
+  expect(result.name).toBe('John');
+  expect(userRepository.save).toHaveBeenCalledWith(expect.objectContaining({ name: 'John' }));
+});
+```
+
+## NestJS Testing Module 설정
+
+### 기본 패턴
+
+```typescript
+import { Test, TestingModule } from '@nestjs/testing';
+
+describe('UserService', () => {
+  let service: UserService;
+  let userRepository: Repository<User>;
+
+  beforeEach(async () => {
+    const module: TestingModule = await Test.createTestingModule({
+      providers: [
+        UserService,
+        {
+          provide: getRepositoryToken(User),
+          useValue: {
+            find: jest.fn(),
+            findOneBy: jest.fn(),
+            save: jest.fn(),
+            delete: jest.fn(),
+          },
+        },
+      ],
+    }).compile();
+
+    service = module.get<UserService>(UserService);
+    userRepository = module.get<Repository<User>>(getRepositoryToken(User));
+  });
+});
+```
+
+### Mock 규칙
+
+| 패턴 | 용도 | 예시 |
+|------|------|------|
+| `useValue` + `jest.fn()` | Repository mock | `{ save: jest.fn() }` |
+| `jest.spyOn` | 특정 호출에 반환값 지정 | `jest.spyOn(repo, 'findOneBy').mockResolvedValue(...)` |
+| `jest.mocked` | 이미 mock된 함수 타입 캐스팅 | `jest.mocked(repo.save).mockResolvedValue(...)` |
+
+- `useValue`로 mock 객체를 주입하고, 각 테스트에서 `jest.spyOn` 또는 `jest.mocked`로 반환값을 지정한다
+- `jest.mock()` 모듈 레벨 mock은 외부 라이브러리에만 사용한다
+
+### 외부 의존성 Mock
+
+```typescript
+// Service가 다른 Service에 의존할 때
+{
+  provide: EmailService,
+  useValue: {
+    sendWelcomeEmail: jest.fn().mockResolvedValue(undefined),
+  },
+},
+```
+
+</rules>
+
+<examples>
+
+### describe 구조
+
+<example type="good">
+```typescript
+// Service > Method > Scenario 명확
+describe('OrderService', () => {
+  describe('cancel', () => {
+    it('should cancel order when status is pending', async () => {});
+    it('should throw BadRequestException when status is shipped', async () => {});
+  });
+});
+```
+</example>
+<example type="bad">
+```typescript
+// 메서드별 그룹 없이 플랫하게 나열
+describe('OrderService', () => {
+  it('cancel pending order', async () => {});
+  it('cancel shipped order throws', async () => {});
+  it('create order', async () => {});
+});
+```
+</example>
+
+### Assertion
+
+<example type="good">
+```typescript
+// 구체적 assertion
+expect(result.status).toBe('cancelled');
+expect(orderRepository.save).toHaveBeenCalledWith(
+  expect.objectContaining({ status: 'cancelled' }),
+);
+```
+</example>
+<example type="bad">
+```typescript
+// 의미 없는 assertion
+expect(result).toBeDefined();
+expect(result).toBeTruthy();
+```
+</example>
+
+### Exception 테스트
+
+<example type="good">
+```typescript
+// 예외 타입 검증
+await expect(service.cancel(orderId)).rejects.toThrow(BadRequestException);
+```
+</example>
+<example type="bad">
+```typescript
+// try-catch로 예외 테스트
+try {
+  await service.cancel(orderId);
+  fail('should have thrown');
+} catch (e) {
+  expect(e).toBeDefined();
+}
+```
+</example>
+
+</examples>
+
+<checklist>
+
+## 체크리스트
+
+- [ ] describe 네스팅이 Service > Method > Scenario 구조인가?
+- [ ] it 문구가 `"should [결과] when [조건]"` 형식인가?
+- [ ] 각 it 블록에 `// Given` / `// When` / `// Then` 주석이 있는가?
+- [ ] `Test.createTestingModule`로 테스트 모듈을 구성했는가?
+- [ ] Repository mock은 `useValue` + `jest.fn()`으로 주입했는가?
+- [ ] assertion이 구체적인 값을 검증하는가?
+- [ ] 예외 테스트에 `rejects.toThrow`를 사용했는가?
+
+</checklist>

package/templates/project/workflows/context-gen.yml

@@ -75,8 +75,12 @@ jobs:
             exit 0
           fi
 
+          SOURCE_PR_TITLE="${{ github.event.pull_request.title }}"
+          CONTEXT_PR_TITLE="${SOURCE_PR_TITLE} - Docs"
+          PR_NUMBER="${{ github.event.pull_request.number }}"
+
           CONTEXT_BRANCH="${{ github.head_ref }}-generated-context"
-          git commit -m "docs: context 문서 자동 업데이트"
+          git commit -m "${CONTEXT_PR_TITLE}"
           git push --force origin HEAD:refs/heads/${CONTEXT_BRANCH}
 
           # 기존 PR이 있으면 스킵, 없으면 생성
@@ -85,11 +89,11 @@ jobs:
             gh pr create \
               --head "${CONTEXT_BRANCH}" \
               --base "${{ github.head_ref }}" \
-              --title "docs: context 문서 자동 업데이트" \
-              --body "$(cat <<'PREOF'
+              --title "${CONTEXT_PR_TITLE}" \
+              --body "$(cat <<PREOF
           ## Auto-generated Context
 
-          PR의 코드 변경을 분석하여 자동 생성된 context 문서입니다.
+          #${PR_NUMBER} 의 코드 변경을 분석하여 자동 생성된 context 문서입니다.
           선택적으로 머지하여 반영할 수 있습니다.
 
           > 이 PR은 re-sync 시 force push로 업데이트됩니다.