docs(changelog): backfill 9 merged PRs before the 0.96.0 cut#1895
Conversation
….96.0 cut Nine PRs merged since v0.95.1 shipped without a changelog entry, so the generated release notes (updater + Discord, single-source from CHANGELOG) would have omitted a headline feature and a security-relevant default change. Backfill before rolling the release: - Added: Hermes first-class runtime (#1889 + hardening #1891) - Changed: filesystem.allow_run defaults OFF on the headless tier (#1888) - Fixed: fleet members no longer inherit hub identity (#1886/#1882/#1881), whole trace on a dedicated Langfuse project (#1893), background knowledge ingest wakes the agent (#1887), quieter plugin hot-reload logs (#1885) The already-present entries (#1880 tracing, #1890 plugin-view 401, #1879 subagent max_turns) are unchanged. Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
|
👀 Quinn is reviewing — verdict (PASS / WARN / FAIL) + findings to follow. |
|
Important Review skippedReview was skipped due to path filters ⛔ Files ignored due to path filters (1)
CodeRabbit blocks several paths by default. You can override this behavior by explicitly including those paths in the path filters. For example, including ⚙️ Run configurationConfiguration used: Organization UI Review profile: ASSERTIVE Plan: Pro Plus Run ID: You can disable this status message by setting the Use the checkbox below for a quick retry:
✨ Finishing Touches🧪 Generate unit tests (beta)
Comment |
There was a problem hiding this comment.
QA Audit — PR #1895 | docs(changelog): backfill 9 merged PRs before the 0.96.0 cut
VERDICT: PASS
CI Status
- 8 checks in progress (build, Python tests, lint, Fleet integration, A2A smoke, workspace config, Web E2E, gitleaks) — non-terminal. Approve-on-green policy promotes this review to APPROVED automatically once every check settles terminal-green.
Diff Review
- Single file:
CHANGELOG.md, +37/-0, pure documentation. - Backfills
[Unreleased]with 9 merged PRs: Hermes runtime (Added),filesystem.allow_rundefault (Changed), and 6 Fixes (fleet identity inheritance, Langfuse tracing, knowledge-ingest wake, plugin logs). - Existing entries (#1880, #1890, #1879) untouched.
- Entries are well-written, correctly categorized under Added/Changed/Fixed per semver convention, and cite the PR numbers they document.
Observations
- No code changes — no correctness risk, no regression surface.
- No unresolved review threads.
- Structural review skipped (1 file, 37 lines, docs-only — optional per diff_summary).
— Quinn, QA Engineer
|
Submitted COMMENT review on #1895. |
There was a problem hiding this comment.
QA panel review — PASS
code-review · head 7473f0ee20e9 · shadow — comment-only
[review-synthesizer completed: workflow code-review:report]
Prose Brief
Overall risk: negligible. This is a changelog-only PR — a routine [Unreleased] roll into [0.54.0] aggregated from 9 merged PRs. No code changes, no behavioral impact, nothing deployable breaks.
Fix-first item: None. The verifier confirmed all 9 referenced PRs (#1881–#1893, excl. gaps) match their changelog descriptions; section ordering follows Keep a Changelog; the prepare-release.yml script is format-agnostic beyond the ## [Unreleased] marker. Structural and factual accuracy both check out.
Panel disagreement: None — the synthesizer arrived at an empty findings array, and the verifier corroborated it. No findings survived to dispute.
What verification changed: Nothing. The verifier resolved the synthesizer's earlier caveat about unverified factual accuracy by fetching and confirming every referenced PR. The empty array stands as final.
[]There was a problem hiding this comment.
CI terminal-green, no blockers on prior review — auto-approving on green (#748).
|
✅ CI went terminal-green with no blockers on the prior review — promoting it to APPROVED per the approve-on-green policy (#748). |
Nine PRs merged since v0.95.1 without a CHANGELOG entry. Our release notes are single-source (the desktop updater notes and the Discord changelog are generated from
CHANGELOG.md), so cutting 0.96.0 as-is would ship notes missing a headline feature (Hermes runtime) and a security-relevant default change (filesystem.allow_run). This backfills[Unreleased]soprepare-release.ymlrolls a complete section.Added
protoagent hermes— one-command Hermes preset +runtime use/list(ADR 0033) #1889) + hardening (fix(runtime): harden the ACP/Hermes runtime path — /v1 switch, instance pinning, serialization, context usage #1891)Changed
filesystem.allow_rundefaults OFF on the headless tier (fix(config): default filesystem.allow_run OFF on the headless UI tier #1888)Fixed
The three entries already present (#1880, #1890, #1879) are untouched. Verified locally that
scripts/changelog.py rollproduces a clean 9-bullet[0.96.0]section (Added×2 / Changed×1 / Fixed×6) and resets[Unreleased].Next step after merge: dispatch
prepare-release.ymlwithbump=minor→ 0.96.0.🤖 Generated with Claude Code