@mrciphersmith/keryx 0.3.58 → 0.3.61

package/dist/core.js

@@ -9801,7 +9801,7 @@ var init_skill_length_ceilings = __esm(() => {
     "orchestration/context-collector": 671,
     "orchestration/feature-analyzer": 447,
     "orchestration/feature-dev": 177,
-    "orchestration/flow-orchestrator": 696,
+    "orchestration/flow-orchestrator": 694,
     "orchestration/issue-analyzer": 373,
     "orchestration/job-documenter": 414,
     "orchestration/job-orchestrator": 2243,
@@ -24100,6 +24100,8 @@ __export(exports_service2, {
   AC_KINDS: () => AC_KINDS,
   CONFIRMATION_CAVEAT: () => CONFIRMATION_CAVEAT,
   DEFAULT_OUTCOME_AUTHOR: () => DEFAULT_OUTCOME_AUTHOR,
+  ORIGIN_KINDS: () => ORIGIN_KINDS,
+  ORIGIN_READINGS: () => ORIGIN_READINGS,
   OUTCOME_AUTHORS: () => OUTCOME_AUTHORS,
   OUTCOME_AUTHOR_READINGS: () => OUTCOME_AUTHOR_READINGS,
   OUTCOME_HINT: () => OUTCOME_HINT,
@@ -24119,6 +24121,7 @@ __export(exports_service2, {
   confirmPreconditionError: () => confirmPreconditionError,
   createFlowService: () => createFlowService,
   describeAcKind: () => describeAcKind,
+  effectiveOutcomeAuthor: () => effectiveOutcomeAuthor,
   evaluatedVerdict: () => evaluatedVerdict,
   extractCriterionTokens: () => extractCriterionTokens,
   factsOnlyVerdict: () => factsOnlyVerdict,
@@ -24128,7 +24131,10 @@ __export(exports_service2, {
   hashDiff: () => hashDiff,
   intentNoteForNewFlow: () => intentNoteForNewFlow,
   isFrozen: () => isFrozen,
+  isOriginKind: () => isOriginKind,
   notCheckableVerdict: () => notCheckableVerdict,
+  originDetailLines: () => originDetailLines,
+  originSummary: () => originSummary,
   outcomeBulletsFrom: () => outcomeBulletsFrom,
   parseAcceptanceCriteria: () => parseAcceptanceCriteria,
   parseOutcomeAuthor: () => parseOutcomeAuthor,
@@ -24137,12 +24143,15 @@ __export(exports_service2, {
   readAcCheckEnabled: () => readAcCheckEnabled,
   readAcKindRecords: () => readAcKindRecords,
   readFlow: () => readFlow,
+  readOrigin: () => readOrigin,
+  readOriginKind: () => readOriginKind,
   readOutcomeAuthor: () => readOutcomeAuthor,
   renderAcCheckAdvisoryNotice: () => renderAcCheckAdvisoryNotice,
   renderAcCheckReport: () => renderAcCheckReport,
   renderAcKindDistribution: () => renderAcKindDistribution,
   reportFromRecords: () => reportFromRecords,
   resolveFlowDir: () => resolveFlowDir,
+  resolveOrigin: () => resolveOrigin,
   sectionOf: () => sectionOf,
   selectMatchedHunks: () => selectMatchedHunks,
   statementFrom: () => statementFrom,
@@ -24750,7 +24759,7 @@ function unansweredComments(state) {
 // src/flow/review-gate.ts
 var BLOCKING_SEVERITY_FLOORS = ["blocker", "major", "minor"];
 var REVIEW_GATE_SEVERITY_FLOOR_DEFAULT = "minor";
-var REVIEW_ROUND_CAP = 3;
+var REVIEW_ROUND_CAP = 5;
 var REVIEW_GATE_CONFIG_PATH = ".metaproject/tasks.config.json";
 var DEFAULT_REVIEW_GATE_CONFIG = {
   severityFloor: REVIEW_GATE_SEVERITY_FLOOR_DEFAULT,
@@ -25613,6 +25622,129 @@ function parseOutcomeAuthor(raw, flag = "--outcome-author") {
   const shown = typeof raw === "string" ? `"${raw}"` : "nothing";
   throw new Error(`${flag} must be one of: ${OUTCOME_AUTHORS.join(", ")} (got ${shown}).`);
 }
+
+// src/flow/origin.ts
+var ORIGIN_KINDS = ["human-request", "agent-finding", "agent-proposal"];
+var ORIGIN_READINGS = [...ORIGIN_KINDS, "unknown"];
+function isOriginKind(value) {
+  return typeof value === "string" && ORIGIN_KINDS.includes(value);
+}
+function nonBlank(value) {
+  return typeof value === "string" && value.trim().length > 0;
+}
+function readOrigin(value) {
+  if (value === null || typeof value !== "object" || Array.isArray(value))
+    return;
+  const raw = value;
+  if (!isOriginKind(raw.kind))
+    return;
+  const origin = { kind: raw.kind };
+  if (typeof raw.quote === "string" && raw.quote.length > 0)
+    origin.quote = raw.quote;
+  if (typeof raw.source === "string" && raw.source.length > 0)
+    origin.source = raw.source;
+  return origin;
+}
+function readOriginKind(value) {
+  return readOrigin(value)?.kind ?? "unknown";
+}
+function resolveOrigin(input) {
+  if (!isOriginKind(input.kind)) {
+    return {
+      note: `origin kind "${input.kind}" is not one of: ${ORIGIN_KINDS.join(", ")}; origin stays unknown.`
+    };
+  }
+  const hasQuote = nonBlank(input.quote);
+  const hasSource = nonBlank(input.source);
+  if (input.kind === "human-request") {
+    if (!hasQuote && !hasSource) {
+      return { note: "origin human-request needs a verbatim --quote and a --source; neither was given, origin stays unknown." };
+    }
+    if (!hasQuote) {
+      return { note: "origin human-request needs a verbatim --quote of the human's first message; none was given, origin stays unknown." };
+    }
+    if (!hasSource) {
+      return { note: "origin human-request needs a --source (channel, message id or time); none was given, origin stays unknown." };
+    }
+  } else if (!hasSource) {
+    return { note: `origin ${input.kind} needs a --source (the check, review or test that produced it); none was given, origin stays unknown.` };
+  }
+  const origin = { kind: input.kind };
+  if (hasQuote)
+    origin.quote = input.quote;
+  if (hasSource)
+    origin.source = input.source;
+  return { origin };
+}
+function effectiveOutcomeAuthor(flow) {
+  const explicit = readOutcomeAuthor(flow.outcomeAuthor);
+  if (explicit !== "unknown")
+    return explicit;
+  return readOrigin(flow.origin) === undefined ? "unknown" : "agent";
+}
+function originSummary(origin) {
+  if (origin === undefined)
+    return "unknown";
+  const parts = [origin.kind];
+  if (origin.quote !== undefined)
+    parts.push(`quote: ${JSON.stringify(origin.quote)}`);
+  if (origin.source !== undefined)
+    parts.push(`source: ${origin.source}`);
+  return parts.join("; ");
+}
+function originDetailLines(origin, indent = "") {
+  if (origin === undefined)
+    return [`${indent}origin: unknown`];
+  const lines = [`${indent}origin: ${origin.kind}`];
+  if (origin.quote !== undefined) {
+    const [first = "", ...rest] = origin.quote.split(`
+`);
+    lines.push(`${indent}  quote:  \xAB${first}`, ...rest.map((line) => `${indent}          ${line}`));
+    lines[lines.length - 1] += "\xBB";
+  }
+  if (origin.source !== undefined)
+    lines.push(`${indent}  source: ${origin.source}`);
+  return lines;
+}
+var REQUEST_LABEL = "\u0417\u0430\u043F\u0440\u043E\u0441 (\u0434\u043E\u0441\u043B\u043E\u0432\u043D\u043E)";
+var SOURCE_LABEL = "\u0418\u0441\u0442\u043E\u0447\u043D\u0438\u043A";
+var FORMALIZATION_LABEL = "\u042D\u0444\u0444\u0435\u043A\u0442 (\u0444\u043E\u0440\u043C\u0430\u043B\u0438\u0437\u0430\u0446\u0438\u044F \u0430\u0433\u0435\u043D\u0442\u0430)";
+var OBSERVE_LABEL = "\u041A\u0430\u043A \u043D\u0430\u0431\u043B\u044E\u0434\u0430\u0442\u044C (\u043F\u0440\u0435\u0434\u043B\u043E\u0436\u0435\u043D\u0438\u0435 \u0430\u0433\u0435\u043D\u0442\u0430)";
+var REQUEST_PLACEHOLDER = "<the human's request, verbatim, no translation or paraphrase>";
+var SOURCE_PLACEHOLDER = "<where the finding or proposal came from: check, review, test or discussion>";
+var FORMALIZATION_PLACEHOLDER = "<the effect that has to be true when this flow is done, in your own words>";
+var OBSERVE_PLACEHOLDER = "<how the effect can be observed once the flow is done>";
+var ORIGIN_PLACEHOLDER_BULLETS = [
+  `- ${REQUEST_LABEL}: ${REQUEST_PLACEHOLDER}`,
+  `- ${SOURCE_LABEL}: ${SOURCE_PLACEHOLDER}`,
+  `- ${FORMALIZATION_LABEL}: ${FORMALIZATION_PLACEHOLDER}`,
+  `- ${OBSERVE_LABEL}: ${OBSERVE_PLACEHOLDER}`
+];
+function renderOriginBullets(origin) {
+  const first = [];
+  if (origin.kind === "human-request") {
+    first.push(...bulletWithText(REQUEST_LABEL, origin.quote === undefined ? undefined : `\xAB${origin.quote}\xBB`, REQUEST_PLACEHOLDER));
+    if (origin.source !== undefined)
+      first[first.length - 1] += ` (source: ${origin.source})`;
+  } else {
+    first.push(...bulletWithText(SOURCE_LABEL, origin.source, SOURCE_PLACEHOLDER));
+    if (origin.quote !== undefined)
+      first.push(...bulletWithText(REQUEST_LABEL, `\xAB${origin.quote}\xBB`, REQUEST_PLACEHOLDER));
+  }
+  return [
+    ...first,
+    `- ${FORMALIZATION_LABEL}: ${FORMALIZATION_PLACEHOLDER}`,
+    `- ${OBSERVE_LABEL}: ${OBSERVE_PLACEHOLDER}`
+  ];
+}
+function bulletWithText(label, text, placeholder) {
+  if (text === undefined)
+    return [`- ${label}: ${placeholder}`];
+  const lines = text.split(`
+`);
+  const head = `- ${label}: ${lines[0] ?? ""}`;
+  return [head, ...lines.slice(1).map((line) => `    ${line}`)];
+}
 // src/flow/identity.ts
 function resolveSignerIdentity(input) {
   const stated = input.stated?.trim();
@@ -25908,6 +26040,23 @@ function flowStateSchema() {
         enum: ["agent", "human"],
         description: "Who wrote the outcome criterion. Written by `flow init --outcome-author` (default `agent`; `human` only when the flag says so, never inferred) and changed only by `flow outcome author`, which journals the change. Absent on flows created before the field existed, and read as `unknown` (neither agent nor human). Gates nothing."
       },
+      origin: {
+        type: "object",
+        additionalProperties: false,
+        required: ["kind"],
+        description: "Where the flow came from. Written by `flow init --origin <kind> [--quote] [--source]` and changed only by `flow origin set`, which journals the change. `human-request` is recorded only with a verbatim quote and a source; `agent-finding` and `agent-proposal` only with a source. Absent on flows created before the field existed and on flows whose evidence was not enough, and read as `unknown`. Gates nothing.",
+        properties: {
+          kind: { type: "string", enum: ["human-request", "agent-finding", "agent-proposal"] },
+          quote: {
+            type: "string",
+            description: "The human's request, verbatim and byte for byte: no translation, no paraphrase."
+          },
+          source: {
+            type: "string",
+            description: "Where the request or finding came from: channel, message id, time, or the check that produced it."
+          }
+        }
+      },
       signatures: {
         type: "array",
         description: "Append-only signing record for `flow ac confirm` and a passing `flow complete`. Absent means never signed under this field's existence.",
@@ -26845,17 +26994,24 @@ function outcomeBulletsFrom(markdown) {
   if (body === null)
     return null;
   const bullets = [];
+  let skipping = false;
   for (const line of body.split(/\r?\n/)) {
     const bullet = /^\s*[-*]\s+(.*)$/.exec(line);
     if ((bullet?.[1] ?? line).trim() === OUTCOME_HINT)
       continue;
-    if (bullet !== null)
-      bullets.push((bullet[1] ?? "").trim());
-    else if (line.trim().length > 0 && bullets.length > 0)
+    if (bullet !== null) {
+      skipping = isOriginTemplateBullet(bullet[1] ?? "");
+      if (!skipping)
+        bullets.push((bullet[1] ?? "").trim());
+    } else if (line.trim().length > 0 && !skipping && bullets.length > 0)
       bullets[bullets.length - 1] += ` ${line.trim()}`;
   }
   return bullets.filter((bullet) => bullet.length > 0);
 }
+function isOriginTemplateBullet(text) {
+  const trimmed = text.trim();
+  return trimmed.startsWith(`${REQUEST_LABEL}:`) || trimmed.startsWith(`${SOURCE_LABEL}:`) || ORIGIN_PLACEHOLDER_BULLETS.some((placeholder) => placeholder.slice(2) === trimmed);
+}
 var INIT_INTENT_NOTE = "note: no intent statement extractable from description.md \u2014 the product index will hold no statement for this flow";
 function initIntentNote(description) {
   return flowStatementFrom(description) === null ? INIT_INTENT_NOTE : null;
@@ -27417,7 +27573,9 @@ async function flowFoldersInHead(cwd, dirs) {
 }
 
 // src/flow/templates.ts
-function renderDescription(title, source) {
+function renderDescription(title, source, origin) {
+  const outcome = origin === undefined ? OUTCOME_HINT : renderOriginBullets(origin).join(`
+`);
   return `# ${title}
 
 Status: draft (flow-init skill formalizes this)
@@ -27433,7 +27591,7 @@ What must be true when this flow is done.
 
 ## Outcome criteria
 
-${OUTCOME_HINT}
+${outcome}
 
 ## Out of Scope
 
@@ -27750,7 +27908,24 @@ function createFlowService(deps) {
       if (input.owner !== undefined && !input.owner.trim()) {
         throw new Error(BLANK_OWNER_MESSAGE);
       }
-      const outcomeAuthor = input.outcomeAuthor === undefined ? DEFAULT_OUTCOME_AUTHOR : parseOutcomeAuthor(input.outcomeAuthor);
+      const outcomeAuthor = input.outcomeAuthor === undefined ? undefined : parseOutcomeAuthor(input.outcomeAuthor);
+      let originNote;
+      let origin;
+      let originDraft;
+      if (input.origin !== undefined) {
+        const resolution = resolveOrigin({ kind: input.origin, quote: input.originQuote, source: input.originSource });
+        origin = resolution.origin;
+        originNote = resolution.note;
+        if (isOriginKind(input.origin)) {
+          originDraft = {
+            kind: input.origin,
+            ...input.originQuote?.trim() ? { quote: input.originQuote } : {},
+            ...input.originSource?.trim() ? { source: input.originSource } : {}
+          };
+        }
+      } else if (input.originQuote !== undefined || input.originSource !== undefined) {
+        originNote = "--quote/--source were given without --origin; origin stays unknown.";
+      }
       const trackerReady = deps.tracker ? await deps.tracker.detect() : false;
       const tracker = trackerReady ? deps.tracker : null;
       const issueRef = input.issue && deps.tracker ? deps.tracker.parseRef(input.issue) : null;
@@ -27806,12 +27981,13 @@ function createFlowService(deps) {
           pr: { url: null },
           ...input.baseBranch === undefined ? {} : { baseBranch: input.baseBranch },
           ...input.owner?.trim() ? { owner: ownerIdentity(input.owner.trim(), "`--owner` flag on `flow init`") } : {},
-          outcomeAuthor,
+          ...outcomeAuthor !== undefined ? { outcomeAuthor } : origin === undefined ? { outcomeAuthor: DEFAULT_OUTCOME_AUTHOR } : {},
+          ...origin === undefined ? {} : { origin },
           tasks: DEFAULT_TASKS.map((task) => ({ ...task, status: "todo" })),
           history: [{ at: createdAt, event: "created" }]
         };
         const sourceLabel = input.issue ?? "user description";
-        await writeFileAtomic(path56.join(absolute, "description.md"), renderDescription(title, sourceLabel));
+        await writeFileAtomic(path56.join(absolute, "description.md"), renderDescription(title, originDraft === undefined ? sourceLabel : `${sourceLabel} (origin: ${originDraft.kind})`, originDraft));
         await writeFileAtomic(path56.join(absolute, "context.md"), context.markdown);
         await writeFileAtomic(path56.join(absolute, "plan.md"), renderPlan());
         await writeFileAtomic(path56.join(absolute, "tasks.md"), renderTasksDoc());
@@ -27819,7 +27995,12 @@ function createFlowService(deps) {
         await writeFileAtomic(path56.join(absolute, "journal.md"), renderJournal(createdAt));
         await writeFlow(input.cwd, dir, flow);
         await recordAllocation(scope, { id, dir, at: createdAt, project: scope.project });
-        return { flow, dir: path56.relative(input.cwd, absolute), contextNotes: context.notes };
+        return {
+          flow,
+          dir: path56.relative(input.cwd, absolute),
+          contextNotes: context.notes,
+          ...originNote === undefined ? {} : { originNote }
+        };
       });
     },
     async list({ cwd }) {
@@ -27883,6 +28064,50 @@ function createFlowService(deps) {
       });
       return { flow, previous, changed };
     },
+    async originSet({ cwd, id, kind, reason, quote, source }) {
+      if (!reason?.trim()) {
+        throw new Error('flow origin set requires --reason "<why>"');
+      }
+      validateSingleLineReason(reason);
+      let previous = "unknown";
+      let next = "unknown";
+      let changed = false;
+      let note;
+      const flow = await mutate(cwd, id, async ({ dir, flow: current }) => {
+        const before = readOrigin(current.origin);
+        previous = before?.kind ?? "unknown";
+        let after;
+        if (kind !== "unknown" && !isOriginKind(kind)) {
+          note = `origin kind must be one of: human-request, agent-finding, agent-proposal, unknown (got "${kind}"); the origin is unchanged.`;
+          next = previous;
+          return current;
+        }
+        if (kind !== "unknown") {
+          const resolution = resolveOrigin({
+            kind,
+            quote: quote !== undefined && quote.length > 0 ? quote : before?.quote,
+            source: source !== undefined && source.length > 0 ? source : before?.source
+          });
+          if (resolution.origin === undefined) {
+            note = resolution.note;
+            next = previous;
+            return current;
+          }
+          after = resolution.origin;
+        }
+        next = after?.kind ?? "unknown";
+        if (JSON.stringify(before ?? null) === JSON.stringify(after ?? null)) {
+          return current;
+        }
+        changed = true;
+        if (after === undefined)
+          delete current.origin;
+        else
+          current.origin = after;
+        return save(cwd, dir, current, "origin-set", `${originSummary(before)} -> ${originSummary(after)} (${reason.trim()})`);
+      });
+      return { flow, previous, next, changed, ...note === undefined ? {} : { note } };
+    },
     async freeze({ cwd, id }) {
       return mutate(cwd, id, async ({ dir, flow }) => {
         assertTransition(flow.status, "ready");
@@ -41023,6 +41248,12 @@ var HELP_GROUPS = [
     group: "Managed work",
     summary: "List doc sections that likely went stale because of the working diff, with Jev."
   },
+  {
+    kind: "slash",
+    name: "/flow",
+    group: "Managed work",
+    summary: `Show a flow's origin, or set it with a reason \u2014 /flow origin [<id>] | /flow origin <id> <kind> --reason "...".`
+  },
   {
     kind: "slash",
     name: "/opencomments",

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@mrciphersmith/keryx",
-  "version": "0.3.58",
+  "version": "0.3.61",
   "description": "Version-controlled project context for AI coding agents: code graph, architecture wiki, project memory, relevant tests, quality signals, and task flows.",
   "private": false,
   "publishConfig": {

package/src/gdskills/bundled/skills/orchestration/flow-orchestrator/SKILL.md

@@ -81,8 +81,8 @@ flowchart TD
   H -- "yes" --> I{"Ask user how to finish"}
   I -- "create PR" --> J["create PR and run review/fix loop"]
   J --> K{"review clean, PR mergeable?"}
-  K -- "no, attempts < 3" --> J
-  K -- "no, attempts = 3 or repetition detected" --> R["enrich context and change fix strategy"]
+  K -- "no, attempts < 5" --> J
+  K -- "no, attempts = 5 or repetition detected" --> R["enrich context and change fix strategy"]
   R --> J
   K -- "yes" --> L["merge PR into recorded base branch"]
   L --> M["keryx flow implemented --pr"]
@@ -153,7 +153,7 @@ it already tried. The flow package does.
       ```
 
    6. Apply the Phase 4 attempt budget against the **persisted** count. If
-      `attempts.count` for the task has already reached **three**, do not
+      `attempts.count` for the task has already reached **five**, do not
       re-dispatch the same approach: go to the re-planning step (Phase 4, PR
       review/fix loop, step 4) and record the decision in `journal.md`.
    7. Run the repetition check before spending an attempt, whatever the count
@@ -533,7 +533,7 @@ answered it and who is behind it.
 | `completion: <anything>` as a constraint STRING | **Not an outcome. Refuse it and ask for the typed field.** `constraints[]` is parsed by nothing, so a completion arriving there is never read at all. Such a dispatch is now refused for the missing `completion_outcome` rather than silently accepted — but the refusal is the contract's, not this row's, and a row telling you to honour the string would be a documented bypass of the fence in the file that owns it. |
 
 A constraint that would raise this skill's own attempt budget is **not** obeyed.
-The three-attempt bound and the `keryx review loop` repetition check are this
+The five-attempt bound and the `keryx review loop` repetition check are this
 skill's, they are evidence-backed, and a caller asking for "loop until clean" gets
 the bound plus an escalation — never an unbounded loop.
 
@@ -551,23 +551,21 @@ the bound plus an escalation — never an unbounded loop.
    remaining `info` findings in the completion report rather than fixing them
    under a loop that was not opened for them. A caller may lower the threshold in
    `constraints`; it cannot raise it to merge over a `minor`.
-3. Allow at most **three** review/fix attempts for the current approach. Count
+3. Allow at most **five** review/fix attempts for the current approach. Count
    an attempt when review/check results are available, including a clean result,
    and record it with `keryx flow task attempt <id> <Tn> --outcome ...` so the
    count survives a session restart. Read the budget from that task's
    `attempts.count` in `flow.json`, never from this session's memory.
 
-   Three, and the same three that `task-implementer` and `job-orchestrator`
-   already use. This skill said six, which was an outlier with nothing behind
-   it. The evidence converges on three: *"the first three to four repair
+   Five here by operator decision (message 178020; review findings surface across rounds, and the repetition check below stops a loop that is not converging), three in `task-implementer` and `job-orchestrator`. This skill said six, an outlier with nothing behind it. The evidence for a repair loop converges on three: *"the first three to four repair
    iterations account for most achievable gains"*
    ([arXiv:2607.05197](https://arxiv.org/abs/2607.05197)); correctness falls
    **0.820 -> 0.673** across two forced revisions while cumulative ever-correct
    is **0.847** ([arXiv:2607.24604](https://arxiv.org/abs/2607.24604)) — the
    agent finds the fix and then destroys it, throwing away ~15 points by not
    stopping. Aider hardcodes `max_reflections = 3`; OpenHands' critic uses 3.
-   Rounds four through six were not buying convergence; they were buying
-   regressions.
+   Rounds four through six of a self-fix loop were not buying convergence; they
+   were buying regressions. Review rounds are the exception, as stated above.
 
 4. **Before** spending an attempt, and regardless of how much budget is left,
    run the repetition check:
@@ -583,7 +581,7 @@ the bound plus an escalation — never an unbounded loop.
    emitting the identical failing output three times must be caught on the
    second, not after the budget runs out.
 
-5. If the third attempt is not clean, **or the repetition check escalated
+5. If the fifth attempt is not clean, **or the repetition check escalated
    earlier**, do not blindly repeat the same loop. Enrich context from the
    findings, affected graph, relevant wiki, and health/testing artifacts;
    identify the likely cycle cause; choose a materially different fix strategy

package/src/gdskills/bundled/skills/orchestration/job-orchestrator/SKILL.md

@@ -1265,8 +1265,8 @@ with evidence, not silence.
 
 Only runs if NEEDS_FIX is true. Default max: **3 iterations** (`max_review_iterations`).
 
-Three is the shared round bound: `task-implementer`, `flow-orchestrator` and
-this skill all use it. *"The first three to four repair iterations account for
+Three is the self-fix bound `task-implementer` and this skill share
+(`flow-orchestrator`'s review bound is five, by operator decision). *"The first three to four repair iterations account for
 most achievable gains"* ([arXiv:2607.05197](https://arxiv.org/abs/2607.05197));
 correctness falls **0.820 -> 0.673** across two forced revisions while
 cumulative ever-correct is **0.847**

package/src/gdskills/bundled/skills/orchestration/job-orchestrator/input-contract.schema.json

@@ -106,7 +106,7 @@
           "minimum": 1,
           "maximum": 3,
           "default": 3,
-          "description": "Max review → fix iterations. Three: the same bound `task-implementer`, `flow-orchestrator` and SKILL.md 2.7 carry, pinned across all of them by src/gdskills/round-bound.test.ts. The default was 2 here, which made the contract disagree with the skill it validates."
+          "description": "Max review → fix iterations. Three: the same bound `task-implementer` and SKILL.md 2.7 carry (`flow-orchestrator`'s review bound is five, by operator decision), pinned by src/gdskills/round-bound.test.ts. The default was 2 here, which made the contract disagree with the skill it validates."
         },
         "plan_approval": {
           "type": "boolean",

package/src/gdskills/bundled/skills/orchestration/task-implementer/SKILL.md

@@ -362,8 +362,8 @@ non-zero exit the signal; if tests were created or modified they must pass.
 
 Maximum 3 self-fix attempts per verification step.
 
-Three, and it is the same three `job-orchestrator` and `flow-orchestrator`
-use: one round bound, not four. *"The first three to four repair iterations
+Three, the same three `job-orchestrator` uses; `flow-orchestrator`'s review
+bound is five by operator decision, so two repair bounds, not four. *"The first three to four repair iterations
 account for most achievable gains"*
 ([arXiv:2607.05197](https://arxiv.org/abs/2607.05197)); correctness falls
 **0.820 -> 0.673** across two forced revisions while cumulative ever-correct is

package/src/gdskills/bundled/skills/orchestration/task-implementer/input-contract.schema.json

@@ -188,7 +188,7 @@
           "type": "integer",
           "minimum": 1,
           "maximum": 3,
-          "description": "Fix iteration number. Three, the same bound `flow-orchestrator`, `job-orchestrator` and SKILL.md Phase 5.4 carry and `src/gdskills/round-bound.test.ts` pins; it was 2 here, a fourth disagreeing repair bound."
+          "description": "Fix iteration number. Three, the same bound `job-orchestrator` and SKILL.md Phase 5.4 carry and `src/gdskills/round-bound.test.ts` pins (`flow-orchestrator`'s review bound is five); it was 2 here, a fourth disagreeing repair bound."
         }
       }
     },