juxscript 1.0.81 → 1.0.83

package/machinery/compiler.js

@@ -189,6 +189,12 @@ export async function bundleJuxFilesToRouter(projectRoot, distDir, options = {})
     if (isJux) {
       fileToFunction.set(relativePath, cleanFunctionName);
       fileToFunction.set(relativePath.split(path.sep).join('/'), cleanFunctionName);
+
+      // ✅ FIX: Register lookup without extension to match juxconfig routes like "/path/to/view"
+      const noExt = relativePath.replace(/\.jux$/, '');
+      const noExtNormalized = noExt.split(path.sep).join('/');
+      fileToFunction.set(noExt, cleanFunctionName);
+      fileToFunction.set(noExtNormalized, cleanFunctionName);
     }
 
     const routePath = routePrefix + '/' + (parsedPath.dir ? `${parsedPath.dir}/` : '') + parsedPath.name;
@@ -250,12 +256,20 @@ export async function bundleJuxFilesToRouter(projectRoot, distDir, options = {})
 
   if (pages) {
     const resolveAndAddRoute = (urlPath, targetFile) => {
+      // ✅ FIX: Handle ./ and / prefixes in config paths
       const cleanTarget = targetFile.replace(/^(\.\/|\/)/, '');
-      const funcName = fileToFunction.get(cleanTarget);
+      let funcName = fileToFunction.get(cleanTarget);
+
+      // Try appending .jux if not found and no extension provided
+      if (!funcName && !cleanTarget.endsWith('.jux')) {
+        funcName = fileToFunction.get(cleanTarget + '.jux');
+      }
+
       if (funcName) {
+        console.log(`      📍 Custom Route: ${urlPath.padEnd(25)} -> ${funcName} (${targetFile})`);
         routes.unshift({ path: urlPath, functionName: funcName });
       } else {
-        console.warn(`      ⚠️  Route target not found: ${targetFile}`);
+        console.warn(`      ⚠️  Route target not found: ${targetFile} (Cleaned: ${cleanTarget})`);
       }
     };
     Object.entries(pages).forEach(([key, value]) => {
@@ -287,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) {
@@ -350,9 +398,11 @@ function transformJuxToViewFunction(juxContent, functionName, pageName, relative
               importReplacements.set(spec.local.name, `${spec.imported.name}$${namespace}`);
             }
           });
-          // Remove local imports as they will be inlined in main.js
-          nodesToRemove.push({ start: node.start, end: node.end });
         }
+
+        // 🔥 FIX: Always remove ImportDeclaration from the view function contents.
+        // Leftover imports inside a function body cause SyntaxError.
+        nodesToRemove.push({ start: node.start, end: node.end });
       }
       if (node.type === 'ExportNamedDeclaration' || node.type === 'ExportDefaultDeclaration') {
         nodesToRemove.push({ start: node.start, end: node.end });

package/package.json

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