Skip to content

feat(agentic): route PlaintextFollowup Write through post-write Read#969

Merged
bobleer merged 1 commit into
GCWing:mainfrom
bobleer:feat/plaintext-followup-post-write-read
May 30, 2026
Merged

feat(agentic): route PlaintextFollowup Write through post-write Read#969
bobleer merged 1 commit into
GCWing:mainfrom
bobleer:feat/plaintext-followup-post-write-read

Conversation

@bobleer
Copy link
Copy Markdown
Collaborator

@bobleer bobleer commented May 30, 2026

Summary

  • PlaintextFollowup Write no longer echoes generated file bodies in tool results or persisted assistant tool calls; the runtime injects a synthetic Read after each generated Write so the next model round continues from normal Read output.
  • Fail fast before slow Write content generation when Read is unavailable or unregistered, and align agent prompts plus tool-loop/truncation guidance to prefer Read + Edit over repeated Write on the same path.
  • Resolve auto/default model selectors to primary (remove short first-turn fast-model shortcut) and track Write stats via lines_written.

Test plan

  • cargo test -p bitfun-core round_executor -- --nocapture
  • Manual PlaintextFollowup flow: Write(file_path) → content generation → Write execution → auto Read → Edit from Read result
  • Confirm ACP/InlineContent Write path is unchanged (inline content still in tool call args)
  • Confirm auto/default model selection resolves to primary model

Stop echoing generated file bodies in Write tool results and session history.
Inject a synthetic Read after each PlaintextFollowup Write, fail fast before
slow content generation when Read is unavailable, and align prompts and loop
guidance to continue edits from Read output. Also resolve auto/default model
selectors to primary and drop the short-first-turn fast model shortcut.
@bobleer bobleer merged commit 57bbc9a into GCWing:main May 30, 2026
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant