proof of concept: exclude ts captures

[gennaro-tedesco]

A primitive proof of concept addressing #95: do you think such concept is in scope?

Logic:

• before inserting the node range in the captures table, check if said node must be excluded

Changes:

• modified lua/flash/plugins/treesitter.lua (containing the main logic)
• added utils functions and configuration options

[gennaro-tedesco]

The above should work as follows:

Screen.Recording.2023-07-03.at.16.29.44.mov

[IndianBoy42]

I think the more flexible method is to take a callback that can return a new list with whatever modifications it wants (or mutate the existing list as long as it returns it as well). vim.tbl_filter and vim.list_contains could be used to do this in one/two lines.

This way would also help #52

[gennaro-tedesco]

Yes, it may be a good idea to do so: ultimately let's hear from folke what he thinks, most likely he knows best where/how in the codebase to perform such changes.

[JoseConseco]

Option for callback that could filter / mutate flash results would be great. It would allow to remove first result from treesitter.jump() - which is always the same as word under cursor (and thus redundant)

[folke]

I went ahead and implemented @IndianBoy42 's suggestion.

You can now do:

require("flash").treesitter({
  filter = function(matches)
    ---@param m Flash.Match.TS
    return vim.tbl_filter(function(m)
      return not (m.node and vim.tbl_contains({ "comment" }, m.node:type()))
    end, matches)
  end,
})

You can of course also define that filter function in your config opts.modes.treesitter.filter

@gennaro-tedesco I assume this covers your need?

[gennaro-tedesco]

Yes, it works very well!

[folke]

Great!