Skip to content

Treat fast mode as opt-in - #26

Merged
ReganBell merged 1 commit into
yc-software:mainfrom
BinSquare:fix-fast-mode-default
Jul 30, 2026
Merged

Treat fast mode as opt-in#26
ReganBell merged 1 commit into
yc-software:mainfrom
BinSquare:fix-fast-mode-default

Conversation

@BinSquare

@BinSquare BinSquare commented Jul 30, 2026

Copy link
Copy Markdown
Contributor

pi-harness reads turn.fastMode !== false, so a turn that never mentions fastMode runs in fast mode on any fast-capable model. Only the web UI ever sets the field, so every other entry point — CLI, API clients, integrations — leaves it undefined and silently opts in.

Two things make this look unintended rather than deliberate:

  • claude-harness.ts:475 reads the same field as turn.fastMode && …, i.e. opt-in. The two harnesses disagree on the default for one TurnOptions field, so switching harness silently changes which tier a turn bills against.
  • core/turn-options.ts sets NON_INTERACTIVE_FAST_MODE = false for triggered turns and omits the field otherwise — deliberately keeping turns off fast mode, which pi-harness then reads as "on".

This is generated by QM.


View with [code]smith Autofix with [code]smith
Need help on this PR? Tag @codesmith-bot with what you need. Autofix is disabled.

…lled against a tier the organization may have no quota for
@ReganBell
ReganBell merged commit ebc2e10 into yc-software:main Jul 30, 2026
15 checks 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.

2 participants