chore: consolidate harness delegation guidance (#1906)
This commit is contained in:
@@ -1,15 +1,12 @@
|
||||
---
|
||||
description: Update Kandev harness files from session learnings or explicit cross-platform harness requests.
|
||||
argument-hint: "[learning or requested harness change]"
|
||||
allowed-tools: Read Grep Glob Agent
|
||||
allowed-tools: Bash Read Edit Write Grep Glob Agent
|
||||
model: inherit
|
||||
effort: medium
|
||||
---
|
||||
|
||||
Rely on the root `AGENTS.md`/`CLAUDE.md` planner/worker contract and use
|
||||
`.agents/skills/harness-improvement/SKILL.md`. Delegate edits and validation
|
||||
to the native `Agent` tool; do not edit harness files in the primary command.
|
||||
|
||||
First read the relevant bundled reference files under `.agents/skills/harness-improvement/references/`. For platform-specific formats, use `references/platforms/` as the first source of truth and do not browse unless a bundled reference is missing, contradictory, or the user explicitly asks for latest upstream behavior.
|
||||
|
||||
Honor the user's scope. For subagents, update all requested project-local platform mirrors. Do not commit unless explicitly asked.
|
||||
Use `.agents/skills/harness-improvement/SKILL.md`; it owns artifact selection,
|
||||
inventory, bundled-reference routing, and validation. The root
|
||||
`AGENTS.md`/`CLAUDE.md` planner/worker contract applies. Honor the user's scope,
|
||||
update requested subagent mirrors, and do not commit unless explicitly asked.
|
||||
|
||||
@@ -1,22 +1,16 @@
|
||||
---
|
||||
description: Run the Kandev PR fixup loop for CI failures and automated review threads.
|
||||
argument-hint: "<PR number>"
|
||||
allowed-tools: Read Grep Glob Agent
|
||||
allowed-tools: Bash Read Edit Write Grep Glob Agent
|
||||
model: opus
|
||||
effort: high
|
||||
---
|
||||
|
||||
Rely on the root `AGENTS.md`/`CLAUDE.md` planner/worker contract and coordinate
|
||||
`.agents/skills/pr-fixup/SKILL.md` as the primary planner.
|
||||
|
||||
Delegate PR state collection to `pr-poller` and broad remediation to
|
||||
`implementer`. Commit focused fixes through active hooks, then delegate
|
||||
post-commit checks to `verify` before push. Review
|
||||
each compact result and launch another bounded assignment when needed. Do not
|
||||
run GitHub, edit, test, commit, or push commands in this primary session. If a
|
||||
required worker cannot be launched, stop and report the blocked phase.
|
||||
Use `.agents/skills/pr-fixup/SKILL.md`; the root `AGENTS.md`/`CLAUDE.md`
|
||||
planner/worker contract applies. The planner handles small triage and
|
||||
scope-preserving fixes directly; use `pr-poller` only for long waits,
|
||||
`implementer` for broad remediation, and post-commit `verify` before push.
|
||||
|
||||
If `pr-poller` reports that GitHub access requires approval, surface that gate
|
||||
to the user and stop. Do not relaunch polling after approval is denied,
|
||||
cancelled, or interrupted; follow `.agents/skills/pr-fixup/SKILL.md` for the
|
||||
full distinction between approval gates and transient fetch failures.
|
||||
and stop. Do not relaunch after denial, cancellation, or interruption; the
|
||||
shared skill distinguishes approval gates from transient fetch failures.
|
||||
|
||||
@@ -1,21 +1,14 @@
|
||||
---
|
||||
description: Drive Kandev feature work through spec, plan, independent tasks, implementation, QA, and verification.
|
||||
argument-hint: "[feature or fix goal]"
|
||||
allowed-tools: Read Edit Write Grep Glob Agent
|
||||
allowed-tools: Bash Read Edit Write Grep Glob Agent
|
||||
model: inherit
|
||||
effort: high
|
||||
---
|
||||
|
||||
Rely on the root `AGENTS.md`/`CLAUDE.md` planner/worker contract and use
|
||||
`.agents/skills/spec-driven-development/SKILL.md` for Kandev's planner-driven
|
||||
flow:
|
||||
|
||||
1. Clarify intent with `/interview-me` style questions when needed.
|
||||
2. Create or update the product spec with `/spec`.
|
||||
3. Create the implementation plan with `/plan`.
|
||||
4. Decompose into independent tasks with acceptance criteria, exact verification, likely files, dependencies, and wave ordering.
|
||||
5. Delegate every execution step to `implementer`, `test-engineer`, `qa`, `security-auditor`, `code-review`, `simplify`, and `verify` as appropriate. Use `architect` only for a bounded second opinion on unusually risky design decisions.
|
||||
|
||||
Keep the primary planner responsible for orchestration, integration order, user
|
||||
communication, and final status. It must not implement, test, integrate, verify,
|
||||
or ship directly. If a required worker is unavailable, stop and report it.
|
||||
Use `.agents/skills/spec-driven-development/SKILL.md` for the full flow; the
|
||||
root `AGENTS.md`/`CLAUDE.md` planner/worker contract applies. Keep small,
|
||||
localized execution in the planner session. Delegate only substantial
|
||||
independent packets or required independent gates: `qa`, `security-auditor`,
|
||||
and `code-review` are exceptional, while `verify` remains the final
|
||||
post-commit gate. Stop and report an unavailable required worker.
|
||||
|
||||
Reference in New Issue
Block a user