@vesk/vesk-cli 0.2.50-canary.11 → 0.2.50-canary.12

package/dist/cli.js

@@ -147,7 +147,10 @@ var init_errors = __esm({
       suggestions;
       nextSteps;
       tip;
+      /** The V-diagnostic code, e.g. `V0501`. */
       code;
+      /** The rendered code frame. Used to be written into `code`, which silently replaced the V-code. */
+      frame;
       constructor(message, opts = {}) {
         super(message);
         this.name = "VeskError";
@@ -157,7 +160,8 @@ var init_errors = __esm({
         this.suggestions = opts.suggestions || [];
         this.nextSteps = opts.nextSteps || [];
         this.tip = opts.tip || "";
-        if (opts.code !== void 0) this.code = opts.code;
+        if (opts.code !== void 0 && !opts.code.includes("\n")) this.code = opts.code;
+        if (opts.frame !== void 0) this.frame = opts.frame;
       }
       static notFound(name, candidates = [], context = {}) {
         const allCandidates = [.../* @__PURE__ */ new Set([...candidates, ...VESK_BUILTINS])];
@@ -236,6 +240,84 @@ var init_errors = __esm({
           }
         );
       }
+      /**
+       * A browser-only global used in code the SERVER evaluates.
+       *
+       * On the server these are either absent (`window is not defined`) or, worse,
+       * silently present-but-wrong (`navigator` exists in modern Node, so
+       * `navigator.onLine` is `undefined` and a page renders as if it were offline).
+       * A raw `ReferenceError` at request time is the worst possible report: it
+       * names neither the component nor the fix. This points at the line.
+       */
+      static ssrUnsafeGlobal(name, context = {}) {
+        return new _VeskError(
+          `\`${name}\` is browser-only, and this component body runs during SSR.`,
+          {
+            ...context,
+            code: context.code || "V0501",
+            suggestions: [
+              `Wrap the part that needs the DOM in a {#client} block.`,
+              `Move the code into an event handler \u2014 handlers never run on the server.`,
+              `Do the setup in on_destroy(), which the compiler only emits on the client.`
+            ],
+            nextSteps: [
+              `component ${context.inComponent ?? "this component"}: wrap the \`${name}\` access in {#client}...{/client}.`,
+              `Or guard it: if (typeof ${name} !== 'undefined') { ... }`
+            ],
+            tip: "SSR renders this component before the client exists. Reading a browser global here either throws during the request or (for `navigator`) silently returns undefined."
+          }
+        );
+      }
+      /**
+       * The second name from `track()` called as a function.
+       *
+       * `const &[count, setCount] = track(0)` returns a CELL, not a setter function, so
+       * `setCount(1)` throws `setCount is not a function` from generated code with
+       * no line of user code in the message. The cell is the writer: `.set(v)`.
+       */
+      static cellCalledAsFunction(name, context = {}) {
+        return new _VeskError(
+          `\`${name}\` is a cell, not a function \u2014 cells are written with \`.set()\`.`,
+          {
+            ...context,
+            code: context.code || "V0502",
+            suggestions: [
+              `Use \`${name}.set(...)\` instead of \`${name}(...)\`.`,
+              `Or use the tracked spelling inside an event handler: set(${context.value ?? "value"}, ...)`
+            ],
+            nextSteps: [
+              `const &[${context.value ?? "value"}, ${name}] = track(0) \u2192 \`${name}.set(v)\``,
+              "The value read in markup is the first name; the second is the cell object."
+            ],
+            tip: `\`track()\` returns [value, cell]. \`${name}\` is the cell half.`
+          }
+        );
+      }
+      /**
+       * `set(value, …)` used outside a rewritten expression.
+       *
+       * The compiler rewrites `set(count, v)` to `countCell.set(v)` inside JSX and
+       * event handlers — but a bare statement in the component body is NOT
+       * rewritten, so the runtime `set` receives the value and throws
+       * "Cannot read properties of undefined". Pass the CELL, not the value.
+       */
+      static setWithValueName(name, context = {}) {
+        return new _VeskError(
+          `\`set(${name}, \u2026)\` needs the CELL, and \`${name}\` is the value half of track().`,
+          {
+            ...context,
+            code: context.code || "V0503",
+            suggestions: [
+              `Use \`set(${context.cell ?? "cell"}, \u2026)\` or \`${context.cell ?? "cell"}.set(\u2026)\`.`,
+              `Or move the set() into an event handler, where the compiler rewrites it for you.`
+            ],
+            nextSteps: [
+              `const &[${name}, ${context.cell ?? "cell"}] = track(0) \u2192 \`${context.cell ?? "cell"}.set(v)\``
+            ],
+            tip: "set(cell, value) is only rewritten inside JSX and handlers; in a body statement it reaches the runtime untouched."
+          }
+        );
+      }
       static serverBlockInClient(compName, context = {}) {
         return new _VeskError(
           `{#server} block found in client island "${compName}". Client islands render on both server and client, so {#server} blocks have no effect.`,
@@ -21282,7 +21364,7 @@ function validateBlocks(compName, isClient, body, file, source) {
           const { line, column } = offsetToLineCol2(source, pos);
           opts.line = line;
           opts.column = column;
-          opts.code = codeFrame(source, line, column);
+          opts.frame = codeFrame(source, line, column);
         }
         throw VeskError.serverBlockInClient(compName, opts);
       }
@@ -21294,7 +21376,7 @@ function validateBlocks(compName, isClient, body, file, source) {
           const { line, column } = offsetToLineCol2(source, pos);
           opts.line = line;
           opts.column = column;
-          opts.code = codeFrame(source, line, column);
+          opts.frame = codeFrame(source, line, column);
         }
         throw VeskError.clientBlockInServer(compName, opts);
       }
@@ -38543,7 +38625,10 @@ var init_errors2 = __esm({
       suggestions;
       nextSteps;
       tip;
+      /** The V-diagnostic code, e.g. `V0501`. */
       code;
+      /** The rendered code frame. Used to be written into `code`, which silently replaced the V-code. */
+      frame;
       constructor(message, opts = {}) {
         super(message);
         this.name = "VeskError";
@@ -38553,8 +38638,10 @@ var init_errors2 = __esm({
         this.suggestions = opts.suggestions || [];
         this.nextSteps = opts.nextSteps || [];
         this.tip = opts.tip || "";
-        if (opts.code !== void 0)
+        if (opts.code !== void 0 && !opts.code.includes("\n"))
           this.code = opts.code;
+        if (opts.frame !== void 0)
+          this.frame = opts.frame;
       }
       static notFound(name, candidates = [], context = {}) {
         const allCandidates = [.../* @__PURE__ */ new Set([...candidates, ...VESK_BUILTINS2])];
@@ -38627,6 +38714,75 @@ var init_errors2 = __esm({
           tip: "Use plain objects for data and factory functions for constructors inside .vsk files."
         });
       }
+      /**
+       * A browser-only global used in code the SERVER evaluates.
+       *
+       * On the server these are either absent (`window is not defined`) or, worse,
+       * silently present-but-wrong (`navigator` exists in modern Node, so
+       * `navigator.onLine` is `undefined` and a page renders as if it were offline).
+       * A raw `ReferenceError` at request time is the worst possible report: it
+       * names neither the component nor the fix. This points at the line.
+       */
+      static ssrUnsafeGlobal(name, context = {}) {
+        return new _VeskError(`\`${name}\` is browser-only, and this component body runs during SSR.`, {
+          ...context,
+          code: context.code || "V0501",
+          suggestions: [
+            `Wrap the part that needs the DOM in a {#client} block.`,
+            `Move the code into an event handler \u2014 handlers never run on the server.`,
+            `Do the setup in on_destroy(), which the compiler only emits on the client.`
+          ],
+          nextSteps: [
+            `component ${context.inComponent ?? "this component"}: wrap the \`${name}\` access in {#client}...{/client}.`,
+            `Or guard it: if (typeof ${name} !== 'undefined') { ... }`
+          ],
+          tip: "SSR renders this component before the client exists. Reading a browser global here either throws during the request or (for `navigator`) silently returns undefined."
+        });
+      }
+      /**
+       * The second name from `track()` called as a function.
+       *
+       * `const &[count, setCount] = track(0)` returns a CELL, not a setter function, so
+       * `setCount(1)` throws `setCount is not a function` from generated code with
+       * no line of user code in the message. The cell is the writer: `.set(v)`.
+       */
+      static cellCalledAsFunction(name, context = {}) {
+        return new _VeskError(`\`${name}\` is a cell, not a function \u2014 cells are written with \`.set()\`.`, {
+          ...context,
+          code: context.code || "V0502",
+          suggestions: [
+            `Use \`${name}.set(...)\` instead of \`${name}(...)\`.`,
+            `Or use the tracked spelling inside an event handler: set(${context.value ?? "value"}, ...)`
+          ],
+          nextSteps: [
+            `const &[${context.value ?? "value"}, ${name}] = track(0) \u2192 \`${name}.set(v)\``,
+            "The value read in markup is the first name; the second is the cell object."
+          ],
+          tip: `\`track()\` returns [value, cell]. \`${name}\` is the cell half.`
+        });
+      }
+      /**
+       * `set(value, …)` used outside a rewritten expression.
+       *
+       * The compiler rewrites `set(count, v)` to `countCell.set(v)` inside JSX and
+       * event handlers — but a bare statement in the component body is NOT
+       * rewritten, so the runtime `set` receives the value and throws
+       * "Cannot read properties of undefined". Pass the CELL, not the value.
+       */
+      static setWithValueName(name, context = {}) {
+        return new _VeskError(`\`set(${name}, \u2026)\` needs the CELL, and \`${name}\` is the value half of track().`, {
+          ...context,
+          code: context.code || "V0503",
+          suggestions: [
+            `Use \`set(${context.cell ?? "cell"}, \u2026)\` or \`${context.cell ?? "cell"}.set(\u2026)\`.`,
+            `Or move the set() into an event handler, where the compiler rewrites it for you.`
+          ],
+          nextSteps: [
+            `const &[${name}, ${context.cell ?? "cell"}] = track(0) \u2192 \`${context.cell ?? "cell"}.set(v)\``
+          ],
+          tip: "set(cell, value) is only rewritten inside JSX and handlers; in a body statement it reaches the runtime untouched."
+        });
+      }
       static serverBlockInClient(compName, context = {}) {
         return new _VeskError(`{#server} block found in client island "${compName}". Client islands render on both server and client, so {#server} blocks have no effect.`, {
           ...context,
@@ -63380,12 +63536,14 @@ if (cmd === "build") {
       const l = err.line;
       const c = err.column;
       const code = err.code;
+      const frame = err.frame;
       let out = `vesk build: error \u2014 ${err.message}`;
+      if (code) out = `vesk build: [${code}] ${err.message}`;
       if (f) out += `
   File: ${f}${l ? `:${l}${c ? `:${c}` : ""}` : ""}`;
-      if (code) out += `
+      if (frame) out += `
 
-${code}`;
+${frame}`;
       console.error(out);
       if (err.stack && !String(err.stack).includes(err.message)) console.error(err.stack);
     } else {

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@vesk/vesk-cli",
-  "version": "0.2.50-canary.11",
+  "version": "0.2.50-canary.12",
   "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.50-canary.11",
-    "@vesk/compiler": "^0.2.50-canary.11",
-    "@vesk/plugin-tailwind": "^0.2.50-canary.11",
-    "@vesk/runtime": "^0.2.50-canary.11",
+    "@vesk/adapter": "^0.2.50-canary.12",
+    "@vesk/compiler": "^0.2.50-canary.12",
+    "@vesk/plugin-tailwind": "^0.2.50-canary.12",
+    "@vesk/runtime": "^0.2.50-canary.12",
     "esrap": "^2.3.0",
     "typescript": "^5.8.0",
     "ws": "^8.18.0",