change show_subtree command into ':tree-sitter-subtree' typable command

[the-mikedavis]

Following up on #1453 (comment), this is a default mapping for the new show_subtree command.

To start off I have it under space+t which is what I've been using, since it's pretty similar presentation-wise to the LSP-driven documentation hover space+k. I'm open to suggestions though.

[pickfire]

I don't think it is good to have this as default especially when this is used for debugging purposes. User can just add this to the config when they needed it.

If anything, I think maybe we could have :show_subtree command so that user can run without changing the config. Even with the current one, it is not usable when the screen is small either way. Maybe writing to file is better.

[hovsater]

I agree with @pickfire here. I don't see this meriting being in the default keymap. Adding a :show_subtree command make sense though and users who find themselves using this often may bind it at their disposal. 🙂

[the-mikedavis]

I like the sound of a typable command, I'll switch this PR to do that instead 👍

[the-mikedavis]

Is there a way to translate between commands::Context and compositor::Context? Looks like typable commands take the latter and regular commands the former, and they look similar structure-wise

[pickfire]

I don't think there is a way to convert. But if you find it similar, then maybe you can do From for them if possible to make it easier for future contributors, otherwise I think can just manually create it. I didn't touch work on these two parts but I don't think there is an issue.

[the-mikedavis]

As it turns out, the current! macro for getting the view and document works the same between the two cxs. The way you add callbacks is a bit different though

Now adding the typable command I think it might be better to only have the typable command. I don't see too much utility in being able to have a keymap for it when I can just :sub<tab>. What do you think, should I remove the old show_subtree regular command?

[pickfire]

Now adding the typable command I think it might be better to only have the typable command. I don't see too much utility in being able to have a keymap for it when I can just :sub. What do you think, should I remove the old show_subtree regular command?

Yeah, I think this should be good.

[archseer]

How about prefixing it with tree-sitter? :tree-sitter-subtree, alias :ts-subtree. Would match :tree-sitter-scopes