#1170 - P3 Slice Zero, Task 4 — replace Web free-text selection with the structured harness catalog - stack

Conversation 0 Commits 2 Files Changed 9

+1510 -25

2 Commits

Author	SHA1	Message	Date
jason.woltjeandClaude Opus 4.8	d752db93e4	fix(web): scope model list to provider and resolve composite catalog identity ci/woodpecker/pr/ci Pipeline was successful Details Fix-in-lane for the two Task-4 review findings, red-first TDD, inside the existing nine-file fence. M-A: the composer now filters model <option>s to the intentionally selected provider (harness.providerId). With no provider chosen the model select offers only its placeholder, so a user can never pick a model that belongs to a different provider. M-B: the model <option> value is now the collision-safe composite `${providerId}:${modelId}` (was the bare modelId), the controlled select value mirrors that same identity so the exact catalog row highlights, and the change handler resolves the composite back to the exact catalog row and persists that row's own {harnessId, providerId, modelId}. selectModel(providerId, modelId) no longer combines a bare model id with ambient provider state, so two providers exposing the same modelId stay distinct. New red-first tests prove: cross-provider models absent, identical modelIds under two providers stay distinct and resolve to the intended tuple, a provider change invalidates the old model and keeps send disabled until the new tuple persists, and a model pick does not enable send until its exact PUT resolves. The four existing anti-masking invariants and the 422 tuple-preservation test are intact. Co-Authored-By: Claude Opus 4.8 <[email protected]> Claude-Session: https://claude.ai/code/session_01ESFAnh2t9HmLwng8oW95St	2026-08-11 21:13:40 -05:00
jason.woltjeandClaude Opus 4.8	ef66b0e7d6	feat(web): use structured harness catalog selection Co-Authored-By: Claude Opus 4.8 <[email protected]> Claude-Session: https://claude.ai/code/session_01ESFAnh2t9HmLwng8oW95St	2026-08-11 20:36:48 -05:00

Author

SHA1

Message

Date

jason.woltjeandClaude Opus 4.8

d752db93e4

fix(web): scope model list to provider and resolve composite catalog identity

ci/woodpecker/pr/ci Pipeline was successful

Details

Fix-in-lane for the two Task-4 review findings, red-first TDD, inside the
existing nine-file fence.

M-A: the composer now filters model <option>s to the intentionally selected
provider (harness.providerId). With no provider chosen the model select offers
only its placeholder, so a user can never pick a model that belongs to a
different provider.

M-B: the model <option> value is now the collision-safe composite
`${providerId}:${modelId}` (was the bare modelId), the controlled select value
mirrors that same identity so the exact catalog row highlights, and the change
handler resolves the composite back to the exact catalog row and persists that
row's own {harnessId, providerId, modelId}. selectModel(providerId, modelId) no
longer combines a bare model id with ambient provider state, so two providers
exposing the same modelId stay distinct.

New red-first tests prove: cross-provider models absent, identical modelIds
under two providers stay distinct and resolve to the intended tuple, a provider
change invalidates the old model and keeps send disabled until the new tuple
persists, and a model pick does not enable send until its exact PUT resolves.
The four existing anti-masking invariants and the 422 tuple-preservation test
are intact.

Co-Authored-By: Claude Opus 4.8 <[email protected]>
Claude-Session: https://claude.ai/code/session_01ESFAnh2t9HmLwng8oW95St

2026-08-11 21:13:40 -05:00

jason.woltjeandClaude Opus 4.8

ef66b0e7d6

feat(web): use structured harness catalog selection

Co-Authored-By: Claude Opus 4.8 <[email protected]>
Claude-Session: https://claude.ai/code/session_01ESFAnh2t9HmLwng8oW95St

2026-08-11 20:36:48 -05:00

P3 Slice Zero, Task 4 — replace Web free-text selection with the structured harness catalog #1170