Skip to content

Comments

Add todo-comments plugin for annotation highlighting#165

Merged
rikuson merged 1 commit intomasterfrom
tafia-temporopontine
Feb 11, 2026
Merged

Add todo-comments plugin for annotation highlighting#165
rikuson merged 1 commit intomasterfrom
tafia-temporopontine

Conversation

@rikuson
Copy link
Owner

@rikuson rikuson commented Feb 10, 2026

Summary

  • Adds folke/todo-comments.nvim plugin to Neovim configuration
  • Highlights annotation comments like TODO, FIXME, HACK, WARN, NOTE, etc.
  • Loads lazily on VeryLazy event

Test plan

  • Run make cui to deploy the configuration
  • Open a file with TODO/FIXME comments and verify highlighting

🤖 Generated with Claude Code

Adds folke/todo-comments.nvim to highlight comments like TODO, FIXME,
HACK, WARN, NOTE, etc. in Neovim.

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
@claude
Copy link

claude bot commented Feb 10, 2026

PR Review: Add todo-comments plugin

✅ Positive Feedback

Code Quality

  • The plugin configuration follows the repository's Lazy.nvim patterns correctly
  • Clean, minimal configuration using the opts = {} pattern for default settings
  • Proper dependency declaration (plenary.nvim)
  • Consistent with other plugin files from the same author (folke/flash.nvim, folke/which-key.nvim)

Architecture Alignment

  • Correctly placed in roles/cui/templates/.config/nvim/lua/plugins/ following the modular plugin structure
  • Uses event = "VeryLazy" for lazy loading, consistent with other UI enhancement plugins
  • Will be automatically loaded by Lazy.nvim plugin manager

Security & Performance

  • No security concerns - this is a well-maintained plugin from a trusted author
  • Lazy loading ensures no performance impact on startup
  • No network requests or external dependencies beyond plenary

💡 Suggestions for Enhancement

Consider Adding Telescope Integration
The todo-comments.nvim plugin has excellent Telescope integration that other plugins in your config (like flash.nvim:1-53) actively use. Consider adding keybindings:

return {
  "folke/todo-comments.nvim",
  dependencies = { "nvim-lua/plenary.nvim" },
  event = "VeryLazy",
  opts = {},
  keys = {
    { "<leader>st", "<cmd>TodoTelescope<cr>", desc = "TODO/FIXME" },
    { "]t", function() require("todo-comments").jump_next() end, desc = "Next TODO" },
    { "[t", function() require("todo-comments").jump_prev() end, desc = "Previous TODO" },
  },
}

This would:

  • Add <leader>st (Search TODO) to search all TODOs via Telescope, consistent with your <leader>s* search patterns (telescope.lua:9)
  • Enable navigation between TODO comments with ]t and [t
  • Integrate nicely with your which-key setup (which-key.lua:17)

Alternative: Keep it minimal
If you prefer the minimal approach (which is valid!), the current configuration is perfect. The plugin will work out of the box with highlighting.

📋 Test Plan Enhancement

Your test plan is good. Consider also verifying:

  • Telescope integration works (if keybindings added): :TodoTelescope
  • Default keywords are highlighted: TODO, HACK, WARN, PERF, NOTE, FIX

Summary

Approval Status: ✅ Approved

This is a solid addition with no issues. The code is correct, follows all repository conventions, and adds useful functionality. The enhancement suggestions are optional - the PR is perfectly acceptable as-is for a minimal working configuration.

@rikuson rikuson merged commit 1d08d8d into master Feb 11, 2026
3 checks passed
@rikuson rikuson deleted the tafia-temporopontine branch February 11, 2026 01:08
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