refactor(linter): update tsgolint payload (#13547)

- oxc side of oxc-project/tsgolint#182

Updates the payload that `oxlint` sends to `tsgolint` to match the new format. The new structure is based around sets of rules and configurations first, and then specifying the files those configs act upon. That looks like something like this (output from running `oxlint --type-aware apps/oxlint/fixtures`)

```json
{
  "configs": [
    {
      "file_paths": [
        "/Users/<user>/workspace/oxc/apps/oxlint/fixtures/tsgolint/use-unknown-in-catch-callback-variable/index.ts",
        "/Users/<user>/workspace/oxc/apps/oxlint/fixtures/tsgolint/prefer-reduce-type-parameter/index.ts",
        "/Users/<user>/workspace/oxc/apps/oxlint/fixtures/tsgolint/return-await/index.ts",
        "/Users/<user>/workspace/oxc/apps/oxlint/fixtures/tsgolint/non-tsgolint.ts",
        "..."
        "/Users/<user>/workspace/oxc/apps/oxlint/fixtures/tsgolint/no-unnecessary-template-expression/index.ts",
        "/Users/<user>/workspace/oxc/apps/oxlint/fixtures/tsgolint/no-unsafe-type-assertion/index.ts",
        "/Users/<user>/workspace/oxc/apps/oxlint/fixtures/tsgolint/no-unsafe-member-access/index.ts",
        "/Users/<user>/workspace/oxc/apps/oxlint/fixtures/tsgolint/no-unsafe-call/index.ts",
        "/Users/<user>/workspace/oxc/apps/oxlint/fixtures/tsgolint/no-unnecessary-boolean-literal-compare/index.ts"
      ],
      "rules": [
        { "name": "await-thenable" },
        { "name": "no-floating-promises" },
        { "name": "no-array-delete" },
        { "name": "no-base-to-string" },
        { "name": "no-confusing-void-expression" },
        { "name": "no-duplicate-type-constituents" },
        { "...": "..."},
        { "name": "restrict-plus-operands" },
        { "name": "restrict-template-expressions" },
        { "name": "return-await" },
        { "name": "switch-exhaustiveness-check" },
        { "name": "unbound-method" },
        { "name": "use-unknown-in-catch-callback-variable" }
      ]
    },
    {
      "file_paths": [
        "/Users/<user>/workspace/oxc/apps/oxlint/fixtures/tsgolint/no-floating-promises/src/index.ts",
        "/Users/<user>/workspace/oxc/apps/oxlint/fixtures/tsgolint/no-floating-promises/src/overrides.ts"
      ],
      "rules": [{ "name": "no-floating-promises" }]
    }
  ]
}
```

Author: camchenry

crates/oxc_linter/src/tsgolint.rs

@@ -1,4 +1,5 @@
 use std::{
+    collections::BTreeSet,
     ffi::OsStr,
     io::{ErrorKind, Read, Write},
     path::{Path, PathBuf},
@@ -76,8 +77,7 @@ impl TsGoLintState {
         let mut resolved_configs: FxHashMap<PathBuf, ResolvedLinterState> = FxHashMap::default();
 
         let json_input = self.json_input(paths, &mut resolved_configs);
-
-        if json_input.files.is_empty() {
+        if json_input.configs.is_empty() {
             return Ok(());
         }
 
@@ -469,32 +469,41 @@ impl TsGoLintState {
         &self,
         paths: &[Arc<OsStr>],
         resolved_configs: &mut FxHashMap<PathBuf, ResolvedLinterState>,
-    ) -> TsGoLintInput {
-        TsGoLintInput {
-            files: paths
-                .iter()
-                .filter(|path| SourceType::from_path(Path::new(path)).is_ok())
-                .map(|path| TsGoLintInputFile {
-                    file_path: path.to_string_lossy().to_string(),
-                    rules: {
-                        let path_buf = PathBuf::from(path);
-                        let resolved_config = resolved_configs
-                            .entry(path_buf.clone())
-                            .or_insert_with(|| self.config_store.resolve(&path_buf));
-
-                        // Collect the rules that are enabled for this file
-                        resolved_config
-                            .rules
-                            .iter()
-                            .filter_map(|(rule, status)| {
-                                if status.is_warn_deny() && rule.is_tsgolint_rule() {
-                                    Some(rule.name().to_string())
-                                } else {
-                                    None
-                                }
-                            })
-                            .collect()
-                    },
+    ) -> Payload {
+        let mut config_groups: FxHashMap<BTreeSet<Rule>, Vec<String>> = FxHashMap::default();
+
+        for path in paths {
+            if SourceType::from_path(Path::new(path)).is_ok() {
+                let path_buf = PathBuf::from(path);
+                let file_path = path.to_string_lossy().to_string();
+
+                let resolved_config = resolved_configs
+                    .entry(path_buf.clone())
+                    .or_insert_with(|| self.config_store.resolve(&path_buf));
+
+                let rules: BTreeSet<Rule> = resolved_config
+                    .rules
+                    .iter()
+                    .filter_map(|(rule, status)| {
+                        if status.is_warn_deny() && rule.is_tsgolint_rule() {
+                            Some(Rule { name: rule.name().to_string() })
+                        } else {
+                            None
+                        }
+                    })
+                    .collect();
+
+                config_groups.entry(rules).or_default().push(file_path);
+            }
+        }
+
+        Payload {
+            version: 2,
+            configs: config_groups
+                .into_iter()
+                .map(|(rules, file_paths)| Config {
+                    file_paths,
+                    rules: rules.into_iter().collect(),
                 })
                 .collect(),
         }
@@ -505,26 +514,35 @@ impl TsGoLintState {
 ///
 /// ```json
 /// {
-///   "files": [
+///   "configs": [
 ///     {
-///       "file_path": "/absolute/path/to/file.ts",
-///       "rules": ["rule-1", "another-rule"]
+///       "file_paths": ["/absolute/path/to/file.ts", "/another/file.ts"],
+///       "rules": [
+///         { "name": "rule-1" },
+///         { "name": "another-rule" },
+///       ]
 ///     }
 ///   ]
 /// }
 /// ```
 #[derive(Debug, Clone, Serialize, Deserialize)]
-pub struct TsGoLintInput {
-    pub files: Vec<TsGoLintInputFile>,
+pub struct Payload {
+    pub version: i32,
+    pub configs: Vec<Config>,
 }
 
 #[derive(Debug, Clone, Serialize, Deserialize)]
-pub struct TsGoLintInputFile {
+pub struct Config {
     /// Absolute path to the file to lint
-    pub file_path: String,
+    pub file_paths: Vec<String>,
     /// List of rules to apply to this file
     /// Example: `["no-floating-promises"]`
-    pub rules: Vec<String>,
+    pub rules: Vec<Rule>,
+}
+
+#[derive(Debug, Clone, Serialize, Deserialize, Hash, Eq, PartialEq, PartialOrd, Ord)]
+pub struct Rule {
+    pub name: String,
 }
 
 /// Represents the raw output binary data from `tsgolint`.