Tupychka/dev#20
Merged
Merged
Conversation
Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
…ebar on mode switch Two linked regressions: - New-chat default no longer prefers Full Power when the gateway is up. It's key-based again: OpenRouter key set -> Work; no key + OpenClaw installed -> Full Power; otherwise Work. (Also keeps the 'OpenClaw is off' banner hidden for key users, since it's gated on mode === 'execute'.) - POST /chats/:id/mode no longer broadcasts updatedAt for draft chats, so switching mode on a freshly-created draft stops leaking the empty chat into the sidebar (the client upserts a row on any updatedAt). mode still syncs. Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
The toggle only controlled visibility of the model's chain-of-thought (the actual effort knob, thinkingLevel, was never wired up), and behaved the same on/off in practice. Removed across the stack: - UI: header toggle + icon (header-chat-tools.ejs, header-tool-icon.ejs). - Client: change handler, cross-tab sync, turn-reasoning case, render helpers, finalize calls, draft reveal, dead selectors (iclaw.js); CSS rules. - Route: POST /chats/:id/reasoning. - Gateway: the 'reasoning' event type and the whole patchSession (its only caller was that route); analysis/chain-of-thought items are now dropped. - Store/types/DB: setReasoningMode, reasoning_mode field + column, reasoningMode + turn-reasoning protocol types. - Tests: dropped the reasoning-specific cases. Existing DBs keep a harmless unused reasoning_mode column. Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
Changes
Screenshots / GIFs
How to test
Checklist
devnpm run typecheckpassesnpm run buildpassesCHANGELOG.mdunder[Unreleased]if user-visibleROADMAP.mdif a planned item is now done