@happyvertical/smrt-cli 0.40.61 → 0.40.63

package/AGENTS.md

@@ -15,10 +15,13 @@ smrt db:rollback             # Rollback migrations
 smrt docs:agents             # Generate .agents/smrt-framework.md
 smrt docs:claude             # Deprecated alias writing .claude/smrt-framework.md
 smrt dev:knowledge-*         # Deterministic agent knowledge index/check/diff
+smrt dev:knowledge-index --scope installed # Installed @happyvertical/* deps, versions, AGENTS.md hashes
 smrt knowledge:review-context --scope package --package <name>
-smrt knowledge:architecture-context --scope project|local|package|sdk
+smrt knowledge:architecture-context --scope project|local|package|sdk|installed
 smrt dev:knowledge-check --format markdown|json
-smrt generate:mcp            # Generate MCP server
+smrt generate-mcp            # Generate MCP server (aliases: generate-mcp-server, mcp)
+smrt generate-types          # Generate TypeScript declarations (alias: generate-declarations)
+smrt generate-routes         # Generate SvelteKit API routes (aliases: routes, generate:routes)
 smrt config:export           # Export agent config for SSG
 smrt init                    # Init new project
 smrt gnode                   # Scaffold gnode site
@@ -50,4 +53,13 @@ migrations are manifest-driven through registered objects and project manifests.
 
 - **Test mode detection**: checks `NODE_ENV=test`, `VITEST=true`, `global.it`/`describe` — could conflict with other test runners
 - **External package load failures silenced**: one package failing doesn't prevent others from loading
+- **Generation command names are hyphenated**: `generate-mcp` and `generate-types` have
+  no colon alias. Only `generate-routes` and `generate-register` declare one, so
+  `smrt generate:mcp` is an unknown command (#2279).
+- **Subcommand `--version` beats the global flag**: `parseCliArgs` in
+  `@happyvertical/utils` dispatches to the built-in `version` command for a
+  `--version` token anywhere in argv. `parseCliCommandArgs` rescopes that token to
+  the named subcommand (`smrt generate-mcp --version 0.1.0`) and only leaves it
+  global when it appears before or without a subcommand (#2279). `--help` is
+  deliberately untouched.
 - **Schema history nuance**: `db:status` / `db:history` should distinguish active live drift from superseded failed generated schema repairs instead of treating all failed rows as current blockers

package/README.md

@@ -83,8 +83,30 @@ the tracker or writing to the database.
 
 | Command | Description |
 |---------|------------|
-| `smrt generate:mcp` | Generate MCP server from registered objects |
-| `smrt generate:types` | Generate TypeScript declarations from manifest |
+| `smrt generate-mcp` | Generate MCP server from registered objects (aliases: `generate-mcp-server`, `mcp`) |
+| `smrt generate-types` | Generate TypeScript declarations from manifest (alias: `generate-declarations`) |
+| `smrt generate-routes` | Generate SvelteKit API routes (aliases: `routes`, `generate:routes`) |
+| `smrt generate-register` | Generate `.smrt/register.js` from discovered packages (aliases: `register`, `generate:register`) |
+
+Generation commands are hyphenated. `generate-routes` and `generate-register`
+also answer to a colon alias for backward compatibility; `generate-mcp` and
+`generate-types` do not.
+
+`smrt generate-mcp` writes `.smrt/mcp-server/index.js` by default and emits
+JavaScript for a `.js` target, so `node .smrt/mcp-server/index.js` runs it
+directly. Ask for a `.ts` target to keep the annotated TypeScript for `tsx` or
+Node's type stripping:
+
+```bash
+smrt generate-mcp --output-path .smrt/mcp-server/index.ts
+```
+
+The generated server is always an ES module, so a `.cjs`/`.cts` output path is
+rejected. It imports `@modelcontextprotocol/server`, `@happyvertical/smrt-core`,
+and `@happyvertical/smrt-config` at runtime (plus `@happyvertical/smrt-jobs` for
+task actions and `@happyvertical/smrt-tenancy` for tenant-scoped objects), so
+those have to be resolvable from the project you run it in — under pnpm's strict
+layout that means declaring them, not relying on the CLI's own dependencies.
 
 ### Documentation
 
@@ -95,8 +117,8 @@ the tracker or writing to the database.
 | `smrt dev:knowledge-index --format markdown\|json` | Print the deterministic s-m-r-t + SDK knowledge index |
 | `smrt dev:knowledge-check --format markdown\|json` | Check agent knowledge freshness |
 | `smrt dev:knowledge-diff --format markdown\|json` | Show changed files and affected package experts |
-| `smrt knowledge:review-context --scope project\|local\|package\|sdk --package <name> --format markdown\|json` | Build a model-ready domain review prompt bundle |
-| `smrt knowledge:architecture-context --scope project\|local\|package\|sdk --package <name> --format markdown\|json` | Build a model-ready domain architecture prompt bundle |
+| `smrt knowledge:review-context --scope project\|local\|package\|sdk\|installed --package <name> --format markdown\|json` | Build a model-ready domain review prompt bundle |
+| `smrt knowledge:architecture-context --scope project\|local\|package\|sdk\|installed --package <name> --format markdown\|json` | Build a model-ready domain architecture prompt bundle |
 
 ### Configuration
 
@@ -162,7 +184,7 @@ Custom methods on s-m-r-t objects are auto-discovered from manifests. Method par
 smrt introspect
 
 # Generate an MCP server
-smrt generate:mcp
+smrt generate-mcp
 
 # Scaffold a package playground definition
 smrt playground init

package/dist/{commands-CB3xtl7p.js → commands-Bkdk2-S5.js}

@@ -3157,7 +3157,7 @@ var generateCommands = {
 			},
 			modular: {
 				type: "boolean",
-				description: "Generate modular directory structure (tools/, handlers/, config.ts)",
+				description: "Generate modular directory structure (tools/, handlers/, config)",
 				default: false
 			},
 			debug: {
@@ -5758,7 +5758,7 @@ var devKnowledgeCommands = {
 			},
 			scope: {
 				type: "string",
-				description: "Knowledge scope: project, local, package, or sdk",
+				description: "Knowledge scope: project, local, package, sdk, or installed",
 				default: "project"
 			},
 			package: {
@@ -5807,7 +5807,7 @@ var devKnowledgeCommands = {
 			},
 			scope: {
 				type: "string",
-				description: "Knowledge scope: project, local, package, or sdk",
+				description: "Knowledge scope: project, local, package, sdk, or installed",
 				default: "project"
 			},
 			package: {
@@ -5852,7 +5852,7 @@ var devKnowledgeCommands = {
 			},
 			scope: {
 				type: "string",
-				description: "Knowledge scope: project, local, package, or sdk",
+				description: "Knowledge scope: project, local, package, sdk, or installed",
 				default: "project"
 			},
 			package: {
@@ -5935,7 +5935,7 @@ function contextOptions(defaultFormat) {
 		},
 		scope: {
 			type: "string",
-			description: "Knowledge scope: project, local, package, or sdk",
+			description: "Knowledge scope: project, local, package, sdk, or installed",
 			default: "project"
 		},
 		package: {

package/dist/index.js

@@ -1,6 +1,6 @@
 #!/usr/bin/env node
 import { readFileSync } from "node:fs";
-import { dirname, join } from "node:path";
+import { basename, dirname, join } from "node:path";
 import { createInterface } from "node:readline";
 import { fileURLToPath } from "node:url";
 import { ObjectRegistry, getPackageFromQualifiedName } from "@happyvertical/smrt-core";
@@ -48,56 +48,56 @@ var _docsCommands = null;
 var _playgroundCommands = null;
 async function getGnodeCommands() {
 	if (!_gnodeCommands) {
-		const { gnodeCommands } = await import("./commands-CB3xtl7p.js");
+		const { gnodeCommands } = await import("./commands-Bkdk2-S5.js");
 		_gnodeCommands = gnodeCommands;
 	}
 	return _gnodeCommands;
 }
 async function getGitCommands() {
 	if (!_gitCommands) {
-		const { gitCommands } = await import("./commands-CB3xtl7p.js");
+		const { gitCommands } = await import("./commands-Bkdk2-S5.js");
 		_gitCommands = gitCommands;
 	}
 	return _gitCommands;
 }
 async function getGenerateCommands() {
 	if (!_generateCommands) {
-		const { generateCommands } = await import("./commands-CB3xtl7p.js");
+		const { generateCommands } = await import("./commands-Bkdk2-S5.js");
 		_generateCommands = generateCommands;
 	}
 	return _generateCommands;
 }
 async function getInitCommands() {
 	if (!_initCommands) {
-		const { initCommands } = await import("./commands-CB3xtl7p.js");
+		const { initCommands } = await import("./commands-Bkdk2-S5.js");
 		_initCommands = initCommands;
 	}
 	return _initCommands;
 }
 async function getUtilityCommands() {
 	if (!_utilityCommands) {
-		const { utilityCommands } = await import("./commands-CB3xtl7p.js");
+		const { utilityCommands } = await import("./commands-Bkdk2-S5.js");
 		_utilityCommands = utilityCommands;
 	}
 	return _utilityCommands;
 }
 async function getDispatchCommands() {
 	if (!_dispatchCommands) {
-		const { dispatchCommands } = await import("./commands-CB3xtl7p.js");
+		const { dispatchCommands } = await import("./commands-Bkdk2-S5.js");
 		_dispatchCommands = dispatchCommands;
 	}
 	return _dispatchCommands;
 }
 async function getDocsCommands() {
 	if (!_docsCommands) {
-		const { docsCommands } = await import("./commands-CB3xtl7p.js");
+		const { docsCommands } = await import("./commands-Bkdk2-S5.js");
 		_docsCommands = docsCommands;
 	}
 	return _docsCommands;
 }
 async function getPlaygroundCommands() {
 	if (!_playgroundCommands) {
-		const { playgroundCommands } = await import("./commands-CB3xtl7p.js");
+		const { playgroundCommands } = await import("./commands-Bkdk2-S5.js");
 		_playgroundCommands = playgroundCommands;
 	}
 	return _playgroundCommands;
@@ -122,11 +122,129 @@ function collectRepeatableOptionValues(argv, optionName) {
 	return values;
 }
 /**
+* Option name a subcommand may declare that collides with a global CLI flag.
+*
+* `parseCliArgs` dispatches to the built-in `version` command whenever
+* `--version` appears anywhere in argv, so `smrt generate-mcp --version 0.1.0`
+* never reached the subcommand and exited without generating anything (#2279).
+* Only `--version` is rescoped: `--help` after a subcommand still prints help,
+* which is what someone typing it wants.
+*/
+var SCOPED_GLOBAL_FLAG = "version";
+/**
+* Private option name the scoped flag is parsed under. Renaming the token is
+* what keeps `parseCliArgs` from intercepting it; the value is mapped back onto
+* the declared name before any handler sees it.
+*/
+var SCOPED_GLOBAL_FLAG_ALIAS = "smrt-scoped-version";
+/**
+* Index of the first real argument, mirroring the node/script-path stripping
+* `parseCliArgs` does. The CLI entry point passes `process.argv.slice(2)`, but
+* the shared parser also accepts a full `process.argv`, and this wrapper must
+* read the same tokens the parser will.
+*/
+function firstArgumentIndex(argv) {
+	let index = 0;
+	if (argv.length > index && basename(argv[index]) === "node") index += 1;
+	if (argv.length > index && argv[index].endsWith(".js")) index += 1;
+	return index;
+}
+/**
+* Resolve the command a plain argv addresses, mirroring the multi-word command
+* matching in `parseCliArgs` (up to three leading words, names or aliases).
+*
+* @returns The matched command, if any, and how many leading argv tokens it consumed
+*/
+function matchLeadingCommand(argv, commands, builtInCommands) {
+	for (let width = Math.min(3, argv.length); width > 0; width -= 1) {
+		const candidateName = argv.slice(0, width).join(" ");
+		const found = builtInCommands[candidateName] ?? commands.find((candidate) => candidate.name === candidateName || candidate.aliases?.includes(candidateName));
+		if (found) return {
+			command: found,
+			wordCount: width
+		};
+	}
+	return { wordCount: 0 };
+}
+/**
+* Rewrite a subcommand-scoped `--version` token to a private alias so the
+* shared parser routes it to the subcommand instead of the global version
+* command. Tokens before the subcommand — and a bare `smrt --version` — are
+* left alone, so the global flag still wins when it is used globally.
+*
+* @returns Rewritten argv and the matched command, or `null` when nothing applies
+*/
+function rescopeGlobalFlag(argv, commands, builtInCommands) {
+	const start = firstArgumentIndex(argv);
+	const leading = argv[start];
+	if (!leading || leading.startsWith("-")) return null;
+	const flag = `--${SCOPED_GLOBAL_FLAG}`;
+	const inlinePrefix = `${flag}=`;
+	const aliasFlag = `--${SCOPED_GLOBAL_FLAG_ALIAS}`;
+	const { command, wordCount } = matchLeadingCommand(argv.slice(start), commands, builtInCommands);
+	const firstOptionIndex = start + Math.max(wordCount, 1);
+	let rewritten = false;
+	const rescoped = [...argv];
+	for (let index = firstOptionIndex; index < rescoped.length; index += 1) {
+		const token = rescoped[index];
+		if (token === "--") break;
+		if (token === flag) {
+			rescoped[index] = aliasFlag;
+			rewritten = true;
+			continue;
+		}
+		if (token.startsWith(inlinePrefix)) {
+			rescoped[index] = `${aliasFlag}=${token.slice(inlinePrefix.length)}`;
+			rewritten = true;
+		}
+	}
+	return rewritten ? {
+		argv: rescoped,
+		command
+	} : null;
+}
+/**
+* Swap a matched command's declared `version` option for the private alias the
+* rescoped token is parsed under, so its type, default, and short flag still
+* apply. Commands that do not declare the option are returned untouched.
+*/
+function declareScopedGlobalFlag(command, commands, builtInCommands) {
+	const declaredOptions = command?.options;
+	const declared = declaredOptions?.[SCOPED_GLOBAL_FLAG];
+	if (!command || !declaredOptions || !declared) return {
+		commands,
+		builtInCommands
+	};
+	const { [SCOPED_GLOBAL_FLAG]: _replaced, ...otherOptions } = declaredOptions;
+	const rewrittenCommand = {
+		...command,
+		options: {
+			...otherOptions,
+			[SCOPED_GLOBAL_FLAG_ALIAS]: { ...declared }
+		}
+	};
+	return {
+		commands: commands.map((candidate) => candidate === command ? rewrittenCommand : candidate),
+		builtInCommands: Object.fromEntries(Object.entries(builtInCommands).map(([name, candidate]) => [name, candidate === command ? rewrittenCommand : candidate]))
+	};
+}
+/**
 * Parse one command while preserving every occurrence of options marked
-* `multiple`. Non-repeatable options retain the shared SDK parser behavior.
+* `multiple`, and while letting a subcommand's own options win over the
+* global flags the shared parser intercepts. Everything else retains the
+* shared SDK parser behavior.
 */
 function parseCliCommandArgs(argv, commands, builtInCommands = {}) {
-	const parsed = parseCliArgs(argv, commands, builtInCommands);
+	const rescoped = rescopeGlobalFlag(argv, commands, builtInCommands);
+	let parsed;
+	if (rescoped) {
+		const declared = declareScopedGlobalFlag(rescoped.command, commands, builtInCommands);
+		parsed = parseCliArgs(rescoped.argv, declared.commands, declared.builtInCommands);
+		if (SCOPED_GLOBAL_FLAG_ALIAS in parsed.options) {
+			parsed.options[SCOPED_GLOBAL_FLAG] = parsed.options[SCOPED_GLOBAL_FLAG_ALIAS];
+			delete parsed.options[SCOPED_GLOBAL_FLAG_ALIAS];
+		}
+	} else parsed = parseCliArgs(argv, commands, builtInCommands);
 	const command = parsed.command ? builtInCommands[parsed.command] ?? commands.find((candidate) => candidate.name === parsed.command || candidate.aliases?.includes(parsed.command)) : void 0;
 	if (!command?.options) return parsed;
 	for (const [name, option] of Object.entries(command.options)) {

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@happyvertical/smrt-cli",
-  "version": "0.40.61",
+  "version": "0.40.63",
   "description": "Developer CLI for SMRT framework - introspection, testing, and project management",
   "type": "module",
   "main": "./dist/index.js",
@@ -24,20 +24,20 @@
     }
   },
   "dependencies": {
-    "@happyvertical/ai": "^0.85.5",
-    "@happyvertical/files": "^0.85.5",
-    "@happyvertical/logger": "^0.85.5",
-    "@happyvertical/sql": "^0.85.5",
-    "@happyvertical/utils": "^0.85.5",
+    "@happyvertical/ai": "^0.86.1",
+    "@happyvertical/files": "^0.86.1",
+    "@happyvertical/logger": "^0.86.1",
+    "@happyvertical/sql": "^0.86.1",
+    "@happyvertical/utils": "^0.86.1",
     "acorn": "^8.17.0",
     "fast-glob": "3.3.3",
     "tar": "^7.5.19",
-    "@happyvertical/smrt-agents": "0.40.61",
-    "@happyvertical/smrt-config": "0.40.61",
-    "@happyvertical/smrt-core": "0.40.61",
-    "@happyvertical/smrt-dev-mcp": "0.40.61",
-    "@happyvertical/smrt-playground": "0.40.61",
-    "@happyvertical/smrt-types": "0.40.61"
+    "@happyvertical/smrt-agents": "0.40.63",
+    "@happyvertical/smrt-config": "0.40.63",
+    "@happyvertical/smrt-core": "0.40.63",
+    "@happyvertical/smrt-dev-mcp": "0.40.63",
+    "@happyvertical/smrt-types": "0.40.63",
+    "@happyvertical/smrt-playground": "0.40.63"
   },
   "devDependencies": {
     "@types/node": "24.13.2",