Files
gstack/test/gbrain-detect-shape.test.ts
Garry Tan 8241949357 v1.57.9.0 feat: source-clean gbrain render (dev-setup --out-dir + machine-wide gbrain-refresh) (#1951)
* feat(gbrain-detect): add --is-ok live-detection exit-code gate

Single source of truth for 'is gbrain usable'. Runs live detection (never
reads the possibly-stale gbrain-detection.json) and exits 0 iff status is ok,
so setup, bin/dev-setup, and gstack-config can gate brain-aware rendering on
one shared check instead of re-grepping the JSON.

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>

* feat(gen-skill-docs): add --out-dir with surgical section-path rewrite

--out-dir <abs-dir> mirrors the Claude skill tree (SKILL.md + sections) into a
separate directory instead of writing in place, and rewrites the literal
section-base path (~/.claude/skills/gstack/<skill>/sections/) in generated
content to point at the out-dir. The rewrite is surgical: only /sections/ paths
move; bin/, browse/, docs/ references stay pointed at the global install.
Global extras (proactive-suggestions.json) are skipped in out-dir mode. Default
(no flag) behavior is unchanged.

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>

* feat(dev-setup): render gbrain :user variant to an untracked workspace dir

Stops the dev/Conductor workspace from dirtying tracked SKILL.md source. setup
honors GSTACK_SKIP_GBRAIN_REGEN (passed inline by dev-setup, never exported) and
skips the in-place :user regen; detection is still persisted (PID-unique tmp so
concurrent workspaces can't clobber it). dev-setup instead renders the :user
variant into .claude/gstack-rendered (gitignored, per-workspace) and repoints
the workspace SKILL.md symlinks at it, so the workspace gets brain-aware blocks
while the worktree stays canonical. dev-teardown removes the render.

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>

* feat(dev-skill): refresh the untracked brain-aware render on template change

After the default in-place regen (which keeps the worktree canonical and runs
validation), also re-render the :user variant into .claude/gstack-rendered when
it exists, so live template edits reflect at the workspace's runtime. Never
creates the render dir during plain template dev.

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>

* feat(gstack-config): gbrain-refresh renders brain-aware blocks into the install

Extends gbrain-refresh to render the :user variant into the global install
(~/.claude/skills/gstack) so every project's Claude sessions get brain-aware
blocks, not just the gstack dev workspace. Guarded against mutating the wrong
directory: the target must exist, not be a symlink (a symlinked install points
at a dev worktree), and look like a real gstack clone (VERSION + package.json).
Idempotent and self-documenting. CLAUDE.md's deploy section now notes that
'git reset --hard' reverts the blocks and to re-run gbrain-refresh.

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>

* test: cover gstack-gbrain-detect --is-ok + dev-skill render refresh

Fills the two automated-coverage gaps from the eng review: --is-ok exit-code
gate (no-cli -> nonzero, healthy -> 0, plus an agrees-with-JSON no-skew check
reusing the deterministic fake-gbrain harness) and a static tripwire that
dev-skill re-renders the :user variant into the workspace render dir only when
it already exists.

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>

* chore: bump version and changelog (v1.57.9.0)

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>

* docs: document brain-aware dev-setup render for v1.57.9.0

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>

---------

Co-authored-by: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
2026-06-09 22:29:23 -07:00

320 lines
11 KiB
TypeScript

/**
* Shape regression test for bin/gstack-gbrain-detect.
*
* After the bash→TS rewrite (codex #5), the TS output must stay
* key/type/semantics backward-compatible with the bash version. Downstream
* callers across most gstack skill preambles shell out to this script and
* pipe through jq. Key order may differ between bash+jq and JSON.stringify;
* key NAMES and TYPES must not.
*
* Asserts:
* 1. All 9 pre-existing keys are present
* 2. Each pre-existing key has the same primitive type/union as the bash version
* 3. The new key (gbrain_local_status) is present and a string
* 4. Output is parseable JSON
* 5. No keys removed/renamed
*/
import { describe, it, expect } from "bun:test";
import { execFileSync, spawnSync } from "child_process";
import {
mkdtempSync,
mkdirSync,
writeFileSync,
chmodSync,
rmSync,
} from "fs";
import { tmpdir } from "os";
import { join } from "path";
const DETECT_BIN = join(import.meta.dir, "..", "bin", "gstack-gbrain-detect");
/** Absolute bun path resolved once at module load (uses the test runner's PATH). */
const BUN_BIN = execFileSync("sh", ["-c", "command -v bun"], { encoding: "utf-8" }).trim();
/**
* Run detect with a controlled HOME + PATH so the output is deterministic.
* We invoke via `bun run <path>` instead of the shebang so the test doesn't
* need bun on its PATH. The script's child-process probes still respect
* the controlled PATH.
*/
function runDetect(env: Partial<NodeJS.ProcessEnv>): string {
return execFileSync(BUN_BIN, ["run", DETECT_BIN], {
encoding: "utf-8",
timeout: 15_000,
stdio: ["ignore", "pipe", "pipe"],
env: { ...process.env, ...env },
});
}
/** Run detect with --is-ok and return its exit code (never throws). */
function runIsOk(env: Partial<NodeJS.ProcessEnv>): number {
const r = spawnSync(BUN_BIN, ["run", DETECT_BIN, "--is-ok"], {
timeout: 15_000,
stdio: ["ignore", "pipe", "pipe"],
env: { ...process.env, ...env },
});
return r.status ?? 1;
}
interface DetectShape {
gbrain_on_path: boolean;
gbrain_version: string | null;
gbrain_config_exists: boolean;
gbrain_engine: string | null;
gbrain_doctor_ok: boolean;
gbrain_mcp_mode: string;
gstack_brain_sync_mode: string;
gstack_brain_git: boolean;
gstack_artifacts_remote: string;
gbrain_local_status: string;
}
describe("bin/gstack-gbrain-detect — shape regression", () => {
it("emits valid JSON", () => {
const tmp = mkdtempSync(join(tmpdir(), "detect-shape-"));
try {
const out = runDetect({
HOME: tmp,
PATH: "/usr/bin:/bin",
GSTACK_HOME: tmp,
});
expect(() => JSON.parse(out)).not.toThrow();
} finally {
rmSync(tmp, { recursive: true, force: true });
}
});
it("contains all 9 pre-existing keys + the new gbrain_local_status key", () => {
const tmp = mkdtempSync(join(tmpdir(), "detect-shape-"));
try {
const out = runDetect({
HOME: tmp,
PATH: "/usr/bin:/bin",
GSTACK_HOME: tmp,
});
const parsed = JSON.parse(out) as DetectShape;
// 9 pre-existing keys (must not be removed/renamed):
expect(parsed).toHaveProperty("gbrain_on_path");
expect(parsed).toHaveProperty("gbrain_version");
expect(parsed).toHaveProperty("gbrain_config_exists");
expect(parsed).toHaveProperty("gbrain_engine");
expect(parsed).toHaveProperty("gbrain_doctor_ok");
expect(parsed).toHaveProperty("gbrain_mcp_mode");
expect(parsed).toHaveProperty("gstack_brain_sync_mode");
expect(parsed).toHaveProperty("gstack_brain_git");
expect(parsed).toHaveProperty("gstack_artifacts_remote");
// 1 new key (added by this fix):
expect(parsed).toHaveProperty("gbrain_local_status");
} finally {
rmSync(tmp, { recursive: true, force: true });
}
});
it("preserves field types from the bash version", () => {
const tmp = mkdtempSync(join(tmpdir(), "detect-shape-"));
try {
const out = runDetect({
HOME: tmp,
PATH: "/usr/bin:/bin",
GSTACK_HOME: tmp,
});
const parsed = JSON.parse(out) as Record<string, unknown>;
// Booleans (bash: `true`/`false`; TS: boolean)
expect(typeof parsed.gbrain_on_path).toBe("boolean");
expect(typeof parsed.gbrain_config_exists).toBe("boolean");
expect(typeof parsed.gbrain_doctor_ok).toBe("boolean");
expect(typeof parsed.gstack_brain_git).toBe("boolean");
// String | null unions (bash: `null` when absent; TS: null when absent)
const versionType = parsed.gbrain_version === null ? "null" : typeof parsed.gbrain_version;
expect(versionType === "string" || versionType === "null").toBe(true);
const engineType = parsed.gbrain_engine === null ? "null" : typeof parsed.gbrain_engine;
expect(engineType === "string" || engineType === "null").toBe(true);
// Strings (bash: always emits a string, never null)
expect(typeof parsed.gbrain_mcp_mode).toBe("string");
expect(typeof parsed.gstack_brain_sync_mode).toBe("string");
expect(typeof parsed.gstack_artifacts_remote).toBe("string");
// New field: string enum
expect(typeof parsed.gbrain_local_status).toBe("string");
} finally {
rmSync(tmp, { recursive: true, force: true });
}
});
it("gbrain_mcp_mode is one of the three documented values", () => {
const tmp = mkdtempSync(join(tmpdir(), "detect-shape-"));
try {
const out = runDetect({
HOME: tmp,
PATH: "/usr/bin:/bin",
GSTACK_HOME: tmp,
});
const parsed = JSON.parse(out) as DetectShape;
expect(["local-stdio", "remote-http", "none"]).toContain(parsed.gbrain_mcp_mode);
} finally {
rmSync(tmp, { recursive: true, force: true });
}
});
it("gstack_brain_sync_mode is one of the three documented values", () => {
const tmp = mkdtempSync(join(tmpdir(), "detect-shape-"));
try {
const out = runDetect({
HOME: tmp,
PATH: "/usr/bin:/bin",
GSTACK_HOME: tmp,
});
const parsed = JSON.parse(out) as DetectShape;
expect(["off", "artifacts-only", "full"]).toContain(parsed.gstack_brain_sync_mode);
} finally {
rmSync(tmp, { recursive: true, force: true });
}
});
it("gbrain_local_status is one of the five documented values", () => {
const tmp = mkdtempSync(join(tmpdir(), "detect-shape-"));
try {
const out = runDetect({
HOME: tmp,
PATH: "/usr/bin:/bin",
GSTACK_HOME: tmp,
});
const parsed = JSON.parse(out) as DetectShape;
expect(["ok", "no-cli", "missing-config", "broken-config", "broken-db"]).toContain(
parsed.gbrain_local_status,
);
} finally {
rmSync(tmp, { recursive: true, force: true });
}
});
it("with no gbrain on PATH, returns gbrain_on_path=false and gbrain_local_status=no-cli", () => {
const tmp = mkdtempSync(join(tmpdir(), "detect-shape-"));
try {
const out = runDetect({
HOME: tmp,
PATH: "/usr/bin:/bin", // no gbrain on this PATH
GSTACK_HOME: tmp,
GSTACK_DETECT_NO_CACHE: "1",
});
const parsed = JSON.parse(out) as DetectShape;
expect(parsed.gbrain_on_path).toBe(false);
expect(parsed.gbrain_version).toBeNull();
expect(parsed.gbrain_local_status).toBe("no-cli");
} finally {
rmSync(tmp, { recursive: true, force: true });
}
});
it("with fake gbrain that returns valid JSON, returns gbrain_on_path=true and gbrain_local_status=ok", () => {
const tmp = mkdtempSync(join(tmpdir(), "detect-shape-"));
const bindir = join(tmp, "bin");
const home = join(tmp, "home");
const configDir = join(home, ".gbrain");
const configPath = join(configDir, "config.json");
try {
mkdirSync(bindir, { recursive: true });
mkdirSync(home, { recursive: true });
mkdirSync(configDir, { recursive: true });
writeFileSync(configPath, JSON.stringify({ engine: "pglite" }));
// Fake gbrain: prints valid sources-list JSON
const fake = `#!/bin/sh
case "$1 $2" in
"--version ") echo "gbrain 0.33.1.0"; exit 0 ;;
"sources list") echo '{"sources":[]}'; exit 0 ;;
"doctor "*) echo '{"status":"ok","checks":[]}'; exit 0 ;;
esac
exit 0
`;
const gbrainPath = join(bindir, "gbrain");
writeFileSync(gbrainPath, fake);
chmodSync(gbrainPath, 0o755);
const out = runDetect({
HOME: home,
PATH: `${bindir}:/usr/bin:/bin`,
GSTACK_HOME: tmp,
GSTACK_DETECT_NO_CACHE: "1",
});
const parsed = JSON.parse(out) as DetectShape;
expect(parsed.gbrain_on_path).toBe(true);
expect(parsed.gbrain_version).toBe("gbrain0.33.1.0");
expect(parsed.gbrain_config_exists).toBe(true);
expect(parsed.gbrain_engine).toBe("pglite");
expect(parsed.gbrain_local_status).toBe("ok");
} finally {
rmSync(tmp, { recursive: true, force: true });
}
});
});
describe("bin/gstack-gbrain-detect --is-ok — live gate", () => {
it("exits non-zero when gbrain is not on PATH (no-cli)", () => {
const tmp = mkdtempSync(join(tmpdir(), "detect-isok-"));
try {
const code = runIsOk({
HOME: tmp,
PATH: "/usr/bin:/bin", // no gbrain
GSTACK_HOME: tmp,
GSTACK_DETECT_NO_CACHE: "1",
});
expect(code).not.toBe(0);
} finally {
rmSync(tmp, { recursive: true, force: true });
}
});
it("exits 0 when a fake gbrain reports a healthy engine (ok)", () => {
const tmp = mkdtempSync(join(tmpdir(), "detect-isok-"));
const bindir = join(tmp, "bin");
const home = join(tmp, "home");
const configDir = join(home, ".gbrain");
try {
mkdirSync(bindir, { recursive: true });
mkdirSync(configDir, { recursive: true });
writeFileSync(join(configDir, "config.json"), JSON.stringify({ engine: "pglite" }));
const fake = `#!/bin/sh
case "$1 $2" in
"--version ") echo "gbrain 0.33.1.0"; exit 0 ;;
"sources list") echo '{"sources":[]}'; exit 0 ;;
"doctor "*) echo '{"status":"ok","checks":[]}'; exit 0 ;;
esac
exit 0
`;
const gbrainPath = join(bindir, "gbrain");
writeFileSync(gbrainPath, fake);
chmodSync(gbrainPath, 0o755);
const code = runIsOk({
HOME: home,
PATH: `${bindir}:/usr/bin:/bin`,
GSTACK_HOME: tmp,
GSTACK_DETECT_NO_CACHE: "1",
});
expect(code).toBe(0);
} finally {
rmSync(tmp, { recursive: true, force: true });
}
});
it("exit code agrees with the JSON gbrain_local_status (no skew)", () => {
// Run both surfaces against the same env and assert they never disagree.
const tmp = mkdtempSync(join(tmpdir(), "detect-isok-"));
try {
const env = { HOME: tmp, PATH: "/usr/bin:/bin", GSTACK_HOME: tmp, GSTACK_DETECT_NO_CACHE: "1" };
const status = (JSON.parse(runDetect(env)) as DetectShape).gbrain_local_status;
const code = runIsOk(env);
expect(code === 0).toBe(status === "ok");
} finally {
rmSync(tmp, { recursive: true, force: true });
}
});
});