@bongos/core 1.19.1066 → 1.19.1068

package/modules/hall-ui/public/studio.states.json

@@ -1,5 +1,5 @@
 {
-  "_": "The Studio (artist mode), R06 of goal 1000074 (task 1003117). A calm room that surfaces the artist's real work — backlog review requests (cascade, R04) and open copy flags (R02) — one piece at a time, never a table. Against the kit's own stub the page renders its EMPTY state (no /tasks or /copy-desk feed), which is a state the floors must hold in. TO SEE IT POPULATED, point the kit at the hall-preview harness instead: `node scripts/hall-preview/server.js --port 4629 --fixture-me` then `render.js --base-in http://builders.localhost:4629 --base-out http://builders.localhost:4629`. That harness answers /tasks from fixtures/tasks.json (whose first row IS a cascade artist review), the reviewed task from fixtures/tasks__detail.json (which carries a visual_url), and the visual itself as a generated stand-in — task 1003829, where the review card's picture became the subject of the card and a broken frame stopped being reviewable. An earlier version of this note named a studio__reviews.json fixture; there is no such file and there never was.",
+  "_": "The studio home (task 1003971 / BV2.TW16), built 1:1 on docs/design/mocks/tweak-mode/Main.dc.html: the night room, the greeting and its sub-line (editable in place as a tweak of builders:studio), the tally, Resume, and three quick actions opening glass panels (the Tweak/CopyWrite Flow, the Approval queue, Artist Review Status). Behind a quiet door, the older reviews (R06 of goal 1000074, task 1003117; retired in TW14) still answer the reviews filed before Tweak Mode. Against the kit's own stub the page renders its unread states (no copy-desk feeds). TO SEE IT POPULATED, point the kit at the hall-preview harness: `node scripts/hall-preview/server.js --port 4636 --fixture-me` then `render.js --base-in http://builders.localhost:4636 --base-out http://builders.localhost:4636`. The harness answers the tally, the pages and the next page from copy-desk__tally.json, copy-desk__pages.json and copy-desk__next.json (built by the real composers), Resume's counter from copy-desk__pages__draft.json, the studio's own lines from copy-desk__pages__draft-studio.json, the claim and the draft save from its CANNED WRITES, and the older reviews from tasks.json (whose first row IS a cascade artist review).",
   "page": "/builders/studio",
   "surface": "hall-ui",
   "stub": {
@@ -25,8 +25,9 @@
     "load failed"
   ],
   "states": {
-    "focus": {
+    "home": {
       "auth": true,
+      "_": "The home: Resume, the three quick actions, the tally top right.",
       "actions": [
         [
           "wait",
@@ -35,13 +36,172 @@
       ],
       "expect": {
         "visible": [
-          "#studio-focus"
+          "#studio-greet",
+          "#st-flow",
+          "#studio-approvals",
+          "#st-status"
         ]
       }
     },
     "out": {
       "auth": false
     },
+    "flow": {
+      "auth": true,
+      "_": "The Tweak/CopyWrite Flow: the recommended page, take it, pick another.",
+      "actions": [
+        [
+          "wait",
+          700
+        ],
+        [
+          "click",
+          "#st-flow"
+        ],
+        [
+          "wait",
+          300
+        ]
+      ],
+      "expect": {
+        "visible": [
+          ".st-dialog",
+          "[data-take]"
+        ]
+      }
+    },
+    "flow-pick": {
+      "auth": true,
+      "_": "Pick another: the other pages as chips.",
+      "actions": [
+        [
+          "wait",
+          700
+        ],
+        [
+          "click",
+          "#st-flow"
+        ],
+        [
+          "wait",
+          300
+        ],
+        [
+          "click",
+          "[data-pick]"
+        ],
+        [
+          "wait",
+          200
+        ]
+      ],
+      "expect": {
+        "visible": [
+          "#st-picks"
+        ]
+      }
+    },
+    "status": {
+      "auth": true,
+      "_": "Artist Review Status: three surface lines, N of M tweaked.",
+      "actions": [
+        [
+          "wait",
+          700
+        ],
+        [
+          "click",
+          "#st-status"
+        ],
+        [
+          "wait",
+          300
+        ]
+      ],
+      "expect": {
+        "visible": [
+          ".st-surfaces"
+        ]
+      }
+    },
+    "words-offer": {
+      "auth": true,
+      "_": "A click on the studio's own words offers to take the studio page.",
+      "actions": [
+        [
+          "wait",
+          700
+        ],
+        [
+          "click",
+          "#studio-greet"
+        ],
+        [
+          "wait",
+          500
+        ]
+      ],
+      "expect": {
+        "visible": [
+          "#st-offer",
+          "#st-offer-take"
+        ]
+      }
+    },
+    "words-edit": {
+      "auth": true,
+      "_": "Taken: the lines the studio's reading lists are typed over in place, saved to its draft.",
+      "actions": [
+        [
+          "wait",
+          700
+        ],
+        [
+          "click",
+          "#studio-greet"
+        ],
+        [
+          "wait",
+          500
+        ],
+        [
+          "click",
+          "#st-offer-take"
+        ],
+        [
+          "wait",
+          700
+        ]
+      ],
+      "expect": {
+        "visible": [
+          "#st-editing"
+        ]
+      }
+    },
+    "reviews": {
+      "auth": true,
+      "_": "The quiet door: the reviews filed before Tweak Mode, one at a time.",
+      "actions": [
+        [
+          "wait",
+          700
+        ],
+        [
+          "click",
+          "#st-older"
+        ],
+        [
+          "wait",
+          900
+        ]
+      ],
+      "expect": {
+        "visible": [
+          "#studio-focus"
+        ]
+      }
+    },
     "full": {
       "auth": true,
       "_": "Full screen (task 1004327): the shell switch pressed — the room fills the window, one button brings the chrome back.",
@@ -61,7 +221,7 @@
       ],
       "expect": {
         "visible": [
-          "#studio-focus",
+          "#studio-greet",
           "#fullscreen-toggle"
         ],
         "hidden": [

package/modules/hall-ui/records/studio.md

@@ -0,0 +1,47 @@
+# The studio — /studio, the lofi home with Tweak Mode as its four doors (task 1003971)
+
+**BV2.TW16 (goal 1000095; [ADR 0341](../../../docs/adr/0341-the-page-is-the-unit-of-tweak-mode.md) D2, D5, D8, D11).** Built 1:1 on the owner-approved master design [`docs/design/mocks/tweak-mode/Main.dc.html`](../../../docs/design/mocks/tweak-mode/README.md) ("I like the lofi design for now"): the night room (the window with its moon, stars and city lights, the floor, the lamp and its glow, the mug and the plant), the eyebrow and the tally pill, "Good evening." and its sub-line, "click any words here to rewrite them", Resume, and three glass quick actions that open panels in place.
+
+Files: `studio.html` (the markup), `studio.css` (the artboard's inline styles, one rule per element, then the older reviews' garden in the hall's tokens), `studio-lib.js` (the pure rules, `window.OTBStudioLib`, node-require-safe) and `studio.js` (the reads, the panels, the words, and the garden). It also loads `tweak-editor-lib.js` for the page names and the draft body, and TW12's Approval queue (`approval-queue*.{js,css}`, `records/approval-queue.md`), unchanged.
+
+- **The four reads.** `GET /copy-desk/tally` (the pill: credits, pages tweaked, this week), `GET /copy-desk/next` (the Flow's recommendation and "pick another", and `resume`, the held page), `GET /copy-desk/pages` (Artist Review Status's three surface lines and the whole count), and, when a page is held, `GET /copy-desk/pages/:id/draft` for Resume's "12 of 38 lines". Signed out, none of them is asked.
+- **Resume** links to `/builders/tweak-editor?page=<held page>` (TW10). With nothing held it is not drawn, and the page's own sub-line stands.
+- **The Flow** shows the recommendation (its `why_line`); "pick another" opens the others as chips and a chip swaps the page in place; "take it" is the claim route on THAT page (ADR 0341 D5), then the editor. A refusal stays in the panel in words.
+- **The Approval queue** is TW12's component: `OTBApprovalQueue.mount({ api, apiBase, writes, onCount })`, opened from the quick action, with `onCount` feeding its detail line.
+- **Artist Review Status** is three meters (landing, builders hall, status, "N of M tweaked"), each against its own surface's page count, so studio.js sits in `tests/hall_audit.mjs` `METERS`. The landing line names the instance's own address, stamped by the server from the branding pack (`<main data-apex="{{publicOrigin}}">`).
+
+## The studio's own words are a tweak of the studio page
+
+The greeting and its sub-line are the studio page's own lines (builders:studio), so rewriting them is a round of that page (ADR 0341 D2), never a live edit. The smallest honest flow:
+
+1. A click on the words (or on the hint, which is a button for the keyboard) reads `GET /copy-desk/pages/builders:studio/draft`. If the artist already holds the studio page, the words open for editing at once.
+2. Otherwise a small glass offer says the words are the studio page itself, with "take the studio page" and "not now". Nothing is claimed until the artist says so.
+3. Taken (the claim route), each word on the page is matched to its line in the studio's reading (`matchLine`: a `{…}` hole matches any run of words, the most specific line wins, a line mostly of holes never matches, shell text never), and the matched elements become `contenteditable`, showing the reading's text, holes and all, as the editor does.
+4. Typing autosaves the WHOLE draft of the studio page (the lines typed here and any the editor saved) through `PUT /copy-desk/pages/builders:studio/draft`, debounced and on blur. A strip under the words says "The studio page is yours · N of M rewritten · saved · submit it in the editor". Submit is the editor's: one submit path, not two.
+
+The page keeps its words until the round is submitted, applied, approved and shipped like any other. A word the reading does not list (a sentence newer than the last reading) keeps its words and loses the cue.
+
+**Why the sub-line's sentence is in studio.js, not studio-lib.js.** The copy registry places a line at its `textContent` assignment. Built inside the lib it read as unplaced, and the reader's pattern tier then matched it to the generic `{…} of {…}` row (sky-panel.js). Written where studio.js assigns it, it is placed on studio.js, which is the page's own file and wins.
+
+## The older reviews stay one quiet door away
+
+The garden this page used to be (R06 of goal 1000074; retired in TW14) still answers the reviews filed before Tweak Mode, because they may hold a deploy under the strict artist gate. It sits behind a quiet line under the actions ("1 review from before Tweak Mode still waits"), shown only while a review waits, and opens in its own dialog in the hall's tokens with the focus card, the stones and the two links, unchanged. Its verdict writes are unchanged.
+
+## What it writes
+
+`WRITE_ROUTES` in studio.js, pinned in `tests/hall_studio_world.mjs`: the four verdicts on an older review, the Approval queue's approve and send-back, and TW16's two, `claim` (take a page's round) and `draft` (the studio page's whole draft). Six write sites; none writes a live string.
+
+## Where it differs from the artboard, and what forced it
+
+- **The words are the ledger's**, not the sample's: 180 credits, 6 pages tweaked, +60 this week, "2 of 75 lines", "6 of 38 pages tweaked" on the harness fixtures (built by the real composers). The artboard's are sample data (the mocks README).
+- **The recommendation order is the spec's** (asked, changed, untweaked), so the chips read Tasks, Goals, Privacy, Terms where the artboard's sample has Work, Privacy, Goals, Explore (ADR 0341 pick 8). "Tasks" is builders:work's own title.
+- **The hint is #85858c, not #6f6f76**: #6f6f76 on the room's #07080d is about 4:1, under the AA floor for 12.5px type.
+- **The panels' scrim is .94 over the light hall and .88 on a phone** (the artboard's .42 over the dark room otherwise), TW12's reason: the glass's grey words fall under AA at .42 over a light page.
+- **A quick action is at least 72px and grows**, with 10px of padding, so a long detail wraps rather than clipping.
+- **The window never slides under the words** (its left edge holds at 680px) when the hall chrome narrows the room.
+- **Resume is not drawn when nothing is held**; the artboard always draws it (sample).
+- **Not on the artboard at all:** the hall chrome view (the full-screen switch off, the default since TW20), the full-screen button top right, light mode, the phone layout (the pill across the top as three stacked figures, the window scaled whole to the top right, the desk on a floor at the bottom), the words' offer, edit strip and refusals, the older reviews' door and dialog, and the panels' loading and could-not-read lines.
+
+## The proof
+
+`studio.states.json` renders sixteen states (home, out, flow, flow-pick, status, words-offer, words-edit, reviews, full, full-back and TW12's six) against the hall-preview harness (`node scripts/hall-preview/server.js --port 4636 --fixture-me`), which answers the tally, the pages and the next page from `copy-desk__tally.json`, `copy-desk__pages.json` and `copy-desk__next.json`, the studio's own lines from `copy-desk__pages__draft-studio.json` (a new fallback ahead of the editor's catch-all), and the claim and the save from its CANNED WRITES. All clean at 1440, 390 and 320, dark and light. The studio fixture carries only the studio's PLACED lines: the page reader treats every string a page reads as data, so a fixture holding its unplaced lines would make the reading drop them. Behaviour: `tests/hall_studio_home.mjs` (the pure rules over the real composers, and the real page in a mini-DOM: the four reads, the panels, take it, and the words typed over and saved as the whole draft).

package/modules/lifecycle/cascade.js

@@ -21,7 +21,9 @@
 // mechanism, its idempotency and its non-blocking property are proved by the
 // tests' own declarations, and R04 (task 1003115) writes the first production
 // entry. A mechanism whose first real rule is written by the task that needs it
-// is honest; one that ships a rule nobody asked for is a guess.
+// is honest; one that ships a rule nobody asked for is a guess. (R04's artist
+// review has since been removed by Tweak Mode, TW14; the removal proposal and
+// the idea cascade remain. See the note above COPY_VISUAL_MODULES.)
 //
 // THE FOUR PROPERTIES THAT MAKE IT SAFE, each of which the tests pin:
 //
@@ -87,32 +89,34 @@
 //               board has not triaged has, precisely, no classification yet.
 //
 // ---------------------------------------------------------------------------
-// The first production rule (R04 of goal 1000074, task 1003115).
+// The artist review that USED to be the first rule (removed, TW14).
 // ---------------------------------------------------------------------------
 //
-// THE ARTIST REVIEW. "Every shipped work with copy or visuals becomes a Review
-// task for an artist, in two steps" (the criterion artist-review-task-on-ship).
-// This is the rule that makes that true, and it is the whole of R04's build: the
-// mechanism (ADR 0242) and its four safety properties already exist and are
-// tested; R04 writes the one declaration.
+// R04 (task 1003115) wrote the first production entry here: `artist-review-on-
+// ship`, which filed a two-step artist review at backlog for every ship with
+// copy or visuals. It is GONE (task 1004325 / BV2.TW14, ADR 0341 D9). The reviews
+// were one per ship, and 34 of the 39 it filed were abandoned, 33 with no note.
+// Tweak Mode replaced it: the PAGE is the unit an artist works on, and a ship
+// that changes a page's words now WRITES NOTHING. The page reads "changed since
+// last tweak" because drift is DERIVED (ADR 0341 D8, modules/copy-desk/
+// page-status.js driftFor): the page's current reading against the lines its
+// last round left. Nothing to store, so nothing here to file.
 //
-// TWO GUARDS THAT KEEP IT FROM MISFIRING, both pinned in the tests:
-//   • It excludes `source === CASCADE_SOURCE`. Without that, the artist review
-//     task it generates — discipline 'artist', so it matches its own predicate —
-//     would on ship generate a review OF the review, and so on. The exclusion
-//     makes the generation a single hop, never a chain.
-//   • "Copy or visuals" is a NARROW, explicit predicate (hasCopyOrVisualWork),
-//     not "every ship". A cascade that fired on everything would bury the artist,
-//     and the criterion's "dismissable rather than nagging" would be a lie.
+// Do not bring a per-ship artist rule back to "mark the page": the mark already
+// exists, derived, and a stored one would be a second record of what the
+// readings and the ledger say.
 //
-// It does NOT block anything (ADR 0241 draws the line: an artist may hold a
-// DEPLOY, decided per project, never a builder's ship; that gate is a separate
-// build). The generated task is a `backlog` request that triage may dismiss.
+// What stays, and why:
+//   • `hasCopyOrVisualWork` stays exported. The grader's advisory reader lens
+//     (ADR 0298, scripts/gds/ship-grade.js) is scoped by it.
+//   • The runner still resolves `ctx.artistGate` and `ctx.changedFiles` when a
+//     caller injects them (routes/lifecycle.js does). No production rule reads
+//     either today; the reads are shared with the removal rule's (ship-files.js
+//     memoises them), so they cost nothing, and ADR 0341 D9 keeps the runner.
+//   • Review rows already filed (source 'cascade', ref prefix `cascade-artist-
+//     review-on-ship-`) are ordinary data. The artist gate (ADR 0241) and the
+//     studio still read them; nothing deletes them.
 
-// The modules whose shipped work puts user-facing copy or visuals in front of a
-// person. Kept as an explicit set rather than inferred, because "is this
-// copy/visual work" is a judgement the rule should state out loud and a test
-// should be able to pin — an inferred predicate is the kind that quietly widens.
 // The removal rule's key, lifted out of its declaration so the SURFACE that
 // answers a removal proposal (task 1004095 / BV2.R16) can find those proposals
 // without hand-typing the string. The key is half of every generated row's
@@ -120,6 +124,11 @@
 // reader that silently returns nothing the day the key changes.
 const REMOVAL_RULE_KEY = 'removal-proposed-on-ship';
 
+// The modules whose shipped work puts user-facing copy or visuals in front of a
+// person. Kept as an explicit set rather than inferred, because "is this
+// copy/visual work" is a judgement the predicate should state out loud and a
+// test should be able to pin — an inferred predicate is the kind that quietly
+// widens.
 const COPY_VISUAL_MODULES = Object.freeze([
   'copy-desk',    // the copy registry + proposals (goal 1000074's own spine)
   'hall-ui',      // the builders' hall — the most-seen surface
@@ -142,8 +151,9 @@ function isUiSurfacePath(p) { return typeof p === 'string' && UI_SURFACE_PATH.te
 // `changedFiles` is what the work actually CHANGED (task 1004201). The label
 // alone missed most UI work: of 19 recent ships that changed page files, 14 had
 // module_key null — the goal forest (task 1003480) shipped its overflowing
-// titles with no visual lens and no review. Owner decision 2026-09-23: every UI
-// change gets a visual review, and this is the predicate that files it.
+// titles with no visual lens. Owner decision 2026-09-23: every UI change gets a
+// visual lens. Since TW14 no cascade files a review on it; the grader's reader
+// lens (ADR 0298) is what this predicate scopes.
 function hasCopyOrVisualWork(task, changedFiles) {
   if (!task) return false;
   if (task.source === 'copy-proposal') return true;        // a copy change that landed via the copy desk (ADR 0233)
@@ -153,44 +163,6 @@ function hasCopyOrVisualWork(task, changedFiles) {
   return false;
 }
 
-// The two-step artist review body. STEP 1 records where the grade was blind;
-// STEP 2 remakes the copy/visuals with both routes explicitly offered — R04's
-// brief, verbatim in intent.
-//
-// STEP 1 ASKS FOR COACHING AGAIN, because there is now a lens listening.
-//
-// The history, because the wording is load-bearing: STEP 1 originally said
-// "coach the grader", which asked for something no artist could do (task
-// 1003937, ADR 0298) — the panel's four lenses read completion, code quality,
-// security and runtime cost, and not one read a sentence, so four consecutive
-// reviews found the same wall. The step was rewritten to ask for the RECORD
-// instead, and that version carried an explicit instruction not to restore the
-// coaching wording early, because the wording is the signal for which world we
-// are in.
-//
-// Task 1003977 shipped the fifth lens: an ADVISORY 'reader', scoped to
-// copy-bearing diffs by THIS FILE'S hasCopyOrVisualWork — the same predicate
-// that fires this cascade, so an artist review and the copy lens cover exactly
-// the same set by construction, and widening one widens both. The voice guide
-// it waited on is docs/voice.md (task 1003976). So there is a counterpart to
-// coach, it has a written standard to be coached against, and the record the
-// interim wording collected is what both were built from.
-//
-// What has NOT changed: the reader is non-gating and cannot hold a ship (ADR
-// 0241 keeps the artist's judgement on the deploy), so this review is still a
-// request triage may dismiss.
-function artistReviewBody(task) {
-  return [
-    `This is an ARTIST REVIEW, generated automatically because task ${task.id} put copy or visuals in front of a person. It is a REQUEST, not a gate — it blocks nothing that shipped, and triage may dismiss it.`,
-    '',
-    `STEP 1 — COACH THE READER. Pull how the grader judged task ${task.id} (GET /tasks/${task.id}?include=grade) and find the 'reader' row in signals.workers, with its copy_score in rubric_scores. That lens read the changed copy against docs/voice.md and cited rule ids. Say where it was RIGHT, where it was WRONG, and — the valuable one — what it did not see at all. A finding it filed that you disagree with is worth as much as one it missed: both are corrections to a lens that is listening, which is what this step could not ask for before task 1003977 shipped it (ADR 0298). If the miss is because no rule in docs/voice.md reaches it, that is a gap in the GUIDE — file it against the guide rather than expecting the lens to invent a rule at grade time. Two cases where there will be no reader row and that is not a fault: a ship that took the trivial-diff path (a single generalist shot, no panel), and one where the lens errored — it is advisory, so an outage is recorded and nothing is blocked.`,
-    '',
-    'STEP 2 — REMAKE OR TWEAK the copy/visuals if they need it. Your choice of route, both offered on purpose: OUTSIDE the AI chat (edit the surface directly and land it through the copy desk or the normal pipeline), or WITH it (draft in chat, then land). Either way it lands as a claimed, graded, shipped task like everything else — there is no live CMS (goal 1000074\'s hard rule).',
-    '',
-    'If the copy and visuals are fine and there was nothing the grade missed, DISMISS this in triage. That is a real, expected outcome — not a failure.',
-  ].join('\n');
-}
-
 // The removal proposal's body (task 1004090 / BV2.R15, ADR 0313).
 //
 // IT ASKS, IT DOES NOT TELL. Every line here is written so a reader confirms or
@@ -235,44 +207,25 @@ function removalProposalBody(task, ctx) {
   return lines.join('\n');
 }
 
-// Production rules. The artist review is the first (R04); add another by
-// appending an entry, never by touching the runner (ADR 0242).
+// Production rules. Add one by appending an entry, never by touching the runner
+// (ADR 0242). The artist review (R04) was the first and is gone (TW14, ADR 0341
+// D9; see the note above COPY_VISUAL_MODULES), so the removal proposal is the
+// only ship rule today.
 const CASCADES = Object.freeze([
-  Object.freeze({
-    key: 'artist-review-on-ship',
-    discipline: 'artist',
-    kind: 'unclassified',
-    // The anti-recursion guard (source !== CASCADE_SOURCE) is FIRST so a
-    // cascade-generated task is rejected before any other test runs.
-    //
-    // The project's artist_gate at 'off' means "not generated" (ADR 0241 §2's
-    // table, task 1003575) — the one level at which this rule files nothing. It
-    // is read from ctx, never from branding, so the predicate stays pure; an
-    // absent ctx (a caller that resolved no policy) files, because the gate's
-    // default is 'strict' and silence must not be read as an opt-out.
-    //
-    // A changed-files list that hit GitHub's read cap FILES (task 1004201): a UI
-    // file may sit past the cap, and a padded PR must not skip its review.
-    when: (t, ctx) => (ctx || {}).artistGate !== 'off'
-      && t.source !== CASCADE_SOURCE
-      && (hasCopyOrVisualWork(t, (ctx || {}).changedFiles) || (ctx || {}).changedFilesTruncated === true),
-    title: (t) => `Artist review: the copy and visuals that shipped in task ${t.id}`,
-    body: artistReviewBody,
-  }),
   Object.freeze({
     key: REMOVAL_RULE_KEY,
     // ENGINEER, not artist: confirming asks whether deleted code is genuinely
     // gone or was moved, which is a question about the diff.
     discipline: 'engineer',
     kind: 'unclassified',
-    // The anti-recursion guard FIRST, the same order and for the same reason as
-    // the rule above — a cascade-generated task must never cascade.
+    // The anti-recursion guard FIRST: a cascade-generated task must never
+    // cascade, so it is rejected before any other test runs.
     //
     // The candidates are resolved ONCE into ctx by runCascadesForShip, never
     // here: a `when` is pure and synchronous by contract, and this derivation is
     // a network read. An absent ctx therefore means "nothing was derived" and
-    // files nothing — the opposite default to the artist gate above, and
-    // deliberately so: silence about a removal must never be read as a removal.
+    // files nothing, deliberately: silence about a removal must never be read
+    // as a removal.
     when: (t, ctx) => t.source !== CASCADE_SOURCE
       && Array.isArray((ctx || {}).removals) && (ctx || {}).removals.length > 0,
     title: (t) => `Was this removed? Task ${t.id} deleted earlier shipped work`,
@@ -418,9 +371,9 @@ function cascadesFor(task, rules = CASCADES, ctx = {}, trigger = TRIGGERS.ship)
 
 // The createTask payload for one matched rule. The attribution preamble is
 // prepended here rather than left to each rule, so no rule can forget it.
-// `ctx` reaches body() as a second argument, mirroring when(). The artist rule
-// ignores it; the removal rule needs it, because what a ship deleted is derived
-// per run and is not a property of the task row.
+// `ctx` reaches body() as a second argument, mirroring when(). The removal rule
+// needs it, because what a ship deleted is derived per run and is not a property
+// of the task row.
 function cascadeTaskSpec(rule, task, trigger = TRIGGERS.ship, ctx = {}) {
   const body = String(rule.body(task, ctx) || '').trim();
   const preamble =
@@ -559,9 +512,9 @@ async function runCascadesForShip(shipped, deps = {}) {
       catch (e) { report.errors.push(`cascade: artistGateMode() threw (${e && e.message}) — policy unread`); }
     }
     // WHAT THIS SHIP CHANGED (task 1004201), from the server's own view of the
-    // merged pull request, so the artist review keys on the work rather than a
-    // module_key label. A resolver that throws is reported and leaves the key
-    // unset: the label-based match still stands and the ship costs nothing.
+    // merged pull request, so a rule can key on the work rather than a
+    // module_key label (the artist review did, until TW14 removed it). A resolver
+    // that throws is reported and leaves the key unset, and the ship costs nothing.
     if (typeof deps.changedFiles === 'function') {
       try {
         const changed = await deps.changedFiles(task);
@@ -650,7 +603,6 @@ module.exports = {
   COPY_VISUAL_MODULES,
   hasCopyOrVisualWork,
   isUiSurfacePath,
-  artistReviewBody,
   cascadeSourceRef,
   assertRuleShape,
   validateCascades,

package/modules/lifecycle/routes/lifecycle.js

@@ -79,16 +79,16 @@ function registerSeams() {
       getTask: (id) => db.getTask(id),
       findTaskBySourceRef: (ref) => db.findTaskBySourceRef(ref),
       createTask: (spec) => db.createTask(spec),
-      // The project's artist gate (ADR 0241, task 1003575). Resolved through the
-      // doorway's one reader rather than from branding().project, so the level that
-      // decides whether a review is FILED and the level that decides whether one
-      // HOLDS a deploy can never be read differently. At 'off' the artist-review
-      // rule files nothing.
+      // The project's artist gate (ADR 0241, task 1003575), resolved through the
+      // doorway's one reader rather than from branding().project. The artist-review
+      // rule that read it was removed by Tweak Mode (TW14, ADR 0341 D9), so no
+      // production rule reads it today; the runner still offers it as policy.
       artistGateMode: () => api.artistGate.artistGateMode(),
-      // WHAT THIS SHIP CHANGED (task 1004201): the merged PR's paths, so a UI
-      // change files its artist review whatever its module_key label says. The
-      // NAMED function, not a closure, for the reason removalCandidates gives below;
-      // the arrow only hands it this ship's shared reads.
+      // WHAT THIS SHIP CHANGED (task 1004201): the merged PR's paths. No rule reads
+      // them since TW14, and they cost nothing extra: sharedPrReads memoises the
+      // same two GitHub reads removalCandidates makes. The NAMED function, not a
+      // closure, for the reason removalCandidates gives below; the arrow only hands
+      // it this ship's shared reads.
       changedFiles: (task) => shipFiles.changedFilesForTask(task, pr),
       // WHAT THIS SHIP DELETED (task 1004090 / BV2.R15, ADR 0313). The derivation
       // asks GitHub what the pull request actually changed and blames the deleted