Skip to content

docs: fold the unreleased notes into 0.5.2 and align the benchmark remeasure line - #128

Merged
bompus merged 1 commit into
mainfrom
release/0.5.2-notes
Oct 10, 2026
Merged

bompus merged 1 commit into
mainfrom
release/0.5.2-notes

Conversation

@bompus

@bompus bompus commented Oct 10, 2026 •

Copy link
Copy Markdown
Owner

0.5.2 is bumped on main (#124) but was never tagged or published: the tags stop at v0.5.1, GitHub's latest release is v0.5.1, and npm has no 0.5.2. The three entries added since then sat under Unreleased, which would have shipped a 0.5.2 whose notes missed them. They now sit in the 0.5.2 section and the Unreleased heading is gone. No version field changes.

docs/benchmarks.md said the benchmark results "have not been remeasured for 0.4.0". The README says "0.4.0, 0.5.0, 0.5.1 or 0.5.2". Both now say the README's list. The faster who and inbox query are not in the benchmarked build (67e187a) and were not remeasured.

Release audit against what 0.5.2 will contain (every commit since v0.5.1: #121, #123, #96, #124, #125, #126, #127):

Nothing is tagged or published here.

Summary by CodeRabbit

  • Documentation
    • The version 0.5.2 changelog now includes notes on swarmail who checking each running T3 thread directory once, inbox fetches avoiding message-body reads when bodies aren’t requested, and memory figures for the swarmail-mcp launch command.
    • The benchmark notice now clarifies that results have not been remeasured for versions 0.4.0, 0.5.0, 0.5.1, and 0.5.2.

…measure line

0.5.2 is bumped on main but was never tagged or published (tags stop at v0.5.1,
npm has no 0.5.2), so the three entries under Unreleased belong in its section.
docs/benchmarks.md said the results had not been remeasured for 0.4.0 only,
while the README lists 0.4.0 through 0.5.2; both now list the same versions.
@coderabbitai

coderabbitai Bot commented Oct 10, 2026 •

Copy link
Copy Markdown

Review in Change Stack →Review in Change Stack →

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration
  • Configuration used: Repository YAML (base), Organization UI (inherited)
  • Review profile: CHILL
  • Plan: Advanced
  • Run ID: 8f49d1b6-0745-4fe8-a539-1760ac31ece3

📥 Commits

Reviewing files that changed from the base of the PR and between e034ad7 and 5f0a976.


📒 Files selected for processing (2)
  • CHANGELOG.md
  • docs/benchmarks.md

Included review availability: This review used your included allowance. Your plan provides up to 10 included reviews per hour; 5 remain after this review.



📝 Walkthrough

Walkthrough

The changelog entries moved from Unreleased to version 0.5.2. The benchmark notice now lists versions 0.4.0, 0.5.0, 0.5.1, and 0.5.2 as not remeasured.

Changes

Release documentation

Layer / File(s) Summary
Update release notes and benchmark notice
CHANGELOG.md, docs/benchmarks.md
The three changelog entries now appear under version 0.5.2, and the benchmark notice lists versions 0.4.0, 0.5.0, 0.5.1, and 0.5.2.

Priority: ⬇️ Low

Merge Risk: ⚪ Minimal · up to 5f0a9

The documentation changes match the stated release-note and benchmark-disclosure updates, with no identified issue requiring resolution before merge.

Architecture Summary

Architecture risk: 🔵 Low · up to 5f0a9

The change affects 2 systems.

Changed systems: CHANGELOG.md, docs

Architecture concerns
No architecture-level concerns identified.

Review details

Systems and components

  • observed — CHANGELOG.md (service) was modified; 1 changed file maps to changed impact.
  • observed — docs (service) was modified; 1 changed file maps to changed impact.

Before / after behavior

  • observed — Modified behavior in CHANGELOG.md: Removed the Unreleased heading and its three entries; those entries are now listed under version 0.5.2.
  • observed — Modified behavior in CHANGELOG.md: Added the three entries to version 0.5.2: swarmail who queries each running T3 thread directory once, inbox fetches skip body reads when include_bodies is off, and the README lists memory figures for three swarmail-mcp launch commands.
  • observed — Modified behavior in docs/benchmarks.md: The notice expands the versions not remeasured from only 0.4.0 to 0.4.0, 0.5.0, 0.5.1, and 0.5.2.

Pre-merge checks | Passed 6
✅ Passed checks (6 passed)
Check name Status Explanation
Title check Passed The title clearly summarizes both documentation changes: moving unreleased notes into 0.5.2 and updating the benchmark statement.
Description check Passed The description clearly explains the changes, their purpose, release context, benchmark scope, version checks, and audit results. It does not reproduce the template checkboxes, but it provides the req…
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.
Suppressions Explained Passed The pull request changes only CHANGELOG.md and docs/benchmarks.md. The added lines move changelog text and update a benchmark sentence. No added lint, type-check, formatter, or configuration suppr…
Interface Changes Documented Passed The pull request changes only CHANGELOG.md and docs/benchmarks.md. The diff contains no MCP tool or argument changes, no swarmail command or flag changes, and no environment-variable changes. Th…

✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Commit to this branch
  • Create a new PR

  • Autofix · Keep fixing CodeRabbit findings and required CI, and resolving merge conflicts

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

@bompus
bompus merged commit 5356317 into main Oct 10, 2026
4 of 5 checks passed
@bompus
bompus deleted the release/0.5.2-notes branch October 10, 2026 03:12
bompus added a commit that referenced this pull request Oct 10, 2026
…e at random (#129)

Two tests picked a random port in 20000-40000 and started a server there, so a busy port failed the run with EADDRINUSE (seen on PR #128's first CI run). They now bind port 0; the test that starts its server late asks the kernel for a free port first.
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