@happyvertical/smrt-scanner 0.57.46 → 0.57.48

package/AGENTS.md

@@ -55,7 +55,8 @@ executes the source.
 - `verifyManifestCompleteness({ packageDir })` — publish guard: re-scans `src/`
   and asserts every `@smrt()` object reached `dist/manifest.json` (issue #1483).
   Returns `ok` / `incomplete` / `missing-manifest` / `scan-error` / `skipped`.
-  Driven by `scripts/verify-manifest-completeness.mjs` from `prepack`.
+  Driven by `scripts/verify-manifest-completeness.mjs` (prepack; release
+  `--release` = `requireManifestExport`, #2853).
 - `discoverSourceFiles(options)` — shared bounded source-discovery policy used
   by `OxcScanner` and the core manifest preflight.
 - `extractAgentSurface` / `scanSvelteAgentSurface` / `mergeAgentSurfaces` —
@@ -325,11 +326,11 @@ place, `toKnowledgeAgentSurface` in `vite-plugin/index.ts`.
   access) — `type` still reads `'any'` for compatibility, so that flag is the
   only thing separating it from an authored `any`. `memberTypes` carries the
   members of an INLINE object literal, which `extractTypeName` otherwise
-  flattens to `'object'`; NAMED bags stay unexpanded on purpose. Core's API
-  wire-ability gate fails closed on both (#2686). The bare `object` keyword is
-  resolved for parameters only — `ManifestAdapter.inferFromAnnotation` gives an
-  `object`-typed FIELD a `{}` column default, so naming it in `extractTypeName`
-  would silently change DDL.
+  flattens to `'object'`; NAMED bags stay unexpanded. Core's API
+  wire-ability gate fails closed on both (#2686). Bare `object` resolves for
+  parameters only: `ManifestAdapter.inferFromAnnotation` gives an `object`
+  FIELD a `{}` column default, so naming it in `extractTypeName` would change
+  DDL. Same-module aliases: `resolvedType` (#4327).
 - **`@method()` config is read with `requireLiteralValues`**, like `@smrt()`: an
   unresolvable value is a scan error, because a dropped `expose: false` restores
   the routing the author was withholding. `decoratorConfig` is `undefined` for

package/dist/chunks/{scanner-QMkE5DFY.js → scanner-D-vcuXjW.js}

@@ -1,6 +1,6 @@
 import { readFileSync } from "node:fs";
 import { dirname, isAbsolute, relative, resolve, sep, win32 } from "node:path";
-import { glob } from "tinyglobby";
+import { glob, isDynamicPattern } from "tinyglobby";
 import { parseSync } from "oxc-parser";
 //#region src/source-location.ts
 function getLineColumn(sourceText, offset) {
@@ -725,15 +725,40 @@ var MANDATORY_DISCOVERY_EXCLUDES = Object.freeze([
 ]);
 async function discoverSourceFiles(options) {
 	const cwd = resolve(options.cwd);
-	return glob(options.include.map((pattern) => relativeGlobToCwd(pattern, cwd)), {
-		cwd,
+	const patterns = options.include.map((pattern) => relativeGlobToCwd(pattern, cwd));
+	const exclusions = options.exclude.map((pattern) => relativeGlobToCwd(pattern, cwd));
+	let discoveryCwd = cwd;
+	for (const pattern of [...patterns, ...exclusions]) {
+		const target = resolve(cwd, pattern.startsWith("!") && !pattern.startsWith("!(") ? pattern.slice(1) : pattern);
+		while (relative(discoveryCwd, target).split(sep)[0] === "..") discoveryCwd = dirname(discoveryCwd);
+	}
+	const rebase = (pattern, bound) => {
+		const negative = pattern.startsWith("!") && !pattern.startsWith("!(");
+		const body = negative ? pattern.slice(1) : pattern;
+		const rebased = bound && (body === "**" || body.startsWith("**/")) ? body : normalizeGlobSeparators(relative(discoveryCwd, resolve(cwd, body)));
+		return negative ? `!${rebased}` : rebased;
+	};
+	const sourcePrunes = /* @__PURE__ */ new Set();
+	const files = await glob(patterns.filter((pattern) => {
+		if (pattern.startsWith("!") && !pattern.startsWith("!(")) return true;
+		const parts = normalizeGlobSeparators(relative(cwd, resolve(cwd, pattern))).split("/");
+		const dynamic = parts.findIndex((part) => isDynamicPattern(part));
+		const literal = dynamic < 0 ? parts : parts.slice(0, dynamic);
+		if (literal.some((part) => part === "node_modules" || part.startsWith(".") && part !== "." && part !== "..")) return false;
+		const prefix = dynamic < 0 ? literal.slice(0, -1) : literal;
+		const root = normalizeGlobSeparators(relative(discoveryCwd, resolve(cwd, ...prefix)));
+		for (const prune of MANDATORY_DISCOVERY_EXCLUDES) sourcePrunes.add(root ? `${root}/${prune}` : prune);
+		return true;
+	}).map((pattern) => rebase(pattern, pattern.startsWith("!") && !pattern.startsWith("!("))), {
+		cwd: discoveryCwd,
 		expandDirectories: false,
-		ignore: [...options.exclude.map((pattern) => relativeGlobToCwd(pattern, cwd)), ...MANDATORY_DISCOVERY_EXCLUDES],
+		ignore: [...exclusions.map((pattern) => rebase(pattern, true)), ...sourcePrunes],
 		absolute: true,
 		onlyFiles: true,
 		dot: true,
 		followSymbolicLinks: options.followSymbolicLinks ?? false
 	});
+	return [...new Set(files)].sort();
 }
 function relativeGlobToCwd(pattern, cwd) {
 	const windowsAbsolute = win32.isAbsolute(pattern);
@@ -2606,6 +2631,7 @@ function parseFile(filePath) {
 				const extracted = extractClassFromNode(node, filePath, sourceText, importAliases, ctx);
 				if (extracted) classes.push(extracted);
 			}
+			resolveLocalParameterAliases(classes, typeAliases);
 			reportUnresolvedSpreads(ctx.unresolved, filePath, sourceText, errors);
 			agentSurface = maybeExtractAgentSurface(program, sourceText, filePath);
 			if (sourceMayDeclareRecipe(sourceText)) {
@@ -2746,6 +2772,7 @@ function parseSource(sourceText, filename = "test.ts") {
 				const extracted = extractClassFromNode(node, filename, sourceText, importAliases, ctx);
 				if (extracted) classes.push(extracted);
 			}
+			resolveLocalParameterAliases(classes, typeAliases);
 			reportUnresolvedSpreads(ctx.unresolved, filename, sourceText, errors);
 			agentSurface = maybeExtractAgentSurface(program, sourceText, filename);
 			if (sourceMayDeclareRecipe(sourceText)) {
@@ -2872,6 +2899,27 @@ function extractTypeAliases(body) {
 	}
 	return aliases;
 }
+var MAX_PARAMETER_ALIAS_DEPTH = 8;
+var BARE_TYPE_IDENTIFIER = /^[A-Za-z_$][\w$]*$/;
+function resolveParameterAlias(type, aliases) {
+	if (!type || !BARE_TYPE_IDENTIFIER.test(type) || !Object.hasOwn(aliases, type)) return;
+	const seen = /* @__PURE__ */ new Set();
+	let current = type;
+	while (BARE_TYPE_IDENTIFIER.test(current) && Object.hasOwn(aliases, current)) {
+		if (seen.has(current) || seen.size >= MAX_PARAMETER_ALIAS_DEPTH) return { aliasUnresolved: true };
+		seen.add(current);
+		current = aliases[current].trim();
+	}
+	return { resolvedType: current };
+}
+function resolveLocalParameterAliases(classes, aliases) {
+	if (Object.keys(aliases).length === 0) return;
+	for (const classDef of classes) for (const method of classDef.methods) for (const parameter of method.parameters) {
+		if (parameter.typeUnresolved) continue;
+		const resolution = resolveParameterAlias(parameter.type, aliases);
+		if (resolution) Object.assign(parameter, resolution);
+	}
+}
 function enumMembersToUnion(members) {
 	const values = [];
 	let next = 0;
@@ -4049,7 +4097,10 @@ var ManifestAdapter = class ManifestAdapter {
 	*    and type alias resolution (up to depth 5).
 	* 4. **Numeric literal without annotation** — `version = 1` → `integer`.
 	* 5. **Boolean literal without annotation** — `isRead = false` → `boolean`.
-	* 6. **Default** — falls back to `text`.
+	* 6. **Boolean `@field({ default })` with no annotation or initializer** —
+	*    `@field({ default: false }) archived;` → `boolean`, matching runtime
+	*    decorator-only inference (#4295; code step 3.7).
+	* 7. **Default** — falls back to `text`.
 	*
 	* @param field - The raw field definition to analyse.
 	* @returns A {@link FieldTypeInference} describing the inferred type,
@@ -4081,6 +4132,13 @@ var ManifestAdapter = class ManifestAdapter {
 			defaultValue: field.initializer === "true",
 			source: "heuristic"
 		};
+		const booleanDefault = this.decoratorBooleanDefault(field);
+		if (booleanDefault !== void 0) return {
+			type: "boolean",
+			required: false,
+			defaultValue: booleanDefault,
+			source: "decorator"
+		};
 		const hasDefaultValue = field.initializer !== null;
 		return {
 			type: "text",
@@ -4100,6 +4158,16 @@ var ManifestAdapter = class ManifestAdapter {
 		return null;
 	}
 	/**
+	* The boolean `default` of a `@field()` decorator on `field`, if any.
+	*/
+	decoratorBooleanDefault(field) {
+		for (const decorator of field.decorators) {
+			if (decorator.name !== "field") continue;
+			const fieldDefault = this.parseFieldDecoratorOptions(decorator.arguments[0])?.default;
+			if (typeof fieldDefault === "boolean") return fieldDefault;
+		}
+	}
+	/**
 	* Infer type from field decorator
 	*/
 	inferFromDecorator(decorator, field) {
@@ -4532,7 +4600,9 @@ var ManifestAdapter = class ManifestAdapter {
 				default: p.defaultValue ? this.parseDefaultValue(p.defaultValue, "string") : void 0,
 				...p.typeUnresolved ? { typeUnresolved: true } : {},
 				...p.memberTypes ? { memberTypes: p.memberTypes } : {},
-				...p.unionBranches ? { unionBranches: p.unionBranches } : {}
+				...p.unionBranches ? { unionBranches: p.unionBranches } : {},
+				...p.resolvedType ? { resolvedType: p.resolvedType } : {},
+				...p.aliasUnresolved ? { aliasUnresolved: true } : {}
 			})),
 			returnType: method.returnType || "any",
 			description: method.description || void 0,
@@ -5058,4 +5128,4 @@ var OxcScanner = class {
 //#endregion
 export { isPrunedAgentSurfacePath as C, sourceMayDeclareAgentSurface as E, isAgentSurfaceSourcePath as S, scanSvelteAgentSurface as T, normalizeGlobSeparators as _, extractSmrtImports as a, emptyAgentSurface as b, parseRecipeFile as c, extractRecipes as d, resolveRecipes as f, discoverSourceFiles as g, resolveEnumValues as h, defaultCollectionName as i, parseSource as l, InheritanceResolver as m, parseUiSelectorsFile as n, parseAgentSurfaceFile as o, sourceMayDeclareRecipe as p, ManifestAdapter as r, parseFile as s, OxcScanner as t, deriveHelpFieldRefs as u, relativeGlobToCwd as v, mergeAgentSurfaces as w, extractAgentSurface as x, checkAgentSurfaceToolNames as y };
 
-//# sourceMappingURL=scanner-QMkE5DFY.js.map
+//# sourceMappingURL=scanner-D-vcuXjW.js.map