@jay-framework/compiler-jay-stack 0.16.4 → 0.17.0

package/dist/index.d.ts

@@ -28,7 +28,7 @@ type BuildEnvironment = 'client' | 'server';
  * @param environment - Target environment ('client' or 'server')
  * @returns Transformed code
  */
-declare function transformJayStackBuilder(code: string, filePath: string, environment: BuildEnvironment): {
+declare function transformJayStackBuilder(code: string, filePath: string, environment: BuildEnvironment, stripBuilders?: Set<string>): {
     code: string;
     map?: any;
 };

package/dist/index.js

@@ -25,7 +25,7 @@ function shouldRemoveMethod(methodName, environment) {
     return true;
   return false;
 }
-const { isCallExpression: isCallExpression$1, isPropertyAccessExpression: isPropertyAccessExpression$1, isIdentifier: isIdentifier$2, isStringLiteral } = tsBridge;
+const { isCallExpression: isCallExpression$1, isPropertyAccessExpression: isPropertyAccessExpression$1, isIdentifier: isIdentifier$2, isStringLiteral: isStringLiteral$1 } = tsBridge;
 function findBuilderMethodsToRemove(sourceFile, bindingResolver, environment) {
   const callsToRemove = [];
   const removedVariables = /* @__PURE__ */ new Set();
@@ -54,7 +54,7 @@ function isPartOfJayStackChain(callExpr, bindingResolver) {
       if (isIdentifier$2(current.expression)) {
         const variable = bindingResolver.explain(current.expression);
         const flattened = flattenVariable(variable);
-        if (flattened.path.length === 1 && JAY_BUILDER_FUNCTIONS.has(flattened.path[0]) && isImportModuleVariableRoot(flattened.root) && isStringLiteral(flattened.root.module) && flattened.root.module.text === "@jay-framework/fullstack-component")
+        if (flattened.path.length === 1 && JAY_BUILDER_FUNCTIONS.has(flattened.path[0]) && isImportModuleVariableRoot(flattened.root) && isStringLiteral$1(flattened.root.module) && flattened.root.module.text === "@jay-framework/fullstack-component")
           return true;
       }
       if (isPropertyAccessExpression$1(current.expression)) {
@@ -185,11 +185,14 @@ const {
   isPropertyAccessExpression,
   isImportDeclaration,
   isNamedImports,
-  isIdentifier
+  isIdentifier,
+  isStringLiteral
 } = tsBridge;
-function transformJayStackBuilder(code, filePath, environment) {
+function transformJayStackBuilder(code, filePath, environment, stripBuilders) {
   const sourceFile = createSourceFile(filePath, code, ScriptTarget.Latest, true);
-  const transformers = [mkTransformer(mkJayStackCodeSplitTransformer, { environment })];
+  const transformers = [
+    mkTransformer(mkJayStackCodeSplitTransformer, { environment, stripBuilders })
+  ];
   const printer = createPrinter();
   const result = tsBridge.transform(sourceFile, transformers);
   const transformedFile = result.transformed[0];
@@ -206,10 +209,25 @@ function mkJayStackCodeSplitTransformer({
   factory,
   sourceFile,
   context,
-  environment
+  environment,
+  stripBuilders
 }) {
   const bindingResolver = new SourceFileBindingResolver(sourceFile);
-  const { callsToRemove } = findBuilderMethodsToRemove(sourceFile, bindingResolver, environment);
+  let workingSourceFile = sourceFile;
+  if (stripBuilders && stripBuilders.size > 0) {
+    const filtered = sourceFile.statements.filter((statement) => {
+      const rootName = findChainRootBuilderName(statement, bindingResolver);
+      return !rootName || !stripBuilders.has(rootName);
+    });
+    if (filtered.length !== sourceFile.statements.length) {
+      workingSourceFile = factory.updateSourceFile(sourceFile, filtered);
+    }
+  }
+  const { callsToRemove } = findBuilderMethodsToRemove(
+    workingSourceFile,
+    bindingResolver,
+    environment
+  );
   const transformVisitor = (node) => {
     if (isCallExpression(node) && isPropertyAccessExpression(node.expression)) {
       const variable = bindingResolver.explain(node.expression);
@@ -222,7 +240,7 @@ function mkJayStackCodeSplitTransformer({
     return visitEachChild(node, transformVisitor, context);
   };
   let transformedSourceFile = visitEachChild(
-    sourceFile,
+    workingSourceFile,
     transformVisitor,
     context
   );
@@ -262,6 +280,36 @@ function filterImportDeclaration(statement, unusedImports, factory) {
     statement.assertClause
   );
 }
+function findChainRootBuilderName(statement, bindingResolver) {
+  let expr;
+  if (tsBridge.isVariableStatement(statement)) {
+    for (const decl of statement.declarationList.declarations) {
+      if (decl.initializer && isCallExpression(decl.initializer)) {
+        expr = decl.initializer;
+      }
+    }
+  }
+  if (tsBridge.isExpressionStatement(statement) && isCallExpression(statement.expression)) {
+    expr = statement.expression;
+  }
+  if (!expr)
+    return void 0;
+  let current = expr;
+  while (true) {
+    if (isCallExpression(current) && isPropertyAccessExpression(current.expression)) {
+      current = current.expression.expression;
+    } else if (isCallExpression(current) && isIdentifier(current.expression)) {
+      const variable = bindingResolver.explain(current.expression);
+      const flattened = flattenVariable(variable);
+      if (flattened.path.length === 1 && isImportModuleVariableRoot(flattened.root) && isStringLiteral(flattened.root.module) && flattened.root.module.text === "@jay-framework/fullstack-component") {
+        return flattened.path[0];
+      }
+      return void 0;
+    } else {
+      return void 0;
+    }
+  }
+}
 const actionMetadataCache = /* @__PURE__ */ new Map();
 function clearActionMetadataCache() {
   actionMetadataCache.clear();
@@ -765,6 +813,9 @@ function jayStackCompiler(options = {}) {
         if (!id.endsWith(".ts") && !id.includes(".ts?")) {
           return null;
         }
+        if (id.includes(".jay-html")) {
+          return null;
+        }
         const hasComponent = code.includes("makeJayStackComponent");
         const hasInit = code.includes("makeJayInit");
         if (!hasComponent && !hasInit) {
@@ -874,6 +925,23 @@ function jayStackCompiler(options = {}) {
           }
           const result = lines.join("\n");
           return result;
+        },
+        // Strip inline makeJayAction/makeJayQuery/makeJayStream statements
+        // from component files in client builds. These are server-only
+        // constructs that shouldn't appear in client bundles.
+        transform(code, id, options2) {
+          if (options2?.ssr || isSSRBuild)
+            return null;
+          if (!id.endsWith(".ts") && !id.includes(".ts?"))
+            return null;
+          if (id.includes(".jay-html"))
+            return null;
+          if (isActionImport(id))
+            return null;
+          const ACTION_BUILDERS = ["makeJayAction", "makeJayQuery", "makeJayStream"];
+          if (!ACTION_BUILDERS.some((b) => code.includes(b)))
+            return null;
+          return stripInlineActionBuilders(code, ACTION_BUILDERS);
         }
       };
     })(),
@@ -882,6 +950,9 @@ function jayStackCompiler(options = {}) {
   );
   return plugins;
 }
+function stripInlineActionBuilders(code, builderNames) {
+  return transformJayStackBuilder(code, "action-strip.ts", "client", new Set(builderNames));
+}
 export {
   clearActionMetadataCache,
   createDefaultPluginDetector,

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@jay-framework/compiler-jay-stack",
-  "version": "0.16.4",
+  "version": "0.17.0",
   "type": "module",
   "license": "Apache-2.0",
   "main": "dist/index.js",
@@ -27,16 +27,16 @@
     "test:watch": "vitest"
   },
   "dependencies": {
-    "@jay-framework/compiler": "^0.16.4",
-    "@jay-framework/compiler-shared": "^0.16.4",
-    "@jay-framework/logger": "^0.16.4",
-    "@jay-framework/typescript-bridge": "^0.16.4",
-    "@jay-framework/vite-plugin": "^0.16.4",
+    "@jay-framework/compiler": "^0.17.0",
+    "@jay-framework/compiler-shared": "^0.17.0",
+    "@jay-framework/logger": "^0.17.0",
+    "@jay-framework/typescript-bridge": "^0.17.0",
+    "@jay-framework/vite-plugin": "^0.17.0",
     "typescript": "^5.3.3",
     "vite": "^5.0.11"
   },
   "devDependencies": {
-    "@jay-framework/dev-environment": "^0.16.4",
+    "@jay-framework/dev-environment": "^0.17.0",
     "rimraf": "^5.0.5",
     "tsup": "^8.0.1",
     "vitest": "^1.2.1"