Mark files and toggle hide marked files feature

[florentcigolotti]

Hi, first I would like to thank you a lot for this plugin, I am using it on a daily basis and it is awesome!

I have been working on adding a feature to mark files and add a key binding to toggle hide marked fies, I have been using it for 2 months now without issues and wanted to share if this is something you are willing to integrate.

The fork is here based on latest main: https://github.com/florentcigolotti/diffview-plus.nvim/tree/feature/hide-reviewed-files.

Here is an example with a diff:

With some files / folders marked using w to toggle marks:

And then using H to hide marked files:

Notice the `Changes (33/48)` and the `15 reviewed file(s) hidden` at the bottom

[dlyongemallo]

Is this a request to merge the fork? Are you asking for a code review?

[florentcigolotti]

It is first an open discussion if you think it is an interesting idea that could be integrated, and if so I could create the pull request that can then be reviewed for integration.

[dlyongemallo]

Two things that stand out: the H key should be configurable (via keymap override), and docs/README/help panel explanations are missing. Also, how does this compose with persist_selections? And there are a couple of thing which could probably be cleaned up. Otherwise, it looks fine to create a pull request.

[florentcigolotti]

Thanks for the review!

H uses the regular keymap override mechanism, I have added documentation and a regression test for that.
I've also added the documentation for the action and default mapping.

Regarding persist_selections, when enabled, the hide-reviewed state is stored alongside the selected file keys, using the same repository/revision scope.

Example:

{
  "/home/user/project:HEAD~1..HEAD": {
    "selections": [
      "working:lua/main.lua",
      "staged:README.md"
    ],
    "hide": true,
    "timestamp": 1787234567
  }
}

It is restored when the view is reopened and saved immediately when toggled.

For the migration of existing stores, it remains compatible - a missing hide entry will consider hide = false so files will be visible on load.

I have just added the pull request #309!