helpers: `toLuaObject` fails to escape strings containing `'` about nixvim

[rummik]

A workaround seems to be using double quotes instead of single quotes within the passed string

[pta2002]

Ah, seems to be an issue with using escapeShellArg. Guess it's about time I implement my own escaping function (it escapes ' to ''\''', which works in shell due to implicit concatenation but not in lua!)

[rummik]

@pta2002 I actually have some progress towards this if you'd like me to submit a PR

[rummik]

For now I'd suggest using builtins.toJSON specifically for string conversion, since it should be compatible in most cases

[rummik]

I'm a bit confused about this part of toLuaObject

https://github.com/pta2002/nixvim/blob/a601a75d0cae8229f1416506bd172e4e4aef3ce7/plugins/helpers.nix#L17-L20

Specifically I'm not sure I understand what the special case for filtering out nil and empty tables is for, and I don't know what the significance of @ prefixed keys are in Lua 😅

[rummik]

As far as I can tell, the condition on line 17 is never reached in the current codebase

[pta2002]

I'm a bit confused about this part of toLuaObject

https://github.com/pta2002/nixvim/blob/a601a75d0cae8229f1416506bd172e4e4aef3ce7/plugins/helpers.nix#L17-L20

Specifically I'm not sure I understand what the special case for filtering out nil and empty tables is for, and I don't know what the significance of @ prefixed keys are in Lua sweat_smile

Honeslty I genuinely do not remember why I put that there... I'm filtering out nil so code does not end up with a bunch of x = nil, because that'll be by far the most common value and I don't want to bloat up the vimrc

[rummik]

For now it's just a simple fix. I don't really feel comfortable changing much in helpers without some kind of tests to make sure I don't break everything 😅