@rubytech/create-sitedesk-code 0.1.529 → 0.1.531

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

@@ -4,15 +4,17 @@
  *
  * A fresh install used to ship zero routines, so the only working examples
  * lived in one customer's graph and every operator wrote the prompts from
- * scratch. These eleven are the catalogue that ships with the platform instead.
+ * scratch. These thirteen are the catalogue that ships with the platform
+ * instead.
  *
  * Two properties make a routine shippable, and both are load-bearing:
  *
  *   1. It resolves its own surfaces at run time. Every prompt opens on
  *      `capabilities-here`, which reports the account's live install set, and
  *      ends the turn without replying when the surface it needs is absent. That
- *      is what makes the same eleven rows correct on an estate agency, a web studio
- *      and a field-service install rather than a template someone must edit.
+ *      is what makes the same thirteen rows correct on an estate agency, a web
+ *      studio and a field-service install rather than a template someone must
+ *      edit.
  *
  *   2. It names nobody. No prompt carries a person's name, an email address, a
  *      phone number, a hostname or a database name. `scripts/check-routine-
@@ -35,7 +37,7 @@ exports.ROUTINE_ROSTER_KEYS = exports.ROUTINE_ROSTER = exports.ROUTINE_TEMPLATE_
  *  routine's prompt only when the stored `routineTemplateVersion` is BELOW this
  *  and no operator has edited the row, so an unbumped edit ships to new installs
  *  and never reaches existing ones — bump it in the same commit as the edit. */
-exports.ROUTINE_TEMPLATE_VERSION = 1;
+exports.ROUTINE_TEMPLATE_VERSION = 2;
 const START_OF_DAY = `Prepare the operator's morning briefing and reply with it as a single message.
 
 Call capabilities-here first and work only with the plugins it reports as enabled. Nothing below is mandatory: each part is included when its surface is present and skipped in silence when it is not.
@@ -159,21 +161,129 @@ If every link is connected, end the turn without replying. A message on a health
 When a link is down, reply with one message naming the number it is paired to, the exact error it last reported, and how long it has been down. Say plainly whether the platform is still retrying it or has stopped. A link that was logged out, or replaced by another device, will not come back on its own: re-pairing needs someone holding the phone, so say that rather than implying it will recover by itself.
 
 Never disconnect a link and never begin a re-pair from here. This routine reports; the remedy is a person's decision.`;
-/** The eleven. Order is the display order on `/routines` before the operator
+const GRAPH_HEALTH = `Report on the state of this account's graph and on what its maintenance sweep last found. This routine reads. It performs no graph writes at all.
+
+Call capabilities-here first. If the memory plugin is not enabled, end the turn without replying.
+
+Read the most recent maintenance report with memory-report-read-latest: it carries the run's per-phase counts and the time it ran. Read the two review queues with memory-review-queue. Orphan candidates are nodes the sweep could not attach to anything; citation proposals are timeline claims still waiting for a source.
+
+Report four things: when the sweep last ran, how many orphan candidates are waiting, how large the citation backlog is, and how much compiled truth the last run found stale. A queue that has grown since the previous run matters more than its absolute size, so say which direction it is moving.
+
+Never call memory-dream-run from here. The sweep already runs on its own hourly cadence and decides for itself whether an account is due; a second writer started from a scheduled turn would race it, and the surviving report would be whichever run finished last. Reading the sweep's output is the whole of this routine's authority.
+
+If both queues are empty and the sweep ran within the past day, end the turn without replying. Nothing is wrong and there is nothing to decide.`;
+const FILESYSTEM_HEALTH = `Report what has accumulated under this account's own directory, so growth is visible before it becomes a problem. This routine deletes nothing.
+
+Call capabilities-here first. If the filesystem plugin is not enabled, end the turn without replying.
+
+Walk the account directory with file-list, starting at its root and descending only where something is already large. Use file-glob to count the things that accumulate rather than get cleaned up: uploaded files, session transcripts, generated documents, logs, and anything a plugin writes once per run.
+
+Report the few paths worth a person's attention, each with what is in it and roughly how much. Name every path relative to the account directory, never as an absolute one. Judgement is the point: one directory holding thousands of files is worth naming, and a hundred small ones spread evenly are not.
+
+There is no deletion tool on this surface and this routine must not ask for one. Report what grew and leave the decision with the operator: a routine that removed files on a schedule would eventually remove the one that mattered, and nothing would say which.
+
+If nothing has grown materially since the last run, end the turn without replying.`;
+/** The thirteen. Order is the display order on `/routines` before the operator
  *  retimes anything: the daily rhythm first, then the reconciles, then the
- *  weekly, then the standing check. */
+ *  weekly, then the standing check, then the two health reports.
+ *
+ *  `description` is what the card shows under the title. It says what the row
+ *  does and, where the row is deliberately narrower than its name suggests,
+ *  what it will not do — the send it never makes, the write it never performs,
+ *  the file it never deletes. That second half is the part an operator cannot
+ *  infer from a title, and it is why the sentence is not just the name again. */
 exports.ROUTINE_ROSTER = [
-    { key: "start-of-day", name: "Start of day briefing", recurrence: "30 7 * * 1-5", prompt: START_OF_DAY },
-    { key: "inbound-check", name: "Working-hours inbound check", recurrence: "*/30 9-17 * * 1-5", prompt: INBOUND_CHECK },
-    { key: "end-of-day", name: "End of day wrap-up", recurrence: "0 18 * * 1-5", prompt: END_OF_DAY },
-    { key: "appointment-reminder", name: "Appointment reminders", recurrence: "0 * * * *", prompt: APPOINTMENT_REMINDER },
-    { key: "commitment-chase", name: "Commitment chase", recurrence: "0 10 * * 2", prompt: COMMITMENT_CHASE },
-    { key: "calendar-reconcile", name: "Calendar reconcile", recurrence: "*/15 * * * *", prompt: CALENDAR_RECONCILE },
-    { key: "contact-reconcile", name: "Contact reconcile", recurrence: "45 6 * * 1-5", prompt: CONTACT_RECONCILE },
-    { key: "crm-reconcile", name: "Operational system reconcile", recurrence: "0 3 * * *", prompt: CRM_RECONCILE },
-    { key: "finance-reconcile", name: "Finance reconcile", recurrence: "0 4 * * 1", prompt: FINANCE_RECONCILE },
-    { key: "weekly-digest", name: "Weekly digest", recurrence: "0 16 * * 5", prompt: WEEKLY_DIGEST },
-    { key: "channel-link-check", name: "Channel link check", recurrence: "*/15 * * * *", prompt: CHANNEL_LINK_CHECK },
+    {
+        key: "start-of-day",
+        name: "Start of day briefing",
+        description: "Sends one morning message covering what needs a decision, what is time-bound and what changed overnight; stays silent when there is nothing worth saying.",
+        recurrence: "30 7 * * 1-5",
+        prompt: START_OF_DAY,
+    },
+    {
+        key: "inbound-check",
+        name: "Working-hours inbound check",
+        description: "Triages mail that arrived since the last check and writes replies for you to approve; it never sends anything itself.",
+        recurrence: "*/30 9-17 * * 1-5",
+        prompt: INBOUND_CHECK,
+    },
+    {
+        key: "end-of-day",
+        name: "End of day wrap-up",
+        description: "Closes the day with your unsent drafts, overdue work and tomorrow's diary; says nothing when all three are empty.",
+        recurrence: "0 18 * * 1-5",
+        prompt: END_OF_DAY,
+    },
+    {
+        key: "appointment-reminder",
+        name: "Appointment reminders",
+        description: "Reminds each attendee once before a booked appointment, on a route they already use; it never opens a conversation with a stranger.",
+        recurrence: "0 * * * *",
+        prompt: APPOINTMENT_REMINDER,
+    },
+    {
+        key: "commitment-chase",
+        name: "Commitment chase",
+        description: "Surfaces promises the business made and did not keep, and promises made to it that nobody kept; it reports them rather than chasing them.",
+        recurrence: "0 10 * * 2",
+        prompt: COMMITMENT_CHASE,
+    },
+    {
+        key: "calendar-reconcile",
+        name: "Calendar reconcile",
+        description: "Reconciles the provider calendar against the appointments held in the graph, writing what is missing and marking what was cancelled; a clean run says nothing.",
+        recurrence: "*/15 * * * *",
+        prompt: CALENDAR_RECONCILE,
+    },
+    {
+        key: "contact-reconcile",
+        name: "Contact reconcile",
+        description: "Reconciles the people you correspond with against the people on record, creating and updating contacts; it never merges two records that might be two people.",
+        recurrence: "45 6 * * 1-5",
+        prompt: CONTACT_RECONCILE,
+    },
+    {
+        key: "crm-reconcile",
+        name: "Operational system reconcile",
+        description: "Brings across what changed in whichever operational system this business runs; it reads only and never writes back to that system.",
+        recurrence: "0 3 * * *",
+        prompt: CRM_RECONCILE,
+    },
+    {
+        key: "finance-reconcile",
+        name: "Finance reconcile",
+        description: "Reconciles the accounting system against the internal ledger and reports what does not agree; it never adjusts an entry to make the two sides match.",
+        recurrence: "0 4 * * 1",
+        prompt: FINANCE_RECONCILE,
+    },
+    {
+        key: "weekly-digest",
+        name: "Weekly digest",
+        description: "Writes the week in three paragraphs at most: what changed, what is stuck, and what next week turns on.",
+        recurrence: "0 16 * * 5",
+        prompt: WEEKLY_DIGEST,
+    },
+    {
+        key: "channel-link-check",
+        name: "Channel link check",
+        description: "Reports a messaging link that has gone down and whether it can recover on its own; it never disconnects a link or starts a re-pair.",
+        recurrence: "*/15 * * * *",
+        prompt: CHANNEL_LINK_CHECK,
+    },
+    {
+        key: "graph-health",
+        name: "Graph health report",
+        description: "Reports what the hourly graph sweep last did and what is waiting for review; it reads only and never runs the sweep itself.",
+        recurrence: "0 9 * * 1",
+        prompt: GRAPH_HEALTH,
+    },
+    {
+        key: "filesystem-health",
+        name: "Filesystem health report",
+        description: "Reports what has grown under the account directory and which paths are worth attention; it deletes nothing.",
+        recurrence: "0 9 * * 1",
+        prompt: FILESYSTEM_HEALTH,
+    },
 ];
 /** Every roster key, in roster order. The scheduling heartbeat's `op=roster`
  *  drift check reconciles this expected set against what is actually in the

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

@@ -1 +1 @@
-{"version":3,"file":"roster.js","sourceRoot":"","sources":["../src/roster.ts"],"names":[],"mappings":";AAAA;;;;;;;;;;;;;;;;;;;;;;;;;;;;;GA6BG;;;AAEH;;;gFAGgF;AACnE,QAAA,wBAAwB,GAAG,CAAC,CAAC;AAgB1C,MAAM,YAAY,GAAG;;;;;;;;;;;4OAWuN,CAAC;AAE7O,MAAM,aAAa,GAAG;;;;;;;;;;2NAUqM,CAAC;AAE5N,MAAM,UAAU,GAAG;;;;;;;;;;gGAU6E,CAAC;AAEjG,MAAM,oBAAoB,GAAG;;;;;;;;;;gJAUmH,CAAC;AAEjJ,MAAM,gBAAgB,GAAG;;;;;;;;;;;4DAWmC,CAAC;AAE7D,MAAM,kBAAkB,GAAG;;;;;;;;;;8NAUmM,CAAC;AAE/N,MAAM,iBAAiB,GAAG;;;;;;;;;;4JAUkI,CAAC;AAE7J,MAAM,aAAa,GAAG;;;;;;;;2HAQqG,CAAC;AAE5H,MAAM,iBAAiB,GAAG;;;;;;;;;;gHAUsF,CAAC;AAEjH,MAAM,aAAa,GAAG;;;;;;;;;;;;gEAY0C,CAAC;AAEjE,MAAM,kBAAkB,GAAG;;;;;;;;;;sHAU2F,CAAC;AAEvH;;uCAEuC;AAC1B,QAAA,cAAc,GAA+B;IACxD,EAAE,GAAG,EAAE,cAAc,EAAE,IAAI,EAAE,uBAAuB,EAAE,UAAU,EAAE,cAAc,EAAE,MAAM,EAAE,YAAY,EAAE;IACxG,EAAE,GAAG,EAAE,eAAe,EAAE,IAAI,EAAE,6BAA6B,EAAE,UAAU,EAAE,mBAAmB,EAAE,MAAM,EAAE,aAAa,EAAE;IACrH,EAAE,GAAG,EAAE,YAAY,EAAE,IAAI,EAAE,oBAAoB,EAAE,UAAU,EAAE,cAAc,EAAE,MAAM,EAAE,UAAU,EAAE;IACjG,EAAE,GAAG,EAAE,sBAAsB,EAAE,IAAI,EAAE,uBAAuB,EAAE,UAAU,EAAE,WAAW,EAAE,MAAM,EAAE,oBAAoB,EAAE;IACrH,EAAE,GAAG,EAAE,kBAAkB,EAAE,IAAI,EAAE,kBAAkB,EAAE,UAAU,EAAE,YAAY,EAAE,MAAM,EAAE,gBAAgB,EAAE;IACzG,EAAE,GAAG,EAAE,oBAAoB,EAAE,IAAI,EAAE,oBAAoB,EAAE,UAAU,EAAE,cAAc,EAAE,MAAM,EAAE,kBAAkB,EAAE;IACjH,EAAE,GAAG,EAAE,mBAAmB,EAAE,IAAI,EAAE,mBAAmB,EAAE,UAAU,EAAE,cAAc,EAAE,MAAM,EAAE,iBAAiB,EAAE;IAC9G,EAAE,GAAG,EAAE,eAAe,EAAE,IAAI,EAAE,8BAA8B,EAAE,UAAU,EAAE,WAAW,EAAE,MAAM,EAAE,aAAa,EAAE;IAC9G,EAAE,GAAG,EAAE,mBAAmB,EAAE,IAAI,EAAE,mBAAmB,EAAE,UAAU,EAAE,WAAW,EAAE,MAAM,EAAE,iBAAiB,EAAE;IAC3G,EAAE,GAAG,EAAE,eAAe,EAAE,IAAI,EAAE,eAAe,EAAE,UAAU,EAAE,YAAY,EAAE,MAAM,EAAE,aAAa,EAAE;IAChG,EAAE,GAAG,EAAE,oBAAoB,EAAE,IAAI,EAAE,oBAAoB,EAAE,UAAU,EAAE,cAAc,EAAE,MAAM,EAAE,kBAAkB,EAAE;CAClH,CAAC;AAEF;;6DAE6D;AAChD,QAAA,mBAAmB,GAAsB,sBAAc,CAAC,GAAG,CAAC,CAAC,CAAC,EAAE,EAAE,CAAC,CAAC,CAAC,GAAG,CAAC,CAAC"}
+{"version":3,"file":"roster.js","sourceRoot":"","sources":["../src/roster.ts"],"names":[],"mappings":";AAAA;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;GA+BG;;;AAEH;;;gFAGgF;AACnE,QAAA,wBAAwB,GAAG,CAAC,CAAC;AAqB1C,MAAM,YAAY,GAAG;;;;;;;;;;;4OAWuN,CAAC;AAE7O,MAAM,aAAa,GAAG;;;;;;;;;;2NAUqM,CAAC;AAE5N,MAAM,UAAU,GAAG;;;;;;;;;;gGAU6E,CAAC;AAEjG,MAAM,oBAAoB,GAAG;;;;;;;;;;gJAUmH,CAAC;AAEjJ,MAAM,gBAAgB,GAAG;;;;;;;;;;;4DAWmC,CAAC;AAE7D,MAAM,kBAAkB,GAAG;;;;;;;;;;8NAUmM,CAAC;AAE/N,MAAM,iBAAiB,GAAG;;;;;;;;;;4JAUkI,CAAC;AAE7J,MAAM,aAAa,GAAG;;;;;;;;2HAQqG,CAAC;AAE5H,MAAM,iBAAiB,GAAG;;;;;;;;;;gHAUsF,CAAC;AAEjH,MAAM,aAAa,GAAG;;;;;;;;;;;;gEAY0C,CAAC;AAEjE,MAAM,kBAAkB,GAAG;;;;;;;;;;sHAU2F,CAAC;AAEvH,MAAM,YAAY,GAAG;;;;;;;;;;gJAU2H,CAAC;AAEjJ,MAAM,iBAAiB,GAAG;;;;;;;;;;mFAUyD,CAAC;AAEpF;;;;;;;;iFAQiF;AACpE,QAAA,cAAc,GAA+B;IACxD;QACE,GAAG,EAAE,cAAc;QACnB,IAAI,EAAE,uBAAuB;QAC7B,WAAW,EACT,2JAA2J;QAC7J,UAAU,EAAE,cAAc;QAC1B,MAAM,EAAE,YAAY;KACrB;IACD;QACE,GAAG,EAAE,eAAe;QACpB,IAAI,EAAE,6BAA6B;QACnC,WAAW,EACT,uHAAuH;QACzH,UAAU,EAAE,mBAAmB;QAC/B,MAAM,EAAE,aAAa;KACtB;IACD;QACE,GAAG,EAAE,YAAY;QACjB,IAAI,EAAE,oBAAoB;QAC1B,WAAW,EACT,mHAAmH;QACrH,UAAU,EAAE,cAAc;QAC1B,MAAM,EAAE,UAAU;KACnB;IACD;QACE,GAAG,EAAE,sBAAsB;QAC3B,IAAI,EAAE,uBAAuB;QAC7B,WAAW,EACT,qIAAqI;QACvI,UAAU,EAAE,WAAW;QACvB,MAAM,EAAE,oBAAoB;KAC7B;IACD;QACE,GAAG,EAAE,kBAAkB;QACvB,IAAI,EAAE,kBAAkB;QACxB,WAAW,EACT,2IAA2I;QAC7I,UAAU,EAAE,YAAY;QACxB,MAAM,EAAE,gBAAgB;KACzB;IACD;QACE,GAAG,EAAE,oBAAoB;QACzB,IAAI,EAAE,oBAAoB;QAC1B,WAAW,EACT,gKAAgK;QAClK,UAAU,EAAE,cAAc;QAC1B,MAAM,EAAE,kBAAkB;KAC3B;IACD;QACE,GAAG,EAAE,mBAAmB;QACxB,IAAI,EAAE,mBAAmB;QACzB,WAAW,EACT,+JAA+J;QACjK,UAAU,EAAE,cAAc;QAC1B,MAAM,EAAE,iBAAiB;KAC1B;IACD;QACE,GAAG,EAAE,eAAe;QACpB,IAAI,EAAE,8BAA8B;QACpC,WAAW,EACT,oIAAoI;QACtI,UAAU,EAAE,WAAW;QACvB,MAAM,EAAE,aAAa;KACtB;IACD;QACE,GAAG,EAAE,mBAAmB;QACxB,IAAI,EAAE,mBAAmB;QACzB,WAAW,EACT,sJAAsJ;QACxJ,UAAU,EAAE,WAAW;QACvB,MAAM,EAAE,iBAAiB;KAC1B;IACD;QACE,GAAG,EAAE,eAAe;QACpB,IAAI,EAAE,eAAe;QACrB,WAAW,EACT,wGAAwG;QAC1G,UAAU,EAAE,YAAY;QACxB,MAAM,EAAE,aAAa;KACtB;IACD;QACE,GAAG,EAAE,oBAAoB;QACzB,IAAI,EAAE,oBAAoB;QAC1B,WAAW,EACT,qIAAqI;QACvI,UAAU,EAAE,cAAc;QAC1B,MAAM,EAAE,kBAAkB;KAC3B;IACD;QACE,GAAG,EAAE,cAAc;QACnB,IAAI,EAAE,qBAAqB;QAC3B,WAAW,EACT,6HAA6H;QAC/H,UAAU,EAAE,WAAW;QACvB,MAAM,EAAE,YAAY;KACrB;IACD;QACE,GAAG,EAAE,mBAAmB;QACxB,IAAI,EAAE,0BAA0B;QAChC,WAAW,EACT,6GAA6G;QAC/G,UAAU,EAAE,WAAW;QACvB,MAAM,EAAE,iBAAiB;KAC1B;CACF,CAAC;AAEF;;6DAE6D;AAChD,QAAA,mBAAmB,GAAsB,sBAAc,CAAC,GAAG,CAAC,CAAC,CAAC,EAAE,EAAE,CAAC,CAAC,CAAC,GAAG,CAAC,CAAC"}

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

@@ -37,6 +37,11 @@ const CANONICAL_TOOL_NAMES: ReadonlySet<string> = (() => {
 function citedToolCandidates(prompt: string): string[] {
   const prefixes = [
     "email-",
+    // `file-` is here because the filesystem routine is the first row to cite
+    // the filesystem plugin. Without the prefix its two tools are collected by
+    // nothing, and the registry assertion below passes over that prompt without
+    // ever examining it — a check that reports clean while checking nothing.
+    "file-",
     "outlook-",
     "google-",
     "schedule-",
@@ -60,11 +65,11 @@ function citedToolCandidates(prompt: string): string[] {
 }
 
 describe("routine roster shape", () => {
-  it("ships exactly eleven routines", () => {
-    expect(ROUTINE_ROSTER).toHaveLength(11);
+  it("ships exactly thirteen routines", () => {
+    expect(ROUTINE_ROSTER).toHaveLength(13);
   });
 
-  it("has the eleven keys the task and the drift check agree on", () => {
+  it("has the thirteen keys the task and the drift check agree on", () => {
     expect([...ROUTINE_ROSTER_KEYS].sort()).toEqual(
       [
         "appointment-reminder",
@@ -74,7 +79,9 @@ describe("routine roster shape", () => {
         "contact-reconcile",
         "crm-reconcile",
         "end-of-day",
+        "filesystem-health",
         "finance-reconcile",
+        "graph-health",
         "inbound-check",
         "start-of-day",
         "weekly-digest",
@@ -104,6 +111,42 @@ describe("routine roster shape", () => {
   });
 });
 
+describe("routine descriptions", () => {
+  // The card renders this and nothing else about what the routine does, so an
+  // empty one is a card that says only its title — the state Task 2214 exists
+  // to end. Uniqueness matters because two rows sharing a sentence means one of
+  // them was copied and never rewritten.
+  it("every routine carries a non-empty description", () => {
+    for (const r of ROUTINE_ROSTER) {
+      expect(r.description.trim(), `${r.key} description`).not.toBe("");
+    }
+  });
+
+  it("descriptions are unique", () => {
+    const descriptions = ROUTINE_ROSTER.map((r) => r.description);
+    expect(new Set(descriptions).size).toBe(descriptions.length);
+  });
+
+  it("each description is one sentence under 200 characters", () => {
+    for (const r of ROUTINE_ROSTER) {
+      expect(r.description.length, `${r.key} description length`).toBeLessThan(200);
+      expect(r.description.endsWith("."), `${r.key} must end in a full stop`).toBe(true);
+      // A second sentence wraps to a second line on a card sized for one.
+      expect(r.description.slice(0, -1), `${r.key} must be one sentence`).not.toMatch(/\.\s/);
+    }
+  });
+
+  it("no prompt cites the graph routine's key as though it were a tool", () => {
+    // citedToolCandidates collects `graph-`-prefixed tokens and checks them
+    // against the canonical registry. `graph-health` is a MERGE key, not a
+    // tool, so a prompt that names itself fails the registry assertion with a
+    // message pointing at the registry rather than at this mistake.
+    for (const r of ROUTINE_ROSTER) {
+      expect(r.prompt, `${r.key} must not name the graph routine's key`).not.toContain("graph-health");
+    }
+  });
+});
+
 describe("routine prompts resolve their own surfaces", () => {
   it("every prompt opens on capabilities-here", () => {
     for (const r of ROUTINE_ROSTER) {

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

@@ -34,7 +34,7 @@ interface FakeCall {
 function fakeSession(opts: {
   timezone?: { timezone: string; timezoneSource: string } | null;
   merges: Array<{ key: string; eventId: string; action: RoutineMergeAction }>;
-  verify: { actual: number; suspended: number; nextRunNull: number };
+  verify: { actual: number; suspended: number; nextRunNull: number; descNull: number };
 }): { session: SeedSessionLike; calls: FakeCall[] } {
   const calls: FakeCall[] = [];
   const rec = (obj: Record<string, unknown>) => ({ get: (k: string) => obj[k] });
@@ -65,7 +65,7 @@ const allCreated = ROUTINE_ROSTER.map((r) => ({
   action: "create" as RoutineMergeAction,
 }));
 
-const cleanVerify = { actual: 10, suspended: 10, nextRunNull: 10 };
+const cleanVerify = { actual: 10, suspended: 10, nextRunNull: 10, descNull: 0 };
 
 describe("MERGE statement shape", () => {
   /** The property names assigned inside a `ON CREATE SET` / `ON MATCH SET`
@@ -84,6 +84,7 @@ describe("MERGE statement shape", () => {
     for (const required of [
       "eventId",
       "name",
+      "routineDescription",
       "agentPrompt",
       "recurrence",
       "routineTemplateVersion",
@@ -118,9 +119,30 @@ describe("MERGE statement shape", () => {
     expect(body).toContain("e.sourcePlugin = null");
   });
 
-  it("ON MATCH touches ONLY the prompt, name, version and updatedAt", () => {
+  it("ON MATCH touches ONLY the prompt, name, description, version and updatedAt", () => {
     expect(new Set(propertiesSetIn("ON MATCH SET"))).toEqual(
-      new Set(["agentPrompt", "name", "routineTemplateVersion", "updatedAt"]),
+      new Set(["agentPrompt", "name", "routineDescription", "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.
+    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");
+  });
+
+  it("verify counts the descriptions the graph actually holds", () => {
+    // The failure this exists for writes nothing and reports nothing: the
+    // 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",
     );
   });
 
@@ -211,10 +233,15 @@ describe("seedRoutineRoster", () => {
     return { report, lines, calls };
   };
 
-  it("passes every roster row to the MERGE", async () => {
+  it("passes every roster row to the MERGE, description included", async () => {
     const { calls } = await seed();
     const mergeCall = calls.find((c) => c.query.includes("MERGE (e:Event"))!;
-    expect((mergeCall.params.rows as unknown[]).length).toBe(ROUTINE_ROSTER.length);
+    const rows = mergeCall.params.rows as Array<{ key: string; description: string }>;
+    expect(rows.length).toBe(ROUTINE_ROSTER.length);
+    // The Cypher can reference `row.description` and still write NULL for every
+    // row if the mapper never puts it on the parameter. That is the shape of
+    // the no-event failure, so assert the payload, not just the statement.
+    expect(rows.every((r) => typeof r.description === "string" && r.description.length > 0)).toBe(true);
     expect(mergeCall.params.version).toBe(ROUTINE_TEMPLATE_VERSION);
     expect(mergeCall.params.timezone).toBe("Europe/Lisbon");
   });
@@ -248,14 +275,29 @@ describe("seedRoutineRoster", () => {
     const { report, lines } = await seed({}, {
       timezone: { timezone: "Europe/Lisbon", timezoneSource: "owner-set" },
       merges: allUnchanged,
-      verify: { actual: short, suspended: short, nextRunNull: short },
+      verify: { actual: short, suspended: short, nextRunNull: short, descNull: 0 },
     });
-    expect(report.verify).toEqual({ expected, actual: short, suspended: short, nextRunNull: short });
+    expect(report.verify).toEqual({ expected, actual: short, suspended: short, nextRunNull: short, descNull: 0 });
     expect(lines).toContainEqual(
-      `[routine-seed] op=verify acct=abcd1234 expected=${expected} actual=${short} suspended=${short} nextRunNull=${short}`,
+      `[routine-seed] op=verify acct=abcd1234 expected=${expected} actual=${short} suspended=${short} nextRunNull=${short} descNull=0`,
     );
   });
 
+  it("reports a description that never landed, rather than reporting a healthy seed", async () => {
+    // Task 2214's no-event failure. Every action reads `unchanged`, expected
+    // and actual agree, and the only line that moves is this one — which is
+    // why it is measured back out of the graph rather than counted from the
+    // statements the seed issued.
+    const full = ROUTINE_ROSTER.length;
+    const { report, lines } = await seed({}, {
+      timezone: null,
+      merges: allUnchanged,
+      verify: { actual: full, suspended: full, nextRunNull: full, descNull: 3 },
+    });
+    expect(report.verify.descNull).toBe(3);
+    expect(lines.find((l) => l.includes("op=verify"))).toContain("descNull=3");
+  });
+
   it("logs tz-resolved as null when no owner-set profile exists", async () => {
     const { lines } = await seed({}, { timezone: null, merges: allUnchanged, verify: cleanVerify });
     expect(lines).toContainEqual("[routine-seed] op=tz-resolved acct=abcd1234 tz=null source=absent");
@@ -271,7 +313,7 @@ describe("seedRoutineRoster", () => {
     const { report } = await seed({}, {
       timezone: null,
       merges: mixed,
-      verify: { actual: 4, suspended: 3, nextRunNull: 3 },
+      verify: { actual: 4, suspended: 3, nextRunNull: 3, descNull: 0 },
     });
     expect(report.created).toBe(1);
     expect(report.refreshed).toBe(1);

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

@@ -71,6 +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. */
+  descNull: number;
 }
 
 export interface RoutineSeedReport {
@@ -144,7 +152,15 @@ export async function resolveOwnerTimezone(
  *  `agentChannel`, not `agentDestination`, not `recurrence`, not `timezone`. A
  *  re-seed on upgrade must not re-suspend an enabled routine, unbind a bound
  *  destination, or reset a retimed cron. Only the prompt, the name and the
- *  version move, and only when the row is behind and unedited. */
+ *  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 const MERGE_CYPHER = `
 UNWIND $rows AS row
 OPTIONAL MATCH (pre:Event {accountId: $accountId, routineTemplate: row.key})
@@ -157,6 +173,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.agentPrompt = row.prompt,
                 e.recurrence = row.recurrence,
                 e.routineTemplateVersion = $version,
@@ -176,6 +193,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.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,
@@ -192,7 +210,8 @@ MATCH (e:Event {accountId: $accountId})
 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.nextRun IS NULL THEN 1 ELSE 0 END) AS nextRunNull,
+       sum(CASE WHEN e.routineDescription IS NULL THEN 1 ELSE 0 END) AS descNull`;
 
 export interface SeedRoutineRosterArgs {
   session: SeedSessionLike;
@@ -246,7 +265,13 @@ export async function seedRoutineRoster(args: SeedRoutineRosterArgs): Promise<Ro
     version,
     timezone,
     now,
-    rows: roster.map((r) => ({ key: r.key, name: r.name, prompt: r.prompt, recurrence: r.recurrence })),
+    rows: roster.map((r) => ({
+      key: r.key,
+      name: r.name,
+      description: r.description,
+      prompt: r.prompt,
+      recurrence: r.recurrence,
+    })),
   });
 
   const merges: RoutineMergeRow[] = mergeRes.records.map((rec) => ({
@@ -266,10 +291,11 @@ export async function seedRoutineRoster(args: SeedRoutineRosterArgs): Promise<Ro
     actual: toInt(verifyRec?.get("actual")),
     suspended: toInt(verifyRec?.get("suspended")),
     nextRunNull: toInt(verifyRec?.get("nextRunNull")),
+    descNull: toInt(verifyRec?.get("descNull")),
   };
   log(
     `[routine-seed] op=verify acct=${acct} expected=${verify.expected} actual=${verify.actual} ` +
-      `suspended=${verify.suspended} nextRunNull=${verify.nextRunNull}`,
+      `suspended=${verify.suspended} nextRunNull=${verify.nextRunNull} descNull=${verify.descNull}`,
   );
 
   const created = merges.filter((m) => m.action === "create").length;