Hi! First of all, thanks for the awesome plugin.
I tried configuring this for php like so

require("ts-node-action").setup({
    php = {
        ["arguments"] = require("ts-node-action").toggle_multiline,
    },
})

To check if the default action will work, but I get the:

No action defined for 'php' node type: 'arguments'

Do you have any ideas?

We need a nice way to test bundled functions. I'm working on it :)

1. In Intellij IDEA, when the user presses <C-S-Enter> when the cursor is positioned on a line of the code where the line can be finished by adding a ; at the end of the line, the editor would add a semicolon to finish the statement.

2. If the cursor is positioned on a function declaration or a function definition, <C-S-Enter> would add parentheses and brackets for the function parameters and the function body where it's appropriate, and also moves the cursor to where it's appropriate.

3. When calling a function, IDE would add parentheses when the user presses <C-S-Enter> on the function identifier.

There may be some more use cases what IntelliJ "complete statement" action does that I'm not aware of, but if above functionalities can be implemented it would be great additions to coding experience with Neovim.

If anyone is unfamiliar with what the "complete statement action" is in IntelliJ, see https://www.jetbrains.com/idea/guide/tips/complete-statement/

Before:

After:

I noticed for split actions the new lines are not indented properly. I think this can be addressed by applying = reflow on all multiline replacements? Won't that work?

Great plugin concept by the way. It's extremely powerful and helps showcase what treesitter can really do.

Separately from this I'm hoping to contribute a toggle action for js/ts to convert a template string expression into regular strings and vice versa. Something you couldn't dare attempt with regexes!

Given the 'pre-1.0' nature of this plugin, there are going to be some breaking changes as we figure out a nice API. I'll post those changes here, so subscribe if you're interested.

Hi! I did some afternoon coding and implemented a fun feature for myself. I was wondering if you think this is something that could be included in ts-node-action itself.

Here is a demo:

If you think this might be a good addition, I might have the time to open a PR for further discussion.

Thanks!

I want to use toggle_multiline() in SQL therefore I have specified the two node types in my config like this:

sql = {
    ["select_expression"] = require("ts-node-action.actions").toggle_multiline(),
    ["column_definitions"] = require("ts-node-action.actions").toggle_multiline(),
},

Here is a SQL snippet and Treesitter AST as an example:

create table tab (
    a int,
    b float,
    c text
);

AST:

statement [0, 0] - [4, 2]
  create_table [0, 0] - [4, 1]
    keyword_create [0, 0] - [0, 6]
    keyword_table [0, 7] - [0, 12]
    table_reference [0, 13] - [0, 16]
      name: identifier [0, 13] - [0, 16]
    column_definitions [0, 17] - [4, 1]
      column_definition [1, 4] - [1, 9]
        name: identifier [1, 4] - [1, 5]
        type: int [1, 6] - [1, 9]
          keyword_int [1, 6] - [1, 9]
      column_definition [2, 4] - [2, 11]
        name: identifier [2, 4] - [2, 5]
        type: float [2, 6] - [2, 11]
          keyword_float [2, 6] - [2, 11]
      column_definition [3, 4] - [3, 10]
        name: identifier [3, 4] - [3, 5]
        type: keyword_text [3, 6] - [3, 10]

When I trigger toggle_multiline() to collapse the column_definitions the whitespaces between the identifier and the type are lost

result:

create table tab (aint,bfloat,ctext);

How can I fix this?

I am using the SQL grammer that is shipped with nvim-treesitter.

Pre-PR

DEMO

Currently toggle_multiline for Lua cannot wrap a table in one line:

Curious why did I have to add the tsx = line to my setup function?

local ts_node_action = require("ts-node-action")
...
ts_node_action.setup({
    tsx = ts_node_action.node_actions.typescriptreact,
})

I would assume default node actions would just work but they were not enabled until I added this.

Apologies for asking such a basic question, but I'm having a hard time getting keymaps to work for the builtin functions.

I can successfully create and use the keymap for node_action, as described under Usage, e.g.:

vim.keymap.set(
    { "n" },
    "K",
    require("ts-node-action").node_action,
    { desc = "Trigger Node Action" },
)

But when I try something similar for the builtin actions, they aren't working. For example, I tried a few variations of:

vim.keymap.set(
    { "n" },
    "C",
    require('ts-node-action.actions').cycle_quotes(),
    { desc = "Cycle Quotes" },
)

My setup function is currently empty, e.g. ts_node_action.setup({}). I assumed that for the builtins, I wouldn't need to specify the languages and node types, but maybe that's my issue?

Any help is appreciated!

We can attach more than one action to a given node type, but this forces us to make a choice each time. For example I am going to author a new node action on a property_identifier for a few of the languages I use often which will grab the var name and make a simple debug print out of it in the next line of code.

But I don't want to have to delete the existing actions.cycle_case() mapped to property_identifier, and neither do i want to insert another action to make to trigger this.

I'd prefer to be able to establish different sets of node actions that are mapped to different key binds! So I could have, say, a mapping that will apply "traditional" actions like cycle case and toggle multiline/operator/quote etc, a separate mapping that applies "expanding" refactoring automations as an example, another mapping that applies "contracting" refactoring automations... etc. You get the idea.

Hi!

I just came across your plugin. It's really nice! Just wanted to tell you that there is actually a similar one: Wansmer/treesj.

You may want to unite efforts to create a unified version tree-sitter node editor.

Cheers! 😏

I would like to customize the possible cycle_case formats for variables.

So for example in JS I only use camelCase and SCREAMING_SNAKE_CASE.
I would like to set this like

actions.cycle_case({'camel_case', 'screaming_snake_case'})

For python I also need normal snake_case. However I use camelCase there very seldom.
This would look like

actions.cycle_case({'snake_case', 'screaming_snake_case', 'camel_case'})

Here it is important that we try to remember the priority.
When I execute the action on a variable SOME_VARIABLE I want to to change to the first other case option that is not it's current case option and was not tried yet. So it would change to someVariable. If I hit the action again it should change to someVariable.

It might be that this is to unintuitive. I'm also fine if I can just set the order. So we always pick the style that comes right after the current style given parameter list.
So for SOME_VARIABLE it should become someVariable.
If we go this route maybe we could also add another parameter, backwards to the cycle_case() function, that allows for a forwards and backwards mapping (Maybe <M-A>, <M-X>?)

Ideally use constants for the different styles, not as I did with strings ;-)

According to my test, require("ts-node-action").node_action() doesn't support dot . repeat now.

This way instead of using ui.select() the user can bind the different actions to different keys, if they are common enough to warrant the quicker activation

I would like to integrate the cycle_case action with LSP rename, so that I can change case of all occurrences of a function or say variable at once. Is it possible to bind a code action with another function? Also, is it possible to apply only one type of code action to a node, say maybe I want to bind a key only for the "Cycle Case" code action?

Hi, first of all, thanks for the wonderful framework.

I have configured it to work via null-ls (none-ls actually) by sources = { null_ls.builtins.code_actions.ts_node_action }

And everything I've tried works fine, except the quotes. Rather, they not only don't work, it would be half the trouble, but they lead to an error that knocks down the entire ts-node-action and it no longer works.

The error appears when I move the cursor to the quotation from either side, and a message appears:

[null-ls] failed to run generator: ...are/nvim/lazy/ts-node-action/lua/ts-node-action/init.lua:156: attempt to index local 'tbl' (a boolean value)

Hi there! loving the plugin, really cool work!

I think it might be cool to integrate the node-actions into the lsp code actions.

So if my cursor is on top of a bool and I run vim.lsp.buf.code_action(), the output would also list "Toggle Boolean" as an option.

I have no idea if that is possible nor how to do it, I just though I'd leave the idea hanging ^^

My changes are summarized here: 3a0ddbd

Here is the error I get in nvim. the first 3 lines compose the menu made by vanilla ui.select:

Select Action
1: Conceal
2: Split Join
Type number and <Enter> or click with the mouse (q or empty cancels):
E5108: Error executing lua: vim/_editor.lua:420: nvim_exec2(): Vim(normal):E5108: Error executing lua ...are/nvim/lazy/ts-node-action/lua/ts-node-action/init.lua:130: attempt to index
 local 'choice' (a nil value)
stack traceback:
        ...are/nvim/lazy/ts-node-action/lua/ts-node-action/init.lua:130: in function 'on_choice'
        ...ar/neovim/HEAD-8c6b0a5/share/nvim/runtime/lua/vim/ui.lua:52: in function 'select'
        ...are/nvim/lazy/ts-node-action/lua/ts-node-action/init.lua:124: in function 'fn'
        ...e/nvim/lazy/ts-node-action/lua/ts-node-action/repeat.lua:11: in function <...e/nvim/lazy/ts-node-action/lua/ts-node-action/repeat.lua:10>
        [C]: in function 'nvim_exec2'
        vim/_editor.lua:420: in function 'cmd'
        ...e/nvim/lazy/ts-node-action/lua/ts-node-action/repeat.lua:23: in function <...e/nvim/lazy/ts-node-action/lua/ts-node-action/repeat.lua:5>
stack traceback:
        [C]: in function 'nvim_exec2'
        vim/_editor.lua:420: in function 'cmd'
        ...e/nvim/lazy/ts-node-action/lua/ts-node-action/repeat.lua:23: in function <...e/nvim/lazy/ts-node-action/lua/ts-node-action/repeat.lua:5>
Press ENTER or type command to continue

Feature request:
So node actions are activated by filetype and not by the embedded code. I have SQL embedded in Julia/R and different languages embedded in Markdown. Would be nice to have the possibility to use node actions there as well.