Skip to content

Plan 5e: Honest framing on remaining stubs#19

Merged
theaboutbox merged 1 commit intoadmin-overhaulfrom
admin-overhaul-5e-stub-cleanup
May 1, 2026
Merged

Plan 5e: Honest framing on remaining stubs#19
theaboutbox merged 1 commit intoadmin-overhaulfrom
admin-overhaul-5e-stub-cleanup

Conversation

@theaboutbox
Copy link
Copy Markdown
Contributor

Summary

The six unbuilt sidebar pages (Triggers & channels, Policies & risk, Secrets, MCP server, Webhooks, Updates) used to say "Coming in Plan 5" — that's misleading now that Plan 5 is shipping. This PR replaces that framing with an honest status badge + a copy that explains what (if anything) exists today.

Two status kinds:

  • Not implemented yet (warn badge) — no backend at all. Applies to: Policies & risk, MCP server, Updates.
  • Partially implemented (info badge) — functionality exists elsewhere or partially. Applies to: Triggers & channels, Secrets, Webhooks. Each shows a "See also" footer that links to the page(s) where the existing functionality lives (e.g., Secrets links to Settings + Projects).

stubPage(...) is refactored from positional args to a single spec object since the new shape needs status + optional seeAlso.

No backend or routing changes. 679 tests still pass.

PR base: admin-overhaul.

Test plan

  • Visit each of the 6 stubbed routes; the status badge in the page header reflects whether the page has any backing implementation
  • "Partially implemented" pages link to the actual implementations via "See also"
  • No regressions on built pages

🤖 Generated with Claude Code

The stub pages used to say "Coming in Plan 5" — misleading now that
Plan 5 is shipping. Replace with explicit status:

- "Not implemented yet" (warn badge) for pages with no backing data:
  Policies & risk, MCP server, Updates.
- "Partially implemented" (info badge) for pages whose functionality
  exists elsewhere or partially in the orchestrator: Triggers &
  channels, Secrets, Webhooks. Each links via "See also" to the page(s)
  where the existing functionality lives.

Refactors stubPage(...) to take a single spec object instead of
positional args, since the new shape needs status + optional seeAlso.

Co-Authored-By: Claude Opus 4.7 <noreply@anthropic.com>
@theaboutbox theaboutbox merged commit 2563992 into admin-overhaul May 1, 2026
1 of 2 checks passed
@theaboutbox theaboutbox deleted the admin-overhaul-5e-stub-cleanup branch May 1, 2026 00:40
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