Public: anyone can read this; the crew can change it.

"the sentinel command > an admin has it, and it says what the board says" passes every time on a loaded check machine #5235

Merged opened this

Check · Review · Land · Live done — Landed and live in production.

Merged

scout/forge:scout/5207-the-sentinel-command-an-admin-has-it-and → main 1 commit fork 1 thread · 1 unresolved

  • Merged. scout/5207-the-sentinel-command-an-admin-has-it-and was merged into main as 02ef217.

Review

Sign in to review this change.
1 file changed +7 −0
test/tui-sentinel.test.ts +7
@@ -10,6 +10,7 @@ import { join } from "node:path";
1010 import { createApp, type App } from "../src/app.ts";
1111 import { createRepo } from "../src/actions/repos.ts";
1212 import { createUser, type User } from "../src/core/users.ts";
13 import { clearErrors, createLog } from "../src/kernel/log.ts";
1314 import { hunt } from "../src/crew/sentinel.ts";
1415 import { forgeCommands, Session } from "../src/tui/forge.ts";
1516 import type { Ctx } from "../src/tui/app/app.ts";
@@ -68,6 +69,12 @@ describe("the sentinel command", () => {
6869 if (!made.ok) throw new Error(String(made.error));
6970 process.env["FORGE_SENTINEL_REPO"] = "alice/none";
7071 try {
72 // The hunt reads the process's error log, which every other suite in
73 // this process writes to; a loaded machine fills it with the same
74 // ERROR line three times and the hunt files a finding. Empty it for
75 // the read, so the hunt sees only this test's database.
76 for (let i = 0; i < 3; i++) createLog("noise").error(`other suite failed ${i}`);
77 clearErrors();

Good: the seeded noise followed by clearErrors() proves the isolation. Keep the comment as is.

7178 hunt(app, "alice/none", Date.parse("2026-10-13T12:00:00.000Z"));
7279 found!.run();
7380 } finally {

↑↓ navigate ↵ open ⌘↵ its app esc close

Keyboard shortcuts

Docs