Better code action filtering

[davidosomething]

Is your feature request related to a problem? Please describe.

I might want to filter some code actions out, but not sure what kind it is

Describe the solution you'd like

In the code action menu (showMenuPicker), always show the kind after the name, i.e.

should say Move to file [refactor]
or whatever the kind is

[davidosomething]

Actually I would specifically like to filter out those two show refactor kinds, but NOT ALL refactor kinds (e.g. keep Extract to type alias https://github.com/neoclide/coc-tsserver/blob/938682f1825b1b35de550c72db525eab6d96830e/src/server/features/refactor.ts#L318 )

I think what I really need is a way to filter code action by source (coc-tsserver) and name (Move to file)

Instead of only in getCodeAction

coc.nvim/src/handler/codeActions.ts

Line 57 in 48cad05

public async getCodeActions(doc: Document, range?: Range, only?: CodeActionKind[]): Promise<CodeAction[]> {

maybe have filter fn ? or some config

{
  exclude: [
    { source: 'tsserver', name: 'Move to file' },
    { source: 'tsserver', name: 'Move to a new file' },
  ]
}

[davidosomething]

Related to #5278 (which is for sorting codeactions, not filtering)

[chemzqm]

The coc-tsserver shows unnecessary selection for refactor, I've fixed the issue on coc-tsserver to make it useful

[Lmills71]

Thank you. Sent from Yahoo Mail for iPhone On Saturday, March 22, 2025, 3:02 AM, Qiming zhao ***@***.***> wrote: The coc-tsserver shows unnecessary selection for refactor, I've fixed the issue on coc-tsserver to make it useful — Reply to this email directly, view it on GitHub, or unsubscribe. You are receiving this because you are subscribed to this thread.Message ID: ***@***.***> chemzqm left a comment (neoclide/coc.nvim#5288) The coc-tsserver shows unnecessary selection for refactor, I've fixed the issue on coc-tsserver to make it useful — Reply to this email directly, view it on GitHub, or unsubscribe. You are receiving this because you are subscribed to this thread.Message ID: ***@***.***>

[atitcreate]

If you know the action name(see the response below), you can call it directly, e.g.:

call CocActionAsync('codeAction', 'cursor', "Extract to type alias")

---

Following the guide Debug language server:

1. Add "tsserver.trace.server": "verbose", to coc-settings.json
2. Open the example ts file
3. Open the output channel with :CocCommand workspace.showOutput tsserver
4. Put the cursor on proppertyA
5. :call CocActionAsync('codeAction', 'cursor') and cancel it with Esc.

function Test(parameter: { propertyA: number } & { propertyB: string }) {
  return parameter
}

The output channel will update with something like:

<semantic> Sending request: getApplicableRefactors (14). Response expected: yes. Current queue length: 0
Arguments: {
    "file": "/home/me/src/temp/ts.ts",
    "startLine": 1,
    "startOffset": 26,
    "endLine": 1,
    "endOffset": 26,
    "includeInteractiveActions": true,
    "triggerReason": "invoked"
}

and

Response

 <semantic> Response received: getApplicableRefactors (14). Request took 23 ms. Success: true 
Result: [
    {
        "name": "Convert export",
        "description": "Convert default export to named export",
        "actions": [
            {
                "name": "Convert default export to named export",
                "description": "Convert default export to named export",
                "kind": "refactor.rewrite.export.named",
                "notApplicableReason": "Could not find export statement"
            },
            {
                "name": "Convert named export to default export",
                "description": "Convert named export to default export",
                "kind": "refactor.rewrite.export.default",
                "notApplicableReason": "Could not find export statement"
            }
        ]
    },
    {
        "name": "Convert import",
        "description": "Convert namespace import to named imports",
        "actions": [
            {
                "name": "Convert namespace import to named imports",
                "description": "Convert namespace import to named imports",
                "kind": "refactor.rewrite.import.named",
                "notApplicableReason": "Selection is not an import declaration."
            }
        ]
    },
    {
        "name": "Convert import",
        "description": "Convert named imports to default import",
        "actions": [
            {
                "name": "Convert named imports to default import",
                "description": "Convert named imports to default import",
                "kind": "refactor.rewrite.import.default",
                "notApplicableReason": "Selection is not an import declaration."
            }
        ]
    },
    {
        "name": "Convert import",
        "description": "Convert named imports to namespace import",
        "actions": [
            {
                "name": "Convert named imports to namespace import",
                "description": "Convert named imports to namespace import",
                "kind": "refactor.rewrite.import.namespace",
                "notApplicableReason": "Selection is not an import declaration."
            }
        ]
    },
    {
        "name": "Extract type",
        "description": "Extract type",
        "actions": [
            {
                "name": "Extract to type alias",
                "description": "Extract to type alias",
                "kind": "refactor.extract.type",
                "range": {
                    "start": {
                        "line": 1,
                        "offset": 26
                    },
                    "end": {
                        "line": 1,
                        "offset": 39
                    }
                }
            },
            {
                "name": "Extract to interface",
                "description": "Extract to interface",
                "kind": "refactor.extract.interface",
                "range": {
                    "start": {
                        "line": 1,
                        "offset": 26
                    },
                    "end": {
                        "line": 1,
                        "offset": 39
                    }
                }
            }
        ]
    },
    {
        "name": "Move to file",
        "description": "Move to file",
        "actions": [
            {
                "name": "Move to file",
                "description": "Move to file",
                "kind": "refactor.move.file",
                "range": {
                    "start": {
                        "line": 1,
                        "offset": 1
                    },
                    "end": {
                        "line": 3,
                        "offset": 2
                    }
                }
            }
        ]
    },
    {
        "name": "Move to a new file",
        "description": "Move to a new file",
        "actions": [
            {
                "name": "Move to a new file",
                "description": "Move to a new file",
                "kind": "refactor.move.newFile",
                "range": {
                    "start": {
                        "line": 1,
                        "offset": 1
                    },
                    "end": {
                        "line": 3,
                        "offset": 2
                    }
                }
            }
        ]
    },
    {
        "name": "Add or remove braces in an arrow function",
        "description": "Add or remove braces in an arrow function",
        "actions": [
            {
                "name": "Add braces to arrow function",
                "description": "Add braces to arrow function",
                "kind": "refactor.rewrite.arrow.braces.add",
                "notApplicableReason": "Containing function is not an arrow function"
            },
            {
                "name": "Remove braces from arrow function",
                "description": "Remove braces from arrow function",
                "kind": "refactor.rewrite.arrow.braces.remove",
                "notApplicableReason": "Containing function is not an arrow function"
            }
        ]
    },
    {
        "name": "Convert parameters to destructured object",
        "description": "Convert parameters to destructured object",
        "actions": [
            {
                "name": "Convert parameters to destructured object",
                "description": "Convert parameters to destructured object",
                "kind": "refactor.rewrite.parameters.toDestructured"
            }
        ]
    },
    {
        "name": "Convert to template string",
        "description": "Convert to template string",
        "actions": [
            {
                "name": "Convert to template string",
                "description": "Convert to template string",
                "kind": "refactor.rewrite.string",
                "notApplicableReason": "Can only convert string concatenations and string literals"
            }
        ]
    },
    {
        "name": "Convert to optional chain expression",
        "description": "Convert to optional chain expression",
        "actions": [
            {
                "name": "Convert to optional chain expression",
                "description": "Convert to optional chain expression",
                "kind": "refactor.rewrite.expression.optionalChain",
                "notApplicableReason": "Could not find convertible access expression"
            }
        ]
    },
    {
        "name": "Extract Symbol",
        "description": "Extract function",
        "actions": [
            {
                "name": "Extract Function",
                "description": "Extract function",
                "kind": "refactor.extract.function",
                "notApplicableReason": "Statement or expression expected."
            }
        ]
    },
    {
        "name": "Extract Symbol",
        "description": "Extract constant",
        "actions": [
            {
                "name": "Extract Constant",
                "description": "Extract constant",
                "kind": "refactor.extract.constant",
                "notApplicableReason": "Statement or expression expected."
            }
        ]
    },
    {
        "name": "Generate 'get' and 'set' accessors",
        "description": "Generate 'get' and 'set' accessors",
        "actions": [
            {
                "name": "Generate 'get' and 'set' accessors",
                "description": "Generate 'get' and 'set' accessors",
                "kind": "refactor.rewrite.property.generateAccessors",
                "notApplicableReason": "Could not find property for which to generate accessor"
            }
        ]
    },
    {
        "name": "Infer function return type",
        "description": "Infer function return type",
        "actions": [
            {
                "name": "Infer function return type",
                "description": "Infer function return type",
                "kind": "refactor.rewrite.function.returnType"
            }
        ]
    }
]

Note that in the response array inapplicable actions have the property "notApplicableReason", so if we can get the response, we can easily filter them out, and further filter the remaining actions.

:CocList services shows the id of tsserver is tsserver, but :call CocRequestAsync('tsserver', 'getApplicableRefactors') throws [coc.nvim]: Error on notification "sendRequest": Language server tsserver not found .