@bongos/core 1.21.4 → 1.21.6

package/tests/projects_hub_app_step.mjs

@@ -1,9 +1,11 @@
-// tests/projects_hub_app_step.mjs — the create wizard's "where your app runs" step,
-// EXECUTED (task 1004351, ADR 0345), and its Render setup (task 1004352, ADR 0348).
+// tests/projects_hub_app_step.mjs — the create wizard's "where your app runs" question,
+// EXECUTED (task 1004351, ADR 0345), and its Render setup (task 1004352, ADR 0348). Since
+// task 1004504 the old App step is the review's "App hosting" row: its controls moved into
+// the row whole, and "Use this" runs the check its Next used to.
 //
 // A static regex cannot tell a guard that works from one that is never consulted, so —
 // as tests/projects_hub_module_picker.mjs does for the picker — this slices the block out
-// of projects.html by its own sentinels and runs it in `vm` under a small DOM stub.
+// of projects.html by the code at its edges and runs it in `vm` under a small DOM stub.
 //
 // The load-bearing assertions for Render, because the key reaches the owner's WHOLE
 // Render account:
@@ -13,7 +15,7 @@
 //     must choose, and Next refuses until they do
 //   • a refused key says so at once, next to the field
 //   • a service building another repository cannot be linked
-//   • the review only ever names what the owner actually chose
+//   • the App hosting row only ever names what the owner actually chose
 //
 // Run:  node --test --test-reporter=tap tests/projects_hub_app_step.mjs
 
@@ -29,13 +31,20 @@ const require = createRequire(import.meta.url);
 const ROOT = path.join(path.dirname(fileURLToPath(import.meta.url)), '..');
 const HUB = fs.readFileSync(path.join(ROOT, 'modules', 'public-landing', 'public', 'projects.html'), 'utf8').replace(/\r\n/g, '\n');
 
-const START = '/* the app step (task 1004351)';
-const END = '    function renderReview() {';
-const from = HUB.indexOf(START);
-const to = HUB.indexOf(END, from);
-assert.ok(from > -1, 'the app-step block opens with its sentinel comment');
-assert.ok(to > from, 'the app-step block ends where renderReview begins');
-const BLOCK = HUB.slice(from, to);
+// Two regions since task 1004504: the row's change listener and the Render wiring (from
+// the #appOpts listener to the Look step), and the choice + the row's words (from
+// appPlatformChoice to the review's ROWS list). The row's value comes from rowText(), and
+// whether the row is fixed from appSkipped(); both come along, real.
+function slice(start, end, what) {
+  const a = HUB.indexOf(start), b = HUB.indexOf(end, a);
+  assert.ok(a > -1, what + ' opens where it should');
+  assert.ok(b > a, what + ' ends where it should');
+  return HUB.slice(a, b);
+}
+const BLOCK = slice("    var appOptsEl = root.querySelector('#appOpts');", '    /* ── the Look step (task 1004421', 'the app wiring');
+const CHOICE = slice('    function appPlatformChoice() {', '    var ROWS = [', 'the app choice and its row detail');
+const ROWTEXT = slice('    function rowText(key) {', '    /* paint every row from the draft', "the review rows' words");
+const SKIPPED = slice('    function appSkipped() {', '    /* What the create sends as `detail`.', 'appSkipped()');
 // The Render flow itself is shared with an existing project's page (task 1004184), so it
 // lives outside the wizard; its fence and the page's own node helper come along.
 const CSTART = '  /* ── connecting Render (task 1004352';
@@ -46,7 +55,7 @@ const CONNECT = HUB.slice(cfrom, cto);
 const APPEL = (HUB.match(/function appEl\(tag, cls, text\) \{[\s\S]*?\n  \}/) || [''])[0];
 assert.ok(APPEL, 'the page\'s own appEl() is lifted');
 
-const PANEL = (HUB.match(/<section class="panel" id="panel-app"[\s\S]*?<\/section>/) || [''])[0];
+const PANEL = (HUB.match(/<div class="setRow" data-row="app">[\s\S]*?<span class="acts">/) || [''])[0];
 const ESC = (HUB.match(/function esc\(s\) \{[\s\S]*?\n  \}/) || [''])[0];
 assert.ok(ESC, 'the page\'s own esc() is lifted, not re-implemented');
 
@@ -90,6 +99,7 @@ function load({ draft = {}, answers = {} } = {}) {
   const els = {};
   const $ = (id) => (els[id] = els[id] || el(id));
   let changeHandler = null;
+  let painted = 0;
   const saves = [];
   const calls = [];
   const root = {
@@ -115,14 +125,18 @@ function load({ draft = {}, answers = {} } = {}) {
   const ctx = { root, state, $, document, Object, API: '/api/bongos', fetch, JSON, String, Number, encodeURIComponent, Array, saveDraft: () => saves.push(JSON.stringify(state)),
     // the shared status watch (task 1004353) lives outside this block; record that it is started
     watchApp: (id, box) => { watches.push({ id, box }); return () => {}; },
-    // the planet-physics step skip (task 1004419) lives outside this block too; the
-    // framing that skips this step is wizard_physics_more.mjs's to test
-    physStepSkipped: () => false };
+    // the review repaints its rows on a change (task 1004504); count it
+    paintSetup: () => { painted += 1; },
+    // the planet-physics answers live outside these regions; rowText reads them through phys()
+    phys: () => { state.physics = state.physics || { answers: {} }; return state.physics; } };
   vm.createContext(ctx);
-  vm.runInContext(ESC + '\n' + APPEL + '\n' + CONNECT + '\n' + BLOCK + '\nthis.api = { appPlatformChoice, syncAppStep, appRow, syncRenderSetup, validateRender, renderRequestBody, sendRenderSetup, planLabel };', ctx);
+  vm.runInContext(ESC + '\n' + APPEL + '\n' + CONNECT + '\n' + CHOICE + '\n' + SKIPPED + '\n' + BLOCK + '\n' + ROWTEXT +
+    '\nthis.api = { appPlatformChoice, syncAppStep, appRowDetail, rowText, syncRenderSetup, validateRender, renderRequestBody, sendRenderSetup, planLabel };', ctx);
   const api = ctx.api;
   return {
-    ...api, inputs, state, saves, calls, $, watches,
+    ...api, inputs, state, saves, calls, $, watches, painted: () => painted,
+    // the row as the founder reads it: its value, then the line under it
+    appRow: () => { const t = api.rowText('app'); return t.v + ' | ' + t.s; },
     change: (value) => changeHandler({ target: inputs.find((i) => i.value === value) }),
     typeKey: (v) => { $('f-renderKey').value = v; $('f-renderKey').fire('input', { target: $('f-renderKey') }); },
     check: async () => { $('renderCheck').fire('click', { target: $('renderCheck') }); await flush(); },
@@ -139,32 +153,33 @@ const SERVICES = { services: [
 ] };
 const ANSWERS = { [LOOKUP]: TWO, [LOOKUP + '#' + TEA_A]: SERVICES };
 
-// ── the step itself (task 1004351) ──────────────────────────────────────────
+// ── the question itself (task 1004351), as the review's App hosting row (task 1004504) ──
 
-test('the panel is real markup: Render and Decide later, Decide later the default', () => {
-  assert.ok(PANEL, '#panel-app exists');
+test('the row is real markup: Render and Decide later, Decide later the default', () => {
+  assert.ok(PANEL, 'the review carries a data-row="app" row');
+  assert.match(PANEL, /<span class="k">App hosting<\/span>/, 'the row is named for the app alone');
   assert.match(PANEL, /<input type="radio" name="appPlatform" value="render" aria-describedby="appRenderWhat">/, 'Render can be picked (task 1004352)');
   assert.doesNotMatch(PANEL, /Coming soon/, 'and is no longer marked soon');
   assert.match(PANEL, /<input type="radio" name="appPlatform" value="later" checked>/, 'Decide later is the default');
-  assert.match(PANEL, /We host your Builders' hall/, 'the step says the hall is ours to host (ADR 0345 D1), in the page\'s one spelling of it');
-  assert.match(PANEL, /<form data-next="11">/, 'Next leads to the Look step (task 1004421), then the review');
-  assert.match(PANEL, /data-back="9"/, 'Back returns to the address step');
+  assert.ok(PANEL.indexOf('value="later"') < PANEL.indexOf('value="render"'), 'Decide later comes first');
+  assert.match(PANEL, /<button type="button" class="btn small rowUse" data-row-use>Use this<\/button>/, 'Use this confirms the row');
+  assert.doesNotMatch(HUB, /id="panel-app"/, 'the old App panel is gone');
 });
 
-test('the step sits between Address and the Look step in the panels map and on the rail', () => {
-  assert.match(HUB, /9: 'panel-address', 10: 'panel-app', 11: 'panel-look', 12: 'panel-review', 13: 'panel-done'/);
-  assert.match(HUB, /<li data-step="10"><span class="snum">8<\/span><span class="slbl">App<\/span><\/li>/);
-  assert.match(HUB, /<li data-step="11"><span class="snum">9<\/span><span class="slbl">Look<\/span><\/li>/);
-  assert.match(HUB, /<li data-step="12"><span class="snum">10<\/span><span class="slbl">Create<\/span><\/li>/);
-  assert.match(HUB, /data-back="11">← Back<\/button>\r?\n\s*<button type="button" class="btn big" id="doCreate">/, "the review's Back returns to the Look step");
-  assert.match(HUB, /if \(step === 10\) \{ syncAppStep\(\); syncRenderSetup\(\); \}\r?\n\s*if \(step === 11\) renderLook\(\);\r?\n\s*if \(step === 12\) renderReview\(\);/, 'entering the step repaints it, Render setup included');
-  assert.match(HUB, /if \(step === 10\) return validateRender\(\);/, 'Next on the step runs the Render check');
+test("the row sits in the review, between Address and Extras, and its checks are the old step's (task 1004504)", () => {
+  assert.match(HUB, /var panels = \{ 0: 'panel-gate', 1: 'panel-fork', 2: 'panel-what', 3: 'panel-checks', 4: 'panel-review', 5: 'panel-look', 6: 'panel-done' \};/);
+  assert.match(HUB, /var ROWS = \['code', 'address', 'app', 'extras', 'team', 'rewards', 'run'\];/);
+  assert.doesNotMatch(HUB, /<li[^>]*data-step="\d+"/, 'no rail beat names the step any more');
+  assert.match(HUB, /if \(step === 4\) \{[^}]*syncAppStep\(\); syncRenderSetup\(\);\r?\n\s*paintSetup\(\);\r?\n\s*\}/, 'entering the review repaints the row, Render setup included');
+  assert.match(HUB, /if \(key === 'app'\) \{ syncAppStep\(\); syncRenderSetup\(\); \}/, 'opening the row repaints it too');
+  assert.match(HUB, /if \(key === 'app'\) return rowFixed\('app'\) \|\| validateRender\(\);/, 'confirming the row runs the Render check');
 });
 
 test('a fresh draft reads "decide later", and the Render setup stays hidden', () => {
   const w = load();
   assert.equal(w.appPlatformChoice(), 'later');
-  assert.equal(w.appRow(), 'Not chosen yet — you can decide later');
+  // the row says the hall is ours to host (ADR 0345 D1), in the page's one spelling of it
+  assert.equal(w.appRow(), 'Decide later | We host your builders’ hall. There’s no app to host until your builders write one.');
   w.syncRenderSetup();
   assert.equal(w.$('renderSetup').hidden, true);
   assert.equal(w.validateRender(), true, 'decide later never blocks Next');
@@ -176,6 +191,7 @@ test('choosing Render saves the choice and opens its setup', () => {
   assert.equal(w.state.appPlatform, 'render');
   assert.equal(JSON.parse(w.saves[0]).appPlatform, 'render');
   assert.equal(w.$('renderSetup').hidden, false);
+  assert.equal(w.painted(), 1, 'and the review repaints its rows');
 });
 
 test('an unknown or missing value reads "decide later"', () => {
@@ -184,19 +200,25 @@ test('an unknown or missing value reads "decide later"', () => {
   }
 });
 
-test('the review carries the hall row and the app row in place of the old Shape row', () => {
-  const render = HUB.slice(HUB.indexOf('function renderReview()'));
-  const rows = render.slice(0, render.indexOf("$('reviewList').innerHTML"));
-  assert.ok(rows.includes("['Builders\\' hall', 'Hosted by us, on the platform’s current release'],"), 'the hall row');
-  assert.match(rows, /\['App', appRow\(\)\]/);
-  assert.doesNotMatch(rows, /\['Shape',/, 'the Shape row is gone');
+test('the review names the app alone: no hall row, no Shape row (task 1004504)', () => {
+  assert.doesNotMatch(HUB, /function renderReview\(\)|id="reviewList"/, 'the old ledger is gone');
+  assert.doesNotMatch(HUB, /data-row="(hall|shape)"/, 'the review has no hall or Shape row');
+  assert.doesNotMatch(HUB, /Hosted by us, on the platform’s current release/, 'the hall line left with the ledger');
+});
+
+test('a local or open-source answer makes the row fixed: it says so and asks for nothing', () => {
+  const w = load({ draft: { physics: { answers: { framing: 'local' } } } });
+  assert.equal(w.appRow(), 'Runs on your own machines | As your answer says. Choose a platform later from the project’s page.');
+  w.state.physics.answers.framing = 'open-source';
+  assert.match(w.appRow(), /^Runs on your own machines/);
+  w.state.demo = { people: 2, hours_each: 3 };
+  assert.match(w.appRow(), /^Decide later/, 'a demo is all managed, whatever framing a walk left behind');
 });
 
-test('the rail grid has one column per step, so adding a step never wraps the rail', () => {
+test('the rail grid left with the rail (task 1004504), so adding a step has no column count to keep in step', () => {
   const css = fs.readFileSync(path.join(ROOT, 'modules', 'public-landing', 'public', 'assets', 'cosmos.css'), 'utf8');
-  const cols = Number((css.match(/#view-new \.steps\{display:grid;grid-template-columns:repeat\((\d+),1fr\)/) || [])[1]);
-  const beats = [...HUB.matchAll(/<li[^>]*data-step="\d+"/g)].length;
-  assert.equal(cols, beats, 'the #view-new rail grid carries exactly one column per rail step');
+  assert.doesNotMatch(css, /#view-new \.steps/, 'no #view-new rail grid is left in cosmos.css');
+  assert.equal([...HUB.matchAll(/<li[^>]*data-step="\d+"/g)].length, 0, 'and no rail step is left on the page');
 });
 
 test('the hall is spelled one way on this page', () => {
@@ -377,7 +399,7 @@ test('Free is on the plan list with its catch spelled out, and Starter stays the
   const dbOpts = w.$('f-renderDbPlan').children.map((o) => o.value);
   assert.equal(dbOpts.includes('free'), false, 'the database stays paid-only');
   const r = load({ draft: { appPlatform: 'render', renderOwnerName: 'Mine', renderMode: 'greenfield', renderRuntime: 'node', renderPlan: 'free' } });
-  assert.equal(r.appRow(), 'Render, workspace Mine — a new app (Node.js, Free, $0, but it sleeps when idle and takes about a minute to wake up), no database. Render bills you directly.');
+  assert.equal(r.appRow(), 'Render | Workspace Mine — a new app (Node.js, Free, $0, but it sleeps when idle and takes about a minute to wake up), no database. Render bills you directly.');
 });
 
 test('what Create sends: greenfield carries the build and the plans, a link only the service', async () => {
@@ -430,7 +452,7 @@ test('a brand-new repo choosing Render asks for no key, says when it happens, an
   assert.equal(w.$('renderLater').hidden, false, 'the note shows');
   assert.equal(w.$('renderSetup').hidden, true, 'no key, workspace or plan questions');
   assert.equal(w.validateRender(), true, 'Next is not blocked on a key nobody needs yet');
-  assert.match(w.appRow(), /^Render — set up from your project’s page once your app has code\. Nothing is created on Render now\.$/);
+  assert.equal(w.appRow(), 'Render | Set up from your project’s page once your app has code. Nothing is created on Render now.');
   w.sendRenderSetup(42);
   await flush();
   assert.equal(w.calls.filter((c) => c.url.endsWith('/render')).length, 0, 'nothing is sent to Render');
@@ -484,13 +506,15 @@ test('a resumed draft asks for the key again, and says why', () => {
   assert.equal(w.validateRender(), false);
 });
 
-test('the review names exactly what was chosen, prices included', () => {
+test('the row names exactly what was chosen, prices included', () => {
   const g = load({ draft: { appPlatform: 'render', renderOwnerName: 'Mine', renderMode: 'greenfield', renderRuntime: 'node', renderPlan: 'starter', renderDb: true, renderDbPlan: 'basic_256mb' } });
-  assert.equal(g.appRow(), 'Render, workspace Mine — a new app (Node.js, Starter, about $7/month), with a database (Basic 256 MB, about $6/month). Render bills you directly.');
+  assert.equal(g.appRow(), 'Render | Workspace Mine — a new app (Node.js, Starter, about $7/month), with a database (Basic 256 MB, about $6/month). Render bills you directly.');
   const a = load({ draft: { appPlatform: 'render', renderOwnerName: 'Mine', renderMode: 'adopt', renderServiceName: '<shop>' } });
-  assert.equal(a.appRow(), 'Render, workspace Mine — link <code>&lt;shop&gt;</code>, changing nothing on it', 'a name from Render is escaped');
+  assert.equal(a.appRow(), 'Render | Workspace Mine — link <shop>, changing nothing on it.', 'a name from Render stays plain text');
+  // ...because the row is painted as TEXT, never parsed as markup
+  assert.match(HUB, /row\.querySelector\('\[data-row-v\]'\)\.textContent = t\.v;\r?\n\s*row\.querySelector\('\[data-row-s\]'\)\.textContent = t\.s;/);
 });
 
 test('Create hands the app to Render straight after the project is filed', () => {
-  assert.match(HUB, /show\(13, true\);\r?\n\s*sendRenderSetup\(data\.instance && data\.instance\.id\);/);
+  assert.match(HUB, /show\(6, true\);\r?\n\s*sendRenderSetup\(data\.instance && data\.instance\.id\);/);
 });

package/tests/projects_hub_dns_ready.mjs

@@ -46,8 +46,11 @@ for (const [name, region] of [['renderProgress', PROGRESS], ['addressReady', OVE
 
 function boot() {
   const liveProgress = { innerHTML: '' };
+  // the done panel's heading (task 1004504): "Setting up <name>" until the project answers
+  const doneH = { textContent: 'Setting up Lantern' };
   const sandbox = {
-    $: (id) => (id === 'liveProgress' ? liveProgress : null),
+    $: (id) => (id === 'liveProgress' ? liveProgress : id === 'done-h' ? doneH : null),
+    state: { name: 'Lantern', slug: 'lantern' },
     esc: (s) => String(s == null ? '' : s).replace(/[&<>"']/g, (c) => (
       { '&': '&amp;', '<': '&lt;', '>': '&gt;', '"': '&quot;', "'": '&#39;' }[c])),
     projectPageLink: (label) => '<a href="?view=manage&id=1">' + label + '</a>',
@@ -61,7 +64,7 @@ function boot() {
   };
   vm.createContext(sandbox);
   vm.runInContext(PROGRESS + '\n' + OVERLAY + '\n' + ADDRESS + '\n' + ORB, sandbox);
-  return { sandbox, progress: () => liveProgress.innerHTML };
+  return { sandbox, progress: () => liveProgress.innerHTML, heading: () => doneH.textContent };
 }
 
 const NOW = () => new Date().toISOString();
@@ -89,9 +92,14 @@ test('once the lookup answers, the done panel links the address', () => {
   const terminal = h.sandbox.renderProgress({ ...base, dns_resolves: true });
   assert.equal(terminal, false, 'resolving is not yet answering — the panel still waits for the probe');
   assert.match(h.progress(), ADDRESS_LINK);
+  assert.equal(h.heading(), 'Setting up Lantern', 'and the heading claims nothing yet (task 1004504)');
   const live = boot();
   assert.equal(live.sandbox.renderProgress({ ...base, dns_resolves: true, last_verified_up: true, last_verified_at: NOW() }), true);
   assert.match(live.progress(), ADDRESS_LINK);
+  assert.equal(live.heading(), 'Lantern is alive.', 'the alive moment, only on a verified answer');
+  const down = boot();
+  down.sandbox.renderProgress({ ...base, dns_resolves: true, last_verified_up: false, last_verified_at: NOW() });
+  assert.equal(down.heading(), 'Setting up Lantern', 'a down probe is never "alive"');
 });
 
 test('the card and the manage page: no link and no name before the lookup answers, in every pre-live state', () => {

package/tests/projects_hub_look.mjs

@@ -5,13 +5,13 @@
 //   • the previews paint the REAL looks: every colour lookKit carries equals its
 //     pack.json, and the night shares equal src/night-tint.js — a preview that drifted
 //     from the hall would promise a look the hall does not wear;
-//   • the step sits between App and Review (look 10, review 11, done 12, draft v7), on
-//     the rail as a beat of its own, and every kit walk past App passes it;
+//   • the screen sits after the review and holds Create (review 4, look 5, done 6,
+//     draft v9 since task 1004504; no rail), and every kit walk files only from it;
 //   • EXECUTED by the vm-slice idiom: the picker offers the four looks and "make your
 //     own", shows the accent only for your own, shows the ADJUSTED colour in the
 //     preview and says so, paints a black night when the tint is off, refuses an SVG
 //     or oversized logo before reading it, and sends the look then the logo;
-//   • Create sends the look right after the project is filed, and the review names it;
+//   • Create sends the look right after the project is filed, from the screen it was chosen on;
 //   • the manage page reads the look once per project, never on the minute tick.
 //
 // Run:  node --test tests/projects_hub_look.mjs
@@ -105,33 +105,39 @@ test('every colour the previews paint is its pack\'s, and the night shares are t
   assert.equal(kit.NIGHT.card, '#101012');
 });
 
-test('the step sits between App and Review: look 10, review 11, done 12, draft v7, a beat on the rail', () => {
-  assert.match(HUB, /10: 'panel-app', 11: 'panel-look', 12: 'panel-review', 13: 'panel-done' \};/);
-  assert.match(HUB, /var DRAFT_KEY = 'cb-create-draft-v8';/);
-  assert.match(HUB, /<li data-step="11"><span class="snum">9<\/span><span class="slbl">Look<\/span><\/li>\r?\n\s*<li data-step="12"><span class="snum">10<\/span><span class="slbl">Create<\/span><\/li>/);
+test('the screen sits after the review and holds Create: review 4, look 5, done 6, draft v9, no rail (task 1004504)', () => {
+  // task 1004504 blended the wizard into six screens; the look moved AFTER the review and
+  // became the screen Create stands on, so the look is the last thing chosen
+  assert.match(HUB, /4: 'panel-review', 5: 'panel-look', 6: 'panel-done' \};/);
+  assert.match(HUB, /var DRAFT_KEY = 'cb-create-draft-v9';/);
+  assert.doesNotMatch(HUB, /<li[^>]*data-step="\d+"/, 'no rail beat is left on the page');
+  assert.doesNotMatch(HUB, /id="panel-app"/, 'the App step is the review\'s App hosting row now');
   const panel = (HUB.match(/<section class="panel" id="panel-look"[\s\S]*?<\/section>/) || [''])[0];
-  assert.match(panel, /<h1 id="look-h">Give it a look<\/h1>/);
-  assert.match(panel, /<form data-next="12">/);
-  assert.match(panel, /data-back="10"/);
+  assert.match(panel, /<h1 id="look-h">Give it a look\.<\/h1>/);
+  assert.doesNotMatch(panel, /<form|data-next=|data-back=/, 'no form and no per-panel Back: the one door is Create, the one Back is the wizard\'s');
   assert.match(panel, /<div class="lookPick" id="wizLook"><\/div>/);
-  assert.ok(HUB.indexOf('id="panel-app"') < HUB.indexOf('id="panel-look"') && HUB.indexOf('id="panel-look"') < HUB.indexOf('id="panel-review"'), 'in document order too');
-  assert.match(HUB, /if \(step === 11\) renderLook\(\);\r?\n\s*if \(step === 12\) renderReview\(\);/);
-  assert.match(fs.readFileSync(path.join(LANDING, 'assets', 'cosmos.css'), 'utf8'), /#view-new \.steps\{display:grid;grid-template-columns:repeat\(10,1fr\);/, 'a track per numbered step');
+  assert.match(panel, /<p class="wizRecap" id="wizRecap"><\/p>/, 'the one-line recap of what Create makes');
+  assert.match(panel, /<button type="button" class="btn big" id="doCreate">Create my project<\/button>/, 'Create stands on this screen');
+  assert.ok(HUB.indexOf('id="panel-review"') < HUB.indexOf('id="panel-look"') && HUB.indexOf('id="panel-look"') < HUB.indexOf('id="panel-done"'), 'in document order too');
+  assert.match(HUB, /if \(step === 5\) \{ renderLook\(\); renderRecap\(\); \}/);
+  assert.doesNotMatch(fs.readFileSync(path.join(LANDING, 'assets', 'cosmos.css'), 'utf8'), /#view-new \.steps/, 'and no rail grid in cosmos.css');
 });
 
-test('every kit walk past the App step passes the Look step, and the kit has a state on it', () => {
+test('every kit walk that files a project passes the review and the Look screen first, and the kit has a state on it', () => {
   const states = JSON.parse(fs.readFileSync(path.join(LANDING, 'projects.states.json'), 'utf8')).states;
   const probes = JSON.parse(fs.readFileSync(path.join(LANDING, 'projects.probes.json'), 'utf8')).probes;
   const walks = [...Object.entries(states).map(([k, v]) => [k, v.actions]), ...probes.map((p) => [p.name, p.steps])];
   let past = 0;
   for (const [name, steps] of walks) {
-    const at = (steps || []).findIndex((a) => a[1] === '#panel-app button[type=submit]');
-    if (at < 0 || /^in-new-look/.test(name)) continue;   // the Look states stop ON the step
+    const at = (steps || []).findIndex((a) => a[1] === '#reviewGo');
+    if (at < 0) continue;
     past++;
-    const look = steps.findIndex((a) => a[1] === '#panel-look button[type=submit]');
-    assert.ok(look > at, `${name}: submits the Look step after the App step`);
+    // the retired App and Look submits must not linger in a walk (they would stop it dead)
+    assert.ok(!steps.some((a) => a[1] === '#panel-app button[type=submit]' || a[1] === '#panel-look button[type=submit]'), `${name}: walks no retired step`);
+    const create = steps.findIndex((a) => a[1] === '#doCreate');
+    if (create >= 0) assert.ok(create > at, `${name}: files only after the review's Continue lands it on the Look screen`);
   }
-  assert.ok(past >= 5, `sanity: the walks that reach the review were found (${past})`);
+  assert.ok(past >= 4, `sanity: the walks that leave the review were found (${past})`);
   assert.deepEqual(states['in-new-look'].expect.visible.slice(0, 2), ['#panel-look', '#wizLook']);
   assert.ok(states['in-new-look-custom'].actions.some((a) => a[1] === '#wizLook-custom'), 'a state pictures "make your own"');
 });
@@ -233,13 +239,17 @@ test('executed: sendLook sends the look, then the logo — and a removed logo is
   assert.deepEqual(calls.map((c) => [c.method, c.url.split('/').pop(), c.body]), [['PATCH', 'look', { look: 'grove', night_tint: 'on' }], ['DELETE', 'logo', null]]);
 });
 
-test('Create sends the look right after the project is filed, and the review names it', () => {
+test('Create sends the look right after the project is filed, and the look is chosen on the screen Create stands on', () => {
   assert.match(HUB, /sendRenderSetup\(data\.instance && data\.instance\.id\);[^\n]*\n\s*sendLookSetup\(data\.instance && data\.instance\.id\);/);
   assert.match(HUB, /lookKit\.sendLook\(instanceId, state\.look \|\| defaultLook\(\)\)/);
   assert.match(HUB, /function defaultLook\(\) \{ return \{ look: 'chrome-world', nightTint: true, accent: '#b48cf2', logo: null \}; \}/, 'the default is the platform\'s look with the tint ON (owner D2)');
-  assert.match(HUB, /\['Look', lookRow\(\)\]/);
+  // the ledger review and its Look row are gone (task 1004504): the picker and Create share
+  // one screen, so the look being filed is the one on screen beside the button
+  assert.doesNotMatch(HUB, /function lookRow\(|\['Look', lookRow\(\)\]/, 'no review row restates the look');
+  const panel = (HUB.match(/<section class="panel" id="panel-look"[\s\S]*?<\/section>/) || [''])[0];
+  assert.ok(panel.indexOf('id="wizLook"') > -1 && panel.indexOf('id="wizLook"') < panel.indexOf('id="doCreate"'), 'the picker sits above Create on the same screen');
   assert.match(HUB, /<p class="hint" id="doneLook" role="status" hidden><\/p>/, 'a refused look is said on the done panel');
-  assert.match(HUB, /look: null, termsAck: false \};/, 'the draft carries the look (and its reset clears it)');
+  assert.equal((HUB.match(/look: null, termsAck: false, confirmed: \{\} \};/g) || []).length, 2, 'the draft carries the look (and its reset clears it)');
 });
 
 test('the manage page has "Its look", read once per project — never on the minute tick', () => {