@grexx/grexxlinter 0.2.2764 → 0.2.2770

package/cli.js

@@ -15659,6 +15659,23 @@ function pushUncovered(push, coverage, casetype) {
   const n = coverage.uncoveredCount(casetype);
   if (n > 0) push("unknown-casetype", "", casetype, `${n} rule clause(s) for "${casetype}" are behaviour-only and not checked by the schema`);
 }
+function isInactive(node) {
+  return !!node && typeof node === "object" && node.inactive === true;
+}
+function underInactive(root, pointer) {
+  if (isInactive(root)) return true;
+  let cur = root;
+  for (const raw of pointer.split("/").slice(1)) {
+    if (cur == null || typeof cur !== "object") return false;
+    cur = cur[raw.replaceAll("~1", "/").replaceAll("~0", "~")];
+    if (isInactive(cur)) return true;
+  }
+  return false;
+}
+function dropInactive(messages, root) {
+  if (isInactive(root)) return [];
+  return messages.filter((m) => !underInactive(root, m.path));
+}
 function dedupeMessages(messages) {
   const seen = /* @__PURE__ */ new Set();
   return messages.filter((m) => {
@@ -15690,7 +15707,7 @@ function lintDocument(ctx, input) {
     });
   }
   if (settings.reportUncovered) pushUncovered(push, ctx.coverage, casetype);
-  const deduped = dedupeMessages(messages);
+  const deduped = dropInactive(dedupeMessages(messages), input.value);
   const ok = !deduped.some((m) => meetsThreshold(m.severity, settings.failOn));
   return { file: input.file, casetype, messages: deduped, skipped: false, ok };
 }
@@ -15769,7 +15786,7 @@ function lintFiles(files, ctx, settings) {
 // src/cli.ts
 var PARALLEL_THRESHOLD = 200;
 var MAX_WORKERS = 8;
-var VERSION = "0.2.2764";
+var VERSION = "0.2.2770";
 function isNewer(latest, current) {
   const a = latest.split(".").map(Number);
   const b = current.split(".").map(Number);

package/lsp.js

@@ -25608,6 +25608,23 @@ function pushUncovered(push, coverage, casetype) {
   const n = coverage.uncoveredCount(casetype);
   if (n > 0) push("unknown-casetype", "", casetype, `${n} rule clause(s) for "${casetype}" are behaviour-only and not checked by the schema`);
 }
+function isInactive(node) {
+  return !!node && typeof node === "object" && node.inactive === true;
+}
+function underInactive(root, pointer) {
+  if (isInactive(root)) return true;
+  let cur = root;
+  for (const raw of pointer.split("/").slice(1)) {
+    if (cur == null || typeof cur !== "object") return false;
+    cur = cur[raw.replaceAll("~1", "/").replaceAll("~0", "~")];
+    if (isInactive(cur)) return true;
+  }
+  return false;
+}
+function dropInactive(messages, root) {
+  if (isInactive(root)) return [];
+  return messages.filter((m) => !underInactive(root, m.path));
+}
 function dedupeMessages(messages) {
   const seen = /* @__PURE__ */ new Set();
   return messages.filter((m) => {
@@ -25639,7 +25656,7 @@ function lintDocument(ctx2, input) {
     });
   }
   if (settings2.reportUncovered) pushUncovered(push, ctx2.coverage, casetype);
-  const deduped = dedupeMessages(messages);
+  const deduped = dropInactive(dedupeMessages(messages), input.value);
   const ok = !deduped.some((m) => meetsThreshold(m.severity, settings2.failOn));
   return { file: input.file, casetype, messages: deduped, skipped: false, ok };
 }

package/package.json

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

package/schemas/mapping.schema.json

@@ -240,6 +240,21 @@
         }
       }
     },
+    {
+      "$comment": "A mapping always names a destination. Which field carries it depends on toAttributeType (usersession -> toKey, everything else -> toAttribute), and the generated branches already require toKey for usersession and activityAttribute for taskAttribute -- but nothing requires the ordinary toAttribute, so a mapping with no destination at all linted clean. Zero findings across packages/schemas/def-yamls-142 and apps/backend/test/fixtures/compile-6685 (1837 active mappings).",
+      "anyOf": [
+        {
+          "required": [
+            "toAttribute"
+          ]
+        },
+        {
+          "required": [
+            "toKey"
+          ]
+        }
+      ]
+    },
     {
       "if": {
         "properties": {

package/schemas/trigger.schema.json

@@ -120,7 +120,23 @@
   ],
   "title": "trigger",
   "type": "object",
+  "$comment": "Hand-maintained overlay (deep-merged by scripts/extend.mjs into extended/trigger.schema.json). A trigger always names an activity to run, whatever its `toTriggerType` -- see the allOf below. The manager exports this per toTriggerType value, but the `page` group's `when` is \"[To page] is not «empty» AND [To type] is \\\"page\\\"\", and condToSchema cannot express the not-empty conjunct (`!==` is outside EQ_OPS and `''` is in BAD_VALUES), so that one group is deferred and never reaches the schema. Stating it unconditionally covers `page`, and additionally covers a trigger with no `toTriggerType` at all -- which the per-value groups all miss, since each `if` requires the field to be present. Conditionals generated by packages/schemas/scripts/extend.mjs from rules/trigger.json.",
   "allOf": [
+    {
+      "$comment": "Zero findings across packages/schemas/def-yamls-142 and apps/backend/test/fixtures/compile-6685 (659 active triggers).",
+      "anyOf": [
+        {
+          "required": [
+            "activityTo"
+          ]
+        },
+        {
+          "required": [
+            "activityTag"
+          ]
+        }
+      ]
+    },
     {
       "if": {
         "properties": {
@@ -453,6 +469,5 @@
         ]
       }
     }
-  ],
-  "$comment": "Conditionals generated by packages/schemas/scripts/extend.mjs from rules/trigger.json."
+  ]
 }