@wairon/cli 5.1.1-dev.27 → 5.1.1-dev.29

package/dist/index.js

@@ -1113,7 +1113,7 @@ var init_defaults = __esm({
       copilot: ".github/prompts",
       codex: ".codex/agents"
     };
-    WAIRON_VERSION = "5.1.1-dev.27";
+    WAIRON_VERSION = "5.1.1-dev.29";
     GITHUB_REPO = "SYW-Apps/Waffle-AIron";
     ARCHITECT_AGENT_ID = "agent-architect";
     ARCHITECT_TEMPLATE_ID = "architect";
@@ -14560,13 +14560,21 @@ function groupsSharingCollaborators(calledBy) {
   }
   return [...groups.values()];
 }
+function isForwardingMethod(narrative) {
+  let handoffs = 0;
+  for (const step of narrative) {
+    if (step.type === "call" || step.type === "dispatch") handoffs++;
+    else if (step.type !== "return") return false;
+  }
+  return handoffs === 1;
+}
 var methodCohesionRule;
 var init_method_cohesion = __esm({
   "src/core/rules/heuristic/method-cohesion.ts"() {
     "use strict";
     methodCohesionRule = {
       name: "method-cohesion",
-      description: "An Orchestrator's methods share collaborators; when they split into groups that call no component in common, the component holds more than one responsibility. Reported only where two or more groups each hold two or more methods, so a single specialized method is not a finding.",
+      description: "An Orchestrator's methods share collaborators; when they split into groups that call no component in common, the component holds more than one responsibility. Reported only where two or more groups each hold two or more methods, so a single specialized method is not a finding \u2014 and never on a pure forwarder, whose every narrated method is a single hand-off and whose responsibility therefore lives in what it forwards to.",
       codes: [
         { code: "INCOHESIVE_METHODS", defaultSeverity: "warning", summary: "Orchestrator's methods form two or more groups of two or more that share no called component" }
       ],
@@ -14576,6 +14584,8 @@ var init_method_cohesion = __esm({
           const intf = ctx.interfaceMap.get(impl.contract);
           const comp = intf ? ctx.componentMap.get(intf.component) : void 0;
           if (!comp || comp.componentType !== "Orchestrator") continue;
+          const narrated = impl.methods.filter((m) => (m.narrative ?? []).length > 0);
+          if (narrated.length > 0 && narrated.every((m) => isForwardingMethod(m.narrative ?? []))) continue;
           const calledBy = /* @__PURE__ */ new Map();
           for (const method of impl.methods) {
             const reached = /* @__PURE__ */ new Set();
@@ -14597,7 +14607,7 @@ var init_method_cohesion = __esm({
           ctx.addIssue(
             "warning",
             "INCOHESIVE_METHODS",
-            `Orchestrator "${component.id}" holds ${groups.length} groups of methods that share no called component: ${described}. Split it along those groups, or acknowledge a deliberate facade with a reasoned lint.allow for INCOHESIVE_METHODS.`,
+            `Orchestrator "${component.id}" holds ${groups.length} groups of methods that share no called component: ${described}. Split it along those groups, move the logic out until every method is a single hand-off (a pure forwarder is exempt), or acknowledge a deliberate facade with a reasoned lint.allow for INCOHESIVE_METHODS.`,
             component.id,
             ctx.isComponentDraft(component.id)
           );