Skip to content

feat: add thread discussion support#13

Merged
choplin merged 2 commits into
mainfrom
feat/thread-comments
Jul 8, 2025
Merged

feat: add thread discussion support#13
choplin merged 2 commits into
mainfrom
feat/thread-comments

Conversation

@choplin

@choplin choplin commented Jul 8, 2025

Copy link
Copy Markdown
Owner

Summary

  • Add thread management for comment discussions
  • Implement 1-thread-1-file storage structure
  • Support multiple threads per line with reply functionality

Key Features

  • Thread Comments: Each thread is stored in a single markdown file
  • Reply Support: Reply to existing comments with <leader>rr
  • Multiple Threads: Support multiple independent threads on the same line
  • Improved UI: Virtual text shows latest comment, thread count for multiple threads
  • Automatic Migration: Old comment format automatically migrated to new structure

Commands

  • <leader>rc: Create new comment (always creates new thread)
  • <leader>rr: Reply to existing comment (with thread selection if multiple)
  • <leader>rs: Show comments at cursor

Breaking Changes

Storage structure changed to support threads. Old comments will be automatically migrated to new structure.

Test Plan

  • Create new comment thread
  • Reply to existing comment
  • Create multiple threads on same line
  • Verify old comments are migrated correctly
  • Test thread selection UI when multiple threads exist
  • Verify virtual text displays correctly

choplin added 2 commits July 8, 2025 12:05
- Add thread management for comment discussions
- Implement 1-thread-1-file storage structure
- Support multiple threads per line
- Add reply functionality with thread selection
- Improve UI with latest comment display
- Simplify storage structure (remove comments/ subdirectory)
- Parse timestamps correctly when reading files
- Fix async thread selection behavior
- Remove auto-reply behavior from add comment

BREAKING CHANGE: Storage structure changed to support threads.
Old comments will be automatically migrated to new structure.
- Fix unused variable warnings
- Fix variable shadowing issues
- Resolve empty if branch warnings with luacheck ignore
- Apply stylua formatting
@choplin choplin merged commit b572454 into main Jul 8, 2025
3 checks passed
@choplin choplin deleted the feat/thread-comments branch July 8, 2025 03:38
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