noe.trevino/haunt.nvim

Fork 1

feat: add quickfix list integration #9

Merged

menisadi merged 4 commits from feature/quickfix-integration into main

2026-01-20 13:22:33 -06:00

menisadi commented

2026-01-20 06:39:33 -06:00

(Migrated from github.com)

This PR adds an optional quickfix list integration.

Motivation:

The README mentions quickfix integration as a possible improvement.
haunt.sidekick already exposes annotation location. So I just built on that.

What’s included:

A helper that converts haunt annotations into quickfix items
A :HauntQuickfix command (with ! for current-buffer only)

Design notes:

Logic lives in sidekick and returns data, not UI decisions
No behavior changes unless the command is used
Keeps the feature fully optional
It doesn't conflict or break any existing stuff

As a non-snacks user this is a nice patch for me (I'd be happy with a Telescope integration either, but tbh, I never built such integration myself, but quickfix felt straightforward).

This PR adds an optional quickfix list integration. Motivation: - The README mentions quickfix integration as a possible improvement. - haunt.sidekick already exposes annotation location. So I just built on that. What’s included: - A helper that converts haunt annotations into quickfix items - A `:HauntQuickfix` command (with `!` for current-buffer only) Design notes: - Logic lives in sidekick and returns data, not UI decisions - No behavior changes unless the command is used - Keeps the feature fully optional - It doesn't conflict or break any existing stuff As a non-snacks user this is a nice patch for me (I'd be happy with a Telescope integration either, but tbh, I never built such integration myself, but quickfix felt straightforward).

noe.trevino requested changes

2026-01-20 11:14:26 -06:00

noe.trevino left a comment

This is in the wrong place, but otherwise, a wonderful PR!

If you could also remove the todo in the readme, and update it with these changes so users know its available, i would appreciate it.

This is in the wrong place, but otherwise, a wonderful PR! If you could also remove the todo in the readme, and update it with these changes so users know its available, i would appreciate it.

lua/haunt/sidekick.lua

this module is specifically meant for the integration with the sidekick plugin from folke. I actually think that this specific function should go into the store module. That module represent data persistence, and fetching (a database).

Lets move this function over there

this module is specifically meant for the integration with the sidekick plugin from folke. I actually think that this specific function should go into the `store` module. That module represent data persistence, and fetching (a database). Lets move this function over there

This one should go into the api module, since it is more public facing. The name and all can stay the same.

This one should go into the `api` module, since it is more public facing. The name and all can stay the same.

Nice!! That is freaking slick

Lets add a vim.notify("haunt.nvim: no bookmarks found.", INFO)

We don't want users to be confused in any way when using the plugin.

Soon, I am going to make a custom logger/notification config prop that users can toggle. For now, adding a vim.notify is pretty important IMO

Lets add a `vim.notify("haunt.nvim: no bookmarks found.", INFO)` We don't want users to be confused in any way when using the plugin. Soon, I am going to make a custom logger/notification config prop that users can toggle. For now, adding a `vim.notify` is pretty important IMO

menisadi commented

2026-01-20 12:42:57 -06:00

(Migrated from github.com)

Cool!
Okay so I’ve pushed updates:

moved get_quickfix_items to store
moved to_quickfix to api
added vim.notify when no bookmarks are found
updated README and marked the TODO as done. If you want it can be removed but it is your choice :)

Cool! Okay so I’ve pushed updates: - moved get_quickfix_items to store - moved to_quickfix to api - added vim.notify when no bookmarks are found - updated README and marked the TODO as done. If you want it can be removed but it is your choice :)

noe.trevino commented

2026-01-20 13:12:02 -06:00

Owner

@menisadi
Nice! I am going to approve this. I will be adding tests for this later today. I don't expect you to do that, but I will add you as a reviewer.
Thanks for the PR!

@menisadi Nice! I am going to approve this. I will be adding tests for this later today. I don't expect you to do that, but I will add you as a reviewer. Thanks for the PR!

🎉 1

menisadi commented

2026-01-20 13:17:46 -06:00

(Migrated from github.com)

Thanks for the great plugin. Been using it already today and I like it alot.

❤️ 1

noe.trevino commented

2026-01-20 13:22:52 -06:00

Owner

I am glad you like it!

noe.trevino commented

2026-01-20 14:50:37 -06:00

Owner

@menisadi I just realized, you said there was a helper for the quickfix, a user command. I don't see it. Were you wanting to add that as well? Just wanted to see if that was a mistake.

menisadi commented

2026-01-21 16:49:17 -06:00

(Migrated from github.com)

Oh you are right. There was in the first version. In the moving process I dropped it for some reason. It is not crucial I think. If someone wants a command they can bind the function easily.

No reviewers

No milestone

No project

No assignees

2 participants

Notifications

Due date

The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference

noe.trevino/haunt.nvim!9

No description provided.

Rows
Columns