Skip to content

Add reasoning effort control and selectable messages - #32

Merged
windzu merged 1 commit into
mainfrom
agent/fix-message-selection-and-reasoning-effort
Aug 3, 2026
Merged

Add reasoning effort control and selectable messages#32
windzu merged 1 commit into
mainfrom
agent/fix-message-selection-and-reasoning-effort

Conversation

@windzu

@windzu windzu commented Aug 3, 2026

Copy link
Copy Markdown
Owner

Summary

  • allow user and assistant message text to be selected and copied inside the Obsidian item view
  • add a reasoning-effort control next to the conversation model picker
  • populate effort choices dynamically from each Codex model's supportedReasoningEfforts
  • persist explicit effort choices per conversation and forward them to Codex on every turn
  • clear an incompatible explicit effort when the conversation model changes

Root cause

Message content did not override the non-selectable behavior inherited from Obsidian's application-style view shell. The model catalog integration also discarded supportedReasoningEfforts and defaultReasoningEffort, even though model/list already returned both fields, leaving the composer with model names only.

User impact

Users can select and copy rendered responses. They can also choose the reasoning level supported by the active Codex model; leaving the control on Default uses that model's advertised default.

Validation

  • npm run typecheck
  • npm test (70 tests passed)
  • npm run build
  • queried the installed Codex app-server model/list and verified model-specific effort catalogs
  • deployed the exact build to the local development Vault and verified artifact hashes match

Screenshots

Not attached because the message-selection fix has no static visual delta and the effort menu is rendered by Obsidian's native Menu; the control was validated in the local development Vault.

@windzu
windzu marked this pull request as ready for review August 3, 2026 06:52
@windzu
windzu merged commit 8ffafd1 into main Aug 3, 2026
1 check passed
@windzu
windzu deleted the agent/fix-message-selection-and-reasoning-effort branch August 3, 2026 06:52
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