refactor(neovim): reorganize plugin configuration into individual files

[iammohitsakhuja]

Summary

Refactored Neovim plugin configuration from category-based files to individual plugin files for better maintainability and easier navigation.

Why This Change?

The Neovim configuration had grown considerably since its initial setup, with plugins organized into 9 category-based files (editor.lua, ui.lua, completion.lua, etc.). This structure made it increasingly difficult to:

• Find specific plugin configurations - Had to remember which category each plugin belonged to
• Navigate the codebase - Large files with multiple plugins made scrolling and searching tedious
• Review changes in git - Modifications to one plugin would show up alongside unrelated plugins in the same category file
• Add or remove plugins - Required editing category files and manually managing the plugin list

What Changed

Before

lua/plugins/
├── init.lua (manual imports)
├── editor.lua (12 plugins)
├── ui.lua (11 plugins)  
├── completion.lua (2 plugins)
├── git.lua (1 plugin)
├── ai.lua (4 plugins)
├── formatter.lua (1 plugin)
├── linter.lua (1 plugin)
├── treesitter.lua (4 plugins)
└── lsp/init.lua (3 plugins)

After

lua/plugins/
├── init.lua (auto-discovery)
├── alpha-nvim.lua
├── auto-session.lua
├── blink-cmp.lua
├── bufferline.lua
... (39 individual plugin files)

Technical Changes

• Split 9 category files into 39 individual plugin files - One file per plugin in a flat structure
• Added LazySpec type annotations - Better LSP support and type checking for all plugin configurations
• Simplified plugins/init.lua - Now returns empty table, leveraging lazy.nvim's native auto-discovery
• Preserved all functionality - No behavioral changes, purely organizational refactoring

Benefits

Developer Experience

• ✅ Easy navigation - Alphabetically sorted files, instantly find any plugin
• ✅ Simple maintenance - Add/remove plugins by creating/deleting a single file
• ✅ Better git diffs - Changes isolated to individual files for clearer review
• ✅ Auto-discovery - New plugin files automatically loaded by lazy.nvim

Code Quality

• ✅ Type safety - LazySpec annotations provide better LSP autocomplete and error checking
• ✅ Consistency - Uniform structure across all plugin files
• ✅ Modularity - Each plugin is self-contained with its config, keymaps, and dependencies

Testing

• Neovim starts without errors
• All plugins load correctly via lazy.nvim auto-discovery
• No broken symlinks remain after stowing
• Pre-commit hooks pass (stylua formatting applied)

Migration Notes

This is a pure refactoring - no functional changes. The plugin configurations remain identical to their previous versions, just reorganized into individual files.

🤖 Generated with Claude Code

[claude]

Claude finished @iammohitsakhuja's task —— View job

---

Code Review Complete

I've thoroughly reviewed this Neovim plugin refactoring PR. Here's my comprehensive analysis:

Review Checklist

• Analyze the refactoring approach and overall structure
• Review the auto-discovery mechanism changes
• Check individual plugin file consistency and type annotations
• Identify potential bugs or configuration issues
• Assess performance and security considerations
• Evaluate maintainability improvements
• Provide comprehensive feedback

---

✅ Overall Assessment: EXCELLENT REFACTORING

This is a well-executed organizational refactoring that significantly improves the codebase structure without changing functionality.

🎯 Code Quality & Best Practices

Strengths:

• Consistent TypeScript annotations: All files properly use ---@module "lazy" and ---@type LazySpec for excellent LSP support
• Uniform structure: Each plugin file follows the same pattern - clean and predictable
• Proper auto-discovery: Smart use of lazy.nvim's native auto-discovery by returning empty table in init.lua
• Excellent documentation: Clear comments explaining the auto-discovery mechanism
• Alphabetical organization: 39 plugin files are naturally sorted, making navigation intuitive

Minor suggestions:

• Consider adding brief header comments in complex plugin files like blink-cmp.lua and nvim-treesitter-textobjects.lua to explain their purpose
• The TODO comment in blink-cmp.lua:154 should be tracked or resolved

🐛 Potential Issues Found

Low Impact:

1. Telescope extension loading (telescope.lua:64): Loading fidget extension without explicit dependency check
2. AI mode conditional logic (lualine.lua:38): Dynamic dependency on vim.g.ai_mode - ensure this variable is set before plugin loading
3. Environment variable usage (copilot-chat.lua:13): vim.env.USER fallback is good practice

No Breaking Changes Detected: All configurations appear to be exact transfers from the original category files.

🚀 Performance Considerations

Positive impacts:

• Reduced memory footprint: Auto-discovery eliminates manual import table
• Faster startup: lazy.nvim can efficiently scan and load individual files
• Better caching: Smaller, focused files improve Lua module caching
• Parallel loading: Individual files can be processed independently

No performance regressions identified.

🔒 Security Assessment

Clean and secure:

• No hardcoded secrets or sensitive information
• Proper conditional loading based on ai_mode settings
• Safe file operations and plugin configurations
• No suspicious external dependencies

📊 Maintainability Improvements

Major benefits achieved:

• Easy plugin management: Add/remove by creating/deleting single files
• Clear git diffs: Changes isolated to individual plugin files
• Better navigation: Jump directly to any plugin alphabetically
• Reduced cognitive load: No need to remember category classifications
• LSP-friendly: Type annotations provide excellent autocomplete and error checking

🧪 Test Coverage

Current status: No automated tests detected for Neovim configuration.

Recommendations:

• Consider adding basic integration tests for critical plugins (LSP, completion, telescope)
• Test auto-discovery mechanism with a minimal setup
• Validate plugin loading order and dependencies

📝 Migration Quality

Excellent execution:

• All 39 plugins correctly migrated from 9 category files
• Type annotations added consistently
• No functional changes - pure organizational refactoring
• Proper dependency management preserved

🎯 Specific File Reviews

blink-cmp.lua: Complex but well-structured completion configuration with good error handling
nvim-lspconfig.lua: Clean LSP setup with comprehensive language server coverage
telescope.lua: Solid configuration with proper trouble.nvim integration
lualine.lua: Sophisticated statusline with truncation logic and AI integration

🏆 Final Recommendation

APPROVE - This refactoring significantly improves maintainability while preserving all functionality. The migration is clean, consistent, and follows Neovim configuration best practices.

Next steps:

1. Resolve the TODO in blink-cmp.lua
2. Consider adding basic integration tests
3. Monitor plugin loading performance after deployment

---