@happyvertical/smrt-dev-mcp 0.40.37 → 0.40.38

package/dist/index.js

@@ -1,14 +1,14 @@
 #!/usr/bin/env node
-import { a as checkKnowledgeFreshnessFromIndex, i as checkKnowledgeFreshness, l as smrtArchitecture, n as buildKnowledgeIndex, r as buildReviewContext, t as buildArchitectureContext, u as smrtReview } from "./knowledge-BPbgCtVJ.js";
+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-DUUhTM5u.js";
 import { existsSync, readFileSync, realpathSync } from "node:fs";
 import { dirname, isAbsolute, join, relative, resolve, sep } from "node:path";
 import { fileURLToPath, pathToFileURL } from "node:url";
 import { Server } from "@modelcontextprotocol/sdk/server/index.js";
 import { StdioServerTransport } from "@modelcontextprotocol/sdk/server/stdio.js";
 import { CallToolRequestSchema, ErrorCode, GetPromptRequestSchema, ListPromptsRequestSchema, ListResourcesRequestSchema, ListToolsRequestSchema, McpError, ReadResourceRequestSchema } from "@modelcontextprotocol/sdk/types.js";
+import { ManifestAdapter, OxcScanner } from "@happyvertical/smrt-scanner";
 import { access, readFile, readdir } from "node:fs/promises";
 import { ManifestGenerator } from "@happyvertical/smrt-core/scanner";
-import { ManifestAdapter, OxcScanner } from "@happyvertical/smrt-scanner";
 //#region src/agent-skills.ts
 var AGENT_SKILLS = [{
 	name: "smrt-code-review",
@@ -255,8 +255,20 @@ var RELATIONSHIP_TYPES = /* @__PURE__ */ new Set([
 	"oneToMany",
 	"manyToMany"
 ]);
+/**
+* Response budget in characters. Generous enough that a 42-object summary never
+* truncates, while still capping a runaway payload instead of emitting one.
+*/
+var DEFAULT_MAX_CHARS = 5e4;
+var DEFAULT_MCP_OPERATIONS = [
+	"list",
+	"get",
+	"create",
+	"update",
+	"delete"
+];
 async function introspectProject(args) {
-	const { directory = process.cwd(), includeFields = true, includeRelationships = true, includeMethods = true } = args;
+	const { directory = process.cwd(), includeFields = true, includeRelationships = true, includeMethods = true, detail = "summary", maxChars = DEFAULT_MAX_CHARS } = args;
 	const projectPath = resolve(directory);
 	if (!await pathExists$1(projectPath)) return JSON.stringify({
 		projectPath,
@@ -271,7 +283,13 @@ async function introspectProject(args) {
 	const packageMetadata = await readPackageMetadata(projectPath);
 	const tenantScopes = await scanTenantScopes(projectPath);
 	const manifestResult = await loadManifestArtifact(projectPath, args.manifestPath) ?? await scanSourceManifest(projectPath, packageMetadata);
-	const objects = Object.entries(manifestResult.manifest.objects ?? {}).map(([manifestKey, object]) => formatObject({
+	const entries = Object.entries(manifestResult.manifest.objects ?? {});
+	const objects = detail === "summary" ? entries.map(([manifestKey, object]) => summarizeObject({
+		manifestKey,
+		object,
+		projectPath,
+		tenantScope: tenantScopes.get(object.className)
+	})) : entries.map(([manifestKey, object]) => formatObject({
 		manifestKey,
 		object,
 		projectPath,
@@ -279,21 +297,77 @@ async function introspectProject(args) {
 		includeRelationships,
 		includeMethods,
 		tenantScope: tenantScopes.get(object.className)
-	})).sort((left, right) => left.className.localeCompare(right.className));
+	}));
+	objects.sort((left, right) => left.className.localeCompare(right.className));
+	const { kept, omitted } = applyObjectBudget(objects, maxChars);
 	const output = {
 		projectPath,
 		manifestSource: manifestResult.source,
 		manifestPath: "path" in manifestResult ? relative(projectPath, manifestResult.path) : void 0,
 		packageName: manifestResult.manifest.packageName ?? packageMetadata.name ?? void 0,
 		packageVersion: manifestResult.manifest.packageVersion ?? packageMetadata.version ?? void 0,
+		detail,
 		objectCount: objects.length,
 		scannedFileCount: manifestResult.scannedFileCount,
 		parseTimeMs: manifestResult.parseTimeMs,
-		objects,
+		...omitted > 0 ? { truncated: {
+			returnedObjectCount: kept.length,
+			omittedObjectCount: omitted,
+			budgetChars: maxChars,
+			guidance: "The object list hit its character budget; metadata and diagnostics are still complete. Narrow the scan with `directory` (a single package), keep `detail: \"summary\"`, or raise `maxChars` deliberately. Object names are sorted alphabetically, so omitted objects are the alphabetical tail."
+		} } : {},
+		objects: kept,
 		diagnostics: manifestResult.diagnostics
 	};
 	return JSON.stringify(output, null, 2);
 }
+/**
+* Trims the object list to the character budget, always keeping at least one so
+* a caller sees the shape rather than an empty list.
+*/
+function applyObjectBudget(objects, maxChars) {
+	const kept = [];
+	let used = 0;
+	for (const object of objects) {
+		const size = JSON.stringify(object, null, 2).length + 2;
+		if (used + size > maxChars && kept.length > 0) break;
+		used += size;
+		kept.push(object);
+	}
+	return {
+		kept,
+		omitted: objects.length - kept.length
+	};
+}
+function summarizeObject({ manifestKey, object, projectPath, tenantScope }) {
+	const fields = Object.entries(object.fields ?? {});
+	const decoratorConfig = object.decoratorConfig ?? {};
+	return compactObject({
+		manifestKey,
+		className: object.className,
+		qualifiedName: object.qualifiedName,
+		filePath: sanitizePath$1(projectPath, object.filePath),
+		extends: object.extends,
+		collection: object.collection,
+		tableName: object.schema?.tableName ?? stringFromConfig(decoratorConfig.tableName) ?? object.collection,
+		tenantScope: tenantScope ?? normalizeTenantScopedConfig(decoratorConfig.tenantScoped),
+		fieldCount: fields.length,
+		relationships: fields.filter(([, field]) => RELATIONSHIP_TYPES.has(field.type)).map(([name, field]) => `${name} -> ${field.related ?? ""} (${field.type})`).join(", ") || void 0,
+		mcpOperations: mcpOperationsFromConfig(decoratorConfig.mcp)
+	});
+}
+/**
+* An omitted `mcp` config means full CRUD, not a closed surface — the same rule
+* the knowledge index applies.
+*/
+function mcpOperationsFromConfig(config) {
+	if (config === false) return [];
+	if (typeof config !== "object" || config === null || Array.isArray(config)) return [...DEFAULT_MCP_OPERATIONS];
+	const record = config;
+	const include = Array.isArray(record.include) ? record.include.filter((item) => typeof item === "string") : DEFAULT_MCP_OPERATIONS;
+	const exclude = new Set(Array.isArray(record.exclude) ? record.exclude.filter((item) => typeof item === "string") : []);
+	return include.filter((operation) => !exclude.has(operation));
+}
 async function loadManifestArtifact(projectPath, manifestPath) {
 	const candidates = manifestPath ? [resolve(projectPath, manifestPath)] : DEFAULT_MANIFEST_PATHS.map((candidate) => join(projectPath, candidate));
 	const diagnostics = [];
@@ -1127,7 +1201,7 @@ var TOOLS = [
 	},
 	{
 		name: "introspect-project",
-		description: "Scan current directory for SMRT objects",
+		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: {
@@ -1139,17 +1213,27 @@ var TOOLS = [
 					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"
+					description: "Include field details (detail: \"full\" only)"
 				},
 				includeRelationships: {
 					type: "boolean",
-					description: "Analyze relationships"
+					description: "Analyze relationships (detail: \"full\" only)"
 				},
 				includeMethods: {
 					type: "boolean",
-					description: "Include public method details"
+					description: "Include public method details (detail: \"full\" only)"
 				}
 			}
 		}
@@ -1265,7 +1349,13 @@ var TOOLS = [
 					items: { type: "string" }
 				},
 				focus: { type: "string" },
-				documentation: { 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."
+				}
 			}
 		}
 	},
@@ -1292,7 +1382,13 @@ var TOOLS = [
 					],
 					default: "project"
 				},
-				package: { type: "string" }
+				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."
+				}
 			}
 		}
 	},
@@ -1317,6 +1413,12 @@ var TOOLS = [
 						"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."
 				}
 			}
 		}
@@ -1330,7 +1432,13 @@ var TOOLS = [
 				rootDir: { type: "string" },
 				idea: { type: "string" },
 				documentation: { type: "string" },
-				focus: { 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."
+				}
 			}
 		}
 	},
@@ -1354,7 +1462,13 @@ var TOOLS = [
 					],
 					default: "project"
 				},
-				package: { type: "string" }
+				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."
+				}
 			}
 		}
 	},
@@ -1367,7 +1481,13 @@ var TOOLS = [
 				rootDir: { type: "string" },
 				idea: { type: "string" },
 				documentation: { type: "string" },
-				focus: { 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."
+				}
 			}
 		}
 	},
@@ -1611,6 +1731,8 @@ async function main() {
 						smrtPackageCount: index.smrtPackages.length,
 						sdkPackageCount: index.sdkPackages.length,
 						relationshipsV2: index.relationshipsV2,
+						coverage: index.coverage,
+						diagnostics: index.diagnostics,
 						freshness
 					}, null, 2);
 					break;
@@ -1626,6 +1748,8 @@ async function main() {
 						domainKnowledgePackageCount: index.packages.filter((pkg) => pkg.hasDomainKnowledge).length,
 						missingDomainKnowledgePackages: index.packages.filter((pkg) => pkg.exportKeys.includes("./smrt-knowledge.json") && !pkg.hasDomainKnowledge).map((pkg) => pkg.name),
 						relationshipsV2: index.relationshipsV2,
+						coverage: index.coverage,
+						diagnostics: index.diagnostics,
 						freshness
 					}, null, 2);
 					break;
@@ -1637,22 +1761,18 @@ async function main() {
 					result = JSON.stringify(await checkKnowledgeFreshness(args), null, 2);
 					break;
 				case "build-review-context":
-					result = JSON.stringify(await buildReviewContext(args), null, 2);
-					break;
 				case "build-domain-review-context":
-					result = JSON.stringify(await buildReviewContext(args), null, 2);
+					result = JSON.stringify(compactContextResult(await buildReviewContext(args), detailArg(args)), null, 2);
 					break;
 				case "smrt-review":
-					result = JSON.stringify(await smrtReview(args), null, 2);
+					result = JSON.stringify(compactContextResult(await smrtReview(args), detailArg(args)), null, 2);
 					break;
 				case "build-architecture-context":
-					result = JSON.stringify(await buildArchitectureContext(args), null, 2);
-					break;
 				case "build-domain-architecture-context":
-					result = JSON.stringify(await buildArchitectureContext(args), null, 2);
+					result = JSON.stringify(compactContextResult(await buildArchitectureContext(args), detailArg(args)), null, 2);
 					break;
 				case "smrt-architecture":
-					result = JSON.stringify(await smrtArchitecture(args), null, 2);
+					result = JSON.stringify(compactContextResult(await smrtArchitecture(args), detailArg(args)), null, 2);
 					break;
 				case "list-agent-skills":
 					result = JSON.stringify({ skills: listAgentSkills() }, null, 2);
@@ -1759,10 +1879,56 @@ function sanitizeKnowledgeIndex(index) {
 		sdkPackages: packages.filter((pkg) => pkg.kind === "sdk")
 	};
 }
+/**
+* Compact projection of a context result for MCP callers (#2143).
+*
+* `selectedPackages` carries each package's whole authored `agentDoc`, its
+* module-doc contents, and its full domain manifest. A three-package downstream
+* product serialized to 329,003 characters that way, which is what made these
+* tools unusable as planning aids. `detail: "full"` returns the whole shape.
+*/
+function compactContextResult(result, detail) {
+	const source = result;
+	if (detail === "full") return source;
+	const compact = {
+		...source,
+		detail: "summary"
+	};
+	for (const key of ["selectedPackages", "selectedSdkPackages"]) {
+		const packages = source[key];
+		if (!Array.isArray(packages)) continue;
+		compact[key] = packages.map(compactKnowledgePackage);
+	}
+	return compact;
+}
+function compactKnowledgePackage(pkg) {
+	return {
+		name: pkg.name,
+		version: pkg.version,
+		kind: pkg.kind,
+		directory: pkg.relativeDirectory,
+		objectSource: pkg.objectSource,
+		...pkg.objectSourceReason ? { objectSourceReason: pkg.objectSourceReason } : {},
+		docs: packageDocPaths(pkg),
+		domainKnowledgePath: pkg.domainKnowledgePath,
+		relationshipFeatures: pkg.relationshipFeatures,
+		smrtDependencies: pkg.smrtDependencies,
+		sdkDependencies: pkg.sdkDependencies,
+		exportKeys: pkg.exportKeys,
+		objectCount: pkg.objects.length,
+		objects: pkg.objects.map((object) => object.tableName ? `${object.qualifiedName ?? object.className} (${object.tableName})` : object.qualifiedName ?? object.className),
+		mcpToolCount: pkg.mcpTools.length
+	};
+}
+function detailArg(args) {
+	const detail = args?.detail;
+	return typeof detail === "string" ? detail : void 0;
+}
 function sanitizeKnowledgePackage(pkg) {
-	const { directory: _directory, objects, ...rest } = pkg;
+	const { directory: _directory, objects, checkedObjectPaths, ...rest } = pkg;
 	return {
 		...rest,
+		checkedObjectPaths: checkedObjectPaths.map((path) => sanitizePath(path) ?? path),
 		objects: objects.map((object) => ({
 			...object,
 			filePath: sanitizePath(object.filePath)