@stacksjs/stx 0.2.162 → 0.2.163

package/dist/variable-extractor.d.ts

@@ -1,3 +1,25 @@
+/**
+ * The runtime bindings the engine hands a `<script server>` block as function
+ * parameters, in the order the parameters are declared.
+ *
+ * This drives the `new Function` parameter list rather than sitting beside it,
+ * because the two must not drift: the value list below it is positional, so a
+ * name added in one place and not the other silently rebinds every argument
+ * after it.
+ *
+ * It is also the set a `from 'stx'` import must not redeclare. A server script
+ * writing the documented `import { defineProps } from 'stx'` was rewritten to
+ * `const { defineProps } = await import('stx')`, and that `const` shadows the
+ * parameter of the same name. The package's own `defineProps` reads
+ * `globalThis.__STX_CURRENT_PROPS__`, which nothing on the server path ever
+ * sets — so props came back `{}` and every component rendered empty.
+ *
+ * It went unnoticed because `stx` does not resolve inside this repo (the
+ * published package is `@stacksjs/stx`; the root `stx` is private), so the
+ * generated `await import('stx')` rejected and the shadowing binding was never
+ * created. The failure needed a real install to appear.
+ */
+export declare const STX_ENGINE_BINDING_NAMES: readonly ['module', 'exports', 'require', 'props', '$props', 'defineProps', 'withDefaults', 'defineClientPayload', 'state', 'derived', 'effect', 'batch', 'onMount', 'onDestroy', 'definePageMeta', 'useRoute', 'useRouter', 'useHead', 'useSeoMeta', 'ref', 'reactive', 'computed', 'watch', 'onMounted', 'onUnmounted', 'nextTick', 'defineEmits', 'defineExpose', 'defineSlots', 'provide', 'inject', 'useColorMode', 'useDark', 'useMediaQuery', 'useScrollLock', 'usePreferredDark', 'usePreferredLight', 'usePreferredReducedMotion', 'usePreferredContrast', 'window', 'document', 'console', 'confirm', 'alert', 'fetch', 'params'];
 /**
  * Strip TypeScript-specific syntax to make it executable as JavaScript
  *

package/dist/variable-extractor.js

@@ -1,5 +1,54 @@
 var {require}=import.meta;import path from "node:path";
 import process from "node:process";
+export const STX_ENGINE_BINDING_NAMES = [
+  "module",
+  "exports",
+  "require",
+  "props",
+  "$props",
+  "defineProps",
+  "withDefaults",
+  "defineClientPayload",
+  "state",
+  "derived",
+  "effect",
+  "batch",
+  "onMount",
+  "onDestroy",
+  "definePageMeta",
+  "useRoute",
+  "useRouter",
+  "useHead",
+  "useSeoMeta",
+  "ref",
+  "reactive",
+  "computed",
+  "watch",
+  "onMounted",
+  "onUnmounted",
+  "nextTick",
+  "defineEmits",
+  "defineExpose",
+  "defineSlots",
+  "provide",
+  "inject",
+  "useColorMode",
+  "useDark",
+  "useMediaQuery",
+  "useScrollLock",
+  "usePreferredDark",
+  "usePreferredLight",
+  "usePreferredReducedMotion",
+  "usePreferredContrast",
+  "window",
+  "document",
+  "console",
+  "confirm",
+  "alert",
+  "fetch",
+  "params"
+];
+const ENGINE_BINDINGS = new Set(STX_ENGINE_BINDING_NAMES), STX_RUNTIME_SPECIFIERS = new Set(["stx", "@stacksjs/stx"]);
 import { findMatchingDelimiter } from "./parser/tokenizer.js";
 import { mergeHeadConfigs, seoMetaToHeadConfig } from "./head.js";
 import { getPublicEnvDefine } from "./public-env.js";
@@ -481,7 +530,7 @@ export async function extractVariables(scriptContent, context, filePath, options
       scriptContextKeys.push(filteredContextKeys[i]);
       scriptContextValues.push(filteredContextValues[i]);
     }
-    const result = await Function("module", "exports", "require", "props", "$props", "defineProps", "withDefaults", "defineClientPayload", "state", "derived", "effect", "batch", "onMount", "onDestroy", "definePageMeta", "useRoute", "useRouter", "useHead", "useSeoMeta", "ref", "reactive", "computed", "watch", "onMounted", "onUnmounted", "nextTick", "defineEmits", "defineExpose", "defineSlots", "provide", "inject", "useColorMode", "useDark", "useMediaQuery", "useScrollLock", "usePreferredDark", "usePreferredLight", "usePreferredReducedMotion", "usePreferredContrast", "window", "document", "console", "confirm", "alert", "fetch", "params", ...propArgNames, ...scriptContextKeys, `return (async () => {
+    const result = await Function(...STX_ENGINE_BINDING_NAMES, ...propArgNames, ...scriptContextKeys, `return (async () => {
         ${convertedScript}
         ${reSyncCode}
         return module.exports
@@ -629,7 +678,7 @@ export function convertToCommonJS(scriptContent, filePath) {
         convertedLines.push(`const ${name} = await import('${resolved}')`);
         convertedLines.push(`module.exports.${name} = ${name};`);
       } else if (namedImportMatch) {
-        const [, names, source] = namedImportMatch, resolved = resolveSource(source), specifiers = [], exported = [];
+        const [, names, source] = namedImportMatch, resolved = resolveSource(source), specifiers = [], exported = [], isStxRuntime = STX_RUNTIME_SPECIFIERS.has(source);
         for (const raw of names.split(",")) {
           const specifier = raw.trim();
           if (!specifier)
@@ -637,14 +686,24 @@ export function convertToCommonJS(scriptContent, filePath) {
           const aliased = specifier.split(/\s+as\s+/);
           if (aliased.length === 2) {
             const [imported, local] = [aliased[0].trim(), aliased[1].trim()];
+            if (isStxRuntime && ENGINE_BINDINGS.has(imported)) {
+              convertedLines.push(`const ${local} = ${imported};`);
+              exported.push(local);
+              continue;
+            }
             specifiers.push(`${imported}: ${local}`);
             exported.push(local);
           } else {
+            if (isStxRuntime && ENGINE_BINDINGS.has(specifier)) {
+              exported.push(specifier);
+              continue;
+            }
             specifiers.push(specifier);
             exported.push(specifier);
           }
         }
-        convertedLines.push(`const { ${specifiers.join(", ")} } = await import('${resolved}')`);
+        if (specifiers.length > 0)
+          convertedLines.push(`const { ${specifiers.join(", ")} } = await import('${resolved}')`);
         for (const local of exported)
           convertedLines.push(`module.exports.${local} = ${local};`);
       } else if (sideEffectMatch) {

package/package.json

@@ -1,7 +1,7 @@
 {
   "name": "@stacksjs/stx",
   "type": "module",
-  "version": "0.2.162",
+  "version": "0.2.163",
   "description": "A performant UI Framework. Powered by Bun.",
   "author": "Chris Breuer <chris@stacksjs.org>",
   "license": "MIT",
@@ -117,12 +117,12 @@
   "dependencies": {
     "@cwcss/crosswind": "^0.2.14",
     "@stacksjs/clapp": "^0.2.0",
-    "@stacksjs/desktop": "0.2.162",
-    "@stacksjs/sanitizer": "0.2.162",
+    "@stacksjs/desktop": "0.2.163",
+    "@stacksjs/sanitizer": "0.2.163",
     "@stacksjs/ts-i18n": "^0.1.11",
-    "bun-plugin-stx": "0.2.162",
+    "bun-plugin-stx": "0.2.163",
     "bunfig": "^0.15.17",
-    "stx-router": "0.2.162",
+    "stx-router": "0.2.163",
     "ts-broadcasting": "^0.0.9",
     "ts-images": "^0.1.9",
     "ts-syntax-highlighter": "^0.2.1"