@mutmutco/cli 4.2.3 → 4.2.5

package/dist/main.cjs

@@ -11802,10 +11802,10 @@ var rollout_plan_default = {
     note: "The v4.0.0 stamp happens at cut time (D6e #4463); until then the candidate is the origin/development head artifacts (built cli/dist + npm pack), identity proven by dist content hash (D6a)."
   },
   baseline: {
-    version: "4.2.3",
-    tag: "v4.2.3",
-    commit: "a54078f2ac92",
-    npm: "@mutmutco/cli@4.2.3"
+    version: "4.2.5",
+    tag: "v4.2.5",
+    commit: "258de06d0766",
+    npm: "@mutmutco/cli@4.2.5"
   },
   exitCriterion: "fleet-n-of-n",
   hubOnlyShortcut: "forbidden",
@@ -11822,14 +11822,14 @@ var rollout_plan_default = {
           repo: "mutmutco/mmi-hub",
           role: "canary",
           schedule: "train",
-          v3Target: "v4.2.3"
+          v3Target: "v4.2.5"
         }
       ],
       rollbackTrigger: "Any red inside the post-contract soak window: `devops train gate` FAIL attributable to the v4 doors, Hub endpoint health probe failure, a pre-v4 client admitted instead of receiving actionable HTTP 426, or npm consumer install/doctor failure on the v4-only dist.",
       rollback: {
         independent: true,
-        mechanism: "npm dist-tag latest -> 4.2.3 and redeploy the Hub Lambda from tag v4.2.3 (a54078f2ac92); installed clients repair via `mmi-cli doctor`. No other cohort is touched.",
-        v3Target: "v4.2.3 (@mutmutco/cli@4.2.3, tag commit a54078f2ac92 \u2014 last known-good release carrying the repo-index v4-only contract)"
+        mechanism: "npm dist-tag latest -> 4.2.5 and redeploy the Hub Lambda from tag v4.2.5 (258de06d0766); installed clients repair via `mmi-cli doctor`. No other cohort is touched.",
+        v3Target: "v4.2.5 (@mutmutco/cli@4.2.5, tag commit 258de06d0766 \u2014 last known-good release carrying the repo-index v4-only contract)"
       }
     },
     {
@@ -25163,6 +25163,16 @@ var WAIVABLE_KINDS = [
   "stale-satisfied-by"
 ];
 var TEST_WORK_KIND = "unrequested-test-file";
+var TEST_FILE_RE = /\.(?:test|spec)\.[cm]?[jt]sx?$/;
+var PY_TEST_FILE_RE = /(?:^|\/)test_[^/]*\.py$|_test\.py$/;
+function isTestPath(path2) {
+  return typeof path2 === "string" && (TEST_FILE_RE.test(path2) || PY_TEST_FILE_RE.test(path2));
+}
+function editsExistingTest(paths, addedPaths = []) {
+  const added = new Set(Array.isArray(addedPaths) ? addedPaths : []);
+  const list = Array.isArray(paths) ? paths : [];
+  return list.some((path2) => isTestPath(path2) && !added.has(path2));
+}
 function translateGlob(glob) {
   let out = "";
   for (let i = 0; i < glob.length; i += 1) {
@@ -25205,7 +25215,7 @@ function matchedMandatoryGlobs(paths, mandatory) {
     return list.some((path2) => re.test(path2));
   });
 }
-function evaluateTestCommandPolicy({ paths, mandatory, regulated = true, override = null } = {}) {
+function evaluateTestCommandPolicy({ paths, mandatory, regulated = true, override = null, addedPaths = [] } = {}) {
   const configuredMandatoryCount = mandatoryGlobList(mandatory).length;
   if (!regulated) {
     return {
@@ -25213,18 +25223,23 @@ function evaluateTestCommandPolicy({ paths, mandatory, regulated = true, overrid
       matchedMandatoryGlobs: [],
       matchedMandatoryCount: 0,
       testCommandsAllowed: true,
+      editsExistingTest: false,
       reasonId: null,
       commandClasses: { allowed: [TEST_COMMAND_CLASS], refused: [] }
     };
   }
   const matched = matchedMandatoryGlobs(paths, mandatory);
   const overrideAuthorizes = Array.isArray(override?.kinds) && override.kinds.includes(TEST_WORK_KIND);
-  const testCommandsAllowed = matched.length > 0 || overrideAuthorizes;
+  const existingTestEdited = editsExistingTest(paths, addedPaths);
+  const testCommandsAllowed = matched.length > 0 || overrideAuthorizes || existingTestEdited;
   return {
     configuredMandatoryCount,
     matchedMandatoryGlobs: matched,
     matchedMandatoryCount: matched.length,
     testCommandsAllowed,
+    /** #5842: true when the diff edits a test file that already existed — one of the facts that can
+     *  authorize the class, named so a refusal can say what was missing. */
+    editsExistingTest: existingTestEdited,
     reasonId: testCommandsAllowed ? null : "test-command-outside-mandatory-zone",
     commandClasses: {
       allowed: testCommandsAllowed ? [TEST_COMMAND_CLASS] : [],
@@ -25302,8 +25317,8 @@ function readOverride(base, cwd) {
 
 // src/test-policy-core.ts
 var POLICY_FILE = "test-policy.json";
-var TEST_RE = /\.(?:test|spec)\.[cm]?[jt]sx?$/;
-var PY_TEST_RE = /(?:^|\/)test_[^/]*\.py$|_test\.py$/;
+var TEST_RE = TEST_FILE_RE;
+var PY_TEST_RE = PY_TEST_FILE_RE;
 function translate(glob) {
   let out = "";
   for (let i = 0; i < glob.length; i++) {
@@ -25338,7 +25353,7 @@ function translate(glob) {
 function globToRegExp2(glob) {
   return new RegExp(`^${translate(glob)}$`);
 }
-function isTestPath(path2) {
+function isTestPath2(path2) {
   return TEST_RE.test(path2) || PY_TEST_RE.test(path2);
 }
 var SUPPORTED_SOURCE = /\.[cm]?[jt]s$/;
@@ -25572,7 +25587,7 @@ function annotateChangeMeaning(changed, policy, read) {
   const matchers = (policy.mandatory ?? []).map((m) => globToRegExp2(m.glob));
   return changed.map((file) => {
     if (file.status !== "M" || !SUPPORTED_SOURCE.test(file.path)) return file;
-    if (!matchers.some((re) => re.test(file.path)) && !isTestPath(file.path)) return file;
+    if (!matchers.some((re) => re.test(file.path)) && !isTestPath2(file.path)) return file;
     let pair;
     try {
       pair = read(file.path);
@@ -25609,7 +25624,7 @@ function removedPaths(changed) {
 function classify(changed, policy, present = () => false) {
   const matchers = (policy.mandatory ?? []).map((m) => ({ ...m, re: globToRegExp2(m.glob) }));
   const mandatoryHits = changed.filter((f) => matchers.some((m) => m.re.test(f.path)));
-  const testChanges = changed.filter((f) => isTestPath(f.path));
+  const testChanges = changed.filter((f) => isTestPath2(f.path));
   const addedTests = testChanges.filter((f) => f.status === "A");
   const removed = removedPaths(changed);
   const discharged = (m) => (m.satisfiedBy?.length ?? 0) > 0 && m.satisfiedBy.every((p) => present(p) && !removed.has(p));
@@ -25798,6 +25813,9 @@ function runTestPolicy(root, deps = {}) {
   const findings = blocking.length > 0 ? blocking : sift(evaluate(changed, policy, present));
   const commandPolicy = evaluateTestCommandPolicy({
     paths: changed.map((f) => f.path),
+    // #5842: `A` is added, `R` is a rename whose new name did not exist at the base either. Both are
+    // test work this diff CREATED, which only a waiver may authorize (#5804).
+    addedPaths: changed.filter((f) => f.status === "A" || f.status === "R").map((f) => f.path),
     mandatory: policy.mandatory,
     regulated: policy.declared !== false,
     override: override && lookup.refusals.length === 0 ? { kinds: override.kinds } : null

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@mutmutco/cli",
-  "version": "4.2.3",
+  "version": "4.2.5",
   "description": "MMI Future CLI — the org dev toolbox and shared cross-IDE engine for every registry-declared MMI coding surface.",
   "type": "module",
   "license": "UNLICENSED",