Skip to content

Conversation

@iammarkps
Copy link
Member

@iammarkps iammarkps commented Jan 19, 2026

  • Minimize root CLAUDE.md to essentials only (17 lines)
  • Create .claude/docs/ with consolidated pattern files:
    • react-patterns.md: Server/Client components, styling
    • api-patterns.md: Validation (Zod→TypeBox migration noted)
    • database-patterns.md: Prisma queries, migrations
    • auth-patterns.md: getServerUser(), admin checks
  • Update agents to reference docs instead of duplicating
  • Simplify skills to quick reference + link to full docs

Reduces duplication across 6 files (-77 lines net).

Summary by CodeRabbit

  • Documentation
    • Consolidated development guidance into a new pattern reference structure.
    • Created new documentation guides for API validation, authentication, database access, and React best practices.
    • Reorganized core development resources with streamlined references to external pattern documentation.
    • Updated code and security review standards with refined criteria and pattern references.
    • Simplified skill-level documentation to focus on key rules rather than detailed examples.

✏️ Tip: You can customize this high-level summary in your review settings.

- Minimize root CLAUDE.md to essentials only (17 lines)
- Create .claude/docs/ with consolidated pattern files:
  - react-patterns.md: Server/Client components, styling
  - api-patterns.md: Validation (Zod→TypeBox migration noted)
  - database-patterns.md: Prisma queries, migrations
  - auth-patterns.md: getServerUser(), admin checks
- Update agents to reference docs instead of duplicating
- Simplify skills to quick reference + link to full docs

Reduces duplication across 6 files (-77 lines net).
@coderabbitai
Copy link

coderabbitai bot commented Jan 19, 2026

📝 Walkthrough

Walkthrough

This PR restructures Claude AI agent guidance and documentation by creating comprehensive pattern reference files for API, authentication, database, and React patterns. It replaces inline code examples and detailed guidance in agent configurations and skill files with references to these new external documentation files, consolidating patterns into dedicated markdown resources.

Changes

Cohort / File(s) Summary
Agent Configuration Updates
.claude/agents/code-reviewer.md, .claude/agents/security-reviewer.md
Reworded review criteria and replaced concrete code patterns with generic references to external pattern documentation; removed explicit examples for auth and input validation.
New Pattern Documentation
.claude/docs/api-patterns.md, .claude/docs/auth-patterns.md, .claude/docs/database-patterns.md, .claude/docs/react-patterns.md
Introduced four new comprehensive pattern documentation files covering validation strategy (Zod to Elysia migration), authentication best practices, database access patterns with Prisma, and React component patterns with server/client separation guidance.
Skill File Simplification
.claude/skills/api-development/SKILL.md, .claude/skills/component-development/SKILL.md, .claude/skills/database-changes/SKILL.md
Condensed skill files by removing detailed code examples, inline guidance, and sections; replaced with concise rules and references to external pattern documentation.
Main Configuration Restructure
CLAUDE.md
Restructured from "Core Principles" and "Key Patterns" sections to "Quick Reference" and "Patterns"; removed "Commands" and "Gotchas" sections; replaced inline code examples with four external documentation links.

Estimated code review effort

🎯 3 (Moderate) | ⏱️ ~20 minutes

Possibly related PRs

  • programming.in.th/programming.in.th#989: Modifies the same .claude configuration, agent, and documentation files with overlapping edits to pattern references and guidance structures.

Suggested reviewers

  • leomotors
  • MasterIceZ
  • plohan
  • pongsaphol

Poem

🐰 A-hopping through docs with a spring in my feet,
Patterns reorganized, neat and complete!
From scattered examples to reference so clear,
Claude guides his path with no lingering fear.

🚥 Pre-merge checks | ✅ 3
✅ Passed checks (3 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title clearly describes the main refactoring objective: applying progressive disclosure by consolidating scattered documentation and reducing root CLAUDE.md to essentials with external references.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.


Comment @coderabbitai help to get the list of available commands and usage tips.

@github-actions
Copy link

Coverage Report

Status Category Percentage Covered / Total
🔵 Lines 1.32% 135 / 10207
🔵 Statements 1.32% 135 / 10207
🔵 Functions 48.81% 103 / 211
🔵 Branches 51.78% 116 / 224
File CoverageNo changed files found.
Generated in workflow #2201 for commit 994b608 by the Vitest Coverage Report Action

@pongsaphol pongsaphol merged commit 79962f2 into main Jan 23, 2026
6 checks passed
@pongsaphol pongsaphol deleted the claude/refactor-agents-disclosure-U771w branch January 23, 2026 04:28
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants