docs: update changelog for recent merges#900
Conversation
📝 WalkthroughWalkthroughUpdates CHANGELOG.md Unreleased with breaking changes, migration guidance, added/changed/fixed items for delivery modes, CLI/Broker features, messaging/PTY/telemetry/SDK notes; adds a changelog-style guideline to AGENTS.md. ChangesUnreleased Changelog Updates
Estimated code review effort🎯 2 (Simple) | ⏱️ ~10 minutes Suggested reviewers
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)
Comment |
2dace2e to
55fda1e
Compare
There was a problem hiding this comment.
No issues found across 1 file
You’re at about 98% of the monthly reviewed-line limit. You may want to disable incremental reviews to conserve quota. Reviews will continue until that limit is exceeded. If you need help avoiding interruptions, please contact contact@cubic.dev.
Tip: cubic could auto-approve low-risk PRs like this, if it thinks it's safe to merge. Learn more
Re-trigger cubic
55fda1e to
b75f406
Compare
There was a problem hiding this comment.
Actionable comments posted: 1
🤖 Prompt for all review comments with AI agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.
Inline comments:
In `@CHANGELOG.md`:
- Line 19: The changelog currently references two different env var names
(POSTHOG_PROJECT_KEY and AGENT_RELAY_POSTHOG_KEY); reconcile them by either
standardizing to a single canonical name across the file or adding an explicit
note that POSTHOG_PROJECT_KEY (used in GitHub Actions/release publishing) maps
to AGENT_RELAY_POSTHOG_KEY at runtime, and show the exact mapping example (e.g.,
export AGENT_RELAY_POSTHOG_KEY=$POSTHOG_PROJECT_KEY) so readers won’t set the
wrong variable; update the lines mentioning POSTHOG_PROJECT_KEY and
AGENT_RELAY_POSTHOG_KEY accordingly.
🪄 Autofix (Beta)
Fix all unresolved CodeRabbit comments on this PR:
- Push a commit to this branch (recommended)
- Create a new PR with the fixes
ℹ️ Review info
⚙️ Run configuration
Configuration used: Organization UI
Review profile: CHILL
Plan: Pro Plus
Run ID: d405fae8-892a-4903-b3af-dd399965e2d4
📒 Files selected for processing (2)
AGENTS.mdCHANGELOG.md
✅ Files skipped from review due to trivial changes (1)
- AGENTS.md
| - See the updated [Worker Orchestration guide](/docs/guides/worker-orchestration) for retry patterns. | ||
| - Pass `--from` to `agent-relay send` when a script requires a specific sender identity. | ||
| - Handle `success: false` from `relay.spawn()` calls that pass `task`; spawns without a task are unchanged. | ||
| - Set `POSTHOG_PROJECT_KEY` in GitHub Actions repository variables before publishing telemetry-enabled artifacts. |
There was a problem hiding this comment.
Clarify telemetry key naming to avoid migration misconfiguration.
Line 19 tells users to set POSTHOG_PROJECT_KEY, while Line 53 documents runtime gating on AGENT_RELAY_POSTHOG_KEY. Please reconcile or explicitly explain the mapping so release setup doesn’t use the wrong variable.
Also applies to: 53-53
🤖 Prompt for AI Agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.
In `@CHANGELOG.md` at line 19, The changelog currently references two different
env var names (POSTHOG_PROJECT_KEY and AGENT_RELAY_POSTHOG_KEY); reconcile them
by either standardizing to a single canonical name across the file or adding an
explicit note that POSTHOG_PROJECT_KEY (used in GitHub Actions/release
publishing) maps to AGENT_RELAY_POSTHOG_KEY at runtime, and show the exact
mapping example (e.g., export AGENT_RELAY_POSTHOG_KEY=$POSTHOG_PROJECT_KEY) so
readers won’t set the wrong variable; update the lines mentioning
POSTHOG_PROJECT_KEY and AGENT_RELAY_POSTHOG_KEY accordingly.
Summary
Verification
Note: the isolated worktree did not have its own node_modules install, so the commit hook could not find prettier there; formatting was checked manually with the repo checkout's existing Prettier binary.