veryfront 0.1.49 → 0.1.51

package/src/src/transforms/esm/import-parser.ts

@@ -24,7 +24,7 @@ export interface MissingImport {
   reason: string;
 }
 
-export interface ParseLocalImportsResult {
+interface ParseLocalImportsResult {
   imports: LocalImport[];
   cssImports: LocalImport[];
   crossProjectImports: CrossProjectImport[];
@@ -127,7 +127,8 @@ async function checkFileExists(path: string, adapter?: RuntimeAdapter): Promise<
     const fs = adapter?.fs.stat ? adapter.fs : createFileSystem();
     const stat = await fs.stat(path);
     return stat.isFile;
-  } catch {
+  } catch (_) {
+    /* expected: file may not exist */
     return false;
   }
 }
@@ -145,7 +146,8 @@ async function resolveLocalImportPath(
       const normalizedPath = basePath.replace(/^\/+/, "");
       const resolved = await adapter.fs.resolveFile(normalizedPath);
       if (resolved) return resolved;
-    } catch {
+    } catch (_) {
+      /* expected: resolveFile may not be supported */
       // Fall through to traditional resolution
     }
   }
@@ -180,7 +182,8 @@ async function resolveAliasImportPath(
     try {
       const resolved = await adapter.fs.resolveFile(normalizedPath);
       if (resolved) return resolved;
-    } catch {
+    } catch (_) {
+      /* expected: resolveFile may not be supported */
       // Fall through to manual resolution
     }
   }
@@ -190,7 +193,8 @@ async function resolveAliasImportPath(
     try {
       const stat = await fs.stat(absolutePath);
       return stat.isFile ? absolutePath : null;
-    } catch {
+    } catch (_) {
+      /* expected: file may not exist */
       return null;
     }
   }
@@ -212,7 +216,8 @@ async function findFirstExistingFile(
       try {
         const stat = await fs.stat(path);
         return stat.isFile ? path : null;
-      } catch {
+      } catch (_) {
+        /* expected: file may not exist */
         return null;
       }
     }),

package/src/src/transforms/esm/in-flight-manager.ts

@@ -21,7 +21,7 @@ import type { HttpCacheLike } from "./http-cache-helpers.js";
 
 const logger = rendererLogger.component("http-cache");
 
-export const DISTRIBUTED_REFRESH_INTERVAL_MS = 4 * 60 * 60 * 1000;
+const DISTRIBUTED_REFRESH_INTERVAL_MS = 4 * 60 * 60 * 1000;
 
 /** Per-request accumulator for bundle metadata during cacheHttpImportsToLocal. */
 export const bundleAccumulatorStorage = new AsyncLocalStorage<BundleEntry[]>();
@@ -31,7 +31,7 @@ export const processingStackStorage = new AsyncLocalStorage<Set<string>>();
 export const inFlightHttpFetches = new Map<string, Promise<string | null>>();
 
 /** Maximum time to wait for an in-flight fetch from another request before retrying */
-export const IN_FLIGHT_WAIT_TIMEOUT_MS = 30_000;
+const IN_FLIGHT_WAIT_TIMEOUT_MS = 30_000;
 
 /**
  * Clear all in-flight HTTP fetches.
@@ -42,7 +42,7 @@ export function __clearInFlightHttpFetches(): void {
 }
 
 /** Jitter to spread out timeout retries and prevent thundering herd (0-5s) */
-export const IN_FLIGHT_JITTER_MS = 5_000;
+const IN_FLIGHT_JITTER_MS = 5_000;
 
 /**
  * Wait for an in-flight fetch with timeout + jitter.
@@ -129,14 +129,17 @@ export function trackBundleAccumulator(
 ): void {
   const accumulator = bundleAccumulatorStorage.getStore();
   if (accumulator) {
-    createFileSystem().stat(cachePath).then((stat) => {
-      accumulator.push({
-        hash: String(hash),
-        url: normalizedUrl,
-        sizeBytes: stat?.size ?? 0,
-      });
-    }).catch(() => {
-      // Ignore stat errors
-    });
+    void (async () => {
+      try {
+        const stat = await createFileSystem().stat(cachePath);
+        accumulator.push({
+          hash: String(hash),
+          url: normalizedUrl,
+          sizeBytes: stat?.size ?? 0,
+        });
+      } catch {
+        // Ignore stat errors
+      }
+    })();
   }
 }

package/src/src/transforms/esm/package-registry.ts

@@ -55,19 +55,6 @@ export function esmShReact(
   );
 }
 
-/**
- * Generate esm.sh URL for browser.
- */
-export function getEsmShUrl(
-  pkg: string,
-  version: string,
-  external?: readonly string[],
-): string {
-  const params = ["target=es2022"];
-  if (external?.length) params.push(`external=${external.join(",")}`);
-  return `https://esm.sh/${pkg}@${version}?${params.join("&")}`;
-}
-
 /**
  * Get React esm.sh URLs with consistent versioning.
  */
@@ -90,13 +77,6 @@ export function getReactImportMap(version?: string): Record<string, string> {
   return getReactImportMapFromRewriter(version ?? DEFAULT_REACT_VERSION);
 }
 
-/**
- * Get React esm.sh URLs for Deno SSR.
- */
-export function getDenoNpmReactMap(version?: string): Record<string, string> {
-  return getReactUrls(version);
-}
-
 /**
  * Strip semver range prefixes (^, ~, >=, >, <=, <, =) from a version string.
  */
@@ -154,8 +134,8 @@ export async function resolveProjectReactVersion(options: {
         const stripped = stripSemverRange(rawVersion);
         return normalizeReactVersion(stripped);
       }
-    } catch {
-      // package.json not found or unreadable - fall through to default
+    } catch (_) {
+      /* expected: package.json may not exist or be unreadable */
     }
   }
 

package/src/src/transforms/esm/source-url-embed.ts

@@ -8,8 +8,8 @@
  */
 
 /** Preserved comment format that survives minification */
-export const VF_SOURCE_PREFIX = "/*! @vf-source: ";
-export const VF_SOURCE_SUFFIX = " */\n";
+const VF_SOURCE_PREFIX = "/*! @vf-source: ";
+const VF_SOURCE_SUFFIX = " */\n";
 
 /**
  * Embed the source URL in bundle code as a preserved comment.

package/src/src/transforms/esm/specifier-resolver.ts

@@ -28,7 +28,7 @@ export type CacheHttpModuleFn = (url: string, options: CacheOptions) => Promise<
  * Handles npm:, http(s)://, relative, and bare specifiers.
  * Returns null if the specifier should not be rewritten.
  */
-export async function resolveSpecifier(
+async function resolveSpecifier(
   specifier: string,
   baseUrl: string | undefined,
   options: CacheOptions,

package/src/src/transforms/esm/transform-cache.ts

@@ -25,7 +25,7 @@ const FALLBACK_MAX_ENTRIES = 500;
 const UNRESOLVED_VF_MODULES_PATTERN =
   /from\s*["']((?:file:\/\/)?\/?\/?_vf_modules\/_veryfront\/[^"']+)["']/;
 
-export interface TransformCacheEntry {
+interface TransformCacheEntry {
   code: string;
   hash: string;
   timestamp: number;
@@ -39,7 +39,7 @@ let cacheInitPromise: Promise<void> | null = null;
 
 const defaultLocalFallback = new Map<string, TransformCacheEntry>();
 
-export interface LocalFallbackLike<K, V> {
+interface LocalFallbackLike<K, V> {
   get(key: K): V | undefined;
   set(key: K, value: V): this;
   delete(key: K): boolean;
@@ -85,7 +85,7 @@ export function __injectCachesForTests(
  * Reset initialization state for testing.
  * This allows tests to simulate fresh initialization.
  */
-export function __resetInitStateForTests(): void {
+function __resetInitStateForTests(): void {
   cacheInitialized = false;
   cacheInitPromise = null;
   cacheGateway = null;
@@ -123,17 +123,7 @@ export async function initializeTransformCache(): Promise<boolean> {
   return cacheGateway?.type !== "memory";
 }
 
-export function isDistributedCacheEnabled(): boolean {
-  const gateway = getEffectiveCacheGateway();
-  if (!gateway) return false;
-  // Use gateway's isDistributed() if available, otherwise check type
-  if ("isDistributed" in gateway && typeof gateway.isDistributed === "function") {
-    return (gateway as TokenizingCacheGateway).isDistributed();
-  }
-  return gateway.type !== "memory";
-}
-
-export interface CacheKeyOptions {
+interface CacheKeyOptions {
   depsHash?: string;
   configHash?: string;
   projectId?: string;
@@ -284,7 +274,7 @@ export async function getDistributedTransformBackend(): Promise<CacheBackend | n
   return gateway as CacheBackend;
 }
 
-export interface TransformCacheResult {
+interface TransformCacheResult {
   code: string;
   /** Bundle manifest ID if the cached entry has one (for manifest-based validation) */
   bundleManifestId?: string;
@@ -325,142 +315,3 @@ export async function getOrComputeTransform(
 
   return { code, cacheHit: false };
 }
-
-export function getTransformCacheStats(): {
-  fallbackEntries: number;
-  maxFallbackEntries: number;
-  backend: string;
-} {
-  return {
-    fallbackEntries: getLocalFallback().size,
-    maxFallbackEntries: FALLBACK_MAX_ENTRIES,
-    backend: getEffectiveCacheGateway()?.type ?? "uninitialized",
-  };
-}
-
-export interface WarmupEntry {
-  key: string;
-  code: string;
-  hash: string;
-  bundleManifestId?: string;
-}
-
-export interface WarmupResult {
-  success: number;
-  failed: number;
-  skipped: number;
-  durationMs: number;
-}
-
-export async function warmupTransformCache(
-  entries: WarmupEntry[],
-  ttlSeconds: number = 3600,
-): Promise<WarmupResult> {
-  const start = performance.now();
-  let success = 0;
-  let failed = 0;
-  let skipped = 0;
-
-  await initializeTransformCache();
-
-  if (!isDistributedCacheEnabled()) {
-    logger.warn("Warmup skipped - no distributed cache available");
-    return {
-      success: 0,
-      failed: 0,
-      skipped: entries.length,
-      durationMs: Math.round(performance.now() - start),
-    };
-  }
-
-  const BATCH_SIZE = 50;
-  for (let i = 0; i < entries.length; i += BATCH_SIZE) {
-    const batch = entries.slice(i, i + BATCH_SIZE);
-    const results = await Promise.allSettled(
-      batch.map(async (entry) => {
-        const existing = await getCachedTransformAsync(entry.key);
-        if (existing && existing.hash === entry.hash) {
-          skipped++;
-          return;
-        }
-
-        await setCachedTransformAsync(
-          entry.key,
-          entry.code,
-          entry.hash,
-          ttlSeconds,
-          entry.bundleManifestId,
-        );
-        success++;
-      }),
-    );
-
-    for (const result of results) {
-      if (result.status === "rejected") {
-        failed++;
-        logger.debug("Warmup entry failed", { error: result.reason });
-      }
-    }
-  }
-
-  const durationMs = Math.round(performance.now() - start);
-  logger.info("Warmup complete", {
-    success,
-    failed,
-    skipped,
-    total: entries.length,
-    durationMs,
-    backend: getEffectiveCacheGateway()?.type,
-  });
-
-  return { success, failed, skipped, durationMs };
-}
-
-export async function prewarmProjectTransforms(
-  projectId: string,
-  filePaths: string[],
-): Promise<number> {
-  await initializeTransformCache();
-  const gateway = getEffectiveCacheGateway();
-
-  if (!gateway || gateway.type === "memory") {
-    logger.debug("Prewarm skipped - no distributed cache");
-    return 0;
-  }
-
-  let prewarmed = 0;
-  for (const filePath of filePaths) {
-    try {
-      const pattern = `v*:${projectId}:${filePath}:*:ssr`;
-      const scan =
-        (gateway as unknown as { scan?: (pattern: string, count: number) => Promise<string[]> })
-          .scan;
-      if (typeof scan !== "function") continue;
-
-      const keys = await scan.call(gateway, pattern, 10);
-      for (const key of keys) {
-        const cached = await getCachedTransformAsync(key);
-        if (!cached) continue;
-
-        setLocalFallback(key, cached);
-        prewarmed++;
-      }
-    } catch (error) {
-      logger.error("Prewarm failed for path", {
-        projectId,
-        filePath,
-        error: error instanceof Error ? error.message : String(error),
-      });
-    }
-  }
-
-  const logFn = prewarmed < filePaths.length ? logger.warn : logger.info;
-  logFn.call(logger, "Prewarm complete", {
-    projectId,
-    prewarmed,
-    total: filePaths.length,
-    incomplete: prewarmed < filePaths.length,
-  });
-
-  return prewarmed;
-}

package/src/src/transforms/import-rewriter/strategies/import-map-strategy.ts

@@ -21,7 +21,8 @@ function extractEsmShPackage(url: string): string | null {
 
     const pkg = (pathname.split("@")[0] ?? "").split("/")[0] ?? "";
     return pkg || null;
-  } catch {
+  } catch (_) {
+    /* expected: URL may be malformed */
     return null;
   }
 }
@@ -47,7 +48,8 @@ function extractEsmShSubpath(url: string): string {
 
     const restPath = pathname.slice(firstSlash);
     return restPath.startsWith("/") ? restPath : "";
-  } catch {
+  } catch (_) {
+    /* expected: URL may be malformed */
     return "";
   }
 }

package/src/src/transforms/md/utils.ts

@@ -10,7 +10,7 @@
  * Check if a file path is inside pages/ or app/ directories.
  * Files in these directories are routed normally, not as standalone markdown.
  */
-export function isInRoutableDir(filePath: string | undefined): boolean {
+function isInRoutableDir(filePath: string | undefined): boolean {
   if (!filePath) return false;
 
   return (

package/src/src/transforms/mdx/compiler/import-rewriter.ts

@@ -17,7 +17,8 @@ function toAbsPath(spec: string, basedir: string): string {
     if (!spec.startsWith(".") && !spec.startsWith("/")) return spec;
 
     return pathResolve(join(basedir, spec));
-  } catch {
+  } catch (_) {
+    /* expected: specifier may not be a valid URL or path */
     return spec;
   }
 }

package/src/src/transforms/mdx/esm-module-loader/cache/index.ts

@@ -27,7 +27,11 @@ export type CacheLookupResult =
   | { status: "miss" }
   | { status: "corrupted"; reason: string; filePath: string };
 
-export const verifiedModuleDeps = new LRUCache<string, true>({ maxEntries: 2000 });
+const MAX_VERIFIED_MODULE_DEPS = 2_000;
+
+export const verifiedModuleDeps = new LRUCache<string, true>({
+  maxEntries: MAX_VERIFIED_MODULE_DEPS,
+});
 
 const FILE_PATH_PATTERN = /file:\/\/([^"'\s]+)/gi;
 
@@ -95,7 +99,8 @@ async function findMissingFileDependencies(code: string): Promise<string[]> {
       if (!stat?.isFile) {
         missing.push(cleanPath);
       }
-    } catch {
+    } catch (_) {
+      /* expected: file dependency may not exist on disk */
       missing.push(cleanPath);
     }
   }
@@ -158,8 +163,8 @@ export async function getModulePathCache(cacheDir: string): Promise<Map<string,
       cache.set(path, cachePath);
     }
     logger.debug(`${LOG_PREFIX_MDX_LOADER} Loaded module index: ${cache.size} entries`);
-  } catch {
-    // Index doesn't exist yet
+  } catch (_) {
+    /* expected: _index.json may not exist yet */
   }
 
   modulePathCacheLoaded.add(cacheDir);
@@ -265,12 +270,16 @@ export function invalidateModulePaths(changedPaths: string[]): void {
       try {
         await localFs.remove(mjsPath);
         logger.debug(`${LOG_PREFIX_MDX_LOADER} Deleted stale cached module`, { mjsPath });
-      } catch {
-        // File may already be gone — not an error
+      } catch (_) {
+        /* expected: file may already be gone */
       }
     }
   };
-  _pendingDiskCleanup = _pendingDiskCleanup.then(cleanup, cleanup);
+  _pendingDiskCleanup = _pendingDiskCleanup.then(cleanup, cleanup).catch((error) => {
+    logger.warn(`${LOG_PREFIX_MDX_LOADER} Disk cleanup failed`, {
+      error: error instanceof Error ? error.message : String(error),
+    });
+  });
 }
 
 export async function clearESMDiskCache(): Promise<void> {
@@ -366,8 +375,8 @@ export async function lookupMdxEsmCache(
 
       try {
         await getLocalFs().remove(cachedPath);
-      } catch {
-        // ignore removal errors
+      } catch (_) {
+        /* expected: cached file may already be removed */
       }
 
       return {
@@ -390,7 +399,7 @@ export async function lookupMdxEsmCache(
       // Delete the stale file so it gets recreated
       try {
         await getLocalFs().remove(cachedPath);
-      } catch { /* ignore removal errors */ }
+      } catch (_) { /* expected: stale cached file may already be removed */ }
       return {
         status: "corrupted",
         reason: "Unresolved _vf_modules imports in cached code",
@@ -411,7 +420,7 @@ export async function lookupMdxEsmCache(
       // Delete the stale file so it gets recreated
       try {
         await getLocalFs().remove(cachedPath);
-      } catch { /* ignore removal errors */ }
+      } catch (_) { /* expected: stale cached file may already be removed */ }
       return {
         status: "corrupted",
         reason: `Missing file dependencies: ${missingDeps.slice(0, 3).join(", ")}`,
@@ -434,7 +443,8 @@ export async function lookupMdxEsmCache(
       `${LOG_PREFIX_MDX_LOADER} SSR reusing MDX-ESM cache: ${filePath} -> ${cachedPath}`,
     );
     return { status: "hit", path: cachedPath };
-  } catch {
+  } catch (_) {
+    /* expected: cached file may be inaccessible or deleted between checks */
     cache.delete(cacheKey);
     return { status: "corrupted", reason: "Cached file inaccessible", filePath };
   }

package/src/src/transforms/mdx/esm-module-loader/constants.ts

@@ -24,6 +24,5 @@ export const ESBUILD_JSX_FACTORY = "React.createElement";
 export const ESBUILD_JSX_FRAGMENT = "React.Fragment";
 
 export const DIRECTORY_PREFIXES = ["", "src/"];
-export const PREFIXES_TO_STRIP = ["components/", "pages/", "lib/", "app/"];
 
 export { HASH_SEED_FNV1A } from "../../../utils/constants/hash.js";

package/src/src/transforms/mdx/esm-module-loader/import-transformer.ts

@@ -143,14 +143,22 @@ export async function transformJsxImports(
               };
             }
           }
-        } catch {
-          // Not cached
+        } catch (_) {
+          /* expected: cached JSX module may not exist yet */
         }
 
         const isFrameworkFile = filePath.startsWith(FRAMEWORK_ROOT);
-        const jsxCode = isFrameworkFile
-          ? await getLocalFs().readTextFile(filePath)
-          : await adapter!.fs.readFile(filePath);
+        let jsxCode: string | Uint8Array;
+        if (isFrameworkFile) {
+          jsxCode = await getLocalFs().readTextFile(filePath);
+        } else if (adapter) {
+          jsxCode = await adapter.fs.readFile(filePath);
+        } else {
+          logger.warn(
+            `${LOG_PREFIX_MDX_LOADER} No adapter available to read JSX file: ${filePath}`,
+          );
+          return null;
+        }
 
         const loaderMap: Record<string, "js" | "jsx" | "ts" | "tsx"> = {
           tsx: "tsx",

package/src/src/transforms/mdx/esm-module-loader/jsx/runtime-loader.ts

@@ -6,13 +6,12 @@ export interface JSXRuntime {
 }
 
 export async function loadJSXRuntime(): Promise<JSXRuntime> {
-  // deno-lint-ignore no-explicit-any
-  const runtime = (await import("react/jsx-dev-runtime")) as any;
+  const runtime = (await import("react/jsx-dev-runtime")) as unknown as Record<string, unknown>;
 
   return {
     Fragment: runtime.Fragment,
-    jsx: runtime.jsx ?? runtime.jsxDEV,
-    jsxs: runtime.jsxs ?? runtime.jsxDEV,
-    jsxDEV: runtime.jsxDEV ?? runtime.jsx,
-  } as JSXRuntime;
+    jsx: (runtime.jsx ?? runtime.jsxDEV) as JSXRuntime["jsx"],
+    jsxs: (runtime.jsxs ?? runtime.jsxDEV) as JSXRuntime["jsxs"],
+    jsxDEV: (runtime.jsxDEV ?? runtime.jsx) as JSXRuntime["jsxDEV"],
+  };
 }

package/src/src/transforms/mdx/esm-module-loader/loader-helpers.ts

@@ -10,6 +10,7 @@
 
 import { join } from "../../../platform/compat/path/index.js";
 import { rendererLogger as logger } from "../../../utils/index.js";
+import { INVALID_ARGUMENT } from "../../../errors/index.js";
 import { withSpan } from "../../../observability/tracing/otlp-setup.js";
 import { SpanNames } from "../../../observability/tracing/span-names.js";
 import { getMdxEsmCacheDir } from "../../../utils/cache-dir.js";
@@ -32,7 +33,8 @@ export async function findMissingFrameworkBundles(paths: string[]): Promise<stri
       if (!(await fsExists(path))) {
         missing.push(path);
       }
-    } catch {
+    } catch (_) {
+      /* expected: file may not exist */
       missing.push(path);
     }
   }
@@ -46,9 +48,10 @@ export function resolveProjectDir(context: ESMLoaderContext): string {
     context.adapter?.env.get("VF_PROJECT_DIR");
   if (envProjectDir) return envProjectDir;
 
-  throw new Error(
-    "[MDX] projectDir is required for import map resolution. Pass it explicitly to loadModuleESM.",
-  );
+  throw INVALID_ARGUMENT.create({
+    detail:
+      "[MDX] projectDir is required for import map resolution. Pass it explicitly to loadModuleESM.",
+  });
 }
 
 /**
@@ -59,14 +62,14 @@ export async function initializeCacheDir(context: ESMLoaderContext): Promise<str
   if (context.esmCacheDir) return context.esmCacheDir;
 
   if (!context.projectId) {
-    throw new Error(
-      `Missing projectId for MDX ESM cache directory (projectSlug: ${context.projectSlug})`,
-    );
+    throw INVALID_ARGUMENT.create({
+      detail: `Missing projectId for MDX ESM cache directory (projectSlug: ${context.projectSlug})`,
+    });
   }
   if (!context.contentSourceId) {
-    throw new Error(
-      `Missing contentSourceId for MDX ESM cache directory (project: ${context.projectId})`,
-    );
+    throw INVALID_ARGUMENT.create({
+      detail: `Missing contentSourceId for MDX ESM cache directory (project: ${context.projectId})`,
+    });
   }
 
   const localFs = getLocalFs();
@@ -81,7 +84,8 @@ export async function initializeCacheDir(context: ESMLoaderContext): Promise<str
     context.esmCacheDir = persistentCacheDir;
     logger.debug(`${LOG_PREFIX_MDX_LOADER} Using persistent cache dir: ${persistentCacheDir}`);
     return persistentCacheDir;
-  } catch {
+  } catch (_) {
+    /* expected: persistent cache dir may not be writable, fall through to temp dir */
     const tempDir = await localFs.makeTempDir({ prefix: `veryfront-mdx-esm-${projectKey}-` });
     context.esmCacheDir = tempDir;
     return tempDir;
@@ -136,9 +140,9 @@ export async function processVfModuleImports(
   }
 
   if (!context.projectId) {
-    throw new Error(
-      `Missing projectId for module fetching (projectSlug: ${context.projectSlug})`,
-    );
+    throw INVALID_ARGUMENT.create({
+      detail: `Missing projectId for module fetching (projectSlug: ${context.projectSlug})`,
+    });
   }
 
   const fetcherContext = createModuleFetcherContext(

package/src/src/transforms/mdx/esm-module-loader/metadata/string-parser.ts

@@ -62,7 +62,8 @@ export function parseJsonish(value: string): unknown {
 
   try {
     return JSON.parse(jsonish);
-  } catch {
+  } catch (_) {
+    /* expected: value may not be valid JSON */
     return value;
   }
 }

package/src/src/transforms/mdx/esm-module-loader/missing-module.ts

@@ -3,6 +3,7 @@
  */
 
 import { getErrorCollector } from "../../../observability/error-collector.js";
+import { MODULE_NOT_FOUND } from "../../../errors/index.js";
 import { extractNamedImports } from "./utils/stub-module.js";
 
 type MissingModuleContext = {
@@ -38,7 +39,7 @@ export function buildMissingModuleError(ctx: MissingModuleContext): Error {
   ].filter(Boolean);
 
   const message = parts.join(" ");
-  const error = new Error(message);
+  const error = MODULE_NOT_FOUND.create({ detail: message });
   error.name = "MissingModuleError";
 
   try {
@@ -48,8 +49,8 @@ export function buildMissingModuleError(ctx: MissingModuleContext): Error {
       importStatement: ctx.importStatement,
       projectSlug: ctx.projectSlug,
     });
-  } catch {
-    // Error collector may not be initialized in all contexts
+  } catch (_) {
+    /* expected: error collector may not be initialized in all contexts */
   }
 
   return error;