@molecule/api-code-sandbox-flyio 1.3.2 → 1.3.3

package/dist/provider.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"provider.d.ts","sourceRoot":"","sources":["../src/provider.ts"],"names":[],"mappings":"AAAA;;;;;;;GAOG;AAMH,OAAO,KAAK,EAYV,eAAe,EAGhB,MAAM,4BAA4B,CAAA;AAGnC,OAAO,EAAE,YAAY,EAAgC,MAAM,UAAU,CAAA;AAwBrE,OAAO,EAAmD,KAAK,WAAW,EAAE,MAAM,cAAc,CAAA;AAEhG,OAAO,KAAK,EAIV,WAAW,EAUZ,MAAM,YAAY,CAAA;AAkBnB;;;;;;;;;;;;;;;;;;;;;;;;;;GA0BG;AACH,wBAAgB,2BAA2B,CAAC,KAAK,EAAE,OAAO,EAAE,YAAY,CAAC,EAAE,MAAM,GAAG,IAAI,GAAG,OAAO,CAIjG;AAED;;;;;;;;;;;;;;GAcG;AACH,wBAAgB,kBAAkB,CAAC,KAAK,EAAE,MAAM,GAAG,IAAI,GAAG,SAAS,GAAG,OAAO,CAE5E;AAkyFD;;;;;;;;;GASG;AACH,wBAAgB,cAAc,CAC5B,MAAM,CAAC,EAAE,WAAW,EACpB,MAAM,CAAC,EAAE,YAAY,EACrB,KAAK,CAAC,EAAE,WAAW,GAAG,IAAI,GACzB,eAAe,CAEjB"}
+{"version":3,"file":"provider.d.ts","sourceRoot":"","sources":["../src/provider.ts"],"names":[],"mappings":"AAAA;;;;;;;GAOG;AAOH,OAAO,KAAK,EAYV,eAAe,EAGhB,MAAM,4BAA4B,CAAA;AAGnC,OAAO,EAAE,YAAY,EAAgC,MAAM,UAAU,CAAA;AAwBrE,OAAO,EAAmD,KAAK,WAAW,EAAE,MAAM,cAAc,CAAA;AAEhG,OAAO,KAAK,EAIV,WAAW,EAUZ,MAAM,YAAY,CAAA;AAkBnB;;;;;;;;;;;;;;;;;;;;;;;;;;GA0BG;AACH,wBAAgB,2BAA2B,CAAC,KAAK,EAAE,OAAO,EAAE,YAAY,CAAC,EAAE,MAAM,GAAG,IAAI,GAAG,OAAO,CAIjG;AAED;;;;;;;;;;;;;;GAcG;AACH,wBAAgB,kBAAkB,CAAC,KAAK,EAAE,MAAM,GAAG,IAAI,GAAG,SAAS,GAAG,OAAO,CAE5E;AA8yFD;;;;;;;;;GASG;AACH,wBAAgB,cAAc,CAC5B,MAAM,CAAC,EAAE,WAAW,EACpB,MAAM,CAAC,EAAE,YAAY,EACrB,KAAK,CAAC,EAAE,WAAW,GAAG,IAAI,GACzB,eAAe,CAEjB"}

package/dist/provider.js

@@ -7,7 +7,8 @@
  * @module
  */
 import { createHash } from 'node:crypto';
-import { gzipSync } from 'node:zlib';
+import { promisify } from 'node:util';
+import { gzip } from 'node:zlib';
 import { getLogger } from '@molecule/api-bond';
 import { t } from '@molecule/api-i18n';
 import { FlyApiClient, FlyApiError, normalizeApiUrl } from './api.js';
@@ -272,6 +273,13 @@ const IMPORT_WRITE_CHUNK_BASE64 = 12_000;
  * the single download happen back-to-back.
  */
 const IMPORT_PRESIGN_EXPIRY_SECONDS = 3600;
+/**
+ * zlib's gzip on the libuv thread pool. {@link Sandbox.importFiles} compresses
+ * a whole deploy archive (hundreds of MB), and the synchronous form ran on the
+ * caller's event loop for its full duration — seconds in which a server process
+ * hosting this provider answered no request at all.
+ */
+const gzipAsync = promisify(gzip);
 /** Extra seconds allowed for the probe exec beyond its connection budget. */
 const PROBE_EXEC_HEADROOM_SECONDS = 10;
 /**
@@ -2336,7 +2344,8 @@ class FlyioSandboxProvider {
              *
              * The Fly Machines exec endpoint is a single request/response with no
              * stdin, so the archive cannot be piped to `tar` directly. It is buffered,
-             * gzipped (fewer bytes = fewer exec round trips over the base64 channel),
+             * gzipped on the thread pool, never on the caller's event loop (fewer
+             * bytes = fewer exec round trips over the base64 channel),
              * written to a temp file in `WRITE_CHUNK_BASE64`-sized pieces the same way
              * `writeFile` does, then extracted.
              *
@@ -2351,7 +2360,10 @@ class FlyioSandboxProvider {
                 const parts = [];
                 for await (const chunk of archive)
                     parts.push(Buffer.from(chunk));
-                const gz = gzipSync(Buffer.concat(parts));
+                // Off the event loop: the archive is a whole deploy artifact, and
+                // compressing it synchronously stalled every other request this process
+                // was serving for as long as the compression took.
+                const gz = await gzipAsync(Buffer.concat(parts));
                 const quotedPath = shellQuote(path);
                 const failIfError = (result, step) => {
                     if (result.exitCode !== 0) {

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@molecule/api-code-sandbox-flyio",
-  "version": "1.3.2",
+  "version": "1.3.3",
   "description": "Fly.io Machines (Firecracker microVM) code sandbox provider",
   "type": "module",
   "main": "dist/index.js",