juxscript 1.0.82 → 1.0.83

package/machinery/compiler.js

@@ -301,44 +301,78 @@ export async function bundleJuxFilesToRouter(projectRoot, distDir, options = {})
 }
 
 function extractSharedModule(juxContent, moduleName, sourceFilePath) {
-  const exportLines = [];
-  try {
-    const ast = acorn.parse(juxContent, { ecmaVersion: 'latest', sourceType: 'module' });
-    ast.body.forEach(node => {
-      if (node.type === 'ExportNamedDeclaration' && node.declaration) {
-        const exportCode = juxContent.slice(node.start, node.end).replace(/^export\s+/, '');
-        exportLines.push(`// From: ${sourceFilePath}\n${namespaceExportedIdentifiers(exportCode, sourceFilePath)}`);
-      } else if (node.type === 'ExportDefaultDeclaration') {
-        const exportCode = juxContent.slice(node.start, node.end).replace(/^export\s+default\s+/, '');
-        exportLines.push(`// From: ${sourceFilePath}\n${namespaceExportedIdentifiers(exportCode, sourceFilePath)}`);
-      }
-    });
-  } catch (err) {
-    throw new Error(`Invalid JavaScript syntax in ${sourceFilePath}.`);
-  }
-  return exportLines.join('\n\n');
-}
-
-function namespaceExportedIdentifiers(code, sourceFilePath) {
-  // ✅ FIX: Strip .jux AND .js extensions for stable namespacing
+  // 1. Calculate Namespace
+  // Ensure we have a valid JS identifier suffix
   const namespace = sourceFilePath.replace(/\.(jux|js)$/, '').replace(/[\/\\]/g, '$').replace(/[^a-zA-Z0-9$]/g, '_');
+
+  let ast;
   try {
-    const ast = acorn.parse(code, { ecmaVersion: 'latest', sourceType: 'module' });
-    const identifiers = [];
-    ast.body.forEach(node => {
-      if (node.type === 'FunctionDeclaration' && node.id) identifiers.push(node.id);
-      else if (node.type === 'VariableDeclaration') node.declarations.forEach(d => { if (d.id.type === 'Identifier') identifiers.push(d.id); });
-      else if (node.type === 'ClassDeclaration' && node.id) identifiers.push(node.id);
-    });
-    identifiers.sort((a, b) => b.start - a.start);
-    let result = code;
-    identifiers.forEach(id => {
-      result = result.slice(0, id.start) + `${id.name}$${namespace}` + result.slice(id.end);
-    });
-    return result;
+    ast = acorn.parse(juxContent, { ecmaVersion: 'latest', sourceType: 'module' });
   } catch (err) {
-    return code;
+    throw new Error(`Invalid JavaScript syntax in ${sourceFilePath}: ${err.message}`);
   }
+
+  // 2. Identify ALL Top-Level Identifiers (Exported or Local) to rename
+  // We must rename everything to avoid global collision in the flat bundle
+  const identifiersToRename = new Map();
+
+  const addId = (idNode) => {
+    if (idNode && idNode.type === 'Identifier') {
+      identifiersToRename.set(idNode.name, `${idNode.name}$${namespace}`);
+    }
+  };
+
+  ast.body.forEach(node => {
+    if (node.type === 'FunctionDeclaration' || node.type === 'ClassDeclaration') {
+      addId(node.id);
+    } else if (node.type === 'VariableDeclaration') {
+      node.declarations.forEach(d => addId(d.id));
+    } else if ((node.type === 'ExportNamedDeclaration' || node.type === 'ExportDefaultDeclaration') && node.declaration) {
+      const d = node.declaration;
+      if (d.type === 'FunctionDeclaration' || d.type === 'ClassDeclaration') addId(d.id);
+      else if (d.type === 'VariableDeclaration') d.declarations.forEach(v => addId(v.id));
+    }
+  });
+
+  // 3. Reconstruct Code Line-by-Line (Node-by-Node)
+  const outputLines = [`// From: ${sourceFilePath}`];
+
+  // Sort keys by length desc to avoid partial replacement issues
+  const sortedKeys = Array.from(identifiersToRename.keys()).sort((a, b) => b.length - a.length);
+
+  ast.body.forEach(node => {
+    // Skip Imports (handled globally in bundling phase)
+    if (node.type === 'ImportDeclaration') return;
+    // Skip "export { a, b }" (re-exports) as 'a' and 'b' are already declared/renamed locally
+    if (node.type === 'ExportNamedDeclaration' && !node.declaration) return;
+
+    // Extract relevant code slice
+    let code = '';
+    if (node.type === 'ExportNamedDeclaration' || node.type === 'ExportDefaultDeclaration') {
+      if (node.declaration) {
+        code = juxContent.slice(node.declaration.start, node.declaration.end);
+      }
+    } else {
+      code = juxContent.slice(node.start, node.end);
+    }
+
+    if (!code) return;
+
+    // Replace Identifiers in this block
+    // Regex:
+    // (?<!\.) -> Negative lookbehind to avoid property access (obj.foo)
+    // \bkey\b -> Word boundary
+    // (?!\s*:) -> Negative lookahead to avoid object keys ({ foo: 1 })
+    for (const key of sortedKeys) {
+      const namespaced = identifiersToRename.get(key);
+      const regex = new RegExp(`(?<!\\.)\\b${key}\\b(?!\\s*:)`, 'g');
+      code = code.replace(regex, namespaced);
+    }
+
+    outputLines.push(code);
+  });
+
+  return outputLines.join('\n\n');
 }
 
 function transformJuxToViewFunction(juxContent, functionName, pageName, relativePath, sharedModules, bundledPaths) {
@@ -367,9 +401,7 @@ function transformJuxToViewFunction(juxContent, functionName, pageName, relative
         }
 
         // 🔥 FIX: Always remove ImportDeclaration from the view function contents.
-        // External imports are hoisted to 'allImports' and placed at the top of main.js.
-        // Local/Bundled imports are hoisted via sharedModules mechanism with namespacing.
-        // Leaving import statements inside the function body causes SyntaxError.
+        // Leftover imports inside a function body cause SyntaxError.
         nodesToRemove.push({ start: node.start, end: node.end });
       }
       if (node.type === 'ExportNamedDeclaration' || node.type === 'ExportDefaultDeclaration') {

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "juxscript",
-  "version": "1.0.82",
+  "version": "1.0.83",
   "type": "module",
   "description": "A JavaScript UX authorship platform",
   "main": "lib/jux.js",