Skip to content

Make the new-workbench composer intent-first - #658

Merged
TheGreatAxios merged 6 commits into
corbitsdev:mainfrom
pratikbuilds:fix/workbench-chat-composer
Sep 7, 2026
Merged

TheGreatAxios merged 6 commits into
corbitsdev:mainfrom
pratikbuilds:fix/workbench-chat-composer

Conversation

@pratikbuilds

@pratikbuilds pratikbuilds commented Sep 6, 2026 •

Copy link
Copy Markdown
Collaborator

Summary

Redesign the new-workbench chat composer as the primary way to start a workbench.

  • Put a large freeform prompt first, with example prompts and a single Start action.
  • Let people select, search, and remove agents before starting; chosen agents join with the opening message.
  • Keep templates below the composer as a secondary "Or start from a template" path, with an explicit empty-workbench option.
  • Polish the active composer state: filled selected prompt chips, an accent caret, no Enter-key footer hint, and responsive layout.
  • If the room exists but its opening message or title fails, take the person into that room and give them a retry path instead of stranding them on setup.

Supporting reliability

  • Correlate each agent reply to its exact child run so a late or overlapping response cannot complete, appear in, or disappear behind another workbench conversation.

Validation

  • bun run typecheck
  • bun run lint
  • bun run test
  • Focused new-workbench composer, agent-selection, and reply-correlation coverage

@pratikbuilds
pratikbuilds marked this pull request as draft September 6, 2026 10:24
@pratikbuilds pratikbuilds changed the title Restore reliable workbench replies Make the new-workbench composer intent-first Sep 6, 2026
@pratikbuilds
pratikbuilds force-pushed the fix/workbench-chat-composer branch from e2dd3aa to fd62988 Compare September 6, 2026 10:35
@pratikbuilds
pratikbuilds force-pushed the fix/workbench-chat-composer branch from fd62988 to d00b6b3 Compare September 6, 2026 10:45
pratikbuilds and others added 2 commits September 7, 2026 12:48
… de-flake search tests

Three review fixes:

- Enter inside the agent search with zero matches no longer falls through
  to the form's implicit submission, which created a workbench the
  operator did not ask for; the keydown is now always cancelled and a
  picker test asserts the cancellation (red without the fix).
- The workbench-create catch reported only the post-create failure branch
  through reportError; every other cause now reports with operation
  context and the toast carries the refId.
- The use-entity-search debounce test asserted a transient that evaporates
  once the 5ms debounce timer outruns the assertion on a loaded runner
  (two CI failures, unreproducible locally); a gated source now holds
  loading observable deterministically.
- agent-turns: the sectionRunId doc matches the per-agent warm run.
@TheGreatAxios
TheGreatAxios marked this pull request as ready for review September 7, 2026 17:35
@TheGreatAxios
TheGreatAxios merged commit 18bb6d4 into corbitsdev:main Sep 7, 2026
46 of 48 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.

2 participants