Skip to content

Add bash tool - #16

Draft
barryceelen wants to merge 39 commits into
mainfrom
feature/bash
Draft

Add bash tool#16
barryceelen wants to merge 39 commits into
mainfrom
feature/bash

Conversation

@barryceelen

Copy link
Copy Markdown
Owner

No description provided.

Resolved conflicts:
- api/api.py: combine check_cancelled() from main with iteration limit check from feature/bash
- chat/ask_question.py: use unconditional clear_tool_status() from main
…es section

- Introduce _stream_one_turn as a shared SSE helper and route both
  stream_response and run_with_text_editor_loop through it, so the
  text editor / bash loop now streams text live between tool calls
  instead of dumping each turn at end_turn.
- Reassemble tool_use input from input_json_delta fragments; echo
  full assistant content (text + tool_use + server/web_search blocks)
  back on each round-trip; accumulate usage and sources across turns.
- Render web search results as a foldable ## Sources block appended
  after the answer (defer_to_end), with the handler inserting the
  correct blank-line separator regardless of the streamed tail.
- Insert a newline between run-on sentences ("once!Based") per
  character so the fix works across chunk boundaries and in the
  previously non-streaming agent path.
- Drop the now-unused _request_non_streaming path and the
  insert_after_response_header plumbing.
- Make items clickable
- Update help text
- Update top margin
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