@happyvertical/smrt-dev-mcp 0.57.30 → 0.57.32

package/AGENTS.md

@@ -274,6 +274,12 @@ launcher or a small wrapper script with an absolute Node path.
 
 ## Validation
 
+`src/mcp-conformance.spec.ts` runs the production HTTP handler and official
+client against an isolated consumer workspace with model manifests and docs.
+It checks protocol behavior and resource freshness with the official deadline;
+it does not measure large-monorepo indexing latency. Broader discovery coverage
+remains in the knowledge tests. See the [protocol fixture contract](agents/protocol-conformance.md).
+
 Run package `test`, `typecheck`, and `build`, then
 `pnpm --filter @happyvertical/smrt-dev-mcp verify:pack`. The packaging gate packs
 MCP/scanner/types, installs only their production dependencies into a fresh

package/agents/protocol-conformance.md

@@ -0,0 +1,20 @@
+# Protocol conformance fixture (#4317)
+
+`src/mcp-conformance.spec.ts` tests the production HTTP handler with the installed
+official client and the reviewed baseline. Its child process owns an isolated
+consumer workspace containing package/model manifests and authored docs. It does
+not inherit the arbitrary size of the caller's checkout or change global cwd.
+
+The official request deadline, assertions, baseline and production TTL0 freshness
+behavior stay unchanged. Discovery and subsequent reads after package/doc edits
+must remain observable. This fixture proves protocol behavior; it is not evidence
+of an arbitrary-workspace latency SLA. Broader workspace, discovery and freshness
+coverage remains in `src/knowledge/` tests.
+
+| Behavior | Trigger | Positive | Negative | Actor/context | Executor/transaction | Runtime/dialect | External edge | Level | Command |
+| --- | --- | --- | --- | --- | --- | --- | --- | --- | --- |
+| Use the actual protocol implementation | Official client connects to isolated host | Reviewed suite/baseline passes | Unexpected scenario still fails | MCP client and production handler in a real child | HTTP/child process; no SQL transaction | Node 26/MCP 2026-07-28; no DB | Official client and unchanged deadline | Conformance integration | `pnpm --filter @happyvertical/smrt-dev-mcp exec vitest run src/mcp-conformance.spec.ts` |
+| Retain metadata/header contracts | Modern requests/discovery | Cache/result metadata and reads match assertions | Header/method mismatch is rejected | HTTP client | HTTP; no SQL transaction | Node 26/MCP; no DB | Production HTTP adapter | Integration | `pnpm --filter @happyvertical/smrt-dev-mcp exec vitest run src/mcp-conformance.spec.ts` |
+| Meaningful bounded discovery | Fixture has model/package/docs | Exact fixture package resources and model field are present | Accidental monorepo discovery cannot satisfy exact set | Consumer workspace | Filesystem/index read; no SQL transaction | Node 26; no DB | Real manifest/doc discovery | Integration | `pnpm --filter @happyvertical/smrt-dev-mcp exec vitest run src/mcp-conformance.spec.ts` |
+| Preserve live freshness | Add package and edit authored docs | Subsequent list/read reflects changes | Stale/empty success fails assertions | Consumer editing project files | Filesystem and HTTP; no SQL transaction | Node 26/MCP; no DB | TTL0 resource contract | Integration | `pnpm --filter @happyvertical/smrt-dev-mcp exec vitest run src/mcp-conformance.spec.ts` |
+| Isolate and clean execution | Host startup, exit or teardown | Child cwd and temporary files are scoped and removed | Startup/termination failure is bounded and reported | Test integration owner | Child process/filesystem; no SQL transaction | Node 26; no DB | IPC, signals and temporary resources | Integration | `pnpm --filter @happyvertical/smrt-dev-mcp exec vitest run src/mcp-conformance.spec.ts` |

package/agents/runtime-fixture-isolation.md

@@ -0,0 +1,29 @@
+# Runtime fixture isolation (#3760)
+
+Missing-dependency tests must not resolve packages installed in ancestor
+`node_modules` directories. `project-runtime.test.ts` selects the first parent
+(tmpdir then homedir) whose ancestry has none, matching the packed-consumer
+fixture isolation rule. If both parents are contaminated, it asks for a clean
+TMPDIR instead of silently borrowing packages. Production runtime resolution
+continues to support ordinary hoisted workspace dependencies.
+
+| Behavior / invariant | Trigger | Positive | Negative / failure | Actor / context | Executor / transaction | Runtime / dialect | External edge | Test level / validation |
+| --- | --- | --- | --- | --- | --- | --- | --- | --- |
+| Hermetic missing-dependency fixture | Normal project-runtime suite | Missing config preserves runtime_dependency_unavailable | Ancestor node_modules cannot alter classification | Test process with polluted /tmp; no production actor | N/A: no application writes; test-owned filesystem only | Node26 / Node ESM lookup; SQL N/A | Host ancestor installations | Focused project-runtime.test.ts |
+| Ancestor selection | Fixture allocation | Clean candidate selected | Candidate under node_modules ancestor rejected | Test-owned clean and polluted directory graph | N/A transaction: temporary dirs removed after each case | Node filesystem | Hoisted package lookup | Deterministic selector case in same file |
+
+Regression evidence on the unchanged base: the exact three runtime setup cases
+failed 1/3 with default `/tmp` ancestry. `/tmp/node_modules/@happyvertical/smrt-config`
+was an existing unowned symlink to a separate 0.55.4 checkout; it was preserved.
+An owned clean TMPDIR passed 3/3, confirming fixture contamination rather than a
+runtime defect. That diagnosis does not replace the documented gate: the fixed
+fixture must pass with the ordinary default environment and unchanged assertions.
+
+Run `pnpm --filter @happyvertical/smrt-dev-mcp exec vitest run src/project-runtime.test.ts`.
+Full outputs are retained under the release evidence `search/dev-mcp-*.log`.
+
+Head evidence: the complete project-runtime file passes all 30 cases with the
+ordinary default environment, including the unchanged missing-config assertion,
+real config loading, hoisted workspace resolution and the new ancestor-selector
+case. Biome passes the changed test file. No runtime code or packaged application
+artifact changed; the coordinator reruns the documented combined source gate.

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@happyvertical/smrt-dev-mcp",
-  "version": "0.57.30",
+  "version": "0.57.32",
   "description": "Development MCP server for SMRT framework knowledge, review context, architecture context, and code generation",
   "type": "module",
   "main": "./dist/index.js",
@@ -33,6 +33,7 @@
     "README.md",
     "dist",
     "AGENTS.md",
+    "agents",
     "plugin.json",
     "mcp.json",
     "skills",
@@ -53,14 +54,14 @@
   "homepage": "https://github.com/happyvertical/smrt/tree/main/packages/smrt-dev-mcp#readme",
   "license": "MIT",
   "dependencies": {
-    "@happyvertical/smrt-scanner": "0.57.30",
+    "@happyvertical/smrt-scanner": "0.57.32",
     "@modelcontextprotocol/node": "2.0.0",
     "@modelcontextprotocol/server": "2.3.1",
-    "@happyvertical/smrt-types": "0.57.30"
+    "@happyvertical/smrt-types": "0.57.32"
   },
   "devDependencies": {
-    "@happyvertical/smrt-config": "0.57.30",
-    "@happyvertical/smrt-core": "0.57.30",
+    "@happyvertical/smrt-config": "0.57.32",
+    "@happyvertical/smrt-core": "0.57.32",
     "@happyvertical/sql": "^0.108.0",
     "@modelcontextprotocol/client": "2.0.0",
     "@modelcontextprotocol/conformance": "0.2.0-alpha.11",