feat(gate): drop reasoning field from selection (~1.5s) #90

Merged

michal merged 1 commits from feat/gate-drop-reasoning into main 2026-07-25 02:07:31 +00:00

Conversation 0 Commits 1 Files Changed 1 +6 -6

michal commented 2026-07-25 02:07:29 +00:00

Owner

Copy Link

Answer is now just {selectedNames} → ~1.5s (validated live), comfortable under the 8s gate budget. Completes the fast-selection tightening.

Answer is now just {selectedNames} → ~1.5s (validated live), comfortable under the 8s gate budget. Completes the fast-selection tightening.

michal added 1 commit 2026-07-25 02:07:29 +00:00

feat(gate): drop the reasoning field from selection → ~1.5s (comfortable margin) ... 34e00af731

Even "<=8 words" was ignored — the model wrote a full-sentence reasoning, pushing
begin_session to ~7s (thin under the 8s budget). The gate only needs the names, so
request just {"selectedNames":[...]}. Validated live vs vllm-current
(glm-4.6-reap-fast): 1.5s, valid JSON. (reasoning defaults to '' in extractSelection.)

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>

michal merged commit cf85e36ede into main 2026-07-25 02:07:31 +00:00

michal referenced this issue from a commit 2026-07-25 02:07:32 +00:00

Merge pull request 'feat(gate): drop reasoning field from selection (~1.5s)' (#90) from feat/gate-drop-reasoning into main

Sign in to join this conversation.

Reviewers

No Reviewers

Labels

No items

No Label

Milestone

No items

No Milestone

Projects

Clear projects

No project

Assignees

Clear assignees

michal

No Assignees

1 Participants

Notifications

Subscribe

Due Date

No due date set.

Dependencies

No dependencies set.

Reference: michal/mcpctl#90