Grilling
Implement Spec Execution Policy Grill Status
Implement Spec Execution Policy Grill Status
Phase State
- Topic:
implement-specexecution throughput and assurance policy. - Scope boundary: requirements only; no shared-skill implementation.
- Shared-understanding confirmation: confirmed.
Branch Dashboard
| Branch | Completion | Locked direction | Still-open items |
|---|---|---|---|
| Task scheduling | 100% | Release a dependent after its prerequisite passes task-local RED/GREEN, relevant typecheck and lint, parent acceptance, and Active Write Scope checks. Verification joins the Task Gate only when the dependent consumes its visible result. Shared bookkeeping may follow dispatch but precedes any Architecture Checkpoint or finalization. | None. |
| Worker ownership | 100% | Workers write in the current branch under disjoint active write scopes. No worktrees or merger agents. | None. |
| Task context | 100% | Keep a small inline execution kernel; use durable Context Pointers for specification detail, code symbols, architecture views, and exact skill guidance. | None. |
| Assurance and architecture | 100% | Keep applicable TDD, runtime, UI, provider, acceptance, and architecture proof. Only declared Architecture Checkpoints are hard barriers. | None. |
| Pull request lifecycle | 100% | Create or reuse the draft PR after the branch/base gate and first meaningful pushed commit; never create an empty commit only for the PR. | None. |
| Review cost | 100% | implement-spec completes implementation and Verification first. The subsequent delivery-phase review step runs one Full Code Review Pass by default with one comprehensive primary reviewer and explicit five-axis coverage, then uses focused repair checks and affected Verification reruns; bounded independent risk-focused challenge may be added when useful and a second pass is risk-triggered. | Detailed topology is authoritative in the Delivery Phase Flow Optimization specification. |
Technical Grounding
| Branch | Evidence anchors | Applicable dimensions | Open technical decisions | Grounding |
|---|---|---|---|---|
| Task scheduling | .agents/skills/implement-spec/references/parallel-orchestration.md:Step 3; implement-spec-execution-policy-grill-log.md:Q8; project-verification capability SPEC.md:AC-001 | scheduler topology, dependency release, Verification timing, shared artifact persistence | none | grounded |
| Worker ownership | .agents/skills/implement-spec/SKILL.md:Branch and PR invariant; .agents/skills/implement-spec/references/parallel.md:Contract | ownership, write isolation, integration seam | none | grounded |
| Task context | .agents/skills/create-plan/references/plan-schema.md:Task contract; .agents/skills/implement-spec/references/parallel-worker-brief.md:Required Context; project-verification capability SPEC.md:AC-033 | inline execution kernel, context boundary, progressive disclosure, pointer freshness | none | grounded |
| Assurance and architecture | .agents/skills/implement-spec/references/lifecycle.md:Shared execution invariants; .agents/skills/implement-spec/references/architecture-conformance.md:Cumulative conformance checkpoint | test, runtime, UI, provider, architecture, acceptance | none | grounded |
| Pull request lifecycle | .agents/skills/implement-spec/SKILL.md:Branch and PR invariant; stack-workflow-grill-status.md:Implementation Branching Policy | branch/base intent, stack metadata, meaningful pushed state, PR visibility | none | grounded |
| Review cost | .agents/skills/delivery-phase/phases/review.md:Prepare Review; .agents/skills/review-phase/phases/run-review.md:Review Execution; project-verification capability SPEC.md:US-007, AC-027, AC-028 | Verification and Code Review authority, phase order, frozen target, retained evidence, repair validation, pass budget | none | grounded |
Current Round
- Round: 2026-09-05 review-topology follow-up (umbrella R6)
- Prior round: R1 (closed)
- Current frontier: empty
- Shared-understanding confirmation: confirmed
| Question id | Prerequisites | Question | State |
|---|---|---|---|
| Q8 | Q1, Q2 | What exact evidence releases a dependent task, and may plan/notes bookkeeping finish after release? | answered |
| Q9 | Q3, Q4, Q5 | Which task fields stay inline, and what may become a Context Pointer? | answered |
| Q10 | Q6 | At which durable branch checkpoint should the draft PR be created? | answered |
| Q11 | Q7, Q4, Q5 | After implement-spec completes, how many Full Code Review Passes does the subsequent delivery-phase review step run, and what happens after review-triggered repairs? | answered |
| Q12 | Q1, Q2 | What planning information must constrain concurrent evidence when write scopes are disjoint? | answered |
| Q13 | Q3 | When may a compact brief retain derived context rather than only a pointer? | answered |
| Q14 | Q8 | When may diagnosis continue before another repair or handback? | answered |
| Q15 | Q11 | How are interrupted, invalidated, and after-pass review attempts counted and routed? | answered |
Parked Branches
- Isolated worker worktrees and merger agents are rejected for the current model. Reconsider only if shared-branch isolation proves inadequate with disjoint write scopes.
- External GitHub or Codex PR reviewer integration remains outside this grill.
Glossary
Terms
- Execution Frontier: Tasks whose dependencies have passed, whose required write scopes are available, and whose declared read, write, and shared-runtime conflicts permit release now.
- Task Gate: The smallest validation and parent check that makes one completed task safe for its dependents to consume.
- Architecture Checkpoint: A declared cumulative barrier for ownership, dependency, public-seam, responsibility, and migration conformance.
- Active Write Scope: Paths exclusively owned by one currently running worker in the shared branch.
- Context Pointer: A stable reference to authoritative spec, plan, code, test, or skill content that avoids copying that content into the task brief.
- Full Code Review Pass: One complete
review-phaserun over a frozen change with all mandatory Code Review lenses. It does not run the product or perform Verification. Avoid: full review. - Focused Repair Validation: Checks tied to accepted findings and the surfaces changed by their repairs.
- Verification: Exercise observable product behavior through a real user path and retain evidence of the action, result, and applicable side effects.
- Project Verifier: Project-owned app-specific executable knowledge consumed through the single
verify-behaviorentrypoint.
Relationships
- The Execution Frontier is recomputed after each Task Gate result.
- A task enters the Execution Frontier only when all dependencies pass their Task Gates, its Active Write Scope is available, and its declared read, write, and shared-runtime conflicts permit release.
- An Architecture Checkpoint or declared relevant-input conflict may block otherwise eligible tasks.
implement-specperforms Verification throughverify-behaviorand the Project Verifier, completes implementation, and returns control todelivery-phase.- The subsequent
delivery-phasereview step invokesreview-phasefor a Full Code Review Pass over the frozen implemented change. review-phasemay inspect retained Verification evidence, but it does not run or refresh Verification.- Focused Repair Validation may replace another Full Code Review Pass only under an accepted review policy.
Axioms
- Unrelated slow work does not delay an otherwise safe dependent task.
- Concurrent workers write in one current branch and never share active write scope.
- Architecture, runtime, UI, provider, and acceptance proof remain when applicable.
- Shared artifacts remain authoritative even when their update is moved off the dependency-release critical path.
- An eager draft PR must preserve accepted branch/base and stack intent.
- Verification and Code Review remain separate gates with separate evidence.
Flagged Ambiguities
- None. Ordinary scheduling uses Execution Frontier with declared conflict checks; cumulative architecture barriers use Architecture Checkpoint.
- A passed Task Gate can release dependents. Shared-artifact reconciliation and final acceptance remain later completion states.
- Full Code Review Pass means only one
review-phaserun afterimplement-spec; it never means Verification.
Accepted Review Policy
implement-spec first completes implementation, task checks, and Verification through verify-behavior and the Project Verifier. It then returns control to delivery-phase. The subsequent review step invokes review-phase against the frozen implemented change.
Accepted policy:
- Run one Full Code Review Pass in the subsequent
delivery-phasereview step. - Repair accepted findings.
- Run Focused Repair Validation for those findings and rerun only Verification scenarios invalidated by the repair.
- Do not automatically run a second Full Code Review Pass.
- Run one second Full Code Review Pass only when the repair changes architecture, security or authorization, a public contract, deployment or runtime topology, or accepted scope.
- Never run more than two Full Code Review Passes in one delivery goal without explicit human direction.
Recommended Next Direction
Compile a focused agent-ready specification for the accepted execution-policy change. Backlog projection and implementation planning remain downstream actions.
Accepted Follow-Up: Review Topology Alignment (2026-09-05)
- A Full Code Review Pass uses one comprehensive primary reviewer with explicit coverage of all five mandatory axes; it does not require five separate review workers.
autoreviewis the primary reviewer and no extra advisory scan is required. One independent risk-focused challenger is mandatory for a bounded independent risk area; additional challengers may be used when larger changes require separate independent risk areas.- Primary and challenger reviewers consume the same frozen factual packet. Challengers complete their independent checks before consuming primary conclusions.
- The parent adjudicates findings and owns the report. Per-axis gaps or incomplete challenger work are incomplete and cannot be clean.
- Detailed topology is authoritative in the Delivery Phase Flow Optimization specification.
- Existing one-pass default, focused repairs, affected Verification reruns, risk-triggered second pass, and two-pass ceiling remain unchanged. No experiment or admission-policy change is added.
Follow-up Accepted Decisions (2026-09-05)
| Branch | Completion | Locked direction | Still-open items |
|---|---|---|---|
| Concurrent evidence | 100% | The swarm-planner primitive of the create-plan step of delivery-phase records Read Dependencies and Shared Runtime Resources alongside disjoint Active Write Scopes. Frontier release respects declared read/write/runtime conflicts; relevant-input changes invalidate affected evidence. This remains a planning and evidence rule inside the native harness. | None. |
| Context disclosure | 100% | Bounded source-attributed derived context excerpts or rationale may accompany a resolvable Context Pointer and freshness marker when they avoid predictable lookups. The authority source remains controlling. | None. |
| Recovery diagnosis | 100% | Authorized diagnosis may gather discriminating evidence before another repair. Handback occurs at changed requirements, weaker gates, out-of-bounds decision/access, or exhausted useful diagnosis; stagnant repair loops do not repeat. | None. |
| Review attempt accounting | 100% | During-pass semantic invalidation requires a fresh frozen-target pass. After-pass ordinary repairs use focused validation and affected Verification reruns. Interrupted or invalidated attempts are incomplete and do not consume the completed-pass allowance; the two-pass ceiling remains. | None. |
Final Handoff
- Shared understanding confirmed on 2026-09-04.
- Every active branch is closed at 100%; no technical decision remains open.
- External PR reviewer integration remains parked outside this capability.
- Requirements are ready for
create-spec. - Follow-up decisions Q12-Q15 were accepted on 2026-09-05. Experiments remain unchanged and were not added to the specification.