feat(admin): graph row glyphs in the styler's twin tables (issue #54)

arrowRight, foldOpen, foldClosed and rearm join the overlay glyph
twins, width-identical across both tables on purpose: a graph row's
padding math must not depend on which table is active, or the 80-col
invariant would break only on ASCII terminals, the least debuggable
place for it. Key parity and the twin widths are pinned; the ascii
table stays pure ASCII by assertion.

Signed-off-by: Rob Boerman <robboerman@live.nl>

Author: edgehero

admin/src/style.mjs

@@ -47,8 +47,10 @@ export const PLAIN_THEME = {
 // switch (`setGlyphs`) must not restyle overlays behind a styler's back, so the overlay opts in per
 // styler instance via `makeStyler(theme, { ascii })`. (The sparkline ramp is the one exception -- its
 // quantization geometry lives only in panel.mjs, so `styler.sparkline` follows panel's active table.)
-const OVERLAY_GLYPHS = { tl: "┌", tr: "┐", bl: "└", br: "┘", ml: "├", mr: "┤", h: "─", v: "│", full: "█", empty: "░", ellipsis: "…" };
-const OVERLAY_ASCII = { tl: "+", tr: "+", bl: "+", br: "+", ml: "+", mr: "+", h: "-", v: "|", full: "#", empty: ".", ellipsis: "..." };
+// The graph rows (issue #54) add four keys; every twin pair below is width-identical on purpose, so a
+// renderer's padding math never depends on which table is active.
+const OVERLAY_GLYPHS = { tl: "┌", tr: "┐", bl: "└", br: "┘", ml: "├", mr: "┤", h: "─", v: "│", full: "█", empty: "░", ellipsis: "…", arrowRight: "─▶", foldOpen: "▾", foldClosed: "▸", rearm: "↻" };
+const OVERLAY_ASCII = { tl: "+", tr: "+", bl: "+", br: "+", ml: "+", mr: "+", h: "-", v: "|", full: "#", empty: ".", ellipsis: "...", arrowRight: "->", foldOpen: "v", foldClosed: ">", rearm: "~" };
 
 /** Per-class colors for `fmtCost`: an estimate must LOOK provisional, and plan coverage must not look free. */
 const COST_COLORS = {

admin/test/style.test.mjs

@@ -116,6 +116,20 @@ test("makeStyler ascii option swaps frame/meter/divider glyphs with identical ge
   assert.equal(s.cell("abcdefgh", 5), "ab...", "the 3-char ellipsis still lands on exactly width");
 });
 
+test("the graph glyphs (issue #54) keep key parity and identical widths across both tables", () => {
+  // Width-identical twins are the contract: a graph row's padding math must not depend on which
+  // table is active, or the 80-col invariant breaks only for ASCII terminals -- the least debuggable
+  // place for it to break.
+  const box = makeStyler(PLAIN_THEME).glyphs;
+  const ascii = makeStyler(PLAIN_THEME, { ascii: true }).glyphs;
+  assert.deepEqual(Object.keys(ascii).sort(), Object.keys(box).sort(), "key parity between the twin tables");
+  for (const key of ["arrowRight", "foldOpen", "foldClosed", "rearm"]) {
+    assert.ok(typeof box[key] === "string" && box[key].length > 0, key);
+    assert.equal(ascii[key].length, box[key].length, `${key}: twin widths must match`);
+  }
+  assert.doesNotMatch(Object.values(ascii).join(""), /[─│┌┐├┤└┘▾▸↻▶]/, "no non-ASCII glyph leaks into the ascii table");
+});
+
 test("styler.sparkline under PLAIN_THEME is byte-identical to the plain sparkline", () => {
   const s = makeStyler(PLAIN_THEME);
   const values = [0, 1, null, 4, 2];