@embeddables/cli 0.7.10 → 0.7.13

package/dist/cli.js

@@ -40,10 +40,10 @@ async function requireLogin(commandName) {
     stdout.print(pc.gray('Run "embeddables login" first.'));
     await exit(1);
 }
-/** Set Sentry context from config and auth for the current command. */
-async function setSentryContextForCommand() {
+/** Set Sentry context from config and auth for the current command. Config is set immediately; user is set in background so we don't block on auth. */
+function setSentryContextForCommand() {
     setSentryContext(getSentryContextFromProjectConfig());
-    await setSentryUserFromAuth();
+    void setSentryUserFromAuth();
 }
 const program = new Command();
 program
@@ -57,7 +57,7 @@ program
     .option('-y, --yes', 'Skip prompts and use defaults')
     .action(async (opts) => {
     await requireLogin('init');
-    await setSentryContextForCommand();
+    setSentryContextForCommand();
     await runInit({ projectId: opts.projectId, yes: opts.yes });
 });
 program
@@ -69,7 +69,7 @@ program
     .option('--pageKeyFrom <mode>', 'filename|export', 'filename')
     .action(async (opts) => {
     await requireLogin('build');
-    await setSentryContextForCommand();
+    setSentryContextForCommand();
     await runBuild(opts);
 });
 program
@@ -85,7 +85,7 @@ program
     .option('--pageKeyFrom <mode>', 'filename|export', 'filename')
     .action(async (opts) => {
     await requireLogin('dev');
-    await setSentryContextForCommand();
+    setSentryContextForCommand();
     // --local flag overrides --engine to use local engine
     if (opts.local) {
         opts.engine = 'http://localhost:8787';
@@ -115,7 +115,7 @@ program
     .option('-p, --preserve', 'Preserve component order in config for forward compile')
     .action(async (opts) => {
     await requireLogin('pull');
-    await setSentryContextForCommand();
+    setSentryContextForCommand();
     await runPull({
         ...opts,
         version: opts.version,
@@ -131,7 +131,7 @@ program
     .option('--from-version <number>', 'Base version number (auto-detected from local files if not provided)')
     .action(async (opts) => {
     await requireLogin('save');
-    await setSentryContextForCommand();
+    setSentryContextForCommand();
     await runSave({
         id: opts.id,
         label: opts.label,
@@ -152,7 +152,7 @@ program
     .option('-i, --id <id>', 'Embeddable ID (will prompt if not provided)')
     .action(async (opts) => {
     await requireLogin('branch');
-    await setSentryContextForCommand();
+    setSentryContextForCommand();
     await runBranch(opts);
 });
 const experiments = program.command('experiments').description('Manage embeddable experiments');
@@ -164,7 +164,7 @@ experiments
     .option('--experiment-key <key>', 'Experiment key (required if --experiment-id is set)')
     .action(async (opts) => {
     await requireLogin('experiments connect');
-    await setSentryContextForCommand();
+    setSentryContextForCommand();
     await runExperimentsConnect({
         id: opts.id,
         experimentId: opts.experimentId,

package/dist/compiler/index.js

@@ -1018,16 +1018,14 @@ async function loadComputedFields(embeddableId) {
  */
 async function loadDataOutputs(embeddableId, config = null) {
     const actionsDir = path.join('embeddables', embeddableId, 'actions');
-    if (!fs.existsSync(actionsDir)) {
-        return [];
-    }
+    const hasActionsDir = fs.existsSync(actionsDir);
     // Find all JS files in actions directory
-    const jsFiles = await fg('**/*.js', {
-        cwd: actionsDir,
-        dot: false,
-    });
-    if (jsFiles.length === 0) {
-        return [];
+    let jsFiles = [];
+    if (hasActionsDir) {
+        jsFiles = await fg('**/*.js', {
+            cwd: actionsDir,
+            dot: false,
+        });
     }
     // Use provided config or read it if not provided
     if (!config) {
@@ -1042,6 +1040,10 @@ async function loadDataOutputs(embeddableId, config = null) {
             }
         }
     }
+    const hasConfigDataOutputs = config && config.dataOutputs && Array.isArray(config.dataOutputs) && config.dataOutputs.length > 0;
+    if (jsFiles.length === 0 && !hasConfigDataOutputs) {
+        return [];
+    }
     // Build a map of actions by various identifiers for quick lookup
     const dataOutputsMap = new Map();
     for (const jsFile of jsFiles) {

package/dist/compiler/reverse.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"reverse.d.ts","sourceRoot":"","sources":["../../src/compiler/reverse.ts"],"names":[],"mappings":"AAQA,OAAO,KAAK,EAAE,QAAQ,EAAiB,MAAM,YAAY,CAAA;AAqlBzD,wBAAsB,cAAc,CAClC,UAAU,EAAE;IACV,EAAE,CAAC,EAAE,MAAM,CAAA;IACX,KAAK,EAAE,QAAQ,EAAE,CAAA;IACjB,MAAM,CAAC,EAAE,MAAM,CAAC,MAAM,EAAE,GAAG,CAAC,CAAA;IAC5B,cAAc,CAAC,EAAE,GAAG,EAAE,CAAA;IACtB,WAAW,CAAC,EAAE,GAAG,EAAE,CAAA;IACnB,UAAU,CAAC,EAAE,GAAG,EAAE,CAAA;IAClB,CAAC,GAAG,EAAE,MAAM,GAAG,GAAG,CAAA;CACnB,EACD,YAAY,EAAE,MAAM,EACpB,IAAI,CAAC,EAAE;IACL,GAAG,CAAC,EAAE,OAAO,CAAA;IACb,QAAQ,CAAC,EAAE,OAAO,CAAA;IAClB,YAAY,CAAC,EAAE;QAAE,OAAO,CAAC,EAAE,MAAM,CAAC;QAAC,QAAQ,CAAC,EAAE,MAAM,CAAC;QAAC,UAAU,CAAC,EAAE,MAAM,CAAA;KAAE,CAAA;CAC5E,iBAoFF;AAgiDD;;GAEG;AACH,wBAAgB,gBAAgB,CAAC,GAAG,EAAE,MAAM,GAAG,MAAM,CAKpD"}
+{"version":3,"file":"reverse.d.ts","sourceRoot":"","sources":["../../src/compiler/reverse.ts"],"names":[],"mappings":"AAQA,OAAO,KAAK,EAAE,QAAQ,EAAiB,MAAM,YAAY,CAAA;AAqlBzD,wBAAsB,cAAc,CAClC,UAAU,EAAE;IACV,EAAE,CAAC,EAAE,MAAM,CAAA;IACX,KAAK,EAAE,QAAQ,EAAE,CAAA;IACjB,MAAM,CAAC,EAAE,MAAM,CAAC,MAAM,EAAE,GAAG,CAAC,CAAA;IAC5B,cAAc,CAAC,EAAE,GAAG,EAAE,CAAA;IACtB,WAAW,CAAC,EAAE,GAAG,EAAE,CAAA;IACnB,UAAU,CAAC,EAAE,GAAG,EAAE,CAAA;IAClB,CAAC,GAAG,EAAE,MAAM,GAAG,GAAG,CAAA;CACnB,EACD,YAAY,EAAE,MAAM,EACpB,IAAI,CAAC,EAAE;IACL,GAAG,CAAC,EAAE,OAAO,CAAA;IACb,QAAQ,CAAC,EAAE,OAAO,CAAA;IAClB,YAAY,CAAC,EAAE;QAAE,OAAO,CAAC,EAAE,MAAM,CAAC;QAAC,QAAQ,CAAC,EAAE,MAAM,CAAC;QAAC,UAAU,CAAC,EAAE,MAAM,CAAA;KAAE,CAAA;CAC5E,iBAoFF;AAggDD;;GAEG;AACH,wBAAgB,gBAAgB,CAAC,GAAG,EAAE,MAAM,GAAG,MAAM,CAKpD"}

package/dist/compiler/reverse.js

@@ -1093,52 +1093,27 @@ function camelCase(str) {
         .join('')
         .replace(/[^a-zA-Z0-9]/g, '');
 }
-function escapeString(str) {
-    // Escape backslashes first, then quotes, then all newline variants
-    return str
-        .replace(/\\/g, '\\\\')
-        .replace(/"/g, '\\"')
-        .replace(/\r\n/g, '\\n') // Windows line endings
-        .replace(/\r/g, '\\n') // Old Mac line endings
-        .replace(/\n/g, '\\n'); // Unix line endings
-}
 function escapeStringForJSX(str) {
-    // Check if string contains double quotes - if so, use template literals
     const hasDoubleQuotes = str.includes('"');
-    if (hasDoubleQuotes) {
-        // Use template literals (backticks) - escape backticks, backslashes, and ${
+    const hasNewlines = /[\r\n]/.test(str);
+    // Use a template literal expression {`...`} when JS escape processing is needed:
+    // - String contains double quotes (can't use them as JSX attribute delimiter)
+    // - String contains actual newline characters (JSX attribute strings can't represent these)
+    if (hasDoubleQuotes || hasNewlines) {
         let escaped = str
-            .replace(/\\/g, '\\\\') // Escape backslashes first
+            .replace(/\\/g, '\\\\') // Escape backslashes (template literals process escape sequences)
             .replace(/`/g, '\\`') // Escape backticks
             .replace(/\$\{/g, '\\${'); // Escape template literal expressions
-        // Escape newlines
         escaped = escaped
             .replace(/\r\n/g, '\\n') // Windows line endings
             .replace(/\r/g, '\\n') // Old Mac line endings
             .replace(/\n/g, '\\n'); // Unix line endings
         return `{\`${escaped}\`}`;
     }
-    // No double quotes - use regular quote selection
-    const doubleQuoteCount = (str.match(/"/g) || []).length;
-    const singleQuoteCount = (str.match(/'/g) || []).length;
-    // Escape backslashes first
-    let escaped = str.replace(/\\/g, '\\\\');
-    // Escape newlines
-    escaped = escaped
-        .replace(/\r\n/g, '\\n') // Windows line endings
-        .replace(/\r/g, '\\n') // Old Mac line endings
-        .replace(/\n/g, '\\n'); // Unix line endings
-    // Use single quotes if there are more double quotes, otherwise use double quotes
-    if (doubleQuoteCount > singleQuoteCount) {
-        // Use single quotes and escape single quotes
-        escaped = escaped.replace(/'/g, "\\'");
-        return `'${escaped}'`;
-    }
-    else {
-        // Use double quotes and escape double quotes
-        escaped = escaped.replace(/"/g, '\\"');
-        return `"${escaped}"`;
-    }
+    // JSX attribute strings treat all characters literally (no escape processing).
+    // Backslashes, \n, etc. are NOT interpreted — they are stored as-is.
+    // Since there are no double quotes in the string, use double quotes as delimiter.
+    return `"${str}"`;
 }
 /**
  * Converts HTML string to JSX children code.
@@ -1244,31 +1219,25 @@ function htmlToJSXChildren(html, indent) {
                 // So if the value contains double quotes, use single quotes for the outer delimiter
                 const hasDoubleQuotes = attrValue.includes('"');
                 const hasSingleQuotes = attrValue.includes("'");
-                if (hasDoubleQuotes && !hasSingleQuotes) {
-                    // Use single quotes - only need to escape backslashes and newlines
-                    const escapedValue = attrValue
-                        .replace(/\\/g, '\\\\')
-                        .replace(/\n/g, '\\n')
-                        .replace(/\r/g, '\\r');
-                    attrs.push(`${jsxAttrName}='${escapedValue}'`);
-                }
-                else if (hasDoubleQuotes && hasSingleQuotes) {
-                    // Value has both quote types - use JSX expression with template literal
+                const hasNewlines = /[\r\n]/.test(attrValue);
+                if (hasDoubleQuotes && hasSingleQuotes || hasNewlines) {
+                    // Use JSX expression with template literal (JS escape processing applies)
                     const escapedValue = attrValue
                         .replace(/\\/g, '\\\\')
                         .replace(/`/g, '\\`')
-                        .replace(/\$\{/g, '\\${') // Escape template literal expressions
-                        .replace(/\n/g, '\\n')
-                        .replace(/\r/g, '\\r');
+                        .replace(/\$\{/g, '\\${')
+                        .replace(/\r\n/g, '\\n')
+                        .replace(/\r/g, '\\n')
+                        .replace(/\n/g, '\\n');
                     attrs.push(`${jsxAttrName}={\`${escapedValue}\`}`);
                 }
+                else if (hasDoubleQuotes) {
+                    // JSX attribute strings are literal — no backslash escaping needed
+                    attrs.push(`${jsxAttrName}='${attrValue}'`);
+                }
                 else {
-                    // No double quotes - use double quotes as before
-                    const escapedValue = attrValue
-                        .replace(/\\/g, '\\\\')
-                        .replace(/\n/g, '\\n')
-                        .replace(/\r/g, '\\r');
-                    attrs.push(`${jsxAttrName}="${escapedValue}"`);
+                    // JSX attribute strings are literal — no backslash escaping needed
+                    attrs.push(`${jsxAttrName}="${attrValue}"`);
                 }
             }
             pos += fullMatch.length;

package/dist/prompts/embeddables.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"embeddables.d.ts","sourceRoot":"","sources":["../../src/prompts/embeddables.ts"],"names":[],"mappings":"AAOA,MAAM,WAAW,cAAc;IAC7B,EAAE,EAAE,MAAM,CAAA;IACV,KAAK,EAAE,MAAM,GAAG,IAAI,CAAA;CACrB;AAED,MAAM,WAAW,kBAAkB;IACjC,KAAK,EAAE,MAAM,GAAG,IAAI,CAAA;IACpB,QAAQ,EAAE,OAAO,CAAA;IACjB,UAAU,EAAE,MAAM,GAAG,IAAI,CAAA;CAC1B;AAED,MAAM,WAAW,eAAe;IAC9B,EAAE,EAAE,MAAM,CAAA;IACV,KAAK,EAAE,MAAM,GAAG,IAAI,CAAA;CACrB;AAED,MAAM,WAAW,0BAA0B;IACzC,oCAAoC;IACpC,OAAO,CAAC,EAAE,MAAM,CAAA;IAChB,kFAAkF;IAClF,eAAe,CAAC,EAAE,OAAO,CAAA;CAC1B;AAED;;GAEG;AACH,wBAAsB,uBAAuB,CAAC,SAAS,EAAE,MAAM,GAAG,OAAO,CAAC,cAAc,EAAE,CAAC,CA0B1F;AAED;;;GAGG;AACH,wBAAsB,uBAAuB,CAC3C,YAAY,EAAE,MAAM,GACnB,OAAO,CAAC,kBAAkB,GAAG,IAAI,CAAC,CA2BpC;AAED;;;GAGG;AACH,wBAAsB,mBAAmB,CACvC,SAAS,EAAE,MAAM,EACjB,OAAO,GAAE,0BAA+B,GACvC,OAAO,CAAC,MAAM,GAAG,IAAI,CAAC,CA4ExB;AAED;;GAEG;AACH,wBAAsB,wBAAwB,IAAI,OAAO,CAAC,eAAe,EAAE,CAAC,CAqC3E;AAED;;;GAGG;AACH,wBAAsB,wBAAwB,CAC5C,OAAO,GAAE,0BAA+B,GACvC,OAAO,CAAC,MAAM,GAAG,IAAI,CAAC,CAuCxB"}
+{"version":3,"file":"embeddables.d.ts","sourceRoot":"","sources":["../../src/prompts/embeddables.ts"],"names":[],"mappings":"AAOA,MAAM,WAAW,cAAc;IAC7B,EAAE,EAAE,MAAM,CAAA;IACV,KAAK,EAAE,MAAM,GAAG,IAAI,CAAA;CACrB;AAED,MAAM,WAAW,kBAAkB;IACjC,KAAK,EAAE,MAAM,GAAG,IAAI,CAAA;IACpB,QAAQ,EAAE,OAAO,CAAA;IACjB,UAAU,EAAE,MAAM,GAAG,IAAI,CAAA;CAC1B;AAED,MAAM,WAAW,eAAe;IAC9B,EAAE,EAAE,MAAM,CAAA;IACV,KAAK,EAAE,MAAM,GAAG,IAAI,CAAA;CACrB;AAED,MAAM,WAAW,0BAA0B;IACzC,oCAAoC;IACpC,OAAO,CAAC,EAAE,MAAM,CAAA;IAChB,kFAAkF;IAClF,eAAe,CAAC,EAAE,OAAO,CAAA;CAC1B;AAED;;GAEG;AACH,wBAAsB,uBAAuB,CAAC,SAAS,EAAE,MAAM,GAAG,OAAO,CAAC,cAAc,EAAE,CAAC,CA2B1F;AAED;;;GAGG;AACH,wBAAsB,uBAAuB,CAC3C,YAAY,EAAE,MAAM,GACnB,OAAO,CAAC,kBAAkB,GAAG,IAAI,CAAC,CA2BpC;AAED;;;GAGG;AACH,wBAAsB,mBAAmB,CACvC,SAAS,EAAE,MAAM,EACjB,OAAO,GAAE,0BAA+B,GACvC,OAAO,CAAC,MAAM,GAAG,IAAI,CAAC,CA4ExB;AAED;;GAEG;AACH,wBAAsB,wBAAwB,IAAI,OAAO,CAAC,eAAe,EAAE,CAAC,CAqC3E;AAED;;;GAGG;AACH,wBAAsB,wBAAwB,CAC5C,OAAO,GAAE,0BAA+B,GACvC,OAAO,CAAC,MAAM,GAAG,IAAI,CAAC,CAuCxB"}

package/dist/prompts/embeddables.js

@@ -17,6 +17,7 @@ export async function fetchProjectEmbeddables(projectId) {
             .from('flows')
             .select('id, title')
             .eq('project_id', projectId)
+            .not('archived', 'is', 'true')
             .order('title', { ascending: true });
         if (error) {
             stdout.warn(pc.yellow(`Could not fetch embeddables: ${error.message}`));

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@embeddables/cli",
-  "version": "0.7.10",
+  "version": "0.7.13",
   "type": "module",
   "bin": {
     "embeddables": "./bin/embeddables.mjs"