fix(sea): make @napi-rs/cli optional + prettier-format loader test

Two CI failures, both unrelated to driver behavior:

1. unit-test/lint/e2e intermittently failed at `npm ci` with ECONNRESET
   fetching @napi-rs/cli@2.18.4 from the internal npm proxy (cold-cache
   matrix jobs; the warm-cache node-14 job kept passing). @napi-rs/cli is
   only used by `build:native`, which CI never runs. Move it from
   devDependencies to optionalDependencies — matching the existing lz4
   pattern — so a proxy hiccup is a non-fatal skip instead of failing the
   whole install. `build:native`'s `npx --no-install` still resolves it on
   dev machines where the optional install succeeds.

2. prettier flagged loader.test.ts: the two-arg `new SeaNativeLoader(cb, fn)`
   calls needed multi-line formatting. Ran prettier --write.

Co-authored-by: Isaac
Signed-off-by: Madhavendra Rathore <madhavendra.rathore@databricks.com>

Author: msrathore-db

package-lock.json

@@ -49,7 +49,6 @@
   ],
   "license": "Apache-2.0",
   "devDependencies": {
-    "@napi-rs/cli": "2.18.4",
     "@types/chai": "^4.3.14",
     "@types/http-proxy": "^1.17.14",
     "@types/lz4": "^0.6.4",
@@ -92,6 +91,7 @@
     "winston": "^3.8.2"
   },
   "optionalDependencies": {
-    "lz4": "^0.6.5"
+    "lz4": "^0.6.5",
+    "@napi-rs/cli": "2.18.4"
   }
 }

package.json

@@ -126,16 +126,22 @@ describe('SeaNativeLoader', () => {
   describe('Node-version gate', () => {
     it('fails closed on a Node version below the floor', () => {
       let loadCalled = false;
-      const loader = new SeaNativeLoader(() => {
-        loadCalled = true;
-        return stubBinding();
-      }, () => 16);
+      const loader = new SeaNativeLoader(
+        () => {
+          loadCalled = true;
+          return stubBinding();
+        },
+        () => 16,
+      );
       expect(() => loader.get()).to.throw(/requires Node >=18/);
       expect(loadCalled, 'load() must not be attempted on an unsupported Node').to.equal(false);
     });
 
     it('fails closed when the Node version is unparseable (NaN)', () => {
-      const loader = new SeaNativeLoader(() => stubBinding(), () => NaN);
+      const loader = new SeaNativeLoader(
+        () => stubBinding(),
+        () => NaN,
+      );
       expect(() => loader.get()).to.throw(/requires Node >=18/);
     });
   });