feat: one route family — every capability is a docs page - #1010
Merged
Conversation
Co-Authored-By: Claude Fable 5.1 <noreply@anthropic.com>
Co-Authored-By: Claude Fable 5.1 <noreply@anthropic.com>
Six capabilities used to share a docs page with a sibling (durable-execution/persistence, ag-ui tool-views/json-render/ client-tools/subagents, render repeat-loops/specs) and relied on PRIMARY_CAPABILITY_BY_DOCS_PATH to pick a primary owner for the route. Give each its own docs path instead, so the override table and its validate-manifest ambiguity check can go away. The six new docs paths (e.g. /docs/langgraph/guides/ durable-execution) do not have pages yet - that lands in a follow-up task alongside removing the /workspace route. Co-Authored-By: Claude Fable 5.1 <noreply@anthropic.com>
The durable-execution capability fixture in workspace-presentation.spec.ts still pinned the old shared docs path (/docs/langgraph/guides/persistence) that the registry no longer publishes for it. Update the expectation to /docs/langgraph/guides/durable-execution, its own path. No other cockpit-shell/workspace-react fixture pins one of the other five moved capabilities' old shared paths (the one remaining hit on /docs/render/guides/specs belongs to spec-rendering, the primary sibling, and is unaffected). Co-Authored-By: Claude Fable 5.1 <noreply@anthropic.com>
…ar descriptors The Python siblings for the six capabilities that got their own docs path (durable-execution, ag-ui tool-views/json-render/client-tools/ subagents, render repeat-loops) were updated, but each Angular mirror in cockpit/*/*/angular/src/index.ts still carried the old shared path. The Website drift guard (apps/website/src/lib/cockpit-docs-links.spec.ts) scans every cockpit/**/src/index.ts, so the Angular copies needed the same update. Co-Authored-By: Claude Fable 5.1 <noreply@anthropic.com>
…h move deployments/ag-ui-dev/deps/** is a generated mirror of the cockpit/ag-ui/*/python sources. The docsPath moves for tool-views, json-render, client-tools, and subagents (plus the earlier prettier reflow of two of those sources) left the generated copies stale, which .github/workflows/deploy-ag-ui.yml would have caught post-merge via `git diff --exit-code -- deployments/ag-ui-dev/`. Regenerated with scripts/generate-ag-ui-deployment-config.ts. Co-Authored-By: Claude Fable 5.1 <noreply@anthropic.com>
…moke and wires cockpit-shell into CI Co-Authored-By: Claude Fable 5.1 <noreply@anthropic.com>
Co-Authored-By: Claude Fable 5.1 <noreply@anthropic.com>
…nerative UI title Co-Authored-By: Claude Fable 5.1 <noreply@anthropic.com>
…s page Co-Authored-By: Claude Fable 5.1 <noreply@anthropic.com>
…swap; drop dead workspace helpers - resolveDocsWorkspace never returns null (a miss resolves docs-only), so the docs index's Run href guard is now gated on resolution.kind === 'mapped' instead of a dead null check, moved into a lib module with a unit test that would have caught the regression - drop dead ?? fallbacks on the now-nonnullable WorkspaceIdentity.docsPath in workspace-presentation.ts - delete the unused getWebsiteWorkspaceHref helper - fix WebsiteWorkspace.spec.tsx tests that override docsSlot: undefined even though the docs route they model always supplies a slot Co-Authored-By: Claude Fable 5.1 <noreply@anthropic.com>
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
Contributor
…th; docsPath is non-nullable Co-Authored-By: Claude Fable 5.1 <noreply@anthropic.com>
Contributor
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Part A of
docs/superpowers/specs/2026-09-04-docs-workspace-unification-design.md(plan:docs/superpowers/plans/2026-09-04-docs-workspace-unification.md).What
PRIMARY_CAPABILITY_BY_DOCS_PATHoverride table is gone and the manifest validator rejects a duplicate or missing docs path.guides/durable-execution; AG-UIguides/client-tools,guides/tool-views,guides/json-render(titled Generative UI),guides/subagents; Renderguides/repeat-loops./workspace/[product]/[topic]route is deleted (old URLs 404 by design),WebsiteWorkspacealways uses documentation labels, andWorkspaceIdentity.docsPathis non-nullable.docsPathconstants in the twelve example descriptors and the generateddeployments/ag-ui-devartifacts are updated so both drift gates stay green.cockpit-shelljoins the CI test list.Verification
Website unit suite 1212/1212; cockpit, cockpit-registry, cockpit-shell, workspace-react green; workflow guards 41/41; website build succeeds with no
workspaceroutes emitted;npx nx e2e websitegreen locally; deployments regeneration is a no-op.Follow-ups: Part B (surface rename), Part C (redirect service retirement), Part D (Vercel/DNS/secrets cleanup).
🤖 Generated with Claude Code