Retain scratch comments across buffer lifecycle changes #2

Closed
eleith wants to merge 13 commits from feat/session-retained-comments into main
pull from: feat/session-retained-comments
Owner

Summary

  • Keep comments in memory when source buffers unload, delete, or wipe; reattach by buffer name when the saved text matches.
  • Let the built-in :CommentList jump to and delete comments by stable ID, including comments from closed buffers.
  • Cover ordinary files and delayed virtual-buffer reads, and document session retention.

Validation

  • make check passed before and after every commit (164 tests at HEAD).

Review note

Asynchronous URI sources without a generic completion signal remain active/parked on an unresolved mismatch instead of being incorrectly marked inactive. No CodeDiff-specific integration or disk persistence is added.

## Summary - Keep comments in memory when source buffers unload, delete, or wipe; reattach by buffer name when the saved text matches. - Let the built-in `:CommentList` jump to and delete comments by stable ID, including comments from closed buffers. - Cover ordinary files and delayed virtual-buffer reads, and document session retention. ## Validation - `make check` passed before and after every commit (164 tests at HEAD). ## Review note Asynchronous URI sources without a generic completion signal remain active/parked on an unresolved mismatch instead of being incorrectly marked inactive. No CodeDiff-specific integration or disk persistence is added.
eleith added 4 commits 2026-09-29 23:19:44 +00:00
eleith added 1 commit 2026-09-29 23:33:04 +00:00
eleith added 1 commit 2026-09-29 23:42:53 +00:00
Tighten README wording for session retention
ci/woodpecker/pr/ci Pipeline was successful
b4877b79cf
eleith added 1 commit 2026-09-30 00:26:24 +00:00
eleith added 1 commit 2026-09-30 12:13:14 +00:00
Make the comment list the sole review manager
ci/woodpecker/pr/ci Pipeline was successful
071a04582b
eleith added 1 commit 2026-09-30 12:20:23 +00:00
Open existing comments from their exact range
ci/woodpecker/pr/ci Pipeline was successful
9e2cd57d8f
eleith added 1 commit 2026-09-30 12:54:29 +00:00
Open comment cards only on quickfix activation
ci/woodpecker/pr/ci Pipeline was successful
b2b8abf835
eleith added 1 commit 2026-09-30 13:52:34 +00:00
Keep comment cards inside the editing window
ci/woodpecker/pr/ci Pipeline was successful
d3be09d99b
eleith added 1 commit 2026-09-30 14:11:14 +00:00
eleith added 1 commit 2026-09-30 14:57:13 +00:00
Separate quickfix navigation from comment editing
ci/woodpecker/pr/ci Pipeline was successful
3baf51a14f
Author
Owner

Superseded by #3. It has the same final file tree, rebuilt from main as four reviewable commits. Please review #3 instead.

Superseded by #3. It has the same final file tree, rebuilt from `main` as four reviewable commits. Please review #3 instead.
eleith closed this pull request 2026-09-30 15:21:50 +00:00

Pull request closed

Please reopen this pull request to perform a merge.
Sign in to join this conversation.
No Reviewers
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: eleith/scratch-comments.nvim#2