Skip to content

release: HyperFix.nvim v2.0.0 - #4

Merged
TheOnliestMattastic merged 1 commit into
mainfrom
update
May 21, 2026
Merged

release: HyperFix.nvim v2.0.0#4
TheOnliestMattastic merged 1 commit into
mainfrom
update

Conversation

@TheOnliestMattastic

Copy link
Copy Markdown
Owner

Significant upgrade focused on minimalism and mnemonic speed:

  • Integrated sidekick.nvim (NES + AI CLI) replacing amp-ai.
  • Reorganized entire keybinding system into intuitive mnemonic groups.
  • Implemented 'double-tap' logic for high-frequency actions.
  • Slimmed down config by removing redundant plugins (DAP, Trouble, Grug-Far).
  • Standardized project name and ADHD-friendly documentation.
  • Enhanced Legend of Zelda thematic visual anchors.

Significant upgrade focused on minimalism and mnemonic speed:
- Integrated sidekick.nvim (NES + AI CLI) replacing amp-ai.
- Reorganized entire keybinding system into intuitive mnemonic groups.
- Implemented 'double-tap' logic for high-frequency actions.
- Slimmed down config by removing redundant plugins (DAP, Trouble, Grug-Far).
- Standardized project name and ADHD-friendly documentation.
- Enhanced Legend of Zelda thematic visual anchors.
@qodo-code-review

Copy link
Copy Markdown
Contributor

Review Summary by Qodo

HyperFix.nvim v2.0.0: Minimalist redesign with sidekick.nvim integration

✨ Enhancement

Grey Divider

Walkthroughs

Description
• Renamed project to HyperFix.nvim with standardized branding
• Replaced amp-ai with sidekick.nvim for modern AI integration
• Reorganized keybindings into intuitive mnemonic groups
• Implemented double-tap logic for high-frequency actions
• Removed redundant plugins (DAP, Trouble, Grug-Far)
• Cleaned up documentation and removed legacy files
Diagram
flowchart LR
  A["amp-ai plugin"] -->|replaced| B["sidekick.nvim + copilot.lua"]
  C["Legacy keymaps<br/>with brackets"] -->|reorganized| D["Mnemonic groups<br/>double-tap logic"]
  E["DAP, Trouble,<br/>Grug-Far plugins"] -->|removed| F["Focused config"]
  G["AGENTS.md,<br/>KEYMAPS.md"] -->|deleted| H["Streamlined docs"]
  B --> I["HyperFix.nvim v2.0.0"]
  D --> I
  F --> I
  H --> I

Loading

File Changes

1. init.lua 📝 Documentation +1/-1

Updated project name casing

init.lua


2. lua/config/keymaps.lua ✨ Enhancement +18/-18

Removed bracket mnemonics from descriptions

lua/config/keymaps.lua


3. lua/plugins/amp.lua Miscellaneous +0/-89

Deleted legacy AMP AI plugin

lua/plugins/amp.lua


View more (19)
4. lua/plugins/conform.lua ✨ Enhancement +2/-2

Changed format keymap to double-tap pattern

lua/plugins/conform.lua


5. lua/plugins/dap.lua Miscellaneous +0/-99

Removed debugging plugin entirely

lua/plugins/dap.lua


6. lua/plugins/flash.lua 📝 Documentation +4/-4

Cleaned up keymap descriptions

lua/plugins/flash.lua


7. lua/plugins/grug-far.lua Miscellaneous +0/-36

Removed search and replace plugin

lua/plugins/grug-far.lua


8. lua/plugins/mason.lua ✨ Enhancement +1/-1

Updated Mason keymap to quit group

lua/plugins/mason.lua


9. lua/plugins/mini.lua ✨ Enhancement +4/-19

Added dual keymaps for file explorer

lua/plugins/mini.lua


10. lua/plugins/render-markdown.lua 📝 Documentation +1/-1

Cleaned up toggle description

lua/plugins/render-markdown.lua


11. lua/plugins/sidekick.lua ✨ Enhancement +60/-0

New AI sidekick integration plugin

lua/plugins/sidekick.lua


12. lua/plugins/snacks.lua ✨ Enhancement +111/-243

Reorganized keymaps into mnemonic groups

lua/plugins/snacks.lua


13. lua/plugins/todo-comments.lua ✨ Enhancement +4/-21

Removed Trouble integration keymaps

lua/plugins/todo-comments.lua


14. lua/plugins/treesitter.lua 📝 Documentation +1/-1

Cleaned up toggle description

lua/plugins/treesitter.lua


15. lua/plugins/trouble.lua Miscellaneous +0/-72

Removed diagnostics sidebar plugin

lua/plugins/trouble.lua


16. lua/plugins/which-key.lua 📝 Documentation +25/-38

Removed bracket mnemonics from menus

lua/plugins/which-key.lua


17. lua/snacks-dashboard.lua 📝 Documentation +1/-1

Updated project name in dashboard

lua/snacks-dashboard.lua


18. AGENTS.md Miscellaneous +0/-287

Deleted AI assistant guidelines document

AGENTS.md


19. CHANGELOG.md 📝 Documentation +19/-4

Added v2.0.0 release notes

CHANGELOG.md


20. KEYMAPS.md Miscellaneous +0/-456

Deleted comprehensive keymaps reference

KEYMAPS.md


21. README.md 📝 Documentation +47/-205

Updated branding and feature descriptions

README.md


22. lazy-lock.json Dependencies +3/-8

Updated plugin dependencies

lazy-lock.json


Grey Divider

Qodo Logo

@qodo-code-review

qodo-code-review Bot commented May 21, 2026

Copy link
Copy Markdown
Contributor

Code Review by Qodo

🐞 Bugs (3) 📘 Rule violations (0)

Grey Divider


Action required

1. <leader>qd keymap collision 🐞 Bug ≡ Correctness
Description
lua/plugins/snacks.lua maps <leader>qd to Snacks.dashboard(), but lua/config/keymaps.lua
also maps <leader>qd to mini.sessions.select("delete"); since config.keymaps is required after
config.lazy, the later mapping overrides the earlier and one feature becomes unreachable. This is
a regression introduced by changing the dashboard key from <leader>qD to <leader>qd.
Code

lua/plugins/snacks.lua[R317-320]

Evidence
init.lua loads config.lazy before config.keymaps, and both Snacks and config keymaps define
<leader>qd with different actions, guaranteeing an override.

init.lua[24-27]
lua/plugins/snacks.lua[316-320]
lua/config/keymaps.lua[313-318]

Agent prompt
The issue below was found during a code review. Follow the provided context and guidance below and implement a solution

## Issue description
`<leader>qd` is defined twice (dashboard and delete-session). Because `init.lua` loads `config.keymaps` after `config.lazy`, whichever mapping is applied last wins, making the other action unreachable.

## Issue Context
- `snacks.nvim` is `lazy = false`, so its `keys` are registered during Lazy startup.
- `config.keymaps` runs immediately after `config.lazy` and will override conflicting mappings.

## Fix Focus Areas
- lua/plugins/snacks.lua[316-320]
- lua/config/keymaps.lua[313-318]
- init.lua[24-27]

## Suggested fix
Pick one owner for `<leader>qd` and move the other mapping:
- Option A (minimal regression fix): change Snacks dashboard back to `<leader>qD`.
- Option B: keep dashboard on `<leader>qd` and move session delete to a different chord (e.g. `<leader>qD` or `<leader>qX`).
Then update any which-key registrations/docs accordingly.

ⓘ Copy this prompt and use it to remediate the issue with your preferred AI generation tools



Remediation recommended

2. Implicit copilot dependency 🐞 Bug ⚙ Maintainability
Description
lua/plugins/sidekick.lua configures sidekick NES behavior alongside a separate Copilot spec, but
the sidekick spec does not declare Copilot as a dependencies relationship, making the coupling
implicit and harder to reason about/test. This can lead to fragile configuration as the two plugins
evolve or are toggled independently.
Code

lua/plugins/sidekick.lua[R11-32]

Evidence
The sidekick plugin is configured with NES enabled and the file separately defines copilot.lua, but
without an explicit dependency edge between them.

lua/plugins/sidekick.lua[11-32]

Agent prompt
The issue below was found during a code review. Follow the provided context and guidance below and implement a solution

## Issue description
`folke/sidekick.nvim` is configured to use Copilot/NES, but the spec does not explicitly declare `zbirenbaum/copilot.lua` as a dependency of sidekick. This makes the relationship implicit and increases the chance of future breakage when adjusting load conditions.

## Issue Context
The file currently returns two separate specs (Copilot and Sidekick) without an explicit `dependencies = { ... }` link.

## Fix Focus Areas
- lua/plugins/sidekick.lua[11-32]

## Suggested fix
Make the relationship explicit:
- Add `dependencies = { "zbirenbaum/copilot.lua" }` to the `folke/sidekick.nvim` spec.
- Optionally move the Copilot spec under `dependencies` (as a full spec table) and remove the top-level Copilot entry, so sidekick fully owns its required backend configuration.

ⓘ Copy this prompt and use it to remediate the issue with your preferred AI generation tools



Advisory comments

3. Todo keymap docs stale 🐞 Bug ⚙ Maintainability
Description
The header comment in lua/plugins/todo-comments.lua still advertises <leader>xt/<leader>xT
sidebar keymaps, but those mappings no longer exist in the plugin’s keys list after this PR. This
will mislead maintainers/users relying on in-file documentation.
Code

lua/plugins/todo-comments.lua[R89-98]

Evidence
The file’s KEYMAPS comment lists <leader>xt/<leader>xT, but the keys table only defines ]t,
[t, <leader>st, and <leader>sT.

lua/plugins/todo-comments.lua[2-11]
lua/plugins/todo-comments.lua[82-118]

Agent prompt
The issue below was found during a code review. Follow the provided context and guidance below and implement a solution

## Issue description
`lua/plugins/todo-comments.lua` comments list keymaps that are no longer defined (`<leader>xt/<leader>xT`).

## Issue Context
This PR removed the Trouble sidebar keymaps but did not update the file header comment.

## Fix Focus Areas
- lua/plugins/todo-comments.lua[2-11]
- lua/plugins/todo-comments.lua[82-118]

## Suggested fix
Update the header comment to reflect the current bindings (only `]t`, `[t`, `<leader>st`, `<leader>sT`) or reintroduce the missing keymaps if still desired.

ⓘ Copy this prompt and use it to remediate the issue with your preferred AI generation tools


Grey Divider

Qodo Logo

@gemini-code-assist gemini-code-assist Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Code Review

This pull request rebrands the project to HyperFix.nvim and introduces version 2.0.0, featuring a significant overhaul of the AI integration and keybinding system. The legacy amp-ai has been replaced with sidekick.nvim and copilot.lua, while several plugins like nvim-dap, trouble.nvim, and grug-far.nvim were removed to simplify the setup. Keybindings have been reorganized into mnemonic groups with new double-tap logic for common actions. Feedback suggests improving the plugin specification by moving copilot.lua into the dependencies of sidekick.nvim and generalizing configuration comments to make the template more accessible to a broader audience.

Comment thread lua/plugins/sidekick.lua
Comment thread lua/plugins/sidekick.lua
Comment thread lua/plugins/snacks.lua
@TheOnliestMattastic
TheOnliestMattastic merged commit 60ffebb into main May 21, 2026
2 checks passed
@TheOnliestMattastic
TheOnliestMattastic deleted the update branch May 21, 2026 18:04
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