Skip to content

feat(journal): create a new journal in incremental auto-vacuum - #682

Merged
klNuno merged 4 commits into
mainfrom
feat/journal-incremental-at-creation
Oct 20, 2026
Merged

klNuno merged 4 commits into
mainfrom
feat/journal-incremental-at-creation

Conversation

@klNuno

@klNuno klNuno commented Oct 20, 2026 •

Copy link
Copy Markdown
Contributor

Note

Clanker GPT-6.1 Sol is responding on behalf of meetsu

new installations reclaim unused database space incrementally from their first cleanup, without a later startup copy.

what changed

  • enable incremental auto-vacuum before the first journal page is written; existing journals keep their one-time conversion.
  • cover fresh journals and preserve legacy conversion coverage in journal-vacuum.test.ts.
  • document the two journal paths in docs/architecture.md.

verified

  • bun test packages/core/test/journal-vacuum.test.ts packages/core/test/journal.test.ts tests/e2e/agent-startup.test.ts: 40 pass, 0 fail, 342 assertions.
  • bun run check:e2e, core typecheck and bun run check:architecture pass after integrating main.
  • main already contains the startup-test bootstrap fix; conflict resolution keeps that version unchanged.
  • not verified: installed application and full local suite; CI on the new head is pending.

A new journal gives free pages back from its first compaction ticks and never needs the one-time VACUUM at start.
@github-actions github-actions Bot added documentation Improvements or additions to documentation core packages/core and packages/contracts labels Oct 20, 2026
@coderabbitai

coderabbitai Bot commented Oct 20, 2026 •

Copy link
Copy Markdown

Warning

Review limit reached

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

Next included review available in 16 minutes.

Check out review usage here.

View limit details

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

Learn how review limits work.

Review configuration:

⚙️ Run configuration
  • Configuration used: Repository: beboite/boite/.coderabbit.yaml
  • Review profile: CHILL
  • Plan: Advanced
  • Run ID: 1ba5f29a-3665-42db-a269-be110ffbc589
📥 Commits

Reviewing files that changed from the base of the PR and between 876592e and d19492e.

📒 Files selected for processing: 4
  • docs/architecture.md
  • packages/core/src/journal.ts
  • packages/core/src/journal/retention.ts
  • packages/core/test/journal-vacuum.test.ts
  • Autofix · Keep fixing CodeRabbit findings and required CI, and resolving merge conflicts

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.

@klNuno
klNuno enabled auto-merge October 20, 2026 21:35
@klNuno
klNuno added this pull request to the merge queue Oct 20, 2026
Merged via the queue into main with commit c652781 Oct 20, 2026
33 of 34 checks passed
@klNuno
klNuno deleted the feat/journal-incremental-at-creation branch October 20, 2026 22:31
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

core packages/core and packages/contracts documentation Improvements or additions to documentation

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant