gentle-pi 3.2.0 → 3.3.0
This diff represents the content of publicly available package versions that have been released to one of the supported registries. The information contained in this diff is provided for informational purposes only and reflects changes between package versions as they appear in their respective public registries.
- package/assets/orchestrator-delegation.md +13 -8
- package/assets/orchestrator.md +2 -2
- package/docs/gentle-shell.md +40 -17
- package/docs/readme-reference.md +41 -7
- package/docs/review-integration.md +25 -11
- package/extensions/gentle-agents.ts +85 -17
- package/extensions/gentle-ai.ts +179 -12
- package/extensions/gentle-shell.ts +408 -38
- package/extensions/gentle-todo.ts +19 -1
- package/lib/agents-view.ts +41 -14
- package/lib/agents-widget.ts +84 -13
- package/lib/command-palette-catalog.ts +1 -0
- package/lib/double-esc-cancel-policy.ts +138 -0
- package/lib/inprocess-reviewer.ts +260 -0
- package/lib/model-routing-authority.ts +1 -1
- package/lib/native-review-cli.ts +23 -0
- package/lib/odd-runtime-delegation-gate.ts +88 -0
- package/lib/review-host-relay.ts +262 -94
- package/lib/review-integration-v2.ts +110 -26
- package/lib/shell-bar.ts +158 -29
- package/lib/shell-card.ts +19 -9
- package/lib/shell-changes-view.ts +43 -5
- package/lib/shell-changes.ts +92 -5
- package/lib/shell-hover.ts +39 -0
- package/lib/shell-prompt.ts +10 -1
- package/lib/shell-sidebar-layout.ts +111 -15
- package/lib/shell-sidebar.ts +16 -0
- package/lib/shell-todo.ts +7 -1
- package/lib/shell-usage-view.ts +98 -10
- package/lib/shell-usage.ts +226 -10
- package/package.json +2 -1
- package/runtime/native-review-cli.mjs +23 -0
- package/runtime/review-integration-v2.mjs +110 -26
- package/scripts/gentle-ai-installer.mjs +10 -10
- package/scripts/maintainer/provider-relay-matrix.mjs +118 -47
- package/scripts/mirror-odd-routing.mjs +242 -0
- package/scripts/verify-package-files.mjs +3 -3
- package/tests/agents-grouping.test.ts +75 -18
- package/tests/agents-view.test.ts +28 -18
- package/tests/agents-widget.test.ts +100 -12
- package/tests/command-palette.test.ts +1 -0
- package/tests/devbinary/pi-host-relay.devtest.ts +176 -138
- package/tests/double-esc-cancel-policy.test.ts +194 -0
- package/tests/gentle-agents.test.ts +528 -5
- package/tests/gentle-ai-binary.test.ts +1 -1
- package/tests/gentle-ai-installer.test.ts +47 -47
- package/tests/gentle-ai.test.ts +69 -5
- package/tests/gentle-shell.test.ts +903 -25
- package/tests/gentle-todo.test.ts +17 -4
- package/tests/inprocess-reviewer.test.ts +368 -0
- package/tests/maintainer/provider-relay.maintest.ts +101 -143
- package/tests/native-review-capability-contract.test.ts +32 -1
- package/tests/odd-routing-canonical-ratchet.test.ts +293 -0
- package/tests/odd-routing-contract.test.ts +57 -0
- package/tests/odd-runtime-delegation-gate.test.ts +212 -0
- package/tests/orchestrator-rdd-ownership.test.ts +3 -3
- package/tests/package-manifest.test.ts +6 -6
- package/tests/review-controller-native-routing.test.ts +60 -1
- package/tests/review-host-relay-routing.test.ts +77 -0
- package/tests/review-host-relay.test.ts +285 -239
- package/tests/review-integration-v2-forward.test.ts +61 -0
- package/tests/review-integration-v2.test.ts +116 -1
- package/tests/review-relay-transport-agent.test.ts +83 -0
- package/tests/runtime-harness.mjs +11 -0
- package/tests/session-changes-shell.test.ts +27 -0
- package/tests/session-worktree-registry.test.ts +41 -0
- package/tests/shell-bar.test.ts +224 -6
- package/tests/shell-card.test.ts +5 -3
- package/tests/shell-changes-view.test.ts +47 -0
- package/tests/shell-changes.test.ts +177 -0
- package/tests/shell-hover.test.ts +19 -0
- package/tests/shell-prompt.test.ts +20 -0
- package/tests/shell-sidebar-fullscreen.test.ts +59 -0
- package/tests/shell-sidebar-layout.test.ts +243 -5
- package/tests/shell-sidebar.test.ts +25 -1
- package/tests/shell-todo.test.ts +36 -0
- package/tests/shell-usage-view.test.ts +123 -3
- package/tests/shell-usage.test.ts +254 -6
- package/lib/opaque-pi-reviewer-adapter.ts +0 -284
- package/tests/opaque-pi-reviewer-adapter.test.ts +0 -266
|
@@ -0,0 +1,59 @@
|
|
|
1
|
+
import assert from "node:assert/strict";
|
|
2
|
+
import test from "node:test";
|
|
3
|
+
import { Container, Spacer, TuiAltScreen, type Component } from "@earendil-works/pi-tui";
|
|
4
|
+
import { createChatViewport } from "../node_modules/@earendil-works/pi-coding-agent/dist/modes/interactive/chat-viewport.js";
|
|
5
|
+
import { installSidebar } from "../lib/shell-sidebar-layout.ts";
|
|
6
|
+
import { sidebarHeader, sidebarPart } from "../lib/shell-sidebar.ts";
|
|
7
|
+
|
|
8
|
+
// End-to-end through pi-tui's real alt-screen renderer and Pi's real chat
|
|
9
|
+
// viewport: the unit fixtures elsewhere hand-build layout nodes, which is how
|
|
10
|
+
// two earlier attempts at this row passed their tests and still failed live.
|
|
11
|
+
const theme = { fg: (_color: string, text: string) => text, bold: (text: string) => text };
|
|
12
|
+
const strip = (text: string) => text.replace(/\x1b\[[0-9;?]*[A-Za-z]/g, "").replace(/\x1b\][^\x07]*\x07/g, "").replace(/\s+$/, "");
|
|
13
|
+
const lines = (rows: string[]): Component => ({ render: () => rows, invalidate() {} });
|
|
14
|
+
|
|
15
|
+
async function renderViewport(columns: number, rows: number, sidebar: boolean): Promise<string[]> {
|
|
16
|
+
const terminal = {
|
|
17
|
+
start() {}, stop() {}, async drainInput() {}, write() {},
|
|
18
|
+
get columns() { return columns; }, get rows() { return rows; },
|
|
19
|
+
moveBy() {}, hideCursor() {}, showCursor() {}, clearLine() {}, clearFromCursor() {}, clearScreen() {}, setTitle() {}, setProgress() {},
|
|
20
|
+
kittyProtocolActive: false, getBufferedInput() { return ""; }, hasPendingInput() { return false; },
|
|
21
|
+
} as never;
|
|
22
|
+
const above = new Container();
|
|
23
|
+
above.addChild(new Spacer(1)); // Pi adds a spacer above the editor when no widget is shown
|
|
24
|
+
const footer = new Container();
|
|
25
|
+
const viewport = createChatViewport({
|
|
26
|
+
document: lines(Array.from({ length: 12 }, (_, i) => `transcript ${i + 1}`)),
|
|
27
|
+
pendingMessages: new Container(), status: new Container(), widgetsAbove: above,
|
|
28
|
+
editor: lines(["╭─ editor ─╮", "│ prompt │", "╰──────────╯"]),
|
|
29
|
+
widgetsBelow: new Container(), footer, scrollbar: "auto",
|
|
30
|
+
});
|
|
31
|
+
const tui = new TuiAltScreen(terminal, false);
|
|
32
|
+
tui.setLayoutRoot(viewport.root);
|
|
33
|
+
if (sidebar) {
|
|
34
|
+
footer.addChild(sidebarPart(tui, "footer", lines(["BOTTOM BAR"]), { render: () => ["Status card"], invalidate() {} }));
|
|
35
|
+
sidebarHeader(tui, { render: (width: number) => [`HEADER ${width}`], invalidate() {} });
|
|
36
|
+
installSidebar(tui, theme);
|
|
37
|
+
} else {
|
|
38
|
+
footer.addChild(lines(["BOTTOM BAR"]));
|
|
39
|
+
}
|
|
40
|
+
tui.start();
|
|
41
|
+
tui.requestRender(true);
|
|
42
|
+
await new Promise((resolve) => setTimeout(resolve, 20));
|
|
43
|
+
tui.stop();
|
|
44
|
+
return ((tui as unknown as { previousScreen: string[] }).previousScreen ?? []).map(strip);
|
|
45
|
+
}
|
|
46
|
+
|
|
47
|
+
test("fullscreen with the sidebar leaves no blank row between the editor and the terminal edge", async () => {
|
|
48
|
+
const screen = await renderViewport(140, 24, true);
|
|
49
|
+
assert.equal(screen[0], "HEADER 138", "the header owns the first row");
|
|
50
|
+
assert.match(screen[screen.length - 1] ?? "", /^╰/, `the editor's bottom border sits on the last row, got ${JSON.stringify(screen.slice(-3))}`);
|
|
51
|
+
assert.ok(screen.some((row) => row.startsWith("╭─ editor")), "the editor is still painted");
|
|
52
|
+
assert.ok(screen.some((row) => row.includes("Status card")), "the rail is still painted");
|
|
53
|
+
});
|
|
54
|
+
|
|
55
|
+
test("fullscreen without the sidebar keeps Pi's bottom bar on the last row", async () => {
|
|
56
|
+
const screen = await renderViewport(140, 24, false);
|
|
57
|
+
assert.equal(screen[screen.length - 1], "BOTTOM BAR");
|
|
58
|
+
assert.match(screen[screen.length - 2] ?? "", /^╰/);
|
|
59
|
+
});
|
|
@@ -1,9 +1,9 @@
|
|
|
1
1
|
import assert from "node:assert/strict";
|
|
2
2
|
import test from "node:test";
|
|
3
|
-
import { CURSOR_MARKER, ScrollView, VStack, visibleWidth, type TUI, type TuiMouseEvent } from "@earendil-works/pi-tui";
|
|
3
|
+
import { CURSOR_MARKER, ScrollView, VStack, visibleWidth, type Component, type TUI, type TuiMouseEvent } from "@earendil-works/pi-tui";
|
|
4
4
|
import { getScrollViewsAt, renderLayoutFrame, type LayoutBox } from "@earendil-works/pi-tui/dist/layout.js";
|
|
5
5
|
import { installSidebar, invalidateSidebar } from "../lib/shell-sidebar-layout.ts";
|
|
6
|
-
import { sidebarPart, sidebarState } from "../lib/shell-sidebar.ts";
|
|
6
|
+
import { sidebarHeader, sidebarPart, sidebarState } from "../lib/shell-sidebar.ts";
|
|
7
7
|
import { renderShellSidebarBar } from "../lib/shell-bar.ts";
|
|
8
8
|
import { renderTodoCard, type TodoState } from "../lib/shell-todo.ts";
|
|
9
9
|
|
|
@@ -23,6 +23,20 @@ function rail(f: ReturnType<typeof fixture>): ScrollView {
|
|
|
23
23
|
assert.equal(node.type, "hstack");
|
|
24
24
|
return node.entries[1].component;
|
|
25
25
|
}
|
|
26
|
+
type HstackNode = { type: string; gap: number; align: string; entries: { component: unknown; basis: number; grow: number; shrink: number; minSize: number }[] };
|
|
27
|
+
// Finds the [left, scroll] hstack regardless of whether it is returned
|
|
28
|
+
// directly (no active header) or nested one level under the header vstack.
|
|
29
|
+
function hstackOf(f: ReturnType<typeof fixture>): HstackNode {
|
|
30
|
+
const node = f.root[NODE]() as unknown as { type: string; entries: { component: { [NODE]?(): HstackNode } }[] };
|
|
31
|
+
if (node.type === "hstack") return node as unknown as HstackNode;
|
|
32
|
+
assert.equal(node.type, "vstack");
|
|
33
|
+
const nested = node.entries[1]?.component[NODE]?.();
|
|
34
|
+
assert.ok(nested, "the vstack's second entry must expose the hstack via NODE");
|
|
35
|
+
return nested!;
|
|
36
|
+
}
|
|
37
|
+
function railWithHeader(f: ReturnType<typeof fixture>): ScrollView {
|
|
38
|
+
return hstackOf(f).entries[1].component as ScrollView;
|
|
39
|
+
}
|
|
26
40
|
|
|
27
41
|
test("grouped Status preserves structured fields and opaque integration text", () => {
|
|
28
42
|
const lines = renderShellSidebarBar({
|
|
@@ -32,13 +46,14 @@ test("grouped Status preserves structured fields and opaque integration text", (
|
|
|
32
46
|
}, theme, 46);
|
|
33
47
|
const text = lines.join("\n");
|
|
34
48
|
let previous = -1;
|
|
35
|
-
for (const heading of ["Status", "Project", "Changes", "
|
|
49
|
+
for (const heading of ["Status", "Project", "Changes", "Integrations"]) {
|
|
36
50
|
const index = text.indexOf(heading);
|
|
37
51
|
assert.ok(index > previous, heading);
|
|
38
52
|
previous = index;
|
|
39
53
|
}
|
|
40
54
|
assert.match(text, /opaque integration/);
|
|
41
55
|
assert.match(text, /Branch.*main/);
|
|
56
|
+
assert.doesNotMatch(text, /Usage/);
|
|
42
57
|
});
|
|
43
58
|
|
|
44
59
|
test("scrollable TODO keeps every task while bottom and collapsed cards stay bounded", () => {
|
|
@@ -265,14 +280,17 @@ test("real layout frames reuse unchanged sidebar output and invalidate at state
|
|
|
265
280
|
assert.deepEqual(f.bottom.render(80), ["Status"]);
|
|
266
281
|
f.host.terminal.columns = 140;
|
|
267
282
|
renderLayoutFrame(f.root, 140, 20, () => {});
|
|
268
|
-
|
|
283
|
+
// Dipping below the breakpoint and back nulls the whole-rail `prepared`
|
|
284
|
+
// memo, but the per-section cache survives underneath it: the revision
|
|
285
|
+
// never moved, so no section actually needed to re-render.
|
|
286
|
+
assert.deepEqual(counts, { footer: 2, changes: 0, agents: 2, todo: 2 });
|
|
269
287
|
|
|
270
288
|
f.host.mode = "regular";
|
|
271
289
|
renderLayoutFrame(f.root, 140, 20, () => {});
|
|
272
290
|
assert.deepEqual(f.bottom.render(80), ["Status"]);
|
|
273
291
|
f.host.mode = "fullscreen";
|
|
274
292
|
renderLayoutFrame(f.root, 140, 20, () => {});
|
|
275
|
-
assert.deepEqual(counts, { footer:
|
|
293
|
+
assert.deepEqual(counts, { footer: 2, changes: 0, agents: 2, todo: 2 });
|
|
276
294
|
|
|
277
295
|
let replacementRenders = 0;
|
|
278
296
|
sidebarPart(f.tui, "todo", {
|
|
@@ -469,3 +487,223 @@ test("native transcript and sidebar keep separate scroll routing across resize,
|
|
|
469
487
|
assert.equal(result.frame.root.children.some(box => box.component === rail), mode === "fullscreen" && width >= 140);
|
|
470
488
|
}
|
|
471
489
|
});
|
|
490
|
+
|
|
491
|
+
// T3: per-section render memo. Each rail section caches its rendered lines by
|
|
492
|
+
// its own digest, so a section whose digest did not change is never re-run
|
|
493
|
+
// when a sibling section's digest ticks — only the whole rail's assembled
|
|
494
|
+
// `lines`/`hits` are rebuilt from the (possibly cached) per-section lines.
|
|
495
|
+
|
|
496
|
+
test("a footer digest change does not re-render Agents or TODO", (t) => {
|
|
497
|
+
const f = fixture();
|
|
498
|
+
let footerLabel = "one";
|
|
499
|
+
const counts = { footer: 0, agents: 0, todo: 0 };
|
|
500
|
+
sidebarPart(f.tui, "footer", { render: () => ["Status"], invalidate() {} }, {
|
|
501
|
+
digest: () => footerLabel,
|
|
502
|
+
render: () => { counts.footer++; return [`Footer ${footerLabel}`]; },
|
|
503
|
+
invalidate() {},
|
|
504
|
+
});
|
|
505
|
+
for (const key of ["agents", "todo"] as const) {
|
|
506
|
+
sidebarPart(f.tui, key, { render: () => [key], invalidate() {} }, {
|
|
507
|
+
render: () => { counts[key]++; return [key]; },
|
|
508
|
+
invalidate() {},
|
|
509
|
+
});
|
|
510
|
+
}
|
|
511
|
+
t.after(installSidebar(f.tui, theme));
|
|
512
|
+
|
|
513
|
+
assert.match(rail(f).render(50).join("\n"), /Footer one/);
|
|
514
|
+
assert.deepEqual(counts, { footer: 1, agents: 1, todo: 1 });
|
|
515
|
+
|
|
516
|
+
// Re-render with nothing changed at all: the outer unchanged fast path
|
|
517
|
+
// must not call any section's render either.
|
|
518
|
+
assert.match(rail(f).render(50).join("\n"), /Footer one/);
|
|
519
|
+
assert.deepEqual(counts, { footer: 1, agents: 1, todo: 1 });
|
|
520
|
+
|
|
521
|
+
footerLabel = "two";
|
|
522
|
+
assert.match(rail(f).render(50).join("\n"), /Footer two/);
|
|
523
|
+
assert.deepEqual(counts, { footer: 2, agents: 1, todo: 1 }, "only the section whose digest changed re-renders");
|
|
524
|
+
|
|
525
|
+
// invalidateSidebar bumps the global revision, which is only what a
|
|
526
|
+
// no-digest section keys its cache on: footer's own unchanged digest
|
|
527
|
+
// still protects it from re-rendering.
|
|
528
|
+
invalidateSidebar(f.tui);
|
|
529
|
+
assert.match(rail(f).render(50).join("\n"), /Footer two/);
|
|
530
|
+
assert.deepEqual(counts, { footer: 2, agents: 2, todo: 2 }, "revision invalidation reaches only the sections with no digest of their own");
|
|
531
|
+
});
|
|
532
|
+
|
|
533
|
+
test("a throwing digest still degrades to invalidation-only under the per-section memo", (t) => {
|
|
534
|
+
const f = fixture();
|
|
535
|
+
let todoRenders = 0;
|
|
536
|
+
sidebarPart(f.tui, "todo", { render: () => ["Todo"], invalidate() {} }, {
|
|
537
|
+
digest: () => { throw new Error("broken digest"); },
|
|
538
|
+
render: () => { todoRenders++; return ["Todo card"]; },
|
|
539
|
+
invalidate() {},
|
|
540
|
+
});
|
|
541
|
+
t.after(installSidebar(f.tui, theme));
|
|
542
|
+
|
|
543
|
+
assert.match(rail(f).render(50).join("\n"), /Todo card/);
|
|
544
|
+
assert.equal(todoRenders, 1);
|
|
545
|
+
assert.match(rail(f).render(50).join("\n"), /Todo card/);
|
|
546
|
+
assert.equal(todoRenders, 1, "an unchanged frame reuses the cached section even without a digest");
|
|
547
|
+
|
|
548
|
+
invalidateSidebar(f.tui);
|
|
549
|
+
assert.match(rail(f).render(50).join("\n"), /Todo card/);
|
|
550
|
+
assert.equal(todoRenders, 2, "explicit invalidation still reaches a rail without a digest");
|
|
551
|
+
});
|
|
552
|
+
|
|
553
|
+
test("a part replaced under the same key never reuses the previous part's cached lines", (t) => {
|
|
554
|
+
const f = fixture();
|
|
555
|
+
let firstRenders = 0;
|
|
556
|
+
sidebarPart(f.tui, "todo", { render: () => ["Todo"], invalidate() {} }, {
|
|
557
|
+
render: () => { firstRenders++; return ["first"]; },
|
|
558
|
+
invalidate() {},
|
|
559
|
+
});
|
|
560
|
+
t.after(installSidebar(f.tui, theme));
|
|
561
|
+
assert.match(rail(f).render(50).join("\n"), /first/);
|
|
562
|
+
assert.equal(firstRenders, 1);
|
|
563
|
+
|
|
564
|
+
let secondRenders = 0;
|
|
565
|
+
sidebarPart(f.tui, "todo", { render: () => ["Todo"], invalidate() {} }, {
|
|
566
|
+
render: () => { secondRenders++; return ["second"]; },
|
|
567
|
+
invalidate() {},
|
|
568
|
+
});
|
|
569
|
+
const text = rail(f).render(50).join("\n");
|
|
570
|
+
assert.match(text, /second/);
|
|
571
|
+
assert.doesNotMatch(text, /first/);
|
|
572
|
+
assert.equal(secondRenders, 1);
|
|
573
|
+
});
|
|
574
|
+
|
|
575
|
+
// T2: the live header row. A registered "header" part wraps the existing
|
|
576
|
+
// [left, scroll] hstack in a one-row-taller vstack and takes over the brand
|
|
577
|
+
// the banner used to carry inside the rail. Hosts that never register a
|
|
578
|
+
// header (every fixture above) keep the exact old hstack-direct shape.
|
|
579
|
+
|
|
580
|
+
test("an active header wraps the hstack in a vstack and removes the banner from the rail", (t) => {
|
|
581
|
+
const f = fixture();
|
|
582
|
+
sidebarHeader(f.tui, { render: (width: number) => [`HEADER ${width}`], invalidate() {} });
|
|
583
|
+
t.after(installSidebar(f.tui, theme));
|
|
584
|
+
|
|
585
|
+
const node = f.root[NODE]() as unknown as HstackNode;
|
|
586
|
+
assert.equal(node.type, "vstack");
|
|
587
|
+
assert.equal(node.gap, 0);
|
|
588
|
+
assert.equal(node.align, "stretch");
|
|
589
|
+
assert.deepEqual(node.entries.map(({ basis, grow, shrink, minSize }) => ({ basis, grow, shrink, minSize })), [
|
|
590
|
+
{ basis: 1, grow: 0, shrink: 0, minSize: 1 },
|
|
591
|
+
{ basis: 0, grow: 1, shrink: 1, minSize: 1 },
|
|
592
|
+
]);
|
|
593
|
+
const header = node.entries[0].component as { render(width: number): string[] };
|
|
594
|
+
// The rail card ends two columns before the terminal edge (its padding plus
|
|
595
|
+
// the scrollbar column); the header stops there too so its right group
|
|
596
|
+
// lines up with the card border instead of touching the edge.
|
|
597
|
+
assert.deepEqual(header.render(0), ["HEADER 138"], "the header renders at the terminal width minus the rail's right inset, not the rail width");
|
|
598
|
+
|
|
599
|
+
const hstack = hstackOf(f);
|
|
600
|
+
assert.equal(hstack.type, "hstack");
|
|
601
|
+
const scroll = railWithHeader(f);
|
|
602
|
+
const rail = scroll.render(50);
|
|
603
|
+
assert.doesNotMatch(rail.join("\n"), /✿ Gentle Shell ✿/, "the header carries the brand now, not the banner");
|
|
604
|
+
// The banner used to hold the first card away from the top; with the
|
|
605
|
+
// header in its place the rail keeps one blank row so the first card does
|
|
606
|
+
// not sit flush against the header.
|
|
607
|
+
assert.equal(rail[0]?.trim(), "", "the rail opens with a blank row under the header");
|
|
608
|
+
assert.notEqual(rail[1]?.trim(), "", "the first card starts on the second row");
|
|
609
|
+
});
|
|
610
|
+
|
|
611
|
+
test("without a registered header the rail keeps the banner and the plain hstack", (t) => {
|
|
612
|
+
const f = fixture();
|
|
613
|
+
t.after(installSidebar(f.tui, theme));
|
|
614
|
+
const node = f.root[NODE]() as unknown as { type: string };
|
|
615
|
+
assert.equal(node.type, "hstack");
|
|
616
|
+
const scroll = rail(f);
|
|
617
|
+
assert.match(scroll.render(50).join("\n"), /✿ Gentle Shell ✿/);
|
|
618
|
+
});
|
|
619
|
+
|
|
620
|
+
test("a header too narrow to show anything falls back to the plain hstack and the banner", (t) => {
|
|
621
|
+
const f = fixture();
|
|
622
|
+
sidebarHeader(f.tui, { render: () => [""], invalidate() {} });
|
|
623
|
+
t.after(installSidebar(f.tui, theme));
|
|
624
|
+
const node = f.root[NODE]() as unknown as { type: string };
|
|
625
|
+
assert.equal(node.type, "hstack", "a blank header line does not earn its own row");
|
|
626
|
+
const scroll = rail(f);
|
|
627
|
+
assert.match(scroll.render(50).join("\n"), /✿ Gentle Shell ✿/);
|
|
628
|
+
});
|
|
629
|
+
|
|
630
|
+
test("header content changes reach the header row while an unrelated part is untouched", (t) => {
|
|
631
|
+
const f = fixture();
|
|
632
|
+
let label = "one";
|
|
633
|
+
sidebarHeader(f.tui, { digest: () => label, render: () => [`HEADER ${label}`], invalidate() {} });
|
|
634
|
+
t.after(installSidebar(f.tui, theme));
|
|
635
|
+
|
|
636
|
+
const headerLines = () => (f.root[NODE]() as unknown as HstackNode).entries[0].component as { render(width: number): string[] };
|
|
637
|
+
assert.deepEqual(headerLines().render(0), ["HEADER one"]);
|
|
638
|
+
label = "two";
|
|
639
|
+
assert.deepEqual(headerLines().render(0), ["HEADER two"]);
|
|
640
|
+
});
|
|
641
|
+
|
|
642
|
+
test("rail mouse dispatch still maps clicks correctly with the header row above it", (t) => {
|
|
643
|
+
const f = fixture();
|
|
644
|
+
let clicks = 0;
|
|
645
|
+
const todo = {
|
|
646
|
+
render: () => ["Todo header", "Todo body"],
|
|
647
|
+
invalidate() {},
|
|
648
|
+
handleMouse(event: TuiMouseEvent) {
|
|
649
|
+
if (event.type !== "click" || event.button !== "left" || event.y !== 0) return undefined;
|
|
650
|
+
clicks++;
|
|
651
|
+
return { handled: true, render: true };
|
|
652
|
+
},
|
|
653
|
+
};
|
|
654
|
+
sidebarPart(f.tui, "todo", todo);
|
|
655
|
+
sidebarHeader(f.tui, { render: () => ["HEADER"], invalidate() {} });
|
|
656
|
+
const dispose = installSidebar(f.tui, theme);
|
|
657
|
+
t.after(dispose);
|
|
658
|
+
const scroll = railWithHeader(f);
|
|
659
|
+
const content = scroll.render(50);
|
|
660
|
+
scroll.updateLayout(content.length, 5, () => {});
|
|
661
|
+
const headerY = content.findIndex((line) => line.includes("Todo header"));
|
|
662
|
+
assert.ok(headerY >= 0);
|
|
663
|
+
// The mouse event carries coordinates local to the scroll component itself
|
|
664
|
+
// (pi-tui's layout tree translates screen coordinates before dispatch), so
|
|
665
|
+
// nesting the hstack one level deeper under the header row does not shift
|
|
666
|
+
// what the rail receives.
|
|
667
|
+
const hit = scroll.handleMouse({ type: "click", button: "left", x: 2, y: headerY, screenX: 92, screenY: 31 + headerY, width: 50, height: 5, shift: false, alt: false, ctrl: false });
|
|
668
|
+
assert.equal(hit?.handled, true);
|
|
669
|
+
assert.equal(clicks, 1);
|
|
670
|
+
});
|
|
671
|
+
|
|
672
|
+
// T8(c): the header row is a leaf in the layout tree, not inside the rail's
|
|
673
|
+
// ScrollView, so pi-tui's mouse dispatch (tui-alt-screen.js) finds and calls
|
|
674
|
+
// its own handleMouse directly — it never goes through dispatchPartMouse.
|
|
675
|
+
test("the header component's handleMouse delegates to the registered header part", (t) => {
|
|
676
|
+
const f = fixture();
|
|
677
|
+
const received: TuiMouseEvent[] = [];
|
|
678
|
+
let result: { handled: boolean; render?: boolean } | undefined = { handled: true, render: true };
|
|
679
|
+
sidebarHeader(f.tui, {
|
|
680
|
+
render: () => ["HEADER"],
|
|
681
|
+
invalidate() {},
|
|
682
|
+
handleMouse(event: TuiMouseEvent) {
|
|
683
|
+
received.push(event);
|
|
684
|
+
return result;
|
|
685
|
+
},
|
|
686
|
+
});
|
|
687
|
+
t.after(installSidebar(f.tui, theme));
|
|
688
|
+
|
|
689
|
+
const node = f.root[NODE]() as unknown as { type: string; entries: { component: Component }[] };
|
|
690
|
+
assert.equal(node.type, "vstack", "an active header wraps the hstack in a vstack");
|
|
691
|
+
const header = node.entries[0]!.component as { handleMouse(event: TuiMouseEvent): { handled: boolean; render?: boolean } | undefined };
|
|
692
|
+
|
|
693
|
+
const event = { type: "click", button: "left", x: 5, y: 0, screenX: 5, screenY: 0, width: 140, height: 1, shift: false, alt: false, ctrl: false } as TuiMouseEvent;
|
|
694
|
+
assert.deepEqual(header.handleMouse(event), { handled: true, render: true });
|
|
695
|
+
assert.equal(received.length, 1);
|
|
696
|
+
assert.equal(received[0], event, "the event reaches the registered part unmodified");
|
|
697
|
+
|
|
698
|
+
result = undefined;
|
|
699
|
+
assert.equal(header.handleMouse(event), undefined, "an ignored click returns undefined, same as any other component");
|
|
700
|
+
});
|
|
701
|
+
|
|
702
|
+
test("the header component's handleMouse is a harmless no-op when no header part is registered", (t) => {
|
|
703
|
+
const f = fixture();
|
|
704
|
+
t.after(installSidebar(f.tui, theme));
|
|
705
|
+
const node = f.root[NODE]() as unknown as { type: string; entries?: { component: Component }[] };
|
|
706
|
+
// No header registered: the active node stays the plain hstack (T2's
|
|
707
|
+
// backward-compatible fallback), so there is no header leaf to click at all.
|
|
708
|
+
assert.equal(node.type, "hstack");
|
|
709
|
+
});
|
|
@@ -1,7 +1,7 @@
|
|
|
1
1
|
import assert from "node:assert/strict";
|
|
2
2
|
import test from "node:test";
|
|
3
3
|
import type { TUI } from "@earendil-works/pi-tui";
|
|
4
|
-
import { sidebarPart, sidebarState } from "../lib/shell-sidebar.ts";
|
|
4
|
+
import { sidebarHeader, sidebarPart, sidebarState } from "../lib/shell-sidebar.ts";
|
|
5
5
|
|
|
6
6
|
const host = (terminal?: object) => ({ terminal }) as TUI;
|
|
7
7
|
const component = () => ({ render: (_width = 80) => ["bottom"], invalidate() {} });
|
|
@@ -48,3 +48,27 @@ test("disposing an old part preserves its replacement and releases its bottom",
|
|
|
48
48
|
(second as typeof second & { dispose?(): void }).dispose?.();
|
|
49
49
|
assert.equal(sidebarState(tui).parts.size, 0);
|
|
50
50
|
});
|
|
51
|
+
|
|
52
|
+
test("sidebarHeader registers the header part under its own key and disposes it without a bottom widget", () => {
|
|
53
|
+
const tui = host({});
|
|
54
|
+
const rail = { render: () => ["header line"], invalidate() {}, digest: () => "d" };
|
|
55
|
+
const dispose = sidebarHeader(tui, rail);
|
|
56
|
+
assert.equal(sidebarState(tui).parts.get("header"), rail);
|
|
57
|
+
dispose();
|
|
58
|
+
assert.equal(sidebarState(tui).parts.get("header"), undefined);
|
|
59
|
+
});
|
|
60
|
+
|
|
61
|
+
test("sidebarHeader on an unsupported host is a harmless no-op", () => {
|
|
62
|
+
const dispose = sidebarHeader(host(), { render: () => [], invalidate() {} });
|
|
63
|
+
assert.doesNotThrow(dispose);
|
|
64
|
+
});
|
|
65
|
+
|
|
66
|
+
test("sidebarHeader dispose does not remove a replacement rail registered after it", () => {
|
|
67
|
+
const tui = host({});
|
|
68
|
+
const first = { render: () => ["first"], invalidate() {} };
|
|
69
|
+
const dispose = sidebarHeader(tui, first);
|
|
70
|
+
const second = { render: () => ["second"], invalidate() {} };
|
|
71
|
+
sidebarHeader(tui, second);
|
|
72
|
+
dispose();
|
|
73
|
+
assert.equal(sidebarState(tui).parts.get("header"), second);
|
|
74
|
+
});
|
package/tests/shell-todo.test.ts
CHANGED
|
@@ -231,6 +231,42 @@ test("completed titles strike only title cells across wrapped lines, never paddi
|
|
|
231
231
|
}
|
|
232
232
|
});
|
|
233
233
|
|
|
234
|
+
test("renderTodoCard paints the sidebar rail and the bottom widget with the same rose INFO frame", () => {
|
|
235
|
+
const taggedTheme = {
|
|
236
|
+
fg(color: string, text: string) {
|
|
237
|
+
return `<${color}>${text}</${color}>`;
|
|
238
|
+
},
|
|
239
|
+
strikethrough(text: string) {
|
|
240
|
+
return text;
|
|
241
|
+
},
|
|
242
|
+
};
|
|
243
|
+
const sidebar = renderTodoCard(seeded(), taggedTheme, 60, { scrollable: true, collapsed: false, staleTurns: 0 });
|
|
244
|
+
assert.match(sidebar[0], /<border>╭<\/border>/);
|
|
245
|
+
assert.match(sidebar[0], /<accent>❀ Todos/);
|
|
246
|
+
|
|
247
|
+
const bottom = renderTodoCard(seeded(), taggedTheme, 60, { collapsed: false, staleTurns: 0 });
|
|
248
|
+
assert.match(bottom[0], /<border>╭<\/border>/);
|
|
249
|
+
assert.match(bottom[0], /<accent>❀ Todos/);
|
|
250
|
+
});
|
|
251
|
+
|
|
252
|
+
// H1 (odd/tasks/usage-click-and-changes-attribution.md): the same shared
|
|
253
|
+
// hover role every other clickable surface uses.
|
|
254
|
+
test("renderTodoCard paints its clickable header control in the shared hover role when hovered", () => {
|
|
255
|
+
const taggedTheme = {
|
|
256
|
+
fg(color: string, text: string) {
|
|
257
|
+
return `<${color}>${text}</${color}>`;
|
|
258
|
+
},
|
|
259
|
+
strikethrough(text: string) {
|
|
260
|
+
return text;
|
|
261
|
+
},
|
|
262
|
+
};
|
|
263
|
+
const idle = renderTodoCard(seeded(), taggedTheme, 70, { collapsed: false, staleTurns: 0 });
|
|
264
|
+
assert.match(idle[0], /<accent>▾ Collapse<\/accent>/, "idle keeps its ordinary accent role");
|
|
265
|
+
const hovered = renderTodoCard(seeded(), taggedTheme, 70, { collapsed: false, staleTurns: 0, hovered: true });
|
|
266
|
+
assert.match(hovered[0], /<warning>▾ Collapse<\/warning>/, "hovering paints the shared hover role");
|
|
267
|
+
assert.doesNotMatch(hovered[0], /<accent>▾ Collapse/);
|
|
268
|
+
});
|
|
269
|
+
|
|
234
270
|
test("renderTodoCard keeps stale indicators and collapse hints in scrollable lists", () => {
|
|
235
271
|
const tasks = Array.from({ length: 40 }, (_, index) => ({ title: `Task ${index + 1}`, status: index < 25 ? "done" : "pending" }));
|
|
236
272
|
const state = applyTodo(emptyTodo(), { action: "write", tasks }, 1).state;
|
|
@@ -1,6 +1,6 @@
|
|
|
1
1
|
import assert from "node:assert/strict";
|
|
2
2
|
import test from "node:test";
|
|
3
|
-
import { visibleWidth } from "@earendil-works/pi-tui";
|
|
3
|
+
import { visibleWidth, type TuiMouseEvent } from "@earendil-works/pi-tui";
|
|
4
4
|
import { stripAnsi } from "../lib/terminal-theme.ts";
|
|
5
5
|
import { parseCodexUsage, UsageStore } from "../lib/shell-usage.ts";
|
|
6
6
|
import { UsageView } from "../lib/shell-usage-view.ts";
|
|
@@ -34,7 +34,8 @@ test("UsageView frames the panel, keeps every line at width, and shows the empty
|
|
|
34
34
|
for (const line of lines) assert.equal(visibleWidth(line), 90, `"${stripAnsi(line)}" is not 90 wide`);
|
|
35
35
|
const plain = lines.map(stripAnsi);
|
|
36
36
|
assert.match(plain[1], /^│ openai-codex · pro · updated just now +│$/);
|
|
37
|
-
assert.match(plain[
|
|
37
|
+
assert.match(plain[2], /^│ {3}codex week +[▰▱]{16} +40% · resets in 2h 0m +│$/);
|
|
38
|
+
assert.match(plain[3], /r refresh .* esc close/);
|
|
38
39
|
});
|
|
39
40
|
|
|
40
41
|
test("UsageView refetches on r and closes on escape or q", async () => {
|
|
@@ -54,9 +55,128 @@ test("UsageView refetches on r and closes on escape or q", async () => {
|
|
|
54
55
|
view.handleInput("r");
|
|
55
56
|
await new Promise((resolve) => setTimeout(resolve, 0));
|
|
56
57
|
assert.deepEqual(events, ["render", "refresh", "render"]);
|
|
57
|
-
assert.match(stripAnsi(view.render(90)[
|
|
58
|
+
assert.match(stripAnsi(view.render(90)[2]), /55%/);
|
|
58
59
|
assert.match(stripAnsi(view.render(90)[1]), /^│ ✿ openai-codex · pro · updated just now/);
|
|
59
60
|
view.handleInput("\x1b");
|
|
60
61
|
view.handleInput("q");
|
|
61
62
|
assert.equal(events.filter((event) => event === "close").length, 2);
|
|
62
63
|
});
|
|
64
|
+
|
|
65
|
+
function click(x: number, y: number, width: number, height: number): TuiMouseEvent {
|
|
66
|
+
return { type: "click", button: "left", x, y, screenX: x, screenY: y, width, height, shift: false, alt: false, ctrl: false };
|
|
67
|
+
}
|
|
68
|
+
|
|
69
|
+
test("UsageView.handleMouse clicks the footer hints like the matching key", async () => {
|
|
70
|
+
const store = new UsageStore();
|
|
71
|
+
const events: string[] = [];
|
|
72
|
+
let resolveRefresh: (() => void) | undefined;
|
|
73
|
+
const view = new UsageView(store, {
|
|
74
|
+
theme: plainTheme,
|
|
75
|
+
now: () => NOW,
|
|
76
|
+
active: () => undefined,
|
|
77
|
+
onRefresh: () =>
|
|
78
|
+
new Promise<void>((resolve) => {
|
|
79
|
+
events.push("refresh");
|
|
80
|
+
resolveRefresh = resolve;
|
|
81
|
+
}),
|
|
82
|
+
onClose: () => events.push("close"),
|
|
83
|
+
requestRender: () => events.push("render"),
|
|
84
|
+
});
|
|
85
|
+
const width = 90;
|
|
86
|
+
const lines = view.render(width);
|
|
87
|
+
const rowIndex = lines.findIndex((line) => stripAnsi(line).includes("r refresh"));
|
|
88
|
+
assert.ok(rowIndex > 0, "the footer hints row must be present");
|
|
89
|
+
const plain = stripAnsi(lines[rowIndex]);
|
|
90
|
+
const refreshStart = plain.indexOf("r refresh");
|
|
91
|
+
const closeStart = plain.indexOf("esc close");
|
|
92
|
+
assert.ok(refreshStart >= 0 && closeStart >= 0);
|
|
93
|
+
|
|
94
|
+
// A click outside any hint span is ignored, like a click on an empty part of the frame.
|
|
95
|
+
assert.equal(view.handleMouse(click(0, rowIndex, width, lines.length)), undefined);
|
|
96
|
+
assert.deepEqual(events, []);
|
|
97
|
+
|
|
98
|
+
// A click on "r refresh" triggers the same refresh as the r key.
|
|
99
|
+
assert.deepEqual(view.handleMouse(click(refreshStart + 1, rowIndex, width, lines.length)), { handled: true, render: true });
|
|
100
|
+
assert.deepEqual(events, ["render", "refresh"]);
|
|
101
|
+
|
|
102
|
+
// A second click while the refresh is still in flight must not re-enter it.
|
|
103
|
+
assert.deepEqual(view.handleMouse(click(refreshStart + 1, rowIndex, width, lines.length)), { handled: true, render: true });
|
|
104
|
+
assert.deepEqual(events, ["render", "refresh"]);
|
|
105
|
+
resolveRefresh?.();
|
|
106
|
+
await new Promise((resolve) => setTimeout(resolve, 0));
|
|
107
|
+
assert.deepEqual(events, ["render", "refresh", "render"]);
|
|
108
|
+
|
|
109
|
+
// A click on "esc close" closes the overlay, like the esc/q keys.
|
|
110
|
+
assert.deepEqual(view.handleMouse(click(closeStart + 1, rowIndex, width, lines.length)), { handled: true, render: true });
|
|
111
|
+
assert.equal(events.filter((event) => event === "close").length, 1);
|
|
112
|
+
});
|
|
113
|
+
|
|
114
|
+
function move(x: number, y: number, width: number, height: number): TuiMouseEvent {
|
|
115
|
+
return { type: "move", button: "none", x, y, screenX: x, screenY: y, width, height, shift: false, alt: false, ctrl: false };
|
|
116
|
+
}
|
|
117
|
+
|
|
118
|
+
// H1 (odd/tasks/usage-click-and-changes-attribution.md): the same shared
|
|
119
|
+
// hover role every other clickable surface uses.
|
|
120
|
+
test("UsageView.handleMouse paints the shared hover role over a footer hint, and clears it on leave", () => {
|
|
121
|
+
const store = new UsageStore();
|
|
122
|
+
const taggedTheme = { fg: (role: string, text: string) => `<${role}>${text}</${role}>` };
|
|
123
|
+
const view = new UsageView(store, {
|
|
124
|
+
theme: taggedTheme,
|
|
125
|
+
now: () => NOW,
|
|
126
|
+
active: () => undefined,
|
|
127
|
+
onRefresh: async () => {},
|
|
128
|
+
onClose: () => {},
|
|
129
|
+
requestRender: () => {},
|
|
130
|
+
});
|
|
131
|
+
const width = 90;
|
|
132
|
+
const lines = view.render(width);
|
|
133
|
+
// The frame draws "│ " (2 columns) before the fitted footer content, and
|
|
134
|
+
// hints are laid out as plain "key label" text joined by 3 spaces -- these
|
|
135
|
+
// column offsets are computed purely from that plain text, independent of
|
|
136
|
+
// the theme, so they hold under taggedTheme exactly as under plainTheme.
|
|
137
|
+
const rowIndex = lines.length - 2;
|
|
138
|
+
const refreshStart = 2;
|
|
139
|
+
const closeStart = refreshStart + "r refresh".length + 3;
|
|
140
|
+
const height = lines.length;
|
|
141
|
+
|
|
142
|
+
assert.equal(view.handleMouse(move(0, rowIndex, width, height)), undefined, "a move outside any hint is not this component's gesture");
|
|
143
|
+
assert.doesNotMatch(view.render(width).join("\n"), /<warning>/, "idle: nothing painted yet");
|
|
144
|
+
|
|
145
|
+
const entered = view.handleMouse(move(refreshStart + 1, rowIndex, width, height));
|
|
146
|
+
assert.deepEqual(entered, { handled: true, render: true });
|
|
147
|
+
assert.match(view.render(width).join("\n"), /<warning>r refresh<\/warning>/, "hovering the refresh hint paints it, key and label together");
|
|
148
|
+
assert.doesNotMatch(view.render(width).join("\n"), new RegExp(`<warning>esc close`), "the other hint stays unpainted");
|
|
149
|
+
|
|
150
|
+
// Moving straight to the other hint switches which one is painted.
|
|
151
|
+
const switched = view.handleMouse(move(closeStart + 1, rowIndex, width, height));
|
|
152
|
+
assert.deepEqual(switched, { handled: true, render: true });
|
|
153
|
+
assert.match(view.render(width).join("\n"), /<warning>esc close<\/warning>/);
|
|
154
|
+
assert.doesNotMatch(view.render(width).join("\n"), /<warning>r refresh/);
|
|
155
|
+
|
|
156
|
+
const left = view.handleMouse(move(0, rowIndex, width, height));
|
|
157
|
+
assert.deepEqual(left, { handled: true, render: true }, "leaving the last hovered hint still requests a repaint");
|
|
158
|
+
assert.doesNotMatch(view.render(width).join("\n"), /<warning>/, "nothing stays painted once the pointer leaves");
|
|
159
|
+
});
|
|
160
|
+
|
|
161
|
+
test("a rejected refresh from a click or key never escapes as an unhandled rejection", async () => {
|
|
162
|
+
const store = new UsageStore();
|
|
163
|
+
const events: string[] = [];
|
|
164
|
+
const unhandled: unknown[] = [];
|
|
165
|
+
const onUnhandled = (reason: unknown) => unhandled.push(reason);
|
|
166
|
+
process.on("unhandledRejection", onUnhandled);
|
|
167
|
+
try {
|
|
168
|
+
const view = new UsageView(store, { theme: plainTheme, now: () => NOW, active: () => undefined, onRefresh: async () => { throw new Error("provider down"); }, onClose: () => events.push("close"), requestRender: () => events.push("render") });
|
|
169
|
+
const lines = view.render(80);
|
|
170
|
+
const row = lines.length - 2;
|
|
171
|
+
const column = lines[row]!.indexOf("r refresh") + 1;
|
|
172
|
+
view.handleMouse({ type: "click", button: "left", x: column, y: row, width: 80, height: lines.length, screenX: column, screenY: row, shift: false, alt: false, ctrl: false } as never);
|
|
173
|
+
await new Promise((resolve) => setImmediate(resolve));
|
|
174
|
+
view.handleInput("r");
|
|
175
|
+
await new Promise((resolve) => setImmediate(resolve));
|
|
176
|
+
await new Promise((resolve) => setImmediate(resolve));
|
|
177
|
+
assert.deepEqual(unhandled, [], "a failing usage fetch is reported by the panel, never thrown at the process");
|
|
178
|
+
assert.match(view.render(80).join("\n"), /r refresh/, "the refresh hint is back after the failure");
|
|
179
|
+
} finally {
|
|
180
|
+
process.off("unhandledRejection", onUnhandled);
|
|
181
|
+
}
|
|
182
|
+
});
|