@dirtydishes/skills 0.1.2 → 0.1.4

package/skills/dirtyloops/templates/workflows/orchestrator-callback/implementation-thread-prompt.md.template

@@ -15,19 +15,28 @@ You must send your final callback to the exact orchestrator thread id `{{ORCHEST
 - Implementation index: `{{IMPLEMENT_MD}}`
 - Turn doc: `{{TURN_DOC}}`
 - Branch/worktree instructions: `{{BRANCH_WORKTREE_INSTRUCTIONS}}`
-- Thread default: standard speed/reasoning unless the orchestrator prompt explicitly says otherwise
+- Launch defaults: speed `standard`, reasoning `high`, inherit_orchestrator_thread_settings `false`, unless Beads metadata explicitly overrides the implementation actor
+- Do not run this worker with inherited UI, recent thread, parent/orchestrator, `fast`, or `xhigh` settings unless Beads metadata explicitly overrides the implementation actor
 
 ## Rules
 
 - Implement exactly this phase.
 - Do not widen scope.
-- Use bounded implementation/scout subagents when useful.
+- Do not do broad solo implementation.
+- For non-trivial phases, run massive bounded swarms before broad edits:
+  - 8-20 scout agents across files, modules, risks, tests, and integration edges
+  - 8-16 slice-plan agents to propose non-overlapping implementation slices
+  - 8-16 implementation-helper agents assigned to slices in parallel
+- If this phase is tiny enough to use fewer than the default agents, record the reason in the turn doc and callback `swarm_summary`.
+- Synthesize swarm reports into one coherent slice plan before broad edits.
+- Integrate helper output yourself; own conflicts, final repairs, gates, branch state, PR state, and callback.
 - Update the existing Markdown turn doc.
 - Run local gates before PR when feasible.
 - Open or update the PR.
 - Do not create the review thread.
 - Call back exactly once to the orchestrator.
 - The callback payload must include `orchestrator_thread_id: "{{ORCHESTRATOR_THREAD_ID}}"` and your `source_thread_id`.
+- The callback payload must include `swarm_summary` with scout, slice-plan, implementation-helper, slice, synthesis, and under-default rationale fields.
 
 ## Callback States
 

package/skills/dirtyloops/templates/workflows/orchestrator-callback/review-thread-prompt.md.template

@@ -22,12 +22,13 @@ Use:
 - PR: `{{PR_URL_OR_ID}}`
 - Branch/commit: `{{BRANCH_OR_COMMIT}}`
 - Required gates: `{{QUALITY_GATES}}`
-- Thread default: standard speed/reasoning unless the orchestrator prompt explicitly says otherwise
+- Launch defaults: speed `standard`, reasoning `high`, inherit_orchestrator_thread_settings `false`, unless Beads metadata explicitly overrides the review actor
+- Do not run this reviewer with inherited UI, recent thread, parent/orchestrator, `fast`, or `xhigh` settings unless Beads metadata explicitly overrides the review actor
 
 ## Rules
 
 - Be ambitious about structural simplification.
-- Use reviewer and CI verification subagents when useful.
+- Use reviewer and CI verification subagents for review and CI evidence. If the phase is tiny enough to use fewer than the default review/CI agents, record why in the turn doc.
 - Own CI through green, repaired-and-green, unavailable-with-evidence, or blocked-with-cause.
 - Apply safe in-scope repairs when assigned by the orchestrator prompt.
 - Update the existing Markdown turn doc.

package/skills/dirtyloops/templates/workflows/orchestrator-callback/run-loop-addendum.md.template

@@ -3,9 +3,15 @@
 This loop uses a main orchestrator thread plus separate implementation and review threads.
 
 - Only the orchestrator creates implementation and review threads.
+- Before the first worker, the orchestrator launches a selector subagent to choose exactly one next ready Beads phase.
+- After every review callback, the orchestrator launches a closeout-selector subagent before closing the phase, stopping, or launching another worker.
+- Selector and closeout-selector subagents return `swarm-report` payloads with mission `selector` or `closeout-selector`; they do not mutate Beads, edit files, implement, review, repair, create threads, or advance the loop.
+- Selector, implementation, review, and closeout-selector threads launch with explicit actor defaults: `speed: standard`, `reasoning: high`, and `inherit_orchestrator_thread_settings: false`, unless Beads metadata overrides that actor.
+- Do not inherit current UI settings, recent Codex defaults, or orchestrator-thread speed/reasoning. `fast` speed and `xhigh` reasoning require explicit Beads metadata for the actor being launched.
 - The orchestrator passes its concrete thread id as the callback target.
 - Before launching a worker or reviewer, verify the actual prompt text includes that exact thread id and does not rely on generic wording like `current orchestrator thread` or `this thread`.
-- Worker and reviewer threads default to standard speed/reasoning unless Beads loop metadata explicitly overrides it.
+- After launching a worker or reviewer, wait for its single callback. Do not repeatedly read, monitor, summarize, or status-check the child thread while waiting.
+- Use a sparse fallback heartbeat around 30 minutes only when the callback is overdue or liveness is uncertain; do not use heartbeat as routine polling.
 - Implementation threads update the existing phase turn doc, open the PR, and call back exactly once when PR-ready or blocked.
 - Implementation callback payloads must echo `orchestrator_thread_id` and include `source_thread_id`.
 - Implementation threads do not create review threads.
@@ -13,5 +19,6 @@ This loop uses a main orchestrator thread plus separate implementation and revie
 - Review threads own CI, repairs, reruns, and evidence.
 - Review threads update the existing phase turn doc and call back exactly once when review and CI are resolved.
 - Review callback payloads must echo `orchestrator_thread_id` and include `source_thread_id`.
-- The orchestrator owns Beads closeout and next selector launch.
-- Use a fallback heartbeat around 30 minutes for long-running threads.
+- The orchestrator validates selector reports, owns Beads closeout, and launches only the selected next phase.
+- Implementation threads are swarm-first for non-trivial phases: 8-20 scout agents, 8-16 slice-plan agents, and 8-16 implementation-helper agents before broad edits.
+- The implementation thread synthesizes swarm reports into the existing turn doc, integrates helper output coherently, and includes `swarm_summary` in the implementation callback.

package/skills/dirtyloops/templates/workflows/orchestrator-callback/selector-subagent-prompt.md.template

@@ -0,0 +1,35 @@
+# Selector Subagent Prompt: {{STREAM_NAME}}
+
+You are the selector subagent for an `orchestrator-callback` dirtyloop.
+
+Canonical tracker: Beads epic `{{EPIC_ID}}`
+
+Mission: select at most one next ready Beads phase for the orchestrator.
+
+Launch defaults:
+
+- speed: `standard`
+- reasoning: `high`
+- inherit_orchestrator_thread_settings: `false`
+
+Do not run this selector with inherited UI, recent thread, parent/orchestrator, `fast`, or `xhigh` settings unless Beads metadata explicitly overrides the selector actor.
+
+Read:
+
+- Beads epic `{{EPIC_ID}}`
+- Beads loop metadata
+- `docs/implementation/{{STREAM_SLUG}}/IMPLEMENT.md`
+- `docs/implementation/{{STREAM_SLUG}}/loop-state.md`
+- linked phase docs for ready or nearly-ready child issues
+
+Rules:
+
+- Do not implement.
+- Do not review.
+- Do not mutate Beads.
+- Do not edit files.
+- Do not create implementation or review threads.
+- Select exactly one phase only when Beads says it is ready and the linked phase doc is coherent.
+- If no phase is ready or the state is ambiguous, report `blocked` with evidence.
+
+Return a compact report matching `schemas/swarm-report.schema.json` with `mission: "selector"`.

package/skills/dirtyloops/templates/workflows/single-thread-subagent/run-loop-addendum.md.template

@@ -3,10 +3,11 @@
 This loop runs in one visible Codex coordinator thread. Single-thread means one coordinator, not solo implementation.
 
 - Do not create separate Codex worker or reviewer threads.
-- Use selector, scout, slice-planning, implementation-helper, reviewer, and CI subagent swarms inside this thread.
-- Use 8+ subagents by default for broad scout, slice, implementation-helper, and review work. If fewer than 8 are used for a broad phase, record why in the turn doc.
-- Before broad implementation, run a slice planning swarm and synthesize the slice plan into the turn doc.
+- For every phase, use selector, scout, slice-planning, implementation-helper, reviewer, and CI subagent swarms inside this thread.
+- Default per-phase sizes: selector 4-8 agents, scout 8-20 agents, slice-plan 8-16 agents, implementation-helper 8-16 agents, reviewer 8-20 agents, and CI verification 4-12 agents.
+- If a phase is tiny enough to use fewer than the default per-phase swarms, record why in the turn doc before implementation.
+- Before implementation, run a slice planning swarm and synthesize the slice plan into the turn doc.
 - The main thread owns integration, branch state, PR state, Beads updates, and closeout.
 - Subagents inspect, compare, critique, split the phase into slices, prepare implementation guidance or patches, verify, and report.
 - Subagents do not own loop state.
-- The coordinator may implement glue, integration, conflict resolution, and final repairs. It must not do most broad discovery or implementation itself.
+- The coordinator may implement glue, integration, conflict resolution, and final repairs. It must not do most discovery or implementation itself.