improvement(workflow): scope the editor by props instead of the URL

[TheodoreSpeaks]

Summary

• The workflow editor no longer reads workspaceId/workflowId from the URL, so <Workflow> works wherever it's mounted (groundwork for rendering the full editor in the org home's resource pane).
• New WorkflowScopeProvider + useWorkflowScope() (throws outside a provider) + useScopedWorkflowId() (throws in a preview). <Workflow> takes both ids as required props; the route page passes them from params.
• Previews (Preview, PreviewWorkflow, PreviewEditor) scope sub-blocks to the workspace with no workflow; /workspace/[id]/w renders Panel under the same null-workflow scope.
• Socket room is only what mounted editors request: Workflow always joins its workflow and releases it on unmount (leaveWorkflow(id)), with a request stack so unmounting one editor never drops another's room. The route-param fallback is removed from socket-provider.
• Editor navigations (invalid-workflow redirect, open logs, duplicate, usage-limit request, command list, delete) go through useResourceRouter, so a containing pane can intercept them.
• useWand, SkillModal, and the custom-tool generation hooks take workspaceId explicitly since they also render outside the editor.

Type of Change

• Improvement / refactor

Testing

• bun run lint, bun run check:audits, block registry check, artifact generators (no drift)
• bun run test: all pass except 6 in lib/cleanup / lib/core that fail identically on staging HEAD
• socket-join-target.test.ts covers the request stack and visibility gate (new cases verified to fail against the matching mutations); sub-block tests now render under WorkflowScopeProvider
• Manual E2E on the routed editor: room join + edits persist, tool/file/KB/table pickers list items, deploy modal (General/API/MCP), wand and file upload send the scoped ids, bogus workflow id redirects, logs execution snapshot renders

Checklist

• Code follows project style guidelines
• Self-reviewed my changes
• Tests added/updated and passing (new tests pass the test-audit authoring gate)
• No new warnings introduced
• I confirm that I have read and agree to the terms outlined in the Contributor License Agreement (CLA)

🤖 Generated with Claude Code

https://claude.ai/code/session_01UMjSAnE9M7r86o5P19ash5

[vercel]

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

1 Skipped Deployment

Project	Deployment	Actions	Updated
docs	Skipped		Oct 13, 2026 10:46pm UTC

[greptile-apps]

Confidence Score: 5/5

[Medium impact] The PR appears safe to merge; no new actionable issues were found.

Summary

The PR gives the workflow editor explicit workspace and workflow ids instead of reading them from the URL.

• Previews now inherit the editor's workspace before falling back to the workspace host.
• Each editor gets its own socket request token and releases it on cleanup.
• The scope hooks now document their return values and errors.
• The previous preview-scope and documentation findings are fixed.
• TheodoreSpeaks intentionally deferred exporting ResourceNavigationProvider: no pane consumes it yet, and an unused export would fail CI. The follow-up pane PR will add both the provider and its consumer.

Diagram

%%{init: {'theme': 'neutral'}}%%
flowchart TD
  Route[Route params] --> Editor[Workflow props]
  Editor --> Scope[WorkflowScopeProvider]
  Scope --> Controls[Editor controls]
  Scope --> Preview[PreviewScope]
  Preview --> PreviewScope[Workspace scope with no workflow]
  Editor --> Join[Request room with unique token]
  Join --> Stack[Most recent request selects room]
  Cleanup[Editor cleanup] --> Release[Remove that token]
  Release --> Stack

Loading

_{Reviews (2) · Last reviewed commit: "fix(workflow): release socket room reque..." · Reviewed by Greptile}

[cubic-dev-ai]

All reported issues were addressed across 62 files

<sub>Reply with feedback, questions, or to request a fix.

Turn on auto-fix | Re-trigger cubic</sub>

[TheodoreSpeaks]

@greptile

[TheodoreSpeaks]

@cubic-dev-ai review this PR

[cubic-dev-ai]

@cubic-dev-ai review this PR

@TheodoreSpeaks I have started the AI code review. It will take a few minutes to complete.

[cubic-dev-ai]

No issues found across 62 files

Confidence score: 5/5

• Automated review surfaced no issues in the provided summaries.
• No files require special attention.

_{Turn on auto-fix | Re-trigger cubic}