@happyvertical/smrt-dev-mcp 0.40.62 → 0.40.64

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`:
@@ -44,6 +48,36 @@ amplify into unbounded filesystem work. Discovery also stops before package
 reads when more than 512 packages match, revalidates confinement immediately
 before reading each package, and runs at most eight scanner fallbacks at once.
 
+### Consumer apps (#2275)
+
+An app that installs the published packages authors none of them, so workspace
+discovery alone sees nothing. Every installed `@happyvertical/smrt-*` package,
+plus the known SDK packages, is resolved from the project's and each workspace
+package's `node_modules/@happyvertical` scope directory and marked
+`isInstalledDependency`. This is an enumeration, not a walk: pnpm materializes
+a store whose entries link back out to their siblings, so a descent reads the
+same package once per path that reaches it. Real paths deduplicate the result;
+each package is still read through its `node_modules` path, because a realpath
+escapes the root wherever an ancestor is a symlink. A workspace package linked
+into a sibling's `node_modules` is authored source and is never marked
+installed.
+
+- `--scope installed` reports exactly those packages. `local` and `package`
+  exclude them; `project` includes them; the index also lists them separately
+  as `installedPackages` (`schemaVersion: 3`). It is a reporting scope:
+  `dev:knowledge-check` has no authored packages to gate under it.
+- Each package carries `agentDocSha256`, the hash of its shipped `AGENTS.md`.
+  That is the drift signal a consumer diffs against a recorded baseline — a
+  version bump is not, because most releases do not touch a documented surface.
+- The freshness gate skips installed packages entirely: indexed, never checked.
+  A consumer cannot add an `AGENTS.md` to a dependency, and a published
+  package's artifact hashes cannot match its republished sources.
+- Coverage and the `no-smrt-objects-discovered` guard count authored packages
+  only. Dependencies ship their own objects, so counting them would make that
+  guard unreachable in exactly the projects most likely to have broken
+  discovery. A project whose own packages contribute nothing while its
+  dependencies do gets the `no-authored-smrt-objects` warning instead.
+
 Per package, objects resolve in this order, and the winner is recorded in
 `objectSource` with a reason:
 
@@ -52,7 +86,7 @@ Per package, objects resolve in this order, and the winner is recorded in
 3. `scanner` — `OxcScanner` + `ManifestAdapter` over the package's sources
 4. `none` — with a machine-readable reason
 
-The index exposes `coverage` and `diagnostics` (`schemaVersion: 2`). Zero
+The index exposes `coverage` and `diagnostics` (added in `schemaVersion: 2`). Zero
 discovered objects is an **error-grade** diagnostic that names the roots and
 artifact paths checked plus the fix, and it propagates into
 `smrt-architecture`/`smrt-review`/`reflect-*` results and prompt bundles.

package/README.md

@@ -124,7 +124,7 @@ dependency edge — by connected component, keeping the copy the others depend o
 and reports `duplicate-object-identity`. Unrelated packages that share a class
 name stay distinct.
 
-The index therefore carries two extra blocks (`schemaVersion: 2`):
+The index therefore carries two extra blocks (added in `schemaVersion: 2`):
 
 - `coverage` — `workspaceGlobs`, `workspaceGlobSource`, `packageDirs`,
   `packagesWithObjects`, and `packagesWithoutObjects` with a reason, the artifact
@@ -134,6 +134,13 @@ The index therefore carries two extra blocks (`schemaVersion: 2`):
   commands to fix it, so an unseen project is never reported as a project with no
   model. `smrt-architecture`, `smrt-review`, and the `reflect-*` tools surface it.
 
+`schemaVersion: 3` adds the consumer-app view: `installedPackages` lists every
+installed `@happyvertical/smrt-*` package plus the known SDK packages — the ones
+the project installs rather than authors — each with `isInstalledDependency`, its
+version, and `agentDocSha256`, the hash of its shipped `AGENTS.md`. That hash is
+what a downstream project diffs against a recorded baseline to find documentation
+drift. Reach it with `--scope installed`.
+
 ## Response Budgets
 
 Knowledge and introspection tools return a **summary** by default and accept
@@ -235,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) |
@@ -300,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`, or `sdk` |
+| `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`
@@ -323,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`, or `sdk` |
+| `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`
@@ -337,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`
 
@@ -353,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`
 
@@ -366,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`
 
@@ -396,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,CAgfrC"}
+{"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"}