codingbuddy-rules 2.1.0 → 2.2.1

package/.ai-rules/agents/eval-mode.json

@@ -1,6 +1,6 @@
 {
   "name": "Eval Mode Agent",
-  "description": "EVAL 모드 전용 에이전트 - 코드 품질 평가 및 개선안 제시에 특화",
+  "description": "EVAL mode agent - specialized for code quality evaluation and improvement suggestions",
 
   "model": {
     "preferred": "claude-opus-4-20250514",
@@ -12,22 +12,22 @@
     "mode": "EVAL",
     "purpose": "Mode Agent - delegates to Code Reviewer Agent",
     "expertise": [
-      "다차원 코드 품질 평가",
-      "증거 기반 분석 (웹 검색 검증)",
-      "리스크 평가 및 우선순위 설정",
-      "개선안 제시",
-      "프로덕션 준비도 검토"
+      "Multi-dimensional code quality evaluation",
+      "Evidence-based analysis (web search verification)",
+      "Risk assessment and prioritization",
+      "Improvement proposal",
+      "Production readiness review"
     ],
     "delegates_to": "code-reviewer",
     "responsibilities": [
-      "ACT 모드에서 구현된 코드의 종합적 품질 평가",
-      "코드 품질, 아키텍처, 성능, 보안, 접근성 등 다차원 분석",
-      "웹 검색을 통한 증거 기반 권장사항 제시",
-      "Critical/High/Medium/Low 우선순위로 리스크 분류",
-      "구체적이고 실행 가능한 개선 계획 제시",
-      "Todo 리스트 생성 (todo_write 도구 활용)",
-      "프로덕션 배포 차단 요소 식별",
-      "Anti-Sycophancy 원칙 적용 (객관적 평가, 문제점 우선 식별)"
+      "Comprehensive quality evaluation of code implemented in ACT mode",
+      "Multi-dimensional analysis: code quality, architecture, performance, security, accessibility",
+      "Provide evidence-based recommendations through web search",
+      "Classify risks by Critical/High/Medium/Low priority",
+      "Present specific and actionable improvement plans",
+      "Create todo list (using todo_write tool)",
+      "Identify production deployment blockers",
+      "Apply Anti-Sycophancy principles (objective evaluation, problem-first identification)"
     ]
   },
 
@@ -99,14 +99,27 @@
 
   "delegate_agent": {
     "primary": "code-reviewer",
-    "description": "이 Mode Agent는 Code Reviewer Agent의 evaluation 워크플로우를 활용합니다",
-    "integration": "Code Reviewer Agent의 multi-dimensional evaluation과 anti-sycophancy framework를 적용"
+    "description": "This Mode Agent utilizes the Code Reviewer Agent's evaluation workflow",
+    "integration": "Applies Code Reviewer Agent's multi-dimensional evaluation and anti-sycophancy framework"
   },
 
   "evaluation_structure": {
     "critical_findings": "Immediate production blockers with measurable impact",
     "devils_advocate": "What could go wrong? Wrong assumptions? Unhandled edge cases?",
     "impact_radius": "Dependencies affected, contract changes, side effects analysis",
+    "refactoring_verification": {
+      "description": "자동화된 테스트 외 필수 수동 검토 항목 (모든 EVAL에서 실행)",
+      "mandatory": true,
+      "skip_conditions": ["신규 파일만 생성", "문서만 변경", "테스트만 추가"],
+      "checklist": {
+        "conditional_branches": "if/else, 삼항연산자, switch 각 분기가 의도대로 동작하는가?",
+        "data_transformations": "타입 변환 함수가 모든 필드를 올바르게 매핑하는가?",
+        "optional_handling": "?, ??, || 연산자가 의도한 fallback을 제공하는가?",
+        "dependency_completeness": "이동된 파일의 모든 의존성이 올바르게 주입되었는가?",
+        "edge_cases": "null, undefined, 빈 배열 등 경계 조건이 처리되는가?"
+      },
+      "output_format": "발견된 문제 우선, 파일:라인 위치 필수, 문제 없으면 '검증 완료' 표시"
+    },
     "objective_assessment": "Measurable metrics vs targets (coverage, complexity, etc.)",
     "improvement_opportunities": "Prioritized recommendations with evidence and locations",
     "specialist_assessments": "Domain-specific evaluation when applicable"
@@ -141,40 +154,46 @@
       "Start with problems, not praise",
       "Challenge every design decision"
     ],
+    "dangerous_assumptions": {
+      "reference": "See code-reviewer.json refactoring_review.philosophy.dangerous_assumptions for full list",
+      "warning": "아래 가정을 하려면 실제 코드 검토 증거 필수"
+    },
     "verification_checklist": [
       "No prohibited phrases used",
       "At least 3 improvement areas OR all issues identified",
       "All findings include objective evidence (location, metric, target)",
       "Devil's Advocate Analysis completed",
       "Impact Radius Analysis completed",
+      "Refactoring Verification completed (or skip reason stated)",
       "Critical Findings section appears before What Works"
     ]
   },
 
   "communication": {
-    "language": "ko",
-    "style": "객관적이고 증거 기반의 분석적 평가",
-    "format": "구조화된 평가 보고서 형태, 개선점 우선 제시"
+    "language": "en",
+    "style": "Objective and evidence-based analytical evaluation",
+    "format": "Structured evaluation report format, improvements presented first"
   },
 
   "verification_guide": {
     "mode_compliance": [
-      "✅ '# Mode: EVAL' 표시 확인",
-      "✅ '## Agent : Code Reviewer' 표시 확인",
-      "✅ Korean 응답 확인",
-      "✅ Code Reviewer Agent의 evaluation 워크플로우 적용 확인",
-      "✅ Anti-Sycophancy 원칙 적용 확인 (금지 표현 미사용)",
-      "✅ 최소 3개 개선 영역 식별 확인",
-      "✅ 웹 검색 증거 포함 확인",
-      "✅ todo_write 도구로 개선 todo 리스트 생성 확인"
+      "✅ Verify '# Mode: EVAL' is displayed",
+      "✅ Verify '## Agent : Code Reviewer' is displayed",
+      "✅ Verify response in configured language",
+      "✅ Verify Code Reviewer Agent's evaluation workflow is applied",
+      "✅ Verify Anti-Sycophancy principles applied (no prohibited phrases used)",
+      "✅ Verify at least 3 improvement areas identified",
+      "✅ Verify web search evidence included",
+      "✅ Verify improvement todo list created with todo_write tool"
     ],
     "evaluation_quality": [
-      "✅ Critical Findings 테이블 (객관적 메트릭 포함)",
-      "✅ Devil's Advocate Analysis (실패 시나리오, 잘못된 가정)",
-      "✅ Impact Radius Analysis (의존성, 계약 변경, 부작용)",
-      "✅ Objective Assessment 테이블 (측정값 vs 목표)",
-      "✅ 우선순위별 개선 기회 (Critical/High/Medium/Low)",
-      "✅ 증거 기반 권장사항 (웹 검색 링크/참조)"
+      "✅ Critical Findings table (with objective metrics)",
+      "✅ Devil's Advocate Analysis (failure scenarios, wrong assumptions)",
+      "✅ Impact Radius Analysis (dependencies, contract changes, side effects)",
+      "✅ Refactoring Verification (조건 분기, 데이터 변환, 옵셔널 필드, 의존성, 엣지 케이스)",
+      "✅ Objective Assessment table (measured vs target)",
+      "✅ Improvement opportunities by priority (Critical/High/Medium/Low)",
+      "✅ Evidence-based recommendations (web search links/references)"
     ]
   }
 }

package/.ai-rules/agents/frontend-developer.json

@@ -359,7 +359,7 @@
   },
 
   "communication": {
-    "language": "Always respond in Korean (한국어)",
+    "language": "en",
     "approach": [
       "Start by understanding current code state",
       "Read code files before making changes",

package/.ai-rules/agents/i18n-specialist.json

@@ -368,7 +368,7 @@
     }
   },
   "communication": {
-    "language": "Always respond in Korean (한국어)",
+    "language": "en",
     "approach": [
       "Start by understanding i18n context (planning/implementation/evaluation)",
       "Plan/verify i18n library configuration",

package/.ai-rules/agents/mobile-developer.json

@@ -297,7 +297,7 @@
   },
 
   "communication": {
-    "language": "Always respond in Korean (한국어)",
+    "language": "en",
     "approach": [
       "Start by understanding target platforms",
       "Read existing project structure and patterns",

package/.ai-rules/agents/performance-specialist.json

@@ -509,7 +509,7 @@
     }
   },
   "communication": {
-    "language": "Always respond in Korean (한국어)",
+    "language": "en",
     "approach": [
       "Start by understanding performance context (planning/implementation/evaluation)",
       "Plan/analyze bundle size and rendering",

package/.ai-rules/agents/plan-mode.json

@@ -1,6 +1,6 @@
 {
   "name": "Plan Mode Agent",
-  "description": "PLAN 모드 전용 에이전트 - 작업 계획 수립 및 설계에 특화",
+  "description": "PLAN mode agent - specialized for work planning and design",
 
   "model": {
     "preferred": "claude-opus-4-20250514",
@@ -12,21 +12,21 @@
     "mode": "PLAN",
     "purpose": "Mode Agent - delegates to Primary Developer Agent",
     "expertise": [
-      "작업 계획 수립",
-      "TDD 관점 설계",
-      "아키텍처 검토",
-      "Todo 리스트 생성",
-      "요구사항 분석"
+      "Work planning",
+      "TDD-oriented design",
+      "Architecture review",
+      "Todo list creation",
+      "Requirements analysis"
     ],
     "delegates_to": "frontend-developer",
     "responsibilities": [
-      "사용자 요구사항 분석 및 명확화",
-      "TDD 관점에서 테스트 케이스 우선 정의",
-      "구현 전 아키텍처 및 설계 검토",
-      "체계적인 구현 계획 수립",
-      "Todo 리스트 작성 (todo_write 도구 활용)",
-      "품질 기준 및 테스트 전략 계획",
-      "파일 구조 및 네이밍 컨벤션 계획"
+      "Analyze and clarify user requirements",
+      "Prioritize test case definition from TDD perspective",
+      "Review architecture and design before implementation",
+      "Establish systematic implementation plans",
+      "Create todo list (using todo_write tool)",
+      "Plan quality standards and test strategy",
+      "Plan file structure and naming conventions"
     ]
   },
 
@@ -84,25 +84,25 @@
 
   "delegate_agent": {
     "primary": "frontend-developer",
-    "description": "이 Mode Agent는 Frontend Developer Agent의 planning 워크플로우를 활용합니다",
-    "integration": "Frontend Developer Agent의 planning 섹션과 mandatory checklist를 적용"
+    "description": "This Mode Agent utilizes the Frontend Developer Agent's planning workflow",
+    "integration": "Applies Frontend Developer Agent's planning section and mandatory checklist"
   },
 
   "communication": {
-    "language": "ko",
-    "style": "계획 수립에 집중한 체계적 접근",
-    "format": "구조화된 markdown 형태로 명확한 섹션 구분"
+    "language": "en",
+    "style": "Systematic approach focused on planning",
+    "format": "Clear section separation in structured markdown format"
   },
 
   "verification_guide": {
     "mode_compliance": [
-      "✅ '# Mode: PLAN' 표시 확인",
-      "✅ '## Agent : Frontend Developer' (또는 적절한 delegate) 표시 확인",
-      "✅ Korean 응답 확인",
-      "✅ todo_write 도구로 todo 리스트 생성 확인",
-      "✅ 모든 todo 항목이 pending 상태로 생성됨 확인",
-      "✅ Delegate Agent의 planning 워크플로우 적용 확인",
-      "✅ 구조화된 계획 (Plan Overview, Implementation Steps, Quality Checklist 등) 확인"
+      "✅ Verify '# Mode: PLAN' is displayed",
+      "✅ Verify '## Agent : Frontend Developer' (or appropriate delegate) is displayed",
+      "✅ Verify response in configured language",
+      "✅ Verify todo list created with todo_write tool",
+      "✅ Verify all todo items created in pending status",
+      "✅ Verify Delegate Agent's planning workflow is applied",
+      "✅ Verify structured plan (Plan Overview, Implementation Steps, Quality Checklist, etc.)"
     ]
   }
 }

package/.ai-rules/agents/security-specialist.json

@@ -446,7 +446,7 @@
     }
   },
   "communication": {
-    "language": "Always respond in Korean (한국어)",
+    "language": "en",
     "approach": [
       "Start by understanding security context (planning/implementation/evaluation)",
       "Plan/verify authentication implementation",

package/.ai-rules/agents/seo-specialist.json

@@ -407,7 +407,7 @@
     }
   },
   "communication": {
-    "language": "Always respond in Korean (한국어)",
+    "language": "en",
     "approach": [
       "Start by understanding SEO context (planning/implementation/evaluation)",
       "Plan/review metadata implementation",

package/.ai-rules/agents/solution-architect.json

@@ -162,8 +162,8 @@
   },
 
   "communication": {
-    "language": "ko",
-    "style": "체계적이고 명확한 접근, 옵션 중심 설계",
+    "language": "en",
+    "style": "Systematic and clear approach, option-oriented design",
     "approach": [
       "Start with brainstorming skill",
       "Ask one question at a time",

package/.ai-rules/agents/technical-planner.json

@@ -198,8 +198,8 @@
   },
 
   "communication": {
-    "language": "ko",
-    "style": "상세하고 실행 가능한 계획, TDD 중심",
+    "language": "en",
+    "style": "Detailed and actionable plans, TDD-focused",
     "approach": [
       "Start with writing-plans skill",
       "Provide complete code (no placeholders)",

package/.ai-rules/agents/test-strategy-specialist.json

@@ -523,7 +523,7 @@
     }
   },
   "communication": {
-    "language": "Always respond in Korean (한국어)",
+    "language": "en",
     "approach": [
       "Start by understanding test context (planning/implementation/evaluation)",
       "Plan/verify TDD vs Test-After approach",

package/.ai-rules/agents/tooling-engineer.json

@@ -64,6 +64,10 @@
         "rule": "Configuration changes MUST be validated (lint, typecheck, build)",
         "verification_key": "validation"
       },
+      "convention_compliance": {
+        "rule": "Code MUST follow project conventions from config files (tsconfig, eslint, prettier, editorconfig, markdownlint)",
+        "verification_key": "convention_compliance"
+      },
       "language": {
         "rule": "MUST respond in Korean as specified in communication.language",
         "verification_key": "language"
@@ -74,6 +78,7 @@
       "backward_compatible": "Check existing tests pass, verify no breaking changes to build output",
       "documentation": "Add inline comments explaining non-obvious settings",
       "validation": "Run yarn lint, yarn typecheck, yarn build after changes",
+      "convention_compliance": "Call get_code_conventions MCP tool and verify code follows all project conventions",
       "language": "All response text in Korean"
     }
   },
@@ -114,6 +119,19 @@
         "5. Test that all features still work",
         "6. Document any breaking changes or migration steps"
       ]
+    },
+    "convention_verification": {
+      "approach": "Verify code against project config files",
+      "applies_to": "ACT mode implementation and EVAL mode review",
+      "steps": [
+        "1. Call get_code_conventions MCP tool to get project conventions",
+        "2. TypeScript: Verify strict mode, noImplicitAny, strictNullChecks compliance",
+        "3. ESLint: Check flat config usage, verify no lint errors",
+        "4. Prettier: Verify formatting (quotes, semicolons, trailing commas, indentation)",
+        "5. EditorConfig: Check indentation style/size, line endings, charset",
+        "6. Markdown: Verify markdownlint rules compliance",
+        "7. Report any convention violations with specific file/line references"
+      ]
     }
   },
 

package/.ai-rules/agents/ui-ux-designer.json

@@ -493,7 +493,7 @@
   },
 
   "communication": {
-    "language": "Always respond in Korean (한국어)",
+    "language": "en",
     "approach": [
       "Start by understanding user goals and context",
       "Apply universal design principles (not system-specific)",

package/.ai-rules/checklists/conventions.json

@@ -0,0 +1,201 @@
+{
+  "domain": "conventions",
+  "icon": "📐",
+  "description": "Code conventions and formatting consistency from project config files",
+  "categories": [
+    {
+      "name": "typescript",
+      "triggers": {
+        "files": ["**/*.ts", "**/*.tsx"],
+        "imports": [],
+        "patterns": []
+      },
+      "items": [
+        {
+          "id": "conv-ts-001",
+          "text": "TypeScript strict mode is enabled (tsconfig.json: strict: true)",
+          "priority": "high",
+          "reason": "Strict mode catches type errors early and improves code quality"
+        },
+        {
+          "id": "conv-ts-002",
+          "text": "No implicit any is enforced (tsconfig.json: noImplicitAny: true)",
+          "priority": "high",
+          "reason": "Explicit types prevent runtime errors"
+        },
+        {
+          "id": "conv-ts-003",
+          "text": "Strict null checks are enabled (tsconfig.json: strictNullChecks: true)",
+          "priority": "high",
+          "reason": "Prevents null/undefined errors at runtime"
+        },
+        {
+          "id": "conv-ts-004",
+          "text": "Path aliases match tsconfig.json paths configuration",
+          "priority": "medium",
+          "reason": "Consistent import paths improve maintainability"
+        }
+      ]
+    },
+    {
+      "name": "eslint",
+      "triggers": {
+        "files": ["**/*.ts", "**/*.tsx", "**/*.js", "**/*.jsx"],
+        "imports": [],
+        "patterns": []
+      },
+      "items": [
+        {
+          "id": "conv-eslint-001",
+          "text": "Using ESLint flat config (eslint.config.js, not .eslintrc)",
+          "priority": "medium",
+          "reason": "Flat config is the new standard and more maintainable"
+        },
+        {
+          "id": "conv-eslint-002",
+          "text": "Code follows ESLint rules defined in project config",
+          "priority": "high",
+          "reason": "Consistent linting prevents code quality issues"
+        },
+        {
+          "id": "conv-eslint-003",
+          "text": "No ESLint errors or warnings in modified files",
+          "priority": "high",
+          "reason": "Lint errors indicate code quality or style violations"
+        }
+      ]
+    },
+    {
+      "name": "prettier",
+      "triggers": {
+        "files": [
+          "**/*.ts",
+          "**/*.tsx",
+          "**/*.js",
+          "**/*.jsx",
+          "**/*.json",
+          "**/*.css",
+          "**/*.scss"
+        ],
+        "imports": [],
+        "patterns": []
+      },
+      "items": [
+        {
+          "id": "conv-prettier-001",
+          "text": "Code uses consistent quote style per .prettierrc (singleQuote setting)",
+          "priority": "medium",
+          "reason": "Consistent quoting improves readability"
+        },
+        {
+          "id": "conv-prettier-002",
+          "text": "Semicolons match .prettierrc setting (semi: true/false)",
+          "priority": "medium",
+          "reason": "Consistent semicolon usage prevents parsing ambiguities"
+        },
+        {
+          "id": "conv-prettier-003",
+          "text": "Trailing commas follow .prettierrc (trailingComma setting)",
+          "priority": "low",
+          "reason": "Consistent trailing commas make diffs cleaner"
+        },
+        {
+          "id": "conv-prettier-004",
+          "text": "Indentation matches .prettierrc (tabWidth setting)",
+          "priority": "medium",
+          "reason": "Consistent indentation is essential for readability"
+        },
+        {
+          "id": "conv-prettier-005",
+          "text": "Arrow function parentheses match .prettierrc (arrowParens setting)",
+          "priority": "low",
+          "reason": "Consistent arrow function style"
+        }
+      ]
+    },
+    {
+      "name": "editorconfig",
+      "triggers": {
+        "files": [
+          "**/*.ts",
+          "**/*.tsx",
+          "**/*.js",
+          "**/*.jsx",
+          "**/*.json",
+          "**/*.css",
+          "**/*.scss",
+          "**/*.md",
+          "**/*.mdx"
+        ],
+        "imports": [],
+        "patterns": []
+      },
+      "items": [
+        {
+          "id": "conv-editor-001",
+          "text": "Indentation style matches .editorconfig (indent_style: space/tab)",
+          "priority": "high",
+          "reason": "Mixed tabs and spaces cause rendering issues"
+        },
+        {
+          "id": "conv-editor-002",
+          "text": "Indent size matches .editorconfig (indent_size setting)",
+          "priority": "high",
+          "reason": "Consistent indentation improves code readability"
+        },
+        {
+          "id": "conv-editor-003",
+          "text": "Line endings match .editorconfig (end_of_line: lf/crlf)",
+          "priority": "medium",
+          "reason": "Consistent line endings prevent git conflicts"
+        },
+        {
+          "id": "conv-editor-004",
+          "text": "File encoding is UTF-8 per .editorconfig (charset setting)",
+          "priority": "medium",
+          "reason": "UTF-8 ensures proper character display"
+        },
+        {
+          "id": "conv-editor-005",
+          "text": "Trailing whitespace removed per .editorconfig",
+          "priority": "low",
+          "reason": "Trailing whitespace creates unnecessary diffs"
+        },
+        {
+          "id": "conv-editor-006",
+          "text": "Files end with newline per .editorconfig (insert_final_newline)",
+          "priority": "low",
+          "reason": "Final newlines are POSIX standard"
+        }
+      ]
+    },
+    {
+      "name": "markdown",
+      "triggers": {
+        "files": ["**/*.md", "**/*.mdx"],
+        "imports": [],
+        "patterns": []
+      },
+      "items": [
+        {
+          "id": "conv-md-001",
+          "text": "Markdown heading style follows .markdownlint.json (MD003 rule)",
+          "priority": "low",
+          "reason": "Consistent heading style improves document structure"
+        },
+        {
+          "id": "conv-md-002",
+          "text": "Markdown list style follows .markdownlint.json (MD004 rule)",
+          "priority": "low",
+          "reason": "Consistent list markers improve readability"
+        },
+        {
+          "id": "conv-md-003",
+          "text": "No markdownlint violations per .markdownlint.json rules",
+          "priority": "medium",
+          "reason": "Markdown lint rules ensure document quality"
+        }
+      ]
+    }
+  ]
+}