fix(test): make the suite pass from any checkout directory (#199)

Two defects made a bare `npm test` unreliable outside a checkout literally
named mcp-automem — the reason the marker-extraction work had to be run as
`npx vitest run --exclude '**/.claude/**'` and still reported two failures.

vitest.config.ts: setting `exclude` replaces vitest's defaults, so the list has
to carry everything. It named 'node_modules/**' and 'dist/**' without the
recursive '**/' form and said nothing about .claude/, so from the main checkout
the glob walked every agent worktree under .claude/worktrees/ — 106 test files
across two worktrees here — running each suite twice, the second time from a
stale copy of the repo.

hermes-real-host.test.ts: the provider derives its project tag from
`basename(os.getcwd())`, and runHermesProviderPrefetchSequence defaulted its cwd
to REPO_ROOT. The tag assertions therefore passed only where the checkout
directory happened to be named mcp-automem, and failed in a worktree, a second
clone, or any CI cache path. The prefetch now runs in a fixture directory named
for the slug, which pins the contract the provider actually promises instead of
the machine's directory layout. Python imports resolve from the Hermes venv, not
cwd, so nothing else depended on REPO_ROOT here.

Both verified load-bearing: dropping the .claude exclude re-globs a planted
worktree fixture, and restoring the REPO_ROOT default fails exactly the two
prefetch tests.

Co-authored-by: Claude Opus 5 <noreply@anthropic.com>

Author: jack-arturo

tests/host-smoke/hermes-real-host.test.ts

@@ -322,10 +322,24 @@ print(json.dumps({
   return JSON.parse(lastJsonLine);
 }
 
+/**
+ * Working directory the prefetch runs in. The provider derives its project tag from
+ * `basename(os.getcwd())`, so this — not the repo — decides what the tag assertions see.
+ *
+ * It used to default to REPO_ROOT, which made the assertions pass only where the
+ * checkout directory happened to be named `mcp-automem`: a git worktree, a second clone,
+ * or a CI cache path under any other name failed on a tag the provider never promised.
+ * A fixture directory named for the slug is deterministic everywhere, and it pins the
+ * contract (tag == basename of cwd) instead of the machine's directory layout. Set per
+ * test in beforeEach; the Python imports resolve from the Hermes venv, not from cwd.
+ */
+const PROJECT_SLUG = 'mcp-automem';
+let projectCwd: string;
+
 async function runHermesProviderPrefetchSequence(
   home: string,
   prompts: string[],
-  cwd: string = REPO_ROOT
+  cwd: string = projectCwd
 ): Promise<string[]> {
   if (!HERMES_PYTHON) {
     throw new Error('Hermes Python is not available');
@@ -481,13 +495,21 @@ describe.skipIf(!HERMES_PYTHON)('Hermes real host integration', () => {
   let tmpDir: string;
   let fakeApi: FakeAutoMemApi;
 
+  let projectRoot: string;
+
   beforeEach(async () => {
     tmpDir = fs.mkdtempSync(path.join(os.tmpdir(), 'automem-hermes-host-'));
+    // See PROJECT_SLUG: the prefetch runs here so the provider's project tag is the
+    // slug under test rather than whatever this checkout's directory is called.
+    projectRoot = fs.mkdtempSync(path.join(os.tmpdir(), 'automem-hermes-project-'));
+    projectCwd = path.join(projectRoot, PROJECT_SLUG);
+    fs.mkdirSync(projectCwd, { recursive: true });
     fakeApi = await startFakeAutoMemApi();
   });
 
   afterEach(async () => {
     fs.rmSync(tmpDir, { recursive: true, force: true });
+    fs.rmSync(projectRoot, { recursive: true, force: true });
     await fakeApi.close();
   });
 
@@ -637,7 +659,7 @@ describe.skipIf(!HERMES_PYTHON)('Hermes real host integration', () => {
     expect(firstContext?.searchParams.get('limit')).toBe('10');
     expect(firstContext?.searchParams.get('time_query')).toBe('last 90 days');
     expect(firstContext?.searchParams.get('format')).toBe('detailed');
-    expect(firstContext?.searchParams.getAll('tags')).toContain('mcp-automem');
+    expect(firstContext?.searchParams.getAll('tags')).toContain(PROJECT_SLUG);
 
     const topicShift = recalls.find(
       (url) => url.searchParams.get('query') === 'Railway deployment status'
@@ -699,7 +721,8 @@ describe.skipIf(!HERMES_PYTHON)('Hermes real host integration', () => {
     });
 
     await runHermesProviderPrefetchSequence(tmpDir, ['do we like Example Contact?']);
-    await runHermesProviderPrefetchSequence(tmpDir, ['what do we know about mcp-automem Hermes?']);
+    const projectPrompt = `what do we know about ${PROJECT_SLUG} Hermes?`;
+    await runHermesProviderPrefetchSequence(tmpDir, [projectPrompt]);
 
     const generalContext = recallRequests(fakeApi).find(
       (url) => url.searchParams.get('query') === 'do we like Example Contact?'
@@ -708,10 +731,10 @@ describe.skipIf(!HERMES_PYTHON)('Hermes real host integration', () => {
     expect(generalContext?.searchParams.getAll('tags')).toEqual([]);
 
     const projectContext = recallRequests(fakeApi).find(
-      (url) => url.searchParams.get('query') === 'what do we know about mcp-automem Hermes?'
+      (url) => url.searchParams.get('query') === projectPrompt
     );
     expect(projectContext?.searchParams.get('limit')).toBe('10');
-    expect(projectContext?.searchParams.getAll('tags')).toContain('mcp-automem');
+    expect(projectContext?.searchParams.getAll('tags')).toContain(PROJECT_SLUG);
   }, 45_000);
 
   it('provider explicit recall clamps large limits before calling AutoMem', async () => {

vitest.config.ts

@@ -4,9 +4,20 @@ export default defineConfig({
   test: {
     globals: true,
     environment: 'node',
-    // Default: exclude integration tests (they need real service)
-    // Run integration tests with: npm run test:integration
-    exclude: ['node_modules/**', 'dist/**', 'tests/integration/automem-service.test.ts'],
+    // Setting `exclude` REPLACES vitest's defaults, so everything that must stay out of
+    // the run has to be listed here — including the recursive `**/` forms, or a nested
+    // copy of the repo brings its own node_modules and dist back into scope.
+    exclude: [
+      '**/node_modules/**',
+      '**/dist/**',
+      // Agent worktrees under .claude/worktrees/ are full checkouts of this repo. Without
+      // this, `npm test` from the main checkout globs every suite twice — once from the
+      // real tree and once from each worktree's (stale) copy — and reports failures for
+      // code that is not the code under test.
+      '**/.claude/**',
+      // Integration tests need a real service; run them with `npm run test:integration`.
+      'tests/integration/automem-service.test.ts',
+    ],
     coverage: {
       provider: 'v8',
       reporter: ['text', 'json', 'html'],