Co-authored-by: Jason Walker <913443@dadeschools.net> Co-committed-by: Jason Walker <913443@dadeschools.net>
This commit was merged in pull request #102.
This commit is contained in:
@@ -337,6 +337,29 @@ touching anything.
|
||||
files, detected secret, or any production/deploy behavior — **stop, report the
|
||||
blocker, and take no mutating action.** Fail closed; never work around a gate.
|
||||
|
||||
## Controller Handoff Summary (required, every task)
|
||||
|
||||
Every task — implementation, review, merge, triage, documentation,
|
||||
discussion-only, or blocked planning — **must end with a
|
||||
`Controller Handoff Summary`** so a controller LLM can pick up the state
|
||||
without rereading the conversation. The canonical format and rules live in the
|
||||
portable skill:
|
||||
[`../skills/llm-project-workflow/SKILL.md`](../skills/llm-project-workflow/SKILL.md) §K.
|
||||
|
||||
Sections (in order): Work performed · Current state (repo, branch/master
|
||||
commit, issue #s, PR #s, complete/blocked/ready-for-review/discussion-only) ·
|
||||
Files changed · Validation · Issues encountered · Review needed? (one of the
|
||||
five fixed answers) · Next recommended action · Safety confirmations
|
||||
(no self-review; no self-merge; no release/tag changes unless requested; no
|
||||
secrets; no production access unless authorized).
|
||||
|
||||
Hard rules: never omit it; never bury blockers earlier only; an opened PR
|
||||
means "Review needed — PR is open"; a blocked merge names the exact gate;
|
||||
discussion-only comments need owner/design feedback, not code review; any
|
||||
touched release state names the exact tag/commit and why. Design debates
|
||||
belong in **discussion/RFC issues** (e.g. #100 `profiles.json v2`) — comment
|
||||
on the issue, create no branches/PRs, and end the comment with this summary.
|
||||
|
||||
## Fail-closed behavior
|
||||
|
||||
Before any mutating action the workflow verifies identity, active profile,
|
||||
|
||||
Reference in New Issue
Block a user