fix: keep insert-at-end order for non-newline captures

[chhoumann]

Summary

• fix a regression in insertAfter.insertAtEnd when the capture text does not end with a newline and the file has trailing EOF blank lines
• avoid shifting insertion into trailing EOF blank lines for non-newline entries, preserving append order across repeated captures
• keep existing spacing-preservation behavior for newline-terminated capture formats
• add regression coverage for # H\nA\n\n + X then Y (no trailing newline), asserting order remains X then Y

Validation

• bun run test -- src/formatters/captureChoiceFormatter-frontmatter.test.ts
• bun run test
• bun run lint
• bun run build-with-lint
• Obsidian CLI (vault=dev) repro check: # H\nA\n\n with format {{value}} now yields # H\nA\nX\nY after two captures

---

Summary by CodeRabbit

• Bug Fixes

  • Fixed insertion order preservation for repeated captures when content lacks trailing newlines, ensuring consistent behavior across successive operations.

• Tests

  • Added test coverage verifying insertion order preservation when formatting has no trailing newline and end-of-file blanks are present.

[vercel]

The latest updates on your projects. Learn more about Vercel for GitHub.

Project	Deployment	Actions	Updated (UTC)
quickadd	Ready	Preview	Feb 24, 2026 7:02am

[coderabbitai]

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between 8bb8ed4 and dd59b6b.

📒 Files selected for processing (2)

• src/formatters/captureChoiceFormatter-frontmatter.test.ts
• src/formatters/captureChoiceFormatter.ts

---

📝 Walkthrough

Walkthrough

The changes add a test case for handling insertion order preservation when formats lack a trailing newline, and update the capture formatter logic to anchor insert positions based on the inserted text's newline status, ensuring consistent ordering across successive captures.

Changes

Cohort / File(s)	Summary
Test Coverage src/formatters/captureChoiceFormatter-frontmatter.test.ts	Added test case "preserves insertion order when format has no trailing newline and EOF blanks exist" to verify deterministic insertion order across successive format operations without trailing newlines.
Core Formatter Logic src/formatters/captureChoiceFormatter.ts	Introduced insertedText parameter to findInsertAfterPositionAtSectionEnd() method; updated insertion positioning logic to anchor position when inserted text lacks trailing newline; replaced condition checking body.endsWith("\n") with fileContent.endsWith("\n").

Estimated code review effort

🎯 3 (Moderate) | ⏱️ ~20 minutes

Possibly related PRs

• PR #1102: Modifies captureChoiceFormatter insertion logic to handle files without a trailing newline and adjust content anchoring at EOF.
• PR #1054: Changes insert-after positioning logic in captureChoiceFormatter for EOF-newline behavior when computing insertion points.
• PR #1119: Modifies findInsertAfterPositionAtSectionEnd() to preserve spacing and ordering at end-of-file conditions.

Suggested labels

released

Poem

🐰 When text gets caught and formats flow,
Order's what we need to know!
No trailing newline? We don't despair,
Our anchored positions place with care,
Insert by insert, side by side,
Deterministic is our pride! ✨

🚥 Pre-merge checks | ✅ 3

✅ Passed checks (3 passed)

Check name	Status	Explanation
Description Check	✅ Passed	Check skipped - CodeRabbit’s high-level summary is enabled.
Title check	✅ Passed	The title clearly and specifically describes the main fix: preserving insertion order when using insert-at-end with captures that lack trailing newlines.
Docstring Coverage	✅ Passed	No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

_{✏️ Tip: You can configure your own custom pre-merge checks in the settings.}

✨ Finishing Touches

• 📝 Generate docstrings (stacked PR)
• 📝 Generate docstrings (commit on current branch)

🧪 Generate unit tests (beta)

• Create PR with unit tests
• Post copyable unit tests in a comment
• Commit unit tests in branch fix-insert-at-end-order-without-trailing-newline

---

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

• X
• Mastodon
• Reddit
• LinkedIn

_{Comment @coderabbitai help to get the list of available commands and usage tips.}

[cloudflare-workers-and-pages]

Deploying quickadd with    Cloudflare Pages

Latest commit:	dd59b6b
Status:	✅  Deploy successful!
Preview URL:	https://eb2f0138.quickadd.pages.dev
Branch Preview URL:	https://fix-insert-at-end-order-with.quickadd.pages.dev

View logs

[devin-ai-integration]

✅ Devin Review: No Issues Found

Devin Review analyzed this PR and found no potential bugs to report.

View in Devin Review to see 3 additional findings.

[github-actions]

🎉 This PR is included in version 2.12.0 🎉

The release is available on GitHub release

Your semantic-release bot 📦🚀