@bongos/core 1.24.82 → 1.24.84

package/modules/hall-ui/public/visor.js

@@ -1,19 +1,21 @@
 // visor.js — the first-person visor: glass over the scene (task 1005148, BONGOS-V3 R08,
 // goal 1000142, criterion `you-are-a-character-in-first-person`). Built to ADR 0399.
 //
-// What is on the glass (the design the owner agreed on 2026-11-10, task 1005148):
-//   - the frame: thin curved visor lines at the top, the bottom and both sides, a small
-//     centre cross, and a compass of nine slanted cells along the top line with the one
-//     salmon marker under its centre. Drawn once as SVG; it is ornament, never a control.
-//   - the place, in a mono line under the compass ("THE SKY", "GOVERNOR CITY", …).
-//   - bottom-left: your character's main action, the one accent pill, with its key.
-//   - bottom-right: your sprite (third person) with your character's name and your credits
+// What is on the glass (the minimal visor, Helm look 3, task 1005195; first built in task
+// 1005148):
+//   - the frame: two thin arcs, one along the top of the glass and one under the foot row,
+//     a small notch under the top arc and a ring-and-dot sight at the centre. Drawn as SVG;
+//     it is ornament, never a control.
+//   - the place, in the name voice under the top arc ("THE SKY", "GOVERNOR CITY", …).
+//   - the foot row. Left: your sprite (third person), your character's name and your credits
 //     (builders.total_credits — credits only, never a payout; the spec's "does not exist").
-//   - top-right: "Actions" (key A), which opens the sheet: your character's actions, then
-//     the keys of the place you are in, then Options on Escape.
+//     Centre: your character's main action, the one accent button, with its key. Right: the
+//     quiet key hints, no boxes: your other actions, "Actions" (key A), which opens the
+//     sheet (your character's actions, then the keys of the place you are in, then Options
+//     on Escape), and Options itself.
 //
 // The rules it keeps (ADR 0399):
-//   - one overlay, #visor, with pointer-events: none; only its pills take a pointer, so a
+//   - one overlay, #visor, with pointer-events: none; only its controls take a pointer, so a
 //     drag, a wheel or a click on the world reaches the scene unchanged (§2.1).
 //   - no camera: a scene registers itself once with OTBVisor.scene({ name, place, actions,
 //     keys }) and a movement action calls the scene's own function (§2.2). A page with no
@@ -34,7 +36,7 @@
 
   // ---------- each character's actions ----------
   // `href` opens a page the hall already has; `tool` opens one of craft-mode's tools on
-  // this page. The first is the character's main action (the accent pill).
+  // this page. The first is the character's main action (the accent button).
   var ACTIONS = {
     artisan: [
       { id: 'claim', label: 'Claim a task', key: 'c', href: '/work' },
@@ -112,23 +114,33 @@
   // ---------- drawing ----------
   var root = null, sheet = null, returnFocus = null;
   var NS = 'http://www.w3.org/2000/svg';
-  // The frame is drawn in a 1000×1000 box stretched over the stage (preserveAspectRatio
-  // none), with non-scaling strokes so every line stays one hairline at any size.
+  // Every mark is drawn twice: a wider stroke of the night under the ice hairline, so the
+  // line reads over a midday sky as well as at night (visor.css .visor__line--under).
+  function twice(shape) { return shape.replace('class="', 'class="visor__line--under ') + shape; }
+  function svgOpen(cls, box, stretch) {
+    return '<svg class="' + cls + '" viewBox="' + box + '"' + (stretch ? ' preserveAspectRatio="none"' : '') + ' aria-hidden="true" focusable="false" xmlns="' + NS + '">';
+  }
+  // The two arcs are each a 1000×100 box stretched across the stage (preserveAspectRatio
+  // none), with non-scaling strokes so each stays one hairline at any size. The notch and the
+  // sight are drawn at their own size, so neither is ever stretched.
   function frameSvg() {
-    var cells = '';
-    for (var i = -4; i <= 4; i++) {
-      var x = 500 + i * 38, y = 30 + Math.pow(i / 4, 2) * 14, s = i * 2;
-      cells += '<path class="visor__cell" d="M' + (x - 15) + ' ' + (y + 4 - s / 2) + 'l30 ' + s + 'l-6 15l-30 ' + (-s) + 'z"/>';
-    }
-    return '<svg class="visor__frame" viewBox="0 0 1000 1000" preserveAspectRatio="none" aria-hidden="true" focusable="false" xmlns="' + NS + '">' +
-      '<path class="visor__line" d="M0 120Q500 -10 1000 120"/>' +
-      '<path class="visor__line" d="M0 880Q500 1010 1000 880"/>' +
-      '<path class="visor__line visor__line--side" d="M18 120Q4 500 18 880M982 120Q996 500 982 880"/>' +
-      '<path class="visor__line" d="M490 500h-12M510 500h12M500 490v-14M500 510v14"/>' +
-      cells + '<path class="visor__mark" d="M494 76l6 9l6 -9z"/>' +
-    '</svg>';
+    return svgOpen('visor__frame visor__frame--top', '0 0 1000 100', true) + twice('<path class="visor__line" d="M76 100Q500 -100 924 100"/>') + '</svg>' +
+      svgOpen('visor__frame visor__frame--foot', '0 0 1000 100', true) + twice('<path class="visor__line" d="M76 0Q500 200 924 0"/>') + '</svg>' +
+      svgOpen('visor__notch', '0 0 12 9') + twice('<path class="visor__line" d="M0 0l6 9l6 -9"/>') + '</svg>' +
+      svgOpen('visor__sight', '0 0 24 24') + twice('<circle class="visor__line" cx="12" cy="12" r="11"/>') + '<circle class="visor__dot" cx="12" cy="12" r="2"/></svg>';
   }
   function keyHtml(k) { k = String(k); return '<kbd class="visor__key">' + esc(k.length === 1 ? k.toUpperCase() : k) + '</kbd>'; }
+  // The count and its unit are two spans, so the number can be white and the word ice; the
+  // text between them is still creditsText's, word for word.
+  function creditsHtml(n) {
+    var t = creditsText(n), at = t.lastIndexOf(' ');
+    if (at < 0) return esc(t);
+    return '<span class="visor__num">' + esc(t.slice(0, at)) + '</span> <span class="visor__unit">' + esc(t.slice(at + 1)) + '</span>';
+  }
+  // A quiet key hint: the key in its small box, then the name. No box round the pair.
+  function hintHtml(a) {
+    return '<a class="visor__hint" data-visor-act="' + esc(a.id) + '" ' + actionAttrs(a) + '>' + keyHtml(a.key) + '<span>' + esc(a.label) + '</span></a>';
+  }
   function actionAttrs(a) {
     return a.href ? 'href="' + esc(hallHref(a.href)) + '"' : 'href="#" role="button"';
   }
@@ -138,14 +150,19 @@
     var n = credits();
     return frameSvg() +
       (scene.place ? '<p class="visor__place">' + esc(scene.place) + '</p>' : '') +
-      '<a class="visor__pill visor__pill--main" data-visor-act="' + esc(main.id) + '" ' + actionAttrs(main) + '>' +
-        '<span>' + esc(main.label) + '</span>' + keyHtml(main.key) + '</a>' +
-      '<button class="visor__pill visor__pill--actions" type="button" data-visor-sheet aria-expanded="false" aria-controls="visor-sheet">' +
-        '<span class="visor__dots" aria-hidden="true"></span><span>Actions</span>' + keyHtml(SHEET_KEY) + '</button>' +
-      '<div class="visor__me">' +
-        (sprite ? '<img class="visor__sprite" src="' + esc(sprite) + '" alt="" width="250" height="290" decoding="async">' : '') +
-        '<span class="visor__who"><span class="visor__name">' + esc(NAMES[id]) + '</span>' +
-        '<span class="visor__credits" data-visor-credits>' + esc(creditsText(n)) + '</span></span>' +
+      '<div class="visor__foot">' +
+        '<div class="visor__me">' +
+          (sprite ? '<img class="visor__sprite" src="' + esc(sprite) + '" alt="" width="250" height="290" decoding="async">' : '') +
+          '<span class="visor__who"><span class="visor__name">' + esc(NAMES[id]) + '</span>' +
+          '<span class="visor__credits" data-visor-credits>' + creditsHtml(n) + '</span></span>' +
+        '</div>' +
+        '<a class="visor__main" data-visor-act="' + esc(main.id) + '" ' + actionAttrs(main) + '>' +
+          '<span>' + esc(main.label) + '</span>' + keyHtml(main.key) + '</a>' +
+        '<div class="visor__hints">' + acts.slice(1).map(hintHtml).join('') +
+          '<button class="visor__hint visor__hint--sheet" type="button" data-visor-sheet aria-expanded="false" aria-controls="visor-sheet">' +
+            '<span class="visor__dots" aria-hidden="true"></span>' + keyHtml(SHEET_KEY) + '<span>Actions</span></button>' +
+          '<a class="visor__hint visor__hint--options" href="' + esc(hallHref('/settings')) + '">' + keyHtml('Esc') + '<span>Options</span></a>' +
+        '</div>' +
       '</div>';
   }
   function sheetHtml(id) {

package/modules/hall-ui/public/watch.html

@@ -11,7 +11,7 @@
   <link rel="icon" href="/modules/ui-design/public/brand/favicon.svg" type="image/svg+xml">
   <link rel="preconnect" href="https://fonts.googleapis.com">
   <link rel="preconnect" href="https://fonts.gstatic.com" crossorigin>
-  <link rel="stylesheet" href="/builders/style.css?v=2026-11-11-view2">
+  <link rel="stylesheet" href="/builders/style.css?v=2026-11-11-helm3">
   <!-- The oversight family's sheet (task 1003469): the figure band, the group
        head, the row, the form. Then the page's own: severities, the report
        reader, the grade trend. The attention tiles + tabs come from style.css's

package/modules/hall-ui/public/work.html

@@ -8,7 +8,7 @@
   <link rel="icon" href="/modules/ui-design/public/brand/favicon.svg" type="image/svg+xml">
   <link rel="preconnect" href="https://fonts.googleapis.com">
   <link rel="preconnect" href="https://fonts.gstatic.com" crossorigin>
-  <link rel="stylesheet" href="/builders/style.css?v=2026-11-11-view2">
+  <link rel="stylesheet" href="/builders/style.css?v=2026-11-11-helm3">
   <!-- Page-local styles (task 2170) — extracted from the old inline <style>
        block per the hall placement rules; shared pager/count-pill styles now
        live in style.css's hall-kit section. -->

package/modules/hall-ui/records/craft-mode.md

@@ -38,6 +38,15 @@ Covers: every shelled hall page (`craft-mode.js`, `shell.js`, `style.css`), and
 - **Always-present tabs widened** (owner, 2026-10-03): `ALWAYS_TABS` is Builders, Collab and Profile, so every mode keeps the You and Community groups.
 - **Governing's header tallies** (task 1004563): `applyTallies()` draws "needs you N · moving M" right after the eyebrow, Governing only, hidden at <=960px (below that they squeezed the jump button under its own label, measured at 800px). Needs you is `GET /blockers?limit=1`'s `page.total`; moving is `GET /moving` (lifecycle, tasks `active` + `confirmed`, owner 2026-10-08), a count in SQL so the page never pulls task rows. Read once per page; a figure that cannot be read is LEFT OUT, never drawn as 0. The warm dot marks needs-you only above zero. It toggles the `hidden` ATTRIBUTE, never `.hidden =` (the nav gate owns that property; `tests/craft_mode_nav.mjs`). Pinned by `tests/craft_mode_tallies.mjs`.
 
+## The pickers in Helm (task 1005194, ADR 0402)
+
+- **Character creation is a screen, not a box**: the night itself (`.craft-picker--create` on `--bg`), the title centred in the name voice, three tall doors, and the one line at the foot. The line is the same `#craft-picker-lede` the dialog is described by; CSS `order` moves it under the doors, the markup and words are unchanged.
+- **A door is pressed, so it is cut** (`--cut-path`; the small cut on a phone, where the three stack as rows). The owner's portrait fills the top of the door under the cut; the name is the largest thing on it, in the name voice. The door the hall suggests (`.craft-door--pre`) is the one thing lit: the accent wash inside the accent edge. The door under the keyboard shows the inset ring and names its key ("Enter", a CSS label, as the start screen's Start does).
+- **The edge is drawn twice.** A door's hairline, the suggested edge and the focus ring are inset shadows, and a portrait that fills the door would cover them, so `.craft-door::after` repeats the door's `box-shadow` over its contents.
+- **The session picker** ("What are you doing now?") shares the rules: a square night panel with the strong hairline over the scrim, the title in the name voice, cut doors. Its four scenes (`.craft-art`) are untouched.
+- **Not taken from the mock:** the two outer doors' perspective tilt (pixel art resamples badly under a 3D transform), the mark above the title, and the "Lands in …" line (no such words exist; the line under each name is still `CHARACTER_LINES`).
+- **Tests:** `tests/hall_helm.mjs` ("character select is a Helm screen"); `tests/character_creation.mjs` still runs the behaviour.
+
 ## The paintbrush (task 1004497)
 
 - **Artist carries the palette and the paintbrush** (and the lightbulb, below): the palette (a feedback doc, `page-notes.js`, record `page-notes.md`) and the paintbrush (`paintbrush.js`). Both are `TOOLS` entries in `craft-mode.js` and load their own script on first press.

package/modules/hall-ui/records/options-menu.md

@@ -13,3 +13,14 @@ Every setting and configuration is reached from one vertical, grouped menu with
 - **Left the sidebar:** My agents, Project settings, Permissions, Modules (and their glyphs). The palette's agent rows open the menu; its Government rows are gone.
 - **Tests:** `tests/hall_options_menu.mjs` (the catalogue, gates, search, addresses, both redirect maps, the embedded page, game-mode Escape); `tests/hall_fullscreen.mjs` (Escape in game mode); `tests/market_watch_pages.mjs` (the settings panel in the frame).
 - **Known:** sounds are switched off (task 1004554), so the Sound entry shows its note, not a toggle.
+
+## The Helm look (task 1005194, ADR 0402)
+
+The menu is a Helm screen, written from the hall tier's tokens alone (`settings.css` types in no colour or corner for it). Words, order, addresses, keys and aria are unchanged.
+
+- **The menu** is one night panel (`--night-glass`) closed left and right by the strong ice hairline. The search is the hall's one field (`controls.css`), on the glass. An entry is a full-width row in the name voice on a soft hairline; the open entry is the accent bar under its dark ink; a group head is an ice label. Focus is the inset ring (`--press-focus`), so it reads on the accent bar too.
+- **The open screen** carries its name over the strong hairline: `options-menu.js` `nameScreen()` writes the open entry's own label into `#opt-screen` (a `<p aria-hidden="true">`; the rail's `aria-current` link and each frame's `aria-label` already say it). Settings' own sections are rows on soft hairlines, not cards, with their titles in the name voice. A frame sits under the name and still fills the column to the window's foot.
+- **Full screen and game mode:** the panel runs the window's full height from the top edge and names itself (`.opt-title`, the nav's own label, hidden while the top bar is up because the bar already says it). The pull to the top edge is the 60px the shell leaves a plain page above its content (`style.css`, `data-fullscreen-plain`); if that changes, change `.opt-rail`'s `margin-top` with it.
+- **Widths:** the panel is `min(--helm-panel-w, 32%)` of the content column; below 860px it stacks above the screen as before. A long label wraps inside its row.
+- **Not built** (the mock shows them on a Controls screen the hall does not have): the key-binding rows, "Reset all keys", and a visible "Back to the city" button. Escape still closes the menu in game mode.
+- **Tests:** `tests/hall_helm.mjs` ("the options menu is a Helm screen"); the kit states in `settings.states.json` render clean at 1440, 390 and 320.

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

@@ -2,9 +2,21 @@
 
 Covers: `visor.js` + `visor.css`, loaded on Home, `/city`, `/goal-map` and `/thinking`, and the three scene registrations (`sky-interaction.js`, `city.js`, `home-city.js`). Spec: `docs/specs/bongos-v3-game-conversion.md` C2 `you-are-a-character-in-first-person`. Game mode itself is in `game-mode.md`.
 
-## What it settled
+## The minimal visor (Helm look 3, task 1005195)
 
-- **The look the owner agreed (2026-11-10, five mock-up rounds).** A quiet layout with "hairline instruments" from the helmet round: thin curved visor lines across the top, bottom and both sides, a small centre cross, and a compass of nine slanted cells along the top line with the one salmon marker under its centre. No tracker or radar (tried and dropped), no clipped corners: the pills stay round (DESIGN.md "pill or nothing"). No glow (DESIGN.md's rejected near-black-plus-neon HUD). The frame is one SVG in a 1000×1000 box stretched over the stage with non-scaling strokes, so it is a hairline at every size.
+The glass was redrawn to the owner's mock-up "Cloud city: in the visor, minimal". Look and layout only: every action, key, id, aria attribute, copy string and data source in `visor.js` is as task 1005148 left it. Where this section and the older bullets below disagree about the look or where a part sits, this section is current.
+
+- **What is on the glass.** Two thin arcs (one along the top, one under the foot row), a notch under the top arc, a ring-and-dot sight at the centre, the place in the name voice under the top arc, and one foot row: the sprite in a cut box with the character's name and the credits (white count, ice word); the one accent button, cut corner, with its key; the quiet key hints (a key in a small box and a name, no box round the pair). The compass cells, the side lines, the centre cross and the round pills are gone.
+- **Where each control went.** The main action is the accent button, centred. The character's other actions (after the game-places filter) are key hints on the right; they were rows in the sheet only. **Actions (A)** is demoted from a top-right pill to a key hint and still opens the same sheet, which now stands over the hints. **Options (Esc)** is the last hint, the same link the sheet's last row is. Nothing was removed.
+- **In the mock-up, not built, because the visor has no data for it:** the four progress segments, the marker and name on the building in view (the visor owns no camera, ADR 0399 §2.2), and the party bars with a speaker's name and a Voice key. Each is a scene or a later task handing the visor real data; none is drawn empty.
+- **It reads over any sky, with no per-hour style.** The glass darkens toward its top and bottom edge (`--night-glass`), type wears `--type-halo` and a stroke of the night painted under each letter, every line is drawn twice (a wider night stroke under the ice hairline, `.visor__line--under`), and a key sits on a chip of the night. No new token.
+- **A phone.** The shared phone block in `style.css` (no breakpoint of the visor's own): the top arc, its notch and the place step aside (the top of a phone is the scene's and the shell's); the foot row stays one line with nothing leaning (the sprite, the count over its word, the accent button whose name may wrap and whose key is left to the sheet, and of the hints only Actions, as three dots on a night chip). Between a phone and a wide desk the hints wrap onto a second line, right-aligned, rather than run into the button.
+- **Scenes.** The top arc, the notch and the place sit inside the top 64px (`--visor-inset-top`), the band the sky already leaves free, so no scene's top row is struck through. Home no longer shifts its greeting row left (there is no pill top-right to clear). The city's door rises above the foot row at every width, because the accent button is centred where the door stood. The sky's `visor-on` offsets are unchanged.
+- **Looking at it.** The stand-in scene in `tests/hall_visor.mjs` over the hall's real sheets; put a bright gradient behind it to check midday.
+
+## What it settled (task 1005148)
+
+- **The first look (2026-11-10, five mock-up rounds; replaced by the minimal visor above).** A quiet layout with "hairline instruments" from the helmet round: thin curved visor lines across the top, bottom and both sides, a small centre cross, and a compass of nine slanted cells along the top line with the one salmon marker under its centre. No tracker or radar (tried and dropped), no clipped corners: the pills stay round (DESIGN.md "pill or nothing"). No glow (DESIGN.md's rejected near-black-plus-neon HUD). The frame is one SVG in a 1000×1000 box stretched over the stage with non-scaling strokes, so it is a hairline at every size.
 - **What sits where.** Bottom-left: the character's main action, the one accent pill, with its key. Bottom-right: the sprite (task 1005147's art, `OTBCraftMode.spriteSrc`) with the character's name and the credit count (`/me` `builder.total_credits` through `OTBCraftMode.me()`; credits only, never a payout). Top-right: Actions (key A), which opens the sheet: the character's actions, then "This place" (the scene's own actions and keys), then Options on Escape. A place line under the compass names the scene; the sky leaves it empty because its own heading names it.
 - **Each character's actions** (`ACTIONS`): Artisan Claim a task C (`/work`), Blockers B, Goal sky G, Your thinking T. Artist Review this page R, Edit its words E, Write an idea I (the three craft-mode tools, through `OTBCraftMode.openTool`), Studio S. Patron Board room O, Governor city V, Blockers B, Watch W. Every one is a page or tool the hall already has; the visor gates nothing, and the server's rank checks answer exactly as before (ADR 0399 §2.5). No key a scene binds is reused (the sky's L 0 / ?, the gallery's F, the palette's K, the city card's digits, arrows and Tab).
 - **Glass, not layout (ADR 0399 §2.1).** `#visor` is `position: fixed` with `pointer-events: none`; only the pills and the sheet take a pointer. A drag across a frame line pans the sky (test-pinned).

package/package-lock.json

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

package/package.json

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

@@ -11595,5 +11595,17 @@
       "id": "1005200",
       "text": "The hall now has two separate views, Game and Classic, and never mixes them. You choose one each time you sign in and can switch only in Options. Game mode stays full screen and only opens the game's places."
     }
+  ],
+  "1.24.83": [
+    {
+      "id": "1005194",
+      "text": "The options menu and the choose-your-character screen now wear the Helm look: a night panel with wide capital rows, and three tall cut-corner doors with the real characters. Fields and tags across the hall are squared to match"
+    }
+  ],
+  "1.24.84": [
+    {
+      "id": "1005195",
+      "text": "The in-game visor is now minimal: two thin arcs, the place name, your credits, one salmon button for the current action and a quiet row of key hints. It stays readable over a bright daytime sky as well as at night."
+    }
   ]
 }

package/src/module-api.js

@@ -81,7 +81,7 @@ const { sleep } = require('./sleep');
 // MAJOR (see allowBoxScope below): passes the request through untouched.
 function deprecatedNoopMiddleware(_req, _res, next) { next(); }
 
-const CORE_VERSION = '1.24.82'; // CI auto-patch carrier (ADR 0161); changelog: docs/module-api-changelog.md
+const CORE_VERSION = '1.24.84'; // CI auto-patch carrier (ADR 0161); changelog: docs/module-api-changelog.md
 
 // A namespaced logger so a module's log lines are attributable + consistent.
 // Usage: const log = api.logger('discord'); log.info('mounted');

package/tests/design_tokens_shared.mjs

@@ -133,13 +133,17 @@ test('the landing, hub, status and raise pages link it before their own sheet',
 
 test('the hall\'s old token names still resolve — each alias names a defined --ds-* token', () => {
   const style = decomment(read('modules', 'hall-ui', 'public', 'style.css'));
-  const want = ['--line', '--radius', '--radius-sm', '--radius-pill', '--sp-hair', '--sp-xs', '--sp-sm', '--sp-md', '--sp-lg', '--sp-xl',
+  const want = ['--line', '--radius', '--radius-sm', '--sp-hair', '--sp-xs', '--sp-sm', '--sp-md', '--sp-lg', '--sp-xl',
     '--shadow-pop', '--z-progress', '--z-topbar', '--z-sidebar', '--z-scrim', '--z-drawer', '--z-modal', '--z-diagram-full', '--label-size'];
   for (const n of want) {
     const m = style.match(new RegExp(`${n}\\s*:\\s*var\\((--ds-[\\w-]+)\\)`));
     assert.ok(m, `${n} is an alias onto the shared sheet`);
     assert.ok(ROOT_TOKENS[m[1]], `${n} → ${m[1]} is defined in tokens.css`);
   }
+  // --radius-pill left the aliases with Helm (task 1005194, ADR 0402): the hall draws no pill,
+  // so its tier sets the name to a square corner; the shared --ds-radius-pill is untouched.
+  assert.match(style, /--radius-pill: 0px;/, 'the hall has no pill');
+  assert.equal(ROOT_TOKENS['--ds-radius-pill'], '999px', 'the shared pill is still a pill for every other surface');
   // the four-step --space-* scale still rides the --sp-* aliases
   for (const n of ['--space-1', '--space-2', '--space-3', '--space-4']) assert.match(style, new RegExp(`${n}\\s*:\\s*var\\(--sp-`));
 });

package/tests/hall_controls.mjs

@@ -161,7 +161,7 @@ test('primary is the 46–48px door; the row size and the text button keep the 2
   }
   const field = sharedRule('select');
   assert.ok(px(prop(field, 'min-height')) >= 24, 'the field clears the floor');
-  assert.equal(prop(field, 'border-radius'), 'var(--radius-pill)', 'a field is a pill too');
+  assert.equal(prop(field, 'border-radius'), 'var(--radius-pill)', 'a field spends the pill name: a pill on the status page, square in the hall, whose tier sets --radius-pill to 0px (tests/hall_helm.mjs)');
 });
 
 // ── 3. AA, measured, in both modes ───────────────────────────────────────────

package/tests/hall_helm.mjs

@@ -14,7 +14,9 @@
 //     cannot see because the link is built at serve time;
 //   * no hall script writes a theme choice, and the shell offers no switch;
 //   * the Helm neutrals: the two dark twins carry the night, the ice and no black;
-//   * the kit's recipes exist once, as hall-tier tokens and shared rules.
+//   * the kit's recipes exist once, as hall-tier tokens and shared rules;
+//   * Helm look 2 (task 1005194): the hall has no pill, and the options menu and
+//     character select are Helm screens written from the tokens alone.
 //
 // Run:  node --test tests/hall_helm.mjs
 import { test } from 'node:test';
@@ -185,3 +187,103 @@ test('the kit\'s recipes are named once: the name voice, the cut, the segment, t
   assert.match(controls, /\n\.seg--done \{ background: var\(--ink-soft\);/, 'a finished step is information, so it is ice, not the accent');
   assert.match(controls, /\n\.seg--now \{ background: var\(--accent\);/, 'the accent is the step you can act on');
 });
+
+// ── Helm look 2 (task 1005194): the square corner, the options menu, character select ──
+
+// one rule's body, by its exact selector line, from a decommented sheet
+const ruleOf = (css, sel) => {
+  const at = css.indexOf('\n' + sel + ' {');
+  assert.notEqual(at, -1, `no rule ${sel}`);
+  const open = css.indexOf('{', at);
+  return css.slice(open + 1, css.indexOf('}', open));
+};
+const NO_COLOUR_LITERAL = /#[0-9a-fA-F]{3,8}\b|\brgba?\(|\bhsla?\(/;
+
+test('the hall has no pill: --radius-pill is a square corner in the hall tier, and only there', () => {
+  assert.equal(root['--radius-pill'], '0px', 'Helm draws no pill');
+  // the panel and the tile keep their corners until each page's own task (DESIGN.md "The hall")
+  assert.equal(root['--radius'], 'var(--ds-radius-panel)');
+  assert.equal(root['--radius-sm'], 'var(--ds-radius-tile)');
+  // the shared scale is untouched, so the landing and the status page keep their pill
+  assert.match(read('modules', 'ui-design', 'public', 'tokens.css'), /--ds-radius-pill:\s*999px;/);
+  const controls = decomment(read('modules', 'hall-ui', 'public', 'controls.css'));
+  // the field and the status tag spend the name, so they are square in the hall with no rule of their own
+  assert.match(controls, /\.input \{[^}]*border-radius: var\(--radius-pill\);/);
+  assert.match(controls, /\ntextarea \{[^}]*border-radius: var\(--press-radius, var\(--radius-sm\)\);/, 'a textarea is square too, the tile where the tier is not loaded');
+});
+
+test('what is pressed in the top bar, and a filter chip, take the small cut and the inset focus ring', () => {
+  const css = decomment(STYLE.replace(/\r\n/g, '\n'));
+  for (const sel of ['.upd-pill', '.chip', 'html[data-fullscreen="on"] .fullscreen-toggle']) {
+    const body = ruleOf(css, sel);
+    assert.match(body, /clip-path: var\(--cut-path-sm\);/, `${sel} is cut`);
+    assert.match(body, /--focus-ring: var\(--press-focus\);/, `${sel}: a cut clips an outline, so the ring is inside`);
+    assert.match(body, /border-radius: var\(--press-radius\);/, `${sel} has no round corner`);
+  }
+  // an outer shadow would be clipped away with the corner, and would outrank the focus ring
+  assert.doesNotMatch(ruleOf(css, 'html[data-fullscreen="on"] .upd-pill'), /box-shadow/);
+});
+
+test('the options menu is a Helm screen: a night panel, name-voice rows, the open one the accent bar', () => {
+  const sheet = decomment(read('modules', 'hall-ui', 'public', 'settings.css').replace(/\r\n/g, '\n'));
+  const menu = sheet.slice(0, sheet.indexOf('.cli-result'));
+  assert.ok(menu.includes('.opt-rail') && menu.includes('.opt-screen'), 'the menu and the open screen are the first part of the sheet');
+  assert.doesNotMatch(menu, NO_COLOUR_LITERAL, 'no colour is typed in');
+  assert.doesNotMatch(menu, /border-radius:\s*\d/, 'no corner is typed in');
+  const rail = ruleOf(menu, '.opt-rail');
+  assert.match(rail, /background: var\(--night-glass\);/);
+  assert.match(rail, /border-left-width: var\(--line-strong\); border-right-width: var\(--line-strong\);/, 'closed left and right by the strong hairline');
+  const link = ruleOf(menu, '.opt-link');
+  assert.match(link, /font-family: var\(--name-face\);/);
+  assert.match(link, /text-transform: var\(--name-case\);/);
+  assert.match(link, /min-height: var\(--ds-space-xl\);/, 'a row clears the 24px floor many times over');
+  assert.match(link, /border-bottom: var\(--line\) solid var\(--rule-soft\);/, 'rows on hairlines, not boxes');
+  assert.match(ruleOf(menu, '.opt-link--active, .opt-link--active:hover'), /color: var\(--accent-on\); background: var\(--accent\);/);
+  assert.match(ruleOf(menu, '.opt-link:focus-visible'), /box-shadow: var\(--press-focus\);/, 'the ring is inside the row, so it reads on the accent bar');
+  const screen = ruleOf(menu, '.opt-screen');
+  assert.match(screen, /font-family: var\(--name-face\);/);
+  assert.match(screen, /border-bottom: var\(--line-strong\) solid var\(--rule\);/);
+  const section = ruleOf(menu, '.settings-groups .scroll');
+  assert.match(section, /background: none;/, 'a section is a row on a hairline, not a card');
+  assert.match(section, /border-bottom: var\(--line\) solid var\(--rule-soft\);/);
+  // the search is the hall's one field: the sheet draws no edge, corner or focus of its own for it
+  assert.doesNotMatch(ruleOf(menu, '.opt-search input'), /border|outline/);
+  assert.doesNotMatch(menu, /\.opt-search input:focus/);
+  // the two names are words the menu already had: its own label, and the open entry's
+  const js = read('modules', 'hall-ui', 'public', 'options-menu.js');
+  assert.match(js, /<p class="opt-title" aria-hidden="true">Options<\/p>/);
+  assert.match(js, /h\.setAttribute\('aria-hidden', 'true'\);/);
+  assert.match(js, /h\.textContent = e\.label;/);
+  assert.match(js, /<nav|opt-search/, 'the search is still drawn');
+  assert.match(read('modules', 'hall-ui', 'public', 'settings.html'), /<nav class="opt-rail" id="settings-rail" aria-label="Options"><\/nav>/);
+});
+
+test('character select is a Helm screen: the night, cut doors, the suggested door lit, the ring inside the cut', () => {
+  const css = decomment(STYLE.replace(/\r\n/g, '\n'));
+  const picker = css.slice(css.indexOf('\n.craft-picker {'), css.indexOf('\n.craft-art {'));
+  assert.doesNotMatch(picker, NO_COLOUR_LITERAL, 'no colour is typed in');
+  assert.doesNotMatch(picker, /border-radius:\s*\d|font-size:\s*\d/, 'no corner or size is typed in');
+  assert.match(ruleOf(css, '.craft-picker--create'), /background: var\(--bg\);/, 'a screen on the night, not a box on a scrim');
+  assert.match(ruleOf(css, '.craft-picker__panel--create'), /background: none;/);
+  const title = ruleOf(css, '.craft-picker__title');
+  assert.match(title, /font-family: var\(--name-face\);/);
+  assert.match(title, /text-transform: var\(--name-case\);/);
+  const door = ruleOf(css, '.craft-door');
+  assert.match(door, /clip-path: var\(--cut-path\);/, 'a door is pressed, so it is cut');
+  assert.match(door, /background: var\(--night-glass\);/);
+  assert.match(ruleOf(css, '.craft-door:focus-visible'), /outline: 0;[^}]*box-shadow: var\(--press-focus\);/);
+  assert.match(ruleOf(css, '.craft-door--pre, .craft-door--pre:hover'), /box-shadow: inset 0 0 0 2px var\(--accent\);/);
+  assert.match(ruleOf(css, '.craft-door::after'), /box-shadow: inherit;/, 'the edge is drawn over the portrait as well as under it');
+  assert.match(ruleOf(css, '.craft-door__name'), /font-family: var\(--name-face\);/);
+  // the real sprites, and the words where they were
+  const js = read('modules', 'hall-ui', 'public', 'craft-mode.js');
+  for (const id of ['artisan', 'artist', 'patron']) assert.ok(fs.existsSync(path.join(HALL, 'characters', id + '.png')), `${id}.png`);
+  assert.match(js, /id="craft-picker-title">Choose your character<\/h2>/);
+  assert.match(js, /aria-describedby="craft-picker-lede"/);
+});
+
+test('the visor reads in the Helm night: no hover falls back to the pack\'s dark accent', () => {
+  const visor = decomment(read('modules', 'hall-ui', 'public', 'visor.css'));
+  assert.doesNotMatch(visor, /--accent-deep/, 'the deep accent is a light-ground colour; on the night it is under 4.5:1');
+  assert.match(visor, /\.visor__row:hover \{ color: var\(--accent-ink\); \}/);
+});

package/tests/hall_topbar_320.mjs

@@ -99,7 +99,9 @@ for (const scheme of ['light', 'dark']) {
   });
 }
 
-test('the top bar search and the mode switch are one shape: a pill with the same hairline', { skip }, async () => {
+// Helm (task 1005194): the hall has no pill, so the one shape the two share is the hall's
+// square corner (--radius-pill is 0px in the hall tier) inside the same hairline.
+test('the top bar search and the mode switch are one shape: the square corner of the hall with the same hairline', { skip }, async () => {
   const ctx = await context(1440, 'dark');
   const page = await ctx.newPage();
   await page.goto(ORIGIN + '/work');
@@ -115,11 +117,12 @@ test('the top bar search and the mode switch are one shape: a pill with the same
       switchRadius: css(sw).borderTopLeftRadius,
       jumpEdge: css(jump).borderTopColor,
       switchShadow: css(sw).boxShadow,
-      pill: root.getPropertyValue('--ds-radius-pill').trim(),
+      pill: root.getPropertyValue('--radius-pill').trim(),
     };
   }).finally(() => ctx.close());
-  assert.equal(s.jumpRadius, s.pill, 'the search is a full pill');
-  assert.equal(s.switchRadius, s.pill, 'the mode switch is a full pill');
+  assert.equal(s.pill, '0px', 'the hall tier squares the pill');
+  assert.equal(s.jumpRadius, s.pill, 'the search takes the corner of the hall');
+  assert.equal(s.switchRadius, s.pill, 'the mode switch takes the same corner');
   assert.ok(s.switchShadow.startsWith(s.jumpEdge), `the two share one hairline colour (${s.jumpEdge} vs ${s.switchShadow})`);
 });