fix: enforce fresh context between molecule steps
Some checks failed
CI / Check for .beads changes (push) Has been skipped
CI / Check embedded formulas (push) Failing after 18s
CI / Test (push) Failing after 1m32s
CI / Lint (push) Failing after 20s
CI / Integration Tests (push) Successful in 1m14s
CI / Coverage Report (push) Has been skipped
Windows CI / Windows Build and Unit Tests (push) Has been cancelled
Some checks failed
CI / Check for .beads changes (push) Has been skipped
CI / Check embedded formulas (push) Failing after 18s
CI / Test (push) Failing after 1m32s
CI / Lint (push) Failing after 20s
CI / Integration Tests (push) Successful in 1m14s
CI / Coverage Report (push) Has been skipped
Windows CI / Windows Build and Unit Tests (push) Has been cancelled
Change molecule step completion instructions to use `gt mol step done` instead of `bd close`. This ensures polecats get fresh context between each step, which is critical for multi-step review workflows like shiny-enterprise where each refinement pass should have unbiased attention. The `gt mol step done` command already: 1. Closes the step 2. Finds the next ready step 3. Respawns the pane for fresh context But polecats were being instructed to use `bd close` directly, which skipped the respawn and let them run through entire workflows in a single session with accumulated context. Updated: - prime_molecule.go: step completion instructions - mol-polecat-work.formula.toml - mol-polecat-code-review.formula.toml - mol-polecat-review-pr.formula.toml Fixes: hq-0kx7ra
This commit is contained in:
@@ -88,9 +88,9 @@ func showMoleculeExecutionPrompt(workDir, moleculeID string) {
|
||||
fmt.Println(style.Bold.Render("→ EXECUTE THIS STEP NOW."))
|
||||
fmt.Println()
|
||||
fmt.Println("When complete:")
|
||||
fmt.Printf(" 1. Close the step: bd close %s\n", step.ID)
|
||||
fmt.Println(" 2. Check for next step: bd ready")
|
||||
fmt.Println(" 3. Continue until molecule complete")
|
||||
fmt.Printf(" gt mol step done %s\n", step.ID)
|
||||
fmt.Println()
|
||||
fmt.Println("This closes the step and respawns your session with fresh context for the next step.")
|
||||
} else {
|
||||
// No next step - molecule may be complete
|
||||
fmt.Println(style.Bold.Render("✓ MOLECULE COMPLETE"))
|
||||
@@ -162,11 +162,10 @@ func outputMoleculeContext(ctx RoleContext) {
|
||||
showMoleculeProgress(b, rootID)
|
||||
|
||||
fmt.Println()
|
||||
fmt.Println("**Molecule Work Loop:**")
|
||||
fmt.Println("1. Complete current step, then `bd close " + issue.ID + "`")
|
||||
fmt.Println("2. Check for next steps: `bd ready --parent " + rootID + "`")
|
||||
fmt.Println("3. Work on next ready step(s)")
|
||||
fmt.Println("4. When all steps done, run `gt done`")
|
||||
fmt.Println("**When step complete:**")
|
||||
fmt.Println(" `gt mol step done " + issue.ID + "`")
|
||||
fmt.Println()
|
||||
fmt.Println("This closes the step and respawns with fresh context for the next step.")
|
||||
break // Only show context for first molecule step found
|
||||
}
|
||||
}
|
||||
|
||||
Reference in New Issue
Block a user