Repository navigation
fix(server): thread split suggestions catch more new tasks - #66
Draft
ScriptedAlchemy wants to merge 4 commits into
Draft
ScriptedAlchemy wants to merge 4 commits into
ScriptedAlchemy wants to merge 4 commits into
Conversation
|
ScriptedAlchemy
force-pushed
the
fix/thread-split-contextual-prompt
branch
from
November 5, 2026 05:02
9fe22cb to
b51f356
Compare
The shipped prompt and 0.94 threshold caught 1 of 33 new tasks on a held-out Claude and Codex test. Use the contextual prompt at 0.6538, and skip drafts under five words, which caught 8 of 33. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
ScriptedAlchemy
force-pushed
the
fix/thread-split-contextual-prompt
branch
from
November 5, 2026 05:41
b51f356 to
8f1a82d
Compare
ScriptedAlchemy
marked this pull request as draft
November 5, 2026 07:15
Ignore rolled-back runs and cancelled runs that never started when reconstructing the prior conversation for split suggestions. Match the associated user message as well as runId because canonical user messages can have a null runId. Keep cancelled turns that did start visible. This prevents abandoned turns from suppressing suggestions or introducing discarded context into model requests. Add regression coverage for all three run states and the exact history sent to the decision model. Also merge current main and preserve the Android test platform variable. Focused tests, the selected check:changed tests, direct server typechecking, formatting, lint, both knip scans, and GraphQL validation pass. The aggregate typecheck launcher is blocked by the environment's seccomp supervisor.
This branch has not been deployed
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
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.
Thread split suggestions almost never fire usefully. On a held-out test of Claude and Codex threads, the prompt and threshold shipped in #55 caught 1 of 33 real new tasks.
This PR switches the shared prompt to the
contextualprompt, lowers the threshold from 0.94 to 0.6538, and skips the model request for drafts under five words. The feature stays passive and opt-in, and the eval keeps the old prompt asbaselinefor comparison.main: baseline, 0.94The table calculates precision from the reported true and false suggestions and excludes uncertain labels, as the evaluator does. The earlier 12.5% and 44.4% figures used a different denominator, and reconciling them needs the private score receipts. These corrections are arithmetic and do not come from a new model evaluation.
Both offline runners now support
--min-message-words 5to reproduce the shipped five-word minimum. The experiment records that choice in its scoring receipts and frozen calibration selection, and it keeps the original baseline's eligibility.mainis within noise, at -0.5 to +2.3 points.The experiment froze the threshold on calibration data and chose the word minimum before it read the test split.
Limits
Verification
main's service code.python3 -m unittest discover -s evals/thread-splitpasses (7 tests), including short-draft and receipt-policy regressions.Still in progress
This PR stays a draft while a local improvement run finishes. It may gain a shorter history or a retrained final layer if either beats this configuration on a fresh sealed test. The run has these results so far:
mlx-decision1.1.0 drops a saved pplx yes/no fit on load, so any trained model needs a fix or a workaround for that before it can ship.GPT-6 Astra Pro in ChatGPT did the follow-up review and fixes.
Claude Opus 5.5 in T3 Code, through Claude Code.
🤖 Generated with Claude Code