@happyvertical/smrt-dev-mcp 0.40.63 → 0.40.65

package/AGENTS.md

@@ -23,6 +23,10 @@ deterministic SMRT ecosystem knowledge, and portable agent workflows.
 | `list-agent-skills` | Lists bundled harness-agnostic agent skills |
 | `get-agent-skill` | Returns a bundled agent skill as Markdown plus optional references |
 
+`pnpm knowledge:check --strict --format markdown` compares this catalog and the
+README parameter tables to the exported `TOOLS` definitions. Add or change a
+tool schema and its authored documentation in the same commit.
+
 ## Knowledge Discovery (#2143)
 
 Discovery reads the workspace globs instead of assuming `<root>/packages`:

package/README.md

@@ -242,14 +242,14 @@ Generate a complete s-m-r-t class with `@smrt()` decorator, fields, and imports.
 | Parameter | Type | Required | Description |
 |-----------|------|----------|-------------|
 | `className` | `string` | Yes | Class name (PascalCase) |
-| `properties` | `array` | Yes | Property definitions (`name`, `type`, `required?`, `nullable?`, `description?`, `defaultValue?`) |
-| `baseClass` | `string` | No | `'SmrtObject'` (default) or `'SmrtCollection'` |
-| `template` | `string` | No | `basic`, `global-catalog`, `optional-catalog`, `tenant-project-object`, `tenant-event-log-object`, or `cross-package-reference` |
+| `properties` | `object[]` | Yes | Property definitions (`name`, `type`, `required?`, `nullable?`, `description?`, `defaultValue?`) |
+| `baseClass` | `'SmrtObject' \| 'SmrtCollection'` | No | `'SmrtObject'` (default) or `'SmrtCollection'` |
+| `template` | `'basic' \| 'global-catalog' \| 'optional-catalog' \| 'tenant-project-object' \| 'tenant-event-log-object' \| 'cross-package-reference'` | No | Generation template (default: `basic`) |
 | `tableName` | `string` | No | Explicit `@smrt({ tableName })` value |
 | `conflictColumns` | `string[]` | No | Explicit upsert natural key columns |
-| `tenantScoped` | `boolean/object` | No | Add `@TenantScoped(...)`; object supports `mode`, `field`, and bypass/filter options |
+| `tenantScoped` | `boolean \| object` | No | Add `@TenantScoped(...)`; object supports `mode`, `field`, and bypass/filter options |
 | `includeTenantIdField` | `boolean` | No | Emit a matching `@tenantId()` field |
-| `relationships` | `array` | No | Relationship definitions for `foreignKey`, `crossPackageRef`, `oneToMany`, or `manyToMany` |
+| `relationships` | `object[]` | No | Relationship definitions for `foreignKey`, `crossPackageRef`, `oneToMany`, or `manyToMany` |
 | `includeCompanionSnippets` | `boolean` | No | Append package wiring notes |
 | `includeApiConfig` | `boolean` | No | Include REST API config (default: true) |
 | `includeMcpConfig` | `boolean` | No | Include MCP config (default: true) |
@@ -307,7 +307,7 @@ coverage, relationship-v2 counts, and freshness status.
 | Parameter | Type | Required | Description |
 |-----------|------|----------|-------------|
 | `rootDir` | `string` | No | Project root directory (default: cwd) |
-| `scope` | `string` | No | `project`, `local`, `package`, `sdk`, or `installed` |
+| `scope` | `'project' \| 'local' \| 'package' \| 'sdk' \| 'installed'` | No | Knowledge source scope (default: `project`) |
 | `package` | `string` | No | Package name or short name to focus |
 
 ### `check-knowledge-freshness`
@@ -330,7 +330,7 @@ Alias over the deterministic checker that emphasizes downstream
 | `rootDir` | `string` | No | Project root directory (default: cwd) |
 | `changed` | `boolean` | No | Limit stale-pattern checks to changed files |
 | `strict` | `boolean` | No | Treat stale-pattern findings as errors |
-| `scope` | `string` | No | `project`, `local`, `package`, `sdk`, or `installed` |
+| `scope` | `'project' \| 'local' \| 'package' \| 'sdk' \| 'installed'` | No | Knowledge source scope (default: `project`) |
 | `package` | `string` | No | Package name or short name to focus |
 
 ### `build-review-context`
@@ -344,11 +344,21 @@ then return a model-ready prompt bundle.
 | `changedFiles` | `string[]` | No | Files to route to package experts |
 | `focus` | `string` | No | Review focus or concern |
 | `documentation` | `string` | No | Additional docs or notes to include |
+| `detail` | `'summary' \| 'full'` | No | Default `summary`; `full` embeds authored docs and full package records |
 
 ### `build-domain-review-context`
 
-Domain-scoped alias for `build-review-context`. Accepts the same fields plus
-`scope` and `package`.
+Domain-scoped alias for `build-review-context`.
+
+| Parameter | Type | Required | Description |
+|-----------|------|----------|-------------|
+| `rootDir` | `string` | No | Project root directory (default: cwd) |
+| `changedFiles` | `string[]` | No | Files to route to package experts |
+| `focus` | `string` | No | Review focus or concern |
+| `documentation` | `string` | No | Additional docs or notes to include |
+| `scope` | `'project' \| 'local' \| 'package' \| 'sdk' \| 'installed'` | No | Knowledge source scope (default: `project`) |
+| `package` | `string` | No | Package name or short name to focus |
+| `detail` | `'summary' \| 'full'` | No | Default `summary`; `full` embeds authored docs and full package records |
 
 ### `smrt-review`
 
@@ -360,7 +370,8 @@ Return deterministic review findings, a prompt bundle, or both.
 | `changedFiles` | `string[]` | No | Files to route to package experts |
 | `focus` | `string` | No | Review focus or concern |
 | `documentation` | `string` | No | Additional docs or notes to include |
-| `mode` | `string` | No | `findings`, `prompt-bundle`, or `both` |
+| `mode` | `'findings' \| 'prompt-bundle' \| 'both'` | No | Response mode (default: `both`) |
+| `detail` | `'summary' \| 'full'` | No | Default `summary`; `full` embeds authored docs and full package records |
 
 ### `build-architecture-context`
 
@@ -373,11 +384,21 @@ then return a model-ready architecture prompt bundle.
 | `idea` | `string` | No | Product or implementation idea |
 | `documentation` | `string` | No | Existing docs or requirements |
 | `focus` | `string` | No | Architecture concern to prioritize |
+| `detail` | `'summary' \| 'full'` | No | Default `summary`; `full` embeds authored docs and full package records |
 
 ### `build-domain-architecture-context`
 
-Domain-scoped alias for `build-architecture-context`. Accepts the same fields
-plus `scope` and `package`.
+Domain-scoped alias for `build-architecture-context`.
+
+| Parameter | Type | Required | Description |
+|-----------|------|----------|-------------|
+| `rootDir` | `string` | No | Project root directory (default: cwd) |
+| `idea` | `string` | No | Product or implementation idea |
+| `documentation` | `string` | No | Existing docs or requirements |
+| `focus` | `string` | No | Architecture concern to prioritize |
+| `scope` | `'project' \| 'local' \| 'package' \| 'sdk' \| 'installed'` | No | Knowledge source scope (default: `project`) |
+| `package` | `string` | No | Package name or short name to focus |
+| `detail` | `'summary' \| 'full'` | No | Default `summary`; `full` embeds authored docs and full package records |
 
 ### `smrt-architecture`
 
@@ -403,7 +424,7 @@ Return a bundled agent skill as Markdown, with optional referenced files.
 
 | Parameter | Type | Required | Description |
 |-----------|------|----------|-------------|
-| `name` | `string` | Yes | Skill name. Currently `smrt-code-review` |
+| `name` | `'smrt-code-review'` | Yes | Skill name |
 | `includeReferences` | `boolean` | No | Include referenced files (default: true) |
 
 ## MCP Resources And Prompts

package/dist/index.d.ts

@@ -1,5 +1,5 @@
-import { Server, Tool } from '@modelcontextprotocol/server';
+import { Server } from '@modelcontextprotocol/server';
+export { TOOLS } from './tool-catalog.js';
 export declare const SERVER_VERSION: string;
-export declare const TOOLS: Tool[];
 export declare function createServer(): Server;
 //# sourceMappingURL=index.d.ts.map

package/dist/index.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"index.d.ts","sourceRoot":"","sources":["../src/index.ts"],"names":[],"mappings":"AAAA;;;;GAIG;AAKH,OAAO,EAGL,MAAM,EACN,KAAK,IAAI,EACV,MAAM,8BAA8B,CAAC;AAqBtC,eAAO,MAAM,cAAc,QAAuB,CAAC;AAwenD,eAAO,MAAM,KAAK,EAAE,IAAI,EAO0C,CAAC;AAEnE,wBAAgB,YAAY,IAAI,MAAM,CAkfrC"}
+{"version":3,"file":"index.d.ts","sourceRoot":"","sources":["../src/index.ts"],"names":[],"mappings":"AAAA;;;;GAIG;AAKH,OAAO,EAGL,MAAM,EACP,MAAM,8BAA8B,CAAC;AAqBtC,OAAO,EAAE,KAAK,EAAE,MAAM,mBAAmB,CAAC;AAG1C,eAAO,MAAM,cAAc,QAAuB,CAAC;AAuBnD,wBAAgB,YAAY,IAAI,MAAM,CAkfrC"}

package/dist/index.js

@@ -1,5 +1,5 @@
 #!/usr/bin/env node
-import { a as checkKnowledgeFreshnessFromIndex, d as smrtReview, i as checkKnowledgeFreshness, n as buildKnowledgeIndex, r as buildReviewContext, s as packageDocPaths, t as buildArchitectureContext, u as smrtArchitecture } from "./knowledge-CCX7oQzQ.js";
+import { a as checkKnowledgeFreshnessFromIndex, d as smrtReview, f as REVIEW_SKILL_NAME, i as checkKnowledgeFreshness, n as buildKnowledgeIndex, p as TOOLS, r as buildReviewContext, s as packageDocPaths, t as buildArchitectureContext, u as smrtArchitecture } from "./knowledge-A7-gIjW4.js";
 import { existsSync, readFileSync, realpathSync } from "node:fs";
 import { dirname, isAbsolute, join, relative, resolve, sep } from "node:path";
 import { fileURLToPath, pathToFileURL } from "node:url";
@@ -1061,7 +1061,6 @@ function pathExistsSyncHint(path) {
 var SERVER_NAME = "smrt-dev-mcp";
 var SERVER_VERSION = readPackageVersion();
 var DEBUG = process.env.DEBUG === "true";
-var REVIEW_SKILL_NAME = "smrt-code-review";
 var REVIEW_SKILL_URI = `smrt-dev-mcp://agent-skills/${REVIEW_SKILL_NAME}`;
 var DOMAIN_CODE_REVIEW_PROMPT = "domain-code-review";
 var DOMAIN_ARCHITECTURE_PROMPT = "domain-architecture";
@@ -1075,504 +1074,6 @@ var LIVE_KNOWLEDGE_CACHE_HINT = {
 	ttlMs: 0,
 	cacheScope: "private"
 };
-var JSON_SCHEMA_2020_12 = "https://json-schema.org/draft/2020-12/schema";
-function compareToolNames(left, right) {
-	return left < right ? -1 : left > right ? 1 : 0;
-}
-/**
-* Stable success/error envelope for development tools. `data` preserves each
-* tool's existing payload exactly; coverage/diagnostics are promoted only when
-* the underlying result actually reports them.
-*/
-var DEV_MCP_OUTPUT_SCHEMA = {
-	$schema: JSON_SCHEMA_2020_12,
-	type: "object",
-	additionalProperties: false,
-	required: [
-		"ok",
-		"coverage",
-		"diagnostics",
-		"data"
-	],
-	properties: {
-		ok: { type: "boolean" },
-		coverage: {
-			type: ["object", "null"],
-			additionalProperties: true
-		},
-		diagnostics: {
-			type: "array",
-			items: {
-				type: "object",
-				additionalProperties: true
-			}
-		},
-		data: {}
-	}
-};
-var TOOLS = [
-	{
-		name: "generate-smrt-class",
-		description: "Generate a complete SMRT class with @smrt() decorator",
-		inputSchema: {
-			type: "object",
-			properties: {
-				className: {
-					type: "string",
-					description: "Name of the class (PascalCase)"
-				},
-				properties: {
-					type: "array",
-					description: "Array of property definitions",
-					items: {
-						type: "object",
-						properties: {
-							name: { type: "string" },
-							type: {
-								type: "string",
-								enum: [
-									"text",
-									"integer",
-									"decimal",
-									"boolean",
-									"datetime",
-									"json"
-								]
-							},
-							required: { type: "boolean" },
-							nullable: { type: "boolean" },
-							description: { type: "string" },
-							defaultValue: { oneOf: [
-								{ type: "string" },
-								{ type: "number" },
-								{ type: "boolean" },
-								{ type: "object" },
-								{ type: "null" }
-							] }
-						},
-						required: ["name", "type"]
-					}
-				},
-				baseClass: {
-					type: "string",
-					enum: ["SmrtObject", "SmrtCollection"],
-					default: "SmrtObject"
-				},
-				template: {
-					type: "string",
-					enum: [
-						"basic",
-						"global-catalog",
-						"optional-catalog",
-						"tenant-project-object",
-						"tenant-event-log-object",
-						"cross-package-reference"
-					],
-					default: "basic"
-				},
-				tableName: { type: "string" },
-				conflictColumns: {
-					type: "array",
-					items: { type: "string" }
-				},
-				tenantScoped: { oneOf: [{ type: "boolean" }, {
-					type: "object",
-					properties: {
-						mode: {
-							type: "string",
-							enum: ["required", "optional"]
-						},
-						field: { type: "string" },
-						autoFilter: { type: "boolean" },
-						autoPopulate: { type: "boolean" },
-						allowSuperAdminBypass: { type: "boolean" }
-					}
-				}] },
-				includeTenantIdField: { type: "boolean" },
-				relationships: {
-					type: "array",
-					items: {
-						type: "object",
-						properties: {
-							name: { type: "string" },
-							type: {
-								type: "string",
-								enum: [
-									"foreignKey",
-									"crossPackageRef",
-									"oneToMany",
-									"manyToMany"
-								]
-							},
-							related: { type: "string" },
-							required: { type: "boolean" },
-							nullable: { type: "boolean" },
-							description: { type: "string" },
-							validate: { type: "boolean" },
-							foreignKey: { type: "string" },
-							through: { type: "string" },
-							sourceKey: { type: "string" },
-							targetKey: { type: "string" }
-						},
-						required: [
-							"name",
-							"type",
-							"related"
-						]
-					}
-				},
-				includeCompanionSnippets: {
-					type: "boolean",
-					default: false
-				},
-				includeApiConfig: {
-					type: "boolean",
-					default: true
-				},
-				includeMcpConfig: {
-					type: "boolean",
-					default: true
-				},
-				includeCliConfig: {
-					type: "boolean",
-					default: true
-				}
-			},
-			required: ["className", "properties"]
-		}
-	},
-	{
-		name: "introspect-project",
-		description: "Scan a directory for SMRT objects. Returns a compact summary by default; pass detail: \"full\" for field, schema, and method details.",
-		inputSchema: {
-			type: "object",
-			properties: {
-				directory: {
-					type: "string",
-					description: "Project directory (default: cwd)"
-				},
-				manifestPath: {
-					type: "string",
-					description: "Optional manifest path. Defaults to .smrt/manifest.json, dist/manifest.json, then source scanning."
-				},
-				detail: {
-					type: "string",
-					enum: ["summary", "full"],
-					default: "summary",
-					description: "summary: one compact record per object. full: complete field/schema/method detail (large)."
-				},
-				maxChars: {
-					type: "number",
-					description: "Character budget for the `objects` payload. Objects past the budget are omitted and reported under `truncated`. Project metadata and diagnostics are always returned in full, so the serialized response is somewhat larger than this budget."
-				},
-				includeFields: {
-					type: "boolean",
-					description: "Include field details (detail: \"full\" only)"
-				},
-				includeRelationships: {
-					type: "boolean",
-					description: "Analyze relationships (detail: \"full\" only)"
-				},
-				includeMethods: {
-					type: "boolean",
-					description: "Include public method details (detail: \"full\" only)"
-				}
-			}
-		}
-	},
-	{
-		name: "review-smrt-project",
-		description: "Advisory ecosystem alignment review for downstream SMRT projects",
-		inputSchema: {
-			type: "object",
-			properties: {
-				directory: {
-					type: "string",
-					description: "Project directory (default: cwd)"
-				},
-				rootDir: {
-					type: "string",
-					description: "Compatibility alias for directory"
-				},
-				includeSourceEvidence: {
-					type: "boolean",
-					description: "Include file and line evidence in findings",
-					default: true
-				},
-				maxFindings: {
-					type: "number",
-					description: "Optional maximum number of findings to return"
-				}
-			}
-		}
-	},
-	{
-		name: "reflect-knowledge",
-		description: "Report deterministic SMRT + HappyVertical SDK knowledge coverage and freshness",
-		inputSchema: {
-			type: "object",
-			properties: { rootDir: {
-				type: "string",
-				description: "Project root directory (default: cwd)"
-			} }
-		}
-	},
-	{
-		name: "reflect-domain-knowledge",
-		description: "Report domain-scoped SMRT knowledge artifacts, SDK packages, and freshness",
-		inputSchema: {
-			type: "object",
-			properties: {
-				rootDir: { type: "string" },
-				scope: {
-					type: "string",
-					enum: [
-						"project",
-						"local",
-						"package",
-						"sdk",
-						"installed"
-					],
-					default: "project"
-				},
-				package: { type: "string" }
-			}
-		}
-	},
-	{
-		name: "check-knowledge-freshness",
-		description: "Run deterministic freshness checks for SMRT agent knowledge",
-		inputSchema: {
-			type: "object",
-			properties: {
-				rootDir: { type: "string" },
-				changed: {
-					type: "boolean",
-					description: "Limit stale-pattern checks to changed files"
-				},
-				strict: {
-					type: "boolean",
-					description: "Treat stale-pattern findings as errors"
-				}
-			}
-		}
-	},
-	{
-		name: "check-domain-knowledge",
-		description: "Run deterministic freshness checks for domain knowledge artifacts",
-		inputSchema: {
-			type: "object",
-			properties: {
-				rootDir: { type: "string" },
-				changed: { type: "boolean" },
-				strict: { type: "boolean" },
-				scope: {
-					type: "string",
-					enum: [
-						"project",
-						"local",
-						"package",
-						"sdk",
-						"installed"
-					],
-					default: "project"
-				},
-				package: { type: "string" }
-			}
-		}
-	},
-	{
-		name: "build-review-context",
-		description: "Build model-ready SMRT review context from changed files and optional focus text",
-		inputSchema: {
-			type: "object",
-			properties: {
-				rootDir: { type: "string" },
-				changedFiles: {
-					type: "array",
-					items: { type: "string" }
-				},
-				focus: { type: "string" },
-				documentation: { type: "string" },
-				detail: {
-					type: "string",
-					enum: ["summary", "full"],
-					default: "summary",
-					description: "summary: authored package docs are listed by path to stay inside tool-result budgets. full: embed them."
-				}
-			}
-		}
-	},
-	{
-		name: "build-domain-review-context",
-		description: "Build domain-scoped model-ready SMRT review context and prompt bundle",
-		inputSchema: {
-			type: "object",
-			properties: {
-				rootDir: { type: "string" },
-				changedFiles: {
-					type: "array",
-					items: { type: "string" }
-				},
-				focus: { type: "string" },
-				documentation: { type: "string" },
-				scope: {
-					type: "string",
-					enum: [
-						"project",
-						"local",
-						"package",
-						"sdk",
-						"installed"
-					],
-					default: "project"
-				},
-				package: { type: "string" },
-				detail: {
-					type: "string",
-					enum: ["summary", "full"],
-					default: "summary",
-					description: "summary: authored package docs are listed by path to stay inside tool-result budgets. full: embed them."
-				}
-			}
-		}
-	},
-	{
-		name: "smrt-review",
-		description: "Return deterministic review findings and/or a reusable model prompt bundle. For a formal downstream review, first call get-agent-skill with { \"name\": \"smrt-code-review\" } or load the smrt-code-review MCP prompt/resource.",
-		inputSchema: {
-			type: "object",
-			properties: {
-				rootDir: { type: "string" },
-				changedFiles: {
-					type: "array",
-					items: { type: "string" }
-				},
-				focus: { type: "string" },
-				documentation: { type: "string" },
-				mode: {
-					type: "string",
-					enum: [
-						"findings",
-						"prompt-bundle",
-						"both"
-					],
-					default: "both"
-				},
-				detail: {
-					type: "string",
-					enum: ["summary", "full"],
-					default: "summary",
-					description: "summary: authored package docs are listed by path to stay inside tool-result budgets. full: embed them."
-				}
-			}
-		}
-	},
-	{
-		name: "build-architecture-context",
-		description: "Build model-ready SMRT architecture context from an idea or documentation",
-		inputSchema: {
-			type: "object",
-			properties: {
-				rootDir: { type: "string" },
-				idea: { type: "string" },
-				documentation: { type: "string" },
-				focus: { type: "string" },
-				detail: {
-					type: "string",
-					enum: ["summary", "full"],
-					default: "summary",
-					description: "summary: authored package docs are listed by path to stay inside tool-result budgets. full: embed them."
-				}
-			}
-		}
-	},
-	{
-		name: "build-domain-architecture-context",
-		description: "Build domain-scoped model-ready SMRT architecture context and prompt bundle",
-		inputSchema: {
-			type: "object",
-			properties: {
-				rootDir: { type: "string" },
-				idea: { type: "string" },
-				documentation: { type: "string" },
-				focus: { type: "string" },
-				scope: {
-					type: "string",
-					enum: [
-						"project",
-						"local",
-						"package",
-						"sdk",
-						"installed"
-					],
-					default: "project"
-				},
-				package: { type: "string" },
-				detail: {
-					type: "string",
-					enum: ["summary", "full"],
-					default: "summary",
-					description: "summary: authored package docs are listed by path to stay inside tool-result budgets. full: embed them."
-				}
-			}
-		}
-	},
-	{
-		name: "smrt-architecture",
-		description: "Suggest SMRT and HappyVertical SDK packages and return an architecture prompt bundle",
-		inputSchema: {
-			type: "object",
-			properties: {
-				rootDir: { type: "string" },
-				idea: { type: "string" },
-				documentation: { type: "string" },
-				focus: { type: "string" },
-				detail: {
-					type: "string",
-					enum: ["summary", "full"],
-					default: "summary",
-					description: "summary: authored package docs are listed by path to stay inside tool-result budgets. full: embed them."
-				}
-			}
-		}
-	},
-	{
-		name: "list-agent-skills",
-		description: "List bundled harness-agnostic agent skills shipped with smrt-dev-mcp",
-		inputSchema: {
-			type: "object",
-			properties: {}
-		}
-	},
-	{
-		name: "get-agent-skill",
-		description: "Return a bundled harness-agnostic agent skill as Markdown plus optional references",
-		inputSchema: {
-			type: "object",
-			properties: {
-				name: {
-					type: "string",
-					enum: [REVIEW_SKILL_NAME],
-					description: "Bundled agent skill name"
-				},
-				includeReferences: {
-					type: "boolean",
-					default: true,
-					description: "Include referenced files with the skill bundle"
-				}
-			},
-			required: ["name"]
-		}
-	}
-].map((tool) => ({
-	...tool,
-	inputSchema: {
-		...tool.inputSchema,
-		$schema: JSON_SCHEMA_2020_12
-	},
-	outputSchema: DEV_MCP_OUTPUT_SCHEMA
-})).sort((left, right) => compareToolNames(left.name, right.name));
 function createServer() {
 	if (DEBUG) console.error(`[${SERVER_NAME}] Starting server v${SERVER_VERSION}`);
 	const server = new Server({
@@ -1680,7 +1181,7 @@ function createServer() {
 	});
 	server.setRequestHandler("prompts/get", async (request) => {
 		const { name } = request.params;
-		if (name === REVIEW_SKILL_NAME) return {
+		if (name === "smrt-code-review") return {
 			description: "Use this procedure when reviewing downstream SMRT projects.",
 			messages: [{
 				role: "user",