@vesk/vesk-cli 0.2.54 → 0.2.56

package/dist/cli.js

@@ -3991,6 +3991,10 @@ function isRouteHref(href) {
   if (!routerMode()) return !href.startsWith("#");
   return !href.startsWith("#") || href.startsWith("#/");
 }
+function addActiveClass(el, activeClass) {
+  const tokens = String(activeClass || "active").split(/\s+/);
+  for (const t of tokens) if (t) el.classList.add(t);
+}
 function setIsHydrating(v) {
   __isHydrating = v;
 }
@@ -4264,7 +4268,7 @@ function NavLink(props, registry, hydrate) {
     const routeHref2 = routeHrefOf(props.href);
     const isActive2 = routeHref2 === path2 || routeHref2 !== "/" && path2.startsWith(routeHref2) && (path2.length === routeHref2.length || path2[routeHref2.length] === "/" || path2[routeHref2.length] === "?");
     if (isActive2) {
-      a2.classList.add(props.activeClass || "active");
+      addActiveClass(a2, props.activeClass);
       if (props.ariaCurrent !== false) a2.setAttribute("aria-current", "page");
     }
     return claimed ? document.createDocumentFragment() : a2;
@@ -4294,7 +4298,7 @@ function NavLink(props, registry, hydrate) {
       const routeHref2 = routeHrefOf(props.href);
       const isActive2 = routeHref2 === path2 || routeHref2 !== "/" && path2.startsWith(routeHref2) && (path2.length === routeHref2.length || path2[routeHref2.length] === "/" || path2[routeHref2.length] === "?");
       if (isActive2) {
-        q.classList.add(props.activeClass || "active");
+        addActiveClass(q, props.activeClass);
         if (props.ariaCurrent !== false) q.setAttribute("aria-current", "page");
       }
       const claimed = q.parentNode !== null;
@@ -4306,7 +4310,7 @@ function NavLink(props, registry, hydrate) {
   const routeHref = routeHrefOf(props.href);
   const isActive = routeHref === path || routeHref !== "/" && path.startsWith(routeHref) && (path.length === routeHref.length || path[routeHref.length] === "/" || path[routeHref.length] === "?");
   if (isActive) {
-    a.classList.add(props.activeClass || "active");
+    addActiveClass(a, props.activeClass);
     if (props.ariaCurrent !== false) a.setAttribute("aria-current", "page");
   }
   return a;
@@ -21870,6 +21874,94 @@ function idxOfQuote(s, from) {
   }
   return -1;
 }
+function warnStaleConstBindings(comp, file) {
+  const reactive = /* @__PURE__ */ new Set();
+  const eachNode = (nodes, fn) => {
+    for (const n of nodes) {
+      fn(n);
+      if (n instanceof ComponentCall) eachNode(n.children, fn);
+      if (n instanceof OpaqueDynamicRegion) {
+        eachNode(n.consequentNodes, fn);
+        eachNode(n.alternateNodes, fn);
+      }
+      if (n instanceof MapRegion) eachNode(n.bodyTemplate, fn);
+      if (n instanceof ForLoop) eachNode(n.bodyTemplate, fn);
+      if (n instanceof WhileLoop) eachNode(n.bodyTemplate, fn);
+      if (n instanceof SwitchBlock) for (const c of n.cases) eachNode(c.body, fn);
+      if (n instanceof TryCatch) {
+        eachNode(n.bodyTemplate, fn);
+        eachNode(n.catchBody, fn);
+      }
+      if (n instanceof StaticNode) eachNode(n.children, fn);
+    }
+  };
+  eachNode(comp.body, (n) => {
+    if (!(n instanceof TrackDecl)) return;
+    reactive.add(n.name);
+    if (n.rawName) reactive.add(n.rawName);
+  });
+  if (reactive.size === 0) return;
+  const stale = /* @__PURE__ */ new Map();
+  eachNode(comp.body, (n) => {
+    if (!(n instanceof RuntimeStatement)) return;
+    const st = n.ast;
+    if (st?.type !== "VariableDeclaration") return;
+    for (const d of st.declarations || []) {
+      if (d.id?.type !== "Identifier" || !d.init) continue;
+      if (reactive.has(d.id.name)) continue;
+      if (d.init.type === "Identifier" && reactive.has(d.init.name)) continue;
+      const seen = /* @__PURE__ */ new Set();
+      collectIdentifiers(d.init, seen);
+      for (const name of seen) {
+        if (!reactive.has(name)) continue;
+        stale.set(d.id.name, name);
+        break;
+      }
+    }
+  });
+  if (stale.size === 0) return;
+  const reported2 = /* @__PURE__ */ new Set();
+  const report = (name) => {
+    if (reported2.has(name)) return;
+    const src2 = stale.get(name);
+    if (!src2) return;
+    reported2.add(name);
+    console.warn(
+      `[vesk] ${comp.name}: \`const ${name} = ...\` is computed once and reads the tracked value \`${src2}\`, so bindings that use \`${name}\` never update. Inline the expression in the binding, or wrap the value in derived().`
+    );
+  };
+  const isEventAttr = (target) => !!target && /^on./.test(target);
+  eachNode(comp.body, (n) => {
+    if (n instanceof DynamicBinding) {
+      if (!isEventAttr(n.target)) {
+        const seen = /* @__PURE__ */ new Set();
+        collectIdentifiers(n.expression.ast, seen);
+        for (const name of seen) if (stale.has(name)) report(name);
+      }
+      return;
+    }
+    if (n instanceof ComponentCall) {
+      for (const pr of n.props) {
+        const seen = /* @__PURE__ */ new Set();
+        collectIdentifiers(pr.value.ast, seen);
+        for (const name of seen) if (stale.has(name)) report(name);
+      }
+    }
+  });
+}
+function collectIdentifiers(node, into) {
+  if (!node || typeof node !== "object") return;
+  if (Array.isArray(node)) {
+    for (const child of node) collectIdentifiers(child, into);
+    return;
+  }
+  if (typeof node.type !== "string") return;
+  if (node.type === "Identifier" && typeof node.name === "string") into.add(node.name);
+  for (const key of Object.keys(node)) {
+    if (key === "loc" || key === "range" || key === "start" || key === "end") continue;
+    collectIdentifiers(node[key], into);
+  }
+}
 function generateIR(ast, source, filename) {
   __vskAnnotations = ast.__vskAnnotations ?? [];
   __slotProps = null;
@@ -22157,6 +22249,7 @@ function generateIR(ast, source, filename) {
         imports.push(`import { ${missing.join(", ")} } from '@vesk/runtime';`);
       }
     }
+    for (const comp of components) warnStaleConstBindings(comp, file);
     const irRoot = new IRRoot(components, imports, importedNames, staticProps, loadFn, topLevelCode, exportAliases, reexportSources);
     irRoot.metadataSource = metadataSource;
     return irRoot;

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@vesk/vesk-cli",
-  "version": "0.2.54",
+  "version": "0.2.56",
   "type": "module",
   "description": "Vesk — compiler-first framework for the post-VDOM web. vesk dev, vesk build.",
   "bin": {
@@ -8,10 +8,10 @@
   },
   "main": "./dist/cli.js",
   "dependencies": {
-    "@vesk/adapter": "^0.2.54",
-    "@vesk/compiler": "^0.2.54",
-    "@vesk/plugin-tailwind": "^0.2.54",
-    "@vesk/runtime": "^0.2.54",
+    "@vesk/adapter": "^0.2.56",
+    "@vesk/compiler": "^0.2.56",
+    "@vesk/plugin-tailwind": "^0.2.56",
+    "@vesk/runtime": "^0.2.56",
     "esrap": "^2.3.0",
     "typescript": "^5.8.0",
     "ws": "^8.18.0",