fix: Distiller agent stdout output — eliminate idea.md filename ambiguity#535
fix: Distiller agent stdout output — eliminate idea.md filename ambiguity#535colehurwitz wants to merge 3 commits into
Conversation
Closes #522 Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
✅ Factory Review: KEEPVerdict: KEEP Experiment: #21 Score Comparison
Guard Checks
Precheck GateCode Review Notes
Posted by Factory CEO |
✅ Factory Review: KEEPVerdict: KEEP Experiment: #21 Score Comparison
Guard Checks
Posted by Factory CEO |
CEO Review — Experiment #21Verdict: ✅ KEEP SummaryPrompt engineering fix across two files ( Closes #522 Changes
Root CauseThe Distiller is the only factory agent expected to output exclusively to stdout. Every other agent writes to explicit file paths (Strategist →
Review Pipeline
Score
No dimension regressions. All 12 dimensions passed. |
PR #530 merged the Distiller into the Strategist. Apply the stdout anti-instruction fix (from #522) to the Strategist's ideation output section instead. Resolve ceo.md conflicts by accepting main's "build plan" wording with our "Print to stdout" fix applied. Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
…n format Fixes 6 CI failures caused by new features merged from main: - test_agents: accept **kwargs in parallel mock functions for review_tag param - test_agents: skip --review-tag patterns in background spawn detection - test_ceo_completion: update research filenames to research-failures/priorart - test_prompts: update strategist ideation assertions for build-plan format Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## main #535 +/- ##
==========================================
+ Coverage 87.92% 87.94% +0.02%
==========================================
Files 70 70
Lines 10120 10180 +60
==========================================
+ Hits 8898 8953 +55
- Misses 1222 1227 +5 ☔ View full report in Codecov by Harness. 🚀 New features to boost your workflow:
|
|
@xukai92 could you review and merge? Pretty small but useful change |
Factory experiment 21. Fixes the Distiller agent writing specs to a file instead of stdout by removing ambiguous idea.md references from distiller.md and ceo.md prompts.
Closes #522
Changes