@molecule/api-code-sandbox-flyio 1.0.3 → 1.0.5

package/dist/provider.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"provider.d.ts","sourceRoot":"","sources":["../src/provider.ts"],"names":[],"mappings":"AAAA;;;;;;;GAOG;AAGH,OAAO,KAAK,EAUV,eAAe,EAEhB,MAAM,4BAA4B,CAAA;AAGnC,OAAO,EAAE,YAAY,EAAgC,MAAM,UAAU,CAAA;AAsBrE,OAAO,EAAmD,KAAK,WAAW,EAAE,MAAM,cAAc,CAAA;AAEhG,OAAO,KAAK,EAEV,WAAW,EAUZ,MAAM,YAAY,CAAA;AA0oDnB;;;;;;;;;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;AAKH,OAAO,KAAK,EAUV,eAAe,EAEhB,MAAM,4BAA4B,CAAA;AAGnC,OAAO,EAAE,YAAY,EAAgC,MAAM,UAAU,CAAA;AAsBrE,OAAO,EAAmD,KAAK,WAAW,EAAE,MAAM,cAAc,CAAA;AAEhG,OAAO,KAAK,EAEV,WAAW,EAUZ,MAAM,YAAY,CAAA;AAwpDnB;;;;;;;;;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

@@ -6,6 +6,7 @@
  *
  * @module
  */
+import { createHash } from 'node:crypto';
 import { getLogger } from '@molecule/api-bond';
 import { t } from '@molecule/api-i18n';
 import { FlyApiClient, FlyApiError, normalizeApiUrl } from './api.js';
@@ -555,16 +556,24 @@ class FlyioSandboxProvider {
         }
     }
     /**
-     * Sanitizes a caller-supplied volume name into a conservative form.
+     * Sanitizes a caller-supplied volume name into a valid Fly volume name.
      *
-     * Fly does NOT document the character rules for volume names, so this reduces
-     * to `[A-Za-z0-9_]`, which is the form every Fly example uses. Callers pass
-     * names like `mol-<uuid>`; the hyphens become underscores.
+     * Fly requires `[a-z0-9_]` (lowercase only) and at most 30 characters — it
+     * rejects anything else with `400 "name only allows lowercase alphanumeric
+     * characters and underscores with at most 30 characters"`. Callers pass names
+     * like `mol-<uuid>` (40 chars, hyphens): lowercase, map every other character
+     * to `_`, and when the result is too long, keep a readable prefix plus a
+     * deterministic hash suffix so the name stays stable (`findVolume` re-derives
+     * the same name on the idempotent path) and unique.
      * @param name - The caller's volume name.
-     * @returns A sanitized Fly volume name.
+     * @returns A sanitized Fly volume name, `[a-z0-9_]` and <= 30 chars.
      */
     volumeName(name) {
-        return name.replace(/[^A-Za-z0-9_]/g, '_');
+        const sanitized = name.toLowerCase().replace(/[^a-z0-9_]/g, '_');
+        if (sanitized.length <= 30)
+            return sanitized;
+        const hash = createHash('sha256').update(name).digest('hex').slice(0, 12);
+        return `${sanitized.slice(0, 17)}_${hash}`;
     }
     /**
      * Idempotently ensures a volume exists in the app and returns its id.
@@ -716,7 +725,14 @@ class FlyioSandboxProvider {
             // read back from the creation response is stale by definition.
             return this.buildSandbox(app, machine.id, 'running');
         }
-        return this.buildSandbox(app, machine.id, mapMachineState(machine.state ?? 'created'));
+        // `POST .../machines` launches the Machine but returns as soon as it is
+        // scheduled — often still `created`/`starting`. The caller's very first
+        // action is an `exec` (to scaffold the workspace), which Fly rejects with
+        // `412 failed_precondition: machine not running` until the Machine is up.
+        // Wait for `started` here so `create` always hands back a live sandbox, as
+        // the JSDoc promises — the template path already does this via restore.
+        await this.waitForStarted(app, machine.id);
+        return this.buildSandbox(app, machine.id, 'running');
     }
     /**
      * Retrieves an existing sandbox by its composite `<app>:<machineId>` id.

package/package.json

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