1tube 0.1.25 → 0.1.26

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "1tube",
-  "version": "0.1.25",
+  "version": "0.1.26",
   "private": false,
   "description": "Self-hosted Supabase Edge Functions gateway for Deno, plus an ESLint plugin for consumers",
   "license": "MIT",

package/src/backends/deno/source-rewriter.ts

@@ -439,7 +439,24 @@ export function createRewriteCache(opts: RewriteCacheOptions): RewriteCache {
       }
       // external / unknown: leave specifier alone.
       if (replacement !== null) {
-        replacements.push({ start: r.start, end: r.end, with: replacement });
+        // For static imports (`import x from "spec"`) the lexer
+        // reports [s, e) over just the specifier text (no quotes),
+        // so writing the bare URL is correct. For dynamic imports
+        // (`await import("spec")`) the slice covers the entire
+        // argument expression including the quotes, so we must
+        // re-quote the replacement or we'd produce
+        // `import(file:///...)` — a parse error. Same goes for
+        // template literals; we normalise to a plain string since
+        // the URL never contains characters that would need
+        // escaping in a JSON string. Detect by sniffing the first
+        // char of the slice.
+        const first = source.charAt(r.start);
+        const isQuoted = first === '"' || first === "'" || first === "`";
+        replacements.push({
+          start: r.start,
+          end: r.end,
+          with: isQuoted ? JSON.stringify(replacement) : replacement,
+        });
       }
     }
 

package/src/server.ts

@@ -907,6 +907,13 @@ if (opts.backend === "workerd") {
   let started = false;
   const padTo = (n: number, width: number) =>
     String(n).padStart(width, " ");
+  // Workers spawn in parallel, so `p.index` (assigned at spawn-start
+  // time) doesn't match the order in which finish lines are printed
+  // — the result reads as a jumbled `[8/53] ... [4/53] ... [7/53]`.
+  // Track a separate "print order" counter so the output reads
+  // monotonically. We still get the correct total + per-fn name +
+  // duration; the only thing that changes is the column we display.
+  let printOrder = 0;
   const { loaded, errors } = await denoWorkerHost.start({
     onSpawnStart: (p) => {
       if (!started) {
@@ -919,8 +926,9 @@ if (opts.backend === "workerd") {
       const w = String(p.total).length;
       const status = p.ok ? "\x1b[32m✓\x1b[0m" : "\x1b[31m✗\x1b[0m";
       const tail = p.ok ? "" : ` — ${p.error ?? "error"}`;
+      const idx = ++printOrder;
       progress.onFinish(
-        `[1tube] [${padTo(p.index, w)}/${p.total}] ${status} ${p.name} ` +
+        `[1tube] [${padTo(idx, w)}/${p.total}] ${status} ${p.name} ` +
           `\x1b[2m(${p.durationMs.toFixed(0)}ms)\x1b[0m${tail}`,
         p.name,
       );