@bongos/core 1.24.81 → 1.24.82

package/modules/hall-ui/public/tweak-editor.html

@@ -10,7 +10,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-helm">
+  <link rel="stylesheet" href="/builders/style.css?v=2026-11-11-view2">
   <link rel="stylesheet" href="/builders/tweak-editor.css?v=2026-09-29-tw27-aqfix">
 </head>
 <!-- THE TWEAK EDITOR (task 1004321 / BV2.TW10, goal 1000095; ADR 0341 D4 and
@@ -143,10 +143,10 @@
 <div id="toast" class="toast" role="status" aria-live="polite" data-visible="0"></div>
 
 <script src="/builders/dom-utils.js?v=2026-11-05-numbers"></script>
-<script src="/builders/shell-game.js?v=2026-11-11-helm"></script>
+<script src="/builders/shell-game.js?v=2026-11-11-view2"></script>
 <script src="/builders/shell-trail.js?v=2026-11-05-numbers"></script>
 <script src="/builders/craft-mode.js?v=2026-11-10-visor"></script>
-<script src="/builders/shell.js?v=2026-11-11-helm"></script>
+<script src="/builders/shell.js?v=2026-11-11-view2"></script>
 <script src="/builders/hall-dialog.js"></script>
 <script src="/builders/palette.js" defer></script>
 <script src="/builders/brand-holes.js?v=2026-09-29-aqfix"></script>

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

@@ -69,7 +69,13 @@
     var mode = typeof C.current === 'function' ? C.current() : null;
     return ACTIONS[mode] ? mode : null;
   }
-  function actionsFor(id) { return ACTIONS[id] ? ACTIONS[id].slice() : []; }
+  // Game mode reaches only the game's places (task 1005200): an action that opens a classic
+  // page is off the glass until that page has a game version. A tool is always this page's.
+  function inGame(a) {
+    var G = window.OTBShellGame;
+    return !a.href || !G || typeof G.isPlace !== 'function' || G.isPlace(a.href);
+  }
+  function actionsFor(id) { return ACTIONS[id] ? ACTIONS[id].filter(inGame) : []; }
   function credits() {
     var C = craft();
     var me = C && typeof C.me === 'function' ? C.me() : null;

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-helm">
+  <link rel="stylesheet" href="/builders/style.css?v=2026-11-11-view2">
   <!-- 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
@@ -269,10 +269,10 @@
 <!-- The app shell (sidebar + top bar). Dependency-free; loaded FIRST so the
      chrome exists before the page script runs. -->
 <script src="/builders/dom-utils.js?v=2026-11-05-numbers"></script>
-<script src="/builders/shell-game.js?v=2026-11-11-helm"></script>
+<script src="/builders/shell-game.js?v=2026-11-11-view2"></script>
 <script src="/builders/shell-trail.js?v=2026-11-05-numbers"></script>
 <script src="/builders/craft-mode.js?v=2026-11-10-visor"></script>
-<script src="/builders/shell.js?v=2026-11-11-helm"></script>
+<script src="/builders/shell.js?v=2026-11-11-view2"></script>
 <script src="/builders/hall-dialog.js"></script>
 <!-- jump palette (Ctrl/Cmd-K). Optional by design: without it the shell's slot stays hidden. -->
 <script src="/builders/palette.js" defer></script>

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-helm">
+  <link rel="stylesheet" href="/builders/style.css?v=2026-11-11-view2">
   <!-- 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. -->
@@ -172,10 +172,10 @@
 <!-- The app shell (sidebar + top bar + footer). Dependency-free; runs at parse
      time so the chrome exists before the deferred page scripts execute. -->
 <script src="/builders/dom-utils.js?v=2026-11-05-numbers"></script>
-<script src="/builders/shell-game.js?v=2026-11-11-helm"></script>
+<script src="/builders/shell-game.js?v=2026-11-11-view2"></script>
 <script src="/builders/shell-trail.js?v=2026-11-05-numbers"></script>
 <script src="/builders/craft-mode.js?v=2026-11-10-visor"></script>
-<script src="/builders/shell.js?v=2026-11-11-helm"></script>
+<script src="/builders/shell.js?v=2026-11-11-view2"></script>
 <script src="/builders/hall-dialog.js"></script>
 <!-- jump palette (Ctrl/Cmd-K). Optional by design: without it the shell's slot stays hidden. -->
 <script src="/builders/palette.js" defer></script>

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

@@ -7,14 +7,15 @@ Covers: every shelled hall page (`shell-game.js`, `shell.js`). Spec: `docs/specs
 - **Game mode is the existing full-screen switch, made hall-wide. There is no second full-screen mode.** The switch was already on every page (task 1004430) but remembered per page, so a reader had to turn it on 33 times. Game mode is one `localStorage` key, `hall-game`; while it is `on`, `shell.js` `storedFullscreen()` answers yes for every page, whatever that page remembers. What hides, the corner button and the Escape hand-off are the shell's, unchanged.
 - **`shell-game.js` holds the mode and nothing else**: `window.OTBShellGame` = `on()`, `set(bool)`, `hint()`. It stamps `html[data-game="on"]` for a sheet or script that wants to read the mode. `shell.js` reads it through a fallback object, so a page without the file is never in game mode. It is its own file because `shell.js` sits on the 1,500-line ratchet; the start screen and options menu (R03, R04) are expected to grow here, not in `shell.js`.
 - **Load order**: `shell-game.js` before `shell-trail.js` / `craft-mode.js` / `shell.js`, loaded the same way (deferred or not) as the shell on that page. Test-pinned.
-- **Leaving is for this page load only.** In game mode the corner button brings the bars back (Escape did too until task 1005144; it opens the options menu now), and nothing is written: the next page opens without them again, and a page's own `hall-fullscreen:<page>` memory is never overwritten. Turning game mode off (`set(false)`) hands each page back what it remembered. `OTBShell.setFullscreen(null)` is the "re-read what should show" call `set()` uses.
+- **Game or classic, never both (task 1005200, owner 2026-11-11).** Inside game mode there is no way back to the bars: the corner button is hidden (`style.css`) and does nothing if pressed, Escape opens the options menu, and the "how to leave" line is never shown. The only switch is Options → Preferences → View (`settings-view.js`), plus the chooser at every sign-in (`records/start-screen.md`). A page's own `hall-fullscreen:<page>` memory is still never overwritten, so switching to classic hands each page back what it remembered.
+- **Game mode reaches only the game's places** (`shell-game.js` `PLACES`: home, city, goal-map, thinking, atlas, settings, profile). Any other hall page `location.replace`s to the city on load, and so does switching to game on one. A screen embedded in the options menu is exempt. The visor drops an action whose `href` is not a place (`visor.js` `inGame`). A page joins `PLACES` when it has a game version.
+- **The choice is kept on the account**: `GET/PATCH /api/bongos/me/hall-view` (`modules/builder-settings/hall-view-prefs.js`, key `hall_view` in `behavior_prefs`, null = never chosen). `hall-game` in localStorage is only the copy a page paints from; `sync()` reads the account on every page load and switches if another browser changed it.
 - **No page may opt out.** `<body data-fullscreen-switch="off">` would keep that page's sidebar in game mode; the test fails on one. A new hall page needs `#app-topbar` and the `shell-game.js` tag, like the rest.
-- **The "how to leave" line shows once per tab in game mode** (`sessionStorage 'hall-game-hint'`), not on every page load. Outside game mode the shell's rule is unchanged.
-- **How it is turned on today**: `?game=on` / `?game=off` on any hall URL, or `OTBShellGame.set()`. Nothing in the hall's interface sets it yet; the start screen (R03) is where that lands.
+- **`?game=on` / `?game=off` are gone** (task 1005200): the address no longer switches the view.
 
 ## For the tasks after this one
 
 - A HUD or options menu that wants Escape must call `preventDefault()` on the press. `shell.js` `onFullscreenKey` waits one tick before leaving so a listener added after it still wins.
-- Since R04 (task 1005144, ADR 0401) the last Escape in game mode opens the options menu (`shell-game.js` `onEscape`), and on the menu it closes it again. The corner button is still the way back to the bars.
+- Since R04 (task 1005144, ADR 0401) the last Escape in game mode opens the options menu (`shell-game.js` `onEscape`), and on the menu it closes it again. Since task 1005200 there is no way back to the bars inside game mode.
 
 Tests: `tests/hall_fullscreen.mjs` (the eight `game mode:` cases run the real `shell-game.js` + `shell.js`), and the script lists in `tests/hall_city.mjs`, `tests/hall_street.mjs`, `tests/hall_my_agents.mjs`, `tests/goal_map_page.mjs`.

package/modules/hall-ui/records/start-screen.md

@@ -23,3 +23,7 @@ Covers: Home (`shell-game.js` `start()`, `shell-game.css`, the hand-off in `welc
 - New art for the start screen (the city in the clouds) is task 1005196; `welcome-boot.js` still hands in the welcome's painting.
 
 Tests: `tests/hall_start_screen.mjs` (the hand-off and pure parts in a vm; both screens, the panel's place at desk and phone widths, the segments, Quit, the no-city and reduced-motion paths in a real browser).
+
+## The view chooser (task 1005200, owner 2026-11-11)
+
+Every sign-in now opens on a chooser before the start screen: **Game** or **Classic**, each with one line saying what it is. The view chosen last time (the browser's `hall-game` copy) is marked "Last time", lit with the accent on its edge, and focused, so Enter keeps it. Game goes on to the start screen as before; Classic closes the screen and the hall is as it always was. Either choice is saved to the account (`PATCH /api/bongos/me/hall-view`). The Extras list lost The docs: it is not a game place. Tests: `tests/hall_start_screen.mjs`.

package/package-lock.json

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

package/package.json

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

@@ -11589,5 +11589,11 @@
       "id": "1005192",
       "text": "The hall now wears the new Helm look: deep night panels, ice-blue text, wide capital titles and cut-corner buttons, all from one shared design system. The start and load screens are rebuilt to match, and the hall is dark only."
     }
+  ],
+  "1.24.82": [
+    {
+      "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."
+    }
   ]
 }

package/scripts/hall-preview/pref-writes.js

@@ -14,7 +14,7 @@
 //
 // What it is NOT: storage. Nothing is written to disk and a restart forgets
 // everything; each recorded write is logged so the harness console shows what the
-// page sent. Only these five routes are recorded — every other write is refused
+// page sent. Only these routes are recorded — every other write is refused
 // exactly as before.
 'use strict';
 
@@ -27,6 +27,7 @@ const wandering = lib('wandering-prefs');
 const models = lib('model-alloc-prefs');
 const render = lib('render-prefs');
 const skills = lib('skill-prefs');
+const hallView = lib('hall-view-prefs'); // task 1005200: game or classic
 
 // apiPath → the /me slice it answers for (null: the skill prefs, not on /me)
 const ROUTES = {
@@ -35,6 +36,7 @@ const ROUTES = {
   'me/model-allocation': 'model_allocation',
   'me/render': 'render',
   'me/skill-prefs': null,
+  'me/hall-view': 'hall_view',
 };
 
 function createPrefRecorder({ log = console.log } = {}) {
@@ -63,6 +65,7 @@ function createPrefRecorder({ log = console.log } = {}) {
     if (slice === 'wandering') return wandering.describeForApi(behavior, rank);
     if (slice === 'model_allocation') return models.describeForApi(behavior, instanceModels);
     if (slice === 'render') return render.describeForApi(behavior);
+    if (slice === 'hall_view') return hallView.describeForApi(behavior);
     return {
       prefs: skillPrefs, effective: skills.listEffective(skillPrefs), defaults: skills.SKILL_DEFAULTS,
       known_skills: skills.KNOWN_SKILLS, allowed_models: skills.ALLOWED_MODELS,
@@ -96,6 +99,10 @@ function createPrefRecorder({ log = console.log } = {}) {
       v = render.validatePatch(b);
       if (!v.ok) return invalid(v.errors);
       behavior = render.applyPatch(behavior, v.patch);
+    } else if (slice === 'hall_view') {
+      v = hallView.validatePatch(b);
+      if (!v.ok) return invalid(v.errors);
+      behavior = hallView.applyView(behavior, v.view);
     } else {
       v = skills.validatePatch(b);
       if (!v.ok) return invalid(v.errors);

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.81'; // CI auto-patch carrier (ADR 0161); changelog: docs/module-api-changelog.md
+const CORE_VERSION = '1.24.82'; // 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/hall_fullscreen.mjs

@@ -94,7 +94,8 @@ function boot({ page = 'thinking', optIn = true, store = {}, storageThrows = fal
     get cookie() { return ''; }, set cookie(_v) {},
   };
   const assigned = []; // task 1005144: where the page sent the browser (game mode's Escape opens the menu)
-  const location = { hostname: 'h', origin: 'https://h', href: 'https://h/' + page + search, pathname: path || '/' + page, hash, search, assign(u) { assigned.push(u); }, replace() {} };
+  const replaced = []; // task 1005200: a classic page in game mode sends the browser to the city
+  const location = { hostname: 'h', origin: 'https://h', href: 'https://h/' + page + search, pathname: path || '/' + page, hash, search, assign(u) { assigned.push(u); }, replace(u) { replaced.push(u); } };
   // `session` is the TAB's sessionStorage: pass one object to several boots and
   // they are page loads in one tab (the trail, task 1004476).
   const sessionStorage = {
@@ -135,6 +136,7 @@ function boot({ page = 'thinking', optIn = true, store = {}, storageThrows = fal
   return {
     store,
     assigned,
+    replaced,
     game: () => windowObj.OTBShellGame,
     gameStamp: () => root.dataset.game === 'on',
     // The "how to leave" line: showing = the shell created it and it is not hidden.
@@ -517,17 +519,36 @@ test('game mode: every hall page carries the switch — none opts out, and each
   }
 });
 
-test('game mode: leaving brings the bars back for THIS page only, and never rewrites a page\'s own memory', () => {
+// task 1005200 (owner, 2026-11-11): game OR classic, never both. Inside game mode there is
+// no way back to the bars — the corner button is hidden (style.css) and, pressed
+// anyway, does nothing; Escape is the options menu's, where the view is switched.
+test('game mode: there is no way back to the bars inside it, and a page\'s own memory is never rewritten', () => {
   const store = { 'hall-game': 'on', 'hall-fullscreen:thinking': 'on' };
-  const goals = boot({ page: 'goals', optIn: false, store });
-  goals.button().click();
-  assert.equal(goals.on(), false, 'the corner button still leaves (Escape opens the options menu, below)');
-  assert.equal(store['hall-fullscreen:goals'], undefined, 'nothing remembered for the page');
-  assert.equal(boot({ page: 'goals', optIn: false, store }).on(), true, 'the next load opens without bars again');
+  const profile = boot({ page: 'profile', optIn: false, store });
+  profile.button().click();
+  assert.equal(profile.on(), true, 'the corner button does not leave game mode');
+  assert.equal(store['hall-fullscreen:profile'], undefined, 'nothing remembered for the page');
   const sky = boot({ page: 'thinking', store });
   sky.button().click();
-  assert.equal(sky.on(), false, 'the corner button leaves too');
+  sky.press('Escape');
+  assert.equal(sky.on(), true, 'nor does Escape');
   assert.equal(store['hall-fullscreen:thinking'], 'on', 'the sky\'s own choice survives game mode');
+  assert.match(read('style.css'), /html\[data-game="on"\] \.fullscreen-toggle \{ display: none; \}/, 'and the button is not drawn');
+});
+
+test('game mode: only the game\'s places are there — a classic page sends the browser to the city', () => {
+  const store = { 'hall-game': 'on' };
+  assert.deepEqual(boot({ page: 'goals', optIn: false, store }).replaced, ['/'], 'Goals is a classic page');
+  assert.deepEqual(boot({ page: 'work', optIn: false, store, path: '/builders/work' }).replaced, ['/builders/'], 'on the apex too');
+  for (const page of ['home', 'city', 'goal-map', 'thinking', 'atlas', 'settings', 'profile']) {
+    assert.deepEqual(boot({ page, optIn: false, store }).replaced, [], `${page} is a game place`);
+  }
+  assert.deepEqual(boot({ page: 'goals', optIn: false, store: {} }).replaced, [], 'classic reaches every page');
+  const G = boot({ page: 'city', store }).game();
+  assert.equal(G.isPlace('/city'), true);
+  assert.equal(G.isPlace('/builders/'), true);
+  assert.equal(G.isPlace('/work'), false);
+  assert.equal(G.isPlace('/blockers?x=1'), false);
 });
 
 // task 1005144 (BONGOS-V3 R04, ADR 0399 §2.4 row 6): in game mode the last Escape is the
@@ -562,35 +583,35 @@ test('game mode: the Escape hand-off is unchanged — a press the page used, or
 
 test('game mode: set() switches the page live, and turning it off hands each page back its own memory', () => {
   const store = { 'hall-fullscreen:thinking': 'on' };
-  const goals = boot({ page: 'goals', optIn: false, store });
+  const goals = boot({ page: 'profile', optIn: false, store });
   goals.game().set(true);
   assert.equal(store['hall-game'], 'on');
   assert.equal(goals.on(), true, 'on at once, no reload');
   assert.ok(goals.resizes() >= 1, 'a page that sizes to its box is told to re-measure');
   goals.game().set(false);
-  assert.equal(goals.on(), false, 'goals remembered nothing, so its bars return');
+  assert.equal(goals.on(), false, 'the profile remembered nothing, so its bars return');
   assert.equal(goals.gameStamp(), false);
-  assert.equal(store['hall-fullscreen:goals'], undefined, 'and switching the mode wrote nothing for the page');
+  assert.equal(store['hall-fullscreen:profile'], undefined, 'and switching the mode wrote nothing for the page');
+  const work = boot({ page: 'work', optIn: false, store: {} });
+  work.game().set(true);
+  assert.deepEqual(work.replaced, ['/'], 'switched to game on a classic page: to the city');
   const sky = boot({ page: 'thinking', store: Object.assign(store, { 'hall-game': 'on' }) });
   sky.game().set(false);
   assert.equal(sky.on(), true, 'the sky had chosen full screen itself, and keeps it');
 });
 
-test('game mode: ?game=on / ?game=off set it from the URL; anything else is ignored', () => {
+test('game mode: the address does not switch it — Options and the sign-in chooser do (task 1005200)', () => {
   const store = {};
-  assert.equal(boot({ page: 'work', optIn: false, store, search: '?game=on' }).on(), true);
-  assert.equal(store['hall-game'], 'on');
-  assert.equal(boot({ page: 'goals', optIn: false, store }).on(), true, 'and it holds on the next page');
-  assert.equal(boot({ page: 'work', optIn: false, store, search: '?game=off' }).on(), false);
-  boot({ page: 'work', optIn: false, store, search: '?game=%22%3E%3Cimg%3E' });
-  assert.equal(store['hall-game'], 'off', 'a value that is not on/off is not stored');
+  assert.equal(boot({ page: 'profile', optIn: false, store, search: '?game=on' }).on(), false);
+  assert.equal(store['hall-game'], undefined, 'nothing stored from the URL');
+  assert.doesNotMatch(read('shell-game.js'), /get\('game'\)/, 'the ?game= reader is gone');
 });
 
-test('game mode: how to leave is said once per tab, not on every page; outside it the shell\'s rule stands', () => {
+test('game mode never says how to leave full screen (there is no leaving); outside it the shell\'s rule stands', () => {
   const store = { 'hall-game': 'on' };
   const session = {};
-  assert.equal(boot({ page: 'goals', optIn: false, store, session }).hintShowing(), true, 'the first page says it');
-  assert.equal(boot({ page: 'work', optIn: false, store, session }).hintShowing(), false, 'the next does not repeat it');
+  assert.equal(boot({ page: 'profile', optIn: false, store, session }).hintShowing(), false, 'no line in game mode');
+  assert.equal(boot({ page: 'city', store, session }).hintShowing(), false);
   const own = { 'hall-fullscreen:thinking': 'on' };
   assert.equal(boot({ page: 'thinking', store: own, session: {} }).hintShowing(), true);
   assert.equal(boot({ page: 'thinking', store: own, session: {} }).hintShowing(), true, 'a remembered page says it on every open, as before');

package/tests/hall_start_screen.mjs

@@ -188,12 +188,21 @@ test('a sign-in passes through the start screen and the load screen before the w
   const { page, errors } = await signedIn(ctx);
   await page.waitForSelector('#game-start');
   assert.deepEqual(await state(page), { start: true, welcome: false, picker: false, game: null, full: null },
-    'the start screen, not the welcome and not the picker; game mode is not on until Start');
+    'the start screen, not the welcome and not the picker; game mode is not on until it is chosen');
+  // task 1005200: first the chooser — game or classic — then, for game, the start screen.
+  const views = await page.evaluate(() => ({
+    items: Array.from(document.querySelectorAll('.gs__items:not([hidden]) .gs__item')).map((e) => e.textContent),
+    focus: document.activeElement.textContent,
+  }));
+  assert.deepEqual(views, { items: ['GameThe city, full screen', 'ClassicThe hall with its sidebar'], focus: 'GameThe city, full screen' });
+  await page.keyboard.press('Enter');
+  await page.waitForSelector('#gs-items:not([hidden])');
+  assert.equal((await state(page)).game, 'on', 'choosing Game turns game mode on');
   const menu = await page.evaluate(() => ({
     title: document.getElementById('gs-title').textContent,
-    items: Array.from(document.querySelectorAll('.gs__items:not(.gs__items--extras) .gs__item')).map((e) => e.textContent),
+    items: Array.from(document.querySelectorAll('.gs__items:not(.gs__items--extras):not([hidden]) .gs__item')).map((e) => e.textContent),
     focus: document.activeElement.textContent,
-    small: Array.from(document.querySelectorAll('.gs__items:not(.gs__items--extras) .gs__item')).filter((e) => e.getBoundingClientRect().height < 46).length,
+    small: Array.from(document.querySelectorAll('.gs__items:not(.gs__items--extras):not([hidden]) .gs__item')).filter((e) => e.getBoundingClientRect().height < 46).length,
   }));
   assert.deepEqual(menu, { title: 'Harbor', items: ['Start', 'Options', 'Achievements', 'Extras', 'Quit'], focus: 'Start', small: 0 });
 
@@ -253,6 +262,29 @@ test('a sign-in passes through the start screen and the load screen before the w
   await ctx.close();
 });
 
+// task 1005200: Classic at the chooser — the screen goes, game mode is off, and the account
+// is told. The view chosen last time is the one marked and focused.
+test('choosing Classic at sign-in closes the screen, leaves game mode off, and saves it to the account', { skip }, async () => {
+  const ctx = await hall();
+  const saved = [];
+  await ctx.route(ORIGIN + '/api/bongos/me/hall-view', (route) => {
+    if (route.request().method() === 'PATCH') saved.push(JSON.parse(route.request().postData()));
+    return route.fulfill({ contentType: 'application/json', body: JSON.stringify({ view: null }) });
+  });
+  const page = await ctx.newPage();
+  await page.addInitScript(() => { try { localStorage.setItem('hall-welcome-signin', '1'); localStorage.setItem('hall-game', 'off'); } catch (_) { /* no storage */ } });
+  await page.goto(ORIGIN + '/');
+  await page.waitForFunction(() => !!window.OTBShell && !!window.OTBWelcome && !!window.OTBShellGame);
+  await page.evaluate(() => { const me = { authed: true, rank: 'metic', builder: { id: 7, display_name: 'Ada', github_login: 'ada' } }; window.OTBShell.applyAccess(me); window.OTBShell.applyAccess(me); });
+  await page.waitForSelector('#game-start');
+  assert.equal(await page.evaluate(() => document.activeElement.textContent), 'ClassicLast timeThe hall with its sidebar', 'the view chosen last time is focused and marked');
+  await page.keyboard.press('Enter');
+  await page.waitForSelector('#game-start', { state: 'detached' });
+  assert.equal((await state(page)).game, null);
+  assert.deepEqual(saved, [{ view: 'classic' }]);
+  await ctx.close();
+});
+
 test('on a phone the panel is the whole width, nothing scrolls sideways and every choice clears the floor (task 1005192)', { skip }, async () => {
   for (const [w, h] of [[390, 844], [320, 700]]) {
     const ctx = await hall({ viewport: { width: w, height: h } });
@@ -266,6 +298,7 @@ test('on a phone the panel is the whole width, nothing scrolls sideways and ever
       };
     });
     assert.deepEqual(m, { x: 0, w, sideways: 0, small: 0 }, `at ${w}px`);
+    await page.click('[data-gs="view-game"]');
     await page.click('[data-gs="start"]');
     await page.waitForSelector('#gs-load:not([hidden])');
     assert.equal(await page.evaluate(() => { const gs = document.getElementById('game-start'); return gs.scrollWidth - gs.clientWidth; }), 0, `the load screen fits ${w}px`);
@@ -279,6 +312,7 @@ test('Quit closes the tab, and says so when the browser keeps it open', { skip }
   const { page } = await signedIn(ctx);
   await page.waitForSelector('#game-start');
   await page.evaluate(() => { window.__closed = 0; window.close = () => { window.__closed += 1; }; });
+  await page.click('[data-gs="view-game"]');
   await page.click('[data-gs="quit"]');
   await page.waitForSelector('#gs-note:not([hidden])');
   assert.equal(await page.evaluate(() => window.__closed), 1);
@@ -291,6 +325,7 @@ test('with no city to draw, or with reduced motion, the still and the tip stand
     const ctx = await hall(opts);
     const { page, errors } = await signedIn(ctx);
     await page.waitForSelector('#game-start');
+    await page.click('[data-gs="view-game"]');
     await page.click('[data-gs="start"]');
     await page.waitForSelector('#gs-load:not([hidden])');
     assert.match(await page.textContent('#gs-tip'), /\.$/);

package/tests/hall_view_prefs.mjs

@@ -0,0 +1,42 @@
+// tests/hall_view_prefs.mjs — the game-or-classic choice kept on the account
+// (task 1005200, modules/builder-settings/hall-view-prefs.js), and the two routes
+// that read and write it.
+import { test } from 'node:test';
+import assert from 'node:assert/strict';
+import fs from 'node:fs';
+import path from 'node:path';
+import { createRequire } from 'node:module';
+import { fileURLToPath } from 'node:url';
+
+const require = createRequire(import.meta.url);
+const ROOT = fileURLToPath(new URL('..', import.meta.url));
+const prefs = require('../modules/builder-settings/hall-view-prefs.js');
+
+test('never chosen is null, not a default view: the sign-in chooser asks', () => {
+  assert.equal(prefs.storedView({}), null);
+  assert.equal(prefs.storedView(null), null);
+  assert.equal(prefs.storedView({ hall_view: 'fullscreen' }), null, 'an unknown value is not a choice');
+  assert.deepEqual(prefs.describeForApi({ hall_view: 'game' }), { view: 'game', views: ['game', 'classic'] });
+});
+
+test('a PATCH takes exactly { view: game | classic }', () => {
+  assert.deepEqual(prefs.validatePatch({ view: 'classic' }), { ok: true, view: 'classic' });
+  assert.equal(prefs.validatePatch({}).ok, false, 'no view');
+  assert.equal(prefs.validatePatch({ view: 'both' }).ok, false);
+  assert.equal(prefs.validatePatch({ view: 'game', extra: 1 }).ok, false, 'unknown keys refused');
+  assert.equal(prefs.validatePatch([]).ok, false);
+});
+
+test('saving the view keeps every other behaviour preference', () => {
+  const before = { wandering: 2, widgets: 'rich', interaction: { verbosity: 'concise' } };
+  const after = prefs.applyView(before, 'game');
+  assert.deepEqual(after, { ...before, hall_view: 'game' });
+  assert.equal(before.hall_view, undefined, 'pure: the input is not changed');
+});
+
+test('the routes: own builder only, GET and PATCH on /me/hall-view', () => {
+  const src = fs.readFileSync(path.join(ROOT, 'modules/builder-settings/routes/settings.js'), 'utf8');
+  assert.match(src, /router\.get\('\/me\/hall-view', api\.requireBuilder,/);
+  assert.match(src, /router\.patch\('\/me\/hall-view', api\.requireBuilder,/);
+  assert.match(src, /hallViewPrefs\.applyView\(existing, v\.view\)/, 'writes through the pure shaper');
+});

package/tests/hall_visor.mjs

@@ -151,7 +151,7 @@ test('in game mode the glass shows the place, the main action, the sprite and th
   }));
   assert.equal(g.shown, 'block');
   assert.equal(g.place, 'Test city');
-  assert.match(g.main, /^Claim a task/);
+  assert.match(g.main, /^Goal sky/, 'Claim a task opens a classic page, so it is off the glass in game mode (task 1005200)');
   assert.equal(g.who, 'Artisan');
   assert.equal(g.credits, '56,021 credits');
   assert.equal(g.sprite, '/builders/characters/artisan.png');
@@ -189,7 +189,8 @@ test('the sheet lists the character\'s actions, then this place, and runs a scen
   const { ctx, page } = await open();
   await page.click('[data-visor-sheet]');
   const rows = await page.$$eval('.visor__sheet .visor__row', (r) => r.map((x) => [...x.children].map((c) => c.textContent).join(' ')));
-  assert.deepEqual(rows, ['Claim a task C', 'Blockers B', 'Goal sky G', 'Your thinking T', 'Back to centre 0', 'Look around drag', 'Options Esc']);
+  // task 1005200: Claim a task and Blockers open classic pages, so they are off the glass.
+  assert.deepEqual(rows, ['Goal sky G', 'Your thinking T', 'Back to centre 0', 'Look around drag', 'Options Esc']);
   await page.click('[data-visor-scene="0"]');
   assert.deepEqual(await page.evaluate(() => ({ centred: window.centred, open: OTBVisor.isOpen() })), { centred: true, open: false });
   await ctx.close();

package/tests/nav_permission_atoms.mjs

@@ -103,7 +103,7 @@ test('the hall-preview fixture carries every atom /me resolves', () => {
 // task 1005070: the remembered nav (the rail paints before /me).
 // task 1005088: the sidebar mark is drawn by reference to the one brand mark.
 // task 1005144: My agents, Project settings, Permissions and Modules left for the options menu.
-const SHELL_SHA = 'a4d41b30b41e49f4';
+const SHELL_SHA = '89f6f166ed22a8ee';
 
 test('shell.js has not changed without its cache-bust being bumped', () => {
   const actual = crypto.createHash('sha256').update(SHELL).digest('hex').slice(0, 16);