Skip to content

Commit 6959152

Browse files
fix: add --auto medium to droid exec and require ExitSpecMode before saving
Background droid exec calls failed for Opus because no --auto flag was set, causing permission denial in non-interactive subprocesses. Also, the agent instructions allowed saving specs directly without user review -- now requires ExitSpecMode approval before writing to specs/active/.
1 parent 28c686a commit 6959152

File tree

2 files changed

+13
-7
lines changed

2 files changed

+13
-7
lines changed

plugins/rfspec/skills/rfspec/SKILL.md

Lines changed: 4 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -29,11 +29,13 @@ Fan out a prompt to multiple models, compare their responses, and help the user
2929
1. Run `/rfspec <user's prompt>` -- fires parallel model calls, returns labeled options (A, B, C).
3030
2. Evaluate the results -- see [references/evaluation-guide.md](references/evaluation-guide.md).
3131
3. Present the choice to the user via AskUser.
32-
4. Save the selected or synthesized result.
32+
4. Present the selected or synthesized spec via ExitSpecMode for user review.
33+
5. Save to `specs/active/` only after the user approves in spec mode.
3334

3435
## Saving
3536

36-
Save the final result (picked or synthesized) to:
37+
**Do not save immediately.** After the user picks or synthesis is complete, present the
38+
final spec via ExitSpecMode for review. Only after approval, save to:
3739

3840
```
3941
specs/active/YYYY-MM-DD-<slug>.md

plugins/rfspec/skills/rfspec/scripts/run.sh

Lines changed: 9 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -27,11 +27,11 @@ MODEL_B="gpt-5.4"; LABEL_B="GPT-5.4"; RE_B="xhigh"
2727
MODEL_C="gemini-3.1-pro-preview"; LABEL_C="Gemini 3.1 Pro"; RE_C="high"
2828

2929
# ── fire all three in parallel ───────────────────────────────────────
30-
droid exec -m "$MODEL_A" -r "$RE_A" -f "$TMPDIR/prompt.md" -o json 2>/dev/null > "$TMPDIR/a.json" &
30+
droid exec -m "$MODEL_A" -r "$RE_A" --auto medium -f "$TMPDIR/prompt.md" -o json 2>/dev/null > "$TMPDIR/a.json" &
3131
PID_A=$!
32-
droid exec -m "$MODEL_B" -r "$RE_B" -f "$TMPDIR/prompt.md" -o json 2>/dev/null > "$TMPDIR/b.json" &
32+
droid exec -m "$MODEL_B" -r "$RE_B" --auto medium -f "$TMPDIR/prompt.md" -o json 2>/dev/null > "$TMPDIR/b.json" &
3333
PID_B=$!
34-
droid exec -m "$MODEL_C" -r "$RE_C" -f "$TMPDIR/prompt.md" -o json 2>/dev/null > "$TMPDIR/c.json" &
34+
droid exec -m "$MODEL_C" -r "$RE_C" --auto medium -f "$TMPDIR/prompt.md" -o json 2>/dev/null > "$TMPDIR/c.json" &
3535
PID_C=$!
3636

3737
FAIL=""
@@ -85,5 +85,9 @@ echo "- Use Option B (${LABEL_B}) as-is"
8585
echo "- Use Option C (${LABEL_C}) as-is"
8686
echo "- Synthesize a refined spec combining the best of all three"
8787
echo "- No -- none of these work (explain why)"
88-
echo "If the user picks synthesis, combine the strongest elements and save"
89-
echo "to specs/active/YYYY-MM-DD-<slug>.md. If rejected, gather feedback."
88+
echo ""
89+
echo "CRITICAL: Do NOT save the spec directly. After the user picks an option"
90+
echo "or requests synthesis, use the ExitSpecMode tool to present the final"
91+
echo "spec content for review. Only save to specs/active/YYYY-MM-DD-<slug>.md"
92+
echo "AFTER the user approves the spec in spec mode. If rejected, gather"
93+
echo "feedback and revise."

0 commit comments

Comments
 (0)