xtrm-cli 2.1.16 → 2.1.19

package/test/hooks.test.ts

@@ -77,7 +77,7 @@ describe('main-guard.mjs — MAIN_GUARD_PROTECTED_BRANCHES', () => {
     );
     expect(r.status).toBe(2);
     const out = parseHookJson(r.stdout);
-    expect(out?.systemMessage).toContain('Bash is restricted');
+    expect(out?.systemMessage).toContain('Bash restricted');
   });
 
   it('allows safe Bash commands on protected branch', () => {
@@ -117,10 +117,19 @@ describe('main-guard.mjs — MAIN_GUARD_PROTECTED_BRANCHES', () => {
       );
       expect(r.status, `expected exit 2 for: ${command}`).toBe(2);
       const out = parseHookJson(r.stdout);
-      expect(out?.systemMessage).toContain('Bash is restricted');
+      expect(out?.systemMessage).toContain('Bash restricted');
     }
   });
 
+  it('allows touch .beads/.memory-gate-done on protected branch', () => {
+    const r = runHook(
+      'main-guard.mjs',
+      { tool_name: 'Bash', tool_input: { command: 'touch .beads/.memory-gate-done' } },
+      { MAIN_GUARD_PROTECTED_BRANCHES: CURRENT_BRANCH },
+    );
+    expect(r.status).toBe(0);
+  });
+
   it('allows Bash when MAIN_GUARD_ALLOW_BASH=1 is set', () => {
     const r = runHook(
       'main-guard.mjs',
@@ -187,9 +196,11 @@ describe('main-guard-post-push.mjs', () => {
         repoDir,
       );
       expect(r.status).toBe(0);
-      const out = parseHookJson(r.stdout);
-      expect(out?.systemMessage).toContain('gh pr create --fill');
-      expect(out?.systemMessage).toContain('gh pr merge --squash');
+      expect(r.stdout).toContain('gh pr create --fill');
+      expect(r.stdout).toContain('gh pr merge --squash');
+      // output must be plain text (agent-only), not a JSON systemMessage banner
+      expect(parseHookJson(r.stdout)).toBeNull();
+
     } finally {
       rmSync(repoDir, { recursive: true, force: true });
     }
@@ -320,7 +331,7 @@ exit 1
         { PATH: `${fake.tempDir}:${process.env.PATH ?? ''}` },
       );
       expect(r.status).toBe(2);
-      expect(r.stderr).toContain('no active claim');
+      expect(r.stderr).toContain('active claim');
     } finally {
       rmSync(fake.tempDir, { recursive: true, force: true });
       rmSync(projectDir, { recursive: true, force: true });
@@ -445,7 +456,7 @@ exit 1
         { PATH: `${fake.tempDir}:${process.env.PATH ?? ''}` },
       );
       expect(r.status).toBe(2);
-      expect(r.stderr).toContain('MEMORY GATE');
+      expect(r.stderr).toContain('Memory gate');
     } finally {
       rmSync(fake.tempDir, { recursive: true, force: true });
       rmSync(projectDir, { recursive: true, force: true });
@@ -542,51 +553,6 @@ exit 2
 });
 
 
-// ── gitnexus-impact-reminder.py ──────────────────────────────────────────────
-
-function runPythonHook(
-  hookFile: string,
-  input: Record<string, unknown>,
-) {
-  return spawnSync('python3', [path.join(HOOKS_DIR, hookFile)], {
-    input: JSON.stringify(input),
-    encoding: 'utf8',
-    env: { ...process.env },
-  });
-}
-
-describe('gitnexus-impact-reminder.py', () => {
-  it('injects additionalContext when prompt contains an edit-intent keyword', () => {
-    const r = runPythonHook('gitnexus-impact-reminder.py', {
-      hook_event_name: 'UserPromptSubmit',
-      prompt: 'fix the broken auth logic in login.ts',
-    });
-    expect(r.status).toBe(0);
-    const out = parseHookJson(r.stdout);
-    expect(out?.hookSpecificOutput?.additionalContext).toContain('gitnexus impact');
-  });
-
-  it('does nothing (no output) when prompt has no edit-intent keywords', () => {
-    const r = runPythonHook('gitnexus-impact-reminder.py', {
-      hook_event_name: 'UserPromptSubmit',
-      prompt: 'explain how the beads gate works',
-    });
-    expect(r.status).toBe(0);
-    expect(r.stdout.trim()).toBe('');
-  });
-
-  it('does nothing for non-UserPromptSubmit events', () => {
-    const r = runPythonHook('gitnexus-impact-reminder.py', {
-      hook_event_name: 'PreToolUse',
-      tool_name: 'Edit',
-      tool_input: { file_path: 'foo.ts' },
-      prompt: 'fix something',
-    });
-    expect(r.status).toBe(0);
-    expect(r.stdout.trim()).toBe('');
-  });
-});
-
 
 // ── beads-gate-core.mjs — decision functions ──────────────────────────────────
 
@@ -704,3 +670,141 @@ process.exit(ok ? 0 : 1);
     });
   });
 });
+
+
+// ── beads-compact-save.mjs ───────────────────────────────────────────────────
+describe('beads-compact-save.mjs', () => {
+  it('exits 0 silently when no .beads directory exists', () => {
+    const r = runHook('beads-compact-save.mjs', { hook_event_name: 'PreCompact', cwd: '/tmp' });
+    expect(r.status).toBe(0);
+    expect(r.stdout).toBe('');
+  });
+
+  it('exits 0 silently and writes no file when no in_progress issues', () => {
+    const projectDir = mkdtempSync(path.join(os.tmpdir(), 'xtrm-compact-save-'));
+    mkdirSync(path.join(projectDir, '.beads'));
+    const fake = withFakeBdDir(`#!/usr/bin/env bash
+if [[ "$1" == "list" ]]; then
+  echo ""
+  echo "--------------------------------------------------------------------------------"
+  echo "Total: 2 issues (2 open, 0 in progress)"
+  exit 0
+fi
+exit 1
+`);
+    try {
+      const r = runHook(
+        'beads-compact-save.mjs',
+        { hook_event_name: 'PreCompact', cwd: projectDir },
+        { PATH: `${fake.tempDir}:${process.env.PATH ?? ''}` },
+      );
+      expect(r.status).toBe(0);
+      const { existsSync } = require('node:fs');
+      expect(existsSync(path.join(projectDir, '.beads', '.last_active'))).toBe(false);
+    } finally {
+      rmSync(fake.tempDir, { recursive: true, force: true });
+      rmSync(projectDir, { recursive: true, force: true });
+    }
+  });
+
+  it('writes .beads/.last_active with in_progress issue IDs', () => {
+    const projectDir = mkdtempSync(path.join(os.tmpdir(), 'xtrm-compact-save-'));
+    mkdirSync(path.join(projectDir, '.beads'));
+    const fake = withFakeBdDir(`#!/usr/bin/env bash
+if [[ "$1" == "list" ]]; then
+  cat <<'EOF'
+◐ proj-abc123 ● P1 First in_progress issue
+◐ proj-def456 ● P2 Second in_progress issue
+
+--------------------------------------------------------------------------------
+Total: 2 issues (0 open, 2 in progress)
+EOF
+  exit 0
+fi
+exit 1
+`);
+    try {
+      const r = runHook(
+        'beads-compact-save.mjs',
+        { hook_event_name: 'PreCompact', cwd: projectDir },
+        { PATH: `${fake.tempDir}:${process.env.PATH ?? ''}` },
+      );
+      expect(r.status).toBe(0);
+      const { readFileSync: rfs } = require('node:fs');
+      const saved = rfs(path.join(projectDir, '.beads', '.last_active'), 'utf8').trim().split('\n');
+      expect(saved).toContain('proj-abc123');
+      expect(saved).toContain('proj-def456');
+    } finally {
+      rmSync(fake.tempDir, { recursive: true, force: true });
+      rmSync(projectDir, { recursive: true, force: true });
+    }
+  });
+});
+
+// ── beads-compact-restore.mjs ────────────────────────────────────────────────
+describe('beads-compact-restore.mjs', () => {
+  it('exits 0 silently when no .beads/.last_active file exists', () => {
+    const projectDir = mkdtempSync(path.join(os.tmpdir(), 'xtrm-compact-restore-'));
+    mkdirSync(path.join(projectDir, '.beads'));
+    try {
+      const r = runHook(
+        'beads-compact-restore.mjs',
+        { hook_event_name: 'SessionStart', cwd: projectDir },
+      );
+      expect(r.status).toBe(0);
+      expect(r.stdout).toBe('');
+    } finally {
+      rmSync(projectDir, { recursive: true, force: true });
+    }
+  });
+
+  it('restores in_progress status, deletes .last_active, and injects additionalSystemPrompt', () => {
+    const projectDir = mkdtempSync(path.join(os.tmpdir(), 'xtrm-compact-restore-'));
+    mkdirSync(path.join(projectDir, '.beads'));
+    const { writeFileSync: wfs } = require('node:fs');
+    wfs(path.join(projectDir, '.beads', '.last_active'), 'proj-abc123\nproj-def456\n', 'utf8');
+
+    const callLog = path.join(projectDir, 'bd-calls.log');
+    const fake = withFakeBdDir(`#!/usr/bin/env bash
+echo "$@" >> "${callLog}"
+exit 0
+`);
+    try {
+      const r = runHook(
+        'beads-compact-restore.mjs',
+        { hook_event_name: 'SessionStart', cwd: projectDir },
+        { PATH: `${fake.tempDir}:${process.env.PATH ?? ''}` },
+      );
+      expect(r.status).toBe(0);
+      // .last_active must be deleted
+      const { existsSync: exs, readFileSync: rfs } = require('node:fs');
+      expect(exs(path.join(projectDir, '.beads', '.last_active'))).toBe(false);
+      // bd update called for each ID
+      const calls = rfs(callLog, 'utf8');
+      expect(calls).toContain('proj-abc123');
+      expect(calls).toContain('proj-def456');
+      // additionalSystemPrompt injected for agent
+      const out = parseHookJson(r.stdout);
+      expect(out?.hookSpecificOutput?.additionalSystemPrompt).toMatch(/Restored 2 in_progress issue/);
+    } finally {
+      rmSync(fake.tempDir, { recursive: true, force: true });
+      rmSync(projectDir, { recursive: true, force: true });
+    }
+  });
+});
+
+
+// ── hooks.json wiring ────────────────────────────────────────────────────────
+describe('hooks.json — beads-compact hooks wiring', () => {
+  it('wires beads-compact-save.mjs to PreCompact event', () => {
+    const cfg = JSON.parse(readFileSync(path.join(__dirname, '../../config/hooks.json'), 'utf8'));
+    const preCompact: Array<{ script: string }> = cfg.hooks.PreCompact ?? [];
+    expect(preCompact.some((h) => h.script === 'beads-compact-save.mjs')).toBe(true);
+  });
+
+  it('wires beads-compact-restore.mjs to SessionStart event', () => {
+    const cfg = JSON.parse(readFileSync(path.join(__dirname, '../../config/hooks.json'), 'utf8'));
+    const sessionStart: Array<{ script: string }> = cfg.hooks.SessionStart ?? [];
+    expect(sessionStart.some((h) => h.script === 'beads-compact-restore.mjs')).toBe(true);
+  });
+});