by NYCU-Chung
An engineering team in a box for Claude Code — 12 specialized agents, 15 automation hooks, and the P7/P9/P10 methodology.
# Add to your Claude Code skills
git clone https://github.com/NYCU-Chung/my-claude-devteamEnglish · 繁體中文
An entire engineering team for Claude Code — 12 specialized agents, 15 automation hooks, and the P7/P9/P10 methodology that keeps them disciplined.
Most people use Claude Code as a single coder. This config turns it into a full engineering org: planner, fullstack-engineer, refactor-specialist, migration-engineer, frontend-designer, critic, vuln-verifier, debugger, db-expert, onboarder, tool-expert, web-researcher — each agent owns a role, each has its own tool permissions, and a strict delegation rulebook decides who touches what.
Backed by corporate-culture-inspired discipline (closure, fact-driven, exhaustiveness) and battle-tested hooks that catch debugger statements, hardcoded secrets, cost overruns, and MCP outages before they hit main.
| Role | Agent | What they do | When they ship |
|------|-------|--------------|---------------|
| 📋 Tech Lead | planner | Breaks down fuzzy requirements into parallelizable Task Prompts with a six-element contract (goal / scope / input / output / acceptance / boundaries). Never writes code. | Task touches 3+ files or 2+ modules |
| 🛠 Senior Engineer | fullstack-engineer | Ships features using the P7 methodology: read reality → design solution → impact analysis → implement → three-question self-review → [P7-COMPLETION] delivery. | Single-feature or cross-module implementation |
| 🔄 Refactor Lead | refactor-specialist | Large-scale safe refactors. Atomic commits, full callsite verification, single-revert rollback. | Renames, file moves, module extraction across 10+ files |
| 🚀 Migration Lead | | Framework / library major-version upgrades. Reads upstream changelog, executes incrementally, verifies at every step. | Next.js 13→14, Vue 2→3, Tailwind 3→4, etc. |
| 🎨 | | Builds landing pages, dashboards, and UI that doesn't look like AI slop. Opinionated aesthetic direction, refuses generic output. | New pages, UI redesigns, visual upgrades |
| 🔍 | | Finds bugs, security holes, logic errors, edge cases, performance issues. Every finding with file path + line number. No "looks good to me". | Pre-commit, pre-deploy, pre-merge |
| 🧪 | | Takes the critic's findings and writes actual PoC tests to prove the vulnerability is real — no false positives, no hand-waving. | After critic flags a security issue |
| 🐛 | | Reads logs, constructs hypotheses, verifies, fixes. Never guesses, always traces root cause. Includes log-analyzer. | Bug reports, service incidents, test failures |
| 🗄 | | Reviews schemas, migrations, queries for safety, indexes, locks, race conditions. Paranoid about data loss. | Schema changes, migrations, query optimization |
| 🗺 | | First-time codebase exploration. Produces a structured mental model — architecture, entry points, suspicious areas. | Joining a new project, evaluating an open-source repo |
| ⚙️ | | Picks the right MCP tools, chains complex workflows, troubleshoots tool failures. Knows every integration in your stack. | MCP tool failures, complex tool chaining |
| 📚 | | Fetches and synthesizes official docs, API specs, error code meanings. The antidote to hallucination. | Uncertain API usage, error code lookups |
No comments yet. Be the first to share your thoughts!
migration-engineerfrontend-designercriticvuln-verifierdebuggerdb-expertonboardertool-expertweb-researcherEach agent is a markdown file under agents/ with its own system prompt, tool permissions, and model selection. Customize them. Fork them. Replace the ones you don't need.
┌─────────────┐
│ Your Task │
└──────┬──────┘
│
┌────────▼────────┐
│ 📋 planner │ ← Breaks into parallel subtasks
│ (Tech Lead) │ if touches 3+ files
└────────┬────────┘
│
┌─────────┼─────────┐
▼ ▼ ▼
┌─────────┐ ┌─────────┐ ┌─────────┐
│ fullstk │ │ fullstk │ │ fullstk │ ← Parallel execution
│ × N │ │ × N │ │ × N │ P7 methodology
└────┬────┘ └────┬────┘ └────┬────┘
└───────────┼───────────┘
▼
┌──────────────┐
│ 🔍 critic │ ← Mandatory pre-deploy review
│ (reviewer) │
└───────┬──────┘
│
┌────────┴────────┐
│ │
▼ ▼
┌──────────────┐ ┌─────────────┐
│ 🐛 debugger │ │ Deploy │
│ (if issues) │ │ │
└──────────────┘ └─────────────┘
Security-sensitive work takes a detour: critic flags → vuln-verifier writes PoC → fix or file PR.
Every agent enforces these. No exceptions. No "close enough".
Not role-play. Operating modes that Claude switches between based on task scope:
| Scope | Mode | Behavior |
|-------|------|----------|
| Single feature | P7 (Senior Engineer) | Design → Impact analysis → Implement → Three-question self-review → [P7-COMPLETION] |
| Multi-module, 3+ files | P9 (Tech Lead) | Decompose into Task Prompts with six elements. Coding is forbidden — your output is prompts, not code. |
| Cross-team, 5+ sprints | P10 (CTO) | Output strategy docs. Goals, success metrics, risks, timeline, resource allocation. |
The team shifts into exhaustive mode when:
We don't keep idle agents. No half-finished work. No excuses.
Fifteen automation hooks wire up at pre-commit, post-tool-use, and stop events. They catch problems before they ship.
| Hook | Trigger | What it catches |
|------|---------|-----------------|
| 💰 cost-tracker.js | After every response | Token usage + estimated cost per model (Opus / Sonnet / Haiku). Running tally in ~/.claude/stats-cache.json |
| ✋ commit-quality.js | Pre-commit | Blocks commits with debugger statements or hardcoded secrets in JS/TS/Python files |
| 🔧 mcp-health.js | MCP tool failures | Detects MCP server outages and suggests restart paths |
| 🛡 config-protection.js | Edit/Write to critical files | Guards important config files from accidental overwrites |
| 🎨 design-quality.js | Frontend changes | Checks for AI-slop indicators in UI code |
| 📝 check-console.js | Pre-commit | Flags stray console.log in production paths |
| 📊 audit-log.js | All tool calls | Keeps an audit trail of significant tool operations |
| 🎯 batch-format.js | Multi-file edits | Runs formatter on modified files in batch |
| 💡 suggest-compact.js | Context pressure | Suggests /compact when context window fills up |
| 📈 accumulator.js | Session tracking | Accumulates session metrics |
| 🚨 log-error.sh | Any error | Unified error logging to ~/.claude/error-log.md |
| 🧪 test-runner.js | After file edit | Finds sibling test file, runs vitest/jest, reports failures (non-blocking) |
| 🔒 branch-protection.js | Pre-Bash | Hard-blocks force pushes and direct commits to main / master / production / release |
| 📏 large-file-warner.js | Pre-Read | Warns at 500 KB, blocks at 2 MB to protect context window |
| 📚 session-summary.js | Stop | Appends session summary to ~/.claude/sessions/ for later search |
Each hook is a self-contained script. Enable / disable / customize in settings.example.json.
Real observations from daily use. Your mileage may vary.
critic is the MVPOn mid-sized modules (500–2000 lines), critic routinely finds 20–30 issues across all severity tiers. On large open-source codebases (tested against OpenClaw 352K⭐, Mermaid 87K⭐, Storybook 85K⭐, React Router 56K⭐), a single focused audit still surfaces 5–10 real bugs that had not been reported in the issue tracker.
Notable catches from real audits:
!== vs safeEqualSecret)writeFileSync race in an auth-adjacent allowlist file that would corrupt state under concurrent access/r1/) that misidentified unrelated models as thinking modelsThe strictness ("assume everything is broken until proven otherwise") is what makes it work.
debugger saves you from face-plantsTwice during one bug-hunting session, the author was about to submit PRs based on seemingly clear reproductions. Both times debugger traced the behavior on HEAD and found the bugs had already been silently fixed in recent commits — the original reporters were on outdated versions. Submitting would have been embarrassing.