@rubytech/create-maxy-code 0.1.546 → 0.1.548

package/dist/index.js

@@ -3405,6 +3405,20 @@ function setupAccount() {
             env: { ...neo4jEnv, PLATFORM_ROOT: join(INSTALL_DIR, "platform") },
         });
     }
+    // Task 2361 — collapse the two routine description properties onto
+    // `:Event.description`, preferring whichever value the operator or the agent
+    // wrote. Runs after the seed for the same reasons the convergence above does:
+    // the schema exists by then, and the same Neo4j env is already built. Scoped to
+    // this install's accounts via PLATFORM_ROOT, so a co-resident brand sharing one
+    // Neo4j is untouched. Idempotent: a converged install matches nothing and
+    // prints converged=0.
+    const convergeRoutineDescription = join(INSTALL_DIR, "platform/plugins/scheduling/mcp/dist/scripts/converge-routine-description.js");
+    if (existsSync(convergeRoutineDescription)) {
+        shell("node", [convergeRoutineDescription], {
+            cwd: INSTALL_DIR,
+            env: { ...neo4jEnv, PLATFORM_ROOT: join(INSTALL_DIR, "platform") },
+        });
+    }
     // Task 165 — register every bundled specialist subagent at
     // $CLAUDE_CONFIG_DIR/agents/<name>.md. Claude Code's `--agent <name>`
     // discovery only resolves under this path; without these symlinks,

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@rubytech/create-maxy-code",
-  "version": "0.1.546",
+  "version": "0.1.548",
   "description": "Install Maxy — AI for Productive People",
   "bin": {
     "create-maxy-code": "./dist/index.js"

package/payload/platform/lib/routine-templates/dist/index.d.ts

@@ -70,13 +70,14 @@ export interface RoutineVerifyCounts {
     actual: number;
     suspended: number;
     nextRunNull: number;
-    /** Task 2214 — roster rows holding no description. Zero is the healthy value
-     *  on every account, not only a fresh one: `VERIFY_CYPHER` matches
-     *  `routineTemplate IS NOT NULL`, so operator-created routines are outside
-     *  its scope and cannot contribute, and `ON MATCH` writes the description
-     *  unconditionally, so an operator-edited row is covered too. Anything above
-     *  zero while `expected === actual` is a MERGE that wrote no description —
-     *  the one failure here that emits no error and does not reproduce. */
+    /** Roster rows holding no description. Zero is the healthy value on every
+     *  account, not only a fresh one: `VERIFY_CYPHER` matches `routineTemplate IS
+     *  NOT NULL`, so operator-created routines are outside its scope and cannot
+     *  contribute. Anything above zero while `expected === actual` is a MERGE that
+     *  wrote no description on create, or a row whose key left the roster before
+     *  the convergence ran — the one failure here that emits no error and does not
+     *  reproduce. `[schedule-audit] op=description-coverage` is the wider twin,
+     *  over every automation row rather than the roster alone. */
     descNull: number;
 }
 export interface RoutineSeedReport {
@@ -124,17 +125,22 @@ export declare function resolveOwnerTimezone(session: SeedSessionLike, accountId
  *  destination, or reset a retimed cron. Only the prompt, the name and the
  *  version move, and only when the row is behind and unedited.
  *
- *  `routineDescription` (Task 2214) is the one property ON MATCH writes on
- *  every match rather than only on the refresh branch, and that is not an
- *  exception to the rule above. It is roster-owned: the routines route projects
- *  it but its PATCH field set refuses it, so no operator edit exists for this
- *  to overwrite. Gating it on `refreshing` would leave every operator-edited
- *  row description-less permanently, because `skip-operator-edited` takes
- *  neither branch, and that row's card would show a title and nothing else. */
-export declare const MERGE_CYPHER = "\nUNWIND $rows AS row\nOPTIONAL MATCH (pre:Event {accountId: $accountId, routineTemplate: row.key})\nWITH row,\n     pre IS NOT NULL AS existed,\n     coalesce(pre.routineTemplateVersion, 0) AS priorVersion,\n     pre.operatorEdited IS NOT NULL AS edited\nWITH row, existed, priorVersion, edited,\n     (existed AND NOT edited AND priorVersion < $version) AS refreshing\nMERGE (e:Event {accountId: $accountId, routineTemplate: row.key})\n  ON CREATE SET e.eventId = randomUUID(),\n                e.name = row.name,\n                e.routineDescription = row.description,\n                e.agentPrompt = row.prompt,\n                e.recurrence = row.recurrence,\n                e.routineTemplateVersion = $version,\n                e.eventStatus = 'suspended',\n                e.nextRun = null,\n                e.agentChannel = null,\n                e.agentDestination = null,\n                e.actionPlugin = null,\n                e.actionTool = null,\n                e.actionArgs = null,\n                e.sourcePlugin = null,\n                e.timezone = $timezone,\n                e.createdBySource = 'installer',\n                e.createdByAgent = 'system',\n                e.createdByTool = 'routine-roster-seed',\n                e.createdAt = $now,\n                e.updatedAt = $now\n  ON MATCH SET e.agentPrompt = CASE WHEN refreshing THEN row.prompt ELSE e.agentPrompt END,\n               e.name = CASE WHEN refreshing THEN row.name ELSE e.name END,\n               e.routineDescription = row.description,\n               e.routineTemplateVersion = CASE WHEN refreshing THEN $version ELSE e.routineTemplateVersion END,\n               e.updatedAt = CASE WHEN refreshing THEN $now ELSE e.updatedAt END\nRETURN row.key AS key,\n       e.eventId AS eventId,\n       CASE WHEN NOT existed THEN 'create'\n            WHEN priorVersion >= $version THEN 'unchanged'\n            WHEN edited THEN 'skip-operator-edited'\n            ELSE 'refresh-version' END AS action";
+ *  `description` (Task 2361) is guarded exactly as the prompt and the name are,
+ *  and that is a change of shape from Task 2214, which introduced it. It used to
+ *  be written on every match, on the grounds that it was roster-owned and the
+ *  routines PATCH refused it, so no operator edit existed for the seed to
+ *  clobber, and that gating it would leave every operator-edited row
+ *  description-less permanently because `skip-operator-edited` takes neither
+ *  branch. Both halves of that argument are gone. The routines PATCH now accepts
+ *  `description` and stamps `operatorEdited`, so an unconditional write would
+ *  destroy the operator's own caption on the next version bump; and the
+ *  scheduling plugin's one-shot `converge-routine-description` has already given
+ *  every pre-existing row a description, so no row is left holding only a
+ *  title. */
+export declare const MERGE_CYPHER = "\nUNWIND $rows AS row\nOPTIONAL MATCH (pre:Event {accountId: $accountId, routineTemplate: row.key})\nWITH row,\n     pre IS NOT NULL AS existed,\n     coalesce(pre.routineTemplateVersion, 0) AS priorVersion,\n     pre.operatorEdited IS NOT NULL AS edited\nWITH row, existed, priorVersion, edited,\n     (existed AND NOT edited AND priorVersion < $version) AS refreshing\nMERGE (e:Event {accountId: $accountId, routineTemplate: row.key})\n  ON CREATE SET e.eventId = randomUUID(),\n                e.name = row.name,\n                e.description = row.description,\n                e.agentPrompt = row.prompt,\n                e.recurrence = row.recurrence,\n                e.routineTemplateVersion = $version,\n                e.eventStatus = 'suspended',\n                e.nextRun = null,\n                e.agentChannel = null,\n                e.agentDestination = null,\n                e.actionPlugin = null,\n                e.actionTool = null,\n                e.actionArgs = null,\n                e.sourcePlugin = null,\n                e.timezone = $timezone,\n                e.createdBySource = 'installer',\n                e.createdByAgent = 'system',\n                e.createdByTool = 'routine-roster-seed',\n                e.createdAt = $now,\n                e.updatedAt = $now\n  ON MATCH SET e.agentPrompt = CASE WHEN refreshing THEN row.prompt ELSE e.agentPrompt END,\n               e.name = CASE WHEN refreshing THEN row.name ELSE e.name END,\n               e.description = CASE WHEN refreshing THEN row.description ELSE e.description END,\n               e.routineTemplateVersion = CASE WHEN refreshing THEN $version ELSE e.routineTemplateVersion END,\n               e.updatedAt = CASE WHEN refreshing THEN $now ELSE e.updatedAt END\nRETURN row.key AS key,\n       e.eventId AS eventId,\n       CASE WHEN NOT existed THEN 'create'\n            WHEN priorVersion >= $version THEN 'unchanged'\n            WHEN edited THEN 'skip-operator-edited'\n            ELSE 'refresh-version' END AS action";
 /** The post-condition read. Counts every roster row the account actually holds,
  *  independently of what the MERGE above believes it wrote. */
-export declare const VERIFY_CYPHER = "\nMATCH (e:Event {accountId: $accountId})\nWHERE e.routineTemplate IS NOT NULL\nRETURN count(e) AS actual,\n       sum(CASE WHEN e.eventStatus = 'suspended' THEN 1 ELSE 0 END) AS suspended,\n       sum(CASE WHEN e.nextRun IS NULL THEN 1 ELSE 0 END) AS nextRunNull,\n       sum(CASE WHEN e.routineDescription IS NULL THEN 1 ELSE 0 END) AS descNull";
+export declare const VERIFY_CYPHER = "\nMATCH (e:Event {accountId: $accountId})\nWHERE e.routineTemplate IS NOT NULL\nRETURN count(e) AS actual,\n       sum(CASE WHEN e.eventStatus = 'suspended' THEN 1 ELSE 0 END) AS suspended,\n       sum(CASE WHEN e.nextRun IS NULL THEN 1 ELSE 0 END) AS nextRunNull,\n       sum(CASE WHEN e.description IS NULL THEN 1 ELSE 0 END) AS descNull";
 export interface SeedRoutineRosterArgs {
     session: SeedSessionLike;
     accountId: string;

package/payload/platform/lib/routine-templates/dist/index.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"index.d.ts","sourceRoot":"","sources":["../src/index.ts"],"names":[],"mappings":"AAAA;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;GA+BG;AACH,OAAO,EAA4C,KAAK,eAAe,EAAE,MAAM,aAAa,CAAC;AAE7F,OAAO,EAAE,cAAc,EAAE,mBAAmB,EAAE,wBAAwB,EAAE,MAAM,aAAa,CAAC;AAC5F,YAAY,EAAE,eAAe,EAAE,MAAM,aAAa,CAAC;AAEnD;;;2EAG2E;AAC3E,MAAM,WAAW,eAAe;IAC9B,GAAG,CAAC,KAAK,EAAE,MAAM,EAAE,MAAM,CAAC,EAAE,MAAM,CAAC,MAAM,EAAE,OAAO,CAAC,GAAG,OAAO,CAAC;QAAE,OAAO,EAAE,KAAK,CAAC;YAAE,GAAG,CAAC,GAAG,EAAE,MAAM,GAAG,OAAO,CAAA;SAAE,CAAC,CAAA;KAAE,CAAC,CAAC;CAClH;AAED;;;;;;;4EAO4E;AAC5E,MAAM,MAAM,kBAAkB,GAAG,QAAQ,GAAG,iBAAiB,GAAG,sBAAsB,GAAG,WAAW,CAAC;AAErG,MAAM,WAAW,eAAe;IAC9B,GAAG,EAAE,MAAM,CAAC;IACZ,OAAO,EAAE,MAAM,CAAC;IAChB,MAAM,EAAE,kBAAkB,CAAC;CAC5B;AAED;;;;;;0EAM0E;AAC1E,MAAM,WAAW,mBAAmB;IAClC,QAAQ,EAAE,MAAM,CAAC;IACjB,MAAM,EAAE,MAAM,CAAC;IACf,SAAS,EAAE,MAAM,CAAC;IAClB,WAAW,EAAE,MAAM,CAAC;IACpB;;;;;;2EAMuE;IACvE,QAAQ,EAAE,MAAM,CAAC;CAClB;AAED,MAAM,WAAW,iBAAiB;IAChC,SAAS,EAAE,MAAM,CAAC;IAClB,OAAO,EAAE,MAAM,CAAC;IAChB,QAAQ,EAAE,MAAM,GAAG,IAAI,CAAC;IACxB,cAAc,EAAE,eAAe,GAAG,QAAQ,CAAC;IAC3C,MAAM,EAAE,eAAe,EAAE,CAAC;IAC1B,OAAO,EAAE,MAAM,CAAC;IAChB,SAAS,EAAE,MAAM,CAAC;IAClB,qBAAqB,EAAE,MAAM,CAAC;IAC9B,SAAS,EAAE,MAAM,CAAC;IAClB,MAAM,EAAE,mBAAmB,CAAC;IAC5B,KAAK,EAAE,MAAM,CAAC;CACf;AAYD;4DAC4D;AAC5D,wBAAgB,OAAO,CAAC,SAAS,EAAE,MAAM,GAAG,MAAM,CAEjD;AAED;;;;;;;;;;GAUG;AACH,wBAAsB,oBAAoB,CACxC,OAAO,EAAE,eAAe,EACxB,SAAS,EAAE,MAAM,GAChB,OAAO,CAAC;IAAE,QAAQ,EAAE,MAAM,GAAG,IAAI,CAAC;IAAC,MAAM,EAAE,eAAe,GAAG,QAAQ,CAAA;CAAE,CAAC,CAa1E;AAED;;;;;;;;;;;;;;;;;;;;+EAoB+E;AAC/E,eAAO,MAAM,YAAY,28DAwCwB,CAAC;AAElD;+DAC+D;AAC/D,eAAO,MAAM,aAAa,+VAMuD,CAAC;AAElF,MAAM,WAAW,qBAAqB;IACpC,OAAO,EAAE,eAAe,CAAC;IACzB,SAAS,EAAE,MAAM,CAAC;IAClB,qEAAqE;IACrE,OAAO,CAAC,EAAE,MAAM,CAAC;IACjB,oEAAoE;IACpE,GAAG,CAAC,EAAE,MAAM,CAAC;IACb,2DAA2D;IAC3D,MAAM,CAAC,EAAE,SAAS,eAAe,EAAE,CAAC;IACpC,+DAA+D;IAC/D,GAAG,CAAC,EAAE,CAAC,IAAI,EAAE,MAAM,KAAK,IAAI,CAAC;IAC7B;sEACkE;IAClE,KAAK,CAAC,EAAE,MAAM,MAAM,CAAC;CACtB;AAED;;;;;;;;;;;;GAYG;AACH,wBAAsB,iBAAiB,CAAC,IAAI,EAAE,qBAAqB,GAAG,OAAO,CAAC,iBAAiB,CAAC,CAgF/F"}
+{"version":3,"file":"index.d.ts","sourceRoot":"","sources":["../src/index.ts"],"names":[],"mappings":"AAAA;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;GA+BG;AACH,OAAO,EAA4C,KAAK,eAAe,EAAE,MAAM,aAAa,CAAC;AAE7F,OAAO,EAAE,cAAc,EAAE,mBAAmB,EAAE,wBAAwB,EAAE,MAAM,aAAa,CAAC;AAC5F,YAAY,EAAE,eAAe,EAAE,MAAM,aAAa,CAAC;AAEnD;;;2EAG2E;AAC3E,MAAM,WAAW,eAAe;IAC9B,GAAG,CAAC,KAAK,EAAE,MAAM,EAAE,MAAM,CAAC,EAAE,MAAM,CAAC,MAAM,EAAE,OAAO,CAAC,GAAG,OAAO,CAAC;QAAE,OAAO,EAAE,KAAK,CAAC;YAAE,GAAG,CAAC,GAAG,EAAE,MAAM,GAAG,OAAO,CAAA;SAAE,CAAC,CAAA;KAAE,CAAC,CAAC;CAClH;AAED;;;;;;;4EAO4E;AAC5E,MAAM,MAAM,kBAAkB,GAAG,QAAQ,GAAG,iBAAiB,GAAG,sBAAsB,GAAG,WAAW,CAAC;AAErG,MAAM,WAAW,eAAe;IAC9B,GAAG,EAAE,MAAM,CAAC;IACZ,OAAO,EAAE,MAAM,CAAC;IAChB,MAAM,EAAE,kBAAkB,CAAC;CAC5B;AAED;;;;;;0EAM0E;AAC1E,MAAM,WAAW,mBAAmB;IAClC,QAAQ,EAAE,MAAM,CAAC;IACjB,MAAM,EAAE,MAAM,CAAC;IACf,SAAS,EAAE,MAAM,CAAC;IAClB,WAAW,EAAE,MAAM,CAAC;IACpB;;;;;;;kEAO8D;IAC9D,QAAQ,EAAE,MAAM,CAAC;CAClB;AAED,MAAM,WAAW,iBAAiB;IAChC,SAAS,EAAE,MAAM,CAAC;IAClB,OAAO,EAAE,MAAM,CAAC;IAChB,QAAQ,EAAE,MAAM,GAAG,IAAI,CAAC;IACxB,cAAc,EAAE,eAAe,GAAG,QAAQ,CAAC;IAC3C,MAAM,EAAE,eAAe,EAAE,CAAC;IAC1B,OAAO,EAAE,MAAM,CAAC;IAChB,SAAS,EAAE,MAAM,CAAC;IAClB,qBAAqB,EAAE,MAAM,CAAC;IAC9B,SAAS,EAAE,MAAM,CAAC;IAClB,MAAM,EAAE,mBAAmB,CAAC;IAC5B,KAAK,EAAE,MAAM,CAAC;CACf;AAYD;4DAC4D;AAC5D,wBAAgB,OAAO,CAAC,SAAS,EAAE,MAAM,GAAG,MAAM,CAEjD;AAED;;;;;;;;;;GAUG;AACH,wBAAsB,oBAAoB,CACxC,OAAO,EAAE,eAAe,EACxB,SAAS,EAAE,MAAM,GAChB,OAAO,CAAC;IAAE,QAAQ,EAAE,MAAM,GAAG,IAAI,CAAC;IAAC,MAAM,EAAE,eAAe,GAAG,QAAQ,CAAA;CAAE,CAAC,CAa1E;AAED;;;;;;;;;;;;;;;;;;;;;;;;;aAyBa;AACb,eAAO,MAAM,YAAY,8+DAwCwB,CAAC;AAElD;+DAC+D;AAC/D,eAAO,MAAM,aAAa,wVAMgD,CAAC;AAE3E,MAAM,WAAW,qBAAqB;IACpC,OAAO,EAAE,eAAe,CAAC;IACzB,SAAS,EAAE,MAAM,CAAC;IAClB,qEAAqE;IACrE,OAAO,CAAC,EAAE,MAAM,CAAC;IACjB,oEAAoE;IACpE,GAAG,CAAC,EAAE,MAAM,CAAC;IACb,2DAA2D;IAC3D,MAAM,CAAC,EAAE,SAAS,eAAe,EAAE,CAAC;IACpC,+DAA+D;IAC/D,GAAG,CAAC,EAAE,CAAC,IAAI,EAAE,MAAM,KAAK,IAAI,CAAC;IAC7B;sEACkE;IAClE,KAAK,CAAC,EAAE,MAAM,MAAM,CAAC;CACtB;AAED;;;;;;;;;;;;GAYG;AACH,wBAAsB,iBAAiB,CAAC,IAAI,EAAE,qBAAqB,GAAG,OAAO,CAAC,iBAAiB,CAAC,CAgF/F"}

package/payload/platform/lib/routine-templates/dist/index.js

@@ -92,13 +92,18 @@ async function resolveOwnerTimezone(session, accountId) {
  *  destination, or reset a retimed cron. Only the prompt, the name and the
  *  version move, and only when the row is behind and unedited.
  *
- *  `routineDescription` (Task 2214) is the one property ON MATCH writes on
- *  every match rather than only on the refresh branch, and that is not an
- *  exception to the rule above. It is roster-owned: the routines route projects
- *  it but its PATCH field set refuses it, so no operator edit exists for this
- *  to overwrite. Gating it on `refreshing` would leave every operator-edited
- *  row description-less permanently, because `skip-operator-edited` takes
- *  neither branch, and that row's card would show a title and nothing else. */
+ *  `description` (Task 2361) is guarded exactly as the prompt and the name are,
+ *  and that is a change of shape from Task 2214, which introduced it. It used to
+ *  be written on every match, on the grounds that it was roster-owned and the
+ *  routines PATCH refused it, so no operator edit existed for the seed to
+ *  clobber, and that gating it would leave every operator-edited row
+ *  description-less permanently because `skip-operator-edited` takes neither
+ *  branch. Both halves of that argument are gone. The routines PATCH now accepts
+ *  `description` and stamps `operatorEdited`, so an unconditional write would
+ *  destroy the operator's own caption on the next version bump; and the
+ *  scheduling plugin's one-shot `converge-routine-description` has already given
+ *  every pre-existing row a description, so no row is left holding only a
+ *  title. */
 exports.MERGE_CYPHER = `
 UNWIND $rows AS row
 OPTIONAL MATCH (pre:Event {accountId: $accountId, routineTemplate: row.key})
@@ -111,7 +116,7 @@ WITH row, existed, priorVersion, edited,
 MERGE (e:Event {accountId: $accountId, routineTemplate: row.key})
   ON CREATE SET e.eventId = randomUUID(),
                 e.name = row.name,
-                e.routineDescription = row.description,
+                e.description = row.description,
                 e.agentPrompt = row.prompt,
                 e.recurrence = row.recurrence,
                 e.routineTemplateVersion = $version,
@@ -131,7 +136,7 @@ MERGE (e:Event {accountId: $accountId, routineTemplate: row.key})
                 e.updatedAt = $now
   ON MATCH SET e.agentPrompt = CASE WHEN refreshing THEN row.prompt ELSE e.agentPrompt END,
                e.name = CASE WHEN refreshing THEN row.name ELSE e.name END,
-               e.routineDescription = row.description,
+               e.description = CASE WHEN refreshing THEN row.description ELSE e.description END,
                e.routineTemplateVersion = CASE WHEN refreshing THEN $version ELSE e.routineTemplateVersion END,
                e.updatedAt = CASE WHEN refreshing THEN $now ELSE e.updatedAt END
 RETURN row.key AS key,
@@ -148,7 +153,7 @@ WHERE e.routineTemplate IS NOT NULL
 RETURN count(e) AS actual,
        sum(CASE WHEN e.eventStatus = 'suspended' THEN 1 ELSE 0 END) AS suspended,
        sum(CASE WHEN e.nextRun IS NULL THEN 1 ELSE 0 END) AS nextRunNull,
-       sum(CASE WHEN e.routineDescription IS NULL THEN 1 ELSE 0 END) AS descNull`;
+       sum(CASE WHEN e.description IS NULL THEN 1 ELSE 0 END) AS descNull`;
 /**
  * Seed (or re-seed) the roster for one account and report what happened.
  *

package/payload/platform/lib/routine-templates/dist/index.js.map

@@ -1 +1 @@
-{"version":3,"file":"index.js","sourceRoot":"","sources":["../src/index.ts"],"names":[],"mappings":";;;AA6GA,0BAEC;AAaD,oDAgBC;AAwGD,8CAgFC;AApUD;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;GA+BG;AACH,2CAA6F;AAE7F,yCAA4F;AAAnF,2GAAA,cAAc,OAAA;AAAE,gHAAA,mBAAmB,OAAA;AAAE,qHAAA,wBAAwB,OAAA;AA+DtE;gFACgF;AAChF,SAAS,KAAK,CAAC,GAAY;IACzB,IAAI,OAAO,GAAG,KAAK,QAAQ;QAAE,OAAO,GAAG,CAAC;IACxC,IAAI,GAAG,IAAI,OAAQ,GAAmC,CAAC,QAAQ,KAAK,UAAU,EAAE,CAAC;QAC/E,OAAQ,GAAkC,CAAC,QAAQ,EAAE,CAAC;IACxD,CAAC;IACD,OAAO,MAAM,CAAC,GAAG,IAAI,CAAC,CAAC,CAAC;AAC1B,CAAC;AAED;4DAC4D;AAC5D,SAAgB,OAAO,CAAC,SAAiB;IACvC,OAAO,SAAS,CAAC,KAAK,CAAC,CAAC,EAAE,CAAC,CAAC,CAAC;AAC/B,CAAC;AAED;;;;;;;;;;GAUG;AACI,KAAK,UAAU,oBAAoB,CACxC,OAAwB,EACxB,SAAiB;IAEjB,MAAM,GAAG,GAAG,MAAM,OAAO,CAAC,GAAG,CAC3B;;;aAGS,EACT,EAAE,SAAS,EAAE,CACd,CAAC;IACF,MAAM,GAAG,GAAG,GAAG,CAAC,OAAO,CAAC,CAAC,CAAC,EAAE,GAAG,CAAC,UAAU,CAAC,CAAC;IAC5C,IAAI,OAAO,GAAG,KAAK,QAAQ,IAAI,GAAG,CAAC,IAAI,EAAE,KAAK,EAAE,EAAE,CAAC;QACjD,OAAO,EAAE,QAAQ,EAAE,GAAG,CAAC,IAAI,EAAE,EAAE,MAAM,EAAE,eAAe,EAAE,CAAC;IAC3D,CAAC;IACD,OAAO,EAAE,QAAQ,EAAE,IAAI,EAAE,MAAM,EAAE,QAAQ,EAAE,CAAC;AAC9C,CAAC;AAED;;;;;;;;;;;;;;;;;;;;+EAoB+E;AAClE,QAAA,YAAY,GAAG;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;iDAwCqB,CAAC;AAElD;+DAC+D;AAClD,QAAA,aAAa,GAAG;;;;;;iFAMoD,CAAC;AAkBlF;;;;;;;;;;;;GAYG;AACI,KAAK,UAAU,iBAAiB,CAAC,IAA2B;IACjE,MAAM,EACJ,OAAO,EACP,SAAS,EACT,OAAO,GAAG,oCAAwB,EAClC,GAAG,GAAG,IAAI,IAAI,EAAE,CAAC,WAAW,EAAE,EAC9B,MAAM,GAAG,0BAAc,EACvB,GAAG,GAAG,CAAC,IAAY,EAAE,EAAE,CAAC,OAAO,CAAC,MAAM,CAAC,KAAK,CAAC,GAAG,IAAI,IAAI,CAAC,EACzD,KAAK,GAAG,GAAG,EAAE,CAAC,IAAI,CAAC,GAAG,EAAE,GACzB,GAAG,IAAI,CAAC;IAET,MAAM,IAAI,GAAG,OAAO,CAAC,SAAS,CAAC,CAAC;IAChC,MAAM,SAAS,GAAG,KAAK,EAAE,CAAC;IAC1B,GAAG,CAAC,gCAAgC,IAAI,WAAW,MAAM,CAAC,MAAM,YAAY,OAAO,EAAE,CAAC,CAAC;IAEvF,MAAM,EAAE,QAAQ,EAAE,MAAM,EAAE,GAAG,MAAM,oBAAoB,CAAC,OAAO,EAAE,SAAS,CAAC,CAAC;IAC5E,GAAG,CAAC,sCAAsC,IAAI,OAAO,QAAQ,IAAI,MAAM,WAAW,MAAM,EAAE,CAAC,CAAC;IAE5F,MAAM,QAAQ,GAAG,MAAM,OAAO,CAAC,GAAG,CAAC,oBAAY,EAAE;QAC/C,SAAS;QACT,OAAO;QACP,QAAQ;QACR,GAAG;QACH,IAAI,EAAE,MAAM,CAAC,GAAG,CAAC,CAAC,CAAC,EAAE,EAAE,CAAC,CAAC;YACvB,GAAG,EAAE,CAAC,CAAC,GAAG;YACV,IAAI,EAAE,CAAC,CAAC,IAAI;YACZ,WAAW,EAAE,CAAC,CAAC,WAAW;YAC1B,MAAM,EAAE,CAAC,CAAC,MAAM;YAChB,UAAU,EAAE,CAAC,CAAC,UAAU;SACzB,CAAC,CAAC;KACJ,CAAC,CAAC;IAEH,MAAM,MAAM,GAAsB,QAAQ,CAAC,OAAO,CAAC,GAAG,CAAC,CAAC,GAAG,EAAE,EAAE,CAAC,CAAC;QAC/D,GAAG,EAAE,MAAM,CAAC,GAAG,CAAC,GAAG,CAAC,KAAK,CAAC,CAAC;QAC3B,OAAO,EAAE,MAAM,CAAC,GAAG,CAAC,GAAG,CAAC,SAAS,CAAC,CAAC;QACnC,MAAM,EAAE,MAAM,CAAC,GAAG,CAAC,GAAG,CAAC,QAAQ,CAAC,CAAuB;KACxD,CAAC,CAAC,CAAC;IAEJ,KAAK,MAAM,CAAC,IAAI,MAAM,EAAE,CAAC;QACvB,GAAG,CAAC,gCAAgC,IAAI,QAAQ,CAAC,CAAC,GAAG,WAAW,CAAC,CAAC,MAAM,YAAY,CAAC,CAAC,OAAO,EAAE,CAAC,CAAC;IACnG,CAAC;IAED,MAAM,SAAS,GAAG,MAAM,OAAO,CAAC,GAAG,CAAC,qBAAa,EAAE,EAAE,SAAS,EAAE,CAAC,CAAC;IAClE,MAAM,SAAS,GAAG,SAAS,CAAC,OAAO,CAAC,CAAC,CAAC,CAAC;IACvC,MAAM,MAAM,GAAwB;QAClC,QAAQ,EAAE,MAAM,CAAC,MAAM;QACvB,MAAM,EAAE,KAAK,CAAC,SAAS,EAAE,GAAG,CAAC,QAAQ,CAAC,CAAC;QACvC,SAAS,EAAE,KAAK,CAAC,SAAS,EAAE,GAAG,CAAC,WAAW,CAAC,CAAC;QAC7C,WAAW,EAAE,KAAK,CAAC,SAAS,EAAE,GAAG,CAAC,aAAa,CAAC,CAAC;QACjD,QAAQ,EAAE,KAAK,CAAC,SAAS,EAAE,GAAG,CAAC,UAAU,CAAC,CAAC;KAC5C,CAAC;IACF,GAAG,CACD,iCAAiC,IAAI,aAAa,MAAM,CAAC,QAAQ,WAAW,MAAM,CAAC,MAAM,GAAG;QAC1F,aAAa,MAAM,CAAC,SAAS,gBAAgB,MAAM,CAAC,WAAW,aAAa,MAAM,CAAC,QAAQ,EAAE,CAChG,CAAC;IAEF,MAAM,OAAO,GAAG,MAAM,CAAC,MAAM,CAAC,CAAC,CAAC,EAAE,EAAE,CAAC,CAAC,CAAC,MAAM,KAAK,QAAQ,CAAC,CAAC,MAAM,CAAC;IACnE,MAAM,SAAS,GAAG,MAAM,CAAC,MAAM,CAAC,CAAC,CAAC,EAAE,EAAE,CAAC,CAAC,CAAC,MAAM,KAAK,iBAAiB,CAAC,CAAC,MAAM,CAAC;IAC9E,MAAM,qBAAqB,GAAG,MAAM,CAAC,MAAM,CAAC,CAAC,CAAC,EAAE,EAAE,CAAC,CAAC,CAAC,MAAM,KAAK,sBAAsB,CAAC,CAAC,MAAM,CAAC;IAC/F,MAAM,SAAS,GAAG,MAAM,CAAC,MAAM,CAAC,CAAC,CAAC,EAAE,EAAE,CAAC,CAAC,CAAC,MAAM,KAAK,WAAW,CAAC,CAAC,MAAM,CAAC;IACxE,MAAM,KAAK,GAAG,KAAK,EAAE,GAAG,SAAS,CAAC;IAElC,GAAG,CACD,8BAA8B,IAAI,UAAU,KAAK,YAAY,OAAO,GAAG;QACrE,WAAW,qBAAqB,cAAc,SAAS,cAAc,SAAS,EAAE,CACnF,CAAC;IAEF,OAAO;QACL,SAAS;QACT,OAAO;QACP,QAAQ;QACR,cAAc,EAAE,MAAM;QACtB,MAAM;QACN,OAAO;QACP,SAAS;QACT,qBAAqB;QACrB,SAAS;QACT,MAAM;QACN,KAAK;KACN,CAAC;AACJ,CAAC"}
+{"version":3,"file":"index.js","sourceRoot":"","sources":["../src/index.ts"],"names":[],"mappings":";;;AA8GA,0BAEC;AAaD,oDAgBC;AA6GD,8CAgFC;AA1UD;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;GA+BG;AACH,2CAA6F;AAE7F,yCAA4F;AAAnF,2GAAA,cAAc,OAAA;AAAE,gHAAA,mBAAmB,OAAA;AAAE,qHAAA,wBAAwB,OAAA;AAgEtE;gFACgF;AAChF,SAAS,KAAK,CAAC,GAAY;IACzB,IAAI,OAAO,GAAG,KAAK,QAAQ;QAAE,OAAO,GAAG,CAAC;IACxC,IAAI,GAAG,IAAI,OAAQ,GAAmC,CAAC,QAAQ,KAAK,UAAU,EAAE,CAAC;QAC/E,OAAQ,GAAkC,CAAC,QAAQ,EAAE,CAAC;IACxD,CAAC;IACD,OAAO,MAAM,CAAC,GAAG,IAAI,CAAC,CAAC,CAAC;AAC1B,CAAC;AAED;4DAC4D;AAC5D,SAAgB,OAAO,CAAC,SAAiB;IACvC,OAAO,SAAS,CAAC,KAAK,CAAC,CAAC,EAAE,CAAC,CAAC,CAAC;AAC/B,CAAC;AAED;;;;;;;;;;GAUG;AACI,KAAK,UAAU,oBAAoB,CACxC,OAAwB,EACxB,SAAiB;IAEjB,MAAM,GAAG,GAAG,MAAM,OAAO,CAAC,GAAG,CAC3B;;;aAGS,EACT,EAAE,SAAS,EAAE,CACd,CAAC;IACF,MAAM,GAAG,GAAG,GAAG,CAAC,OAAO,CAAC,CAAC,CAAC,EAAE,GAAG,CAAC,UAAU,CAAC,CAAC;IAC5C,IAAI,OAAO,GAAG,KAAK,QAAQ,IAAI,GAAG,CAAC,IAAI,EAAE,KAAK,EAAE,EAAE,CAAC;QACjD,OAAO,EAAE,QAAQ,EAAE,GAAG,CAAC,IAAI,EAAE,EAAE,MAAM,EAAE,eAAe,EAAE,CAAC;IAC3D,CAAC;IACD,OAAO,EAAE,QAAQ,EAAE,IAAI,EAAE,MAAM,EAAE,QAAQ,EAAE,CAAC;AAC9C,CAAC;AAED;;;;;;;;;;;;;;;;;;;;;;;;;aAyBa;AACA,QAAA,YAAY,GAAG;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;iDAwCqB,CAAC;AAElD;+DAC+D;AAClD,QAAA,aAAa,GAAG;;;;;;0EAM6C,CAAC;AAkB3E;;;;;;;;;;;;GAYG;AACI,KAAK,UAAU,iBAAiB,CAAC,IAA2B;IACjE,MAAM,EACJ,OAAO,EACP,SAAS,EACT,OAAO,GAAG,oCAAwB,EAClC,GAAG,GAAG,IAAI,IAAI,EAAE,CAAC,WAAW,EAAE,EAC9B,MAAM,GAAG,0BAAc,EACvB,GAAG,GAAG,CAAC,IAAY,EAAE,EAAE,CAAC,OAAO,CAAC,MAAM,CAAC,KAAK,CAAC,GAAG,IAAI,IAAI,CAAC,EACzD,KAAK,GAAG,GAAG,EAAE,CAAC,IAAI,CAAC,GAAG,EAAE,GACzB,GAAG,IAAI,CAAC;IAET,MAAM,IAAI,GAAG,OAAO,CAAC,SAAS,CAAC,CAAC;IAChC,MAAM,SAAS,GAAG,KAAK,EAAE,CAAC;IAC1B,GAAG,CAAC,gCAAgC,IAAI,WAAW,MAAM,CAAC,MAAM,YAAY,OAAO,EAAE,CAAC,CAAC;IAEvF,MAAM,EAAE,QAAQ,EAAE,MAAM,EAAE,GAAG,MAAM,oBAAoB,CAAC,OAAO,EAAE,SAAS,CAAC,CAAC;IAC5E,GAAG,CAAC,sCAAsC,IAAI,OAAO,QAAQ,IAAI,MAAM,WAAW,MAAM,EAAE,CAAC,CAAC;IAE5F,MAAM,QAAQ,GAAG,MAAM,OAAO,CAAC,GAAG,CAAC,oBAAY,EAAE;QAC/C,SAAS;QACT,OAAO;QACP,QAAQ;QACR,GAAG;QACH,IAAI,EAAE,MAAM,CAAC,GAAG,CAAC,CAAC,CAAC,EAAE,EAAE,CAAC,CAAC;YACvB,GAAG,EAAE,CAAC,CAAC,GAAG;YACV,IAAI,EAAE,CAAC,CAAC,IAAI;YACZ,WAAW,EAAE,CAAC,CAAC,WAAW;YAC1B,MAAM,EAAE,CAAC,CAAC,MAAM;YAChB,UAAU,EAAE,CAAC,CAAC,UAAU;SACzB,CAAC,CAAC;KACJ,CAAC,CAAC;IAEH,MAAM,MAAM,GAAsB,QAAQ,CAAC,OAAO,CAAC,GAAG,CAAC,CAAC,GAAG,EAAE,EAAE,CAAC,CAAC;QAC/D,GAAG,EAAE,MAAM,CAAC,GAAG,CAAC,GAAG,CAAC,KAAK,CAAC,CAAC;QAC3B,OAAO,EAAE,MAAM,CAAC,GAAG,CAAC,GAAG,CAAC,SAAS,CAAC,CAAC;QACnC,MAAM,EAAE,MAAM,CAAC,GAAG,CAAC,GAAG,CAAC,QAAQ,CAAC,CAAuB;KACxD,CAAC,CAAC,CAAC;IAEJ,KAAK,MAAM,CAAC,IAAI,MAAM,EAAE,CAAC;QACvB,GAAG,CAAC,gCAAgC,IAAI,QAAQ,CAAC,CAAC,GAAG,WAAW,CAAC,CAAC,MAAM,YAAY,CAAC,CAAC,OAAO,EAAE,CAAC,CAAC;IACnG,CAAC;IAED,MAAM,SAAS,GAAG,MAAM,OAAO,CAAC,GAAG,CAAC,qBAAa,EAAE,EAAE,SAAS,EAAE,CAAC,CAAC;IAClE,MAAM,SAAS,GAAG,SAAS,CAAC,OAAO,CAAC,CAAC,CAAC,CAAC;IACvC,MAAM,MAAM,GAAwB;QAClC,QAAQ,EAAE,MAAM,CAAC,MAAM;QACvB,MAAM,EAAE,KAAK,CAAC,SAAS,EAAE,GAAG,CAAC,QAAQ,CAAC,CAAC;QACvC,SAAS,EAAE,KAAK,CAAC,SAAS,EAAE,GAAG,CAAC,WAAW,CAAC,CAAC;QAC7C,WAAW,EAAE,KAAK,CAAC,SAAS,EAAE,GAAG,CAAC,aAAa,CAAC,CAAC;QACjD,QAAQ,EAAE,KAAK,CAAC,SAAS,EAAE,GAAG,CAAC,UAAU,CAAC,CAAC;KAC5C,CAAC;IACF,GAAG,CACD,iCAAiC,IAAI,aAAa,MAAM,CAAC,QAAQ,WAAW,MAAM,CAAC,MAAM,GAAG;QAC1F,aAAa,MAAM,CAAC,SAAS,gBAAgB,MAAM,CAAC,WAAW,aAAa,MAAM,CAAC,QAAQ,EAAE,CAChG,CAAC;IAEF,MAAM,OAAO,GAAG,MAAM,CAAC,MAAM,CAAC,CAAC,CAAC,EAAE,EAAE,CAAC,CAAC,CAAC,MAAM,KAAK,QAAQ,CAAC,CAAC,MAAM,CAAC;IACnE,MAAM,SAAS,GAAG,MAAM,CAAC,MAAM,CAAC,CAAC,CAAC,EAAE,EAAE,CAAC,CAAC,CAAC,MAAM,KAAK,iBAAiB,CAAC,CAAC,MAAM,CAAC;IAC9E,MAAM,qBAAqB,GAAG,MAAM,CAAC,MAAM,CAAC,CAAC,CAAC,EAAE,EAAE,CAAC,CAAC,CAAC,MAAM,KAAK,sBAAsB,CAAC,CAAC,MAAM,CAAC;IAC/F,MAAM,SAAS,GAAG,MAAM,CAAC,MAAM,CAAC,CAAC,CAAC,EAAE,EAAE,CAAC,CAAC,CAAC,MAAM,KAAK,WAAW,CAAC,CAAC,MAAM,CAAC;IACxE,MAAM,KAAK,GAAG,KAAK,EAAE,GAAG,SAAS,CAAC;IAElC,GAAG,CACD,8BAA8B,IAAI,UAAU,KAAK,YAAY,OAAO,GAAG;QACrE,WAAW,qBAAqB,cAAc,SAAS,cAAc,SAAS,EAAE,CACnF,CAAC;IAEF,OAAO;QACL,SAAS;QACT,OAAO;QACP,QAAQ;QACR,cAAc,EAAE,MAAM;QACtB,MAAM;QACN,OAAO;QACP,SAAS;QACT,qBAAqB;QACrB,SAAS;QACT,MAAM;QACN,KAAK;KACN,CAAC;AACJ,CAAC"}

package/payload/platform/lib/routine-templates/dist/roster.d.ts

@@ -43,9 +43,9 @@ export interface RoutineTemplate {
     /** The title shown on `/routines`. */
     readonly name: string;
     /** One sentence under the title on the card: what this does and what it will
-     *  not do. Roster-owned and read-only — projected by the routines route but
-     *  absent from its PATCH field set, so no operator edit exists for the seed to
-     *  overwrite and the MERGE rewrites it on every match. */
+     *  not do. Seeded onto `e.description`, which the operator can then edit from
+     *  the routines page. The seed rewrites it only on a version refresh of an
+     *  unedited row, so an operator's own wording survives every upgrade. */
     readonly description: string;
     /** The proposed cron. Written ON CREATE only, never refreshed, so an operator
      *  who retimes a routine keeps their expression across upgrades. */

package/payload/platform/lib/routine-templates/dist/roster.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"roster.d.ts","sourceRoot":"","sources":["../src/roster.ts"],"names":[],"mappings":"AAAA;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;GA+BG;AAEH;;;gFAGgF;AAChF,eAAO,MAAM,wBAAwB,IAAI,CAAC;AAE1C,MAAM,WAAW,eAAe;IAC9B;;mEAE+D;IAC/D,QAAQ,CAAC,GAAG,EAAE,MAAM,CAAC;IACrB,sCAAsC;IACtC,QAAQ,CAAC,IAAI,EAAE,MAAM,CAAC;IACtB;;;8DAG0D;IAC1D,QAAQ,CAAC,WAAW,EAAE,MAAM,CAAC;IAC7B;wEACoE;IACpE,QAAQ,CAAC,UAAU,EAAE,MAAM,CAAC;IAC5B,+EAA+E;IAC/E,QAAQ,CAAC,MAAM,EAAE,MAAM,CAAC;CACzB;AAgKD;;;;;;;;iFAQiF;AACjF,eAAO,MAAM,cAAc,EAAE,SAAS,eAAe,EAyGpD,CAAC;AAEF;;6DAE6D;AAC7D,eAAO,MAAM,mBAAmB,EAAE,SAAS,MAAM,EAAqC,CAAC"}
+{"version":3,"file":"roster.d.ts","sourceRoot":"","sources":["../src/roster.ts"],"names":[],"mappings":"AAAA;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;GA+BG;AAEH;;;gFAGgF;AAChF,eAAO,MAAM,wBAAwB,IAAI,CAAC;AAE1C,MAAM,WAAW,eAAe;IAC9B;;mEAE+D;IAC/D,QAAQ,CAAC,GAAG,EAAE,MAAM,CAAC;IACrB,sCAAsC;IACtC,QAAQ,CAAC,IAAI,EAAE,MAAM,CAAC;IACtB;;;6EAGyE;IACzE,QAAQ,CAAC,WAAW,EAAE,MAAM,CAAC;IAC7B;wEACoE;IACpE,QAAQ,CAAC,UAAU,EAAE,MAAM,CAAC;IAC5B,+EAA+E;IAC/E,QAAQ,CAAC,MAAM,EAAE,MAAM,CAAC;CACzB;AAgKD;;;;;;;;iFAQiF;AACjF,eAAO,MAAM,cAAc,EAAE,SAAS,eAAe,EAyGpD,CAAC;AAEF;;6DAE6D;AAC7D,eAAO,MAAM,mBAAmB,EAAE,SAAS,MAAM,EAAqC,CAAC"}

package/payload/platform/lib/routine-templates/src/__tests__/seed.test.ts

@@ -84,7 +84,7 @@ describe("MERGE statement shape", () => {
     for (const required of [
       "eventId",
       "name",
-      "routineDescription",
+      "description",
       "agentPrompt",
       "recurrence",
       "routineTemplateVersion",
@@ -121,20 +121,24 @@ describe("MERGE statement shape", () => {
 
   it("ON MATCH touches ONLY the prompt, name, description, version and updatedAt", () => {
     expect(new Set(propertiesSetIn("ON MATCH SET"))).toEqual(
-      new Set(["agentPrompt", "name", "routineDescription", "routineTemplateVersion", "updatedAt"]),
+      new Set(["agentPrompt", "name", "description", "routineTemplateVersion", "updatedAt"]),
     );
   });
 
-  it("ON MATCH writes the description unconditionally, not only when refreshing", () => {
-    // Task 2214. The description is roster-owned and absent from the routines
-    // PATCH field set, so there is no operator edit for this to clobber.
-    // Gating it on `refreshing` would leave every operator-edited row without
-    // a description permanently: `skip-operator-edited` takes neither branch,
-    // so its card would show a title and nothing else, for ever.
+  it("ON MATCH writes the description only when refreshing, exactly as the prompt and name are", () => {
+    // The 2214 shape wrote it on every match, because gating it would have left
+    // every operator-edited row description-less for ever: skip-operator-edited
+    // takes neither branch. Task 2361's one-shot convergence removes that
+    // argument by giving every pre-existing row a description before the first
+    // guarded MERGE runs, and from here the description is operator-writable, so
+    // an unconditional write would clobber the operator's own text on every
+    // upgrade.
     const start = MERGE_CYPHER.indexOf("ON MATCH SET");
     const body = MERGE_CYPHER.slice(start, MERGE_CYPHER.indexOf("RETURN"));
-    expect(body).toContain("e.routineDescription = row.description");
-    expect(body).not.toContain("e.routineDescription = CASE");
+    expect(body).toContain(
+      "e.description = CASE WHEN refreshing THEN row.description ELSE e.description END",
+    );
+    expect(body).not.toContain("e.routineDescription");
   });
 
   it("verify counts the descriptions the graph actually holds", () => {
@@ -142,7 +146,7 @@ describe("MERGE statement shape", () => {
     // MERGE succeeds, every action reads `refresh-version`, and the property
     // is simply absent. `expected` and `actual` agree in that state.
     expect(VERIFY_CYPHER).toContain(
-      "sum(CASE WHEN e.routineDescription IS NULL THEN 1 ELSE 0 END) AS descNull",
+      "sum(CASE WHEN e.description IS NULL THEN 1 ELSE 0 END) AS descNull",
     );
   });
 

package/payload/platform/lib/routine-templates/src/index.ts

@@ -71,13 +71,14 @@ export interface RoutineVerifyCounts {
   actual: number;
   suspended: number;
   nextRunNull: number;
-  /** Task 2214 — roster rows holding no description. Zero is the healthy value
-   *  on every account, not only a fresh one: `VERIFY_CYPHER` matches
-   *  `routineTemplate IS NOT NULL`, so operator-created routines are outside
-   *  its scope and cannot contribute, and `ON MATCH` writes the description
-   *  unconditionally, so an operator-edited row is covered too. Anything above
-   *  zero while `expected === actual` is a MERGE that wrote no description —
-   *  the one failure here that emits no error and does not reproduce. */
+  /** Roster rows holding no description. Zero is the healthy value on every
+   *  account, not only a fresh one: `VERIFY_CYPHER` matches `routineTemplate IS
+   *  NOT NULL`, so operator-created routines are outside its scope and cannot
+   *  contribute. Anything above zero while `expected === actual` is a MERGE that
+   *  wrote no description on create, or a row whose key left the roster before
+   *  the convergence ran — the one failure here that emits no error and does not
+   *  reproduce. `[schedule-audit] op=description-coverage` is the wider twin,
+   *  over every automation row rather than the roster alone. */
   descNull: number;
 }
 
@@ -154,13 +155,18 @@ export async function resolveOwnerTimezone(
  *  destination, or reset a retimed cron. Only the prompt, the name and the
  *  version move, and only when the row is behind and unedited.
  *
- *  `routineDescription` (Task 2214) is the one property ON MATCH writes on
- *  every match rather than only on the refresh branch, and that is not an
- *  exception to the rule above. It is roster-owned: the routines route projects
- *  it but its PATCH field set refuses it, so no operator edit exists for this
- *  to overwrite. Gating it on `refreshing` would leave every operator-edited
- *  row description-less permanently, because `skip-operator-edited` takes
- *  neither branch, and that row's card would show a title and nothing else. */
+ *  `description` (Task 2361) is guarded exactly as the prompt and the name are,
+ *  and that is a change of shape from Task 2214, which introduced it. It used to
+ *  be written on every match, on the grounds that it was roster-owned and the
+ *  routines PATCH refused it, so no operator edit existed for the seed to
+ *  clobber, and that gating it would leave every operator-edited row
+ *  description-less permanently because `skip-operator-edited` takes neither
+ *  branch. Both halves of that argument are gone. The routines PATCH now accepts
+ *  `description` and stamps `operatorEdited`, so an unconditional write would
+ *  destroy the operator's own caption on the next version bump; and the
+ *  scheduling plugin's one-shot `converge-routine-description` has already given
+ *  every pre-existing row a description, so no row is left holding only a
+ *  title. */
 export const MERGE_CYPHER = `
 UNWIND $rows AS row
 OPTIONAL MATCH (pre:Event {accountId: $accountId, routineTemplate: row.key})
@@ -173,7 +179,7 @@ WITH row, existed, priorVersion, edited,
 MERGE (e:Event {accountId: $accountId, routineTemplate: row.key})
   ON CREATE SET e.eventId = randomUUID(),
                 e.name = row.name,
-                e.routineDescription = row.description,
+                e.description = row.description,
                 e.agentPrompt = row.prompt,
                 e.recurrence = row.recurrence,
                 e.routineTemplateVersion = $version,
@@ -193,7 +199,7 @@ MERGE (e:Event {accountId: $accountId, routineTemplate: row.key})
                 e.updatedAt = $now
   ON MATCH SET e.agentPrompt = CASE WHEN refreshing THEN row.prompt ELSE e.agentPrompt END,
                e.name = CASE WHEN refreshing THEN row.name ELSE e.name END,
-               e.routineDescription = row.description,
+               e.description = CASE WHEN refreshing THEN row.description ELSE e.description END,
                e.routineTemplateVersion = CASE WHEN refreshing THEN $version ELSE e.routineTemplateVersion END,
                e.updatedAt = CASE WHEN refreshing THEN $now ELSE e.updatedAt END
 RETURN row.key AS key,
@@ -211,7 +217,7 @@ WHERE e.routineTemplate IS NOT NULL
 RETURN count(e) AS actual,
        sum(CASE WHEN e.eventStatus = 'suspended' THEN 1 ELSE 0 END) AS suspended,
        sum(CASE WHEN e.nextRun IS NULL THEN 1 ELSE 0 END) AS nextRunNull,
-       sum(CASE WHEN e.routineDescription IS NULL THEN 1 ELSE 0 END) AS descNull`;
+       sum(CASE WHEN e.description IS NULL THEN 1 ELSE 0 END) AS descNull`;
 
 export interface SeedRoutineRosterArgs {
   session: SeedSessionLike;

package/payload/platform/lib/routine-templates/src/roster.ts

@@ -45,9 +45,9 @@ export interface RoutineTemplate {
   /** The title shown on `/routines`. */
   readonly name: string;
   /** One sentence under the title on the card: what this does and what it will
-   *  not do. Roster-owned and read-only — projected by the routines route but
-   *  absent from its PATCH field set, so no operator edit exists for the seed to
-   *  overwrite and the MERGE rewrites it on every match. */
+   *  not do. Seeded onto `e.description`, which the operator can then edit from
+   *  the routines page. The seed rewrites it only on a version refresh of an
+   *  unedited row, so an operator's own wording survives every upgrade. */
   readonly description: string;
   /** The proposed cron. Written ON CREATE only, never refreshed, so an operator
    *  who retimes a routine keeps their expression across upgrades. */

package/payload/platform/plugins/admin/mcp/dist/index.js

@@ -26,6 +26,7 @@ import { resolvePublicHostname } from "./lib/public-hostname.js";
 import { resolveAdminHostname } from "./lib/admin-hostname.js";
 import { publishSite } from "./tools/publish-site.js";
 import { createAgent, updateAgent, projectAgentViaPlatform } from "./tools/agent-write.js";
+import { checkDefaultAgent } from "./tools/account-default-agent.js";
 import { resolveCapabilitiesHere } from "./tools/capabilities-here.js";
 import { resolveIngestToken } from "./tools/linkedin-ingest-token.js";
 import { startCpuTriage, readCpuTriage } from "./tools/cpu-triage.js";
@@ -787,19 +788,30 @@ eagerTool(server, "account-update", "Update a user-configurable setting in accou
         thinkingView: ["default", "expanded", "collapsed"],
         effort: ["low", "medium", "high", "max", "auto"],
     };
-    // defaultAgent: non-empty slug must reference an existing agent; empty string clears the field
+    // defaultAgent: non-empty slug must be slug-shaped AND reference an existing
+    // agent; empty string clears the field.
+    //
+    // Task 2347 — shape before existence. The stored value is read back by
+    // resolveDefaultAgentSlug and handed to readAgentFile, which interpolates
+    // it into a path; a traversal value's config.json exists on any
+    // multi-account install, so existence alone stored a path that led out of
+    // the account directory.
+    // `defaultAgentSlug` is the canonical value to store: the verdict's trimmed
+    // slug, empty for the clear operation. Storing the raw `value` instead
+    // would let `" sales "` be accepted here and then refused by the read
+    // door's shape rule, which sees the untrimmed string.
+    let defaultAgentSlug = "";
     if (field === "defaultAgent") {
-        const slug = value.trim();
-        if (slug) {
-            const agentConfigPath = join(getAccountDir(), "agents", slug, "config.json");
-            if (!existsSync(agentConfigPath)) {
-                return {
-                    content: [{ type: "text", text: `Agent '${slug}' not found — no config.json in agents/${slug}/` }],
-                    isError: true,
-                };
-            }
+        const verdict = checkDefaultAgent(getAccountDir(), value);
+        if (!verdict.ok) {
+            const slug = value.trim();
+            console.error(`[admin:account-update] rejected field=defaultAgent value="${value}" reason=${verdict.reason}`);
+            const text = verdict.reason === "shape"
+                ? `Invalid agent slug "${slug}" — must be lowercase, start with a letter, and be 3 to 50 characters with no path separators.`
+                : `Agent '${slug}' not found — no config.json in agents/${slug}/`;
+            return { content: [{ type: "text", text }], isError: true };
         }
-        // empty slug is valid — handled below as a delete operation
+        defaultAgentSlug = verdict.slug;
     }
     // Model fields accept any non-empty string (model IDs change frequently)
     const FREE_FORM_FIELDS = ["adminModel", "publicModel"];
@@ -822,22 +834,26 @@ eagerTool(server, "account-update", "Update a user-configurable setting in accou
         const configPath = join(getAccountDir(), "account.json");
         const config = await readAccountConfig();
         const oldDefault = field === "defaultAgent" ? config.defaultAgent ?? "(none)" : undefined;
-        if (field === "defaultAgent" && !value.trim()) {
-            delete config.defaultAgent;
+        if (field === "defaultAgent") {
+            if (defaultAgentSlug)
+                config.defaultAgent = defaultAgentSlug;
+            else
+                delete config.defaultAgent;
         }
         else {
             config[field] = value;
         }
+        const writtenValue = field === "defaultAgent" ? defaultAgentSlug : value;
         await writeFile(configPath, JSON.stringify(config, null, 2) + "\n", "utf-8");
-        console.error(`[admin:account-update] wrote field=${field} value="${value}" path=${configPath}`);
+        console.error(`[admin:account-update] wrote field=${field} value="${writtenValue}" path=${configPath}`);
         if (field === "defaultAgent") {
-            if (!value.trim()) {
+            if (!defaultAgentSlug) {
                 return {
                     content: [{ type: "text", text: `Default agent cleared (was "${oldDefault}"). Visitors to the root URL will see the first available agent, or a "no agents" message if none exist. Takes effect on next page load.` }],
                 };
             }
             return {
-                content: [{ type: "text", text: `Default agent changed from "${oldDefault}" to "${value}". Takes effect on next page load.` }],
+                content: [{ type: "text", text: `Default agent changed from "${oldDefault}" to "${defaultAgentSlug}". Takes effect on next page load.` }],
             };
         }
         return {