Thinking-capable OpenAI-compatible models can return tagged reasoning before the final polished answer. The cleanup layer now strips only explicit think-tag blocks after response parsing so existing provider requests, UI, and fallback behavior stay unchanged.
Constraint: Issue #25 asks to adapt thinking output rather than disabling model thinking.
Rejected: Add a provider-specific thinking toggle | broader UI and settings change than needed for the bug.
Rejected: Strip localized heading text | too likely to remove normal user-facing content without a stable provider contract.
Confidence: medium
Scope-risk: narrow
Tested: cargo test polish::tests -- --nocapture
Tested: cargo check
Windows regression currently depends on a tester pressing a real modifier key, which cannot distinguish OS-hook delivery failures from coordinator state-machine failures. Add a debug/test-only injection path that drives handle_pressed and handle_released, logs the coordinator hotkey edge, and cancels the dry-run session automatically.
Constraint: Must not rely on SendInput or a physical keyboard for the coordinator gate.
Constraint: Keep physical hotkey testing as the OS hook coverage layer.
Rejected: Fake log-only gate | would not exercise the coordinator hotkey path.
Rejected: Production IPC command | unnecessary surface area for a test/dev-only gate.
Confidence: high
Scope-risk: narrow
Tested: npm run check:hotkey-injection
Tested: npm run build
Tested: cargo check
Tested: cargo check --release
Tested: git diff --check
Related: https://github.com/appergb/openless/issues/35
* Remove fragile space from active app path
The Tauri app lived under a path containing a literal space, which made CI paths and local commands easy to quote incorrectly. Rename the active tree to openless-all and update repository references so scripts, docs, and release workflows use the same space-free path.
Constraint: Preserve the existing app layout and move only the containing directory name.
Rejected: Add compatibility symlinks | would keep two app paths and leave quoting hazards in place.
Confidence: high
Scope-risk: moderate
Tested: npm run build
Tested: cargo clean && cargo check
Tested: git diff --check
Tested: grep -R for old openless -all and openless%20-all references
* Keep Windows build docs aligned with space-free path
The app directory was renamed to openless-all, but the GNU/MinGW route still described the new name as if it contained a space. Update the wording so the rationale matches the current path and keeps the helper-script guidance focused on consistent GNU setup.
Constraint: Documentation-only code review follow-up.
Confidence: high
Scope-risk: narrow
Tested: git diff --check
baiqing