vitest-auto-spy 5.26.0 → 5.27.0

package/CHANGELOG.md

@@ -10,6 +10,32 @@ The latest released version here must always match the one published on
 
 ## [Unreleased]
 
+## [5.27.0] - 2026-09-23
+
+### Changed
+
+- **The `doctor` and `codemod` repository scan honours the root `.gitignore` for directories.** The
+  hardcoded skip list only knew the usual build and store names, so a repository's own generated or
+  local output (`src/generated/`, `/reports`, `tmp-*/`) was read as source: counted in the report,
+  walked by the import graph and offered to `codemod --write`. The file is parsed directly, with no
+  `git` process and no dependency: comments, `!` negations with the last match winning, anchored and
+  unanchored patterns, `*`, `?`, `**` and bracket classes. Only the root file and only directories
+  are honoured — an ignored file is still listed. A pattern with a `\` escape or a POSIX class is
+  dropped, which only means scanning more, and a `!` rule that cannot be read disables the whole
+  file rather than prune a directory git keeps. `perf-merge`'s report listing is unaffected.
+
+### Fixed
+
+- **The `tsconfig` glob check no longer reports a pattern it could never see matched.** A pattern
+  inside a skipped directory below the root (`libs/app/out-tsc/**/*.ts`) or inside a directory the
+  root `.gitignore` excludes is exempt, and so is a `files` entry there — both were reported as
+  matching nothing or missing, although the scan had simply not descended into them.
+- **The `prefer-create-spy-from-class` reference lists every factory whose arguments are exempt.**
+  The built-in doubles and stubs (`createComponentStub`, `createDirectiveHost`, `createWindowDouble`,
+  `provideRouterDouble` and the rest) were exempt in the rule but missing from the docs, which also
+  still said "four shapes" over a list of eleven. The Russian page gained the four exemptions it was
+  missing (`vi.hoisted`, the options bag, `createFixture`, an RxJS observer).
+
 ## [5.26.0] - 2026-09-23
 
 ### Changed
@@ -7770,7 +7796,8 @@ by hand there, in more than one place, by more than one person.
   `mockAccessorsProp`.
 - Dual ESM + CJS build with type declarations; 100% test coverage.
 
-[Unreleased]: https://github.com/ASDAlexey/vitest-auto-spy/compare/v5.26.0...HEAD
+[Unreleased]: https://github.com/ASDAlexey/vitest-auto-spy/compare/v5.27.0...HEAD
+[5.27.0]: https://github.com/ASDAlexey/vitest-auto-spy/compare/v5.26.0...v5.27.0
 [5.26.0]: https://github.com/ASDAlexey/vitest-auto-spy/compare/v5.25.0...v5.26.0
 [5.25.0]: https://github.com/ASDAlexey/vitest-auto-spy/compare/v5.24.0...v5.25.0
 [5.24.0]: https://github.com/ASDAlexey/vitest-auto-spy/compare/v5.23.0...v5.24.0

package/dist/cli.js

@@ -1,6 +1,6 @@
 #!/usr/bin/env node
-import { resolve, join, dirname, relative, posix, isAbsolute, sep } from 'node:path';
-import { readFileSync, statSync, mkdirSync, writeFileSync, readdirSync, existsSync, lstatSync, rmSync } from 'node:fs';
+import { resolve, join, dirname, posix, isAbsolute, relative, sep } from 'node:path';
+import { readFileSync, statSync, mkdirSync, writeFileSync, existsSync, lstatSync, rmSync, readdirSync } from 'node:fs';
 import { createHash } from 'node:crypto';
 import { fileURLToPath } from 'node:url';
 import { createRequire } from 'node:module';
@@ -98,6 +98,84 @@ function flagEnabled(args, name) {
   }
   return value !== "false";
 }
+
+// src/cli/gitignore.ts
+function parseGitignore(text) {
+  const rules = [];
+  for (const raw of text.split("\n")) {
+    const line = raw.trimEnd();
+    if (line === "" || line.startsWith("#")) {
+      continue;
+    }
+    const negated = line.startsWith("!");
+    const pattern = toRegExp(negated ? line.slice(1) : line);
+    if (pattern !== void 0) {
+      rules.push({ negated, pattern });
+    } else if (negated) {
+      return void 0;
+    }
+  }
+  return rules;
+}
+function isIgnoredDirectory(rules, path) {
+  let ignored = false;
+  for (const rule of rules) {
+    if (rule.pattern.test(path)) {
+      ignored = !rule.negated;
+    }
+  }
+  return ignored;
+}
+function toRegExp(line) {
+  const body = line.replace(/\/+$/, "");
+  if (body === "" || line.includes("\\")) {
+    return void 0;
+  }
+  const anchored = body.includes("/");
+  const segments = (body.startsWith("/") ? body.slice(1) : body).split("/");
+  let source = "";
+  for (const [index, segment] of segments.entries()) {
+    const last = index === segments.length - 1;
+    if (segment === "**") {
+      source += last ? ".*" : "(?:[^/]+/)*";
+      continue;
+    }
+    const converted = segmentSource(segment);
+    if (converted === void 0) {
+      return void 0;
+    }
+    source += last ? converted : `${converted}/`;
+  }
+  try {
+    return new RegExp(`^${anchored ? "" : "(?:.*/)?"}${source}$`);
+  } catch {
+    return void 0;
+  }
+}
+function segmentSource(segment) {
+  let source = "";
+  let index = 0;
+  while (index < segment.length) {
+    const char = segment.charAt(index);
+    if (char === "[") {
+      const negated = /^[!^]/.test(segment.slice(index + 1));
+      const start = index + (negated ? 2 : 1);
+      const end = segment.indexOf("]", start + 1);
+      const members = segment.slice(start, end);
+      if (end === -1 || members.includes("[")) {
+        return void 0;
+      }
+      source += `[${negated ? "^" : ""}${members.replace(/[\]^]/g, "\\$&")}]`;
+      index = end + 1;
+      continue;
+    }
+    source += char === "*" ? "[^/]*" : char === "?" ? "[^/]" : char.replace(/[$()+.^{|}]/, "\\$&");
+    index += 1;
+  }
+  return source;
+}
+
+// src/cli/fs-scan.ts
 var SKIPPED_DIRECTORIES = /* @__PURE__ */ new Set([
   ".angular",
   ".bun",
@@ -174,18 +252,25 @@ function removeFile(path) {
   rmSync(path, { force: true });
 }
 function listRepositoryFiles(root, limit = scanCap()) {
-  return scanRepository(root, limit).files;
+  return scan(root, limit, []).files;
 }
 function scanRepository(root, limit = scanCap()) {
-  const found = [];
-  const truncated = walk(root, root, found, limit);
-  return { files: found.sort(), truncated };
+  return scan(root, limit, readGitignoreRules(root));
+}
+function scan(root, limit, rules) {
+  const walker = { root, rules, found: [], ignored: [], limit };
+  const truncated = walk(walker, root);
+  return { files: walker.found.sort(), ignored: walker.ignored.sort(), truncated };
+}
+function readGitignoreRules(root) {
+  const text = readTextFile(join(root, ".gitignore"));
+  return (text === void 0 ? void 0 : parseGitignore(text)) ?? [];
 }
 function isRepositoryRoot(directory) {
   return existsSync(join(directory, ".git"));
 }
-function walk(root, directory, found, limit) {
-  if (found.length >= limit) {
+function walk(walker, directory) {
+  if (walker.found.length >= walker.limit) {
     return true;
   }
   let entries;
@@ -195,22 +280,30 @@ function walk(root, directory, found, limit) {
     return false;
   }
   for (const entry of entries) {
-    if (found.length >= limit) {
+    if (walker.found.length >= walker.limit) {
       return true;
     }
     const full = join(directory, entry.name);
     if (entry.isDirectory()) {
-      if (!isSkippedDirectory(entry.name) && !isRepositoryRoot(full) && walk(root, full, found, limit)) {
+      if (!isSkippedDirectory(entry.name) && !isRepositoryRoot(full) && !isGitignored(walker, full) && walk(walker, full)) {
         return true;
       }
       continue;
     }
     if (entry.isFile()) {
-      found.push(toPosix(relative(root, full)));
+      walker.found.push(toPosix(relative(walker.root, full)));
     }
   }
   return false;
 }
+function isGitignored(walker, directory) {
+  const path = toPosix(relative(walker.root, directory));
+  if (!isIgnoredDirectory(walker.rules, path)) {
+    return false;
+  }
+  walker.ignored.push(path);
+  return true;
+}
 function parseJsonc(text) {
   try {
     return JSON.parse(stripJsonComments(text));
@@ -640,7 +733,7 @@ function readProfile(cwd) {
   const scripts = stringMap(packageJson["scripts"]);
   const runner = detectRunner(dependencies, scripts);
   const framework = detectFramework(dependencies);
-  const scan2 = scanRepository(cwd);
+  const scan3 = scanRepository(cwd);
   return {
     cwd,
     runner,
@@ -651,8 +744,9 @@ function readProfile(cwd) {
     setupFiles: detectSetupFiles(cwd),
     dependencies,
     scripts,
-    files: scan2.files,
-    filesTruncated: scan2.truncated
+    files: scan3.files,
+    filesTruncated: scan3.truncated,
+    ignoredDirectories: scan3.ignored
   };
 }
 var PACKAGE_NAME = "vitest-auto-spy";
@@ -1059,7 +1153,7 @@ function textOf(context, range) {
 function group(groups, position) {
   return groups.slice(position, position + 1).join("");
 }
-function scan(masked, pattern) {
+function scan2(masked, pattern) {
   return [...masked.matchAll(pattern)].map((match) => ({ index: match.index, whole: group(match, 0), groups: [...match] }));
 }
 
@@ -1139,7 +1233,7 @@ function replacement(start, end, text) {
 function replacements(context, pattern, text) {
   return {
     ...EMPTY_OUTPUT,
-    edits: scan(context.masked, pattern).map((match) => ({
+    edits: scan2(context.masked, pattern).map((match) => ({
       start: match.index,
       end: match.index + match.whole.length,
       text: text(match)
@@ -1182,7 +1276,7 @@ var jasmineStrategies = {
 function withArgsOutput(context) {
   const closes = spyOnCloses(context);
   return mergeOutputs(
-    scan(context.masked, WITH_ARGS).map(
+    scan2(context.masked, WITH_ARGS).map(
       (match) => closes.has(lastCodeBefore(context.masked, match.index)) ? { ...EMPTY_OUTPUT, notes: [withArgsOnSpyOnNote(context, match.index)] } : replacement(match.index, match.index + match.whole.length, ".calledWith(")
     )
   );
@@ -1205,7 +1299,7 @@ function lastCodeBefore(masked, index) {
 }
 function strategyOutputs(context) {
   return mergeOutputs(
-    scan(context.masked, AND_STRATEGY).flatMap((match) => {
+    scan2(context.masked, AND_STRATEGY).flatMap((match) => {
       const name = group(match.groups, 1);
       return HELPER_NAMES.has(name) ? [] : [strategyOutput(context, match, name)];
     })
@@ -1279,12 +1373,12 @@ var jasmineSpyOn = {
   residue: /(?:^|[^\w$.])spyOn(?:Property)?\s*\(/,
   run: (context) => ({
     ...EMPTY_OUTPUT,
-    edits: scan(context.masked, SPY_ON).flatMap((match) => spyOnEdits(context, match))
+    edits: scan2(context.masked, SPY_ON).flatMap((match) => spyOnEdits(context, match))
   })
 };
 function spyOnCloses(context) {
   return new Set(
-    scan(context.masked, SPY_ON).flatMap((match) => {
+    scan2(context.masked, SPY_ON).flatMap((match) => {
       const range = spyOnRange(context, match);
       return range === void 0 ? [] : [range[1]];
     })
@@ -1517,7 +1611,7 @@ var jasmineGlobals = {
   // leftover `jasmine.SpyObj` reported twice would name a transform that never intended to touch it.
   residue: /\bjasmine\s*\.\s*(?!Spy(?:Obj)?\b)/,
   run: (context) => mergeOutputs(
-    scan(context.masked, MEMBER).flatMap((match) => {
+    scan2(context.masked, MEMBER).flatMap((match) => {
       const name = group(match.groups, 1);
       return TYPE_TARGETS[name] === void 0 ? [globalOutput(context, match, name)] : [];
     })
@@ -1592,7 +1686,7 @@ var jasmineTypes = {
   summary: "jasmine.Spy \u2192 Mock from vitest, and jasmine.SpyObj<T> \u2192 Spy<T> from this package.",
   residue: /\bjasmine\s*\.\s*Spy(?:Obj)?\b/,
   run: (context) => mergeOutputs(
-    scan(context.masked, MEMBER).flatMap((match) => {
+    scan2(context.masked, MEMBER).flatMap((match) => {
       const target = TYPE_TARGETS[group(match.groups, 1)];
       return target === void 0 ? [] : [typeOutput(context, match, target)];
     })
@@ -1630,11 +1724,11 @@ var jasmineMatchers = {
       ([from, to]) => replacements(context, new RegExp(String.raw`\.\s*${from}\s*\(`, "g"), () => `.${to}(`)
     ),
     { ...EMPTY_OUTPUT, edits: failEdits(context) },
-    { ...EMPTY_OUTPUT, edits: scan(context.masked, EXPECT).flatMap((match) => withContextEdits(context, match)) }
+    { ...EMPTY_OUTPUT, edits: scan2(context.masked, EXPECT).flatMap((match) => withContextEdits(context, match)) }
   ])
 };
 function failEdits(context) {
-  return scan(context.masked, FAIL).filter((match) => isCallPosition(context.masked, match.index + match.whole.length - 1)).map((match) => ({
+  return scan2(context.masked, FAIL).filter((match) => isCallPosition(context.masked, match.index + match.whole.length - 1)).map((match) => ({
     start: match.index,
     end: match.index + match.whole.length,
     text: `${group(match.groups, 1)}expect.fail(`
@@ -1775,7 +1869,7 @@ var jestNamespace = {
   summary: "jest.<member> \u2192 vi.<member>, reporting the members with no vi twin instead of renaming them.",
   residue: /\bjest\s*\./,
   run: (context) => collect(
-    scan(context.masked, MEMBER2).flatMap((match) => {
+    scan2(context.masked, MEMBER2).flatMap((match) => {
       const name = group(match.groups, 1);
       return TYPE_NAMES[name] === void 0 ? [memberOutput(context, match, name)] : [];
     })
@@ -1847,7 +1941,7 @@ var jestTypes = {
   run: (context) => {
     const functionFirst = importsJestGlobals(context);
     return collect([
-      ...scan(context.masked, MEMBER2).flatMap((match) => {
+      ...scan2(context.masked, MEMBER2).flatMap((match) => {
         const name = group(match.groups, 1);
         const target = TYPE_NAMES[name];
         return target === void 0 ? [] : [typeOutput2(context, match, name, target, functionFirst)];
@@ -1863,7 +1957,7 @@ var GENERIC_CALL = /\bjest\s*\.\s*(fn|spyOn)\s*</g;
 function genericCallNotes(context) {
   return {
     ...EMPTY_OUTPUT,
-    notes: scan(context.masked, GENERIC_CALL).map((match) => unreadableTypeArguments(context, match.index, group(match.groups, 1)))
+    notes: scan2(context.masked, GENERIC_CALL).map((match) => unreadableTypeArguments(context, match.index, group(match.groups, 1)))
   };
 }
 var NO_ARGUMENT = /\.\s*(mockImplementation(?:Once)?)\s*\(\s*\)/g;
@@ -1874,7 +1968,7 @@ var mockImplementationArity = {
   residue: /\.\s*mockImplementation(?:Once)?\s*\(\s*\)/,
   run: (context) => ({
     ...EMPTY_OUTPUT,
-    edits: scan(context.masked, NO_ARGUMENT).map((match) => ({
+    edits: scan2(context.masked, NO_ARGUMENT).map((match) => ({
       start: match.index,
       end: match.index + match.whole.length,
       text: `.${group(match.groups, 1)}(() => undefined)`
@@ -1890,7 +1984,7 @@ var jasmineAliases = {
   run: (context) => ({
     ...EMPTY_OUTPUT,
     edits: Object.entries(JASMINE_ALIASES).flatMap(
-      ([alias, replacement2]) => scan(context.masked, new RegExp(`(^|[^\\w$.])${alias}\\s*\\(`, "g")).filter((match) => isCallPosition(context.masked, match.index + match.whole.length - 1)).map((match) => {
+      ([alias, replacement2]) => scan2(context.masked, new RegExp(`(^|[^\\w$.])${alias}\\s*\\(`, "g")).filter((match) => isCallPosition(context.masked, match.index + match.whole.length - 1)).map((match) => {
         const start = match.index + group(match.groups, 1).length;
         return { start, end: start + alias.length, text: replacement2 };
       })
@@ -1911,7 +2005,7 @@ var jestGlobalsImport = {
 function globalsEdits(context, statement) {
   const specifierAt = context.source.indexOf(JEST_GLOBALS, statement.start);
   const [open, close] = statement.braces ?? [statement.start, statement.start];
-  const renames = scan(context.masked.slice(open, close), /\bjest\b/g).map((match) => ({
+  const renames = scan2(context.masked.slice(open, close), /\bjest\b/g).map((match) => ({
     start: open + match.index,
     end: open + match.index + "jest".length,
     text: "vi"
@@ -2112,7 +2206,7 @@ var injectCast = {
   residue: /\bas\s+Spy\s*</,
   run: (context) => {
     const helper = context.entries === void 0 ? void 0 : entryFor(context.entries, "asSpy", context.preferredEntry);
-    const casts = scan(context.masked, INJECT).flatMap((match) => {
+    const casts = scan2(context.masked, INJECT).flatMap((match) => {
       const cast = castAt(context, match.index, match.whole.length);
       return cast === void 0 ? [] : [cast];
     });
@@ -2124,7 +2218,7 @@ var injectCast = {
 };
 function withHelper(context, casts, need) {
   const covered = new Set(casts.map((cast) => cast.angle));
-  const strays = scan(context.masked, ANY_SPY_CAST).filter((match) => !covered.has(match.index + match.whole.length - 1)).map((match) => castNote(context, match.index));
+  const strays = scan2(context.masked, ANY_SPY_CAST).filter((match) => !covered.has(match.index + match.whole.length - 1)).map((match) => castNote(context, match.index));
   return {
     edits: casts.map((cast) => cast.edit),
     needs: casts.length === 0 ? [] : [need],
@@ -2152,7 +2246,7 @@ var TRANSFORMS = [
 function residueOf(file, text, transforms) {
   const masked = maskComments(text);
   return transforms.flatMap(
-    (transform) => scan(masked, new RegExp(transform.residue.source, `${transform.residue.flags.replace("g", "")}g`)).filter((match) => transform.residueIgnores?.(masked, match) !== true).map((match) => residueNote(file, text, match, transform))
+    (transform) => scan2(masked, new RegExp(transform.residue.source, `${transform.residue.flags.replace("g", "")}g`)).filter((match) => transform.residueIgnores?.(masked, match) !== true).map((match) => residueNote(file, text, match, transform))
   );
 }
 function residueNote(file, text, match, transform) {
@@ -2911,7 +3005,7 @@ function checkForeignPragma(graph) {
 }
 
 // src/cli/checks/export-map.generated.ts
-var EXPORT_MAP_VERSION = "5.25.0";
+var EXPORT_MAP_VERSION = "5.26.0";
 var ENTRY_SPECIFIERS = "vitest-auto-spy vitest-auto-spy/bun vitest-auto-spy/bun-angular vitest-auto-spy/node vitest-auto-spy/rstest vitest-auto-spy/rxjs vitest-auto-spy/console vitest-auto-spy/dom-stubs vitest-auto-spy/diagnostics vitest-auto-spy/jasmine vitest-auto-spy/jasmine-compat vitest-auto-spy/observer-spy vitest-auto-spy/angular vitest-auto-spy/angular/diagnostics vitest-auto-spy/angular/doubles vitest-auto-spy/angular/matchers vitest-auto-spy/angular-http vitest-auto-spy/angular-router vitest-auto-spy/signal-forms vitest-auto-spy/nestjs vitest-auto-spy/react vitest-auto-spy/vue vitest-auto-spy/svelte vitest-auto-spy/setup vitest-auto-spy/zone vitest-auto-spy/eslint-plugin vitest-auto-spy/perf-reporter";
 var EXPORTED_BY = {
   AccessorImplementations: "0 1 2 3 4 12 20 21 22",
@@ -3787,10 +3881,14 @@ function expandInclude(pattern) {
   const base = last === "*" ? trimmed2 : `${trimmed2}${last === "**" ? "/*" : "/**/*"}`;
   return IMPLIED_EXTENSIONS.map((extension) => `${base}${extension}`);
 }
-function isExemptPattern(pattern) {
-  const slash = pattern.indexOf("/");
-  const root = slash === -1 ? pattern : pattern.slice(0, slash);
-  return /\.d\.[cm]?ts$/.test(pattern) || isSkippedDirectory(root);
+function isExemptPattern(pattern, ignoredDirectories = []) {
+  return /\.d\.[cm]?ts$/.test(pattern) || isOutsideScan(pattern, ignoredDirectories);
+}
+function isOutsideScan(path, ignoredDirectories) {
+  const segments = path.split("/");
+  const wildcard = segments.findIndex((segment) => /[*?[]/.test(segment));
+  const literal = wildcard === -1 ? segments : segments.slice(0, wildcard);
+  return literal.some(isSkippedDirectory) || ignoredDirectories.some((directory) => `${path}/`.startsWith(`${directory}/`));
 }
 function literalTail(pattern) {
   const last = pattern.slice(pattern.lastIndexOf("/") + 1);
@@ -3816,12 +3914,13 @@ function stringList(value) {
 function resolveFromConfig(configPath, entry) {
   return posix.normalize(posix.join(posix.dirname(configPath), entry)).replace(/^\.\//, "");
 }
-function checkInclude(configPath, entry, files) {
+function checkInclude(configPath, entry, profile) {
+  const files = profile.files;
   if (entry.includes("${") || entry.startsWith("/")) {
     return void 0;
   }
   const resolved = resolveFromConfig(configPath, entry);
-  if (isExemptPattern(resolved) || matchesAnyFile(resolved, files)) {
+  if (isExemptPattern(resolved, profile.ignoredDirectories) || matchesAnyFile(resolved, files)) {
     return void 0;
   }
   const tail = literalTail(resolved);
@@ -3842,16 +3941,16 @@ function checkInclude(configPath, entry, files) {
     fix: "A pattern that matches nothing type-checks nothing, and `tsc --noEmit` still reports zero errors. Fix the glob or delete the entry."
   };
 }
-function checkOne(cwd, configPath, files) {
-  const text = readTextFile(join(cwd, configPath));
+function checkOne(profile, configPath) {
+  const text = readTextFile(join(profile.cwd, configPath));
   const parsed = text === void 0 ? void 0 : parseJsonc(text);
   if (!isRecord(parsed)) {
     return [];
   }
-  const findings = stringList(parsed["include"]).flatMap((entry) => checkInclude(configPath, entry, files) ?? []);
+  const findings = stringList(parsed["include"]).flatMap((entry) => checkInclude(configPath, entry, profile) ?? []);
   for (const entry of stringList(parsed["files"])) {
     const resolved = resolveFromConfig(configPath, entry);
-    if (!files.includes(resolved)) {
+    if (!profile.files.includes(resolved) && !isOutsideScan(resolved, profile.ignoredDirectories)) {
       findings.push({
         check: "tsconfig-file-missing",
         severity: "error",
@@ -3865,7 +3964,7 @@ function checkOne(cwd, configPath, files) {
 }
 function checkTsconfigGlobs(profile) {
   const configs = profile.files.filter((file) => TSCONFIG_NAME2.test(file));
-  return configs.flatMap((config) => checkOne(profile.cwd, config, profile.files));
+  return configs.flatMap((config) => checkOne(profile, config));
 }
 
 // src/cli/checks/unawaited-helper.ts

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "vitest-auto-spy",
-  "version": "5.26.0",
+  "version": "5.27.0",
   "description": "Auto-generate fully-typed test spies from a class — across Vitest, Bun, node:test and Rstest, with Angular/NestJS/React/Vue/Svelte recipes. Drop-in replacement for jest-auto-spies and jasmine-auto-spies.",
   "keywords": [
     "auto-mock",