Feature request: add a toggle hide/show generated files, open generated file

[prscoelho]

Hey guys.

I can hide generated files but I know this isn't considered best practice, you're supposed to look at the generated file to see if you've messed things up. But having a bunch of generated files in your tree list can be a noisy.

I thought of two commands that might be useful:

• Toggle show/hide generated files (is this something that's doable through lsp?)
• Goto generated file for current buffer nevermind, there's an open compiled command!

[zth]

VSCode just released a feature for grouping files together. That might be something we could implement for the VSCode extension specifically, to nest generated files collapsed under the source file:

https://code.visualstudio.com/updates/v1_67#_explorer-file-nesting

^- Happy to guide/help if someone wants to take a stab at experimenting with/adding that.

[Kingdutch]

No update to the extension is needed if people are willing to edit their own settings :D

Open your VS Code settings and type "editor.filenesting", enable the feature and scroll down to patterns.

I've added two patterns:

This nests implementations under interfaces if they're present and nests all generated files under the main ReScript file. Cleaning up the explorer nicely. As you can see for vite.resi it does not create nesting when both an interface and compiled file are present but instead keeps them both under the top level interface directly :D

I think maybe the only added benefit for adding this to the rescript-vscode extension would be to automatically use the configured extension from bsconfig.json. However, if the extension were to hardcode this pattern we may actually decrease usability because people may want to enable nesting but omit certain patterns (e.g. I personally removed nesting yarn.lock under package.json because I prefer having those separate while using the nesting for ReScript).

[zth]

@Kingdutch very good point! Maybe we should just document how you can add that config yourself in the readme? Does anyone read the readme? 😄

[prscoelho]

Oh I didn't know about file nesting! This is great.

Please do add the open compiled command to the readme as well, I looked for it before opening this issue, but then I found it by looking at the code. I was also searching for "generated", which I guess is the wrong term?