@punks/cli 1.0.1 → 1.0.3

package/dist/data/subagents/manifest.mjs

@@ -62,6 +62,40 @@ const capabilityPacks = {
       "Repo documentation maintenance, routing, and human-facing durable knowledge capture.",
     skills: ["docs-maintenance"],
   },
+  "review-core": {
+    description:
+      "Cross-cutting review, simplification, and architecture-friction analysis for changed code.",
+    skills: ["simplify", "improve-codebase-architecture"],
+  },
+  "python-core": {
+    description:
+      "Python project structure, code style, design boundaries, and everyday implementation guidance.",
+    skills: [
+      "python-project-structure",
+      "python-design-patterns",
+      "python-code-style",
+      "python-testing-patterns",
+    ],
+  },
+  "python-async": {
+    description:
+      "Python asyncio, concurrent I/O, async API boundaries, and async test strategy.",
+    skills: [
+      "async-python-patterns",
+      "python-project-structure",
+      "python-code-style",
+      "python-testing-patterns",
+    ],
+  },
+  "python-quality": {
+    description:
+      "Python pytest coverage, fixture design, style checks, and refactor-safe quality work.",
+    skills: [
+      "python-testing-patterns",
+      "python-code-style",
+      "python-design-patterns",
+    ],
+  },
 };
 
 const skillActivationInstructions = [
@@ -73,7 +107,31 @@ const skillActivationInstructions = [
   "Do not rely on role name or inherited context alone; scoped AGENTS.md skill routing is part of the task contract.",
 ];
 
+const reviewActivationInstructions = [
+  ...skillActivationInstructions,
+  "Stay read-only unless the parent explicitly asks for fixes.",
+  "Lead with findings, sorted by severity, with precise file/line references.",
+  "Use `simplify` to evaluate whether recently changed code can be clearer, smaller, or less stateful.",
+  "Use `improve-codebase-architecture` only when the review reveals cross-module coupling, shallow-module sprawl, testability friction, or AI-navigation problems worth surfacing.",
+];
+
 const agents = [
+  {
+    name: "code-review",
+    description:
+      "Use for cross-cutting code review centered on bugs, regressions, simplification opportunities, architecture friction, and review-ready findings across changed code.",
+    ownedPaths: ["**/*"],
+    guidanceFiles: ["AGENTS.md"],
+    activationInstructions: reviewActivationInstructions,
+    packs: ["review-core"],
+    skills: ["simplify", "improve-codebase-architecture"],
+    boundaries: [
+      "Review first: identify concrete defects, behavioral risks, missing tests, and simplification opportunities before summarizing.",
+      "Do not edit files unless the parent explicitly assigns implementation after the review.",
+      "Keep architecture notes grounded in observed code friction; return candidate refactors or follow-up RFC recommendations instead of speculative rewrites.",
+      "If the task asks for feature work rather than review, tell the parent to use the relevant implementation specialist.",
+    ],
+  },
   {
     name: "docs-maintenance",
     description:
@@ -248,6 +306,96 @@ const agents = [
       "If the task is mainly HTTP transport or route mounting, tell the parent to use server-transport instead.",
     ],
   },
+  {
+    name: "python-app",
+    description:
+      "Use for Python application work centered on module structure, implementation, code style, maintainability, and tests.",
+    ownedPaths: [
+      "src/**",
+      "packages/**",
+      "services/**",
+      "apps/**",
+      "tests/**",
+      "pyproject.toml",
+      "requirements*.txt",
+      "uv.lock",
+      "poetry.lock",
+    ],
+    guidanceFiles: ["AGENTS.md"],
+    activationInstructions: skillActivationInstructions,
+    packs: ["python-core", "python-quality", "simplify"],
+    skills: [
+      "python-project-structure",
+      "python-design-patterns",
+      "python-code-style",
+      "python-testing-patterns",
+      "simplify",
+    ],
+    boundaries: [
+      "Own ordinary Python implementation, project layout, module boundaries, and nearby tests.",
+      "If the task is mainly asyncio, concurrent I/O, WebSockets, or async API behavior, tell the parent to use python-async instead.",
+      "If the task is mainly test suite design, fixture cleanup, or pytest behavior without feature work, tell the parent to use python-testing instead.",
+      "If the task is mainly TypeScript, frontend, Effect, or auth-specific work, tell the parent to use the matching non-Python specialist.",
+    ],
+  },
+  {
+    name: "python-async",
+    description:
+      "Use for Python async work centered on asyncio, concurrent I/O, async APIs, background tasks, and async test coverage.",
+    ownedPaths: [
+      "src/**",
+      "packages/**",
+      "services/**",
+      "apps/**",
+      "tests/**",
+      "pyproject.toml",
+    ],
+    guidanceFiles: ["AGENTS.md"],
+    activationInstructions: skillActivationInstructions,
+    packs: ["python-async", "python-quality", "simplify"],
+    skills: [
+      "async-python-patterns",
+      "python-project-structure",
+      "python-code-style",
+      "python-testing-patterns",
+      "simplify",
+    ],
+    boundaries: [
+      "Own async Python flows, cancellation, task lifetime, backpressure, concurrent I/O, and async test coverage.",
+      "Keep structural refactors limited to what the async change needs; broader project layout belongs to python-app.",
+      "If the task is synchronous business logic or package structure, tell the parent to use python-app instead.",
+      "If the task is only pytest fixture or assertion work, tell the parent to use python-testing instead.",
+    ],
+  },
+  {
+    name: "python-testing",
+    description:
+      "Use for Python test work centered on pytest, fixtures, integration boundaries, mocks, coverage, and refactor-safe assertions.",
+    ownedPaths: [
+      "tests/**",
+      "**/tests/**",
+      "src/**",
+      "packages/**",
+      "services/**",
+      "apps/**",
+      "pyproject.toml",
+    ],
+    guidanceFiles: ["AGENTS.md"],
+    activationInstructions: skillActivationInstructions,
+    packs: ["python-quality", "simplify"],
+    skills: [
+      "python-testing-patterns",
+      "python-code-style",
+      "python-design-patterns",
+      "simplify",
+    ],
+    boundaries: [
+      "Own pytest structure, fixtures, parameterization, integration-test shape, and test readability.",
+      "Prefer tests that exercise real behavior over brittle mocks unless the boundary is genuinely external.",
+      "If fixing production Python code becomes the main task, tell the parent to use python-app or python-async depending on the failure.",
+      "If test work crosses into TypeScript, frontend, or Effect-specific systems, tell the parent to split that work to the matching specialist.",
+    ],
+  },
 ];
 
 export { agents, capabilityPacks };