@embeddable.com/sdk-core 4.0.0-next.1 → 4.0.0

package/src/dev.ts

@@ -53,6 +53,10 @@ let browserWindow: ChildProcess | null = null;
 
 let previewWorkspace: string;
 
+// Build coordination to prevent duplicate plugin builds
+let pluginBuildInProgress = false;
+let pendingPluginBuilds: (() => Promise<void>)[] = [];
+
 const SERVER_PORT = 8926;
 const BUILD_DEV_DIR = ".embeddable-dev-build";
 // NOTE: for backward compatibility, keep the file name as global.css
@@ -62,6 +66,52 @@ export const buildWebComponent = async (config: any) => {
   await generate(config, "sdk-react");
 };
 
+const executePluginBuilds = async (
+  config: ResolvedEmbeddableConfig,
+  watchers: Array<RollupWatcher | FSWatcher>,
+) => {
+  if (pluginBuildInProgress) {
+    // If a plugin build is already in progress, queue this one
+    return new Promise<void>((resolve) => {
+      pendingPluginBuilds.push(async () => {
+        await doPluginBuilds(config, watchers);
+        resolve();
+      });
+    });
+  } else {
+    // Start the plugin build immediately
+    await doPluginBuilds(config, watchers);
+  }
+};
+
+const doPluginBuilds = async (
+  config: ResolvedEmbeddableConfig,
+  watchers: Array<RollupWatcher | FSWatcher>,
+) => {
+  pluginBuildInProgress = true;
+
+  try {
+    for (const getPlugin of config.plugins) {
+      const plugin = getPlugin();
+
+      await plugin.validate(config);
+      const watcher = await plugin.build(config);
+      await configureWatcher(watcher as RollupWatcher, config);
+      watchers.push(watcher as RollupWatcher);
+    }
+  } finally {
+    pluginBuildInProgress = false;
+
+    // Process any pending builds
+    if (pendingPluginBuilds.length > 0) {
+      const nextBuild = pendingPluginBuilds.shift();
+      if (nextBuild) {
+        await nextBuild();
+      }
+    }
+  }
+};
+
 const addToGitingore = async () => {
   try {
     const gitignorePath = path.resolve(process.cwd(), ".gitignore");
@@ -215,17 +265,8 @@ export default async () => {
       await sendBuildChanges(config);
 
       if (config.pushComponents) {
-        for (const getPlugin of config.plugins) {
-          const plugin = getPlugin();
-
-          breadcrumbs.push("validate plugin");
-          await plugin.validate(config);
-          breadcrumbs.push("build plugin");
-          const watcher = await plugin.build(config);
-          breadcrumbs.push("configure watcher");
-          await configureWatcher(watcher as RollupWatcher, config);
-          watchers.push(watcher as RollupWatcher);
-        }
+        breadcrumbs.push("build plugins with coordination");
+        await executePluginBuilds(config, watchers);
 
         const customCanvasCssWatch = globalCustomCanvasWatcher(config);
         watchers.push(customCanvasCssWatch);
@@ -440,13 +481,21 @@ export const sendBuildChanges = async (ctx: ResolvedEmbeddableConfig) => {
     filesList = [...filesList, ...cubeAndSecurityContextFileList];
   }
 
-  const token = await getToken();
-  await archive({
-    ctx,
-    filesList,
-    isDev: true,
-  });
-  await sendBuild(ctx, { workspaceId: previewWorkspace, token });
+    try {
+    const token = await getToken();
+    await archive({
+      ctx,
+      filesList,
+      isDev: true,
+    });
+    await sendBuild(ctx, { workspaceId: previewWorkspace, token });
+  } catch (e: any) {
+    sending.fail(
+      `Data models and/or security context synchronization failed with error: ${e.response?.data?.errorMessage ?? e.message ?? 'Unknown error'}`
+    );
+    return sendMessage("dataModelsAndOrSecurityContextUpdateError");
+  }
+  
   sending.succeed(`Data models and/or security context synchronized`);
   sendMessage("dataModelsAndOrSecurityContextUpdateSuccess");
 };