Use the new snip run and check subcommands

[JoaoCostaIFG]

2 new subcommands were added to snip:

• run - safer way to specify the command to be run
• check - check if a command should be run with snip. For example, shell built-ins like cd and export would be rejected.

This PR:

• Replaces hardcoded unproxyable list with a runtime call to snip check
• Deduplicate findFirstPipe call - the AST was traversed twice
• Replace console.warn with client.log - per opencode recommendations (console.warn in plugin code mangles the UI)

[VincentHardouin]

Thanks for your contribution ! I’ve seen your contribution on snip repo, they haven’t been released yet. I’ll take a closer look at your PR when I have a little more time. At a quick glance, I think we’ll need to check the installed version of snip to prompt users to update it.

[JoaoCostaIFG]

That's a good point, didn't think about that. I'll try to add a check for the snip version to preserve the previous behavior when using older version of snip (without the run/check sub-commands)

[VincentHardouin]

That's a good point, didn't think about that. I'll try to add a check for the snip version to preserve the previous behavior when using older version of snip (without the run/check sub-commands)

I don't think it's important to maintain backward compatibility. I'm more in favor of making sure that people using opencode-snip@latest are aware that they need to update snip.

[harshcurious]

Hey folks, any updates on this?