omp-conductor 0.15.8 → 0.15.9

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "omp-conductor",
-  "version": "0.15.8",
+  "version": "0.15.9",
   "type": "module",
   "license": "MIT",
   "description": "A 24/7 dispatcher that takes ready GitHub issues to green, mergeable PRs using omp coding sessions, with tiered escalation first to an orchestrator session and then to a human.",

package/src/setup.ts

@@ -146,6 +146,12 @@ export interface SetupAnswers {
    * pinned one.
    */
   workerModel?: string;
+  /**
+   * Routable-candidate count below which the tick prompt says to groom. The
+   * wizard never asks for it — it is hand-edited — so it exists here only to
+   * survive an amend of some other area (#369).
+   */
+  groomBelow?: number;
   telegramChatId?: string;
   /** Forum topic for tier-2 Telegram pages; absent keeps flat-chat 0.13 behaviour. */
   telegramTopicId?: number;
@@ -743,9 +749,16 @@ export function buildProject(a: SetupAnswers): ProjectConfig {
     // `planUsage` is the one cap that is not a number, and the wizard does not
     // ask for it: naming a provider allowance window means reading
     // `omp usage --json` on the fleet host first, so it stays a documented
-    // hand-edited key (#110). Skipped here to keep this numeric loop honest.
+    // hand-edited key (#110). Skipped here to keep this numeric loop honest,
+    // and carried below instead — an unrelated amend that deleted the weekly
+    // allowance guard would be exactly the silent loss #369 is about.
     if (key !== "planUsage" && typeof value === "number") caps[key] = value;
   }
+  // `null` is a spelling in its own right — "no allowance guard" — so both it
+  // and a configured window survive an amend unchanged.
+  if (a.caps.planUsage !== undefined) {
+    caps.planUsage = a.caps.planUsage === null ? null : { ...a.caps.planUsage };
+  }
 
   return {
     name: a.projectName,
@@ -757,6 +770,8 @@ export function buildProject(a: SetupAnswers): ProjectConfig {
     ...(a.workerModel !== undefined && a.workerModel.trim().length > 0
       ? { workerModel: a.workerModel.trim() }
       : {}),
+    // A documented hand-edited key, so an unrelated amend must not delete it.
+    ...(a.groomBelow === undefined ? {} : { groomBelow: a.groomBelow }),
     escalation,
     authority: { ...a.authority },
     // Written out in full, never as the legacy string: the file then says which
@@ -957,6 +972,7 @@ export function answersFromProject(p: ProjectConfig): SetupAnswers {
   // Set only when present, never as an explicit `undefined`: an absent key is
   // what keeps the rewritten config identical to the one that was read.
   if (p.workerModel !== undefined) answers.workerModel = p.workerModel;
+  if (p.groomBelow !== undefined) answers.groomBelow = p.groomBelow;
   if (p.escalation.telegramChatId !== undefined) answers.telegramChatId = p.escalation.telegramChatId;
   if (p.escalation.telegramTopicId !== undefined) answers.telegramTopicId = p.escalation.telegramTopicId;
   if (p.recoveryMerges !== undefined) {