feat: Phase D — config, StreamRenderer, Orchestrator/REPL wiring#50
Merged
Conversation
Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
Renders phase transitions, tool calls, edit plans, colored diffs, validation results, and errors during coding agent runs. Uses raw ANSI codes consistent with the rest of the codebase. Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
- Orchestrator creates CodingAgent from config.agent settings - isCodingTask() detects action verbs (add/fix/implement/etc.) while excluding explanatory prompts (explain/describe/how/etc.) - process() routes coding tasks to CodingAgent before classifier - REPL attaches StreamRenderer for real-time phase/plan/diff output Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
Two safety fixes: - CodingAgent now pauses after PLAN phase when auto_confirm is false, showing the edit plan and asking for confirmation before applying - Planner prompt now instructs LLM to use "create" for new files and only "insert/replace" when modifying existing code, preventing overwrites of existing files with unrelated content Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
…include file content Three fixes for coding agent edit failures: - ANALYZE pre-reads files mentioned in the prompt (e.g. "index.ts") instead of relying on the LLM to decide what to read - Planner normalizes invalid operations (edit_file → replace, write_file → create, etc.) instead of passing them through - Execute step includes actual file content in the prompt so the LLM can generate accurate search strings Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
…ile as content The LLM was putting the whole file in "content" for insert operations instead of just the new lines. Now explicitly documents each operation type's semantics and states "content must NEVER contain the entire file". Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
AgentConfigSchemato config withmax_iterations,auto_confirm,show_plan,run_validation,validation_commandAgentStreamandStreamRendererfor real-time CLI output (phases, plans, diffs, validation, errors)CodingAgentintoOrchestrator—isCodingTask()detects action verbs and routes to coding agent before the classifierStreamRendererinto REPL for streaming output during coding tasksThis is Phase D (final phase) of the v0.3 coding agent plan (Tasks 9–11).
v0.3 Complete
All 5 PRs merged/ready:
Test Plan
npm test)npx tsc --noEmit)npm run build)locode→ type a coding task → verify streaming output🤖 Generated with Claude Code