@bongos/core 1.19.1065 → 1.19.1066

package/.bongos-core.json

@@ -2,22 +2,22 @@
   "artifact": "bongos-core",
   "manifest_schema": 1,
   "generator": "scripts/gds/package-core.js",
-  "core_version": "1.19.1065",
-  "core_contract": "1.19.1065",
-  "source_commit": "9eb2bc904c90fb783c60a1f9dc5c25e5e70682cb",
+  "core_version": "1.19.1066",
+  "core_contract": "1.19.1066",
+  "source_commit": "75a6b51abf5c23eb683f15eb0d4f8a7387803a8f",
   "source_ref": "HEAD",
-  "built_at": "2026-09-28T16:27:32.366Z",
+  "built_at": "2026-09-28T16:56:08.158Z",
   "redaction": {
     "model": "docs-redacted+functional-verbatim",
     "docs_redacted": 546,
     "agent_docs_stubbed": 26,
-    "functional_verbatim": 2556,
+    "functional_verbatim": 2557,
     "rules": 3,
     "gate_literals": 3,
     "gate": "passed"
   },
-  "file_count": 3129,
-  "tree_sha256": "663cedc98127e9b0caa2437170740a645d68b7988452df288137ca46fcc0a534",
+  "file_count": 3130,
+  "tree_sha256": "2f41769464ed9567654e2b042a9946102caa4a0e4cdf1d9bf93ccee84aa79855",
   "files": [
     {
       "path": ".claude/skills/ask-for-help/SKILL.md",
@@ -2172,7 +2172,7 @@
     {
       "path": "docs/adr/0341-the-page-is-the-unit-of-tweak-mode.md",
       "mode": "0000644",
-      "sha256": "ab4cefdfbe2505dabe75298a1f2ac98adee1dcbb205c3879d91813d242bdf60f"
+      "sha256": "058c9eb62763fd8c0459277a7ab904010d79589027f8f4920dd6de6ad938dc3b"
     },
     {
       "path": "docs/adr/README.md",
@@ -2197,7 +2197,7 @@
     {
       "path": "docs/branding-contract.md",
       "mode": "0000644",
-      "sha256": "ac14c9d29de36fdcc10553c603d9768e4dccd315a0815f6201439ee80e7177f2"
+      "sha256": "40630abe085d54562abbeea3b8c046c03b4245363a2a37c4eea086940684e19f"
     },
     {
       "path": "docs/branding-fill-prompt.md",
@@ -2747,7 +2747,7 @@
     {
       "path": "docs/module-api-changelog.md",
       "mode": "0000644",
-      "sha256": "218d72a360ed0c079b5428101a3b804789e2df510bb8b223a04d3002a2c7ac99"
+      "sha256": "ea2d1bdda788f09e2665dd8a924c78e9618197b6e602ef1f2dfaddd96b8ca938"
     },
     {
       "path": "docs/modules-contract.md",
@@ -4152,7 +4152,7 @@
     {
       "path": "modules/copy-desk/page-status.js",
       "mode": "0000644",
-      "sha256": "c3f403aed75d46680c4ffde57202ce17f01830d498688081d1c8202865d25900"
+      "sha256": "0b96c931e6e0f8a22a3c4de287f698b9f74a45183f1b4314f66b8dfe643d33f2"
     },
     {
       "path": "modules/copy-desk/pages.js",
@@ -4172,7 +4172,7 @@
     {
       "path": "modules/copy-desk/routes/copy-desk.js",
       "mode": "0000644",
-      "sha256": "b907b07cf1626c49eb30d7f07aede6434b48d7015d9f935ebc805c42a176a106"
+      "sha256": "175a45af668fbe7405dddfba3ea922e0021dbd2d59dcfdfb6e947351c5bfcfe8"
     },
     {
       "path": "modules/copy-desk/tests/copy_docx.mjs",
@@ -4407,7 +4407,7 @@
     {
       "path": "modules/economy/credits.js",
       "mode": "0000644",
-      "sha256": "6b5700a6e1d7689496e93d5f1de9f9c32766842a389571135b4f3af950278ce0"
+      "sha256": "f26603fecd1771d8286fc471f9c9c51ad96774b23967811de621bbf96dd482fc"
     },
     {
       "path": "modules/economy/idea-brightness.js",
@@ -4442,12 +4442,12 @@
     {
       "path": "modules/economy/reward-policy.js",
       "mode": "0000644",
-      "sha256": "817f400faa29aabcfdeb98eabeca45790203dff2dbca2ed01336d69078dd2be2"
+      "sha256": "39b06cfa5ae58fce68633f6b2aef74eb9421bf3346cc54a0658b0186136551cf"
     },
     {
       "path": "modules/economy/reward.js",
       "mode": "0000644",
-      "sha256": "79f225665dfc84fce1c50d01a25bbf81986c91047732d04113f24a68f766d29a"
+      "sha256": "4c54857aea9bc12c59e712410b0b98124d0274dece739bfbd844f52dc6ac6880"
     },
     {
       "path": "modules/economy/routes/cost.js",
@@ -6092,7 +6092,7 @@
     {
       "path": "modules/lifecycle/db-ship.js",
       "mode": "0000644",
-      "sha256": "a774d51dd8bc9d6b1e1bd52f9b0da0f240d0881ff2ab4218a2ffe7af56366fc6"
+      "sha256": "7d43343a2db27ab3a281bb301cc2f3071698aa9a0b66b280b88fe348b7a61b16"
     },
     {
       "path": "modules/lifecycle/db-tasks.js",
@@ -8627,12 +8627,12 @@
     {
       "path": "package-lock.json",
       "mode": "0000644",
-      "sha256": "a2f47f997bc868bfdaa6c70685bb7635796123501fc16f5e0476848819d32925"
+      "sha256": "dedef6af9441180d979a552db8177040fe36401288584d25a19af72bddb17785"
     },
     {
       "path": "package.json",
       "mode": "0000644",
-      "sha256": "7f750a2a729e5cff3f5aa9cbda69a8072378a50e8320f7bb69bcdfe45aff7571"
+      "sha256": "5351891c0be6644fc56740ac01c2c52d8fb7bc84b202b351cfb239f4b34dcff1"
     },
     {
       "path": "public-docs/index.html",
@@ -8652,7 +8652,7 @@
     {
       "path": "release-notes.json",
       "mode": "0000644",
-      "sha256": "4b4f065b5c1c07d6cd4529e5c80c7c18852a6399de67fc6d754ace363005d67e"
+      "sha256": "08071a7100ca28392f74d548016d7cb2ed0e354b7d0d2bce1a0a9c565cea4f10"
     },
     {
       "path": "scripts/bongos-mcp.js",
@@ -10742,7 +10742,7 @@
     {
       "path": "src/module-api.js",
       "mode": "0000644",
-      "sha256": "035ede76c10191aa4ec780853ba1dca6cd8198d46dc06917e1f9b16f5d8c55c1"
+      "sha256": "45f6883c5bc140e80f36240647a5ab4ec0840a9967c504ea959f7eadb7257a1d"
     },
     {
       "path": "src/module-loader/catalog.js",
@@ -11069,6 +11069,11 @@
       "mode": "0000644",
       "sha256": "3337ab686f2fa4bef95d0eabcf9b134a9b919d40480750e07a500b8007ae9659"
     },
+    {
+      "path": "tests/artist_page_credit.mjs",
+      "mode": "0000644",
+      "sha256": "df3266f78209e0ca45fa9cc6be2b52bc299bfb4bf93dd360196054812166e5cc"
+    },
     {
       "path": "tests/asset_version.mjs",
       "mode": "0000644",

package/docs/adr/0341-the-page-is-the-unit-of-tweak-mode.md

@@ -141,6 +141,7 @@ An artist working in the studio runs no session, so stream 2 can never reach the
 - **The applier keeps its normal build reward** (the cost-plus session reward, on land, unchanged). **The shipper's per-task estimate is not paid on a page-tweak task**, because the artist lane takes that stream, so the total is conserved in both reward modes.
 - **Self-applied** (the artist also ran `/tweak`): one artist row plus their own session reward. Nothing is doubled.
 - The tally (D8) reads these rows.
+- **As built** (task 1004324, BV2.TW13): `awardArtistPageCredit` in `modules/economy/credits.js`, called from `awardLandingCredit` in `modules/lifecycle/db-ship.js` in place of the estimate on a `page-tweak` task. The idempotency guard keys on `(task_id, reason)`, one payment per round, which is stricter than "(builder, reason)" and implies it, since a round has one payee. The row must read `shipped` in the ship transaction, so a sent-back round is never paid. A round with no web claim, or a zero priced value, books nothing.
 
 ### D11. The routes, in one place
 

package/docs/branding-contract.md

@@ -36,7 +36,7 @@ env override   →   instance config (config/branding.json)   →   neutral star
 | `repo` | `{ owner, name }` GitHub binding. |
 | `db.database` | DB name Bongos pool connects to when neither `DATABASE_URL` nor `PGHOST`/`PGDATABASE` is set (`src/bongos/pool.js`, ADR 0062 §5). `null` in the neutral starter = **fail loud** rather than silently target a host DB. |
 | `currency` | `{ label, symbol }` — the reward unit label (e.g. "example"). Display only. |
-| `reward` | `{ marginPct, usdPeg, mode }` — the per-instance reward POLICY the economy module reads ([ADR 0054](adr/0054-session-token-reward-cost-plus.md) + ADR 0146, tasks 2009/1002270): the cost-plus margin percent (default `20`), the example-per-USD peg (default `1`), and `mode` — `"cost-plus-and-estimate"` (default; pay both the per-task author estimate AND the cost-plus session reward) or `"cost-plus-only"` (suppress the per-task estimate payout; the cost-plus session reward is the sole equity reward). A committed, code-reviewed change — not a live UI knob. Omit ⇒ code defaults (20% margin, 1:1 peg, pay both streams). |
+| `reward` | `{ marginPct, usdPeg, mode }` — the per-instance reward POLICY the economy module reads ([ADR 0054](adr/0054-session-token-reward-cost-plus.md) + ADR 0146, tasks 2009/1002270): the cost-plus margin percent (default `20`), the example-per-USD peg (default `1`), and `mode` — `"cost-plus-and-estimate"` (default; pay both the per-task author estimate AND the cost-plus session reward) or `"cost-plus-only"` (suppress the per-task estimate payout; the cost-plus session reward is the sole equity reward). Two per-craft lanes sit beside `mode` and are not derived from it: `ideatorCredit` ([ADR 0172](adr/0172-per-craft-compensation-ideator-credit-lane.md)) and `artistCredit` ([ADR 0341](adr/0341-the-page-is-the-unit-of-tweak-mode.md) D10, an approved page tweak pays its artist), each default on and withheld only by an explicit `false`. A committed, code-reviewed change — not a live UI knob. Omit ⇒ code defaults (20% margin, 1:1 peg, pay both streams, both lanes on). |
 | `models` | `{ main, subagentDefault, subagentRoutine }` — the per-instance subagent model-allocation DEFAULT the builder-settings module reads (task 2011): the tier for the main session, subagent-default work, and routine plumbing (tiers: `haiku`/`sonnet`/`opus`/`fable`). A per-builder override resolves over this. Omit ⇒ code defaults (`opus`/`opus`/`haiku`). |
 | `tracks` | The configurable version-track set (replaces the hardcoded `product\|internal`). |
 | `theme` | `{ palette, mode, ui }` — palette file ref + theme mode, plus `ui`, the WEB look `themeMarkup()` splices into every served page as a plain `:root` after the page's stylesheet: `colors` (the thirteen contract tokens — **light-mode values**; each stylesheet derives its own dark tier from them at `:root[data-mode="dark"]` + its `prefers-color-scheme` twin, which outrank the pack), `fonts` (`display`, `body`, `links`), `favicon`, and `backdrop` — a `body{background:…}` rule that outranks every sheet in **both** modes, so a dual-mode world sets it to `var(--bg)` (the neutral pack does since [task 1003298](https://cloudbongos.com/builders#/task/1003298), ADR 0196) rather than a literal. The neutral values are the chrome world's; `config/design-tokens.neutral.json` must mirror `colors` + `fonts` key for key (`tests/design_tokens_sync.mjs`), and the world's derived recipes are recorded in `DESIGN.md`. The ui-design module's **style library** (`modules/ui-design/styles/`, [ADR 0219](adr/0219-a-look-is-a-branding-pack-the-style-library.md)) ships ready-made `theme.ui` blocks (looks) an instance copies into this field; every served sheet reads only these fifteen, so a look re-themes every surface with no page edit. |

package/docs/module-api-changelog.md

@@ -2617,5 +2617,7 @@ is load-bearing: the script throws rather than guess if it is missing, and
         landed since 1.19.1063 with no explicit bump. run 36441707814. (task 1002620)
   1.19.1065 — CI auto-patch (publish-on-merge, ADR 0161): carrier for merges
         landed since 1.19.1064 with no explicit bump. run 36451089716. (task 1002620)
+  1.19.1066 — CI auto-patch (publish-on-merge, ADR 0161): carrier for merges
+        landed since 1.19.1065 with no explicit bump. run 36454504693. (task 1002620)
 ---------------------------------------------------------------------------
 ```

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

@@ -36,8 +36,10 @@ const pages = require('./pages');
 
 const TERMINAL = new Set(['shipped', 'abandoned']);
 
-// The credit reason ADR 0341 D10 books an approved page under. TW13 writes it;
-// until then no row carries it and every tally reads 0, which is the truth.
+// The credit reason ADR 0341 D10 books an approved page under. The economy
+// module writes it at the land (credits.js awardArtistPageCredit, task 1004324,
+// its ARTIST_PAGE_REASON_PREFIX); restated here because a module cannot require
+// another, and tests/artist_page_credit.mjs holds the two strings together.
 const TALLY_REASON_PREFIX = 'tweak.page_approved:';
 const WEEK_MS = 7 * 24 * 60 * 60 * 1000;
 

package/modules/copy-desk/routes/copy-desk.js

@@ -361,8 +361,9 @@ function pageReadsOrFail(res) {
 }
 
 // Every `tweak.page_approved:` credit row the caller has, through the reward
-// port (credit_log is economy's table). No reward port, or no rows yet (TW13
-// has not built the payout), reads as none: a tally of 0 is the truth then.
+// port (credit_log is economy's table). No reward port, or no approved page
+// landed yet (economy books one row per landed round, task 1004324), reads as
+// none: a tally of 0 is the truth then.
 //
 // ONE bounded query. An approved page books one row, so the cap is thousands of
 // pages for one artist; past it the tally says `truncated` (creditLedger's

package/modules/economy/credits.js

@@ -1174,6 +1174,80 @@ async function sumSessionEarnings(executor, builderId, taskIds, sessionId) {
   return rows[0] ? rows[0].total : 0;
 }
 
+// =========================================================================
+// The ARTIST credit lane (task 1004324 / BV2.TW13, ADR 0341 D10).
+// =========================================================================
+// Tweak Mode replaced the per-ship artist review with an approved page, and the
+// owner's rule is the same payouts. A review paid its PRICED value,
+// credits_reward x kind multiplier, as the per-task estimate, which cost-plus-only
+// withholds, so paid literally a studio artist would earn nothing on this
+// instance. So an approved page pays that priced value (30 for a feature round
+// at 30) on its own lane, `artistCredit`, which cost-plus-only does not touch:
+// the ideator lane's precedent (ADR 0172). The lifecycle calls this at the land
+// INSTEAD of the shipper's per-task estimate, so the total is conserved in both
+// modes, and the applier keeps its cost-plus session reward, which is booked
+// elsewhere and is untouched here.
+//
+// The reason prefix is the one copy-desk's tally already sums
+// (modules/copy-desk/page-status.js TALLY_REASON_PREFIX, restated there because
+// a module cannot require another; tests/artist_page_credit.mjs holds the two
+// strings together).
+const ARTIST_PAGE_REASON_PREFIX = 'tweak.page_approved:';
+const PAGE_TWEAK_SOURCE = 'page-tweak';
+
+// Everything is read from the ledger inside the caller's transaction, never
+// taken from the caller (ADR 0185): the task's own source, status and priced
+// value, and the payee, the builder of the round's LATEST WEB claim (both
+// worktree_name and creator_session_id null), the claim the submit released.
+// That is the TW05 reads' artist rule (modules/lifecycle/page-tweak-reads.js),
+// so the changelog's artist and the paid artist are the same person.
+//
+// NEVER PAID UNLESS SHIPPED. The row must read `shipped` in this transaction, so
+// a round that was sent back, abandoned or is still waiting pays nothing, even
+// if a caller reaches this early.
+//
+// ONE PAYMENT PER ROUND. The NOT EXISTS keys on (task_id, reason), served by
+// idx_credit_log_task. That is stricter than ADR 0341's "(builder, reason)": a
+// round has one payee, so keying on the round alone also refuses a second row
+// should the payee ever read differently on a re-sweep, and a self-applied round
+// (the artist also ran /tweak) is still one artist row, never two.
+//
+// Returns { credited, reason, artistId, delta } with a reason on every path:
+// 'paid' · 'already_paid' · 'artist_lane_off' · 'not_found' · 'not_a_page_tweak'
+// · 'not_shipped' · 'no_artist' · 'zero_value'.
+async function awardArtistPageCredit(client, { taskId, config = rewardConfig() } = {}) {
+  if (config && config.artistCredit === false) return { credited: 0, reason: 'artist_lane_off' };
+  const { rows } = await client.query(
+    `SELECT t.source, t.status, t.credits_reward, t.kind,
+            (SELECT c.builder_id FROM claims c
+              WHERE c.task_id = t.id AND c.worktree_name IS NULL AND c.creator_session_id IS NULL
+              ORDER BY c.claimed_at DESC, c.id DESC
+              LIMIT 1) AS artist_id
+       FROM tasks t
+      WHERE t.id = $1`,
+    [taskId]
+  );
+  const t = rows[0];
+  if (!t) return { credited: 0, reason: 'not_found' };
+  if (t.source !== PAGE_TWEAK_SOURCE) return { credited: 0, reason: 'not_a_page_tweak' };
+  if (t.status !== 'shipped') return { credited: 0, reason: 'not_shipped' };
+  if (t.artist_id == null) return { credited: 0, reason: 'no_artist' };
+  // The review's priced value, by the one multiplier rule (rounded once, there).
+  const priced = applyKindMultiplier(t.credits_reward, t.kind);
+  if (priced.effective <= 0) return { credited: 0, reason: 'zero_value', artistId: t.artist_id };
+  const reason = `${ARTIST_PAGE_REASON_PREFIX}${taskId}`;
+  const { rowCount } = await client.query(
+    `INSERT INTO credit_log (builder_id, task_id, delta, reason, description)
+     SELECT $1, $2, $3, $4, $5
+      WHERE NOT EXISTS (SELECT 1 FROM credit_log WHERE task_id = $2 AND reason = $4)`,
+    [t.artist_id, taskId, priced.effective, reason,
+      `Approved page tweak, task #${taskId}: ${priced.description} to its artist (artist lane, ADR 0341 D10)`]
+  );
+  return rowCount > 0
+    ? { credited: priced.effective, reason: 'paid', artistId: t.artist_id, delta: priced.effective }
+    : { credited: 0, reason: 'already_paid', artistId: t.artist_id, delta: priced.effective };
+}
+
 // The three ideator streams (ADR 0172) each book under a reason key that EMBEDS
 // the idea (and, for stream 1, the task) — that string IS the idempotency
 // predicate, which is why it can't be a bare constant:
@@ -1294,4 +1368,7 @@ module.exports = {
   // the ledger read surface (task 1003484, blocker 1000127)
   creditLedger,
   IDEA_STREAM_REASON_PREFIXES,
+  // the artist lane (task 1004324 / BV2.TW13, ADR 0341 D10)
+  awardArtistPageCredit,
+  ARTIST_PAGE_REASON_PREFIX,
 };

package/modules/economy/reward-policy.js

@@ -61,6 +61,12 @@ function rewardPolicy(rawReward) {
   const perTaskCredit = mode !== REWARD_MODE_COST_PLUS_ONLY;
   // Default-on, opt-out-only: only an explicit `false` withholds the ideator lane.
   const ideatorCredit = r.ideatorCredit !== false;
+  // The ARTIST lane (task 1004324 / BV2.TW13, ADR 0341 D10): an approved page
+  // pays its artist on its own lane, the ideator lane's precedent exactly. Not
+  // derived from `mode`, so cost-plus-only does not withhold it: a studio artist
+  // runs no session, so the cost-plus reward can never reach them. Default-on,
+  // opt-out-only, the same rule as ideatorCredit.
+  const artistCredit = r.artistCredit !== false;
   // Validated to [0, 1): see IDEA_DESCENT_DECAY above — a rate of 1 or more does
   // not converge, and a negative one would pay a descent backwards.
   // ABSENT IS THE DEFAULT, NOT ZERO — and Number(null) is 0, which is a legal
@@ -71,7 +77,7 @@ function rewardPolicy(rawReward) {
   const ideaDescentDecay = Number.isFinite(rawDecay) && rawDecay >= 0 && rawDecay < 1
     ? rawDecay
     : IDEA_DESCENT_DECAY;
-  return { marginPct, usdPeg, mode, perTaskCredit, ideatorCredit, ideaDescentDecay };
+  return { marginPct, usdPeg, mode, perTaskCredit, ideatorCredit, artistCredit, ideaDescentDecay };
 }
 
 module.exports = {

package/modules/economy/reward.js

@@ -40,6 +40,10 @@ module.exports = {
   // `client` (docs/specs/board-room-pass-event-contract.md) — the port is how
   // money crosses the wall; the `board.item.passed` event never carries a payment.
   awardIdeaGovernancePassCredit: (...a) => credits.awardIdeaGovernancePassCredit(...a),
+  // BV2.TW13 / ADR 0341 D10 — the artist lane: an approved page, at its land,
+  // pays its artist the review's priced value. The lifecycle's landing hook calls
+  // it INSTEAD of the per-task estimate on a page tweak; the payee is read here.
+  awardArtistPageCredit: (...a) => credits.awardArtistPageCredit(...a),
   awardNetNegativeBonus: (...a) => credits.awardNetNegativeBonus(...a),
   awardSessionTokenReward: (...a) => credits.awardSessionTokenReward(...a),
   // gamification hooks the lifecycle fires at claim/ship

package/modules/lifecycle/db-ship.js

@@ -18,6 +18,7 @@ const doneWhen = require('./done-when.js');
 const { THETES_GRADUATION_THRESHOLD, maybeAutoGraduateToThetes, foldWorktreeName } = require('./db-rank-authz.js');
 const { mostRecentClaimHolderId, sealSessionLog, staleClaimHourPredicate } = require('./db-shared.js');
 const { LIVE_STATUSES } = require('./dead-deps.js');
+const { isPageTweak } = require('./page-tweak-hold.js');
 
 // shouldRefuseClaimResolve — may THIS caller resolve THIS claim? (task 1003767)
 //
@@ -346,8 +347,15 @@ async function resolveClaim({ claimId, builderId, outcome, notesMd, valueSummary
 // row, re-landing books a FRESH 'task.shipped' row — the reward is fully
 // re-earnable, exactly as the ADR promises. Caller passes its ship txn client
 // so the credit commits atomically with the status flip.
-async function awardLandingCredit(client, { taskId, creditedBuilderId, creditsReward, kind }) {
+async function awardLandingCredit(client, { taskId, creditedBuilderId, creditsReward, kind, source = null }) {
   const reward = seams.resolveOptional('reward');
+  // ADR 0341 D10 (task 1004324): a page tweak's per-task stream is its ARTIST's, on the
+  // artist lane, in place of the shipper's estimate (economy reads the payee); the
+  // applier keeps its cost-plus session reward, which is booked elsewhere.
+  if (reward && isPageTweak({ source })) {
+    const artist = reward.awardArtistPageCredit ? await reward.awardArtistPageCredit(client, { taskId }) : null;
+    return { creditsAwarded: 0, ideaBonus: null, artist };
+  }
   if (!reward || creditedBuilderId == null || !(Number(creditsReward) > 0)) {
     return { creditsAwarded: 0, ideaBonus: null };
   }
@@ -510,7 +518,7 @@ async function shipTask({ taskId, builderId, landAssurance }, deps = {}) {
   // under NODE_ENV==='test', so a regression test can drive the txn with a
   // SQL-aware fakeClient. Production callers omit deps → module pool.
   const activePool = (deps.pool && process.env.NODE_ENV === 'test') ? deps.pool : pool;
-  const { shipRow, graduationBuilder, rankGraduation, actualShipperId, closedClaimCount, creditedBuilderId, creditsAwarded, ideaBonus, justUnlocked } = await withTx(async (client) => {
+  const { shipRow, graduationBuilder, rankGraduation, actualShipperId, closedClaimCount, creditedBuilderId, creditsAwarded, ideaBonus, artistCredit, justUnlocked } = await withTx(async (client) => {
     const { rows } = await client.query(
       `UPDATE tasks
           SET status = 'shipped',
@@ -521,7 +529,7 @@ async function shipTask({ taskId, builderId, landAssurance }, deps = {}) {
               needs_rebase_at = NULL,
               updated_at = now()
         WHERE id = $1 AND status = 'confirmed'
-        RETURNING id, title, status, shipped_at, shipped_by, security_sensitive, value_summary, credits_reward, kind`,
+        RETURNING id, title, status, shipped_at, shipped_by, security_sensitive, value_summary, credits_reward, kind, source`,
       [taskId, builderId]
     );
     if (rows.length === 0) {
@@ -626,6 +634,7 @@ async function shipTask({ taskId, builderId, landAssurance }, deps = {}) {
       creditedBuilderId,
       creditsReward: rows[0].credits_reward,
       kind: rows[0].kind,
+      source: rows[0].source,
     });
     // task 1679 pattern: state-based achievement reconcile now that the credit
     // (above) + the ship count are reflected in this txn. SAVEPOINT-wrapped so
@@ -647,7 +656,7 @@ async function shipTask({ taskId, builderId, landAssurance }, deps = {}) {
     return {
       shipRow: rows[0], graduationBuilder, rankGraduation, actualShipperId,
       closedClaimCount: closedClaims.length,
-      creditedBuilderId, creditsAwarded: landing.creditsAwarded, ideaBonus: landing.ideaBonus, justUnlocked,
+      creditedBuilderId, creditsAwarded: landing.creditsAwarded, ideaBonus: landing.ideaBonus, artistCredit: landing.artist || null, justUnlocked,
     };
   }, { pool: activePool });
   if (closedClaimCount > 0) {
@@ -677,7 +686,7 @@ async function shipTask({ taskId, builderId, landAssurance }, deps = {}) {
     // ADR 0120: the per-task reward that landed on THIS ship (0 on a re-ship /
     // reconciler re-sweep — idempotent). The ship route surfaces it so the
     // builder sees "credits landed" at ship, not at confirm.
-    creditedBuilderId, creditsAwarded, ideaBonus, justUnlocked,
+    creditedBuilderId, creditsAwarded, ideaBonus, artistCredit, justUnlocked,
   };
 }
 

package/package-lock.json

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

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@bongos/core",
-  "version": "1.19.1065",
+  "version": "1.19.1066",
   "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

@@ -7691,5 +7691,11 @@
       "id": "1004323",
       "text": "An artist can now approve a tweaked page from the studio. A new Approval queue button opens a glass panel that shows the applied page as real pictures, before and after, on desktop and phone, in light and dark, with the change"
     }
+  ],
+  "1.19.1066": [
+    {
+      "id": "1004324",
+      "text": "When an artist's page rewrite is approved and goes live, the artist is now paid 30 credits for it, the same value an artist review used to carry. The payment lands at the moment the page ships, never at approval, and a page th"
+    }
   ]
 }

package/src/module-api.js

@@ -71,7 +71,7 @@ const { responsibilityFor, ROLE_RESPONSIBILITIES } = require('./role-responsibil
 // there. scripts/gds/bump-version.js still rewrites the literal below; it appends
 // the entry to that file. Look for a version's history there, not here.
 // ---------------------------------------------------------------------------
-const CORE_VERSION = '1.19.1065'; // CI auto-patch carrier (ADR 0161); changelog: docs/module-api-changelog.md
+const CORE_VERSION = '1.19.1066'; // 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('dev-box'); log.info('mounted');

package/tests/artist_page_credit.mjs

@@ -0,0 +1,272 @@
+// tests/artist_page_credit.mjs
+//
+// Task 1004324 / BV2.TW13, ADR 0341 D10: an approved page pays its ARTIST at
+// the land, as the artist reviews it replaced paid.
+//
+// The contract this pins, through the REAL db.shipTask and the REAL economy
+// writer over a small stateful ledger (tasks, claims, credit_log):
+//   * a shipped page tweak books ONE credit_log row, reason
+//     tweak.page_approved:<task id>, to the builder of the round's latest WEB
+//     claim, worth credits_reward x KIND_MULTIPLIERS[kind] (30 for a feature
+//     round at 30);
+//   * the shipper's per-task estimate (task.shipped) is NOT paid on a page
+//     tweak, so the per-task stream's total is the same as an ordinary ship's;
+//   * a self-applied round (the artist also ran /tweak) is one artist row;
+//   * rounding is the one multiplier rule, once;
+//   * a re-ship or a reconciler re-sweep books nothing a second time;
+//   * a sent-back round is never paid, and a re-applied round pays its artist,
+//     never either applier;
+//   * the lane is not withheld by cost-plus-only, and only an explicit
+//     artistCredit:false turns it off;
+//   * the reason prefix is the one the studio tally sums, and the tally reads
+//     the booked row as 30 credits.
+//
+// Run:  node tests/artist_page_credit.mjs
+
+import { strict as assert } from 'node:assert';
+import { createRequire } from 'node:module';
+import { makeRunner, makeSqlAwareClient } from './helpers.mjs';
+
+process.env.NODE_ENV = 'test';
+
+const require = createRequire(import.meta.url);
+const achievements = require('../modules/economy/achievements.js');
+achievements.evaluateAfterShip = async () => [];
+achievements.evaluateAfterCreditAward = async () => [];
+achievements.reconcileBuilderAchievements = async () => [];
+const seams = require('../src/module-seams.js');
+if (!seams.hasProvider('reward')) seams.registerProvider('reward', require('../modules/economy/reward.js'));
+const credits = require('../modules/economy/credits.js');
+const db = require('../modules/lifecycle/db.js');
+const pageStatus = require('../modules/copy-desk/page-status.js');
+
+const { test, summary } = makeRunner();
+
+const ARTIST = '41';
+const APPLIER = '42';
+const APPLIER_2 = '43';
+const OTHER_ARTIST = '44';
+const RECONCILER = '999';
+const LAND_PROOF = { kind: 'proof', reason: 'test fixture: land proven' };
+
+// The claims a round collects, in order. A web claim has no worktree and no
+// session; the applier's CLI claim has a worktree.
+const web = (builder, at) => ({ builder_id: builder, claimed_at: at, worktree_name: null, creator_session_id: null });
+const cli = (builder, at) => ({ builder_id: builder, claimed_at: at, worktree_name: `tw-${builder}-${at}`, creator_session_id: `s-${builder}` });
+
+// A stateful ledger for one task. It answers the statements shipTask and the
+// artist writer send; credit_log is a real list, so the NOT EXISTS guards are
+// evaluated against what was actually booked.
+function ledger({ source = 'page-tweak', status = 'confirmed', creditsReward = 30, kind = 'feature', claims = [] } = {}) {
+  const state = {
+    task: { id: 7001, title: 'Tweak: Home (landing:index)', source, status, credits_reward: creditsReward, kind, shipped_by: null },
+    claims: claims.map((c, i) => ({ id: 9000 + i, released_at: 'earlier', ...c })),
+    credit_log: [],
+  };
+  const latest = (list) => list.slice().sort((a, b) => (a.claimed_at - b.claimed_at) || (a.id - b.id)).slice(-1)[0] || null;
+  const pool = makeSqlAwareClient((s, p) => {
+    const t = state.task;
+    if (/^UPDATE tasks SET status = 'shipped'/.test(s)) {
+      if (t.status !== 'confirmed') return { rows: [] };
+      t.status = 'shipped';
+      t.shipped_by = t.shipped_by ?? p[1];
+      return { rows: [{ id: t.id, title: t.title, status: 'shipped', shipped_at: new Date().toISOString(), shipped_by: t.shipped_by,
+        security_sensitive: false, value_summary: null, credits_reward: t.credits_reward, kind: t.kind, source: t.source }] };
+    }
+    if (/^UPDATE claims/.test(s)) return { rows: [] };
+    if (/^SELECT builder_id FROM claims WHERE task_id = \$1/.test(s)) {
+      const c = latest(state.claims);
+      return { rows: c ? [{ builder_id: c.builder_id }] : [] };
+    }
+    if (/^SELECT 1 FROM credit_log WHERE task_id = \$1 AND reason = 'task\.shipped'/.test(s)) {
+      return { rows: state.credit_log.filter((r) => r.task_id === p[0] && r.reason === 'task.shipped').map(() => ({ one: 1 })) };
+    }
+    if (/^SELECT t\.source, t\.status, t\.credits_reward, t\.kind/.test(s)) {
+      if (p[0] !== t.id) return { rows: [] };
+      assert.match(s, /c\.worktree_name IS NULL AND c\.creator_session_id IS NULL/, 'the payee is read from WEB claims only');
+      const w = latest(state.claims.filter((c) => c.worktree_name == null && c.creator_session_id == null));
+      return { rows: [{ source: t.source, status: t.status, credits_reward: t.credits_reward, kind: t.kind, artist_id: w ? w.builder_id : null }] };
+    }
+    if (/^INSERT INTO credit_log/.test(s)) {
+      const [builder_id, task_id, delta, reason] = p;
+      if (/WHERE NOT EXISTS \(SELECT 1 FROM credit_log WHERE task_id = \$2 AND reason = \$4\)/.test(s)
+        && state.credit_log.some((r) => r.task_id === task_id && r.reason === reason)) return { rows: [], rowCount: 0 };
+      if (/WHERE NOT EXISTS \(SELECT 1 FROM credit_log WHERE builder_id = \$1 AND reason = \$4\)/.test(s)
+        && state.credit_log.some((r) => r.builder_id === builder_id && r.reason === reason)) return { rows: [], rowCount: 0 };
+      state.credit_log.push({ builder_id, task_id, delta, reason, recorded_at: new Date().toISOString() });
+      return { rows: [], rowCount: 1 };
+    }
+    return { rows: [] };
+  });
+  return { pool, state };
+}
+
+const ship = (pool) => db.shipTask({ taskId: 7001, builderId: RECONCILER, landAssurance: LAND_PROOF }, { pool });
+const sent = (pool) => pool._client.queries.map((q) => String(q.sql).replace(/\s+/g, ' ').trim());
+const perTaskRows = (state) => state.credit_log.filter((r) => r.reason === 'task.shipped' || r.reason.startsWith('tweak.page_approved:'));
+
+// =========================================================================
+await test('a shipped page tweak pays its artist 30, on tweak.page_approved:<id>, and the shipper no estimate', async () => {
+  const { pool, state } = ledger({ claims: [web(ARTIST, 1), cli(APPLIER, 2)] });
+  const out = await ship(pool);
+  assert.equal(out.status, 'shipped');
+  assert.equal(out.creditsAwarded, 0, 'the shipper is paid no per-task estimate on a page tweak');
+  assert.equal(String(out.creditedBuilderId), APPLIER, 'the applier is still the ship-credited builder (achievements, session reward)');
+  assert.deepEqual(state.credit_log.map((r) => [String(r.builder_id), r.task_id, r.delta, r.reason]),
+    [[ARTIST, 7001, 30, 'tweak.page_approved:7001']], 'exactly one row: the artist, the priced value, the tally reason');
+  assert.equal(out.artistCredit.reason, 'paid');
+  assert.equal(out.artistCredit.credited, 30);
+  assert.ok(!sent(pool).some((q) => /'task\.shipped'/.test(q)), 'the estimate path is never entered for a page tweak');
+});
+
+await test('self-applied: the artist also ran /tweak, and is paid one artist row, nothing doubled', async () => {
+  const { pool, state } = ledger({ claims: [web(ARTIST, 1), cli(ARTIST, 2)] });
+  const out = await ship(pool);
+  assert.equal(String(out.creditedBuilderId), ARTIST);
+  assert.equal(state.credit_log.length, 1, 'one row, not an artist row plus an estimate row');
+  assert.equal(String(state.credit_log[0].builder_id), ARTIST);
+  assert.equal(state.credit_log[0].delta, 30);
+});
+
+await test('rounding: the priced value is credits_reward x the kind multiplier, rounded once', async () => {
+  for (const [reward, kind, want] of [[30, 'feature', 30], [25, 'bug', 30], [7, 'refactor', 6], [5, 'spike', 6], [3, 'cleanup', 2], [11, 'mystery', 11]]) {
+    const { pool, state } = ledger({ creditsReward: reward, kind, claims: [web(ARTIST, 1), cli(APPLIER, 2)] });
+    await ship(pool);
+    assert.equal(state.credit_log.length, 1, `${reward} x ${kind}: one row`);
+    assert.equal(state.credit_log[0].delta, want, `${reward} x ${kind} pays ${want}`);
+    assert.equal(state.credit_log[0].delta, credits.applyKindMultiplier(reward, kind).effective, 'the same rule as the estimate');
+  }
+});
+
+await test('a zero-reward round books nothing (no zero-value row that reads like a payment)', async () => {
+  const { pool, state } = ledger({ creditsReward: 0, claims: [web(ARTIST, 1), cli(APPLIER, 2)] });
+  const out = await ship(pool);
+  assert.equal(state.credit_log.length, 0);
+  assert.equal(out.artistCredit.reason, 'zero_value');
+});
+
+await test('the per-task stream total is conserved: a page tweak pays the artist what the ordinary ship paid the shipper', async () => {
+  for (const [reward, kind] of [[30, 'feature'], [25, 'bug'], [7, 'refactor']]) {
+    const ordinary = ledger({ source: null, creditsReward: reward, kind, claims: [cli(APPLIER, 2)] });
+    await ship(ordinary.pool);
+    const tweak = ledger({ creditsReward: reward, kind, claims: [web(ARTIST, 1), cli(APPLIER, 2)] });
+    await ship(tweak.pool);
+    const sum = (st) => perTaskRows(st).reduce((a, r) => a + r.delta, 0);
+    const ordinaryRows = perTaskRows(ordinary.state);
+    // An instance in cost-plus-only books no ordinary estimate at all; the
+    // equality then holds against what the estimate WOULD have been.
+    const estimate = ordinaryRows.length ? sum(ordinary.state) : credits.applyKindMultiplier(reward, kind).effective;
+    assert.equal(sum(tweak.state), estimate, `${reward} x ${kind}: the same total, moved to the artist`);
+    assert.ok(!perTaskRows(tweak.state).some((r) => String(r.builder_id) === APPLIER), 'the applier gets no per-task row');
+    assert.ok(!sent(ordinary.pool).some((q) => /^SELECT t\.source, t\.status/.test(q)), 'an ordinary ship never asks the artist lane');
+  }
+});
+
+await test('idempotent: a re-ship or a reconciler re-sweep books nothing a second time', async () => {
+  const { pool, state } = ledger({ claims: [web(ARTIST, 1), cli(APPLIER, 2)] });
+  await ship(pool);
+  // The shipped flip is terminal, so a second shipTask throws before paying.
+  await assert.rejects(() => ship(pool), (e) => e.code === 'TASK_NOT_CONFIRMED');
+  // And the writer itself, asked again inside some later transaction, refuses.
+  const again = await credits.awardArtistPageCredit(pool._client, { taskId: 7001, config: credits.rewardPolicy({}) });
+  assert.equal(again.reason, 'already_paid');
+  assert.equal(again.credited, 0);
+  assert.equal(state.credit_log.length, 1, 'still one row');
+});
+
+await test('one payment per ROUND: a payee that reads differently on a re-sweep is still refused', async () => {
+  const { pool, state } = ledger({ claims: [web(ARTIST, 1), cli(APPLIER, 2)] });
+  await ship(pool);
+  state.claims.push({ id: 9100, released_at: 'later', ...web(OTHER_ARTIST, 3) });
+  const again = await credits.awardArtistPageCredit(pool._client, { taskId: 7001, config: credits.rewardPolicy({}) });
+  assert.equal(again.reason, 'already_paid', 'keyed on (task, reason), not (builder, reason)');
+  assert.equal(state.credit_log.length, 1);
+});
+
+await test('a sent-back round is never paid: it sits at ready, its ship is refused, and the writer refuses too', async () => {
+  const { pool, state } = ledger({ status: 'ready', claims: [web(ARTIST, 1), cli(APPLIER, 2)] });
+  await assert.rejects(() => ship(pool), (e) => e.code === 'TASK_NOT_CONFIRMED');
+  const direct = await credits.awardArtistPageCredit(pool._client, { taskId: 7001, config: credits.rewardPolicy({}) });
+  assert.equal(direct.reason, 'not_shipped', 'a round must read shipped in the transaction to be paid');
+  for (const s of ['completed', 'confirmed', 'active', 'abandoned']) {
+    state.task.status = s;
+    const r = await credits.awardArtistPageCredit(pool._client, { taskId: 7001, config: credits.rewardPolicy({}) });
+    assert.equal(r.reason, 'not_shipped', `${s} is not paid`);
+  }
+  assert.equal(state.credit_log.length, 0);
+});
+
+await test('sent back, re-applied by someone else, approved and landed: the artist is paid once, neither applier', async () => {
+  // web (artist writes) -> CLI (applier 1, sent back, released) -> CLI (applier 2)
+  const { pool, state } = ledger({ claims: [web(ARTIST, 1), cli(APPLIER, 2), cli(APPLIER_2, 3)] });
+  const out = await ship(pool);
+  assert.equal(String(out.creditedBuilderId), APPLIER_2, 'the landing applier is the ship-credited builder');
+  assert.deepEqual(state.credit_log.map((r) => [String(r.builder_id), r.delta]), [[ARTIST, 30]]);
+});
+
+await test('the payee is the LATEST web claim: an artist who resumed the draft after a send-back is the one paid', async () => {
+  const { pool, state } = ledger({ claims: [web(ARTIST, 1), cli(APPLIER, 2), web(OTHER_ARTIST, 3), cli(APPLIER, 4)] });
+  await ship(pool);
+  assert.deepEqual(state.credit_log.map((r) => String(r.builder_id)), [OTHER_ARTIST],
+    'the TW05 reads name the same builder as the round\'s artist');
+});
+
+await test('a page tweak with no web claim pays nobody rather than guessing', async () => {
+  const { pool, state } = ledger({ claims: [cli(APPLIER, 2)] });
+  const out = await ship(pool);
+  assert.equal(out.artistCredit.reason, 'no_artist');
+  assert.equal(state.credit_log.length, 0, 'and the applier is not paid the estimate in its place');
+});
+
+await test('the writer reads the task itself: an ordinary task handed to it is refused', async () => {
+  const { pool, state } = ledger({ source: 'cascade', status: 'shipped', claims: [web(ARTIST, 1)] });
+  const r = await credits.awardArtistPageCredit(pool._client, { taskId: 7001, config: credits.rewardPolicy({}) });
+  assert.equal(r.reason, 'not_a_page_tweak');
+  const missing = await credits.awardArtistPageCredit(pool._client, { taskId: 1, config: credits.rewardPolicy({}) });
+  assert.equal(missing.reason, 'not_found');
+  assert.equal(state.credit_log.length, 0);
+});
+
+await test('the artist lane: cost-plus-only does not withhold it; only an explicit artistCredit:false does', async () => {
+  const p = credits.rewardPolicy({ mode: 'cost-plus-only' });
+  assert.equal(p.perTaskCredit, false);
+  assert.equal(p.artistCredit, true, 'a different craft\'s pay, not the builder estimate');
+  assert.equal(credits.rewardPolicy({}).artistCredit, true, 'absent -> on');
+  for (const v of [true, 1, 'yes', undefined, null]) assert.equal(credits.rewardPolicy({ artistCredit: v }).artistCredit, true);
+  assert.equal(credits.rewardPolicy({ artistCredit: false }).artistCredit, false, 'explicit opt-out');
+
+  const { pool, state } = ledger({ status: 'shipped', claims: [web(ARTIST, 1), cli(APPLIER, 2)] });
+  const paid = await credits.awardArtistPageCredit(pool._client, { taskId: 7001, config: p });
+  assert.equal(paid.credited, 30, 'paid under cost-plus-only');
+  const off = ledger({ status: 'shipped', claims: [web(ARTIST, 1)] });
+  const r = await credits.awardArtistPageCredit(off.pool._client, { taskId: 7001, config: credits.rewardPolicy({ artistCredit: false }) });
+  assert.equal(r.reason, 'artist_lane_off');
+  assert.equal(off.state.credit_log.length, 0);
+  assert.equal(state.credit_log.length, 1);
+});
+
+await test('the booked row is what the studio tally sums: same prefix, and it reads as 30 credits', async () => {
+  assert.equal(credits.ARTIST_PAGE_REASON_PREFIX, pageStatus.TALLY_REASON_PREFIX,
+    'economy writes and copy-desk reads the same prefix (restated in each, since a module cannot require another)');
+  const { pool, state } = ledger({ claims: [web(ARTIST, 1), cli(APPLIER, 2)] });
+  await ship(pool);
+  const tally = pageStatus.composeTally({ builderId: ARTIST, tasks: [], creditRows: state.credit_log, now: new Date() });
+  assert.equal(tally.credits, 30);
+  assert.equal(tally.this_week, 30);
+  const applierTally = pageStatus.composeTally({ builderId: APPLIER, tasks: [], creditRows: state.credit_log.filter((r) => String(r.builder_id) === APPLIER), now: new Date() });
+  assert.equal(applierTally.credits, 0);
+});
+
+await test('the reward port exposes the writer, delegated so a stub bites at call time', async () => {
+  const port = require('../modules/economy/reward.js');
+  assert.equal(typeof port.awardArtistPageCredit, 'function');
+  const real = credits.awardArtistPageCredit;
+  credits.awardArtistPageCredit = async () => ({ credited: 0, reason: 'stubbed' });
+  try {
+    assert.equal((await port.awardArtistPageCredit({}, { taskId: 1 })).reason, 'stubbed');
+  } finally {
+    credits.awardArtistPageCredit = real;
+  }
+});
+
+summary();