Skip to content

[codex] Unify module workbench routes#38

Merged
ActiveInAI merged 1 commit into
mainfrom
codex/unify-module-workbench-routes-20260516
May 16, 2026
Merged

[codex] Unify module workbench routes#38
ActiveInAI merged 1 commit into
mainfrom
codex/unify-module-workbench-routes-20260516

Conversation

@ActiveInAI

Copy link
Copy Markdown
Owner

What changed

  • Use UUID-safe default runtime context headers for frontend API calls so database-backed project routes no longer send dev-tenant.
  • Route every business module, including Digital Twin, through /app/modules/{moduleId} and retire the standalone /app/digital-twin entry.
  • Remove the unused standalone DigitalTwinWorkbench component and update architecture docs to match the unified Open CDE module workbench boundary.

Why

/app/projects was calling the database-backed backend with a non-UUID tenant id, which caused tenant_id must be a UUID. Digital Twin also had two competing surfaces, while the repository source of truth requires the module route to use the same Open CDE workbench as all other modules.

Validation

  • bun x next typegen
  • bun run typecheck
  • bun run lint
  • bun run test
  • bun run build
  • Restarted local database containers, backend gateway, and frontend dev server.
  • Verified /v1/projects returns tenant-scoped projects with UUID headers.
  • Verified /app/projects returns 200 and renders 2 projects.
  • Verified all 14 module routes return 200 under /app/modules/....
  • Verified /app/modules/digital_twin returns 200.
  • Verified retired /app/digital-twin returns 404.

@ActiveInAI ActiveInAI merged commit 166556e into main May 16, 2026
6 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