@codyswann/lisa 4.56.2 → 4.56.4

package/plugins/lisa-wiki-agy/plugin.json

@@ -1,6 +1,6 @@
 {
   "name": "lisa-wiki",
-  "version": "4.56.2",
+  "version": "4.56.4",
   "description": "LLM Wiki — a distributable, git-native markdown knowledge base for Claude Code and Codex",
   "author": {
     "name": "Cody Swann"

package/plugins/lisa-wiki-copilot/.claude-plugin/plugin.json

@@ -1,6 +1,6 @@
 {
   "name": "lisa-wiki",
-  "version": "4.56.2",
+  "version": "4.56.4",
   "description": "LLM Wiki — a distributable, git-native markdown knowledge base for Claude Code and Codex",
   "author": {
     "name": "Cody Swann"

package/plugins/lisa-wiki-cursor/.claude-plugin/plugin.json

@@ -1,6 +1,6 @@
 {
   "name": "lisa-wiki",
-  "version": "4.56.2",
+  "version": "4.56.4",
   "description": "LLM Wiki — a distributable, git-native markdown knowledge base for Claude Code and Codex",
   "author": {
     "name": "Cody Swann"

package/plugins/src/base/hooks/threshold-ratchet-compare.mjs

@@ -231,6 +231,47 @@ function compareAllowList(relPath, base, current) {
   return findings;
 }
 
+/**
+ * Compare Lighthouse assertions, including the shipped detail checker's default.
+ * Only lighthouserc-config.json's top-level forced-reflow setting is read by
+ * check-lighthouse-details.mjs. Other filenames and audits have no inferred
+ * baseline. The real checker/comparator boundary is covered by a parity test.
+ * @param {string} relPath Repo-relative path
+ * @param {unknown} base Parsed baseline config
+ * @param {unknown} current Parsed current config
+ * @returns {Finding[]} Removed or weakened assertions
+ */
+function compareLighthouse(relPath, base, current) {
+  const baseline = extractLighthouseAssertions(base);
+  const updated = extractLighthouseAssertions(current);
+  const findings = compareConstraints(relPath, baseline, updated);
+  const previous = base?.assertions?.forcedReflowInsight?.maxNumericValue;
+  const added = current?.assertions?.forcedReflowInsight?.maxNumericValue;
+  if (
+    relPath.split("/").at(-1) === "lighthouserc-config.json" &&
+    (previous == null ||
+      base?.ci?.assert?.assertions != null ||
+      current?.ci?.assert?.assertions != null) &&
+    (previous == null ||
+      (typeof previous === "number" && Number.isFinite(previous))) &&
+    typeof added === "number" &&
+    Number.isFinite(added)
+  ) {
+    // This checker reads top-level settings even beside canonical assertions.
+    // Keep its comparison separate so a new setting cannot replace a canonical
+    // bound with the same audit name and hide that bound's weakening.
+    const key = "forcedReflowInsight.maxNumericValue";
+    findings.push(
+      ...compareConstraints(
+        relPath,
+        new Map([[key, { value: previous ?? 100, direction: "max" }]]),
+        new Map([[key, { value: added, direction: "max" }]])
+      )
+    );
+  }
+  return findings;
+}
+
 /**
  * Compare one watched file's baseline and current contents and report every
  * weakening. Pure: no filesystem or git access.
@@ -301,11 +342,7 @@ export function compareFile(relPath, baselineText, currentText) {
     case "k6":
       return compareK6(relPath, base, current);
     case "lighthouse":
-      return compareConstraints(
-        relPath,
-        extractLighthouseAssertions(base),
-        extractLighthouseAssertions(current)
-      );
+      return compareLighthouse(relPath, base, current);
     case KIND_ALLOW_LIST:
       return compareAllowList(relPath, base, current);
     default:

package/rails/copy-overwrite/scripts/threshold-ratchet-compare.mjs

@@ -234,6 +234,47 @@ function compareAllowList(relPath, base, current) {
   return findings;
 }
 
+/**
+ * Compare Lighthouse assertions, including the shipped detail checker's default.
+ * Only lighthouserc-config.json's top-level forced-reflow setting is read by
+ * check-lighthouse-details.mjs. Other filenames and audits have no inferred
+ * baseline. The real checker/comparator boundary is covered by a parity test.
+ * @param {string} relPath Repo-relative path
+ * @param {unknown} base Parsed baseline config
+ * @param {unknown} current Parsed current config
+ * @returns {Finding[]} Removed or weakened assertions
+ */
+function compareLighthouse(relPath, base, current) {
+  const baseline = extractLighthouseAssertions(base);
+  const updated = extractLighthouseAssertions(current);
+  const findings = compareConstraints(relPath, baseline, updated);
+  const previous = base?.assertions?.forcedReflowInsight?.maxNumericValue;
+  const added = current?.assertions?.forcedReflowInsight?.maxNumericValue;
+  if (
+    relPath.split("/").at(-1) === "lighthouserc-config.json" &&
+    (previous == null ||
+      base?.ci?.assert?.assertions != null ||
+      current?.ci?.assert?.assertions != null) &&
+    (previous == null ||
+      (typeof previous === "number" && Number.isFinite(previous))) &&
+    typeof added === "number" &&
+    Number.isFinite(added)
+  ) {
+    // This checker reads top-level settings even beside canonical assertions.
+    // Keep its comparison separate so a new setting cannot replace a canonical
+    // bound with the same audit name and hide that bound's weakening.
+    const key = "forcedReflowInsight.maxNumericValue";
+    findings.push(
+      ...compareConstraints(
+        relPath,
+        new Map([[key, { value: previous ?? 100, direction: "max" }]]),
+        new Map([[key, { value: added, direction: "max" }]])
+      )
+    );
+  }
+  return findings;
+}
+
 /**
  * Compare one watched file's baseline and current contents and report every
  * weakening. Pure: no filesystem or git access.
@@ -304,11 +345,7 @@ export function compareFile(relPath, baselineText, currentText) {
     case "k6":
       return compareK6(relPath, base, current);
     case "lighthouse":
-      return compareConstraints(
-        relPath,
-        extractLighthouseAssertions(base),
-        extractLighthouseAssertions(current)
-      );
+      return compareLighthouse(relPath, base, current);
     case KIND_ALLOW_LIST:
       return compareAllowList(relPath, base, current);
     default:

package/typescript/copy-overwrite/scripts/lisa-mutation.mjs

@@ -1848,11 +1848,20 @@ export const parseChangedLineRanges = patch => {
  * @param {string} cwd - Project root.
  * @param {string} base - Merge-base sha.
  * @param {string} file - Repository-relative path.
+ * @param {string} [previousFile] - Old path when the file was renamed.
  * @returns {string[]} `path:start-end` entries for current changed lines.
  */
-export const selectChangedLineRanges = (cwd, base, file) =>
+export const selectChangedLineRanges = (cwd, base, file, previousFile = file) =>
   parseChangedLineRanges(
-    git(cwd, ["diff", "--unified=0", "--diff-filter=ACMR", base, "--", file])
+    git(cwd, [
+      "diff",
+      "--find-renames",
+      "--unified=0",
+      "--diff-filter=ACMR",
+      previousFile === file ? base : `${base}:${previousFile}`,
+      "--",
+      file,
+    ])
   ).map(({ start, end }) => `${file}:${start}-${end}`);
 
 /**
@@ -1870,16 +1879,35 @@ export const selectChangedTargets = (cwd, base, patterns) => {
   // a target whose only change is uncommitted is invisible to committed state,
   // so the gate printed "0 mutate targets" and exited 0 while Stryker, pointed
   // at the same tree, had a changed target in front of it.
-  const changed = git(cwd, ["diff", "--name-only", "--diff-filter=ACMRD", base])
-    .split("\n")
-    .map(file => file.trim())
-    .filter(Boolean);
-  const selectedFiles = changed
-    .filter(file => isMutateTarget(file, patterns))
-    .filter(file => fs.existsSync(path.join(cwd, file)));
-  const byFile = selectedFiles.map(file => ({
+  // Split full rewrites so an old name reused after a move remains a possible
+  // source. Include B as well as M: Git filters rewritten source records as B.
+  const names = git(cwd, [
+    "diff",
+    "--find-renames",
+    "--name-status",
+    "--break-rewrites",
+    "-z",
+    "--diff-filter=ACMRDB",
+    base,
+  ]).split("\0");
+  const changed = [];
+  for (let index = 0; names[index]; ) {
+    const status = names[index++];
+    const previousFile = names[index++];
+    const file = /^[RC]/u.test(status) ? names[index++] : previousFile;
+    changed.push({ file, previousFile });
+  }
+  const selectedFiles = changed.filter(({ file }) =>
+    isMutateTarget(file, patterns)
+  );
+  const byFile = selectedFiles.map(({ file, previousFile }) => ({
     file,
-    ranges: selectChangedLineRanges(cwd, base, file),
+    // Preserve deleted targets for classification, without asking Stryker to
+    // open them. A rename compares the old blob with the current destination;
+    // diffing both paths together can include a new file reusing the old name.
+    ranges: fs.existsSync(path.join(cwd, file))
+      ? selectChangedLineRanges(cwd, base, file, previousFile)
+      : [],
   }));
   return {
     changed: changed.length,
@@ -1890,7 +1918,7 @@ export const selectChangedTargets = (cwd, base, patterns) => {
       .map(entry => entry.file),
     // Reported from the diff, not from the selection, because by construction
     // these can never BE in the selection — that is the whole point of them.
-    uninstrumentable: uninstrumentableGuards(changed),
+    uninstrumentable: uninstrumentableGuards(changed.map(({ file }) => file)),
   };
 };
 

package/typescript/copy-overwrite/scripts/threshold-ratchet-compare.mjs

@@ -234,6 +234,47 @@ function compareAllowList(relPath, base, current) {
   return findings;
 }
 
+/**
+ * Compare Lighthouse assertions, including the shipped detail checker's default.
+ * Only lighthouserc-config.json's top-level forced-reflow setting is read by
+ * check-lighthouse-details.mjs. Other filenames and audits have no inferred
+ * baseline. The real checker/comparator boundary is covered by a parity test.
+ * @param {string} relPath Repo-relative path
+ * @param {unknown} base Parsed baseline config
+ * @param {unknown} current Parsed current config
+ * @returns {Finding[]} Removed or weakened assertions
+ */
+function compareLighthouse(relPath, base, current) {
+  const baseline = extractLighthouseAssertions(base);
+  const updated = extractLighthouseAssertions(current);
+  const findings = compareConstraints(relPath, baseline, updated);
+  const previous = base?.assertions?.forcedReflowInsight?.maxNumericValue;
+  const added = current?.assertions?.forcedReflowInsight?.maxNumericValue;
+  if (
+    relPath.split("/").at(-1) === "lighthouserc-config.json" &&
+    (previous == null ||
+      base?.ci?.assert?.assertions != null ||
+      current?.ci?.assert?.assertions != null) &&
+    (previous == null ||
+      (typeof previous === "number" && Number.isFinite(previous))) &&
+    typeof added === "number" &&
+    Number.isFinite(added)
+  ) {
+    // This checker reads top-level settings even beside canonical assertions.
+    // Keep its comparison separate so a new setting cannot replace a canonical
+    // bound with the same audit name and hide that bound's weakening.
+    const key = "forcedReflowInsight.maxNumericValue";
+    findings.push(
+      ...compareConstraints(
+        relPath,
+        new Map([[key, { value: previous ?? 100, direction: "max" }]]),
+        new Map([[key, { value: added, direction: "max" }]])
+      )
+    );
+  }
+  return findings;
+}
+
 /**
  * Compare one watched file's baseline and current contents and report every
  * weakening. Pure: no filesystem or git access.
@@ -304,11 +345,7 @@ export function compareFile(relPath, baselineText, currentText) {
     case "k6":
       return compareK6(relPath, base, current);
     case "lighthouse":
-      return compareConstraints(
-        relPath,
-        extractLighthouseAssertions(base),
-        extractLighthouseAssertions(current)
-      );
+      return compareLighthouse(relPath, base, current);
     case KIND_ALLOW_LIST:
       return compareAllowList(relPath, base, current);
     default: