expo-desktop 0.1.23 → 0.1.24

package/build/common/apply-windows-cpp-app-template.js

@@ -224,9 +224,6 @@ async function renameCppAppPathsAsync(windowsRoot, filesafeName) {
         });
         const chosen = candidates[0];
         const newRelWin = cppAppRelativePathTransform(chosen.rel, filesafeName);
-        if (newRelWin === chosen.rel) {
-            throw new Error(`Windows cpp-app rename: expected path containing MyApp to change: "${chosen.rel}"`);
-        }
         const newAbs = path.join(windowsRoot, newRelWin);
         if (newAbs === chosen.abs) {
             continue;

package/build/common/child-process.js

@@ -39,7 +39,7 @@ function runPromisifiedSpawn({ command, args, options, logLine, debugLogDir, })
         stdio: stdioEffective,
         env: envWithForcedColorIfPiped({ ...options, stdio: stdioEffective }),
     };
-    const cp = spawn(command, args, spawnOptions);
+    const cp = spawn(`${command} ${args.join(" ")}`, spawnOptions);
     /** Interleaved stdout/stderr lines in arrival order (tagged for readability). */
     const lineBuffer = [];
     const pushLine = (stream, line) => {

package/build/common/shescape.js

@@ -0,0 +1,34 @@
+import { Shescape } from "shescape";
+const defaultShell = Symbol("Default shell");
+const shescapes = {};
+export function getShescape(shell) {
+    const resolvedShell = shell ?? defaultShell;
+    if (shescapes[resolvedShell]) {
+        return shescapes[resolvedShell];
+    }
+    const shescapeOptions = {};
+    if (typeof resolvedShell === "string") {
+        shescapeOptions.shell = resolvedShell;
+    }
+    let shescape;
+    try {
+        shescape = new Shescape(shescapeOptions);
+    }
+    catch (cause) {
+        if (!(cause instanceof Error) || cause.message !== "Shescape does not support the shell sh") {
+            throw new Error("Unable to spawn child process due to error being thrown when constructing Shescape instance", { cause });
+        }
+        // Can't escape for the meta-shell `/bin/sh`. Let's try falling back to a
+        // typical Unix shell and hoping for the best.
+        // https://github.com/ericcornelissen/shescape/issues/2009
+        try {
+            shescapeOptions.shell = process.platform === "darwin" ? "zsh" : "bash";
+            shescape = new Shescape(shescapeOptions);
+        }
+        catch (cause) {
+            throw new Error("Unable to spawn child process due to error being thrown when constructing fallback Shescape instance", { cause });
+        }
+    }
+    shescapes[resolvedShell] = shescape;
+    return shescape;
+}

package/build/common/template.js

@@ -14,6 +14,7 @@ import { pathToFileURL } from "node:url";
 import { applyWindowsCppAppTemplateAsync } from "./apply-windows-cpp-app-template.js";
 import { promisifiedSpawnTask } from "./child-process.js";
 import { getTemplateFilesToRenameAsync, renameTemplateAppNameAsync, } from "./rename-template-app-name.js";
+import { getShescape } from "./shescape.js";
 export async function applySelectedTemplatesAsync({ projectRoot, selection, enabledPlatforms, name, respectTemplateConfig, }) {
     const descriptors = getOrderedTemplateDescriptors(selection, enabledPlatforms);
     if (!descriptors.length) {
@@ -112,6 +113,8 @@ function parseTemplateSource(template) {
     return { type: "npm", spec: template };
 }
 async function prepareTemplateSourceAsync(taskTitle, source) {
+    // We make sure there are no spaces in the path so that we don't need to
+    // quote/escape the shell command.
     const tempRoot = await fs.mkdtemp(path.join(os.tmpdir(), "expo-desktop-template-"));
     const archivePath = path.join(tempRoot, "template.tgz");
     switch (source.type) {
@@ -119,6 +122,8 @@ async function prepareTemplateSourceAsync(taskTitle, source) {
             await fs.copyFile(source.path, archivePath);
             break;
         case "github": {
+            // Don't think it's possible to have spaces in the owner/repo/ref, so no
+            // percent-encoding or quoting needed.
             const tarballUrl = `https://codeload.github.com/${source.owner}/${source.repo}/tar.gz/${source.ref}`;
             const response = await fetch(tarballUrl);
             if (!response.ok || !response.body) {
@@ -129,11 +134,12 @@ async function prepareTemplateSourceAsync(taskTitle, source) {
             break;
         }
         case "npm": {
+            const shescape = getShescape();
             await tasks([
                 promisifiedSpawnTask({
                     title: `npm pack (${source.spec})`,
                     command: "npm",
-                    args: ["pack", source.spec, "--silent"],
+                    args: ["pack", shescape.quote(source.spec), "--silent"],
                     options: { cwd: tempRoot },
                 }),
             ]);

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "expo-desktop",
-  "version": "0.1.23",
+  "version": "0.1.24",
   "description": "Best-effort desktop support for Expo",
   "keywords": [
     "android",
@@ -42,6 +42,7 @@
     "glob": "^10.5.0",
     "kleur": "^4.1.5",
     "mustache": "^4.2.0",
+    "shescape": "^2.1.12",
     "toml": "^4.1.1"
   },
   "devDependencies": {