Skip to content

Unify agent prompt handling and update chat input UX#297

Merged
GCWing merged 1 commit intoGCWing:mainfrom
wgqqqqq:fix/unify-agentic-prompt-read-bounds
Mar 30, 2026
Merged

Unify agent prompt handling and update chat input UX#297
GCWing merged 1 commit intoGCWing:mainfrom
wgqqqqq:fix/unify-agentic-prompt-read-bounds

Conversation

@wgqqqqq
Copy link
Copy Markdown
Collaborator

@wgqqqqq wgqqqqq commented Mar 28, 2026

Summary

  • remove the GPT-only agentic prompt template so all models use the default agentic prompt
  • thread current working directory through tool contexts and tighten file path resolution around the active workspace
  • cap Read tool output with lower default line limits plus a total character budget, and surface truncation explicitly
  • include the local chat input UX and locale updates that were already in the working tree

Verification

  • cargo test -p tool-runtime truncates_when_total_char_budget_is_hit
  • cargo test -p tool-runtime reads_multiple_lines_when_budget_allows
  • cargo test -p bitfun-core always_uses_default_prompt_template
  • cargo check -p bitfun-desktop

Notes

  • This PR intentionally includes all local changes that were present in the worktree, including the web UI input updates.

@wgqqqqq wgqqqqq force-pushed the fix/unify-agentic-prompt-read-bounds branch from df5913d to bc69130 Compare March 30, 2026 01:05
@GCWing GCWing merged commit fc2d527 into GCWing:main Mar 30, 2026
4 checks passed
wsp1911 added a commit to wsp1911/BitFun that referenced this pull request Apr 7, 2026
…Context

This removes the current_working_directory field introduced in GCWing#297.
In the current implementation it overlaps with workspace semantics and is not
meaningfully maintained in the main execution flow, so keeping it adds API and
path-resolution complexity without real value.

- remove current_working_directory from ToolUseContext
- remove cwd-based path resolution branches
- update tool context builders and tool validation paths
- simplify related tests to workspace-root-only behavior
wsp1911 added a commit that referenced this pull request Apr 7, 2026
…Context

This removes the current_working_directory field introduced in #297.
In the current implementation it overlaps with workspace semantics and is not
meaningfully maintained in the main execution flow, so keeping it adds API and
path-resolution complexity without real value.

- remove current_working_directory from ToolUseContext
- remove cwd-based path resolution branches
- update tool context builders and tool validation paths
- simplify related tests to workspace-root-only behavior
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