Fix rapid typing and save sequencing regressions

scenes/bunny.jedit-scene

@@ -1,7 +1,7 @@
 {
   "camera": {
-    "position": [0, 1.45, -2.55],
-    "targetObject": "stanford-bunny"
+    "position": [-1.05, 0.46, 2.15],
+    "target": [-1.05, 1.2, -0.24]
   },
   "environment": {
     "background": [2, 3, 7],
@@ -65,6 +65,42 @@
       "color": [148, 232, 124],
       "reflectivity": 0.13
     },
+    {
+      "label": "chrome-left-orb",
+      "kind": "sphere",
+      "position": [-1.42, 0.62, -1.55],
+      "radius": 0.48,
+      "height": 0.96,
+      "footprintRadius": 0.48,
+      "color": [218, 237, 255],
+      "reflectivity": 0.94,
+      "transparency": 0.04,
+      "refractiveIndex": 1.18
+    },
+    {
+      "label": "chrome-crown-orb",
+      "kind": "sphere",
+      "position": [0.12, 1.05, -1.95],
+      "radius": 0.58,
+      "height": 1.16,
+      "footprintRadius": 0.58,
+      "color": [236, 244, 255],
+      "reflectivity": 0.96,
+      "transparency": 0.03,
+      "refractiveIndex": 1.2
+    },
+    {
+      "label": "chrome-right-orb",
+      "kind": "sphere",
+      "position": [1.5, 0.68, -1.52],
+      "radius": 0.5,
+      "height": 1.0,
+      "footprintRadius": 0.5,
+      "color": [204, 226, 255],
+      "reflectivity": 0.92,
+      "transparency": 0.05,
+      "refractiveIndex": 1.16
+    },
     {
       "label": "amber-glass-tower",
       "kind": "cube",

scenes/continuum-gate.jedit-scene

@@ -0,0 +1,44 @@
+{
+  "camera": {
+    "position": [0, 4.2, 3.4],
+    "target": [0, 1.25, 0]
+  },
+  "environment": {
+    "background": [2, 3, 7],
+    "floor": {
+      "kind": "grid",
+      "dark": [2, 3, 7],
+      "light": [68, 90, 100],
+      "gridScale": 1.2,
+      "fadeDistance": 34
+    }
+  },
+  "objects": [
+    {
+      "label": "stanford-bunny",
+      "kind": "mesh",
+      "mesh": "bunny",
+      "offset": [-0.95, 0, 2.84],
+      "localYaw": { "phase": 2.94, "angularSpeed": 0 },
+      "radius": 1.02,
+      "footprintRadius": 1.02,
+      "height": 2.35,
+      "color": [82, 98, 104],
+      "reflectivity": 0.1,
+      "transparency": 0,
+      "refractiveIndex": 1
+    },
+    {
+      "label": "chrome-mirror-sphere",
+      "kind": "sphere",
+      "position": [0, 1.25, 0],
+      "radius": 1.18,
+      "height": 2.36,
+      "footprintRadius": 1.18,
+      "color": [236, 244, 255],
+      "reflectivity": 0.99,
+      "transparency": 0.005,
+      "refractiveIndex": 1.06
+    }
+  ]
+}

spec/fixtures/title-scene-visual-witness.json

@@ -19,7 +19,7 @@
       "⢕⢵⢝⢵░▓█▒⠕⢅⢕⢵⢝⢽⢕⢥⠝⢽⢝⢽⢝⢝⠕⢅⠕⢅⠕⢽⢝⢽⢝⢵⠝⢕⠝⢵⢝⢵⠕⢅⠕⠭",
       "⡕⡅⡝⡕██▓░⡕⣥⣝⣽⣝⣽⣝⡍⡕⡅⡕⡍⡝⣅⡕⡅⡕⡅⡕⣽⣝⣽⣝⣽⣝⡅⡕⡅⡕⡅⡕⡥⣕⣭"
     ],
-    "foregroundDigest": "cf49e7a0fdb9d729",
+    "foregroundDigest": "0be1378fba7ca909",
     "backgroundDigest": "65a24ea998becce0"
   },
   {
@@ -42,7 +42,7 @@
       "⠕⢅⢝⢅⠕⢅⠕⢅⢕⢵⢝⢵⢝⢽⠝⢝⠝⢽⢝⢽⠕⢅⠕⢅⠕⢅⢝⢵⢕⢅⠕⢵⢝⢵⢝⢵⢝⢅⠕⢅⠝⠭",
       "⣕⡵⣝⡵⣕⡅⣕⣵⣝⣽⣝⣽⣝⡍⡕⡅⡕⡅⡝⣅⡕⡅⡕⡅⡕⣽⣝⣽⣝⣽⣝⡍⡝⡵⣝⡵⣝⡅⡕⡅⡕⣭"
     ],
-    "foregroundDigest": "6b666ae8f7c6f251",
+    "foregroundDigest": "acaad4c34d0e5484",
     "backgroundDigest": "d56a3f804cc5b610"
   },
   {
@@ -52,10 +52,10 @@
     "glyphRows": [
       "                                          ",
       "                 55                       ",
-      "             ,,,XH5,Xi,,,,,,,,            ",
-      ",,,,,iiiiiiiiiiiiG5#B9Miiiiiiiiiiiiii,,,,i",
+      "             ,,,XH2,Xi,,,,,,,,            ",
+      ",,,,,iiiiiiiiiiiiG3#B9Miiiiiiiiiiiiii,,,,i",
       "iiiiiiiiiiiiiiiiiSSS#HhhS2h33iiiiiiiiiiii2",
-      "iiiiiiiiirii#Brrr#G35AXsrMMMM2riiiiiiiiii2",
+      "iiiiiiiiirii#Brrr#G35AAsrMMMM2riiiiiiiiii2",
       "iiiiriiiriii32i;rAr2XsXiX3AAAXXiiiiiiiiir5",
       ",rri,;iri,irs;rr:;,,Xri;:3AAAMi:i,rrii:i, ",
       "iri,irri,,,sr;,:sssr:is,,5AA2:irrri,:iii, ",
@@ -65,7 +65,7 @@
       ",,r:,,,,rsssss;r;sss,,,,,;ssr;,ssssss,,,, ",
       "Xs2s5;X555555,;,;,;Xr,;,;A55555;rrAs5;;,;;"
     ],
-    "foregroundDigest": "e7baf08de00f5c17",
+    "foregroundDigest": "c6c7e3ff04d5fe00",
     "backgroundDigest": "4cc8acca331fa313"
   },
   {
@@ -88,7 +88,7 @@
       "⠕⢅⢕⢵⢕⢅⠕⢅⢕⢵⢝⢵⢝⢽⠕⢝⠝⢽⢝⢝⠕⢅⠕⢅⠕⢽⢕⢥⢕⢅⠕⢵⢝⢵⢝⢵⢕⢅⠝⢕⠝⠭",
       "⣕⡵⣝⡵⣝⡵⣕⣵⣝⣽⣝⣽⡝⡅⡕⡅⡕⡍⣝⣅⡕⡅⡕⡅⡕⣽⣝⣽⣝⣽⣕⡝⡝⣵⣝⡵⣝⡅⡕⡅⡕⣭"
     ],
-    "foregroundDigest": "fd4770907df4d58d",
+    "foregroundDigest": "9db9b0c054e3e9ff",
     "backgroundDigest": "7e30be255634420e"
   }
 ]

spec/settings-drawer.spec.mjs

@@ -47,15 +47,17 @@ test('settings drawer renders structured rows and highlights the selected row',
   const text = surfaceText(surface);
 
   assert.match(text, /Settings/);
-  assert.match(text, /F2\/Esc close/);
-  assert.match(text, /● English Current/);
+  assert.match(text, /F2\/Esc\/q close/);
+  assert.match(text, /\[x\] English/);
   assert.match(text, /↻ Theme/);
   assert.match(text, /☑ Footer/);
   assert.match(text, /↻ Markdown preview/);
   assert.match(text, /↻ Diagnostics Open/);
-  assert.equal(surface.get(2, 8).char, '›');
-  assert.equal(surface.get(2, 8).fg, theme.cursor.normal.fg);
-  assert.equal(surface.get(2, 8).bg, theme.cursor.normal.bg);
+  assert.ok(text.includes('› ↻ Theme'));
+  const selectedMarker = surface.get(2, 8);
+  assert.equal(selectedMarker.char, '›');
+  assert.equal(selectedMarker.fg, theme.cursor.normal.fg);
+  assert.equal(selectedMarker.bg, theme.cursor.normal.bg);
 });
 
 test('settings drawer keeps the focused row visible when section headers consume short drawers', async () => {
@@ -81,3 +83,29 @@ test('settings drawer keeps the focused row visible when section headers consume
 
   assert.match(surfaceText(surface), /› ↻ Markdown preview Source/);
 });
+
+test('settings drawer always returns the requested positive surface size', async () => {
+  const { drawer, settings, themes } = await loadDrawerModules();
+  const theme = themes.availableJeditThemes()[0];
+  const rows = settings.jeditSettingsRows({
+    i18n: createI18nMock(),
+    jeditTheme: theme,
+    footerVisible: true,
+    markdownPreviewActive: true,
+    diagnosticsAvailable: true,
+    viewMode: 'source',
+  });
+
+  for (const [width, height] of [[1, 1], [2, 2], [5, 3], [42, 20]]) {
+    const surface = drawer.renderSettingsDrawer({
+      rows,
+      selectedIndex: 0,
+      theme,
+      width,
+      height,
+    });
+
+    assert.equal(surface.width, width);
+    assert.equal(surface.height, height);
+  }
+});

spec/settings-session.spec.mjs

@@ -5,13 +5,15 @@ import { pathToFileURL } from 'node:url';
 import { createI18nMock } from './i18n-mock.mjs';
 import { REPO_ROOT, ensureDistBuilt } from './dist-helpers.mjs';
 
+const KEYBINDINGS_PATH = path.join(REPO_ROOT, 'dist', 'app', 'keybindings.js');
 const SESSION_PATH = path.join(REPO_ROOT, 'dist', 'app', 'settings-session.js');
 const THEMES_PATH = path.join(REPO_ROOT, 'dist', 'ui', 'jedit-themes.js');
 
 async function loadSettingsModules() {
   await ensureDistBuilt();
 
   return {
+    keybindings: await import(pathToFileURL(KEYBINDINGS_PATH).href),
     settings: await import(pathToFileURL(SESSION_PATH).href),
     themes: await import(pathToFileURL(THEMES_PATH).href),
   };
@@ -46,8 +48,8 @@ test('jedit settings rows expose theme, footer, and markdown preview preferences
   assert.deepEqual(
     rows.map((row) => [row.label, row.valueLabel, row.kind, row.checked === true]),
     [
-      ['English', 'en', settings.JEDIT_SETTING_ROW_KIND.Option, false],
-      ['Français', 'Current', settings.JEDIT_SETTING_ROW_KIND.Option, true],
+      ['English', '', settings.JEDIT_SETTING_ROW_KIND.Option, false],
+      ['Français', '', settings.JEDIT_SETTING_ROW_KIND.Option, true],
       ['Theme', theme.name, settings.JEDIT_SETTING_ROW_KIND.Choice, false],
       ['Light/dark', 'Dark', settings.JEDIT_SETTING_ROW_KIND.Choice, false],
       ['Footer', 'On', settings.JEDIT_SETTING_ROW_KIND.Toggle, true],
@@ -57,17 +59,19 @@ test('jedit settings rows expose theme, footer, and markdown preview preferences
   );
 });
 
-test('settings focus movement clamps to the available rows', async () => {
+test('settings focus movement loops through the available rows', async () => {
   const { settings } = await loadSettingsModules();
 
   assert.equal(settings.moveSettingsFocusIndex(0, 1, 3), 1);
-  assert.equal(settings.moveSettingsFocusIndex(2, 1, 3), 2);
-  assert.equal(settings.moveSettingsFocusIndex(0, -1, 3), 0);
+  assert.equal(settings.moveSettingsFocusIndex(2, 1, 3), 0);
+  assert.equal(settings.moveSettingsFocusIndex(0, -1, 3), 2);
   assert.equal(settings.moveSettingsFocusIndex(3, 0, 3), 2);
+  assert.equal(settings.moveSettingsFocusIndex(0, 1, 0), 0);
+  assert.equal(settings.moveSettingsFocusIndex(4, -1, 0), 0);
 });
 
 test('settings key reducer closes, moves, and activates focused settings rows', async () => {
-  const { settings, themes } = await loadSettingsModules();
+  const { keybindings, settings, themes } = await loadSettingsModules();
   const theme = themes.availableJeditThemes()[0];
   const baseModel = {
     i18n: createI18nMock(),
@@ -109,6 +113,14 @@ test('settings key reducer closes, moves, and activates focused settings rows',
   const [closed] = settings.updateJeditSettingsFromKey({ key: 'escape' }, baseModel, rows, handlers);
   assert.equal(closed.settingsOpen, false);
 
+  const [closedWithQ] = settings.updateJeditSettingsFromKey(
+    { key: keybindings.JEDIT_SETTINGS_CLOSE_KEY },
+    baseModel,
+    rows,
+    handlers,
+  );
+  assert.equal(closedWithQ.settingsOpen, false);
+
   const [moved] = settings.updateJeditSettingsFromKey({ key: 'down' }, baseModel, rows, handlers);
   assert.equal(moved.settingsFocusIndex, 1);
 

spec/title-braille-sampling-policy.spec.mjs

@@ -174,7 +174,7 @@ test("title Braille sampling holds reduced quality while screen activity remains
   );
 });
 
-test("title Braille sampling traces one dot while the camera moves under pressure", async () => {
+test("title Braille sampling keeps full quality while the camera moves", async () => {
   const sampling = await importDist(
     "app",
     "workspace",
@@ -190,8 +190,8 @@ test("title Braille sampling traces one dot while the camera moves under pressur
       cameraMoving: true,
     }),
     {
-      phase: 2,
-      phaseCount: 8,
+      phase: 0,
+      phaseCount: 1,
     },
   );
 });

spec/title-scene-loader.spec.mjs

@@ -56,19 +56,23 @@ function normalize(vector) {
   return vector.map((component) => component / length);
 }
 
-test("loaded mesh scenes do not expose scene-authored mesh position as ray-hit state", async () => {
+test("loaded mesh scenes decode authored mesh transforms without exposing position", async () => {
   const { loader, titleScene, titleMeshLibrary, titleBunnyMesh } =
     await loadTitleSceneLoaderModules();
   const mesh = titleMeshLibrary.createTitleBunnyMesh(
     titleBunnyMesh.loadTitleBunnyMeshSource(),
   );
+  const offset = [0.5, 0.1, -0.25];
+  const localYaw = { phase: 1.25, angularSpeed: 0.125 };
   const scene = loader.parseTitleSceneJson(
     {
       objects: [
         {
           kind: titleScene.TITLE_SCENE_SHAPE_KIND.Mesh,
           mesh: "bunny",
           position: [100, 100, 100],
+          offset,
+          localYaw,
           radius: mesh.footprintRadius,
           footprintRadius: mesh.footprintRadius,
           height: mesh.height,
@@ -80,12 +84,15 @@ test("loaded mesh scenes do not expose scene-authored mesh position as ray-hit s
     { bunny: mesh },
   );
   const loadedMesh = scene.objects[0];
-  const origin = [-1, 1.1, 4.5];
-  const ray = normalize([-0.05, -0.04, -1]);
+  const center = titleScene.titleSceneObjectFootprintCenter(loadedMesh);
+  const origin = [center[0], center[1] + 0.02, center[2] + 4.5];
+  const ray = normalize([0, -0.02, -1]);
   const hit = titleScene.nearestTitleSceneObjectHit(origin, ray, scene.objects);
 
   assert.equal(loadedMesh.kind, titleScene.TITLE_SCENE_SHAPE_KIND.Mesh);
   assert.equal(Object.hasOwn(loadedMesh, "position"), false);
+  assert.deepEqual(loadedMesh.offset, offset);
+  assert.deepEqual(loadedMesh.localYaw, localYaw);
   assert.ok(hit != null);
   assert.equal(hit.object, loadedMesh);
 });