docs(release): finalize the v1.8.0 changelog section - #90
Conversation
Renames '## Unreleased' to the real version heading — release.yml and release-prep.yml both extract the release body with scripts/changelog-section.sh, which matches on '## <version>', so an Unreleased heading strands the tag with no publishable build. Also corrects two things the notes got wrong, since this text ships verbatim as the GitHub Release body friends read before auto-updating: - The developer claim that fresh checkouts 'build and run' without fetch-llama-server.sh contradicted our own README. build.rs writes a DEBUG-only placeholder and its release arm panics on it, so a release-profile build still needs the script. Moved to a 'For developers' section that matches README.md. - The engine download size read ~10-20 MB in the notes, the Settings row, and a store comment. Measured against the pinned b9095 assets it is 8.6-16.1 MB, so all three now read 8-16 MB. Adds the three user-visible changes the notes had omitted: the idle network-footprint increase from the relay presence leg, the AI pane's rewritten privacy line, and a known issue covering the in-session Invite picker, which still renders a limited-connection friend as plain Available. Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
|
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: Organization UI Review profile: CHILL Plan: Pro Plus Run ID: 📒 Files selected for processing (3)
📜 Recent review details⏰ Context from checks skipped due to timeout. (3)
🧰 Additional context used📓 Path-based instructions (5)src/**/*.{ts,tsx,css}📄 CodeRabbit inference engine (CLAUDE.md)
Files:
src/**/*.{ts,tsx}📄 CodeRabbit inference engine (CLAUDE.md)
Files:
**/*.{ts,tsx,js,jsx,css,json,md,rs,toml,yml,yaml}📄 CodeRabbit inference engine (CLAUDE.md)
Files:
src/strings.ts📄 CodeRabbit inference engine (CLAUDE.md)
Files:
CHANGELOG.md📄 CodeRabbit inference engine (CLAUDE.md)
Files:
🔇 Additional comments (3)
📝 WalkthroughWalkthroughUpdates the changelog to v1.8.0 and documents AI engine installation, path resolution, friends presence, privacy, known issues, and development behavior. Adjusts llama.cpp download-size wording in settings documentation and user-facing help text. ChangesRelease documentation
Estimated code review effort: 1 (Trivial) | ~5 minutes Possibly related PRs
Suggested reviewers: 🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 passed)
✨ Finishing Touches📝 Generate docstrings
✨ Simplify code
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. Comment |
Cuts the release notes for v1.8.0 (minor: the range adds a new persisted setting, a new Settings row, and a three-state friends list — the same shape that earned minors for 1.5.0/1.6.0/1.7.0).
This must land before Release-prep can run: both release paths extract the body with
scripts/changelog-section.sh, which matches## <version>. With## Unreleasedin place,changelog-section.sh 1.8.0exits 1 — on the one-click path that aborts before the bump/tag/push (fails safe), but on the manual path it fails after the tag is immutable, stranding a pushed tag with no build.Two corrections, because this text ships verbatim as the Release body
scripts/fetch-llama-server.sh.src-tauri/build.rs:41-55writes a debug-only placeholder and its release arm panics on it — so a release-profile build still requires the script, exactly asREADME.md:330-336says. The changelog was contradicting our own README. Rewritten into a### For developerssection that matches.src/strings.ts), and a store comment. Measured live against the pinned tag-b9095 assets: 8,643,613 B (macos-arm64) to 16,054,993 B (win-cpu-x64). All three now read 8–16 MB, so the release notes, the in-app copy, and the actual bytes agree.Three omissions added
src/strings.ts), changed in this range but undocumented.### Known issuefor the in-session Invite picker, which still renders a limited-connection friend as plain Available — a new failure mode I74 introduces. Deliberately shipped documented rather than fixed: batching a code change here would invalidate the CI-green-on-exact-SHA gate Release-prep requires. Filed for 1.8.1.Also softened "direct connection proven" to account for
LIMITED_CONNECTION_SETTLE_MS(120s), during which a relay-only friend renders plain green.Verification
bash scripts/changelog-section.sh 1.8.0→ exit 0, non-empty body.prettier --check,check-strings,check-tokens,check-contrast,lint,build, and 871 tests all pass. No test or story asserts the old size string;Unreleasedno longer appears anywhere in the tree.Backed by a 22-agent readiness audit whose blocking findings each survived an independent adversarial refutation pass. It separately confirmed the release is safe to auto-deliver: no new SQLite migrations, an additive settings key with a tested absent→default path, an untouched updater pubkey/endpoint, untouched identity derivation, and a relay-presence namespace a 1.7.1 peer neither publishes nor subscribes.
🤖 Generated with Claude Code
Summary by CodeRabbit