-
Notifications
You must be signed in to change notification settings - Fork 198
pstack: operate PDCA harness when present #186
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Changes from all commits
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,14 @@ | ||
| ### Operate PDCA harness | ||
|
|
||
| **You are Orchestrator.** The shelf is authority. Load the kit contract; do not invent board state. | ||
|
|
||
| Triggered when the workspace has a PDCA harness (`docs/OPERATOR.md`, `docs/HARNESS.md`, or `harness/`) and the user wants delivery advanced, validated, or explained as an operator. | ||
|
|
||
| 1. Read `references/pdca-harness.md`, then `docs/OPERATOR.md` (or `docs/HARNESS.md` / `harness/tree/…` equivalents). | ||
| 2. Throughput checkpoint stays one line: `throughput checkpoint: n/a, harness operator` unless a Feature/Bug fix playbook also applies to product code. | ||
| 3. Match the user ask to an **alias** in `docs/OPERATOR.md` (`plan`, `validate`/`pre-do`, `pin wi-N`, `build`/`do`, `status`). Expand to that file’s **Full commands** block. Echo the expansion in `action:`. | ||
| 4. Run one tick of the expanded station skill (or print the full `/loop` text if they only asked what to run). Spawn Planner / Gate Reviewer / Implementer / Reviewer as `poteto-agent` per the harness roles doc. | ||
| 5. Emit the tick status line the skill requires. Stop on the published signals. | ||
| 6. Apply **unslop** to the reply. | ||
|
|
||
| No nested `/loop` from inside a tick. Opening a PR only when a station or product Feature asks for it. | ||
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,43 @@ | ||
| # PDCA harness detection | ||
|
|
||
| When the workspace has a PDCA harness, poteto-mode is the **preferred Orchestrator**. The harness kit owns the protocol. This file does not duplicate it. | ||
|
|
||
| ## Detect | ||
|
|
||
| Any of these means a harness is present: | ||
|
|
||
| - `docs/OPERATOR.md` | ||
| - `docs/HARNESS.md` | ||
| - `harness/tree/docs/OPERATOR.md` or `harness/tree/docs/HARNESS.md` | ||
| - `.cursor/skills/intake-next/SKILL.md` and `.cursor/skills/build-next/SKILL.md` | ||
|
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Harness detection signals disagreeMedium Severity · Logic Bug The Additional Locations (1)Reviewed by Cursor Bugbot for commit 5d72b40. Configure here. |
||
|
|
||
| Prefer `docs/OPERATOR.md` (or the `harness/tree/…` equivalent). Fall back to `docs/HARNESS.md`. | ||
|
|
||
| ## Load | ||
|
|
||
| 1. Read the operator contract (OPERATOR, else HARNESS). | ||
| 2. Do not invent Plan/Do board moves from chat. | ||
| 3. Advance only via station skills: `/intake-next`, `/build-next`, pin `/build-next wi-N`. | ||
| 4. Before HITL yes, expect `python3 scripts/gate_check.py --gate <gate_id>` (exit 1 → Planner, exit 2 → Gate Reviewer, exit 0 → human yes/no). | ||
| 5. Spawn role agents with `subagent_type: "poteto-agent"` per the harness `docs/AGENT-ROLES.md`. | ||
| 6. Pre-implementation stop is `AWAITING_PIN`. Never pin from intake. Never treat `/loop /poteto-mode` as a station. | ||
|
|
||
| ## Aliases | ||
|
|
||
| If the user says a short name, expand using the **Aliases** table in `docs/OPERATOR.md`, then run the matching **Full commands** block. Echo the expansion in the tick `action:`. | ||
|
|
||
| | Alias | Station | | ||
| | --- | --- | | ||
| | `plan` / `intake` | Plan loop | | ||
| | `validate` / `pre-do` | Pre-Do validation loop | | ||
| | `pin wi-N` | `/build-next wi-N` | | ||
| | `build` / `do` | Do loop | | ||
| | `status` | Read-only shelf report | | ||
|
|
||
| Synonyms (“validate upstream”, “run until pre-implementation”, “advance plan”) use the same rows. Do not invent new Cursor slash skills for aliases. | ||
|
|
||
| ## Route phrases | ||
|
|
||
| If the user asks to run delivery or names an alias without a station slash command, expand via OPERATOR.md and run (or print) that full command. Do not freestyle a parallel backlog. | ||
|
|
||
| If only `/poteto-mode` is attached with no station and the request is clearly delivery, load OPERATOR and either expand `plan`/`validate` or name the alias table. Do not freestyle a parallel backlog. | ||


There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Alias lookup ignores contract fallbacks
High Severity · Logic Bug
The PDCA harness logic has inconsistent expectations for contract file paths. While detection and initial loading support
docs/OPERATOR.md,docs/HARNESS.md, andharness/tree/docs/variants, alias expansion, route phrases, and role agent spawning are more rigid. This prevents alias matching, command expansion, and role agent spawning for validHARNESS.mdorharness/tree/docs/based kits.Additional Locations (1)
pstack/skills/poteto-mode/references/pdca-harness.md#L13-L14Reviewed by Cursor Bugbot for commit 5d72b40. Configure here.