Skip to content

Conversation

@john-mutuma
Copy link
Owner

@john-mutuma john-mutuma commented Nov 1, 2025

What does this PR do?

This PR consolidates the plugin ecosystem and cleans up legacy files to maintain a leaner, more maintainable codebase. The main focus is eliminating redundant search/replace tools and standardizing terminal API usage across the configuration.

  • Removed grug-far plugin in favor of scooter for all search/replace operations
  • Refactored scooter to use Snacks.terminal API (replacing toggleterm dependency)
  • Updated Snacks dashboard to use :FindReplace command for search/replace
  • Added "OpenCode:" prefix to all OpenCode keymap descriptions for better discoverability
  • Cleaned up legacy files (install.sh.backup, coc-settings.json, colors.sh, nvim-ide keymap)
  • Moved doc/ → docs/ directory for GitHub Pages compatibility
  • Streamlined AGENTS.md from 360 lines to ~70 lines with concise high-level summaries
  • Maintained all scooter keybindings and backdrop functionality

Why is it needed?

Plugin Consolidation:

  • Having both grug-far and scooter for search/replace created confusion and maintenance overhead
  • scooter provides all necessary search/replace functionality with better terminal integration
  • Using Snacks.terminal API ensures consistent terminal behavior across all features

Repository Cleanup:

  • Legacy files (install.sh.backup, coc-settings.json, colors.sh) were unused and cluttering the repo
  • nvim-ide keymap config referenced a plugin that is not installed
  • doc/ → docs/ naming aligns with GitHub Pages conventions
  • Compact AGENTS.md improves readability while maintaining essential project history

User Experience:

  • OpenCode keymap prefixes make commands easier to discover in which-key popup
  • Single search/replace tool reduces cognitive load for users

How have the changes been tested?

  • Verified :FindReplace command works from Snacks dashboard
  • Tested all scooter keybindings (Leader+sf, Leader+sr, Leader+fw, Leader+fr)
  • Confirmed backdrop functionality works with scooter terminal
  • Verified OpenCode keymaps display with "OpenCode:" prefix in which-key
  • Checked that all removed files were truly unused (no references in codebase)
  • Confirmed Neovim starts without errors after plugin removal
  • Validated lazy-lock.json updates applied cleanly

Screenshots (if applicable)

N/A - Terminal and keymap changes are functional improvements without visual UI changes

Checklist

  • All commits follow conventional commit format
  • Plugin consolidation tested and working
  • Legacy files removed safely (no broken references)
  • AGENTS.md maintains essential project history
  • lazy-lock.json updated to remove grug-far
  • Scooter terminal integration verified
  • OpenCode keymap prefixes added and tested
  • No breaking changes to existing workflows
  • Documentation (AGENTS.md) updated

John Mutuma added 8 commits November 1, 2025 11:07
- Remove nvim-notify dependency from noice.lua (use Snacks notifier)
- Remove zen-mode.nvim plugin (no keybindings configured)
- Delete orphaned nvim-ide keymaps file (plugin not installed)
- Update lazy-lock.json to reflect removed plugins

Consolidates notification system to use Snacks.nvim built-in notifier,
reducing dependencies and simplifying the plugin stack.
- Replace toggleterm.terminal.Terminal with Snacks.terminal()
- Maintain backdrop functionality and on_exit callbacks
- Update EditLineFromScooter and OpenScooterSearchText functions
- Remove toggleterm dependency from plugin list
- Terminal window detection now checks for snacks_terminal buffer pattern
- Remove grug-far plugin from init.lua
- Update Snacks dashboard to use :FindReplace (scooter) instead of :GrugFar
- scooter provides better terminal integration with custom backdrop
- Eliminates redundancy between two search/replace tools
@john-mutuma john-mutuma merged commit d9e6d65 into develop Nov 1, 2025
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.

2 participants