Skip to content

docs: contributor credits + full v0.23.2 PR coverage in CHANGELOG - #396

Merged
100yenadmin merged 2 commits into
mainfrom
docs/v0232-changelog-credits
Aug 27, 2026
Merged

100yenadmin merged 2 commits into
mainfrom
docs/v0232-changelog-credits

Conversation

@100yenadmin

Copy link
Copy Markdown
Member

Owner-requested post-GA attribution audit findings + fix.

Audit result: v0.22.0/v0.23.0/v0.23.1 notes are accurate, PR-mapped, and credit contributors (v0.23.0 names all upstream-carried authors with PR numbers; v0.23.1's window was internal-only). v0.23.2 had gaps: its CHANGELOG section listed only 4 of the window's 14 merged PRs — omitting the #365 SECURITY fix arc and @Tosko4's #364 — and neither the notes nor the release body credited contributors.

This PR: completes the v0.23.2 CHANGELOG section (security arc #365→#366 + #383/#384/#391, #364 credited to @Tosko4, #385→#388 identity, integrity work #352 #358 #368 #373 #381 #382 #390, contributors line) and mirrors v0.23.0's upstream credits into its CHANGELOG section. Docs-only; tagged notes files untouched (byte-identity with tags preserved — the class #393 fixed). The v0.23.2 GitHub release body is updated separately via gh release edit (editable metadata, isImmutable=false).

release-workflow tests: 8 passed.

…GELOG

Post-GA attribution audit (owner-requested): the v0.23.2 CHANGELOG section
covered only #370/#371/#373/#374 — missing the SECURITY fix arc (#365→#366,
#383/#384/#391), @Tosko4's read-only-clones fix (#364), release identity
(#385→#388), and the release/benchmark-integrity work (#352 #358 #368 #373
#381 #382 #390). Added, with a Contributors line crediting the external code
contribution and the review signal. Also mirrored the v0.23.0 notes'
upstream-carried contributor credits (@stephenschoettler @Tosko4 @masidigital
@davidrobertson @TurgutKural) into that CHANGELOG section. The tagged notes
files are untouched (tag/tree byte-identity preserved); the v0.23.2 GitHub
release BODY gets the Contributors section separately (editable metadata).
@coderabbitai

coderabbitai Bot commented Aug 27, 2026 •

Copy link
Copy Markdown

Warning

Review limit reached

Next included review available in 40 minutes.

View limit details

Limit details: You’ve used the included review currently available.

You've used all free OSS reviews for now. Wait for the free limit to reset to keep reviewing this public repository.

Learn how review limits work.

Review configuration:

⚙️ Run configuration

Configuration used: Organization UI

Review profile: ASSERTIVE

Plan: Pro Plus

Run ID: 8f3922c3-756c-43f3-a12c-5e5c3ebbfdb1

📥 Commits

Reviewing files that changed from the base of the PR and between 676bb48 and 97fe5ae.

📒 Files selected for processing (1)
  • CHANGELOG.md

Warning

Billing warning: we have not been able to collect payment for this subscription for more than 72 hours. Please update the payment method or pay any pending invoices in Billing to avoid service interruption.


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.

❤️ Share

Comment @coderabbitai help to get the list of available commands.

…364 characterization, issue→PR notation

F1: #352 is the V1-M benchmark provider-accounting PR (already covered under
#368's boundary rows); the exact-head governance pair is #349 + #358.
F2: #364's clones are Hermes per-agent host clones — the failure mode was a
silent fallback to the built-in lossy compressor under a concurrent WAL
writer (#363); the credit line now states that accurately.
F3: consistent issue→PR notation (#383→#384, #389→#391).
@100yenadmin
100yenadmin merged commit 2e17bbc into main Aug 27, 2026
15 checks passed
@100yenadmin
100yenadmin deleted the docs/v0232-changelog-credits branch August 27, 2026 16:44
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