omp-conductor 0.17.1 → 0.18.0

package/src/config.ts

@@ -28,6 +28,9 @@ import {
   DEFAULT_PROJECT_POLICY,
   DEFAULT_REPORT_POLICY,
   DEFAULT_REPORT_SCOPE,
+  DEFAULT_REVIEW_MAX_ROUNDS,
+  DEFAULT_REVIEW_POLICY,
+  DEFAULT_REVIEW_STRICTNESS,
   DENIED_RELEASE_GRANTS,
   DRAFT_POLICIES,
   LEGACY_RELEASE_POLICIES,
@@ -37,6 +40,9 @@ import {
   RELEASE_REQUIREMENTS,
   RELEASE_SHAPES,
   REPORT_SCOPES,
+  REVIEW_MAX_ROUNDS_MAX,
+  REVIEW_MAX_ROUNDS_MIN,
+  REVIEW_STRICTNESS,
   INTERRUPT_CATEGORIES,
   DIGEST_CADENCES,
   WEEKDAYS,
@@ -53,6 +59,8 @@ import {
   type ReleaseRequirement,
   type ReportScope,
   type ReportingPolicy,
+  type ReviewPolicy,
+  type ReviewStrictness,
   type Weekday,
   type WeeklyAvailability,
   type RepoTarget,
@@ -71,6 +79,7 @@ import {
   ORCHESTRATOR_MODE_LIST,
   RELEASE_REQUIREMENT_LIST,
   RELEASE_SHAPE_LIST,
+  REVIEW_STRICTNESS_LIST,
   REPORT_SCOPE_LIST,
   WEEKDAY_LIST,
   quoteList,
@@ -98,10 +107,13 @@ const REPO_RE = /^[A-Za-z0-9._-]+\/[A-Za-z0-9._-]+$/;
  * Used when a project omits `stateLabels`. Namespaced so a human scanning the
  * tracker can tell dispatcher-written labels from their own.
  */
-const DEFAULT_STATE_LABELS: ProjectConfig["stateLabels"] = {
+export const DEFAULT_STATE_LABELS: ProjectConfig["stateLabels"] = {
   inProgress: "agent:in-progress",
   blocked: "agent:blocked",
   failed: "agent:failed",
+  // The operator's park gesture (#507): unnamed by design — it is theirs, and
+  // the operator adds it by hand until setup grows a confirm step (slice 2).
+  backlog: "backlog",
 };
 
 /**
@@ -330,6 +342,20 @@ export function resolveArmProof(p: ProjectConfig): ArmProof {
   return p.arm?.proof === "claim-only" ? "claim-only" : DEFAULT_ARM_PROOF;
 }
 
+/**
+ * The project's review policy (#678), complete.
+ *
+ * The loader always materialises `review` (finalizeProject), so this is only
+ * ever the fallback for a `ProjectConfig` that never went through it — a
+ * hand-built one in a test, or a config written before the key existed.
+ * Absent resolves to {@link DEFAULT_REVIEW_POLICY}, the documented migration
+ * default every existing install deterministically upgrades to. A fresh object
+ * each call, so mutating the result cannot reach the config it came from.
+ */
+export function resolveReview(p: ProjectConfig): ReviewPolicy {
+  return { ...(p.review ?? DEFAULT_REVIEW_POLICY) };
+}
+
 /**
  * A policy with no array shared with its source.
  *
@@ -861,6 +887,12 @@ function clauseFor(rel: readonly PropertyKey[], issue: {
     if (rel.length === 1 && relStr === "policy") {
       return `policy must be an object with "merge" and "release" sections`;
     }
+    if (rel.length === 1 && relStr === "review") {
+      return `review must be an object with "strictness" and "maxRounds"`;
+    }
+    if (rel.length === 2 && rel[0] === "review" && rel[1] === "maxRounds") {
+      return `review.maxRounds must be an integer between ${REVIEW_MAX_ROUNDS_MIN} and ${REVIEW_MAX_ROUNDS_MAX}, found ${found()}`;
+    }
     if (rel.length === 2 && relStr === "policy") return `${pathStr} must be an object`;
     if (rel.length === 1 && relStr === "caps") return `caps must be an object`;
     if (rel.length === 2 && relStr === "caps" && rel[1] === "planUsage") {
@@ -911,6 +943,7 @@ function clauseFor(rel: readonly PropertyKey[], issue: {
   if (issue.code === "unrecognized_keys") {
     const keys = (issue.keys ?? []).join(", ");
     if (pathStr === "policy") return `policy has unknown key(s): ${keys} — expected "merge" or "release"`;
+    if (pathStr === "review") return `review has unknown key(s): ${keys} — expected "strictness" and "maxRounds"`;
     if (pathStr === "policy.merge") return `${pathStr} has unknown key(s): ${keys} — expected ${POLICY_MERGE_KEYS}`;
     if (pathStr === "policy.release") return `${pathStr} has unknown key(s): ${keys} — expected ${POLICY_RELEASE_KEYS}`;
     if (pathStr === "caps.planUsage") {
@@ -960,6 +993,9 @@ function clauseFor(rel: readonly PropertyKey[], issue: {
 
   // Regex / custom-message failures carry the finding in issue.message.
   if (issue.code === "too_small") {
+    if (pathStr === "review.maxRounds") {
+      return `review.maxRounds must be an integer between ${REVIEW_MAX_ROUNDS_MIN} and ${REVIEW_MAX_ROUNDS_MAX}, found ${found()}`;
+    }
     // Empty array (reporting.interruptOn, availability.days) has its own line.
     if (parentOf(rel) === "reporting.interruptOn") return `reporting.interruptOn must name at least one category`;
     if (pathStr === "reporting.availability.days") return `reporting.availability.days must be a non-empty array of ${WEEKDAY_LIST}`;
@@ -975,6 +1011,9 @@ function clauseFor(rel: readonly PropertyKey[], issue: {
   }
 
   if (issue.code === "too_big") {
+    if (pathStr === "review.maxRounds") {
+      return `review.maxRounds must be an integer between ${REVIEW_MAX_ROUNDS_MIN} and ${REVIEW_MAX_ROUNDS_MAX}, found ${found()}`;
+    }
     if (rel.length === 2 && rel[0] === "caps") return capProblem(rel[1] as string, found());
     if (pathStr === "caps.planUsage.maxUsedFraction") {
       return `caps.planUsage.maxUsedFraction must be a fraction between 0 and 1 (0.85 holds at 85% of the allowance), found ${found()}`;
@@ -1101,6 +1140,7 @@ function finalizeProject(
     );
   }
   const policy = finalizePolicy(p["policy"], label, problems);
+  const review = finalizeReview(p["review"] as Raw | undefined);
   const caps = reconcileCaps(p["caps"], `${label}: caps`, problems, legacyCaps);
   const reporting = finalizeReporting(p["reporting"], label, problems);
   const recoveryMerges = (p["recoveryMerges"] as RecoveryMergeAuthorization[] | undefined)?.map(
@@ -1199,6 +1239,7 @@ function finalizeProject(
       inProgress: pickString(stateLabels?.["inProgress"], DEFAULT_STATE_LABELS.inProgress),
       blocked: pickString(stateLabels?.["blocked"], DEFAULT_STATE_LABELS.blocked),
       failed: pickString(stateLabels?.["failed"], DEFAULT_STATE_LABELS.failed),
+      backlog: pickString(stateLabels?.["backlog"], DEFAULT_STATE_LABELS.backlog),
     },
     routing: { labelPrefix, repos },
     caps,
@@ -1208,6 +1249,7 @@ function finalizeProject(
     ...(effectiveOmpSettings === undefined ? {} : { ompSettings: effectiveOmpSettings }),
     escalation,
     arm,
+    review,
     authority,
     releasePolicy,
     policy,
@@ -1231,6 +1273,23 @@ function finalizeArm(parsed: Raw | undefined): ProjectConfig["arm"] {
   return { proof: (parsed["proof"] as ArmProof | undefined) ?? DEFAULT_ARM_PROOF };
 }
 
+/**
+ * The project's review policy (#678), complete. A config written before the
+ * key existed — or one that never answered — materialises as
+ * {@link DEFAULT_REVIEW_POLICY}, the documented migration default, so an
+ * upgrade changes no existing project's behaviour beyond what its operators
+ * then choose in setup. zod has already rejected anything outside the
+ * strictness vocabulary or the validated round range, so this only fills the
+ * absent-case default.
+ */
+function finalizeReview(parsed: Raw | undefined): ProjectConfig["review"] {
+  if (parsed === undefined) return { ...DEFAULT_REVIEW_POLICY };
+  return {
+    strictness: (parsed["strictness"] as ReviewStrictness | undefined) ?? DEFAULT_REVIEW_STRICTNESS,
+    maxRounds: (parsed["maxRounds"] as number | undefined) ?? DEFAULT_REVIEW_MAX_ROUNDS,
+  };
+}
+
 function finalizeEscalation(parsed: Raw | undefined): ProjectConfig["escalation"] {
   if (parsed === undefined) {
     return { fallbackToIssueComment: true, orchestrator: "embedded" };