@objectstack/lint 11.3.0 → 11.5.0

package/dist/index.cjs

@@ -1,7 +1,9 @@
 "use strict";
+var __create = Object.create;
 var __defProp = Object.defineProperty;
 var __getOwnPropDesc = Object.getOwnPropertyDescriptor;
 var __getOwnPropNames = Object.getOwnPropertyNames;
+var __getProtoOf = Object.getPrototypeOf;
 var __hasOwnProp = Object.prototype.hasOwnProperty;
 var __export = (target, all) => {
   for (var name in all)
@@ -15,6 +17,14 @@ var __copyProps = (to, from, except, desc) => {
   }
   return to;
 };
+var __toESM = (mod, isNodeMode, target) => (target = mod != null ? __create(__getProtoOf(mod)) : {}, __copyProps(
+  // If the importer is in node compatibility mode or this is not an ESM
+  // file that has been converted to a CommonJS file using a Babel-
+  // compatible transform (i.e. "__esModule" has not been set), then set
+  // "default" to the CommonJS "module.exports" for node compatibility.
+  isNodeMode || !mod || !mod.__esModule ? __defProp(target, "default", { value: mod, enumerable: true }) : target,
+  mod
+));
 var __toCommonJS = (mod) => __copyProps(__defProp({}, "__esModule", { value: true }), mod);
 
 // src/index.ts
@@ -23,6 +33,7 @@ __export(index_exports, {
   CHART_CONFIG_MISSING: () => CHART_CONFIG_MISSING,
   CHART_FIELD_UNKNOWN: () => CHART_FIELD_UNKNOWN,
   MEASURE_AGGREGATE_INCOHERENT: () => MEASURE_AGGREGATE_INCOHERENT,
+  PAGE_SOURCE_CLASSNAME: () => PAGE_SOURCE_CLASSNAME,
   STYLE_CLASSNAME_TAILWIND: () => STYLE_CLASSNAME_TAILWIND,
   STYLE_NODE_MISSING_ID: () => STYLE_NODE_MISSING_ID,
   STYLE_RESPONSIVE_NO_BASE: () => STYLE_RESPONSIVE_NO_BASE,
@@ -35,6 +46,9 @@ __export(index_exports, {
   WIDGET_DIMENSION_UNKNOWN: () => WIDGET_DIMENSION_UNKNOWN,
   WIDGET_MEASURE_UNKNOWN: () => WIDGET_MEASURE_UNKNOWN,
   validateJsxPages: () => validateJsxPages,
+  validatePageSourceStyling: () => validatePageSourceStyling,
+  validateReactPageProps: () => validateReactPageProps,
+  validateReactPages: () => validateReactPages,
   validateRecordTitle: () => validateRecordTitle,
   validateResponsiveStyles: () => validateResponsiveStyles,
   validateStackExpressions: () => validateStackExpressions,
@@ -734,7 +748,7 @@ function validateJsxPages(stack, opts = {}) {
   const pages = asArray4(stack.pages);
   for (let p = 0; p < pages.length; p++) {
     const page = pages[p];
-    if (!page || page.kind !== "jsx") continue;
+    if (!page || page.kind !== "html" && page.kind !== "jsx") continue;
     const name = String(page.name ?? `#${p}`);
     const source = page.source;
     if (typeof source !== "string" || source.trim() === "") {
@@ -743,7 +757,7 @@ function validateJsxPages(stack, opts = {}) {
         rule: "jsx-page-empty-source",
         where: `page "${name}"`,
         path: `pages[${p}].source`,
-        message: "kind:'jsx' page has no `source`.",
+        message: `kind:'${page.kind}' page has no \`source\`.`,
         hint: "Author the page as a constrained JSX/Tailwind string in `source`."
       });
       continue;
@@ -763,11 +777,188 @@ function validateJsxPages(stack, opts = {}) {
   return findings;
 }
 
+// src/validate-react-pages.ts
+var import_sucrase = require("sucrase");
+var asArray5 = (v) => Array.isArray(v) ? v : [];
+function validateReactPages(stack) {
+  const findings = [];
+  const pages = asArray5(stack.pages);
+  for (let p = 0; p < pages.length; p++) {
+    const page = pages[p];
+    if (!page || page.kind !== "react") continue;
+    const name = String(page.name ?? `#${p}`);
+    const source = page.source;
+    if (typeof source !== "string" || source.trim() === "") {
+      findings.push({
+        severity: "error",
+        rule: "react-page-empty-source",
+        where: `page "${name}"`,
+        path: `pages[${p}].source`,
+        message: "kind:'react' page has no `source`.",
+        hint: "Author the page as a real React component string in `source`."
+      });
+      continue;
+    }
+    try {
+      (0, import_sucrase.transform)(source, { transforms: ["jsx", "typescript"], production: true });
+    } catch (err) {
+      const message = err instanceof Error ? err.message : String(err);
+      findings.push({
+        severity: "error",
+        rule: "react-page-syntax",
+        where: `page "${name}"`,
+        path: `pages[${p}].source`,
+        message: `kind:'react' source has a syntax error: ${message.split("\n")[0]}`,
+        hint: "The source is transpiled (never executed) at build to catch syntax errors early \u2014 fix the JS/JSX."
+      });
+    }
+  }
+  return findings;
+}
+
+// src/validate-react-page-props.ts
+var import_typescript = __toESM(require("typescript"), 1);
+var import_ui = require("@objectstack/spec/ui");
+var asArray6 = (v) => Array.isArray(v) ? v : [];
+var BLOCKS = new Map(
+  import_ui.REACT_BLOCKS.map((b) => [
+    b.tag,
+    {
+      requiredBindings: b.interactions.filter((i) => i.required).map((i) => i.name),
+      knownProps: new Set(b.interactions.map((i) => i.name))
+    }
+  ])
+);
+function editDistance(a, b, cap = 2) {
+  if (Math.abs(a.length - b.length) > cap) return cap + 1;
+  const dp = Array.from({ length: a.length + 1 }, (_, i) => i);
+  for (let j = 1; j <= b.length; j++) {
+    let prev = dp[0];
+    dp[0] = j;
+    for (let i = 1; i <= a.length; i++) {
+      const tmp = dp[i];
+      dp[i] = Math.min(dp[i] + 1, dp[i - 1] + 1, prev + (a[i - 1] === b[j - 1] ? 0 : 1));
+      prev = tmp;
+    }
+  }
+  return dp[a.length];
+}
+function nearestKnown(prop, known) {
+  if (known.has(prop)) return null;
+  let best = null;
+  let bestD = 3;
+  for (const k of known) {
+    const d = editDistance(prop, k);
+    if (d < bestD) {
+      bestD = d;
+      best = k;
+    }
+  }
+  return bestD <= 2 ? best : null;
+}
+function validateReactPageProps(stack) {
+  const findings = [];
+  const pages = asArray6(stack.pages);
+  for (let p = 0; p < pages.length; p++) {
+    const page = pages[p];
+    if (!page || page.kind !== "react") continue;
+    const source = page.source;
+    if (typeof source !== "string" || source.trim() === "") continue;
+    const name = String(page.name ?? `#${p}`);
+    let sf;
+    try {
+      sf = import_typescript.default.createSourceFile("page.tsx", source, import_typescript.default.ScriptTarget.Latest, true, import_typescript.default.ScriptKind.TSX);
+    } catch {
+      continue;
+    }
+    const visit = (node) => {
+      if (import_typescript.default.isJsxOpeningElement(node) || import_typescript.default.isJsxSelfClosingElement(node)) {
+        const tag = node.tagName.getText(sf);
+        const block = BLOCKS.get(tag);
+        if (block) {
+          let hasSpread = false;
+          const used = /* @__PURE__ */ new Set();
+          for (const a of node.attributes.properties) {
+            if (import_typescript.default.isJsxSpreadAttribute(a)) {
+              hasSpread = true;
+              continue;
+            }
+            if (import_typescript.default.isJsxAttribute(a)) used.add(a.name.getText(sf));
+          }
+          const where = `page "${name}" \u203A <${tag}>`;
+          const path = `pages[${p}].source`;
+          if (!hasSpread) {
+            for (const req of block.requiredBindings) {
+              if (!used.has(req)) {
+                findings.push({
+                  severity: "error",
+                  rule: "react-prop-missing-required",
+                  where,
+                  path,
+                  message: `<${tag}> is missing the required prop "${req}".`,
+                  hint: `Pass ${req}={\u2026}. See the react-tier component contract.`
+                });
+              }
+            }
+          }
+          for (const u of used) {
+            const near = nearestKnown(u, block.knownProps);
+            if (near) {
+              findings.push({
+                severity: "warning",
+                rule: "react-prop-typo",
+                where,
+                path,
+                message: `<${tag}> has prop "${u}" \u2014 did you mean "${near}"?`,
+                hint: "Likely a typo of a contract prop. Fix it or remove it."
+              });
+            }
+          }
+        }
+      }
+      import_typescript.default.forEachChild(node, visit);
+    };
+    visit(sf);
+  }
+  return findings;
+}
+
+// src/validate-page-source-styling.ts
+var PAGE_SOURCE_CLASSNAME = "page-source-className-tailwind";
+var asArray7 = (v) => Array.isArray(v) ? v : [];
+var CLASSNAME_ATTR = /\bclassName\s*=\s*["'{]/g;
+function validatePageSourceStyling(stack) {
+  const findings = [];
+  const pages = asArray7(stack.pages);
+  for (let p = 0; p < pages.length; p++) {
+    const page = pages[p];
+    if (!page) continue;
+    const kind = page.kind;
+    if (kind !== "html" && kind !== "react" && kind !== "jsx") continue;
+    const source = page.source;
+    if (typeof source !== "string" || source.trim() === "") continue;
+    const name = String(page.name ?? `#${p}`);
+    CLASSNAME_ATTR.lastIndex = 0;
+    let count = 0;
+    while (CLASSNAME_ATTR.exec(source) !== null) count++;
+    if (count === 0) continue;
+    findings.push({
+      severity: "warning",
+      rule: PAGE_SOURCE_CLASSNAME,
+      where: `page "${name}"`,
+      path: `pages[${p}].source`,
+      message: `${count} \`className\` attribute${count > 1 ? "s" : ""} in ${String(kind)}-source page \u2014 Tailwind utilities in page source silently produce no CSS (the build never scans authored metadata; ADR-0065).`,
+      hint: kind === "react" ? `Style with inline style={{}} using hsl(var(--token)) theme colors (e.g. color:'hsl(var(--foreground))', background:'hsl(var(--card))'); render drawer/modal via <ObjectForm formType="drawer"|"modal"> instead of hand-rolled overlays.` : `Lay out with the components' structured props (<flex direction gap>, <grid columns>) and add CSS via a JSON style object style={{"color":"hsl(var(--foreground))"}}; do not use Tailwind className.`
+    });
+  }
+  return findings;
+}
+
 // src/validate-record-title.ts
 var import_data2 = require("@objectstack/spec/data");
 var TITLE_FORMAT_RETIRED = "title-format-retired";
 var TITLE_UNRESOLVABLE = "title-unresolvable";
-function asArray5(v) {
+function asArray8(v) {
   if (Array.isArray(v)) return v;
   if (v && typeof v === "object") {
     return Object.entries(v).map(([name, def]) => ({ name, ...def }));
@@ -776,7 +967,7 @@ function asArray5(v) {
 }
 function validateRecordTitle(stack) {
   const findings = [];
-  const objects = asArray5(stack.objects);
+  const objects = asArray8(stack.objects);
   for (let i = 0; i < objects.length; i++) {
     const obj = objects[i];
     const objName = typeof obj.name === "string" ? obj.name : `(object ${i})`;
@@ -811,6 +1002,7 @@ function validateRecordTitle(stack) {
   CHART_CONFIG_MISSING,
   CHART_FIELD_UNKNOWN,
   MEASURE_AGGREGATE_INCOHERENT,
+  PAGE_SOURCE_CLASSNAME,
   STYLE_CLASSNAME_TAILWIND,
   STYLE_NODE_MISSING_ID,
   STYLE_RESPONSIVE_NO_BASE,
@@ -823,6 +1015,9 @@ function validateRecordTitle(stack) {
   WIDGET_DIMENSION_UNKNOWN,
   WIDGET_MEASURE_UNKNOWN,
   validateJsxPages,
+  validatePageSourceStyling,
+  validateReactPageProps,
+  validateReactPages,
   validateRecordTitle,
   validateResponsiveStyles,
   validateStackExpressions,