Skip to content

fix(ai): improve Anthropic reasoning, prefill trimming, and spotlight shortcuts#901

Merged
bobleer merged 1 commit into
GCWing:mainfrom
bobleer:fix/anthropic-reasoning-prefill-and-spotlight-shortcut
May 26, 2026
Merged

fix(ai): improve Anthropic reasoning, prefill trimming, and spotlight shortcuts#901
bobleer merged 1 commit into
GCWing:mainfrom
bobleer:fix/anthropic-reasoning-prefill-and-spotlight-shortcut

Conversation

@bobleer
Copy link
Copy Markdown
Collaborator

@bobleer bobleer commented May 26, 2026

Summary

  • Map ReasoningMode::Enabled to adaptive thinking for Claude adaptive models instead of sending incompatible enabled + budget_tokens payloads.
  • Always attach budget_tokens for enabled Anthropic reasoning on non-adaptive models, including proxy/alias model names.
  • Trim trailing whitespace from final assistant prefills in OpenAI message conversion and remove the newline from <bitfun_contents> prefill to improve Write tool reliability.
  • Register tauri-plugin-global-shortcut and spotlight window capabilities for desktop shortcut support.
  • Restore completed tool item animations in ModelRoundItem by not disabling animation for the completed state.

Test plan

  • cargo test -p bitfun-ai-adapters build_anthropic_request_body
  • cargo test -p bitfun-ai-adapters message_converter
  • cargo test -p bitfun-core round_executor::tests::
  • Verify Write tool prefill no longer leaves a leading blank line in generated file content
  • Verify global shortcuts work for the spotlight window on desktop

… shortcuts

Map ReasoningMode::Enabled to adaptive thinking for Claude adaptive models,
always attach budget_tokens for enabled reasoning on other models, trim
trailing whitespace from assistant prefills for Write reliability, register
global-shortcut capabilities for the spotlight window, and restore completed
tool item animations in ModelRoundItem.
@bobleer bobleer merged commit 3043c5f into GCWing:main May 26, 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