Skip to content

fix(app): show reasoning by default#1820

Merged
benjaminshafii merged 2 commits into
devfrom
task/reasoning-default-on
May 16, 2026
Merged

fix(app): show reasoning by default#1820
benjaminshafii merged 2 commits into
devfrom
task/reasoning-default-on

Conversation

@benjaminshafii
Copy link
Copy Markdown
Member

Summary

  • Turn model reasoning visibility on by default for new/local preferences.
  • Render reasoning as normal prose instead of monospace text and strip leading Thinking/Reasoning labels.
  • Add focused coverage for the default visibility and rendered reasoning presentation.

Evidence

Screenshots

No screenshot committed. Browser-plugin UI verification was not completed after the local opencode-browser install path invoked the wrong extension installer; verification used focused render coverage plus Daytona build checks instead.

Build verification

  • Local: bun test apps/app/tests/reasoning-display.test.ts passed.
  • Local: pnpm --filter @openwork/app typecheck passed.
  • Local: pnpm --filter @openwork/app build passed.
  • Daytona sandbox openwork-eval-reasoning-20260515-161659: bun test apps/app/tests/reasoning-display.test.ts && pnpm --filter @openwork/app typecheck && pnpm --filter @openwork/app build passed.

API verification

No API changes.

UI verification

  • Focused SSR render test confirms reasoning is visible by default, does not include a leading Thinking: label, and does not use font-mono.
  • Console errors: not checked in a live browser session.
  • Failed network requests: not checked in a live browser session.

Test instructions

  1. pnpm install
  2. bun test apps/app/tests/reasoning-display.test.ts
  3. pnpm --filter @openwork/app typecheck
  4. pnpm --filter @openwork/app build
  5. Open Settings -> Preferences and verify "Show model reasoning" defaults to on for fresh preferences.
  6. Open a session with model reasoning and verify reasoning appears as prose without a leading "Thinking" heading.

@benjaminshafii
Copy link
Copy Markdown
Member Author

Verification

Completed checks:

  • Local bun test apps/app/tests/reasoning-display.test.ts passed.
  • Local pnpm --filter @openwork/app typecheck passed.
  • Local pnpm --filter @openwork/app build passed.
  • Daytona sandbox openwork-eval-reasoning-20260515-161659 ran bun test apps/app/tests/reasoning-display.test.ts && pnpm --filter @openwork/app typecheck && pnpm --filter @openwork/app build; all passed.

Live browser-plugin UI verification was not completed because the local _repos/opencode-browser install path invoked its own extension installer, which was not the requested OpenWork UI test path.

@vercel
Copy link
Copy Markdown
Contributor

vercel Bot commented May 15, 2026

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
openwork-app Ready Ready Preview, Comment May 16, 2026 4:48am
openwork-den Ready Ready Preview, Comment May 16, 2026 4:48am
openwork-den-worker-proxy Ready Ready Preview, Comment May 16, 2026 4:48am
openwork-landing Ready Ready Preview, Comment, Open in v0 May 16, 2026 4:48am
openwork-share Ready Ready Preview, Comment May 16, 2026 4:48am

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