@bongos/core 1.23.16 → 1.23.18

package/.bongos-core.json

@@ -2,22 +2,22 @@
   "artifact": "bongos-core",
   "manifest_schema": 1,
   "generator": "scripts/bongos/package-core.js",
-  "core_version": "1.23.16",
-  "core_contract": "1.23.16",
-  "source_commit": "6e661df1836bf2b9f9e4574d98a7fc05b05fb98a",
+  "core_version": "1.23.18",
+  "core_contract": "1.23.18",
+  "source_commit": "cf436c265cc3be4912f3c0ee9500d8a6fc8e2e20",
   "source_ref": "HEAD",
-  "built_at": "2026-10-28T00:12:52.362Z",
+  "built_at": "2026-10-28T00:49:26.518Z",
   "redaction": {
     "model": "docs-redacted+functional-verbatim",
     "docs_redacted": 616,
     "agent_docs_stubbed": 29,
-    "functional_verbatim": 3584,
+    "functional_verbatim": 3587,
     "rules": 3,
     "gate_literals": 3,
     "gate": "passed"
   },
-  "file_count": 4230,
-  "tree_sha256": "7ebfe9f3a223ab157c483e0e2bf97c093ff0225711fdbc542c9a572075287574",
+  "file_count": 4233,
+  "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": "31d8127f92c9fc127bf1ba774ad14e116e66ad5b9a7ef2625721fb9f675d5841"
+      "sha256": "5231965cae7f22d595883889e831987e6f14ad158155df16fb439918dc6e44fc"
     },
     {
       "path": "docs/modules-contract.md",
@@ -4484,6 +4484,11 @@
       "mode": "0000644",
       "sha256": "6bf4b2a9b9468ca6a5d077a5e01584c66e2ba3a82af3039ee7b9d4be7bc4dbde"
     },
+    {
+      "path": "modules/copy-desk/auto-close.js",
+      "mode": "0000644",
+      "sha256": "f9f4e8e009fab44c47fd3651c5d44da94cec1fd4cb7ad0931515fadbd6526280"
+    },
     {
       "path": "modules/copy-desk/docx.js",
       "mode": "0000644",
@@ -4517,7 +4522,7 @@
     {
       "path": "modules/copy-desk/module.json",
       "mode": "0000644",
-      "sha256": "0d98abbe640a1fa9326d29ebc44678b0d05e87d6455548f64bcd80c51ea747d1"
+      "sha256": "e5885c9524764cd3e5a1cafc5d352b47e8b1773509854f53c989570c3d02be5d"
     },
     {
       "path": "modules/copy-desk/page-data.js",
@@ -4532,13 +4537,18 @@
     {
       "path": "modules/copy-desk/page-status.js",
       "mode": "0000644",
-      "sha256": "de12a5dd005cb8ab8f078cc2295af5fb220f09e0ca21192719698ab9037b57be"
+      "sha256": "0d3ae54e54203d62a6b8b7874706a478476d2c22f680ed92dc6d91d2c8285c3e"
     },
     {
       "path": "modules/copy-desk/pages.js",
       "mode": "0000644",
       "sha256": "595946a1ef88b65612779de0f3dafa0b44b2a02e2b3652f0e026863a9ef4d60c"
     },
+    {
+      "path": "modules/copy-desk/pollers/auto-close-rounds.js",
+      "mode": "0000644",
+      "sha256": "1f370c827350735270704e0e1c7bd5c4427449709c071e3917b1251acfc8dc52"
+    },
     {
       "path": "modules/copy-desk/pollers/stuck-page-digest.js",
       "mode": "0000644",
@@ -4592,7 +4602,12 @@
     {
       "path": "modules/copy-desk/tests/copy_no_cms.mjs",
       "mode": "0000644",
-      "sha256": "7d03e68af010a2c2dd8dead6157c3e80643d0ef5956b45359eb1461d224a59e4"
+      "sha256": "3c5401ec7fbf0aa8a255f0a20ce4a25e285b5c7b3ceb90b88ba1434a837f17b6"
+    },
+    {
+      "path": "modules/copy-desk/tests/copy_page_auto_close.mjs",
+      "mode": "0000644",
+      "sha256": "f992174839c4f2be3c3a6f942c27493a5a7081e46ce01293f9075ce9a26330be"
     },
     {
       "path": "modules/copy-desk/tests/copy_page_draft.mjs",
@@ -4607,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",
@@ -10942,12 +10957,12 @@
     {
       "path": "package-lock.json",
       "mode": "0000644",
-      "sha256": "9812601e19d98dc57f1c6327750d37d3f1ba5e9a2e77fcc7aa5efcafeb0c1889"
+      "sha256": "f2aa7d637ca9724249d7da6c5522c3c610d8f8f577ad45800c87bc5c34e85400"
     },
     {
       "path": "package.json",
       "mode": "0000644",
-      "sha256": "8c6f24f0e4d27bb040f8ca7a1a5f8a438638bb94632870110aa18f82662ca9a2"
+      "sha256": "3e4721211b08ace73fce53f51c9f456fc53915ba54df2816c8e9e4c34df20f87"
     },
     {
       "path": "public-docs/index.html",
@@ -10967,7 +10982,7 @@
     {
       "path": "release-notes.json",
       "mode": "0000644",
-      "sha256": "46ad6e108a8bdf1681a0a79efacf6c5bbe83adbe06bfb6bf554d8bfedf8dffc4"
+      "sha256": "48cc837f2857d4db190398862cce83da8cf092ac339e77d043c9b998993ba957"
     },
     {
       "path": "scripts/bongos/CLAUDE.md",
@@ -14842,7 +14857,7 @@
     {
       "path": "src/module-api.js",
       "mode": "0000644",
-      "sha256": "84676e50d010ed084fe27550fc614a050d3d794b2bacadfaed0b6cd9b87b9140"
+      "sha256": "a69374b5230240bb5f2551554f6ce7e1ac40163e1ddd9ab6bcc5027619734e67"
     },
     {
       "path": "src/module-loader/catalog.js",
@@ -15892,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

@@ -3247,5 +3247,9 @@ is load-bearing: the script throws rather than guess if it is missing, and
         landed since 1.23.14 with no explicit bump. run 40604630697. (task 1002620)
   1.23.16 — CI auto-patch (publish-on-merge, ADR 0161): carrier for merges
         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

@@ -0,0 +1,73 @@
+'use strict';
+
+// auto-close.js — a page round with nothing in it closes itself (task 1004899).
+//
+// WHY. A round only moves when a person or a runner acts on it, so a round that
+// holds nothing to act on sat in the Studio as "Needs a look" (or "Submitted,
+// waiting to be applied") forever: someone had to notice it and abandon it by
+// hand. Two shapes hold nothing:
+//
+//   blank             `ready`, no draft and no frozen batch: opened in the
+//                     studio, nothing written, the claim dropped.
+//   nothing_to_apply  `ready` with a frozen batch whose `lines` is empty: every
+//                     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.
+//
+// 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.
+//
+// `closeReason` is pure; `runAutoClose` is the I/O the poller calls. A round
+// someone holds, or one touched within GRACE_MS, is never closed: an artist who
+// just released a draft to come back to it in a minute keeps it.
+
+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, 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);
+  // 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;
+}
+
+// One sweep. Returns what happened per round, so the poller and the tests read
+// the same outcome. The actor is the project owner (a system act, recorded on
+// the task as its abandon reason); no owner, no sweep.
+async function runAutoClose({ now = Date.now(), resolve = (k) => (api.resolveOptional ? api.resolveOptional(k) : null), founder = () => api.getFoundingBuilder() } = {}) {
+  const lifecycle = resolve('lifecycle');
+  if (!lifecycle || typeof lifecycle.listPageTweakTasks !== 'function' || typeof lifecycle.releaseStuckTweak !== 'function') {
+    return { closed: [], skipped: 'no_lifecycle' };
+  }
+  const tasks = await lifecycle.listPageTweakTasks();
+  const due = tasks.map((t) => ({ t, why: closeReason(t, { now }) })).filter((x) => x.why);
+  if (!due.length) return { closed: [] };
+  const owner = await founder();
+  if (!owner) return { closed: [], skipped: 'no_owner' };
+  const closed = [];
+  for (const { t, why } of due) {
+    const r = await lifecycle.releaseStuckTweak({ taskId: t.id, builderId: owner.id, expectedStatus: 'ready', note: NOTES[why] });
+    closed.push({ task_id: String(t.id), why, outcome: r.outcome });
+  }
+  return { closed };
+}
+
+module.exports = { closeReason, runAutoClose };

package/modules/copy-desk/module.json

@@ -7,7 +7,7 @@
   "category": { "parent": "quality" },
   "default": true,
   "maintenance": { "status": "core-maintained" },
-  "contributes": { "routes": ["copy-desk", "copy-desk-artist", "copy-desk-release"], "migrations": true, "skills": ["tweak"], "pollers": ["stuck-page-digest"] },
+  "contributes": { "routes": ["copy-desk", "copy-desk-artist", "copy-desk-release"], "migrations": true, "skills": ["tweak"], "pollers": ["stuck-page-digest", "auto-close-rounds"] },
   "provides": ["copy-desk.artistStats", "copy-desk.stuckPageTweaksFor"],
   "consumes": ["lifecycle"]
 }

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/pollers/auto-close-rounds.js

@@ -0,0 +1,32 @@
+'use strict';
+
+// modules/copy-desk/pollers/auto-close-rounds.js — closes page rounds that hold
+// nothing to act on (task 1004899). Loader-started (contributes.pollers); the
+// rule lives in ../auto-close.js. Every 15 minutes by default
+// (COPY_AUTO_CLOSE_MS); COPY_AUTO_CLOSE_DISABLED=1 turns it off. Best-effort: a
+// failed sweep logs and tries again next interval.
+
+const api = require('../../../src/module-api');
+const autoClose = require('../auto-close');
+
+const log = api.logger('copy-desk');
+const INTERVAL_MS = Number(process.env.COPY_AUTO_CLOSE_MS) || 15 * 60_000;
+let timer = null;
+
+function tick() {
+  autoClose.runAutoClose()
+    .then((r) => { if (r.closed.length) log.info(`[auto-close-rounds] closed ${r.closed.length} empty page round(s)`); })
+    .catch((e) => log.error(e, '[auto-close-rounds] sweep failed'));
+}
+
+module.exports = {
+  start() {
+    if (process.env.COPY_AUTO_CLOSE_DISABLED === '1' || timer) return;
+    timer = setInterval(tick, INTERVAL_MS);
+    if (timer.unref) timer.unref();
+  },
+  stop() {
+    if (timer) clearInterval(timer);
+    timer = null;
+  },
+};

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

@@ -70,7 +70,7 @@ const stripComments = (src) => src
 // Task 1004435 added routes/copy-desk-artist.js (one artist's approved pages, a
 // read) and artist-stats.js's readArtistPages under it.
 const ROUTE_FILES = fs.readdirSync(path.join(MODULE_DIR, 'routes')).filter((f) => f.endsWith('.js')).sort().map((f) => `routes/${f}`);
-const MODULE_FILES = ['registry.js', 'flags.js', 'proposals.js', 'pages.js', 'page-status.js', 'page-data.js', 'docx.js', 'page-decisions.js', 'artist-stats.js', ...ROUTE_FILES];
+const MODULE_FILES = ['registry.js', 'flags.js', 'proposals.js', 'pages.js', 'page-status.js', 'page-data.js', 'docx.js', 'page-decisions.js', 'artist-stats.js', 'auto-close.js', ...ROUTE_FILES];
 
 console.log('copy-desk: NO LIVE CMS');
 
@@ -310,6 +310,10 @@ await test('the module manifest declares no seam another module could write copy
   assert.deepEqual(manifest.provides, ['copy-desk.artistStats', 'copy-desk.stuckPageTweaksFor'], 'a provided port here would be a copy write path for every other module');
   const stuckSrc = stripComments(read('stuck-tweaks.js'));
   assert.equal(/writeFile|\.query\(|UPDATE |INSERT /.test(stuckSrc), false, 'the stuck-tweaks port writes nothing');
+  // task 1004899: the auto-close sweep only ABANDONS an empty round through the
+  // lifecycle port; it never hands a compose, so it cannot write a description.
+  const closeSrc = stripComments(read('auto-close.js'));
+  assert.equal(/writeFile|\.query\(|UPDATE |INSERT |compose/.test(closeSrc), false, 'the auto-close sweep abandons and writes nothing');
   const stats = read('artist-stats.js');
   assert.equal(/\b(pool|query|writeFile|appendFile|INSERT|UPDATE|DELETE)\b/.test(stats.replace(/\/\/.*$/gm, '')), false,
     'the artist-stats port reads through other ports and writes nothing');

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

@@ -0,0 +1,100 @@
+// modules/copy-desk/tests/copy_page_auto_close.mjs — a page round with nothing in
+// it closes itself (task 1004899).
+//
+// The done-when: a blank round and a round whose batch has no lines left are
+// abandoned by the sweep; a round with real lines, a draft, a held round, a
+// round waiting for its artist and a freshly touched one are never touched.
+
+import { strict as assert } from 'node:assert';
+import { createRequire } from 'node:module';
+
+const require = createRequire(import.meta.url);
+const pages = require('../pages.js');
+const autoClose = require('../auto-close.js');
+
+let passed = 0;
+let failed = 0;
+async function test(name, fn) {
+  try { await fn(); passed++; console.log(`  ok  ${name}`); }
+  catch (err) { failed++; console.log(`  FAIL ${name}`); console.log(`     ${err.stack || err.message}`); }
+}
+
+const NOW = Date.parse('2026-10-27T12:00:00.000Z');
+const OLD = '2026-10-20T12:00:00.000Z';
+const PAGE = 'builders:collab';
+
+const line = { key: 'L0001', section: 'Top', before: 'Archive', after: 'Past Collabs', placement: 'placed' };
+const draft = pages.composeDraftBlock({ page_id: PAGE, reading_hash: 'h', saved_at: OLD, lines: [line] });
+const batch = (lines) => pages.composeBatchBlock({ page_id: PAGE, reading_hash: 'h', submitted_at: OLD, lines, unplaced_tasks: lines.length ? [] : ['1004692'] });
+
+const round = (over) => ({ id: '1', status: 'ready', description: 'Page tweak round 1.', holder_id: null, updated_at: OLD, created_at: OLD, ...over });
+
+const ROUNDS = {
+  blank: round({ id: '11' }),
+  empty_batch: round({ id: '12', description: `${draft}\n\n${batch([])}` }),
+  real_batch: round({ id: '13', description: `${draft}\n\n${batch([line])}` }),
+  draft_only: round({ id: '14', description: draft }),
+  held: round({ id: '15', status: 'active', holder_id: '7' }),
+  waiting: round({ id: '16', status: 'completed', description: `${draft}\n\n${batch([line])}` }),
+  fresh_blank: round({ id: '17', updated_at: new Date(NOW - 5 * 60_000).toISOString() }),
+  ready_but_held: round({ id: '18', holder_id: '7' }),
+};
+
+console.log('copy-desk: empty page rounds close themselves');
+
+await test('a blank round and an empty batch are the only two that close', () => {
+  const got = Object.fromEntries(Object.entries(ROUNDS).map(([k, t]) => [k, autoClose.closeReason(t, { now: NOW })]));
+  assert.deepEqual(got, {
+    blank: 'blank',
+    empty_batch: 'nothing_to_apply',
+    real_batch: null,
+    draft_only: null,
+    held: null,
+    waiting: null,
+    fresh_blank: null,
+    ready_but_held: null,
+  });
+});
+
+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);
+  assert.equal(autoClose.closeReason(t, { now: NOW, graceMs: 30 * 60_000 }), 'blank');
+});
+
+await test('the sweep abandons through the lifecycle port, as the owner, with a plain note and no rewrite', async () => {
+  const calls = [];
+  const lifecycle = {
+    listPageTweakTasks: async () => Object.values(ROUNDS),
+    releaseStuckTweak: async (args) => { calls.push(args); return { outcome: 'abandoned' }; },
+  };
+  const r = await autoClose.runAutoClose({ now: NOW, resolve: () => lifecycle, founder: async () => ({ id: '1' }) });
+  assert.deepEqual(r.closed.map((c) => [c.task_id, c.why, c.outcome]), [['11', 'blank', 'abandoned'], ['12', 'nothing_to_apply', 'abandoned']]);
+  for (const c of calls) {
+    assert.equal(c.builderId, '1');
+    assert.equal(c.expectedStatus, 'ready');
+    assert.equal('compose' in c, false, 'no compose: the round is abandoned, never rewritten');
+    assert.ok(c.note.length >= pages.MIN_NOTE_CHARS && c.note.length <= pages.MAX_NOTE_CHARS, 'the note fits the release note floor and cap');
+  }
+});
+
+await test('nothing due, no owner read; no owner, no sweep; no lifecycle, no sweep', async () => {
+  let asked = 0;
+  const quiet = { listPageTweakTasks: async () => [ROUNDS.real_batch], releaseStuckTweak: async () => { throw new Error('must not run'); } };
+  assert.deepEqual(await autoClose.runAutoClose({ now: NOW, resolve: () => quiet, founder: async () => { asked++; return { id: '1' }; } }), { closed: [] });
+  assert.equal(asked, 0);
+  const due = { listPageTweakTasks: async () => [ROUNDS.blank], releaseStuckTweak: async () => { throw new Error('must not run'); } };
+  assert.equal((await autoClose.runAutoClose({ now: NOW, resolve: () => due, founder: async () => null })).skipped, 'no_owner');
+  assert.equal((await autoClose.runAutoClose({ now: NOW, resolve: () => null })).skipped, 'no_lifecycle');
+});
+
+console.log(`\n${passed} passed, ${failed} failed`);
+process.exit(failed ? 1 : 0);

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.16",
+  "version": "1.23.18",
   "lockfileVersion": 3,
   "requires": true,
   "packages": {
     "": {
       "name": "@bongos/core",
-      "version": "1.23.16",
+      "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.16",
+  "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

@@ -10485,5 +10485,17 @@
       "id": "1004900",
       "text": "An artist's rewritten page words are now approved in the Studio before anything changes on the page. Only approved words go on to be applied, and words sent back become a draft the artist can change again."
     }
+  ],
+  "1.23.17": [
+    {
+      "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.16'; // 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 });