@objectstack/lint 11.9.0 → 12.0.0

package/dist/index.cjs

@@ -1,9 +1,7 @@
 "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)
@@ -17,14 +15,6 @@ 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
@@ -34,6 +24,9 @@ __export(index_exports, {
   CHART_FIELD_UNKNOWN: () => CHART_FIELD_UNKNOWN,
   FIELD_GROUP_EMPTY: () => FIELD_GROUP_EMPTY,
   FIELD_GROUP_UNDECLARED: () => FIELD_GROUP_UNDECLARED,
+  FORM_COLSPAN_ABSOLUTE: () => FORM_COLSPAN_ABSOLUTE,
+  FORM_FIELD_UNKNOWN: () => FORM_FIELD_UNKNOWN,
+  LIST_VIEW_FILTERS_IN_VIEWS_MODE: () => LIST_VIEW_FILTERS_IN_VIEWS_MODE,
   MEASURE_AGGREGATE_INCOHERENT: () => MEASURE_AGGREGATE_INCOHERENT,
   PAGE_SOURCE_CLASSNAME: () => PAGE_SOURCE_CLASSNAME,
   SEMANTIC_ROLE_FIELD_UNKNOWN: () => SEMANTIC_ROLE_FIELD_UNKNOWN,
@@ -48,7 +41,9 @@ __export(index_exports, {
   WIDGET_DATASET_UNKNOWN: () => WIDGET_DATASET_UNKNOWN,
   WIDGET_DIMENSION_UNKNOWN: () => WIDGET_DIMENSION_UNKNOWN,
   WIDGET_MEASURE_UNKNOWN: () => WIDGET_MEASURE_UNKNOWN,
+  validateFormLayout: () => validateFormLayout,
   validateJsxPages: () => validateJsxPages,
+  validateListViewMode: () => validateListViewMode,
   validatePageSourceStyling: () => validatePageSourceStyling,
   validateReactPageProps: () => validateReactPageProps,
   validateReactPages: () => validateReactPages,
@@ -433,6 +428,60 @@ function validateStackExpressions(stack) {
   return issues;
 }
 
+// src/validate-list-view-mode.ts
+var LIST_VIEW_FILTERS_IN_VIEWS_MODE = "list-view-filters-in-views-mode";
+var FORBIDDEN_FIELDS = ["userFilters", "quickFilters"];
+function asArray3(v) {
+  if (Array.isArray(v)) return v;
+  if (v && typeof v === "object") {
+    return Object.entries(v).map(([name, def]) => ({
+      name,
+      ...def
+    }));
+  }
+  return [];
+}
+function scanView(view, where, path, out) {
+  if (!view || typeof view !== "object") return;
+  const rec = view;
+  for (const field of FORBIDDEN_FIELDS) {
+    if (rec[field] == null) continue;
+    out.push({
+      severity: "error",
+      rule: LIST_VIEW_FILTERS_IN_VIEWS_MODE,
+      where,
+      path: `${path}.${field}`,
+      message: `\`${field}\` is a page filters-mode control and is ignored on an object list view ("views" mode) \u2014 the ViewTabBar is the only nav control here.`,
+      hint: `Move \`${field}\` to a page list (InterfaceListPage, "filters" mode), or remove it. See ADR-0053.`
+    });
+  }
+}
+function scanListViews(listViews, wherePrefix, pathPrefix, out) {
+  if (!listViews || typeof listViews !== "object") return;
+  for (const [name, view] of Object.entries(listViews)) {
+    scanView(
+      view,
+      `${wherePrefix} \u203A listViews.${name}`,
+      `${pathPrefix}.listViews.${name}`,
+      out
+    );
+  }
+}
+function validateListViewMode(stack) {
+  const out = [];
+  asArray3(stack.objects).forEach((obj, i) => {
+    const label = typeof obj.name === "string" ? `object "${obj.name}"` : `objects[${i}]`;
+    scanListViews(obj.listViews, label, `objects[${i}]`, out);
+  });
+  asArray3(stack.views).forEach((view, i) => {
+    const named = typeof view.objectName === "string" ? view.objectName : typeof view.name === "string" ? view.name : void 0;
+    const label = named ? `view "${named}"` : `views[${i}]`;
+    scanView(view.list, `${label} \u203A list`, `views[${i}].list`, out);
+    scanListViews(view.listViews, label, `views[${i}]`, out);
+  });
+  return out;
+}
+
 // src/validate-responsive-styles.ts
 var STYLE_NODE_MISSING_ID = "style-node-missing-id";
 var STYLE_CLASSNAME_TAILWIND = "style-classname-tailwind";
@@ -636,7 +685,7 @@ function looksLikeTailwind(className) {
     return false;
   });
 }
-function asArray3(v) {
+function asArray4(v) {
   if (Array.isArray(v)) return v;
   if (v && typeof v === "object") {
     return Object.entries(v).map(([name, def]) => ({ name, ...def }));
@@ -729,13 +778,13 @@ function checkNode(node, pageName, path, findings) {
 }
 function validateResponsiveStyles(stack) {
   const findings = [];
-  const pages = asArray3(stack.pages);
+  const pages = asArray4(stack.pages);
   for (let p = 0; p < pages.length; p++) {
     const page = pages[p];
     const pageName = typeof page.name === "string" ? page.name : `pages[${p}]`;
-    const regions = asArray3(page.regions);
+    const regions = asArray4(page.regions);
     for (let r = 0; r < regions.length; r++) {
-      const components = asArray3(regions[r].components);
+      const components = asArray4(regions[r].components);
       for (let c = 0; c < components.length; c++) {
         checkNode(components[c], pageName, `pages[${p}].regions[${r}].components[${c}]`, findings);
       }
@@ -746,10 +795,10 @@ function validateResponsiveStyles(stack) {
 
 // src/validate-jsx-pages.ts
 var import_sdui_parser = require("@objectstack/sdui-parser");
-var asArray4 = (v) => Array.isArray(v) ? v : [];
+var asArray5 = (v) => Array.isArray(v) ? v : [];
 function validateJsxPages(stack, opts = {}) {
   const findings = [];
-  const pages = asArray4(stack.pages);
+  const pages = asArray5(stack.pages);
   for (let p = 0; p < pages.length; p++) {
     const page = pages[p];
     if (!page || page.kind !== "html" && page.kind !== "jsx") continue;
@@ -782,11 +831,25 @@ function validateJsxPages(stack, opts = {}) {
 }
 
 // src/validate-react-pages.ts
-var import_sucrase = require("sucrase");
-var asArray5 = (v) => Array.isArray(v) ? v : [];
+var import_node_module = require("module");
+var import_meta = {};
+var cachedTransform = null;
+function loadSucraseTransform() {
+  if (cachedTransform) return cachedTransform;
+  const anchor = typeof import_meta !== "undefined" && import_meta.url ? import_meta.url : typeof __filename !== "undefined" ? __filename : process.cwd() + "/";
+  try {
+    cachedTransform = (0, import_node_module.createRequire)(anchor)("sucrase").transform;
+  } catch (err) {
+    throw new Error(
+      `@objectstack/lint: validating a kind:'react' page requires the "sucrase" package, which could not be loaded (${err instanceof Error ? err.message : String(err)}). It is a declared dependency of @objectstack/lint \u2014 if this deployment prunes packages, keep "sucrase" in the image; it is only loaded when a react-source page is validated.`
+    );
+  }
+  return cachedTransform;
+}
+var asArray6 = (v) => Array.isArray(v) ? v : [];
 function validateReactPages(stack) {
   const findings = [];
-  const pages = asArray5(stack.pages);
+  const pages = asArray6(stack.pages);
   for (let p = 0; p < pages.length; p++) {
     const page = pages[p];
     if (!page || page.kind !== "react") continue;
@@ -803,8 +866,9 @@ function validateReactPages(stack) {
       });
       continue;
     }
+    const transform = loadSucraseTransform();
     try {
-      (0, import_sucrase.transform)(source, { transforms: ["jsx", "typescript"], production: true });
+      transform(source, { transforms: ["jsx", "typescript"], production: true });
     } catch (err) {
       const message = err instanceof Error ? err.message : String(err);
       findings.push({
@@ -821,9 +885,23 @@ function validateReactPages(stack) {
 }
 
 // src/validate-react-page-props.ts
-var import_typescript = __toESM(require("typescript"), 1);
+var import_node_module2 = require("module");
 var import_ui = require("@objectstack/spec/ui");
-var asArray6 = (v) => Array.isArray(v) ? v : [];
+var import_meta2 = {};
+var cachedTs = null;
+function loadTypeScript() {
+  if (cachedTs) return cachedTs;
+  const anchor = typeof import_meta2 !== "undefined" && import_meta2.url ? import_meta2.url : typeof __filename !== "undefined" ? __filename : process.cwd() + "/";
+  try {
+    cachedTs = (0, import_node_module2.createRequire)(anchor)("typescript");
+  } catch (err) {
+    throw new Error(
+      `@objectstack/lint: validating a kind:'react' page requires the "typescript" package, which could not be loaded (${err instanceof Error ? err.message : String(err)}). It is a declared dependency of @objectstack/lint \u2014 if this deployment prunes packages, keep "typescript" in the image; it is only loaded when a react-source page is validated.`
+    );
+  }
+  return cachedTs;
+}
+var asArray7 = (v) => Array.isArray(v) ? v : [];
 var BLOCKS = new Map(
   import_ui.REACT_BLOCKS.map((b) => [
     b.tag,
@@ -862,32 +940,33 @@ function nearestKnown(prop, known) {
 }
 function validateReactPageProps(stack) {
   const findings = [];
-  const pages = asArray6(stack.pages);
+  const pages = asArray7(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}`);
+    const tsc = loadTypeScript();
     let sf;
     try {
-      sf = import_typescript.default.createSourceFile("page.tsx", source, import_typescript.default.ScriptTarget.Latest, true, import_typescript.default.ScriptKind.TSX);
+      sf = tsc.createSourceFile("page.tsx", source, tsc.ScriptTarget.Latest, true, tsc.ScriptKind.TSX);
     } catch {
       continue;
     }
     const visit = (node) => {
-      if (import_typescript.default.isJsxOpeningElement(node) || import_typescript.default.isJsxSelfClosingElement(node)) {
+      if (tsc.isJsxOpeningElement(node) || tsc.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)) {
+            if (tsc.isJsxSpreadAttribute(a)) {
               hasSpread = true;
               continue;
             }
-            if (import_typescript.default.isJsxAttribute(a)) used.add(a.name.getText(sf));
+            if (tsc.isJsxAttribute(a)) used.add(a.name.getText(sf));
           }
           const where = `page "${name}" \u203A <${tag}>`;
           const path = `pages[${p}].source`;
@@ -920,7 +999,7 @@ function validateReactPageProps(stack) {
           }
         }
       }
-      import_typescript.default.forEachChild(node, visit);
+      tsc.forEachChild(node, visit);
     };
     visit(sf);
   }
@@ -929,11 +1008,11 @@ function validateReactPageProps(stack) {
 
 // src/validate-page-source-styling.ts
 var PAGE_SOURCE_CLASSNAME = "page-source-className-tailwind";
-var asArray7 = (v) => Array.isArray(v) ? v : [];
+var asArray8 = (v) => Array.isArray(v) ? v : [];
 var CLASSNAME_ATTR = /\bclassName\s*=\s*["'{]/g;
 function validatePageSourceStyling(stack) {
   const findings = [];
-  const pages = asArray7(stack.pages);
+  const pages = asArray8(stack.pages);
   for (let p = 0; p < pages.length; p++) {
     const page = pages[p];
     if (!page) continue;
@@ -962,7 +1041,7 @@ function validatePageSourceStyling(stack) {
 var import_data2 = require("@objectstack/spec/data");
 var TITLE_FORMAT_RETIRED = "title-format-retired";
 var TITLE_UNRESOLVABLE = "title-unresolvable";
-function asArray8(v) {
+function asArray9(v) {
   if (Array.isArray(v)) return v;
   if (v && typeof v === "object") {
     return Object.entries(v).map(([name, def]) => ({ name, ...def }));
@@ -971,7 +1050,7 @@ function asArray8(v) {
 }
 function validateRecordTitle(stack) {
   const findings = [];
-  const objects = asArray8(stack.objects);
+  const objects = asArray9(stack.objects);
   for (let i = 0; i < objects.length; i++) {
     const obj = objects[i];
     const objName = typeof obj.name === "string" ? obj.name : `(object ${i})`;
@@ -1006,7 +1085,7 @@ function validateRecordTitle(stack) {
 var FIELD_GROUP_UNDECLARED = "field-group-undeclared";
 var FIELD_GROUP_EMPTY = "field-group-empty";
 var SEMANTIC_ROLE_FIELD_UNKNOWN = "semantic-role-field-unknown";
-function asArray9(v) {
+function asArray10(v) {
   if (Array.isArray(v)) return v;
   if (v && typeof v === "object") {
     return Object.entries(v).map(([name, def]) => ({ name, ...def }));
@@ -1015,7 +1094,7 @@ function asArray9(v) {
 }
 function validateSemanticRoles(stack) {
   const findings = [];
-  const objects = asArray9(stack.objects);
+  const objects = asArray10(stack.objects);
   for (let i = 0; i < objects.length; i++) {
     const obj = objects[i];
     if (!obj || typeof obj !== "object") continue;
@@ -1081,12 +1160,94 @@ function validateSemanticRoles(stack) {
   }
   return findings;
 }
+
+// src/validate-form-layout.ts
+var FORM_FIELD_UNKNOWN = "form-field-unknown";
+var FORM_COLSPAN_ABSOLUTE = "absolute-colspan-discouraged";
+function asArray11(v) {
+  if (Array.isArray(v)) return v;
+  if (v && typeof v === "object") {
+    return Object.entries(v).map(([name, def]) => ({ name, ...def }));
+  }
+  return [];
+}
+function fieldNameOf(entry) {
+  if (typeof entry === "string") return entry.length > 0 ? entry : null;
+  if (entry && typeof entry === "object" && !Array.isArray(entry)) {
+    const f = entry.field;
+    return typeof f === "string" && f.length > 0 ? f : null;
+  }
+  return null;
+}
+function boundObject(view) {
+  const data = view.data;
+  if (data && typeof data === "object" && typeof data.object === "string") {
+    return data.object;
+  }
+  return typeof view.objectName === "string" ? view.objectName : void 0;
+}
+function validateFormLayout(stack) {
+  const findings = [];
+  const objectFields = /* @__PURE__ */ new Map();
+  for (const obj of asArray11(stack.objects)) {
+    const name = typeof obj.name === "string" ? obj.name : void 0;
+    if (!name) continue;
+    const fields = obj.fields && typeof obj.fields === "object" && !Array.isArray(obj.fields) ? Object.keys(obj.fields) : [];
+    objectFields.set(name, new Set(fields));
+  }
+  const views = asArray11(stack.views);
+  for (let i = 0; i < views.length; i++) {
+    const view = views[i];
+    if (!view || typeof view !== "object") continue;
+    const sections = Array.isArray(view.sections) ? view.sections : null;
+    if (!sections) continue;
+    const viewName = typeof view.name === "string" ? view.name : `(view ${i})`;
+    const objName = boundObject(view);
+    const known = objName ? objectFields.get(objName) : void 0;
+    const where = `view "${viewName}"`;
+    const base = `views[${i}]`;
+    for (let s = 0; s < sections.length; s++) {
+      const sec = sections[s];
+      const secFields = sec && typeof sec === "object" && Array.isArray(sec.fields) ? sec.fields : [];
+      for (let f = 0; f < secFields.length; f++) {
+        const entry = secFields[f];
+        const fname = fieldNameOf(entry);
+        const fpath = `${base}.sections[${s}].fields[${f}]`;
+        if (fname && known && !known.has(fname)) {
+          findings.push({
+            severity: "warning",
+            rule: FORM_FIELD_UNKNOWN,
+            where,
+            path: fpath,
+            message: `${viewName}: field "${fname}" is not a field on object "${objName}" \u2014 it is silently skipped and never renders on the form`,
+            hint: `Fix the field name, or add "${fname}" to ${objName}. Section field references must match the object's field names exactly.`
+          });
+        }
+        const colSpan = entry && typeof entry === "object" && !Array.isArray(entry) ? entry.colSpan : void 0;
+        if (colSpan != null) {
+          findings.push({
+            severity: "warning",
+            rule: FORM_COLSPAN_ABSOLUTE,
+            where,
+            path: `${fpath}.colSpan`,
+            message: `${viewName}: field "${fname ?? "?"}" sets absolute colSpan ${String(colSpan)} \u2014 the form's column count is derived per surface (mobile 1 / modal 2 / page 3-4), so a fixed span only aligns at one width`,
+            hint: `Prefer span: 'full' (whole row at any column count), or omit for auto width. The renderer clamps colSpan to the current column count.`
+          });
+        }
+      }
+    }
+  }
+  return findings;
+}
 // Annotate the CommonJS export names for ESM import in node:
 0 && (module.exports = {
   CHART_CONFIG_MISSING,
   CHART_FIELD_UNKNOWN,
   FIELD_GROUP_EMPTY,
   FIELD_GROUP_UNDECLARED,
+  FORM_COLSPAN_ABSOLUTE,
+  FORM_FIELD_UNKNOWN,
+  LIST_VIEW_FILTERS_IN_VIEWS_MODE,
   MEASURE_AGGREGATE_INCOHERENT,
   PAGE_SOURCE_CLASSNAME,
   SEMANTIC_ROLE_FIELD_UNKNOWN,
@@ -1101,7 +1262,9 @@ function validateSemanticRoles(stack) {
   WIDGET_DATASET_UNKNOWN,
   WIDGET_DIMENSION_UNKNOWN,
   WIDGET_MEASURE_UNKNOWN,
+  validateFormLayout,
   validateJsxPages,
+  validateListViewMode,
   validatePageSourceStyling,
   validateReactPageProps,
   validateReactPages,