@walkeros/cli 1.0.1 → 1.0.2-next-1769158278557

package/CHANGELOG.md

@@ -1,5 +1,98 @@
 # @walkeros/cli
 
+## 1.0.2-next-1769158278557
+
+### Patch Changes
+
+- 2709933: Add `$code:` prefix support for inline JavaScript in flow.json
+
+  Values prefixed with `$code:` are output as raw JavaScript instead of quoted
+  strings in the bundled output. This enables features like `fn:` callbacks and
+  `condition:` predicates directly in JSON configuration files.
+
+  Example:
+
+  ```json
+  { "fn": "$code:(value) => value.toUpperCase()" }
+  ```
+
+  Outputs:
+
+  ```javascript
+  {
+    fn: (value) => value.toUpperCase();
+  }
+  ```
+
+- 04469bb: Auto-detect default export for sources and destinations
+
+  Sources and destinations now automatically use their package's default export,
+  eliminating the need to specify `imports` for the main function.
+
+  Before (verbose):
+
+  ```json
+  "@walkeros/web-source-browser": { "imports": ["sourceBrowser"] }
+  ```
+
+  After (simpler):
+
+  ```json
+  "@walkeros/web-source-browser": {}
+  ```
+
+  The `imports` field is now only needed for utility functions. Explicit `code`
+  still works for packages without default exports.
+
+- 544a79e: Implicit collector: auto-add @walkeros/collector when
+  sources/destinations exist
+
+  The CLI now automatically adds `@walkeros/collector` and imports `startFlow`
+  when your flow has sources or destinations. No need to declare the collector
+  package.
+
+  Before (verbose):
+
+  ```json
+  "packages": {
+    "@walkeros/collector": { "imports": ["startFlow"] },
+    "@walkeros/web-source-browser": {},
+    "@walkeros/destination-demo": {}
+  }
+  ```
+
+  After (simpler):
+
+  ```json
+  "packages": {
+    "@walkeros/web-source-browser": {},
+    "@walkeros/destination-demo": {}
+  }
+  ```
+
+  You only need to specify `@walkeros/collector` when you want to pin a specific
+  version or use a local path for development.
+
+- 4da2ef3: Fix CLI commands hanging after completion
+
+  Commands (`bundle`, `simulate`, `push`) would hang indefinitely after
+  completing successfully due to open handles keeping the Node.js event loop
+  alive.
+
+  Root cause: esbuild worker threads and pacote HTTP keep-alive connections were
+  not being cleaned up.
+
+  Fixes:
+  - Add `esbuild.stop()` after builds to terminate worker threads
+  - Add explicit `process.exit(0)` on successful completion for all CLI commands
+
+- 2f82a2e: Fix simulate command JSON output to use consistent `result` property
+  instead of `elbResult`
+- Updated dependencies [b65b773]
+- Updated dependencies [20eca6e]
+  - @walkeros/core@1.0.1-next-1769158278557
+  - @walkeros/server-core@1.0.1-next-1769158278557
+
 ## 1.0.1
 
 ### Patch Changes

package/dist/examples/flow-complete.json

@@ -4,8 +4,8 @@
   "variables": {
     "currency": "EUR",
     "flowVersion": "1.0.0",
-    "ga4MeasurementId": "${GA4_MEASUREMENT_ID:G-DEMO123456}",
-    "apiUrl": "${API_URL:http://localhost:8080/collect}"
+    "ga4MeasurementId": "$env.GA4_MEASUREMENT_ID:G-DEMO123456",
+    "apiUrl": "$env.API_URL:http://localhost:8080/collect"
   },
   "definitions": {
     "ga4ItemsLoop": {
@@ -212,7 +212,7 @@
           "config": {
             "consent": { "marketing": true },
             "settings": {
-              "ga4": { "measurementId": "$variables.ga4MeasurementId" }
+              "ga4": { "measurementId": "$var.ga4MeasurementId" }
             },
             "mapping": {
               "page": {
@@ -233,7 +233,7 @@
                     "map": {
                       "currency": {
                         "key": "data.currency",
-                        "value": "$variables.currency"
+                        "value": "$var.currency"
                       },
                       "value": "data.price",
                       "items": {
@@ -258,7 +258,7 @@
                     "map": {
                       "currency": {
                         "key": "data.currency",
-                        "value": "$variables.currency"
+                        "value": "$var.currency"
                       },
                       "value": "data.price",
                       "items": {
@@ -286,11 +286,11 @@
                       "value": "data.total",
                       "currency": {
                         "key": "data.currency",
-                        "value": "$variables.currency"
+                        "value": "$var.currency"
                       },
                       "tax": "data.tax",
                       "shipping": "data.shipping",
-                      "items": { "$ref": "#/definitions/ga4ItemsLoop" }
+                      "items": "$def.ga4ItemsLoop"
                     }
                   }
                 }
@@ -307,7 +307,7 @@
           "package": "@walkeros/web-destination-api",
           "config": {
             "settings": {
-              "url": "$variables.apiUrl",
+              "url": "$var.apiUrl",
               "batch": 5
             },
             "mapping": {
@@ -322,7 +322,7 @@
                         "key": "user.email",
                         "consent": { "marketing": true }
                       },
-                      "items": { "$ref": "#/definitions/ga4ItemsLoop" }
+                      "items": "$def.ga4ItemsLoop"
                     }
                   }
                 }
@@ -347,7 +347,7 @@
         },
         "globals": {
           "environment": "demo",
-          "version": "$variables.flowVersion"
+          "version": "$var.flowVersion"
         },
         "custom": {
           "campaign": "flow-demo"
@@ -357,8 +357,8 @@
     "server": {
       "server": {},
       "variables": {
-        "metaPixelId": "${META_PIXEL_ID:123456789012345}",
-        "metaAccessToken": "${META_ACCESS_TOKEN:demo_token}"
+        "metaPixelId": "$env.META_PIXEL_ID:123456789012345",
+        "metaAccessToken": "$env.META_ACCESS_TOKEN:demo_token"
       },
       "packages": {
         "@walkeros/collector": {},
@@ -444,8 +444,8 @@
           "before": "fingerprint",
           "config": {
             "settings": {
-              "pixelId": "$variables.metaPixelId",
-              "accessToken": "$variables.metaAccessToken",
+              "pixelId": "$var.metaPixelId",
+              "accessToken": "$var.metaAccessToken",
               "user_data": {
                 "external_id": {
                   "set": ["user.device", "user.session"]
@@ -481,7 +481,7 @@
                       "value": "data.price",
                       "currency": {
                         "key": "data.currency",
-                        "value": "$variables.currency"
+                        "value": "$var.currency"
                       },
                       "content_type": { "value": "product" },
                       "content_ids": {
@@ -497,7 +497,7 @@
                       "value": "data.price",
                       "currency": {
                         "key": "data.currency",
-                        "value": "$variables.currency"
+                        "value": "$var.currency"
                       },
                       "content_type": { "value": "product" },
                       "contents": {
@@ -523,10 +523,10 @@
                       "value": "data.total",
                       "currency": {
                         "key": "data.currency",
-                        "value": "$variables.currency"
+                        "value": "$var.currency"
                       },
                       "order_id": "data.id",
-                      "contents": { "$ref": "#/definitions/metaContentsLoop" }
+                      "contents": "$def.metaContentsLoop"
                     }
                   }
                 }
@@ -562,7 +562,7 @@
         },
         "globals": {
           "environment": "demo",
-          "version": "$variables.flowVersion",
+          "version": "$var.flowVersion",
           "platform": "server"
         }
       }

package/dist/index.js

@@ -870,6 +870,14 @@ async function bundleCore(flowConfig, buildOptions, logger2, showStats = false)
   }
   try {
     await fs7.ensureDir(TEMP_DIR);
+    const hasSourcesOrDests = Object.keys(
+      flowConfig.sources || {}
+    ).length > 0 || Object.keys(
+      flowConfig.destinations || {}
+    ).length > 0;
+    if (hasSourcesOrDests && !buildOptions.packages["@walkeros/collector"]) {
+      buildOptions.packages["@walkeros/collector"] = {};
+    }
     logger2.debug("Downloading packages");
     const packagesArray = Object.entries(buildOptions.packages).map(
       ([name, packageConfig]) => ({
@@ -935,6 +943,8 @@ async function bundleCore(flowConfig, buildOptions, logger2, showStats = false)
         buildError,
         buildOptions.code || ""
       );
+    } finally {
+      await esbuild.stop();
     }
     const outputStats = await fs7.stat(outputPath);
     const sizeKB = (outputStats.size / 1024).toFixed(1);
@@ -1127,30 +1137,31 @@ function generateImportStatements(packages, destinationPackages, sourcePackages,
   for (const [packageName, packageConfig] of Object.entries(packages)) {
     const isUsedByDestOrSource = usedPackages.has(packageName);
     const hasExplicitCode = explicitCodeImports.has(packageName);
+    const namedImportsToGenerate = [];
+    if (isUsedByDestOrSource && !hasExplicitCode) {
+      const varName = packageNameToVariable(packageName);
+      importStatements.push(`import ${varName} from '${packageName}';`);
+    }
+    if (hasExplicitCode) {
+      const codes = Array.from(explicitCodeImports.get(packageName));
+      namedImportsToGenerate.push(...codes);
+    }
     if (packageConfig.imports && packageConfig.imports.length > 0) {
       const uniqueImports = [...new Set(packageConfig.imports)];
-      const defaultImports = [];
-      const namedImports = [];
       for (const imp of uniqueImports) {
         if (imp.startsWith("default as ")) {
-          defaultImports.push(imp.replace("default as ", ""));
+          if (!isUsedByDestOrSource || hasExplicitCode) {
+            const defaultImportName = imp.replace("default as ", "");
+            importStatements.push(
+              `import ${defaultImportName} from '${packageName}';`
+            );
+          }
         } else {
-          namedImports.push(imp);
-        }
-      }
-      if (defaultImports.length > 0) {
-        for (const defaultImport of defaultImports) {
-          importStatements.push(
-            `import ${defaultImport} from '${packageName}';`
-          );
+          if (!namedImportsToGenerate.includes(imp)) {
+            namedImportsToGenerate.push(imp);
+          }
         }
       }
-      if (namedImports.length > 0) {
-        const importList = namedImports.join(", ");
-        importStatements.push(
-          `import { ${importList} } from '${packageName}';`
-        );
-      }
       const examplesImport = uniqueImports.find(
         (imp) => imp.includes("examples as ")
       );
@@ -1166,14 +1177,13 @@ function generateImportStatements(packages, destinationPackages, sourcePackages,
           );
         }
       }
-    } else if (hasExplicitCode) {
-      const codes = Array.from(explicitCodeImports.get(packageName));
-      importStatements.push(
-        `import { ${codes.join(", ")} } from '${packageName}';`
-      );
-    } else if (isUsedByDestOrSource) {
-      const varName = packageNameToVariable(packageName);
-      importStatements.push(`import ${varName} from '${packageName}';`);
+    }
+    if (packageName === "@walkeros/collector" && !namedImportsToGenerate.includes("startFlow")) {
+      namedImportsToGenerate.push("startFlow");
+    }
+    if (namedImportsToGenerate.length > 0) {
+      const importList = namedImportsToGenerate.join(", ");
+      importStatements.push(`import { ${importList} } from '${packageName}';`);
     }
   }
   return { importStatements, examplesMappings };
@@ -1269,7 +1279,37 @@ ${destinationsEntries.join(",\n")}
 }`;
 }
 function processConfigValue(value) {
-  return JSON.stringify(value, null, 2);
+  return serializeWithCode(value, 0);
+}
+function serializeWithCode(value, indent) {
+  const spaces = "  ".repeat(indent);
+  const nextSpaces = "  ".repeat(indent + 1);
+  if (typeof value === "string") {
+    if (value.startsWith("$code:")) {
+      return value.slice(6);
+    }
+    return JSON.stringify(value);
+  }
+  if (Array.isArray(value)) {
+    if (value.length === 0) return "[]";
+    const items = value.map(
+      (v) => nextSpaces + serializeWithCode(v, indent + 1)
+    );
+    return `[
+${items.join(",\n")}
+${spaces}]`;
+  }
+  if (value !== null && typeof value === "object") {
+    const entries = Object.entries(value);
+    if (entries.length === 0) return "{}";
+    const props = entries.map(
+      ([k, v]) => `${nextSpaces}${JSON.stringify(k)}: ${serializeWithCode(v, indent + 1)}`
+    );
+    return `{
+${props.join(",\n")}
+${spaces}}`;
+  }
+  return JSON.stringify(value);
 }
 function generatePlatformWrapper(configObject, userCode, buildOptions) {
   if (buildOptions.platform === "browser") {
@@ -1434,6 +1474,7 @@ Build Summary: ${successCount}/${results.length} succeeded`
         throw new Error(`${failureCount} flow(s) failed to build`);
       }
     }
+    process.exit(0);
   } catch (error) {
     const duration = timer.getElapsed() / 1e3;
     const errorMessage = getErrorMessage(error);
@@ -2004,17 +2045,15 @@ async function simulateCommand(options) {
       ...result,
       duration: (Date.now() - startTime) / 1e3
     };
+    const output = formatSimulationResult(resultWithDuration, {
+      json: options.json
+    });
     if (options.json) {
-      logger2.json(resultWithDuration);
+      console.log(output);
     } else {
-      const output = formatSimulationResult(resultWithDuration, {
-        json: false
-      });
       logger2.log(output);
     }
-    if (!result.success) {
-      process.exit(1);
-    }
+    process.exit(result.success ? 0 : 1);
   } catch (error) {
     const errorMessage = getErrorMessage(error);
     if (options.json) {
@@ -2132,6 +2171,7 @@ async function pushCommand(options) {
         process.exit(1);
       }
     }
+    process.exit(0);
   } catch (error) {
     const duration = Date.now() - startTime;
     const errorMessage = getErrorMessage(error);