[LSP] `source.fixAll` code action should be `source.fixAll.pyrefly`

[rchl]

Describe the Bug

Pyrefly advertizes support for source.fixAll code action in the initialize response:

{
  "capabilities": {
    "callHierarchyProvider": true,
    "codeActionProvider": {
      "codeActionKinds": [
        "quickfix",
        "refactor.extract",
        "refactor.rewrite",
        "refactor.delete",
        "refactor.move",
        "refactor.inline",
        "source.fixAll"
      ]
    },
  },
  // ...
}

This should actually be source.fixAll.pyrefly which then would allow the editor to only target purefly-specific fixes with:

    "editor.codeActionsOnSave": {
        "source.fixAll.pyrefly": true
    },

Otherwise it's only possible to use:

    "editor.codeActionsOnSave": {
        "source.fixAll": true
    },

which then can trigger fixes from other servers that might be running on the file.

Sandbox Link

No response

(Only applicable for extension issues) IDE Information

No response

[NathanTempest]

Thanks for reporting @rchl

You're correct, this should be source.fixAll.pyrefly to follow the LSP convention used by ESLint, Ruff, Stylelint, etc. This allows users to selectively enable/disable Pyrefly's fix-all independently when multiple language servers are active. Without this, when using the automatic code actions, there is a high chance of conflicts.

PR #3039 from @Arths17 is linked to fix this

cc @kinto0 for review.