Skip to content

Put the unreleased entries back in the right sections - #370

Merged
btsouth merged 1 commit into
mainfrom
fix/changelog-sections
Aug 22, 2026
Merged

Put the unreleased entries back in the right sections#370
btsouth merged 1 commit into
mainfrom
fix/changelog-sections

Conversation

@btsouth

@btsouth btsouth commented Aug 22, 2026

Copy link
Copy Markdown
Owner

Why this matters before a release

The unreleased section currently reads:

Section Bullets
Added 1
Security 17
Internal 1
Fixed 1

Cutting 1.5.35 from that would announce seventeen security fixes. There is one: Gemini treats a missing home directory as not logged in — the entry that created the section, because it stopped live OAuth tokens being written into a checked-in fixture.

This is my fault. Landing 21 PRs today meant resolving twenty-odd CHANGELOG conflicts, and my resolution merged bullets by position without regard for the heading above them. Entries kept landing in whichever section the conflict happened to open in, and ### Security sat at the top of the drift.

What

Every bullet restored to the section the commit that introduced it had it in — recovered from git log -S per bullet, not from my judgement of what looks security-ish. Section order matches 1.5.33: Added, Security, Fixed, Internal.

Section Before After
Added 1 1
Security 17 1
Fixed 1 17
Internal 1 1

No bullet text changed, and none was lost — still 20.

Three entries (#351, #354, #355) were introduced after the drift began, so history could not place them cleanly. Their PR diffs add no ### Security heading and none is a security fix, so they sit under Fixed.


Note

Low Risk
Documentation-only CHANGELOG reordering; no product or security code changes.

Overview
Fixes a merge-conflict mix-up in the Unreleased notes that had put seventeen bullets under Security.

Section order is now Added, Security, Fixed, Internal. The Gemini missing-home-directory credential fallback stays the only Security entry. The other sixteen notes (plus the mid-scan price-fingerprint item that had been under Internal) move to Fixed. Bullet text is unchanged.

Reviewed by Cursor Bugbot for commit 8c561c6. Bugbot is set up for automated code reviews on this repo. Configure here.

Note

Move unreleased changelog fix entries back under the ### Fixed section

Reorganizes CHANGELOG.md by adding a ### Fixed subsection under the Unreleased heading and relocating the price-change-scan fix bullet into it. Removes a duplicate ### Fixed subsection and stray blank lines that appeared after the ### Internal section.

Macroscope summarized 8c561c6.

Summary by CodeRabbit

  • Documentation
    • Updated the unreleased changelog with a Fixed section.
    • Reorganized unreleased entries under the appropriate headings.

Sixteen ordinary bug fixes had drifted under ### Security. That is my
doing: resolving twenty-odd CHANGELOG conflicts today merged bullets by
position without regard for the heading above them, so entries kept
landing in whichever section the conflict happened to open in.

Shipping that would have told everyone this release fixed seventeen
security issues. It fixed one: the Gemini missing-home token write, which
is the entry that created the section.

Each bullet was put back where the commit that introduced it had it, and
the section order matches 1.5.33: Added, Security, Fixed, Internal. No
bullet text changed and none was lost - still twenty.
@cloudflare-workers-and-pages

cloudflare-workers-and-pages Bot commented Aug 22, 2026

Copy link
Copy Markdown
Contributor

Deploying with  Cloudflare Workers  Cloudflare Workers

The latest updates on your project. Learn more about integrating Git with Workers.

Status Name Latest Commit Preview URL Updated (UTC)
✅ Deployment successful!
View logs
ceiling 8c561c6 Commit Preview URL

Branch Preview URL
Aug 22 2026, 09:43 PM

@coderabbitai

coderabbitai Bot commented Aug 22, 2026

Copy link
Copy Markdown

Review Change Stack

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro Plus

Run ID: 6ed7e7b1-9975-4758-9c43-7c38aab78644

📥 Commits

Reviewing files that changed from the base of the PR and between bd664f1 and 8c561c6.

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

Included review availability: Your plan provides up to 1 included review per hour; 0 remain after this review.


📝 Walkthrough

Walkthrough

The unreleased changelog adds a ### Fixed heading and reorganizes the internal cleanup and price-fingerprint entries under the appropriate sections.

Changes

Unreleased changelog organization

Layer / File(s) Summary
Reorganize unreleased entries
CHANGELOG.md
Adds the ### Fixed subsection and places the price-change scan consistency entry under Fixed. The internal cleanup entry appears before the ### Internal heading.

Estimated code review effort: 1 (Trivial) | ~2 minutes

Merge Risk: ⚪ Minimal · up to 8c561

This is a localized documentation-only changelog reordering with no product or security code impact; no actionable merge-blocking risk remains after normal checks and review.

Suggested reviewers: finesssee, itsmernb

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title clearly describes restoring unreleased changelog entries to their correct sections.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check. Docstring coverage is scoped to functions touched by this diff. Analyzed 0 functions across 0 files. (1 skipped: 1 unsupported.)
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.
✨ Finishing Touches 💡 1
🛠️ Fix failing CI checks 💡
  • Create stacked PR
  • Commit on current branch
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch fix/changelog-sections

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.

@btsouth
btsouth merged commit fcfdb07 into main Aug 22, 2026
13 of 14 checks passed
@btsouth
btsouth deleted the fix/changelog-sections branch August 22, 2026 21:48
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