@bongos/core 1.19.1059 → 1.19.1061

package/tests/copy_desk_page_next.mjs

@@ -0,0 +1,172 @@
+// tests/copy_desk_page_next.mjs — "recommend the next page" and "take next",
+// the WHOLE path (task 1004318 / BV2.TW07, ADR 0341 D5, D8, D11).
+//
+// The done-when: the recommendation follows the order on a fixture, skips held
+// pages, and "take next" claims the recommended page in one call.
+//
+// modules/copy-desk/tests/copy_page_next.mjs proves the order on hand-shaped
+// rows. This drives the path those rows travel: GET /copy-desk/next through
+// the real route, the real lifecycle read (listPageTweakTasks) and the real
+// page-ask summary SQL, then POST the recommendation's own `take` path through
+// the real page claim (webClaimPageTweak, createTask), all against one stateful
+// fake ledger (tests/fixtures/page-ledger.mjs). So a claim made by "take next"
+// is what the next read sees, with no hand-built rows in between.
+
+import { strict as assert } from 'node:assert';
+import { createRequire } from 'node:module';
+import fs from 'node:fs';
+import os from 'node:os';
+import path from 'node:path';
+import { makeRunner } from './helpers.mjs';
+import { installPageLedger } from './fixtures/page-ledger.mjs';
+
+process.env.NODE_ENV = 'test';
+const require = createRequire(import.meta.url);
+const F = require('../modules/copy-desk/tests/fixtures/page-tweak.cjs');
+const pages = require('../modules/copy-desk/pages.js');
+const api = require('../src/module-api.js');
+
+const { db } = installPageLedger(api, {
+  builders: [
+    { id: '7', github_login: 'alice', display_name: 'Alice', preferred_disciplines: ['artist'] },
+    { id: '8', github_login: 'bob', display_name: 'Bob', preferred_disciplines: ['artist'] },
+    { id: '9', github_login: 'carol', display_name: 'Carol', preferred_disciplines: ['engineer'] },
+  ],
+});
+const ALICE = '7';
+const BOB = '8';
+const CAROL = '9';
+
+api.requireBuilder = (req, res, next) => {
+  req.builder = { id: String(req.headers['x-builder'] || ALICE), rank: 'thetes' };
+  next();
+};
+
+const seams = require('../src/module-seams.js');
+if (!seams.hasProvider('lifecycle')) seams.registerProvider('lifecycle', require('../modules/lifecycle/lifecycle.js'));
+
+const pageData = require('../modules/copy-desk/page-data.js');
+const express = require('express');
+const { attachFail } = require('../src/bongos/middleware/error-envelope.js');
+const buildRouter = require('../modules/copy-desk/routes/copy-desk.js');
+
+const app = express();
+app.use(express.json());
+app.use(attachFail);
+app.use(buildRouter());
+const server = await new Promise((resolve) => { const s = app.listen(0, '127.0.0.1', () => resolve(s)); });
+const base = `http://127.0.0.1:${server.address().port}`;
+
+async function call(method, p, builder, body) {
+  const r = await fetch(base + p, {
+    method, headers: { 'content-type': 'application/json', 'x-builder': builder },
+    ...(body === undefined ? {} : { body: JSON.stringify(body) }),
+  });
+  return { status: r.status, body: await r.json() };
+}
+const getNext = (builder) => call('GET', '/copy-desk/next', builder);
+const errCode = (b) => (b && b.error && b.error.code) || null;
+const order = (b) => [b.recommended, ...b.alternatives].filter(Boolean).map((c) => c.page_id);
+
+const tmp = fs.mkdtempSync(path.join(os.tmpdir(), 'page-next-'));
+fs.mkdirSync(path.join(tmp, 'docs'), { recursive: true });
+fs.writeFileSync(path.join(tmp, 'docs/page-inventory.json'), JSON.stringify(F.INVENTORY));
+fs.writeFileSync(path.join(tmp, 'docs/page-readings.json'), JSON.stringify(F.READINGS));
+
+// The world before anyone asks: builders:studio was tweaked once by alice
+// (her web claim, released at submit), and has drifted since ("Welcome home"
+// became "Welcome back"). Every other page is untweaked.
+db.tasks.push({
+  id: '4001', status: 'shipped', title: 'Tweak: Studio (builders:studio)', source: 'page-tweak',
+  source_ref: pages.roundSourceRef('builders:studio', 1), touches: F.STUDIO_FILES,
+  shipped_at: '2026-09-20T10:00:00.000Z',
+  description: F.describe({
+    pageId: 'builders:studio', submittedAt: '2026-09-19T10:00:00Z',
+    lines: [{ key: 'L0002', section: 'greeting', before: 'Welcome', after: 'Welcome home' }],
+    appliedAt: '2026-09-19T18:00:00Z', linesAfter: ['Welcome home', 'Resume'],
+  }),
+});
+db.claims.push({ id: '8001', task_id: '4001', builder_id: ALICE, worktree_name: null, creator_session_id: null, claimed_at: '2026-09-18T10:00:00.000Z', released_at: '2026-09-19T10:00:00.000Z' });
+db.grades.set('4001', true);
+
+const { test, summary } = makeRunner();
+
+try {
+  pageData._setRoot(tmp);
+
+  await test('anyone asks; the recommendation reads asked, then changed, then untweaked', async () => {
+    assert.equal((await call('POST', '/copy-desk/pages/landing:projects/asks', CAROL, { reason: 'The project cards all read the same.' })).status, 201);
+    assert.equal((await call('POST', '/copy-desk/pages/landing:projects/asks', BOB, { reason: 'Agreed, the cards need a line each.' })).status, 201);
+    assert.equal((await call('POST', '/copy-desk/pages/status:explore/asks', CAROL, { reason: 'Explore does not say what it shows.' })).status, 201);
+
+    const { status, body } = await getNext(ALICE);
+    assert.equal(status, 200, JSON.stringify(body));
+    assert.deepEqual(order(body), ['landing:projects', 'status:explore', 'builders:studio', 'landing:index']);
+    assert.equal(body.recommended.why, '2 people asked for it');
+    assert.equal(body.recommended.why_line, 'landing · 2 people asked for it · untweaked');
+    assert.equal(body.alternatives[1].why, '1 line changed since your last pass', 'the round artist is the web claim, alice');
+    assert.equal(body.asked_pages, 2);
+    assert.equal(body.resume, null);
+    assert.deepEqual(body.skipped, []);
+    assert.equal(body.readings.ok, true);
+  });
+
+  await test('"take next" claims the recommended page in one call', async () => {
+    const { body: next } = await getNext(ALICE);
+    const before = db.tasks.length;
+    const take = await call(next.recommended.take.method, next.recommended.take.path, ALICE, {});
+    assert.equal(take.status, 201, JSON.stringify(take.body));
+    assert.equal(take.body.outcome, 'created');
+    assert.equal(take.body.page_id, next.recommended.page_id);
+    assert.equal(take.body.round.source_ref, 'page-tweak/landing:projects/r1');
+    assert.equal(db.tasks.length, before + 1);
+    const claim = db.claims.find((c) => c.task_id === take.body.round.task_id && c.released_at == null);
+    assert.deepEqual([claim.builder_id, claim.worktree_name, claim.creator_session_id], [ALICE, null, null], 'a web claim');
+  });
+
+  await test('the held page is Resume for its holder, and skipped (holder named) for everyone else', async () => {
+    const mine = (await getNext(ALICE)).body;
+    assert.equal(mine.resume.page_id, 'landing:projects');
+    assert.ok(!order(mine).includes('landing:projects'));
+    assert.equal(mine.recommended.page_id, 'status:explore');
+
+    const bobs = (await getNext(BOB)).body;
+    assert.equal(bobs.resume, null);
+    assert.ok(!order(bobs).includes('landing:projects'), 'never a page someone else holds, however many asked');
+    assert.deepEqual(bobs.skipped, [{ page_id: 'landing:projects', reason: 'page_held', state: 'writing', holder: { id: ALICE, login: 'alice' } }]);
+    assert.equal(bobs.recommended.page_id, 'status:explore');
+    assert.equal(bobs.alternatives[0].why, '1 line changed since the last pass', 'bob did not make that pass');
+  });
+
+  await test('take next for a second artist gets the next page, not the held one', async () => {
+    const { body: next } = await getNext(BOB);
+    const take = await call('POST', next.recommended.take.path, BOB, {});
+    assert.equal(take.status, 201, JSON.stringify(take.body));
+    assert.equal(take.body.page_id, 'status:explore');
+    const after = (await getNext(CAROL)).body;
+    assert.deepEqual(after.skipped.map((s) => s.page_id).sort(), ['landing:projects', 'status:explore']);
+    assert.deepEqual(order(after), ['builders:studio', 'landing:index']);
+  });
+
+  await test('a submitted round is in flight: skipped, and the claim agrees', async () => {
+    const round = db.tasks.find((t) => t.source_ref === 'page-tweak/status:explore/r1');
+    db.claims.find((c) => c.task_id === round.id && c.released_at == null).released_at = new Date().toISOString();
+    round.status = 'ready';
+    round.description += '\n\n' + pages.composeBatchBlock({ page_id: 'status:explore', reading_hash: 'h', submitted_at: 'x', lines: [{ key: 'L0001', section: 'hero', before: 'Explore', after: 'Look around', placement: 'placed', file: 'x', line: 1, string_id: 'a' }] });
+    const next = (await getNext(ALICE)).body;
+    assert.deepEqual(next.skipped, [{ page_id: 'status:explore', reason: 'page_in_flight', state: 'submitted' }]);
+    const refused = await call('POST', '/copy-desk/pages/status:explore/claim', ALICE, {});
+    assert.equal(errCode(refused.body), 'page_in_flight');
+  });
+
+  await test('no page inventory: a named 503, never a crash', async () => {
+    pageData._setRoot(fs.mkdtempSync(path.join(os.tmpdir(), 'page-next-empty-')));
+    const r = await getNext(ALICE);
+    assert.equal(r.status, 503);
+    assert.equal(errCode(r.body), 'page_inventory_missing');
+  });
+} finally {
+  server.close();
+  pageData._setRoot(null);
+}
+summary();

package/tests/copy_desk_page_submit.mjs

@@ -0,0 +1,289 @@
+// tests/copy_desk_page_submit.mjs — save a page draft, submit it, and split the
+// unplaced lines out, the WHOLE path (task 1004319 / BV2.TW08, ADR 0341 D4, D7).
+//
+// The done-when: draft save, submit and the unplaced split work end to end.
+//
+// Driven through the real copy-desk routes, the real lifecycle port (the page
+// claim, updateHeldTaskDescription, submitPageTweak, and the real createTask
+// under it) and the real page reads, against the shared stateful fake ledger
+// (tests/fixtures/page-ledger.mjs), whose page lock and task row lock are real
+// mutexes. The reading is a file on disk, read through the real page-data.js.
+
+import { strict as assert } from 'node:assert';
+import { createRequire } from 'node:module';
+import fs from 'node:fs';
+import os from 'node:os';
+import path from 'node:path';
+import { makeRunner } from './helpers.mjs';
+import { installPageLedger } from './fixtures/page-ledger.mjs';
+
+process.env.NODE_ENV = 'test';
+const require = createRequire(import.meta.url);
+const F = require('../modules/copy-desk/tests/fixtures/page-tweak.cjs');
+const pages = require('../modules/copy-desk/pages.js');
+const api = require('../src/module-api.js');
+
+const { db, seen } = installPageLedger(api, {
+  builders: [
+    { id: '7', github_login: 'alice', display_name: 'Alice', preferred_disciplines: ['artist'] },
+    { id: '8', github_login: 'bob', display_name: 'Bob', preferred_disciplines: ['artist'] },
+  ],
+});
+const ALICE = '7';
+const BOB = '8';
+
+api.requireBuilder = (req, res, next) => {
+  req.builder = { id: String(req.headers['x-builder'] || ALICE), rank: 'metic' };
+  next();
+};
+
+const seams = require('../src/module-seams.js');
+const lifecyclePort = require('../modules/lifecycle/lifecycle.js');
+if (!seams.hasProvider('lifecycle')) seams.registerProvider('lifecycle', lifecyclePort);
+
+const pageData = require('../modules/copy-desk/page-data.js');
+const express = require('express');
+const { attachFail } = require('../src/bongos/middleware/error-envelope.js');
+const buildRouter = require('../modules/copy-desk/routes/copy-desk.js');
+
+const app = express();
+app.use(express.json());
+app.use(attachFail);
+app.use(buildRouter());
+const server = await new Promise((resolve) => { const s = app.listen(0, '127.0.0.1', () => resolve(s)); });
+const base = `http://127.0.0.1:${server.address().port}`;
+
+async function call(method, p, builder, body = {}) {
+  const r = await fetch(base + p, { method, headers: { 'content-type': 'application/json', 'x-builder': builder }, body: method === 'GET' ? undefined : JSON.stringify(body) });
+  return { status: r.status, body: await r.json() };
+}
+const errCode = (b) => (b && b.error && b.error.code) || null;
+const errDetail = (b) => (b && b.error && b.error.details) || {};
+
+// The studio reading, in page-reader.js's shape: a shared line with its
+// registry row, a shared line without one, placed lines, one with {…} holes,
+// and an unplaced line.
+const L = (key, section, text, placement, row) => ({ key, section, text, placement, ...(row || {}) });
+const STUDIO_LINES = [
+  L('L0001', 'top bar', 'Studio', 'shared', { file: 'modules/hall-ui/public/shell.js', line: 40, string_id: 'sh0001' }),
+  L('L0002', 'top bar', 'Ctrl K', 'shared'),
+  L('L0003', 'greeting', 'Welcome back', 'placed', { file: 'modules/hall-ui/public/studio.html', line: 12, string_id: 'st0003' }),
+  L('L0004', 'greeting', '{…} pages tweaked', 'placed', { file: 'modules/hall-ui/public/studio.js', line: 88, string_id: 'st0004' }),
+  L('L0005', 'greeting', 'Your desk is quiet today', 'unplaced'),
+];
+const INDEX_LINES = [
+  L('L0001', 'hero', 'Build here', 'placed', { file: 'modules/public-landing/public/index.html', line: 3, string_id: 'ix0001' }),
+  L('L0002', 'hero', 'Start a project', 'placed', { file: 'modules/public-landing/public/index.html', line: 4, string_id: 'ix0002' }),
+];
+const readingsWith = (studio, index) => ({
+  schema: 1,
+  pages: [
+    { id: 'builders:studio', reading_hash: studio.hash, lines: studio.lines },
+    { id: 'landing:index', reading_hash: index.hash, lines: index.lines },
+  ],
+});
+
+const tmp = fs.mkdtempSync(path.join(os.tmpdir(), 'page-submit-'));
+fs.mkdirSync(path.join(tmp, 'docs'), { recursive: true });
+fs.writeFileSync(path.join(tmp, 'docs/page-inventory.json'), JSON.stringify(F.INVENTORY));
+function writeReadings(r) {
+  fs.writeFileSync(path.join(tmp, 'docs/page-readings.json'), JSON.stringify(r));
+  pageData._setRoot(tmp); // drop the (mtime, size) cache
+}
+
+const { test, summary } = makeRunner();
+const STUDIO = '/copy-desk/pages/builders:studio';
+const INDEX = '/copy-desk/pages/landing:index';
+const studioTask = () => db.tasks.find((t) => t.source_ref === 'page-tweak/builders:studio/r1');
+const DRAFT = {
+  reading_hash: 'studio-h1',
+  lines: [
+    { key: 'L0003', after: 'Welcome home' },
+    { key: 'L0004', after: '{…} pages rewritten so far' },
+    { key: 'L0005', after: 'Nothing is waiting for you' },
+    { key: 'L0002', after: 'Control K' },
+  ],
+};
+
+try {
+  writeReadings(readingsWith({ hash: 'studio-h1', lines: STUDIO_LINES }, { hash: 'index-h1', lines: INDEX_LINES }));
+
+  await test('no round yet: a draft save is no_open_round, and nothing is written', async () => {
+    const r = await call('PUT', `${STUDIO}/draft`, ALICE, DRAFT);
+    assert.equal(r.status, 409);
+    assert.equal(errCode(r.body), 'no_open_round');
+    assert.equal(db.tasks.length, 0);
+  });
+
+  await test('the holder saves a draft: it rests on the round task, resolved from the reading', async () => {
+    assert.equal((await call('POST', `${STUDIO}/claim`, ALICE)).status, 201);
+    const r = await call('PUT', `${STUDIO}/draft`, ALICE, DRAFT);
+    assert.equal(r.status, 200, JSON.stringify(r.body));
+    assert.equal(r.body.outcome, 'saved');
+    assert.deepEqual(r.body.counter, { changed: 4, total: 5 });
+    const parsed = pages.parseDraftBlock(studioTask().description);
+    assert.equal(parsed.ok, true);
+    assert.deepEqual(parsed.draft.lines.map((l) => [l.key, l.before, l.after, l.placement]), [
+      ['L0002', 'Ctrl K', 'Control K', 'shared'],
+      ['L0003', 'Welcome back', 'Welcome home', 'placed'],
+      ['L0004', '{…} pages tweaked', '{…} pages rewritten so far', 'placed'],
+      ['L0005', 'Your desk is quiet today', 'Nothing is waiting for you', 'unplaced'],
+    ]);
+    assert.match(studioTask().description, /^Page tweak round 1/, 'the round body stays; the block is added below it');
+    assert.equal(studioTask().status, 'active');
+  });
+
+  await test('the same autosave again writes nothing: unchanged, and the task is not touched', async () => {
+    const writes = db.writes;
+    const stamp = studioTask().updated_at;
+    const before = seen.length;
+    const r = await call('PUT', `${STUDIO}/draft`, ALICE, DRAFT);
+    assert.equal(r.status, 200);
+    assert.equal(r.body.outcome, 'unchanged');
+    assert.equal(db.writes, writes);
+    assert.equal(studioTask().updated_at, stamp);
+    assert.equal(seen.slice(before).some((q) => /^UPDATE tasks/.test(q)), false);
+    // The save reads this page's rounds only, never the whole instance's.
+    assert.equal(db.pageScopes[db.pageScopes.length - 1], 'page-tweak/builders:studio/');
+  });
+
+  await test('a whole-draft replace: a line left out of the next save leaves the draft', async () => {
+    const r = await call('PUT', `${STUDIO}/draft`, ALICE, { ...DRAFT, lines: DRAFT.lines.slice(0, 3) });
+    assert.equal(r.body.outcome, 'saved');
+    assert.deepEqual(pages.parseDraftBlock(studioTask().description).draft.lines.map((l) => l.key), ['L0003', 'L0004', 'L0005']);
+    const back = await call('PUT', `${STUDIO}/draft`, ALICE, DRAFT);
+    assert.equal(back.body.outcome, 'saved');
+    assert.equal((studioTask().description.match(/```page-tweak-draft/g) || []).length, 1, 'replaced in place, never a second block');
+  });
+
+  await test('another artist cannot save it: page_not_held, naming the holder', async () => {
+    const r = await call('PUT', `${STUDIO}/draft`, BOB, DRAFT);
+    assert.equal(r.status, 409);
+    assert.equal(errCode(r.body), 'page_not_held');
+    assert.deepEqual(errDetail(r.body).holder, { id: ALICE, login: 'alice', name: 'Alice' });
+  });
+
+  await test('a draft built on an older reading is refused reading_moved, and nothing is written', async () => {
+    const writes = db.writes;
+    const r = await call('PUT', `${STUDIO}/draft`, ALICE, { ...DRAFT, reading_hash: 'studio-h0' });
+    assert.equal(r.status, 409);
+    assert.equal(errCode(r.body), 'reading_moved');
+    assert.equal(errDetail(r.body).reading_hash, 'studio-h1');
+    assert.equal(db.writes, writes);
+  });
+
+  await test('the body is strict: a target field at the top level is refused', async () => {
+    const r = await call('PUT', `${STUDIO}/draft`, ALICE, { ...DRAFT, file: 'modules/hall-ui/public/studio.html' });
+    assert.equal(errCode(r.body), 'validation_failed');
+  });
+
+  await test('another artist cannot submit it', async () => {
+    const r = await call('POST', `${STUDIO}/submit`, BOB);
+    assert.equal(r.status, 409);
+    assert.equal(errCode(r.body), 'page_not_held');
+    assert.equal(pages.blockPresence(studioTask().description).batch, false);
+  });
+
+  await test('submit freezes the batch, files an engineer task per unplaced line, and releases the claim', async () => {
+    const r = await call('POST', `${STUDIO}/submit`, ALICE);
+    assert.equal(r.status, 200, JSON.stringify(r.body));
+    assert.equal(r.body.outcome, 'submitted');
+    assert.deepEqual(r.body.round, { task_id: studioTask().id, state: 'submitted', status: 'ready' });
+    assert.equal(r.body.batch.line_count, 2);
+    assert.deepEqual(r.body.next, { path: '/copy-desk/next' });
+
+    // The batch: placed lines with their registry rows, from the reading.
+    const b = pages.parseBatchBlock(studioTask().description);
+    assert.equal(b.ok, true);
+    assert.deepEqual(b.batch.lines.map((l) => [l.key, l.file, l.line, l.string_id, l.before, l.after]), [
+      ['L0003', 'modules/hall-ui/public/studio.html', 12, 'st0003', 'Welcome back', 'Welcome home'],
+      ['L0004', 'modules/hall-ui/public/studio.js', 88, 'st0004', '{…} pages tweaked', '{…} pages rewritten so far'],
+    ]);
+    assert.equal(b.batch.reading_hash, 'studio-h1');
+
+    // The unplaced split: the unplaced line, and the shared line with no row.
+    const eng = db.tasks.filter((t) => t.source === 'page-tweak-unplaced');
+    assert.deepEqual(eng.map((t) => t.source_ref).sort(), [
+      `page-tweak-unplaced/${studioTask().id}/L0002`,
+      `page-tweak-unplaced/${studioTask().id}/L0005`,
+    ]);
+    for (const t of eng) {
+      assert.deepEqual([t.status, t.discipline, t.kind, t.version_id, t.goal_id, t.created_by], ['backlog', 'engineer', 'feature', 'V9', '77', ALICE]);
+      assert.match(t.description, /builders:studio/);
+    }
+    const quiet = eng.find((t) => t.source_ref.endsWith('/L0005'));
+    assert.match(quiet.description, /"Your desk is quiet today"/);
+    assert.match(quiet.description, /"Nothing is waiting for you"/);
+    assert.deepEqual([...b.batch.unplaced_tasks].sort(), eng.map((t) => t.id).sort(), 'the batch names the engineer tasks');
+    assert.deepEqual(r.body.unplaced_tasks.map((u) => u.created), [true, true]);
+
+    // The claim is released and the round is ready: the /tweak queue.
+    assert.equal(studioTask().status, 'ready');
+    const claim = db.claims.find((c) => c.task_id === studioTask().id);
+    assert.ok(claim.released_at, 'the web claim is released');
+    assert.equal(claim.outcome, 'partial');
+    // The draft stays beside the batch (ADR 0341 D7: it may be amended later).
+    assert.equal(pages.parseDraftBlock(studioTask().description).ok, true);
+  });
+
+  await test('the page reads submitted, and nobody can claim it back into writing', async () => {
+    const page = await call('GET', STUDIO, BOB);
+    assert.equal(page.status, 200);
+    assert.equal(page.body.open_round.state, 'submitted');
+    const claim = await call('POST', `${STUDIO}/claim`, BOB);
+    assert.equal(errCode(claim.body), 'page_in_flight');
+  });
+
+  await test('a repeated submit is already_submitted: no second batch, no second engineer task', async () => {
+    const tasks = db.tasks.length;
+    const r = await call('POST', `${STUDIO}/submit`, ALICE);
+    assert.equal(r.status, 200);
+    assert.equal(r.body.outcome, 'already_submitted');
+    assert.equal(r.body.unplaced_tasks.length, 2);
+    assert.equal(db.tasks.length, tasks);
+    assert.equal((studioTask().description.match(/```page-tweak\n/g) || []).length, 1);
+  });
+
+  await test('after submit the draft is frozen: a save is page_in_flight', async () => {
+    const r = await call('PUT', `${STUDIO}/draft`, ALICE, DRAFT);
+    assert.equal(r.status, 409);
+    assert.equal(errCode(r.body), 'page_in_flight');
+  });
+
+  await test('a line whose reading moved is refused target_gone at submit; the round stays with its artist', async () => {
+    assert.equal((await call('POST', `${INDEX}/claim`, BOB)).status, 201);
+    const saved = await call('PUT', `${INDEX}/draft`, BOB, { reading_hash: 'index-h1', lines: [{ key: 'L0001', after: 'Build it here' }, { key: 'L0002', after: 'Start yours' }] });
+    assert.equal(saved.body.outcome, 'saved');
+    // A ship rewords the hero line: the page no longer shows "Build here".
+    writeReadings(readingsWith({ hash: 'studio-h1', lines: STUDIO_LINES }, {
+      hash: 'index-h2',
+      lines: [L('L0001', 'hero', 'Build anything here', 'placed', { file: 'x', line: 3, string_id: 'ix0009' }), INDEX_LINES[1]],
+    }));
+    const writes = db.writes;
+    const r = await call('POST', `${INDEX}/submit`, BOB);
+    assert.equal(r.status, 409);
+    assert.equal(errCode(r.body), 'lines_refused');
+    assert.deepEqual(errDetail(r.body).refused, [{ key: 'L0001', section: 'hero', before: 'Build here', code: 'target_gone' }]);
+    const t = db.tasks.find((x) => x.source_ref === 'page-tweak/landing:index/r1');
+    assert.equal(t.status, 'active');
+    assert.equal(pages.blockPresence(t.description).batch, false);
+    assert.equal(db.writes, writes);
+    assert.ok(db.claims.find((c) => c.task_id === t.id && c.released_at == null), 'the claim is still held');
+  });
+
+  await test('two submits at once: one freezes, the other is the same double-click, and one engineer task per line', async () => {
+    // Bob drops the gone line on the current reading, adds an unplaced-free
+    // rewrite, and double-submits.
+    const t = db.tasks.find((x) => x.source_ref === 'page-tweak/landing:index/r1');
+    const saved = await call('PUT', `${INDEX}/draft`, BOB, { reading_hash: 'index-h2', lines: [{ key: 'L0002', after: 'Start yours' }] });
+    assert.equal(saved.body.outcome, 'saved');
+    const [a, b] = await Promise.all([call('POST', `${INDEX}/submit`, BOB), call('POST', `${INDEX}/submit`, BOB)]);
+    assert.deepEqual([a.body.outcome, b.body.outcome].sort(), ['already_submitted', 'submitted'], JSON.stringify([a.body, b.body]));
+    assert.equal((t.description.match(/```page-tweak\n/g) || []).length, 1);
+    assert.equal(t.status, 'ready');
+  });
+} finally {
+  server.close();
+  pageData._setRoot(null);
+}
+summary();