Support wrapping a target with knausj common functions

[pkozlows]

Some helpful links:

• https://github.com/cursorless-dev/cursorless/blob/main/cursorless-talon/src/cursorless.talon#L19-L20
• https://github.com/cursorless-dev/cursorless/blob/main/cursorless-talon/src/actions/wrap.py
• https://github.com/knausj85/knausj_talon/blob/c830318a4aa96d3855e537c576fc79a5641aa98c/lang/tags/functions_common.py#L13

Can basically add an extra disjunct to the rule for cursorless_wrapper

• [ ] Also update https://github.com/cursorless-dev/cursorless/wiki/Talon-home-requirements to indicate we now depend on the capture <user.code_common_function>

[auscompgeek]

Workaround: take bat funk wrap <function> should work, but will change your cursor selection

[pokey]

Actually, now that I think about it, we used to support this, but got rid of it for some reason. See the first command of https://youtu.be/Py9xjeIhxOg @AndreasArvidsson any idea why we dropped that action?

[AndreasArvidsson]

I guess we were trying to limit Cursorless dependency on user/knausj implementations and we tried to avoid code insertion in Cursorless in favor of the change command.

[pokey]

Ok I did some git history sleuthing, and looks like this one hit the chopping block during DFA optimisation. I don't think that's a big concern anymore, so probably fine?

[pkozlows]

Take 'target' funk 'function' wrap. Is simply selecting the target and replacing it with the, for example, print function for me. Rap is getting recognized as tab, which doesn't usually happen, and i take as a sign that I'm doing the wrong command. If this type of workaround worked, I would actually have no problem. atm, doing "carve 'target'" "funk 'function'" "paste that" which works, but is more cumbersome.

[pokey]

Whoops; got the command wrong. I just edited the comment