stacklok / toolhive-studio / 25908146373

70%

main: 70%

Build Type

push

github

Committed by web-flow

Commit Message

feat(playground): per-thread model, MCP, and skills selection (#2253)

* feat(playground): make model, MCP, and skills selection per-thread

Each chat thread now persists its own model/provider, enabled MCP tools,
and enabled skills (agent was already per-thread). Switching threads no
longer reshuffles the toolbar; new threads inherit the last-used
selection so the UX is unchanged for the first pick.

Globals are kept as the "default for new threads" source. Every
per-thread mutation dual-writes the global so the next new thread
inherits it. A one-time migration snapshots current globals into every
existing thread so nothing regresses on upgrade.

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

* test(playground): fix CI after per-thread settings change

- Add migration 007 to the test-helpers DB bootstrap so writers/readers
  and registry tests see the new columns.
- Make thread-settings-storage stop importing thread-integration so it
  no longer pulls in threads-storage's electron-store side effect; this
  was breaking tests that don't mock electron. Row materialisation now
  happens via db primitives directly.
- Mock the new thread-settings-storage / thread-settings IPC module in
  the colocated unit tests that previously loaded them transitively
  without electron available.
- Update streaming.test.ts assertion for the new
  createBuiltinAgentTools(key, { threadId }) signature.

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

* fix(playground): preserve per-thread isolation and dual-write contract

Address review:

- writeThread now snapshots current globals into the per-thread columns
  when materialising a brand-new row. Previously the row was created
  with NULL provider/model/MCP/skills, which broke per-thread isolation
  for draft threads and any path that creates a thread without
  explicitly carrying the settings (e.g. reconcile-from-store).

- Dual-write call sites (model mutation, MCP server sel... (continued)

Coverage Stats

5194 of 7997 branches covered (64.95%)

130 of 219 new or added lines in 18 files covered. (59.36%)

6 existing lines in 3 files now uncovered.

7706 of 10933 relevant lines covered (70.48%)

114.87 hits per line