Skip to content

Enable Workspace creation in the demo - #762

Draft
luokerenx4 wants to merge 1 commit into
devfrom
codex/demo-workspace-create
Draft

Enable Workspace creation in the demo#762
luokerenx4 wants to merge 1 commit into
devfrom
codex/demo-workspace-create

Conversation

@luokerenx4

Copy link
Copy Markdown
Contributor

Summary

  • replace the Demo Workspace creation stub with an in-memory implementation that follows the production response contract
  • validate tags and templates, reject duplicate tags with renderable top-level errors, and expose newly created Workspaces through the list endpoint
  • return a complete Workspace record so the real UI can navigate directly into the new empty desk
  • add handler coverage for creation, list visibility, duplicate tags, and invalid tags

Root cause

The Demo POST /api/workspaces route returned a nested { error: { ... } } object even though the client expects the production { error, message } shape. The form stored that object as renderable error text, causing React to crash to a blank page. The route also disabled a prominent mutation despite the Demo banner promising non-persistent mutations.

Verification

  • pnpm vitest run ui/src/demo/handlers/workspaces-create.spec.ts
  • npx tsc --noEmit
  • cd ui && npx tsc -b
  • pnpm test (358 passed, 1 skipped; 3391 tests passed, 9 skipped)
  • git diff --check
  • Demo browser walkthrough: created chat-jul29, navigated to /workspaces/demo-created-ws-1, confirmed the empty Workspace/session surface renders, then retried the same tag and confirmed an inline duplicate error without a crash

Boundary touch

  • none; Demo-only Workspace handlers and tests

@vercel

vercel Bot commented Jul 28, 2026

Copy link
Copy Markdown

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
openalice-demo Ready Ready Preview, Comment Jul 28, 2026 9:54pm

Request Review

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