docs(changelog): record the relay dashboard removal#1179
Conversation
Add Removed / Breaking Changes / Migration Guidance entries to [Unreleased] for the local web dashboard removal landed in #1146: dropped `up` flags (--no-dashboard/--port/--foreground), AGENT_RELAY_DASHBOARD_PORT -> AGENT_RELAY_BROKER_PORT, installer no longer fetches the dashboard, and the `human_dashboard` ActionSource is retired. Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
|
Warning You have reached your daily quota limit. Please wait up to 24 hours and I will start processing your requests again! |
|
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: Organization UI Review profile: CHILL Plan: Pro Plus Run ID: 📒 Files selected for processing (1)
📝 WalkthroughWalkthrough
ChangesDashboard Removal Release Notes
Estimated code review effort🎯 1 (Trivial) | ⏱️ ~2 minutes Poem
🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. ✨ Finishing Touches🧪 Generate unit tests (beta)
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: b44ea675d5
ℹ️ About Codex in GitHub
Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".
|
|
||
| ### Migration Guidance | ||
|
|
||
| - Replace `agent-relay up --no-dashboard` with `agent-relay up --background`. |
There was a problem hiding this comment.
Point migration guidance at
local up
When users follow this migration literally, agent-relay up --background is not a registered command in the current CLI: createProgram() attaches the broker lifecycle commands only under the local group (packages/cli/src/cli/bootstrap.ts:290-292), and the command-surface test lists the leaf as local up (packages/cli/src/cli/bootstrap.test.ts:6-10). The changelog should direct users to agent-relay local up --background (and qualify the surrounding up references) so the upgrade note does not send them to an unknown command.
Useful? React with 👍 / 👎.
Adds the
[Unreleased]changelog entry for the local web dashboard removal (landed in #1146), kept out of that PR per request.Removed
agent-relay upno longer starts one; installer no longer fetchesrelay-dashboard-server/UI or@agent-relay/dashboard-server;updrops--no-dashboard/--port/--foreground.human_dashboardActionSourceretired (CLI + broker); HTTP-API spawns reporthuman_cli.Breaking Changes
agent-relay upis broker-only, attached by default; use--backgroundto detach (replaces the old--no-dashboarddetach behavior). Removed flags now error.AGENT_RELAY_DASHBOARD_PORT→AGENT_RELAY_BROKER_PORT.Migration Guidance included for each.
This is a SemVer-major (breaking) change set; the version bump is intentionally left out for maintainers to decide.