codingbuddy-rules 2.0.0 → 2.2.0

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

@@ -0,0 +1,202 @@
+{
+  "name": "Tooling Engineer",
+  "description": "Project configuration, build tools, and development environment specialist",
+  "model": {
+    "preferred": "claude-sonnet-4-20250514",
+    "reason": "Balanced for configuration tasks and tooling setup"
+  },
+
+  "role": {
+    "title": "Tooling Engineer",
+    "type": "primary",
+    "expertise": [
+      "Project Configuration (codingbuddy.config.js, .env)",
+      "TypeScript Configuration (tsconfig.json, paths)",
+      "Linting & Formatting (ESLint, Prettier, Stylelint)",
+      "Build Tools (Vite, Webpack, Next.js config, Rollup)",
+      "Package Management (package.json, yarn workspaces, dependencies)",
+      "MCP Tools & IDE Integration",
+      "Development Environment Setup"
+    ],
+    "responsibilities": [
+      "Configure and optimize project settings",
+      "Set up and maintain build tool configurations",
+      "Manage linter and formatter rules",
+      "Handle package dependencies and workspace configuration",
+      "Configure TypeScript compiler options",
+      "Set up development environment and IDE settings",
+      "Integrate MCP tools with development workflow"
+    ]
+  },
+
+  "context_files": [".ai-rules/rules/core.md", ".ai-rules/rules/project.md"],
+
+  "activation": {
+    "trigger": "When user works with config files, build tools, or development environment setup",
+    "explicit_patterns": [
+      "설정 파일",
+      "config file",
+      "tsconfig",
+      "eslint",
+      "prettier",
+      "package.json",
+      "vite.config",
+      "next.config",
+      "webpack",
+      "빌드 설정",
+      "build config",
+      "tooling-engineer로"
+    ],
+    "mandatory_checklist": {
+      "schema_compliance": {
+        "rule": "Configuration changes MUST maintain valid schema structure",
+        "verification_key": "schema_compliance"
+      },
+      "backward_compatible": {
+        "rule": "Changes MUST NOT break existing configurations or build processes",
+        "verification_key": "backward_compatible"
+      },
+      "documentation": {
+        "rule": "Non-obvious configuration options MUST be documented with comments",
+        "verification_key": "documentation"
+      },
+      "validation": {
+        "rule": "Configuration changes MUST be validated (lint, typecheck, build)",
+        "verification_key": "validation"
+      },
+      "language": {
+        "rule": "MUST respond in Korean as specified in communication.language",
+        "verification_key": "language"
+      }
+    },
+    "verification_guide": {
+      "schema_compliance": "Run schema validation if available (e.g., tsc --showConfig for tsconfig)",
+      "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",
+      "language": "All response text in Korean"
+    }
+  },
+
+  "workflow": {
+    "config_modification": {
+      "approach": "Incremental change with validation",
+      "applies_to": "Modifying existing configuration files",
+      "steps": [
+        "1. Analyze current configuration and understand existing settings",
+        "2. Identify the specific change needed",
+        "3. Make minimal, targeted changes",
+        "4. Add comments for non-obvious settings",
+        "5. Validate changes (lint, typecheck, build)",
+        "6. Test that existing functionality still works"
+      ]
+    },
+    "tool_setup": {
+      "approach": "Best practices implementation",
+      "applies_to": "Setting up new tools or configurations",
+      "steps": [
+        "1. Research current best practices for the tool",
+        "2. Check project's existing patterns and conventions",
+        "3. Create configuration following project standards",
+        "4. Add necessary dependencies to package.json",
+        "5. Document setup in comments or README",
+        "6. Verify integration with existing toolchain"
+      ]
+    },
+    "dependency_management": {
+      "approach": "Safe updates with compatibility check",
+      "applies_to": "Managing package dependencies",
+      "steps": [
+        "1. Check current dependency versions and constraints",
+        "2. Research compatibility between packages",
+        "3. Update package.json with appropriate version ranges",
+        "4. Run install and verify lock file changes",
+        "5. Test that all features still work",
+        "6. Document any breaking changes or migration steps"
+      ]
+    }
+  },
+
+  "config_patterns": {
+    "typescript": {
+      "tsconfig.json": "TypeScript compiler configuration",
+      "paths": "Module path aliases for cleaner imports",
+      "strict_mode": "Always enable strict mode for type safety"
+    },
+    "linting": {
+      "eslint.config.js": "ESLint flat config (modern approach)",
+      "prettier": "Code formatting rules",
+      "integration": "Ensure ESLint and Prettier work together"
+    },
+    "build_tools": {
+      "vite.config.ts": "Vite bundler configuration",
+      "next.config.js": "Next.js framework configuration",
+      "webpack.config.js": "Webpack bundler configuration"
+    },
+    "package_management": {
+      "package.json": "Dependencies, scripts, and metadata",
+      "workspaces": "Monorepo workspace configuration",
+      "engines": "Node.js version requirements"
+    }
+  },
+
+  "best_practices": {
+    "general": [
+      "Use specific versions or version ranges (^, ~) appropriately",
+      "Keep configuration files minimal and well-documented",
+      "Follow project conventions over personal preferences",
+      "Validate changes before committing",
+      "Consider cross-platform compatibility"
+    ],
+    "typescript": [
+      "Enable strict mode for better type safety",
+      "Use path aliases for cleaner imports",
+      "Configure appropriate target and module settings",
+      "Include necessary type definitions"
+    ],
+    "linting": [
+      "Use ESLint flat config for modern projects",
+      "Configure rules that match team conventions",
+      "Integrate with Prettier for formatting",
+      "Set up pre-commit hooks for enforcement"
+    ],
+    "build": [
+      "Optimize for development and production separately",
+      "Configure appropriate source maps",
+      "Set up tree-shaking and code splitting",
+      "Consider bundle size implications"
+    ]
+  },
+
+  "code_quality_checklist": [
+    "Configuration is valid and parseable",
+    "All settings are documented or self-explanatory",
+    "No breaking changes to existing functionality",
+    "Build process completes successfully",
+    "Lint and typecheck pass",
+    "Development workflow still works correctly"
+  ],
+
+  "communication": {
+    "language": "en",
+    "style": "Technical and precise, focusing on configuration details",
+    "approach": [
+      "Start by understanding the current configuration state",
+      "Explain the purpose and impact of changes",
+      "Provide context for non-obvious settings",
+      "Verify changes work before completing"
+    ]
+  },
+
+  "reference": {
+    "project_rules": ".ai-rules/rules/",
+    "config_schema_reference": "See project's package.json and existing configs",
+    "official_docs": {
+      "typescript": "https://www.typescriptlang.org/tsconfig",
+      "eslint": "https://eslint.org/docs/latest/use/configure/configuration-files",
+      "prettier": "https://prettier.io/docs/configuration",
+      "vite": "https://vite.dev/config/",
+      "nextjs": "https://nextjs.org/docs/app/api-reference/config/next-config-js"
+    }
+  }
+}

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/accessibility.json

@@ -0,0 +1,132 @@
+{
+  "domain": "accessibility",
+  "icon": "♿",
+  "description": "Accessibility (a11y) review items for WCAG 2.1 AA compliance",
+  "categories": [
+    {
+      "name": "forms",
+      "triggers": {
+        "files": [
+          "**/form/**",
+          "**/input/**",
+          "**/login/**",
+          "**/signup/**",
+          "**/contact/**"
+        ],
+        "imports": ["react-hook-form", "formik", "@radix-ui/react-form"],
+        "patterns": ["<form", "<input", "<select", "<textarea", "onSubmit"]
+      },
+      "items": [
+        {
+          "id": "a11y-form-001",
+          "text": "Associate labels with form inputs using htmlFor/id",
+          "priority": "critical",
+          "reason": "Screen readers need label association to announce field purpose",
+          "reference": "WCAG 1.3.1"
+        },
+        {
+          "id": "a11y-form-002",
+          "text": "Provide clear error messages with aria-describedby",
+          "priority": "high",
+          "reason": "Users need to understand what went wrong"
+        },
+        {
+          "id": "a11y-form-003",
+          "text": "Mark required fields with aria-required",
+          "priority": "medium",
+          "reason": "Assistive technologies should announce required fields"
+        },
+        {
+          "id": "a11y-form-004",
+          "text": "Ensure form can be submitted with keyboard (Enter key)",
+          "priority": "high",
+          "reason": "Keyboard-only users must be able to submit forms"
+        }
+      ]
+    },
+    {
+      "name": "interactive_elements",
+      "triggers": {
+        "files": [
+          "**/button/**",
+          "**/modal/**",
+          "**/dialog/**",
+          "**/dropdown/**",
+          "**/menu/**",
+          "**/components/**"
+        ],
+        "imports": [
+          "@radix-ui/*",
+          "@headlessui/*",
+          "react-aria",
+          "react-modal"
+        ],
+        "patterns": ["onClick", "onKeyDown", "role=", "aria-", "<button", "<a "]
+      },
+      "items": [
+        {
+          "id": "a11y-interactive-001",
+          "text": "Ensure all interactive elements are keyboard accessible",
+          "priority": "critical",
+          "reason": "Not all users can use a mouse",
+          "reference": "WCAG 2.1.1"
+        },
+        {
+          "id": "a11y-interactive-002",
+          "text": "Provide visible focus indicators",
+          "priority": "critical",
+          "reason": "Keyboard users need to see where focus is",
+          "reference": "WCAG 2.4.7"
+        },
+        {
+          "id": "a11y-interactive-003",
+          "text": "Use semantic HTML elements (button, a) instead of div with onClick",
+          "priority": "high",
+          "reason": "Semantic elements have built-in accessibility"
+        },
+        {
+          "id": "a11y-interactive-004",
+          "text": "Manage focus when opening/closing modals",
+          "priority": "high",
+          "reason": "Focus should move to modal and return when closed"
+        }
+      ]
+    },
+    {
+      "name": "content",
+      "triggers": {
+        "files": ["**/page/**", "**/layout/**", "**/*.tsx", "**/*.jsx"],
+        "imports": ["next/image", "next/link"],
+        "patterns": ["<img", "<Image", "<h1", "<h2", "<main", "<nav", "<aside"]
+      },
+      "items": [
+        {
+          "id": "a11y-content-001",
+          "text": "Provide alt text for all meaningful images",
+          "priority": "critical",
+          "reason": "Screen reader users cannot see images",
+          "reference": "WCAG 1.1.1"
+        },
+        {
+          "id": "a11y-content-002",
+          "text": "Use proper heading hierarchy (h1 > h2 > h3)",
+          "priority": "high",
+          "reason": "Headings provide document structure for navigation"
+        },
+        {
+          "id": "a11y-content-003",
+          "text": "Ensure sufficient color contrast (4.5:1 for text)",
+          "priority": "high",
+          "reason": "Low contrast text is hard to read",
+          "reference": "WCAG 1.4.3"
+        },
+        {
+          "id": "a11y-content-004",
+          "text": "Use landmark regions (main, nav, aside, footer)",
+          "priority": "medium",
+          "reason": "Landmarks help screen reader users navigate"
+        }
+      ]
+    }
+  ]
+}

package/.ai-rules/checklists/code-quality.json

@@ -0,0 +1,97 @@
+{
+  "domain": "code-quality",
+  "icon": "📏",
+  "description": "Code quality and maintainability review items",
+  "categories": [
+    {
+      "name": "general",
+      "triggers": {
+        "files": ["**/*.ts", "**/*.tsx", "**/*.js", "**/*.jsx"],
+        "imports": [],
+        "patterns": []
+      },
+      "items": [
+        {
+          "id": "cq-gen-001",
+          "text": "Use explicit TypeScript types (avoid 'any')",
+          "priority": "high",
+          "reason": "Type safety catches errors at compile time"
+        },
+        {
+          "id": "cq-gen-002",
+          "text": "Keep functions small (10-20 lines max)",
+          "priority": "medium",
+          "reason": "Small functions are easier to understand and test"
+        },
+        {
+          "id": "cq-gen-003",
+          "text": "Use meaningful variable and function names",
+          "priority": "high",
+          "reason": "Code should be self-documenting"
+        },
+        {
+          "id": "cq-gen-004",
+          "text": "Remove dead code and unused imports",
+          "priority": "medium",
+          "reason": "Dead code adds confusion and maintenance burden"
+        }
+      ]
+    },
+    {
+      "name": "solid_principles",
+      "triggers": {
+        "files": ["**/service/**", "**/util/**", "**/lib/**", "**/core/**"],
+        "imports": [],
+        "patterns": ["class ", "interface ", "abstract "]
+      },
+      "items": [
+        {
+          "id": "cq-solid-001",
+          "text": "Single Responsibility: Each module has one reason to change",
+          "priority": "high",
+          "reason": "Focused modules are easier to maintain"
+        },
+        {
+          "id": "cq-solid-002",
+          "text": "Dependency Inversion: Depend on abstractions, not concretions",
+          "priority": "medium",
+          "reason": "Makes code more flexible and testable"
+        },
+        {
+          "id": "cq-solid-003",
+          "text": "DRY: Eliminate code duplication",
+          "priority": "high",
+          "reason": "Duplicated code means duplicated bugs"
+        }
+      ]
+    },
+    {
+      "name": "error_handling",
+      "triggers": {
+        "files": ["**/api/**", "**/service/**", "**/handler/**"],
+        "imports": [],
+        "patterns": ["try", "catch", "throw", "Error", "Promise"]
+      },
+      "items": [
+        {
+          "id": "cq-error-001",
+          "text": "Handle all error cases explicitly",
+          "priority": "high",
+          "reason": "Unhandled errors crash applications"
+        },
+        {
+          "id": "cq-error-002",
+          "text": "Provide meaningful error messages",
+          "priority": "medium",
+          "reason": "Good error messages help debugging"
+        },
+        {
+          "id": "cq-error-003",
+          "text": "Use custom error types for domain errors",
+          "priority": "low",
+          "reason": "Enables specific error handling"
+        }
+      ]
+    }
+  ]
+}

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

@@ -0,0 +1,47 @@
+{
+  "version": "1.0.0",
+  "domains": [
+    {
+      "id": "security",
+      "name": "Security",
+      "icon": "🔒",
+      "file": "security.json",
+      "description": "Security-related review items (authentication, input validation, data protection)"
+    },
+    {
+      "id": "accessibility",
+      "name": "Accessibility",
+      "icon": "♿",
+      "file": "accessibility.json",
+      "description": "WCAG 2.1 AA compliance items (forms, interactive elements, content)"
+    },
+    {
+      "id": "performance",
+      "name": "Performance",
+      "icon": "⚡",
+      "file": "performance.json",
+      "description": "Performance optimization items (rendering, loading, data fetching)"
+    },
+    {
+      "id": "testing",
+      "name": "Testing",
+      "icon": "🧪",
+      "file": "testing.json",
+      "description": "Testing strategy items (unit, component, API testing)"
+    },
+    {
+      "id": "code-quality",
+      "name": "Code Quality",
+      "icon": "📏",
+      "file": "code-quality.json",
+      "description": "Code quality items (TypeScript, SOLID principles, error handling)"
+    },
+    {
+      "id": "seo",
+      "name": "SEO",
+      "icon": "🔍",
+      "file": "seo.json",
+      "description": "SEO optimization items (metadata, structure, links)"
+    }
+  ]
+}

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

@@ -0,0 +1,97 @@
+{
+  "domain": "performance",
+  "icon": "⚡",
+  "description": "Performance optimization review items",
+  "categories": [
+    {
+      "name": "rendering",
+      "triggers": {
+        "files": ["**/components/**", "**/page/**", "**/*.tsx", "**/*.jsx"],
+        "imports": ["react", "next", "useMemo", "useCallback", "memo"],
+        "patterns": ["useState", "useEffect", "useMemo", "useCallback", "memo("]
+      },
+      "items": [
+        {
+          "id": "perf-render-001",
+          "text": "Memoize expensive calculations with useMemo",
+          "priority": "medium",
+          "reason": "Prevents unnecessary recalculations on re-render"
+        },
+        {
+          "id": "perf-render-002",
+          "text": "Use useCallback for callback props to child components",
+          "priority": "medium",
+          "reason": "Prevents unnecessary child re-renders"
+        },
+        {
+          "id": "perf-render-003",
+          "text": "Avoid inline object/array creation in JSX",
+          "priority": "low",
+          "reason": "New references cause unnecessary re-renders"
+        },
+        {
+          "id": "perf-render-004",
+          "text": "Use React.memo for pure presentational components",
+          "priority": "low",
+          "reason": "Skip re-renders when props haven't changed"
+        }
+      ]
+    },
+    {
+      "name": "loading",
+      "triggers": {
+        "files": ["**/page/**", "**/layout/**", "**/app/**"],
+        "imports": ["next/dynamic", "next/image", "react-lazy-load"],
+        "patterns": ["dynamic(", "lazy(", "Suspense", "loading"]
+      },
+      "items": [
+        {
+          "id": "perf-load-001",
+          "text": "Use code splitting with dynamic imports",
+          "priority": "high",
+          "reason": "Reduces initial bundle size"
+        },
+        {
+          "id": "perf-load-002",
+          "text": "Optimize images (WebP, proper sizing, lazy loading)",
+          "priority": "high",
+          "reason": "Images are often the largest assets"
+        },
+        {
+          "id": "perf-load-003",
+          "text": "Implement loading skeletons for async content",
+          "priority": "medium",
+          "reason": "Improves perceived performance"
+        }
+      ]
+    },
+    {
+      "name": "data_fetching",
+      "triggers": {
+        "files": ["**/api/**", "**/hook/**", "**/query/**"],
+        "imports": ["@tanstack/react-query", "swr", "axios"],
+        "patterns": ["useQuery", "useSWR", "fetch(", "getServerSideProps"]
+      },
+      "items": [
+        {
+          "id": "perf-data-001",
+          "text": "Implement proper caching strategy",
+          "priority": "high",
+          "reason": "Reduces unnecessary network requests"
+        },
+        {
+          "id": "perf-data-002",
+          "text": "Use stale-while-revalidate pattern where appropriate",
+          "priority": "medium",
+          "reason": "Shows cached data while fetching fresh data"
+        },
+        {
+          "id": "perf-data-003",
+          "text": "Paginate or virtualize large lists",
+          "priority": "high",
+          "reason": "Rendering thousands of items kills performance"
+        }
+      ]
+    }
+  ]
+}