@plasmicapp/cli 0.1.300 → 0.1.302

package/dist/actions/sync.d.ts

@@ -13,6 +13,7 @@ export interface SyncArgs extends CommonArgs {
     allFiles?: boolean;
     loaderConfig?: string;
     skipFormatting?: boolean;
+    skipBuffering?: boolean;
 }
 /**
  * Sync will always try to sync down a set of components that are version-consistent among specified projects.

package/dist/index.js

@@ -735439,6 +735439,7 @@ async function fixAllImportStatements(context, baseDir, summary) {
   logger.info("Fixing import statements...");
   const config = context.config;
   const fixImportContext = mkFixImportContext(config);
+  let lastError = void 0;
   for (const project of config.projects) {
     for (const compConfig of project.components) {
       const compSummary = summary == null ? void 0 : summary.get(compConfig.id);
@@ -735447,17 +735448,34 @@ async function fixAllImportStatements(context, baseDir, summary) {
       }
       const fixSkeletonModule = compSummary ? compSummary.skeletonModuleModified : true;
       if (!summary || compSummary) {
-        await fixComponentImportStatements(
-          context,
-          compConfig,
-          fixImportContext,
-          fixSkeletonModule,
-          baseDir
-        );
+        try {
+          await fixComponentImportStatements(
+            context,
+            compConfig,
+            fixImportContext,
+            fixSkeletonModule,
+            baseDir
+          );
+        } catch (err) {
+          logger.error(
+            `Error encountered while fixing imports for ${compConfig.name}: ${err}`
+          );
+          lastError = err;
+        }
       }
     }
   }
-  fixGlobalContextImportStatements(context, fixImportContext, baseDir);
+  try {
+    fixGlobalContextImportStatements(context, fixImportContext, baseDir);
+  } catch (err) {
+    logger.error(
+      `Error encountered while fixing imports for global contexts: ${err}`
+    );
+    lastError = err;
+  }
+  if (lastError) {
+    throw lastError;
+  }
 }
 async function fixComponentImportStatements(context, compConfig, fixImportContext, fixSkeletonModule, baseDir) {
   if (compConfig.type !== "mapped" && isLocalModulePath(compConfig.importSpec.modulePath) && fixSkeletonModule) {
@@ -736636,7 +736654,10 @@ async function startAuth(opts) {
     let prompt;
     const initToken = v4_default();
     const url = `${opts.host}/auth/plasmic-init/${initToken}`;
-    (0, import_open.default)(url);
+    (0, import_open.default)(url).then((subprocess) => {
+      subprocess.on("error", () => {
+      });
+    });
     logger.info(
       `
 If your browser doesn't automatically open, enter the following URL:
@@ -738940,7 +738961,7 @@ async function sync(opts, metadataDefaults) {
   context.api.attachProjectIdsAndTokens(projectIdsAndTokens);
   const externalNpmPackages = /* @__PURE__ */ new Set();
   const externalCssImports = /* @__PURE__ */ new Set();
-  await withBufferedFs(async () => {
+  const doSync = async () => {
     var _a2;
     for (const projectMeta of projectsToSync) {
       await syncProject(
@@ -739009,7 +739030,12 @@ async function sync(opts, metadataDefaults) {
       }
     });
     await updateConfig(context, context.config, baseDir);
-  });
+  };
+  if (opts.skipBuffering) {
+    await doSync();
+  } else {
+    await withBufferedFs(doSync);
+  }
   await checkExternalPkgs(
     context,
     baseDir,
@@ -739775,6 +739801,10 @@ function configureSyncArgs(yags, includeQuietOption = true) {
     type: "boolean",
     describe: "Disables formatting on generated code",
     default: false
+  }).option("skip-buffering", {
+    type: "boolean",
+    describe: "Write files directly to disk, instead of buffering and only writing if sync completes successfully",
+    default: false
   }).option("all-files", {
     type: "boolean",
     describe: "Sync all files, including those that haven't changed since last sync",

package/dist/lib.js

@@ -730779,6 +730779,7 @@ async function fixAllImportStatements(context, baseDir, summary) {
   logger.info("Fixing import statements...");
   const config = context.config;
   const fixImportContext = mkFixImportContext(config);
+  let lastError = void 0;
   for (const project of config.projects) {
     for (const compConfig of project.components) {
       const compSummary = summary == null ? void 0 : summary.get(compConfig.id);
@@ -730787,17 +730788,34 @@ async function fixAllImportStatements(context, baseDir, summary) {
       }
       const fixSkeletonModule = compSummary ? compSummary.skeletonModuleModified : true;
       if (!summary || compSummary) {
-        await fixComponentImportStatements(
-          context,
-          compConfig,
-          fixImportContext,
-          fixSkeletonModule,
-          baseDir
-        );
+        try {
+          await fixComponentImportStatements(
+            context,
+            compConfig,
+            fixImportContext,
+            fixSkeletonModule,
+            baseDir
+          );
+        } catch (err) {
+          logger.error(
+            `Error encountered while fixing imports for ${compConfig.name}: ${err}`
+          );
+          lastError = err;
+        }
       }
     }
   }
-  fixGlobalContextImportStatements(context, fixImportContext, baseDir);
+  try {
+    fixGlobalContextImportStatements(context, fixImportContext, baseDir);
+  } catch (err) {
+    logger.error(
+      `Error encountered while fixing imports for global contexts: ${err}`
+    );
+    lastError = err;
+  }
+  if (lastError) {
+    throw lastError;
+  }
 }
 async function fixComponentImportStatements(context, compConfig, fixImportContext, fixSkeletonModule, baseDir) {
   if (compConfig.type !== "mapped" && isLocalModulePath(compConfig.importSpec.modulePath) && fixSkeletonModule) {
@@ -731976,7 +731994,10 @@ async function startAuth(opts) {
     let prompt;
     const initToken = v4_default();
     const url = `${opts.host}/auth/plasmic-init/${initToken}`;
-    (0, import_open.default)(url);
+    (0, import_open.default)(url).then((subprocess) => {
+      subprocess.on("error", () => {
+      });
+    });
     logger.info(
       `
 If your browser doesn't automatically open, enter the following URL:
@@ -734236,7 +734257,7 @@ async function sync(opts, metadataDefaults) {
   context.api.attachProjectIdsAndTokens(projectIdsAndTokens);
   const externalNpmPackages = /* @__PURE__ */ new Set();
   const externalCssImports = /* @__PURE__ */ new Set();
-  await withBufferedFs(async () => {
+  const doSync = async () => {
     var _a2;
     for (const projectMeta of projectsToSync) {
       await syncProject(
@@ -734305,7 +734326,12 @@ async function sync(opts, metadataDefaults) {
       }
     });
     await updateConfig(context, context.config, baseDir);
-  });
+  };
+  if (opts.skipBuffering) {
+    await doSync();
+  } else {
+    await withBufferedFs(doSync);
+  }
   await checkExternalPkgs(
     context,
     baseDir,

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@plasmicapp/cli",
-  "version": "0.1.300",
+  "version": "0.1.302",
   "description": "plasmic cli for syncing local code with Plasmic designs",
   "engines": {
     "node": ">=12"
@@ -78,5 +78,5 @@
     "wrap-ansi": "^7.0.0",
     "yargs": "^15.4.1"
   },
-  "gitHead": "d77514328c5ed3df3af84ff5c9f35f351fc1c276"
+  "gitHead": "98f55a13c36ec563fe7e8f286167d0c3f241fee8"
 }

package/src/actions/sync.ts

@@ -75,6 +75,7 @@ export interface SyncArgs extends CommonArgs {
   allFiles?: boolean;
   loaderConfig?: string;
   skipFormatting?: boolean;
+  skipBuffering?: boolean;
 }
 
 async function ensureRequiredPackages(
@@ -319,8 +320,7 @@ export async function sync(
   const externalNpmPackages = new Set<string>();
   const externalCssImports = new Set<string>();
 
-  // Perform the actual sync
-  await withBufferedFs(async () => {
+  const doSync = async () => {
     // Sync in sequence (no parallelism)
     // going in reverse to get leaves of the dependency tree first
     for (const projectMeta of projectsToSync) {
@@ -408,7 +408,14 @@ export async function sync(
     });
     // Write the new ComponentConfigs to disk
     await updateConfig(context, context.config, baseDir);
-  });
+  };
+
+  // Perform the actual sync
+  if (opts.skipBuffering) {
+    await doSync();
+  } else {
+    await withBufferedFs(doSync);
+  }
 
   await checkExternalPkgs(
     context,

package/src/index.ts

@@ -366,6 +366,12 @@ function configureSyncArgs(
       describe: "Disables formatting on generated code",
       default: false,
     })
+    .option("skip-buffering", {
+      type: "boolean",
+      describe:
+        "Write files directly to disk, instead of buffering and only writing if sync completes successfully",
+      default: false,
+    })
     .option("all-files", {
       type: "boolean",
       describe:

package/src/utils/auth-utils.ts

@@ -102,7 +102,12 @@ export async function startAuth(opts: Partial<CommonArgs> & { host: string }) {
 
     const initToken = uuidv4();
     const url = `${opts.host}/auth/plasmic-init/${initToken}`;
-    open(url);
+    open(url).then((subprocess) => {
+      subprocess.on("error", () => {
+        // Browser failed to start. This needs to be handled to avoid
+        // crashing the CLI.
+      });
+    });
     logger.info(
       `\nIf your browser doesn't automatically open, enter the following URL:\n${url}\n`
     );

package/src/utils/code-utils.ts

@@ -504,6 +504,7 @@ export async function fixAllImportStatements(
   logger.info("Fixing import statements...");
   const config = context.config;
   const fixImportContext = mkFixImportContext(config);
+  let lastError: any = undefined;
   for (const project of config.projects) {
     for (const compConfig of project.components) {
       const compSummary = summary?.get(compConfig.id);
@@ -514,17 +515,36 @@ export async function fixAllImportStatements(
         ? compSummary.skeletonModuleModified
         : true;
       if (!summary || compSummary) {
-        await fixComponentImportStatements(
-          context,
-          compConfig,
-          fixImportContext,
-          fixSkeletonModule,
-          baseDir
-        );
+        try {
+          await fixComponentImportStatements(
+            context,
+            compConfig,
+            fixImportContext,
+            fixSkeletonModule,
+            baseDir
+          );
+        } catch (err) {
+          logger.error(
+            `Error encountered while fixing imports for ${compConfig.name}: ${err}`
+          );
+          lastError = err;
+        }
       }
     }
   }
-  fixGlobalContextImportStatements(context, fixImportContext, baseDir);
+
+  try {
+    fixGlobalContextImportStatements(context, fixImportContext, baseDir);
+  } catch (err) {
+    logger.error(
+      `Error encountered while fixing imports for global contexts: ${err}`
+    );
+    lastError = err;
+  }
+
+  if (lastError) {
+    throw lastError;
+  }
 }
 
 async function fixComponentImportStatements(