refactor(napi/oxlint): re-order code (#12571)

Pure refactor - does not change any code, just moves it around. Reverse the order of Rust code relating to `loadPlugin` and `lintFile`, for consistency with JS-side code, which has `loadPlugin` first.

Author: overlookmotel

napi/oxlint2/src/lib.rs

@@ -22,38 +22,52 @@ mod generated {
 use generated::raw_transfer_constants::{BLOCK_ALIGN, BUFFER_SIZE};
 
 #[napi]
-pub type JsLintFileCb = ThreadsafeFunction<
+pub type JsLoadPluginCb = ThreadsafeFunction<
     // Arguments
-    FnArgs<(
-        String,             // Absolute path of file to lint
-        u32,                // Buffer ID
-        Option<Uint8Array>, // Buffer (optional)
-        Vec<u32>,           // Array of rule IDs
-    )>,
+    String, // Absolute path of plugin file
     // Return value
-    String, // `Vec<LintFileResult>`, serialized to JSON
+    Promise<String>, // `PluginLoadResult`, serialized to JSON
     // Arguments (repeated)
-    FnArgs<(String, u32, Option<Uint8Array>, Vec<u32>)>,
+    String,
     // Error status
     Status,
     // CalleeHandled
     false,
 >;
 
 #[napi]
-pub type JsLoadPluginCb = ThreadsafeFunction<
+pub type JsLintFileCb = ThreadsafeFunction<
     // Arguments
-    String, // Absolute path of plugin file
+    FnArgs<(
+        String,             // Absolute path of file to lint
+        u32,                // Buffer ID
+        Option<Uint8Array>, // Buffer (optional)
+        Vec<u32>,           // Array of rule IDs
+    )>,
     // Return value
-    Promise<String>, // `PluginLoadResult`, serialized to JSON
+    String, // `Vec<LintFileResult>`, serialized to JSON
     // Arguments (repeated)
-    String,
+    FnArgs<(String, u32, Option<Uint8Array>, Vec<u32>)>,
     // Error status
     Status,
     // CalleeHandled
     false,
 >;
 
+fn wrap_load_plugin(cb: JsLoadPluginCb) -> ExternalLinterLoadPluginCb {
+    let cb = Arc::new(cb);
+    Arc::new(move |plugin_name| {
+        Box::pin({
+            let cb = Arc::clone(&cb);
+            async move {
+                let result = cb.call_async(plugin_name).await?.into_future().await?;
+                let plugin_load_result: PluginLoadResult = serde_json::from_str(&result)?;
+                Ok(plugin_load_result)
+            }
+        })
+    })
+}
+
 fn wrap_lint_file(cb: JsLintFileCb) -> ExternalLinterLintFileCb {
     let cb = Arc::new(cb);
     Arc::new(move |file_path: String, rule_ids: Vec<u32>, allocator: &Allocator| {
@@ -155,20 +169,6 @@ fn wrap_lint_file(cb: JsLintFileCb) -> ExternalLinterLintFileCb {
     })
 }
 
-fn wrap_load_plugin(cb: JsLoadPluginCb) -> ExternalLinterLoadPluginCb {
-    let cb = Arc::new(cb);
-    Arc::new(move |plugin_name| {
-        Box::pin({
-            let cb = Arc::clone(&cb);
-            async move {
-                let result = cb.call_async(plugin_name).await?.into_future().await?;
-                let plugin_load_result: PluginLoadResult = serde_json::from_str(&result)?;
-                Ok(plugin_load_result)
-            }
-        })
-    })
-}
-
 #[expect(clippy::allow_attributes)]
 #[allow(clippy::trailing_empty_array, clippy::unused_async)] // https://github.com/napi-rs/napi-rs/issues/2758
 #[napi]