Skip to content

Job_chat: remove redundant job code edits#467

Merged
josephjclark merged 3 commits intorelease/nextfrom
dedupe-apply-edits
Apr 21, 2026
Merged

Job_chat: remove redundant job code edits#467
josephjclark merged 3 commits intorelease/nextfrom
dedupe-apply-edits

Conversation

@hanna-paasivirta
Copy link
Copy Markdown
Contributor

Short Description

Removes unnecessary second apply_code_edits in job_chat.

Implementation Details

The streaming path applies edits mid-stream to emit suggested_code in the changes event, but the post-stream path was then calling parse_and_apply_edits in all scenarios and redoing the same edits. Now we cache the streaming-resolved result on self and skip the post-stream apply when it's already been run. The non-streaming path still falls through to parse_and_apply_edits as before.

AI Usage

Please disclose how you've used AI in this work (it's cool, we just want to know!):

  • Code generation (copilot but not intellisense)
  • Learning or fact checking
  • Strategy / design
  • Optimisation / refactoring
  • Translation / spellchecking / doc gen
  • Other
  • I have not used AI

You can read more details in our Responsible AI Policy

@josephjclark josephjclark changed the base branch from main to release/next April 21, 2026 09:26
@josephjclark josephjclark merged commit 3c4a688 into release/next Apr 21, 2026
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.

2 participants