@mutmutco/cli 4.5.15 → 4.5.17

package/dist/main.cjs

@@ -7401,6 +7401,18 @@ function surfaceInferenceSuffix(inferred) {
   const alts = inferred.alternatives?.length ? ` Other candidates: ${inferred.alternatives.join(", ")}.` : "";
   return ` This pick is low-confidence \u2014 pass --surface <value> to choose explicitly.${alts}`;
 }
+function uncertainSurfaceRefusal(inferred, command = "issue create") {
+  if (!inferred.uncertain) return void 0;
+  const alternatives = inferred.alternatives?.length ? ` Other candidates: ${inferred.alternatives.join(", ")}.` : "";
+  return {
+    message: `${command}: surface inference is uncertain (${inferred.reason}) \u2014 pass --surface <value> to choose explicitly or --no-surface only when the filing is genuinely exempt.${alternatives}`,
+    payload: {
+      code: ERROR_CODES.ERR_MISSING_FLAG,
+      offending_flag: "--surface",
+      expected: ["--surface <value>", "--no-surface for a genuinely exempt filing"]
+    }
+  };
+}
 function surfaceInferenceNote(inferred) {
   const base = `mmi-cli: no --surface given \u2014 filed under ${inferred.label} (${inferred.reason}).`;
   return inferred.uncertain ? `${base}${surfaceInferenceSuffix(inferred)}
@@ -7438,7 +7450,9 @@ async function checkSurfaceRequirement(input, deps = {}) {
   }
   if (known.length === 0) return { enforcing: false, taxonomyAbsent: true };
   if (labelsCarrySurface(input.labels)) return { enforcing: true };
-  return { enforcing: true, inferred: inferSurface(known, { title: input.title, type: input.type, body: input.body }) };
+  const inferred = inferSurface(known, { title: input.title, type: input.type, body: input.body });
+  const refusal = uncertainSurfaceRefusal(inferred, input.command ?? "issue create");
+  return refusal ? { enforcing: true, refusal } : { enforcing: true, inferred };
 }
 function conflictingSurfaceInputs(surfaceFlag, labels) {
   if (!surfaceFlag) return void 0;
@@ -11076,6 +11090,7 @@ var ITEM_FIELD_VALUES_PAGE = 25;
 var ITEM_LABELS_PAGE = 25;
 var ITEM_ASSIGNEES_PAGE = 10;
 var ITEM_PROJECT_ITEMS_PAGE = 50;
+var LINKED_REPOS_PAGE = 20;
 var CONTENT_LABELS_AND_ASSIGNEES = `
               labels(first: ${ITEM_LABELS_PAGE}) { pageInfo { hasNextPage } nodes { name } }
               assignees(first: ${ITEM_ASSIGNEES_PAGE}) { pageInfo { hasNextPage } nodes { login } }`;
@@ -11086,6 +11101,7 @@ query($owner: String!, $number: Int!, $after: String, $query: String) {
     projectV2(number: $number) {
       id
       title
+      repositories(first: ${LINKED_REPOS_PAGE}) { pageInfo { hasNextPage } nodes { nameWithOwner } }
       items(first: 100, after: $after, query: $query) {
         pageInfo { hasNextPage endCursor }
         nodes {
@@ -11126,6 +11142,44 @@ query($owner: String!, $number: Int!, $after: String, $query: String) {
     }
   }
 }`;
+var INDEX_LAG_RECOVERY_PAGE = 50;
+var INDEX_LAG_RECOVERY_ITEMS_PAGE = 10;
+var INDEX_LAG_RECOVERY_MAX_PAGES = 10;
+var indexLagRecoveryQuery = (connection) => `
+query($repoOwner: String!, $repoName: String!, $after: String) {
+  repository(owner: $repoOwner, name: $repoName) {
+    ${connection}(first: ${INDEX_LAG_RECOVERY_PAGE}, after: $after, states: OPEN) {
+      pageInfo { hasNextPage endCursor }
+      nodes {
+        __typename
+        id
+        number
+        title
+        url
+        state
+        updatedAt
+        repository { nameWithOwner }${CONTENT_LABELS_AND_ASSIGNEES}
+        projectItems(first: ${INDEX_LAG_RECOVERY_ITEMS_PAGE}) {
+          pageInfo { hasNextPage }
+          nodes {
+            id
+            project { id }
+            fieldValues(first: ${ITEM_FIELD_VALUES_PAGE}) {
+              pageInfo { hasNextPage }
+              nodes {
+                ... on ProjectV2ItemFieldSingleSelectValue {
+                  name
+                  optionId
+                  field { ... on ProjectV2SingleSelectField { id name } }
+                }
+              }
+            }
+          }
+        }
+      }
+    }
+  }
+}`;
 var ACTIVE_BOARD_ITEMS_FILTER = "-status:Done -is:closed";
 var DOCTOR_BOARD_ITEMS_FILTER = "-status:Done";
 var ARCHIVE_BOARD_ITEMS_FILTER = "status:Done is:closed";
@@ -11349,6 +11403,52 @@ function shouldSkipInactiveBoardNode(node, cfg) {
   const status = parseSingleSelectFields(node.fieldValues?.nodes, cfg).status?.name;
   return status === "Done";
 }
+async function recoverIndexLaggedItems(graphql, projectId, linkedRepos, listed, isInactive, warnings) {
+  const known = new Set(listed.map((node) => node.id));
+  const nodes = [];
+  let partial = false;
+  if (linkedRepos?.pageInfo?.hasNextPage) {
+    warnings.push(`partial board read: index-lag recovery covers only the first ${LINKED_REPOS_PAGE} linked repos`);
+    partial = true;
+  }
+  for (const repo of (linkedRepos?.nodes ?? []).map((node) => node.nameWithOwner).filter((name) => Boolean(name))) {
+    const [repoOwner, repoName2] = repo.split("/");
+    let recoveredHere = 0;
+    for (const connection of ["issues", "pullRequests"]) {
+      let after;
+      let pages = 0;
+      try {
+        do {
+          const page = (await graphql(indexLagRecoveryQuery(connection), { repoOwner, repoName: repoName2, ...after ? { after } : {} })).repository?.[connection];
+          for (const content of page?.nodes ?? []) {
+            const { projectItems, ...contentFields } = content;
+            const item = (projectItems?.nodes ?? []).find((entry) => entry.project?.id === projectId);
+            if (!item?.id || known.has(item.id)) continue;
+            const node = { id: item.id, fieldValues: item.fieldValues, content: contentFields };
+            if (isInactive(node)) continue;
+            known.add(item.id);
+            nodes.push(node);
+            recoveredHere += 1;
+          }
+          after = page?.pageInfo?.hasNextPage ? page.pageInfo.endCursor ?? void 0 : void 0;
+          pages += 1;
+          if (after && pages >= INDEX_LAG_RECOVERY_MAX_PAGES) {
+            warnings.push(`partial board read: index-lag recovery for ${repo} ${connection} capped at ${pages * INDEX_LAG_RECOVERY_PAGE} open items`);
+            partial = true;
+            after = void 0;
+          }
+        } while (after);
+      } catch (e) {
+        warnings.push(`partial board read: index-lag recovery for ${repo} ${connection} failed (${e.message})`);
+        partial = true;
+      }
+    }
+    if (recoveredHere) {
+      warnings.push(`board read: ${recoveredHere} active item(s) from ${repo} were missing from GitHub's project items listing (Projects index lag) \u2014 recovered from their own project membership`);
+    }
+  }
+  return { nodes, partial };
+}
 async function collectBoardItems(cfg, options, deps) {
   const client = deps.client ?? defaultGitHubClient();
   const currentRepo = await resolveCurrentRepo(options, deps);
@@ -11361,6 +11461,7 @@ async function collectBoardItems(cfg, options, deps) {
   let projectId = cfg.projectId;
   let partial = false;
   let pageCount = 0;
+  let linkedRepos;
   do {
     try {
       const page = await fetchProjectPage(client, cfg, after, itemsQuery);
@@ -11369,6 +11470,7 @@ async function collectBoardItems(cfg, options, deps) {
       if (!project2) throw new Error(`project ${cfg.projectOwner}#${cfg.projectNumber} not found`);
       projectId = project2.id;
       projectTitle ||= project2.title;
+      linkedRepos ??= project2.repositories;
       pageCount += 1;
       const pageNodes = project2.items.nodes ?? [];
       if (options.activeOnly) {
@@ -11410,6 +11512,18 @@ async function collectBoardItems(cfg, options, deps) {
       partial = true;
     }
   }
+  if (options.activeOnly) {
+    const recovered = await recoverIndexLaggedItems(
+      (query, variables) => client.graphql(query, variables),
+      projectId,
+      linkedRepos,
+      nodes,
+      (node) => shouldSkipInactiveBoardNode(node, cfg),
+      warnings
+    );
+    nodes.push(...recovered.nodes);
+    partial ||= recovered.partial;
+  }
   const read = nodesToItems(nodes, warnings, cfg);
   if (read.failures.length) {
     const message2 = `partial board read: ${read.failures.length} item(s) came back truncated and were not read \u2014 ${read.failures.join(" | ")}`;
@@ -16001,10 +16115,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.5.15",
-    tag: "v4.5.15",
-    commit: "fa2cf924d900",
-    npm: "@mutmutco/cli@4.5.15"
+    version: "4.5.17",
+    tag: "v4.5.17",
+    commit: "be54d6077479",
+    npm: "@mutmutco/cli@4.5.17"
   },
   exitCriterion: "fleet-n-of-n",
   hubOnlyShortcut: "forbidden",
@@ -16021,14 +16135,14 @@ var rollout_plan_default = {
           repo: "mutmutco/mmi-hub",
           role: "canary",
           schedule: "train",
-          v3Target: "v4.5.15"
+          v3Target: "v4.5.17"
         }
       ],
       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.5.15 and redeploy the Hub Lambda from tag v4.5.15 (fa2cf924d900); installed clients repair via `mmi-cli doctor`. No other cohort is touched.",
-        v3Target: "v4.5.15 (@mutmutco/cli@4.5.15, tag commit fa2cf924d900 \u2014 last known-good release carrying the repo-index v4-only contract)"
+        mechanism: "npm dist-tag latest -> 4.5.17 and redeploy the Hub Lambda from tag v4.5.17 (be54d6077479); installed clients repair via `mmi-cli doctor`. No other cohort is touched.",
+        v3Target: "v4.5.17 (@mutmutco/cli@4.5.17, tag commit be54d6077479 \u2014 last known-good release carrying the repo-index v4-only contract)"
       }
     },
     {
@@ -35691,10 +35805,12 @@ function validateBatchSpecs(specs) {
   }
   return { ok: errors.length === 0, errors, validated };
 }
-async function preflightBatchSurfaces(validated, rowRepo, options) {
+async function preflightBatchSurfaces(validated, rowRepo, options, deps = {}) {
+  const surfaceApplies = deps.surfaceLabelApplies ?? surfaceLabelApplies;
+  const readLabels = deps.readRepoSurfaceLabels ?? readRepoSurfaceLabels;
   const applies = /* @__PURE__ */ new Map();
   const appliesTo = async (repo) => {
-    if (!applies.has(repo)) applies.set(repo, await surfaceLabelApplies(repo));
+    if (!applies.has(repo)) applies.set(repo, await surfaceApplies(repo));
     return applies.get(repo);
   };
   if (options.surface) {
@@ -35709,7 +35825,7 @@ async function preflightBatchSurfaces(validated, rowRepo, options) {
   const censusFor = (repo) => {
     let pending = censusCache.get(repo);
     if (!pending) {
-      pending = readRepoSurfaceLabels(repo);
+      pending = readLabels(repo);
       censusCache.set(repo, pending);
     }
     return pending;
@@ -35727,9 +35843,14 @@ async function preflightBatchSurfaces(validated, rowRepo, options) {
     }
     if (known.length === 0) continue;
     const inferred = inferSurface(known, { title: spec.title, type: spec.type, body: spec.body });
+    const refusal = uncertainSurfaceRefusal(inferred, `issue create --batch row ${row}`);
+    if (refusal) {
+      errors.push({ row, error: refusal.message });
+      continue;
+    }
     spec.labels = [...spec.labels ?? [], inferred.label];
     process.stderr.write(
-      `mmi-cli: no --surface given \u2014 row ${row} filed under ${inferred.label} (${inferred.reason}). Pass --surface to choose.${surfaceInferenceSuffix(inferred)}
+      `mmi-cli: no --surface given \u2014 row ${row} filed under ${inferred.label} (${inferred.reason}). Pass --surface to choose.
 `
     );
   }
@@ -42229,7 +42350,7 @@ function registerCollaborationCommands(program3) {
   }
   const issue = program3.command("issue").description("issues \u2014 create and view with structured JSON (view; show, get and read alias view for board-verb/gh-shaped callers). Claims are board mutations: use `oracle board claim`, never `oracle issue claim` or the flat `board claim`");
   withExamples(mutating(
-    issue.command("create").description("create an issue (type \u2014 label) and print {number,url,label} JSON").addOption(new Option("--type <type>", "bug | feature | task (sets the matching label; required unless --batch)").choices([...ISSUE_TYPES])).option("--title <title>", "issue title").option("--title-file <path|->", "read the issue title from a UTF-8 file, or from stdin with -").option("--body <body>", "issue body (markdown)").option("--body-file <path|->", "read issue body from a UTF-8 file. `-` (stdin) needs a heredoc, which the agent inline-body guard denies (#1473/#2125) \u2014 prefer a real path; a title with backticks needs --title-file for the same reason (#3381)").option("--priority <priority>", "urgent | high | medium | low (defaults to medium; sets the board Priority field only \u2014 never a priority:* label, #416)").option("--repo <owner/repo>", "target repo (defaults to the current repo)").option("--label <label...>", "extra label(s) to attach (repeatable; auto-created if missing)").option("--surface <surface>", "issue surface, with or without the surface: prefix (#3789). Any value satisfies the one-surface-label board rule, so this is not a closed enum. Omit it on an enforcing repo and one is inferred from the title/body (#1164) \u2014 the pick is printed and included in --json as surface_inferred/surface/surface_reason").option("--no-surface", "file without a surface label on a repo that requires one \u2014 for a genuinely exempt filing (e.g. a coop proof issue that spans every surface). Suppresses inference too").option("--parent <ref>", "file as a native sub-issue of this parent (#123, owner/repo#123, or URL)").option("--no-related", "skip the auto related-issues comment"),
+    issue.command("create").description("create an issue (type \u2014 label) and print {number,url,label} JSON").addOption(new Option("--type <type>", "bug | feature | task (sets the matching label; required unless --batch)").choices([...ISSUE_TYPES])).option("--title <title>", "issue title").option("--title-file <path|->", "read the issue title from a UTF-8 file, or from stdin with -").option("--body <body>", "issue body (markdown)").option("--body-file <path|->", "read issue body from a UTF-8 file. `-` (stdin) needs a heredoc, which the agent inline-body guard denies (#1473/#2125) \u2014 create tmp/issue.md with a file-write tool (create tmp/ first if needed), then pass --body-file tmp/issue.md; a title with backticks needs --title-file for the same reason (#3381)").option("--priority <priority>", "urgent | high | medium | low (defaults to medium; sets the board Priority field only \u2014 never a priority:* label, #416)").option("--repo <owner/repo>", "target repo (defaults to the current repo)").option("--label <label...>", "extra label(s) to attach (repeatable; auto-created if missing)").option("--surface <surface>", "issue surface, with or without the surface: prefix (#3789). Any value satisfies the one-surface-label board rule, so this is not a closed enum. Omit it on an enforcing repo and one is inferred from the title/body (#1164) \u2014 the pick is printed and included in --json as surface_inferred/surface/surface_reason").option("--no-surface", "file without a surface label on a repo that requires one \u2014 for a genuinely exempt filing (e.g. a coop proof issue that spans every surface). Suppresses inference too").option("--parent <ref>", "file as a native sub-issue of this parent (#123, owner/repo#123, or URL)").option("--no-related", "skip the auto related-issues comment"),
     // --dry-run/--validate-only plan: resolve the same title source and validate the same type, priority,
     // and surface contract as the real action. A plan that echoes the title-file PATH instead of its value
     // is not a plan of the mutation that will run (#3914).

package/package.json

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