Skip to content

chore: remove new thread suggestions - #113

Merged
ScriptedAlchemy merged 2 commits into
mainfrom
t3/remove-new-thread-suggestions
Nov 13, 2026
Merged

ScriptedAlchemy merged 2 commits into
mainfrom
t3/remove-new-thread-suggestions

Conversation

@ScriptedAlchemy

Copy link
Copy Markdown
Owner

The experimental "Suggest new threads" Labs feature (#55) asked a local decision model whether a composer draft was independent of its thread and offered to start a fresh one. We no longer want it.

Removed end to end:

  • Server: ThreadSplitSuggestions service, its layer, the orchestration.suggestThreadSplit RPC handler, instrumentation, and test mocks.
  • Contracts: threadSplit.ts, the RPC, its permission scope, and the threadSplitSuggestionsEnabled / threadSplitModelEndpoint settings (old values in settings.json are ignored on decode).
  • Web: composer suggestion, Labs → Zack settings rows, settings search entry, and the isCompacting composer prop that only the suggestion read.
  • Mobile: composer suggestion and the Thread suggestions settings section.
  • Shared prompt JSON, evals/thread-split, and the user guide section.

vp run check:changed passes (format, lint, knip, typecheck of 42 packages, tests).

Done by Claude Opus 5.5 in Claude Code.

🤖 Generated with Claude Code

The "Suggest new threads" Labs feature asked a local decision model whether a
draft was independent of its thread and offered to start a fresh one. Remove
it end to end: the server service and RPC, contracts and settings, the web and
mobile composer suggestion and settings rows, the settings search entry, the
user guide section, and the thread-split eval.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
@ScriptedAlchemy
ScriptedAlchemy enabled auto-merge (squash) November 12, 2026 21:24
@changeset-bot

changeset-bot Bot commented Nov 12, 2026 •

Copy link
Copy Markdown

⚠️ No Changeset found

Latest commit: 55da771

Merging this PR will not cause a version bump for any packages. If these changes should not result in a new version, you're good to go. If these changes should result in a version bump, you need to add a changeset.

Click here to learn what changesets are, and how to add one.

Click here if you're a maintainer who wants to add a changeset to this PR

@ScriptedAlchemy
ScriptedAlchemy merged commit d57f643 into main Nov 13, 2026
16 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.

1 participant