feat(nav): IA restructure — every page findable, one deployments URL, an honest AI menu #5558
ccantynzAI Reviewcommented 5d ago
AI Triage
(no summary)
Priority: medium Risk area: mixed
Suggested labels: (no label suggestions) Suggested reviewers: (no reviewer suggestions)
Suggestions only — nothing has been applied. The PR author stays in control.
gluecron[bot]🤖 botAI Reviewcommented 5d ago
AI review unavailable
The platform's AI balance is exhausted, so AI generation is temporarily unavailable. Nothing was lost. You can queue this as a repair for the internal agent from the repository's Health page, or try again once the balance is restored. The PR is otherwise unchanged.
Cross-repo impact
See what breaks downstream if this PR merges.
⮌ Merged
This pull request was merged into main.
c comment · e edit title · m merge · a approve · r request changes · ? shortcuts
The owner's verdict after the wiring-level unification: "the platform navigation is not very good we need improvements." This is the structural answer.
REPO_NAV_MORE_GROUPS, same pattern as the AI menu): Repository (Commits, Branches, Tags, Releases, Packages, Symbols, Stale branches), Planning (Milestones, Projects, Discussions, Wiki, Contributors, Moderation queue), Delivery (Gates, Merge queue, Merge checks, Deployments, Pipeline, Agents), Activity (Health report, Pulse, Push log, Traffic). Eleven formerly URL-only pages are now findable, each with a real active key — which came free, since the key union derives from the arrays. Claude sessions joined the AI menu./deploymentsis authoritative; cloud-provider runs render on it vialistCloudRuns();/cloud-deployments301s (private-repo gate still runs first — no existence leak); all links repointed. The hand-rolledHealthNavfork is deleted; the dead duplicate/workspaceregistration removed with a do-not-re-add note.ANTHROPIC_API_KEYis unset, the 18-item ✨ menu shows one banner ("AI features are not enabled on this instance — admins can set a key in Admin → Integrations") and mutes items rather than hiding them — their fallback pages are good.min(92vw, 760px).Triage kept out of nav deliberately:
/coupling+/dependencies(Insights-hub cards, pinned),/timeline/:ref(file-scoped),/rollback(POST-only),/memory(demo-gated fabricated data — retirement candidate). Standing finding for a future pass:deps.tsx:467registers a dead duplicateGET /:owner/:repo/dependencies(insights mounts first).Re-verified at integration on current main: typecheck clean, design gate 0 banned, nav suites 54 pass.