@bongos/core 1.23.17 → 1.23.18

package/.bongos-core.json

@@ -2,11 +2,11 @@
   "artifact": "bongos-core",
   "manifest_schema": 1,
   "generator": "scripts/bongos/package-core.js",
-  "core_version": "1.23.17",
-  "core_contract": "1.23.17",
-  "source_commit": "4a502e79527cf82113e88b2023d3008dfbae984f",
+  "core_version": "1.23.18",
+  "core_contract": "1.23.18",
+  "source_commit": "cf436c265cc3be4912f3c0ee9500d8a6fc8e2e20",
   "source_ref": "HEAD",
-  "built_at": "2026-10-28T00:35:19.416Z",
+  "built_at": "2026-10-28T00:49:26.518Z",
   "redaction": {
     "model": "docs-redacted+functional-verbatim",
     "docs_redacted": 616,
@@ -17,7 +17,7 @@
     "gate": "passed"
   },
   "file_count": 4233,
-  "tree_sha256": "99139b31ab3e5e0d6fb9464e6ad1295547d3b7dad361ee272235333df79fc8a2",
+  "tree_sha256": "eba7a9ced17fd552ba4b96d63bed4bf6de2b1b6b5b8825168fc83c764da4724a",
   "files": [
     {
       "path": ".claude/skills/ask-for-help/SKILL.md",
@@ -2937,7 +2937,7 @@
     {
       "path": "docs/module-api-changelog.md",
       "mode": "0000644",
-      "sha256": "4bc6873970f7d2632aea896316a31aaef5a2cc6fd51cfa69f1609974b999cb67"
+      "sha256": "5231965cae7f22d595883889e831987e6f14ad158155df16fb439918dc6e44fc"
     },
     {
       "path": "docs/modules-contract.md",
@@ -4487,7 +4487,7 @@
     {
       "path": "modules/copy-desk/auto-close.js",
       "mode": "0000644",
-      "sha256": "e27d2856bdc6e61116e486c053a353bde07c36ac5d850dce278e5d3c1e4135d0"
+      "sha256": "f9f4e8e009fab44c47fd3651c5d44da94cec1fd4cb7ad0931515fadbd6526280"
     },
     {
       "path": "modules/copy-desk/docx.js",
@@ -4537,7 +4537,7 @@
     {
       "path": "modules/copy-desk/page-status.js",
       "mode": "0000644",
-      "sha256": "de12a5dd005cb8ab8f078cc2295af5fb220f09e0ca21192719698ab9037b57be"
+      "sha256": "0d3ae54e54203d62a6b8b7874706a478476d2c22f680ed92dc6d91d2c8285c3e"
     },
     {
       "path": "modules/copy-desk/pages.js",
@@ -4607,7 +4607,7 @@
     {
       "path": "modules/copy-desk/tests/copy_page_auto_close.mjs",
       "mode": "0000644",
-      "sha256": "ec7b6b3ec7f974f79abe5ffdcfb08668ec11477599b021303335374b3e7c59ec"
+      "sha256": "f992174839c4f2be3c3a6f942c27493a5a7081e46ce01293f9075ce9a26330be"
     },
     {
       "path": "modules/copy-desk/tests/copy_page_draft.mjs",
@@ -4622,7 +4622,7 @@
     {
       "path": "modules/copy-desk/tests/copy_page_status.mjs",
       "mode": "0000644",
-      "sha256": "f968af0d0f0bd8d1202816ae1d660c9bd75f706997167a75c2a09c8a9d8ed579"
+      "sha256": "e90260a6fc9b95c65efd1aa1488c88f7344d94e6d315055a52648d4c92f416dc"
     },
     {
       "path": "modules/copy-desk/tests/copy_proposals.mjs",
@@ -10957,12 +10957,12 @@
     {
       "path": "package-lock.json",
       "mode": "0000644",
-      "sha256": "22b8b7bf76a70bbd3bbb57ed0ab3f53657cf2a989eb6acc8578022311932e5c1"
+      "sha256": "f2aa7d637ca9724249d7da6c5522c3c610d8f8f577ad45800c87bc5c34e85400"
     },
     {
       "path": "package.json",
       "mode": "0000644",
-      "sha256": "a9218f9cc690dc0c929a5fe7b6a4fccb6a0103f08c1637d09be5c2d23b96b035"
+      "sha256": "3e4721211b08ace73fce53f51c9f456fc53915ba54df2816c8e9e4c34df20f87"
     },
     {
       "path": "public-docs/index.html",
@@ -10982,7 +10982,7 @@
     {
       "path": "release-notes.json",
       "mode": "0000644",
-      "sha256": "d1193b47af93db407c4fe4ec5a227fdd5c97bd8ebeaa5d1f2c9b0a4c0fab32ca"
+      "sha256": "48cc837f2857d4db190398862cce83da8cf092ac339e77d043c9b998993ba957"
     },
     {
       "path": "scripts/bongos/CLAUDE.md",
@@ -14857,7 +14857,7 @@
     {
       "path": "src/module-api.js",
       "mode": "0000644",
-      "sha256": "e28e39ed8813e02cf4f2689fdf369d45ea3631a6fb6f2994cfaa0089eb4898c1"
+      "sha256": "a69374b5230240bb5f2551554f6ce7e1ac40163e1ddd9ab6bcc5027619734e67"
     },
     {
       "path": "src/module-loader/catalog.js",
@@ -15907,7 +15907,7 @@
     {
       "path": "tests/copy_stuck_page_tweaks.mjs",
       "mode": "0000644",
-      "sha256": "d0a860520d482be8014a036d183e570f789195597891c367c16ee7e817825e2a"
+      "sha256": "e53e4b7c768b0e1b5cb2343d174eba8886586678cc3d3a70560d107faa88a1f5"
     },
     {
       "path": "tests/core_258_box_session_claims_db.mjs",

package/docs/module-api-changelog.md

@@ -3249,5 +3249,7 @@ is load-bearing: the script throws rather than guess if it is missing, and
         landed since 1.23.15 with no explicit bump. run 40607196561. (task 1002620)
   1.23.17 — CI auto-patch (publish-on-merge, ADR 0161): carrier for merges
         landed since 1.23.16 with no explicit bump. run 40609586769. (task 1002620)
+  1.23.18 — CI auto-patch (publish-on-merge, ADR 0161): carrier for merges
+        landed since 1.23.17 with no explicit bump. run 40611101597. (task 1002620)
 ---------------------------------------------------------------------------
 ```

package/modules/copy-desk/auto-close.js

@@ -13,8 +13,11 @@
 //                     rewrite left the batch for an engineer task at submit
 //                     (unplaced, shared or removed lines), so /tweak has no line
 //                     to apply and the round can never move on its own.
+//   draft_expired     `ready` with a draft and no batch, untouched for
+//                     page-status's DRAFT_EXPIRES_AFTER (14 days, task 1004901):
+//                     its artist was reminded at 7 days and never came back.
 //
-// Both are abandoned through the lifecycle port's releaseStuckTweak (no compose,
+// All three are abandoned through the lifecycle port's releaseStuckTweak (no compose,
 // so the note is the reason and nothing is written to the description). The
 // engineer tasks a submit filed are untouched: they carry the artist's words.
 //
@@ -24,20 +27,23 @@
 
 const api = require('../../src/module-api');
 const pages = require('./pages');
+const pageStatus = require('./page-status');
 
 const GRACE_MS = Number(process.env.COPY_AUTO_CLOSE_GRACE_MS) || 60 * 60_000;
 
 const NOTES = Object.freeze({
   blank: 'Closed by itself: the round was opened but nothing was written in it.',
   nothing_to_apply: 'Closed by itself: every rewritten line went to an engineer task, so nothing was left to apply on the page.',
+  draft_expired: 'Closed by itself: the draft sat untouched for two weeks. Its words stay on this task; claim the page again to start over.',
 });
 
-function closeReason(task, { now = Date.now(), graceMs = GRACE_MS } = {}) {
+function closeReason(task, { now = Date.now(), graceMs = GRACE_MS, draftExpiresMs = pageStatus.DRAFT_EXPIRES_AFTER } = {}) {
   if (!task || task.status !== 'ready' || task.holder_id != null) return null;
   const touched = Date.parse(task.updated_at || task.created_at || '');
   if (!Number.isFinite(touched) || now - touched < graceMs) return null;
   const p = pages.blockPresence(task.description);
-  if (!p.batch) return p.draft ? null : 'blank';
+  // task 1004901: a draft its artist left untouched for two weeks expires.
+  if (!p.batch) return p.draft ? (now - touched >= draftExpiresMs ? 'draft_expired' : null) : 'blank';
   const b = pages.parseBatchBlock(task.description);
   if (b.ok && b.batch.lines.length === 0) return 'nothing_to_apply';
   return null;

package/modules/copy-desk/page-status.js

@@ -306,22 +306,29 @@ function orderedSurfaces(inventory) {
 // (applied rounds only) stayed empty: the artist's finished work looked lost.
 // This is every page with an open round, where it is, since when (the round's
 // last update), and whether it has sat there too long. Waiting for review is
-// the artist's own move and a draft is theirs to finish; a draft is never stuck,
-// and a review is stuck only after 3 days (task 1004777, so its artist is told);
-// a state the reads cannot name always needs to be reviewed.
+// the artist's own move and a draft is theirs to finish: a draft or a page
+// being written is stuck after 7 days and a draft expires after 14 (task
+// 1004901); a review is stuck after 3 days (task 1004777, so its artist is
+// told); a state the reads cannot name always needs to be reviewed.
 // ---------------------------------------------------------------------------
 
 const DAY_MS = 24 * 60 * 60 * 1000;
+// task 1004901: every state names WHO moves it next (waits_on: 'artist' is the
+// round's own artist, 'appliers' the Metic and up builders who apply and land
+// work) and WHEN it is overdue (stuck_after: then it is pushed to that person on
+// Home and in the daily digest). A draft nobody touches also EXPIRES
+// (expires_after): the auto-close sweep abandons it, its words kept on the task.
+const DRAFT_EXPIRES_AFTER = 14 * DAY_MS;
 const IN_FLIGHT = Object.freeze({
-  draft: { word: 'Being written', stuck_after: null },
-  writing: { word: 'Being written', stuck_after: null },
-  words_review: { word: 'Words waiting for review', stuck_after: 3 * DAY_MS },
-  submitted: { word: 'Approved, waiting to be applied', stuck_after: 3 * DAY_MS },
-  applying: { word: 'Being applied', stuck_after: DAY_MS },
+  draft: { word: 'Being written', stuck_after: 7 * DAY_MS, waits_on: 'artist', expires_after: DRAFT_EXPIRES_AFTER },
+  writing: { word: 'Being written', stuck_after: 7 * DAY_MS, waits_on: 'artist' },
+  words_review: { word: 'Words waiting for review', stuck_after: 3 * DAY_MS, waits_on: 'artist' },
+  submitted: { word: 'Approved, waiting to be applied', stuck_after: 3 * DAY_MS, waits_on: 'appliers' },
+  applying: { word: 'Being applied', stuck_after: DAY_MS, waits_on: 'appliers' },
   // task 1004777: a review left 3 days is stuck too, so its artist is told on Home.
-  waiting_for_artist: { word: 'Waiting for your review', stuck_after: 3 * DAY_MS },
-  landing: { word: 'Approved, going live', stuck_after: DAY_MS },
-  unknown: { word: 'Needs to be reviewed', stuck_after: 0 },
+  waiting_for_artist: { word: 'Waiting for your review', stuck_after: 3 * DAY_MS, waits_on: 'artist' },
+  landing: { word: 'Approved, going live', stuck_after: DAY_MS, waits_on: 'appliers' },
+  unknown: { word: 'Needs to be reviewed', stuck_after: 0, waits_on: 'appliers' },
 });
 const IN_FLIGHT_ORDER = ['unknown', 'landing', 'applying', 'submitted', 'waiting_for_artist', 'words_review', 'writing', 'draft'];
 
@@ -344,6 +351,9 @@ function inFlightFor(list, byPage, nowMs) {
       state_word: kind.word,
       since,
       stuck: kind.stuck_after != null && (kind.stuck_after === 0 || (age != null && age > kind.stuck_after)),
+      waits_on: kind.waits_on,
+      due_at: since && kind.stuck_after != null ? new Date(Date.parse(since) + kind.stuck_after).toISOString() : null,
+      expires_at: since && kind.expires_after != null ? new Date(Date.parse(since) + kind.expires_after).toISOString() : null,
       holder: o.holder,
       artist_id: r.artist_id != null ? String(r.artist_id) : null,
     });
@@ -380,8 +390,8 @@ function stuckPageTweaks({ inventory, tasks, now = Date.now() }) {
       state_word: f.state_word,
       since: f.since,
       days: ms == null ? null : Math.max(0, Math.floor(ms / DAY_MS)),
-      // Submitted words (task 1004900) wait on their artist's review, like an applied page.
-      waits_on: f.state === 'waiting_for_artist' || f.state === 'words_review' ? 'artist' : 'appliers',
+      // The state's own owner (task 1004901), one table for the Studio and the pushes.
+      waits_on: f.waits_on,
       artist_id: f.artist_id,
     };
   });
@@ -968,4 +978,6 @@ module.exports = {
   stuckPageTweaks,
   stuckPageTweaksFor,
   stuckDigestText,
+  // task 1004901: when an untouched draft expires (the auto-close sweep reads it).
+  DRAFT_EXPIRES_AFTER,
 };

package/modules/copy-desk/tests/copy_page_auto_close.mjs

@@ -56,6 +56,14 @@ await test('a blank round and an empty batch are the only two that close', () =>
   });
 });
 
+await test('a draft left untouched for two weeks expires; a younger one stays (task 1004901)', () => {
+  const D = 86400000;
+  const at = (days) => round({ description: draft, updated_at: new Date(NOW - days * D).toISOString() });
+  assert.equal(autoClose.closeReason(at(13.9), { now: NOW }), null);
+  assert.equal(autoClose.closeReason(at(14.1), { now: NOW }), 'draft_expired');
+  assert.equal(autoClose.closeReason({ ...at(30), holder_id: '7' }, { now: NOW }), null, 'a held draft is being written, never expired');
+});
+
 await test('the grace period is honoured: a round touched within it stays', () => {
   const t = round({ updated_at: new Date(NOW - 59 * 60_000).toISOString() });
   assert.equal(autoClose.closeReason(t, { now: NOW, graceMs: 60 * 60_000 }), null);

package/modules/copy-desk/tests/copy_page_status.mjs

@@ -257,7 +257,7 @@ ${pages.composeWordsApprovedBlock({ at: ago(9), by: '7' })}`;
     round(2, 'landing:b', { status: 'ready', description: `${pages.roundTaskBody(pg('landing:b'), 1)}\n\n${batch('landing:b')}`, updated_at: ago(1) }),
     round(3, 'landing:c', { status: 'confirmed', updated_at: ago(17) }),
     round(4, 'builders:d', { status: 'completed', grade_passed: true, updated_at: ago(20) }),
-    round(5, 'builders:e', { status: 'active', holder_id: '7', holder_login: 'me', holder_worktree: false, updated_at: ago(30) }),
+    round(5, 'builders:e', { status: 'active', holder_id: '7', holder_login: 'me', holder_worktree: false, updated_at: ago(5) }),
     round(6, 'builders:f', { status: 'ready', updated_at: ago(0.2) }),
     round(7, 'builders:g', { status: 'shipped', shipped_at: ago(3), updated_at: ago(3) }),
     round(8, 'status:h', { status: 'active', holder_id: '8', holder_worktree: true, updated_at: ago(2) }),
@@ -271,7 +271,7 @@ ${pages.composeWordsApprovedBlock({ at: ago(9), by: '7' })}`;
     ['builders:d', 'waiting_for_artist', 'Waiting for your review', true],
     ['landing:b', 'submitted', 'Approved, waiting to be applied', false],
     ['builders:e', 'writing', 'Being written', false],
-  ], 'a shipped round is not in flight; writing is never stuck, and a review left 20 days is (task 1004777)');
+  ], 'a shipped round is not in flight; writing is not overdue before 7 days (task 1004901), and a review left 20 days is (task 1004777)');
   assert.equal(out.in_flight[1].since, ago(17));
   assert.equal(out.in_flight[1].task_id, '3');
   assert.equal(out.pages.find((p) => p.id === 'landing:a').status_label, 'Untweaked', 'the status still counts only what shipped');

package/package-lock.json

@@ -1,12 +1,12 @@
 {
   "name": "@bongos/core",
-  "version": "1.23.17",
+  "version": "1.23.18",
   "lockfileVersion": 3,
   "requires": true,
   "packages": {
     "": {
       "name": "@bongos/core",
-      "version": "1.23.17",
+      "version": "1.23.18",
       "license": "AGPL-3.0-or-later",
       "dependencies": {
         "express": "^4.21.2",

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@bongos/core",
-  "version": "1.23.17",
+  "version": "1.23.18",
   "description": "Cloud Bongos — the AI-first build platform core (GDS + platform surfaces + module system), installed as a versioned dependency (ADR 0108).",
   "license": "AGPL-3.0-or-later",
   "main": "src/platform-server.js",

package/release-notes.json

@@ -10491,5 +10491,11 @@
       "id": "1004899",
       "text": "Page tweak rounds that hold nothing to do now close themselves after an hour, so the Studio no longer shows empty rounds as needing a look forever."
     }
+  ],
+  "1.23.18": [
+    {
+      "id": "1004901",
+      "text": "Every stage of a page tweak now has a named person and a deadline. Drafts left alone get a reminder after a week and close after two, so no page sits in the Studio without someone being told."
+    }
   ]
 }

package/src/module-api.js

@@ -81,7 +81,7 @@ const { sleep } = require('./sleep');
 // MAJOR (see allowBoxScope below): passes the request through untouched.
 function deprecatedNoopMiddleware(_req, _res, next) { next(); }
 
-const CORE_VERSION = '1.23.17'; // CI auto-patch carrier (ADR 0161); changelog: docs/module-api-changelog.md
+const CORE_VERSION = '1.23.18'; // CI auto-patch carrier (ADR 0161); changelog: docs/module-api-changelog.md
 
 // A namespaced logger so a module's log lines are attributable + consistent.
 // Usage: const log = api.logger('discord'); log.info('mounted');

package/tests/copy_stuck_page_tweaks.mjs

@@ -34,8 +34,8 @@ const TASKS = [
   round(2, 'landing:account', { status: 'completed', grade_passed: true, updated_at: ago(5) }),
   // submitted yesterday: NOT stuck
   round(3, 'builders:studio', { status: 'ready', description: `${pages.roundTaskBody(pg('builders:studio'), 1)}\n\n${batch('builders:studio')}`, updated_at: ago(1) }),
-  // being written for a month: a draft is never stuck
-  round(4, 'landing:projects', { status: 'active', holder_id: '7', holder_worktree: false, updated_at: ago(30) }),
+  // being written for 3 days: not overdue (that is 7 days, task 1004901)
+  round(4, 'landing:projects', { status: 'active', holder_id: '7', holder_worktree: false, updated_at: ago(3) }),
 ];
 const stuck = () => status.stuckPageTweaks({ inventory: INVENTORY, tasks: TASKS, now: NOW });
 
@@ -46,6 +46,43 @@ await test('the composer: only stuck pages, each with where it sits, how long, a
   ]);
 });
 
+// task 1004901: every state names who moves it next and when it is overdue.
+await test('every state has an owner and a deadline: overdue only past it, told to that owner', () => {
+  const D = 86400000;
+  const p = (id) => pg(id);
+  const desc = (id, blocks) => [pages.roundTaskBody(p(id), 1), ...blocks].join('\n\n');
+  const draftBlock = (id) => pages.composeDraftBlock({ page_id: id, reading_hash: 'h', saved_at: ago(1), lines: [] });
+  const batchBlock = (id) => pages.composeBatchBlock({ page_id: id, reading_hash: 'h', submitted_at: ago(1), lines: [] });
+  const approved = pages.composeWordsApprovedBlock({ at: ago(1), by: ARTIST });
+  // [state, task shape, owner, days until overdue]
+  const table = [
+    ['draft', (id) => ({ status: 'ready', description: desc(id, [draftBlock(id)]) }), 'artist', 7],
+    ['writing', () => ({ status: 'active', holder_id: ARTIST, holder_worktree: false }), 'artist', 7],
+    ['words_review', (id) => ({ status: 'ready', description: desc(id, [batchBlock(id)]) }), 'artist', 3],
+    ['submitted', (id) => ({ status: 'ready', description: desc(id, [batchBlock(id), approved]) }), 'appliers', 3],
+    ['applying', () => ({ status: 'active', holder_id: '99', holder_worktree: true }), 'appliers', 1],
+    ['waiting_for_artist', () => ({ status: 'completed', grade_passed: true }), 'artist', 3],
+    ['landing', () => ({ status: 'confirmed' }), 'appliers', 1],
+    ['unknown', () => ({ status: 'ready' }), 'appliers', 0],
+  ];
+  for (const [state, shape, owner, days] of table) {
+    const id = 'landing:projects';
+    const at = (age) => status.composePages({ inventory: INVENTORY, readings: null, tasks: [round(50, id, { ...shape(id), updated_at: ago(age) })], now: NOW }).in_flight[0];
+    const before = at(Math.max(days - 0.5, 0));
+    assert.equal(before.state, state);
+    assert.equal(before.waits_on, owner, `${state} waits on ${owner}`);
+    assert.equal(Date.parse(before.due_at) - Date.parse(before.since), days * D, `${state} is overdue after ${days} day(s)`);
+    if (days > 0) assert.equal(before.stuck, false, `${state} is not overdue before its deadline`);
+    const after = at(days + 0.5);
+    assert.equal(after.stuck, true, `${state} is overdue after its deadline`);
+    const told = status.stuckPageTweaksFor({ builderId: ARTIST, rank: 'xenos', stuck: status.stuckPageTweaks({ inventory: INVENTORY, tasks: [round(50, id, { ...shape(id), updated_at: ago(days + 0.5) })], now: NOW }) });
+    assert.equal(told.length, owner === 'artist' ? 1 : 0, `${state}: its artist is told only when it waits on them`);
+  }
+  const draft = status.composePages({ inventory: INVENTORY, readings: null, tasks: [round(51, 'landing:projects', { status: 'ready', description: desc('landing:projects', [draftBlock('landing:projects')]), updated_at: ago(2) })], now: NOW }).in_flight[0];
+  assert.equal(Date.parse(draft.expires_at) - Date.parse(draft.since), status.DRAFT_EXPIRES_AFTER, 'a draft says when it expires');
+  assert.equal(status.DRAFT_EXPIRES_AFTER, 14 * D);
+});
+
 await test('submitted words left unreviewed wait on their artist, not the appliers (task 1004900)', () => {
   const words = round(9, 'landing:projects', { status: 'ready', description: `${pages.roundTaskBody(pg('landing:projects'), 1)}\n\n${pages.composeBatchBlock({ page_id: 'landing:projects', reading_hash: 'h', submitted_at: ago(4), lines: [] })}`, updated_at: ago(4) });
   const got = status.stuckPageTweaks({ inventory: INVENTORY, tasks: [words], now: NOW });