fix(rstack): skip unconfigured Rstest captures

Author: ScriptedAlchemy

packages/rstack/package.json

@@ -81,7 +81,7 @@
     "@rsbuild/core": "catalog:",
     "@rslib/core": "catalog:",
     "@rslint/core": "catalog:",
-    "@rstackjs/context": "git+https://github.com/rstackjs/context.git#6c992fac8c8e2b22a1a66f8f0d9a954437649bf7",
+    "@rstackjs/context": "git+https://github.com/rstackjs/context.git#f2408b0497a0ee56ce0a9c72fa2603beb8375b42",
     "@rstest/core": "catalog:",
     "prettier": "catalog:",
     "tinypool": "catalog:",

packages/rstack/src/mcp.ts

@@ -8,7 +8,7 @@ import {
   createContextMcpServer,
   resolveContextWorkspace,
 } from '@rstackjs/context';
-import { withRstackConfigTarget } from './config.ts';
+import { loadRstackConfig, withRstackConfigTarget } from './config.ts';
 import { resolveRelatedTests } from './relatedTests.ts';
 
 declare const RSTACK_VERSION: string;
@@ -47,6 +47,11 @@ const runContextMcpServer = async (startPath: string): Promise<void> => {
         wrapperConfigPath: path.join(import.meta.dirname, 'rstestConfig.js'),
         withConfigTarget: withRstackConfigTarget,
         resolveRelatedTests,
+        isTestConfigured: ({ packageRoot, configPath }) =>
+          withRstackConfigTarget(packageRoot, configPath, async () => {
+            const { configs } = await loadRstackConfig();
+            return configs.test !== undefined;
+          }),
       }),
   });
 

packages/rstack/tests/context/mcp.test.ts

@@ -1091,7 +1091,7 @@ test('reads the current store for every project status call without workspace pa
       expect(firstResult.content).toEqual([
         {
           type: 'text',
-          text: 'Rstack project status: 1 current context (0 ready, 1 pending); 0 context-store/read issues. See structuredContent for details.',
+          text: 'Rstack project status: 1 recorded context identity (0 ready, 1 pending); 0 context-store/read issues. See structuredContent for compact selection details.',
         },
       ]);
 
@@ -1111,7 +1111,14 @@ test('reads the current store for every project status call without workspace pa
             producer: run.producer,
             context,
             state: 'ready',
-            latestSnapshot: snapshot,
+            latestSnapshot: {
+              snapshotId: snapshot.snapshotId,
+              observedAt: snapshot.observedAt,
+              status: snapshot.status,
+              completeness: snapshot.completeness,
+              facets: ['summary'],
+              summary: {},
+            },
             freshness: { state: 'unknown', changedPaths: [] },
           },
         ],
@@ -1120,7 +1127,7 @@ test('reads the current store for every project status call without workspace pa
       expect(secondResult.content).toEqual([
         {
           type: 'text',
-          text: 'Rstack project status: 1 current context (1 ready, 0 pending); 0 context-store/read issues. See structuredContent for details.',
+          text: 'Rstack project status: 1 recorded context identity (1 ready, 0 pending); 0 context-store/read issues. See structuredContent for compact selection details.',
         },
       ]);
     });
@@ -1144,7 +1151,7 @@ test('returns a valid empty project status', async () => {
       expect(result.content).toEqual([
         {
           type: 'text',
-          text: 'Rstack project status: 0 current contexts (0 ready, 0 pending); 0 context-store/read issues. See structuredContent for details.',
+          text: 'Rstack project status: 0 recorded context identities (0 ready, 0 pending); 0 context-store/read issues. See structuredContent for compact selection details.',
         },
       ]);
     });
@@ -1793,6 +1800,50 @@ define.test({ include: ['./tests/*.never.ts'], passWithNoTests: ${fixture.testSt
   });
 });
 
+test('does not capture tests when the selected Rstack package has no test config', async () => {
+  await withTempWorkspace(async (workspaceRoot) => {
+    const rstackEntry = pathToFileURL(path.resolve('dist/index.js')).href;
+    await writeFile(
+      path.join(workspaceRoot, 'package.json'),
+      JSON.stringify({ name: '@repo/app', type: 'module' }),
+    );
+    await writeFile(
+      path.join(workspaceRoot, 'rstack.config.ts'),
+      `import { define } from ${JSON.stringify(rstackEntry)};
+define.app({});
+`,
+    );
+
+    const transport = new StdioClientTransport({
+      command: process.execPath,
+      args: [path.resolve('bin/rs.js'), 'mcp'],
+      cwd: workspaceRoot,
+      env: getDefaultEnvironment(),
+      stderr: 'pipe',
+    });
+    const client = new Client({
+      name: 'rstack-unconfigured-test-capture',
+      version: '1.0.0',
+    });
+
+    try {
+      await client.connect(transport);
+      const result = await client.callTool({ name: 'test_snapshot', arguments: {} });
+
+      expect(result.isError).toBe(true);
+      expect(result.content).toEqual([
+        {
+          type: 'text',
+          text: 'Rstest is not configured for package root ".".',
+        },
+      ]);
+      expect((await readProjectStatus(workspaceRoot)).contexts).toEqual([]);
+    } finally {
+      await client.close();
+    }
+  });
+});
+
 test('rejects unexpected fields at every Phase 3/4 MCP input boundary', async () => {
   await withTempWorkspace(async (workspaceRoot) => {
     await withMcpClient(workspaceRoot, async (client) => {

pnpm-lock.yaml

@@ -4,7 +4,7 @@ packages:
   - website
 
 allowBuilds:
-  '@rstackjs/context@https://codeload.github.com/rstackjs/context/tar.gz/6c992fac8c8e2b22a1a66f8f0d9a954437649bf7': true
+  '@rstackjs/context@https://codeload.github.com/rstackjs/context/tar.gz/f2408b0497a0ee56ce0a9c72fa2603beb8375b42': true
   core-js: false
   simple-git-hooks: false