@grexx/grexxlinter 0.2.1318 → 0.2.1372

package/cli.js

@@ -14547,8 +14547,7 @@ function buildCoverageIndex(doc) {
   return {
     clauseCovered(casetype, field, modality, when) {
       if (!casetype || !field) return null;
-      const v = map.get(key(casetype, field, modality, when));
-      return v === void 0 ? null : v;
+      return map.get(key(casetype, field, modality, when)) ?? null;
     },
     uncoveredCount: (casetype) => uncovered.get(casetype) ?? 0
   };
@@ -15039,7 +15038,8 @@ function lintDocument(ctx, input) {
           if (!emittedReqAny.has(key2)) {
             emittedReqAny.add(key2);
             const state = describeState(schema, g.err, input.value, ctx.registry.resolveRef);
-            push("required-field", g.instancePath, null, `missing at least one of: ${g.fields.join(", ")}${state ? ` ${state}` : ""}`);
+            const stateSuffix = state ? ` ${state}` : "";
+            push("required-field", g.instancePath, null, `missing at least one of: ${g.fields.join(", ")}${stateSuffix}`);
           }
           continue;
         }
@@ -15063,11 +15063,14 @@ function lintDocument(ctx, input) {
       }
       if (e.keyword === "type") {
         const actual = valueAtPointer(input.value, e.instancePath);
-        const recv = actual === null ? "null" : Array.isArray(actual) ? "array" : typeof actual;
+        let recv = typeof actual;
+        if (actual === null) recv = "null";
+        else if (Array.isArray(actual)) recv = "array";
         let val = JSON.stringify(actual) ?? "";
         if (val.length > 60) val = `${val.slice(0, 57)}\u2026"`;
         const expected = String(e.params.type);
-        push("invalid-type", path, field, `expected ${expected} but received ${recv}${actual === void 0 ? "" : ` with value ${val}`}`);
+        const withValue = actual === void 0 ? "" : ` with value ${val}`;
+        push("invalid-type", path, field, `expected ${expected} but received ${recv}${withValue}`);
         continue;
       }
       const rule = ruleForKeyword(e.keyword);
@@ -15124,7 +15127,7 @@ function toJSOrError(doc) {
 // src/cli.ts
 var PARALLEL_THRESHOLD = 200;
 var MAX_WORKERS = 8;
-var VERSION = true ? "0.2.1318" : "0.0.0-dev";
+var VERSION = true ? "0.2.1372" : "0.0.0-dev";
 function isNewer(latest, current) {
   const a = latest.split(".").map(Number);
   const b = current.split(".").map(Number);
@@ -15195,6 +15198,7 @@ function discover(paths, exclude) {
   for (const p of paths) walk(p);
   return files;
 }
+var KEY_SEP = String.fromCodePoint(0);
 var COLORS = { error: "\x1B[31m", warning: "\x1B[33m", info: "\x1B[36m" };
 var RESET = "\x1B[0m";
 var useColor = process.stdout.isTTY;
@@ -15205,7 +15209,8 @@ function printText(results, settings, quiet, errorsOnly) {
     const msgs = errorsOnly ? r.messages.filter((m) => m.severity === "error") : r.messages;
     if (quiet && msgs.length === 0) continue;
     const rel = r.file ? relative(process.cwd(), r.file) : "<input>";
-    const head = `${rel}${r.casetype ? ` (${r.casetype})` : ""}`;
+    const casetypeTag = r.casetype ? ` (${r.casetype})` : "";
+    const head = `${rel}${casetypeTag}`;
     if (msgs.length === 0) {
       if (!quiet) console.log(`${paint("ok", COLORS.info)}  ${head}`);
       continue;
@@ -15218,9 +15223,12 @@ function printText(results, settings, quiet, errorsOnly) {
   }
   const s = summarize(results.filter((r) => !r.skipped));
   const skipped = results.filter((r) => r.skipped).length;
+  const skippedNote = skipped ? `, ${skipped} skipped` : "";
+  const errorsText = paint(`${s.errors} error(s)`, COLORS.error);
+  const warningsText = paint(`${s.warnings} warning(s)`, COLORS.warning);
   console.log(
     `
-${s.files} file(s)${skipped ? `, ${skipped} skipped` : ""} \u2014 ${paint(`${s.errors} error(s)`, COLORS.error)}, ${paint(`${s.warnings} warning(s)`, COLORS.warning)}, ${s.infos} info(s) [fail-on: ${settings.failOn}]`
+${s.files} file(s)${skippedNote} \u2014 ${errorsText}, ${warningsText}, ${s.infos} info(s) [fail-on: ${settings.failOn}]`
   );
 }
 function printReport(results, kind, errorsOnly) {
@@ -15243,7 +15251,7 @@ function printReport(results, kind, errorsOnly) {
   }
   const by = /* @__PURE__ */ new Map();
   for (const m of msgs) {
-    const key2 = `${m.casetype ?? "?"}\0${m.rule}\0${m.severity}\0${m.message}`;
+    const key2 = [m.casetype ?? "?", m.rule, m.severity, m.message].join(KEY_SEP);
     const e = by.get(key2) ?? { count: 0, casetype: m.casetype ?? "?", severity: m.severity, message: m.message };
     e.count++;
     by.set(key2, e);
@@ -15363,5 +15371,5 @@ async function main() {
   }
   process.exit(summarize(results.filter((r) => !r.skipped)).exitCode);
 }
-if (isMainThread) void main();
+if (isMainThread) await main();
 else runWorker();

package/lsp.js

@@ -23616,8 +23616,7 @@ function buildCoverageIndex(doc) {
   return {
     clauseCovered(casetype, field, modality, when) {
       if (!casetype || !field) return null;
-      const v = map.get(key(casetype, field, modality, when));
-      return v === void 0 ? null : v;
+      return map.get(key(casetype, field, modality, when)) ?? null;
     },
     uncoveredCount: (casetype) => uncovered.get(casetype) ?? 0
   };
@@ -23997,7 +23996,8 @@ function lintDocument(ctx2, input) {
           if (!emittedReqAny.has(key2)) {
             emittedReqAny.add(key2);
             const state = describeState(schema, g.err, input.value, ctx2.registry.resolveRef);
-            push("required-field", g.instancePath, null, `missing at least one of: ${g.fields.join(", ")}${state ? ` ${state}` : ""}`);
+            const stateSuffix = state ? ` ${state}` : "";
+            push("required-field", g.instancePath, null, `missing at least one of: ${g.fields.join(", ")}${stateSuffix}`);
           }
           continue;
         }
@@ -24021,11 +24021,14 @@ function lintDocument(ctx2, input) {
       }
       if (e.keyword === "type") {
         const actual = valueAtPointer(input.value, e.instancePath);
-        const recv = actual === null ? "null" : Array.isArray(actual) ? "array" : typeof actual;
+        let recv = typeof actual;
+        if (actual === null) recv = "null";
+        else if (Array.isArray(actual)) recv = "array";
         let val = JSON.stringify(actual) ?? "";
         if (val.length > 60) val = `${val.slice(0, 57)}\u2026"`;
         const expected = String(e.params.type);
-        push("invalid-type", path, field, `expected ${expected} but received ${recv}${actual === void 0 ? "" : ` with value ${val}`}`);
+        const withValue = actual === void 0 ? "" : ` with value ${val}`;
+        push("invalid-type", path, field, `expected ${expected} but received ${recv}${withValue}`);
         continue;
       }
       const rule = ruleForKeyword(e.keyword);

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@grexx/grexxlinter",
-  "version": "0.2.1318",
+  "version": "0.2.1372",
   "description": "Lint Grexx Studio casetype YAML against the studio JSON Schemas.",
   "type": "module",
   "bin": {

package/schemas/widget.schema.json

@@ -250,6 +250,280 @@
   "title": "widget",
   "type": "object",
   "allOf": [
+    {
+      "$comment": "Shape of the legacy options blob for grids — the only structure it has, since widget.options is declared `additionalProperties: true`. Closed at every level: an unknown key is an additional-field finding. The key set and the types are the union observed over 1275 grid widgets (def-yamls-142 plus three real projects); where the stored spelling and the studio's mapper disagree, both are declared, because both are live.",
+      "if": {
+        "properties": {
+          "widgetType": {
+            "enum": [
+              "grid",
+              "grid-v2"
+            ]
+          }
+        },
+        "required": [
+          "widgetType"
+        ]
+      },
+      "then": {
+        "properties": {
+          "options": {
+            "type": "object",
+            "additionalProperties": false,
+            "properties": {
+              "type": {
+                "type": "string"
+              },
+              "usesCases": {
+                "type": "boolean"
+              },
+              "dataset": {
+                "type": "string"
+              },
+              "extraclasses": {
+                "$comment": "Extra CSS classes on the widget wrapper. No editor writes it; it rides through from the legacy designer.",
+                "type": "string"
+              },
+              "grid": {
+                "type": "object",
+                "additionalProperties": false,
+                "properties": {
+                  "showHeader": {
+                    "type": "boolean"
+                  },
+                  "title": {
+                    "type": "string"
+                  },
+                  "showToolbar": {
+                    "type": "boolean"
+                  },
+                  "showSimpleSearch": {
+                    "type": "boolean"
+                  },
+                  "showAdvancedSearch": {
+                    "type": "boolean"
+                  },
+                  "showLinenumbers": {
+                    "type": "boolean"
+                  },
+                  "showColumnHeaders": {
+                    "type": "boolean"
+                  },
+                  "showColumnSelector": {
+                    "type": "boolean"
+                  },
+                  "showFooter": {
+                    "type": "boolean"
+                  },
+                  "noFullCount": {
+                    "type": "boolean"
+                  },
+                  "selectColumn": {
+                    "type": "boolean"
+                  },
+                  "autoHeight": {
+                    "type": "boolean"
+                  },
+                  "saveSearch": {
+                    "type": "string"
+                  },
+                  "orderColumn": {
+                    "type": "string"
+                  },
+                  "height": {
+                    "$comment": "Every stored value is a string (\"640\"); the studio's mapper writes a number on edit.",
+                    "type": [
+                      "string",
+                      "number"
+                    ]
+                  },
+                  "pagination": {
+                    "type": "object",
+                    "additionalProperties": false,
+                    "properties": {
+                      "rowsPerPage": {
+                        "type": [
+                          "number",
+                          "string"
+                        ]
+                      },
+                      "inifiteScroll": {
+                        "$comment": "Misspelled in the atalanta runtime that reads it — this is the wire format.",
+                        "type": "boolean"
+                      }
+                    }
+                  },
+                  "gridAggregationRowType": {
+                    "$comment": "Mirror of the first-class gridAggregationRowType; stored as an empty array on all but one of 475 occurrences, so both spellings are allowed.",
+                    "type": [
+                      "string",
+                      "array"
+                    ]
+                  },
+                  "gridAggregationRowPosition": {
+                    "$comment": "Mirror of the first-class gridAggregationRowPosition; stored as an empty array, as above.",
+                    "type": [
+                      "string",
+                      "array"
+                    ]
+                  }
+                }
+              },
+              "row": {
+                "type": "object",
+                "additionalProperties": false,
+                "properties": {
+                  "clickHandler": {
+                    "type": "string"
+                  },
+                  "clickHandlerColumnCase": {
+                    "type": "string"
+                  },
+                  "rowHeight": {
+                    "type": [
+                      "string",
+                      "number"
+                    ]
+                  },
+                  "viewToOpen": {
+                    "$comment": "Only read when clickHandler is viewHandler; the studio drops both keys when it is not.",
+                    "type": "string"
+                  },
+                  "customViewModalClasses": {
+                    "type": "string"
+                  }
+                }
+              },
+              "subgrid": {
+                "type": "object",
+                "additionalProperties": false,
+                "properties": {
+                  "enabled": {
+                    "type": "boolean"
+                  },
+                  "height": {
+                    "type": [
+                      "string",
+                      "number"
+                    ]
+                  }
+                }
+              },
+              "columns": {
+                "$comment": "Keyed by the dataset column's atalanta case id. On a case-converted grid the entry degrades to a caseId pointer at the gridColumn; on a grid that was never converted it keeps the full inline column settings instead, and those 2798 legacy entries are what the types below are read off. `size` stays untyped: it is a nested map whose own key set has no second source to check against.",
+                "type": "object",
+                "additionalProperties": {
+                  "type": "object",
+                  "additionalProperties": false,
+                  "properties": {
+                    "caseId": {
+                      "type": "string"
+                    },
+                    "name": {
+                      "type": "string"
+                    },
+                    "column": {
+                      "type": "string"
+                    },
+                    "title": {
+                      "type": "string"
+                    },
+                    "hidden": {
+                      "type": "boolean"
+                    },
+                    "hideable": {
+                      "type": "boolean"
+                    },
+                    "searchable": {
+                      "type": "boolean"
+                    },
+                    "sortable": {
+                      "type": "boolean"
+                    },
+                    "resizable": {
+                      "type": "boolean"
+                    },
+                    "wordwrap": {
+                      "type": "boolean"
+                    },
+                    "valueAsRowClass": {
+                      "type": "boolean"
+                    },
+                    "render": {
+                      "type": "string"
+                    },
+                    "order": {
+                      "type": [
+                        "integer",
+                        "string"
+                      ]
+                    },
+                    "clickHandler": {
+                      "type": "string"
+                    },
+                    "clickHandlerColumnCase": {
+                      "type": "string"
+                    },
+                    "aggregationOperation": {
+                      "$comment": "Written here by the studio's mapper, but absent from every stored legacy entry — no observed value to type it from, so the key is accepted untyped."
+                    },
+                    "size": {
+                      "type": "object"
+                    }
+                  }
+                }
+              },
+              "activities": {
+                "$comment": "Keyed by slot — addActivityId / editActivity / finishActivity / multiNN. Unlike columns these keep their full legacy field set alongside the caseId pointer, so every key here is observed and typed.",
+                "type": "object",
+                "additionalProperties": {
+                  "type": "object",
+                  "additionalProperties": false,
+                  "properties": {
+                    "caseId": {
+                      "type": "string"
+                    },
+                    "activity": {
+                      "type": "string"
+                    },
+                    "enabled": {
+                      "type": "boolean"
+                    },
+                    "title": {
+                      "type": "string"
+                    },
+                    "useActivityTitle": {
+                      "type": "boolean"
+                    },
+                    "icon": {
+                      "type": "string"
+                    },
+                    "type": {
+                      "type": "string"
+                    },
+                    "checkRights": {
+                      "type": "string"
+                    },
+                    "whenDoneRefreshType": {
+                      "type": "string"
+                    },
+                    "breaker": {
+                      "type": "boolean"
+                    },
+                    "order": {
+                      "type": [
+                        "integer",
+                        "string"
+                      ]
+                    }
+                  }
+                }
+              }
+            }
+          }
+        }
+      }
+    },
     {
       "if": {
         "properties": {