@hyperfrontend/project-scope 0.1.0 → 0.2.1

package/index.cjs.js

@@ -13,7 +13,6 @@ var node_os = require('node:os');
  *
  * @module @hyperfrontend/immutable-api-utils/built-in-copy/console
  */
-// Capture references at module initialization time
 const _console = globalThis.console;
 /**
  * (Safe copy) Outputs a message to the console.
@@ -88,44 +87,6 @@ _console.timeEnd.bind(_console);
  */
 _console.timeLog.bind(_console);
 
-/**
- * Safe copies of Array built-in static methods.
- *
- * These references are captured at module initialization time to protect against
- * prototype pollution attacks. Import only what you need for tree-shaking.
- *
- * @module @hyperfrontend/immutable-api-utils/built-in-copy/array
- */
-// Capture references at module initialization time
-const _Array = globalThis.Array;
-/**
- * (Safe copy) Determines whether the passed value is an Array.
- */
-const isArray = _Array.isArray;
-/**
- * (Safe copy) Creates an array from an array-like or iterable object.
- */
-const from = _Array.from;
-
-/**
- * Safe copies of JSON built-in methods.
- *
- * These references are captured at module initialization time to protect against
- * prototype pollution attacks. Import only what you need for tree-shaking.
- *
- * @module @hyperfrontend/immutable-api-utils/built-in-copy/json
- */
-// Capture references at module initialization time
-const _JSON = globalThis.JSON;
-/**
- * (Safe copy) Converts a JavaScript Object Notation (JSON) string into an object.
- */
-const parse = _JSON.parse;
-/**
- * (Safe copy) Converts a JavaScript value to a JavaScript Object Notation (JSON) string.
- */
-const stringify = _JSON.stringify;
-
 /**
  * Safe copies of Object built-in methods.
  *
@@ -134,7 +95,6 @@ const stringify = _JSON.stringify;
  *
  * @module @hyperfrontend/immutable-api-utils/built-in-copy/object
  */
-// Capture references at module initialization time
 const _Object = globalThis.Object;
 /**
  * (Safe copy) Prevents modification of existing property attributes and values,
@@ -162,30 +122,25 @@ const defineProperty = _Object.defineProperty;
  */
 const defineProperties = _Object.defineProperties;
 
+const registeredClasses = [];
+
 /**
- * Safe copies of Set built-in via factory function.
- *
- * Since constructors cannot be safely captured via Object.assign, this module
- * provides a factory function that uses Reflect.construct internally.
+ * Safe copies of Array built-in static methods.
  *
  * These references are captured at module initialization time to protect against
  * prototype pollution attacks. Import only what you need for tree-shaking.
  *
- * @module @hyperfrontend/immutable-api-utils/built-in-copy/set
+ * @module @hyperfrontend/immutable-api-utils/built-in-copy/array
  */
-// Capture references at module initialization time
-const _Set = globalThis.Set;
-const _Reflect$3 = globalThis.Reflect;
+const _Array = globalThis.Array;
 /**
- * (Safe copy) Creates a new Set using the captured Set constructor.
- * Use this instead of `new Set()`.
- *
- * @param iterable - Optional iterable of values.
- * @returns A new Set instance.
+ * (Safe copy) Determines whether the passed value is an Array.
  */
-const createSet = (iterable) => _Reflect$3.construct(_Set, iterable ? [iterable] : []);
-
-const registeredClasses = [];
+const isArray = _Array.isArray;
+/**
+ * (Safe copy) Creates an array from an array-like or iterable object.
+ */
+const from = _Array.from;
 
 /**
  * Returns the data type of the target.
@@ -221,9 +176,8 @@ const getType = (target) => {
  *
  * @module @hyperfrontend/immutable-api-utils/built-in-copy/error
  */
-// Capture references at module initialization time
 const _Error = globalThis.Error;
-const _Reflect$2 = globalThis.Reflect;
+const _Reflect$3 = globalThis.Reflect;
 /**
  * (Safe copy) Creates a new Error using the captured Error constructor.
  * Use this instead of `new Error()`.
@@ -232,7 +186,7 @@ const _Reflect$2 = globalThis.Reflect;
  * @param options - Optional error options.
  * @returns A new Error instance.
  */
-const createError = (message, options) => _Reflect$2.construct(_Error, [message, options]);
+const createError = (message, options) => _Reflect$3.construct(_Error, [message, options]);
 
 /**
  * Safe copies of Map built-in via factory function.
@@ -245,9 +199,8 @@ const createError = (message, options) => _Reflect$2.construct(_Error, [message,
  *
  * @module @hyperfrontend/immutable-api-utils/built-in-copy/map
  */
-// Capture references at module initialization time
 const _Map = globalThis.Map;
-const _Reflect$1 = globalThis.Reflect;
+const _Reflect$2 = globalThis.Reflect;
 /**
  * (Safe copy) Creates a new Map using the captured Map constructor.
  * Use this instead of `new Map()`.
@@ -255,7 +208,7 @@ const _Reflect$1 = globalThis.Reflect;
  * @param iterable - Optional iterable of key-value pairs.
  * @returns A new Map instance.
  */
-const createMap = (iterable) => _Reflect$1.construct(_Map, iterable ? [iterable] : []);
+const createMap = (iterable) => _Reflect$2.construct(_Map, iterable ? [iterable] : []);
 
 /**
  * Safe copies of Date built-in via factory function and static methods.
@@ -268,11 +221,10 @@ const createMap = (iterable) => _Reflect$1.construct(_Map, iterable ? [iterable]
  *
  * @module @hyperfrontend/immutable-api-utils/built-in-copy/date
  */
-// Capture references at module initialization time
 const _Date = globalThis.Date;
-const _Reflect = globalThis.Reflect;
+const _Reflect$1 = globalThis.Reflect;
 function createDate(...args) {
-    return _Reflect.construct(_Date, args);
+    return _Reflect$1.construct(_Date, args);
 }
 /**
  * (Safe copy) Returns the number of milliseconds elapsed since January 1, 1970 00:00:00 UTC.
@@ -287,15 +239,11 @@ const dateNow = _Date.now;
  *
  * @module @hyperfrontend/immutable-api-utils/built-in-copy/math
  */
-// Capture references at module initialization time
 const _Math = globalThis.Math;
 /**
  * (Safe copy) Returns the value of a number rounded to the nearest integer.
  */
 const round = _Math.round;
-// ============================================================================
-// Min/Max
-// ============================================================================
 /**
  * (Safe copy) Returns the larger of zero or more numbers.
  */
@@ -305,6 +253,28 @@ const max = _Math.max;
  */
 const min = _Math.min;
 
+/**
+ * Safe copies of Set built-in via factory function.
+ *
+ * Since constructors cannot be safely captured via Object.assign, this module
+ * provides a factory function that uses Reflect.construct internally.
+ *
+ * These references are captured at module initialization time to protect against
+ * prototype pollution attacks. Import only what you need for tree-shaking.
+ *
+ * @module @hyperfrontend/immutable-api-utils/built-in-copy/set
+ */
+const _Set = globalThis.Set;
+const _Reflect = globalThis.Reflect;
+/**
+ * (Safe copy) Creates a new Set using the captured Set constructor.
+ * Use this instead of `new Set()`.
+ *
+ * @param iterable - Optional iterable of values.
+ * @returns A new Set instance.
+ */
+const createSet = (iterable) => _Reflect.construct(_Set, iterable ? [iterable] : []);
+
 /* eslint-disable @typescript-eslint/no-explicit-any */
 /**
  * Creates a wrapper function that only executes the wrapped function if the condition function returns true.
@@ -468,6 +438,24 @@ function notFnMsg(label) {
 
 createLogger(error, warn, log, info, debug);
 
+/**
+ * Safe copies of JSON built-in methods.
+ *
+ * These references are captured at module initialization time to protect against
+ * prototype pollution attacks. Import only what you need for tree-shaking.
+ *
+ * @module @hyperfrontend/immutable-api-utils/built-in-copy/json
+ */
+const _JSON = globalThis.JSON;
+/**
+ * (Safe copy) Converts a JavaScript Object Notation (JSON) string into an object.
+ */
+const parse = _JSON.parse;
+/**
+ * (Safe copy) Converts a JavaScript value to a JavaScript Object Notation (JSON) string.
+ */
+const stringify = _JSON.stringify;
+
 /**
  * Global log level registry.
  * Tracks all created scoped loggers to allow global log level changes.
@@ -611,14 +599,11 @@ function formatMessage(namespace, message, meta) {
  */
 function createScopedLogger(namespace, options = {}) {
     const { level = 'error', sanitizeSecrets = true } = options;
-    // Create wrapper functions that add namespace prefix and sanitization
     const createLogFn = (baseFn) => (message, meta) => {
         const processedMeta = sanitizeSecrets && meta ? sanitize(meta) : meta;
         baseFn(formatMessage(namespace, message, processedMeta));
     };
-    // Create base logger with wrapped functions
     const baseLogger = createLogger(createLogFn(error), createLogFn(warn), createLogFn(log), createLogFn(info), createLogFn(debug));
-    // Set initial log level (use global override if set)
     baseLogger.setLogLevel(globalLogLevel ?? level);
     const scopedLogger = freeze({
         error: (message, meta) => baseLogger.error(message, meta),
@@ -629,7 +614,6 @@ function createScopedLogger(namespace, options = {}) {
         setLogLevel: baseLogger.setLogLevel,
         getLogLevel: baseLogger.getLogLevel,
     });
-    // Register logger for global level management
     loggerRegistry.add(scopedLogger);
     return scopedLogger;
 }
@@ -1065,17 +1049,14 @@ function readDirectoryRecursive(dirPath, options) {
             entries = readDirectory(currentPath);
         }
         catch {
-            // Skip inaccessible directories
             fsDirLogger.debug('Skipping inaccessible directory', { path: currentPath });
             return;
         }
         for (const entry of entries) {
-            // Skip hidden files/dirs if not included
             if (!includeHidden && entry.name.startsWith('.')) {
                 continue;
             }
             results.push({ ...entry, depth });
-            // Recurse into directories
             if (entry.isDirectory || (entry.isSymlink && followSymlinks && isDirectory(entry.path))) {
                 walk(entry.path, depth + 1);
             }
@@ -1143,7 +1124,6 @@ function joinPosix(...paths) {
 function normalizePath(filePath) {
     if (!filePath)
         return '';
-    // Normalize path and convert backslashes to forward slashes
     const normalized = node_path.normalize(filePath);
     return node_path.sep === '\\' ? normalized.replace(/\\/g, '/') : normalized;
 }
@@ -1177,7 +1157,11 @@ function normalizeToNative(filePath) {
  * @returns Path with trailing slashes removed
  */
 function removeTrailingSlash(filePath) {
-    return filePath.replace(/[/\\]+$/, '');
+    let i = filePath.length;
+    while (i > 0 && (filePath[i - 1] === '/' || filePath[i - 1] === '\\')) {
+        i--;
+    }
+    return filePath.slice(0, i);
 }
 /**
  * Append a forward slash to the path if not already present.
@@ -1351,7 +1335,6 @@ function traverseUpward(startPath, predicate) {
         }
         currentPath = node_path.dirname(currentPath);
     }
-    // Check root directory
     if (predicate(rootPath)) {
         fsTraversalLogger.debug('Upward traversal found match at root', { startPath, foundPath: rootPath });
         return rootPath;
@@ -1723,29 +1706,23 @@ const depsLogger = createScopedLogger('project-scope:heuristics:deps');
  */
 function extractImports(content) {
     const imports = [];
-    // ES import with 'from': import X from 'path' or import { X } from 'path'
-    // Use non-greedy match and avoid nested quantifiers by matching "from" keyword directly
     const esImportFromRegex = /import\s+.+?\s+from\s+['"]([^'"]+)['"]/g;
     let match;
     while ((match = esImportFromRegex.exec(content)) !== null) {
         imports.push(match[1]);
     }
-    // Side-effect import: import 'path'
     const sideEffectImportRegex = /import\s+['"]([^'"]+)['"]/g;
     while ((match = sideEffectImportRegex.exec(content)) !== null) {
         imports.push(match[1]);
     }
-    // Dynamic import: import('path')
     const dynamicImportRegex = /import\s*\(\s*['"]([^'"]+)['"]\s*\)/g;
     while ((match = dynamicImportRegex.exec(content)) !== null) {
         imports.push(match[1]);
     }
-    // require: require('path')
     const requireRegex = /require\s*\(\s*['"]([^'"]+)['"]\s*\)/g;
     while ((match = requireRegex.exec(content)) !== null) {
         imports.push(match[1]);
     }
-    // Re-export: export * from 'path' or export { X } from 'path'
     const exportFromRegex = /export\s+.+?\s+from\s+['"]([^'"]+)['"]/g;
     while ((match = exportFromRegex.exec(content)) !== null) {
         imports.push(match[1]);
@@ -2059,7 +2036,6 @@ const cacheRegistry = createSet();
 function createCache(options) {
     const { ttl, maxSize } = options ?? {};
     const store = createMap();
-    // Track insertion order for FIFO eviction
     const insertionOrder = [];
     /**
      * Check if an entry is expired.
@@ -2110,12 +2086,10 @@ function createCache(options) {
             return entry.value;
         },
         set(key, value) {
-            // If key exists, remove from order first
             if (store.has(key)) {
                 removeFromOrder(key);
             }
             else {
-                // Evict if needed before adding new entry
                 evictIfNeeded();
             }
             // eslint-disable-next-line workspace/no-unsafe-builtin-methods -- Date.now() is needed for Jest fake timers compatibility
@@ -2148,7 +2122,6 @@ function createCache(options) {
             return [...insertionOrder];
         },
     };
-    // Register cache for global operations
     cacheRegistry.add(cache);
     return freeze(cache);
 }
@@ -2228,7 +2201,6 @@ function memoize(fn, options) {
         cache.set(key, result);
         return result;
     };
-    // Attach cache for direct access
     defineProperty(memoized, 'cache', {
         value: cache,
         writable: false,
@@ -2237,10 +2209,6 @@ function memoize(fn, options) {
     return memoized;
 }
 
-/**
- * Pattern matching utilities with ReDoS protection.
- * Uses character-by-character matching instead of regex where possible.
- */
 /**
  * Match path against glob pattern using safe character iteration.
  * Avoids regex to prevent ReDoS attacks.
@@ -2278,17 +2246,14 @@ function matchGlobPattern(path, pattern) {
  * @returns True if remaining segments match
  */
 function matchSegments(pathParts, patternParts, pathIdx, patternIdx) {
-    // Base cases
     if (pathIdx === pathParts.length && patternIdx === patternParts.length) {
-        return true; // Both exhausted = match
+        return true;
     }
     if (patternIdx >= patternParts.length) {
-        return false; // Pattern exhausted but path remains
+        return false;
     }
     const patternPart = patternParts[patternIdx];
-    // Handle ** (globstar) - matches zero or more directories
     if (patternPart === '**') {
-        // Try matching rest of pattern against current position and all future positions
         for (let i = pathIdx; i <= pathParts.length; i++) {
             if (matchSegments(pathParts, patternParts, i, patternIdx + 1)) {
                 return true;
@@ -2297,10 +2262,9 @@ function matchSegments(pathParts, patternParts, pathIdx, patternIdx) {
         return false;
     }
     if (pathIdx >= pathParts.length) {
-        return false; // Path exhausted but pattern remains (and it's not **)
+        return false;
     }
     const pathPart = pathParts[pathIdx];
-    // Match current segment
     if (matchSegment(pathPart, patternPart)) {
         return matchSegments(pathParts, patternParts, pathIdx + 1, patternIdx + 1);
     }
@@ -2320,12 +2284,10 @@ function matchSegment(text, pattern) {
     while (patternIdx < pattern.length) {
         const char = pattern[patternIdx];
         if (char === '*') {
-            // * matches zero or more characters
             patternIdx++;
             if (patternIdx === pattern.length) {
-                return true; // * at end matches rest of string
+                return true;
             }
-            // Try matching rest of pattern at each position in text
             for (let i = textIdx; i <= text.length; i++) {
                 if (matchSegmentFrom(text, i, pattern, patternIdx)) {
                     return true;
@@ -2334,7 +2296,6 @@ function matchSegment(text, pattern) {
             return false;
         }
         else if (char === '?') {
-            // ? matches exactly one character
             if (textIdx >= text.length) {
                 return false;
             }
@@ -2342,10 +2303,8 @@ function matchSegment(text, pattern) {
             patternIdx++;
         }
         else if (char === '{') {
-            // {a,b,c} matches any alternative
             const closeIdx = findClosingBrace(pattern, patternIdx);
             if (closeIdx === -1) {
-                // Unmatched brace, treat as literal
                 if (textIdx >= text.length || text[textIdx] !== char) {
                     return false;
                 }
@@ -2363,7 +2322,6 @@ function matchSegment(text, pattern) {
             }
         }
         else {
-            // Literal character
             if (textIdx >= text.length || text[textIdx] !== char) {
                 return false;
             }
@@ -2789,7 +2747,8 @@ const entryPointLogger = createScopedLogger('project-scope:heuristics:entry-poin
  */
 const entryPointCache = createCache({ ttl: 60000, maxSize: 50 });
 /**
- * Common entry point patterns.
+ * Common entry point patterns by project type.
+ * Used for convention-based entry point discovery.
  */
 const ENTRY_POINT_PATTERNS = {
     /** Library entry patterns */
@@ -3072,7 +3031,6 @@ function collectAllDependencies(packageJson) {
 function parseVersionString(versionString) {
     if (versionString === undefined || versionString === null)
         return undefined;
-    // Manual parsing instead of regex to avoid ReDoS
     let start = 0;
     while (start < versionString.length) {
         const char = versionString[start];
@@ -3133,7 +3091,6 @@ function expressDetector(projectPath, packageJson) {
         version = parseVersionString(deps['express']);
         sources.push({ type: 'package.json', field: 'dependencies.express' });
     }
-    // @types/express (indicates usage)
     if (deps['@types/express']) {
         confidence += 10;
         sources.push({ type: 'package.json', field: 'dependencies.@types/express' });
@@ -3169,13 +3126,11 @@ function nestDetector(projectPath, packageJson) {
     let version;
     let configPath;
     const deps = collectAllDependencies(pkg);
-    // @nestjs/core package
     if (deps['@nestjs/core']) {
         confidence += 70;
         version = parseVersionString(deps['@nestjs/core']);
         sources.push({ type: 'package.json', field: 'dependencies.@nestjs/core' });
     }
-    // @nestjs/common
     if (deps['@nestjs/common']) {
         confidence += 15;
         sources.push({ type: 'package.json', field: 'dependencies.@nestjs/common' });
@@ -3226,7 +3181,6 @@ function fastifyDetector(projectPath, packageJson) {
         confidence += 15;
         sources.push({ type: 'package.json', field: 'dependencies (fastify plugins)' });
     }
-    // @types/fastify (older versions)
     if (deps['@types/fastify']) {
         confidence += 5;
         sources.push({ type: 'package.json', field: 'dependencies.@types/fastify' });
@@ -3261,7 +3215,6 @@ function koaDetector(projectPath, packageJson) {
         version = parseVersionString(deps['koa']);
         sources.push({ type: 'package.json', field: 'dependencies.koa' });
     }
-    // @types/koa
     if (deps['@types/koa']) {
         confidence += 10;
         sources.push({ type: 'package.json', field: 'dependencies.@types/koa' });
@@ -3840,7 +3793,6 @@ function remixDetector(projectPath, packageJson) {
     let confidence = 0;
     let version;
     const deps = collectAllDependencies(pkg);
-    // @remix-run packages
     if (deps['@remix-run/react']) {
         confidence += 70;
         version = parseVersionString(deps['@remix-run/react']);
@@ -4116,7 +4068,6 @@ function sveltekitDetector(projectPath, packageJson) {
     let confidence = 0;
     let version;
     const deps = collectAllDependencies(pkg);
-    // @sveltejs/kit package
     if (deps['@sveltejs/kit']) {
         confidence += 70;
         version = parseVersionString(deps['@sveltejs/kit']);
@@ -4195,13 +4146,11 @@ function qwikDetector(projectPath, packageJson) {
     let confidence = 0;
     let version;
     const deps = collectAllDependencies(pkg);
-    // @builder.io/qwik package
     if (deps['@builder.io/qwik']) {
         confidence += 70;
         version = parseVersionString(deps['@builder.io/qwik']);
         sources.push({ type: 'package.json', field: 'dependencies.@builder.io/qwik' });
     }
-    // @builder.io/qwik-city
     if (deps['@builder.io/qwik-city']) {
         confidence += 20;
         sources.push({ type: 'package.json', field: 'dependencies.@builder.io/qwik-city' });
@@ -4312,23 +4261,19 @@ function angularJSDetector(projectPath, packageJson) {
     let confidence = 0;
     let version;
     const deps = collectAllDependencies(pkg);
-    // AngularJS package (angular, not @angular/core)
     if (deps['angular']) {
         confidence += 70;
         version = parseVersionString(deps['angular']);
         sources.push({ type: 'package.json', field: 'dependencies.angular' });
     }
-    // AngularJS router
     if (deps['angular-route']) {
         confidence += 15;
         sources.push({ type: 'package.json', field: 'dependencies.angular-route' });
     }
-    // AngularJS resource
     if (deps['angular-resource']) {
         confidence += 10;
         sources.push({ type: 'package.json', field: 'dependencies.angular-resource' });
     }
-    // AngularJS animate
     if (deps['angular-animate']) {
         confidence += 5;
         sources.push({ type: 'package.json', field: 'dependencies.angular-animate' });
@@ -4359,23 +4304,19 @@ function backboneDetector(projectPath, packageJson) {
     let confidence = 0;
     let version;
     const deps = collectAllDependencies(pkg);
-    // Backbone package
     if (deps['backbone']) {
         confidence += 70;
         version = parseVersionString(deps['backbone']);
         sources.push({ type: 'package.json', field: 'dependencies.backbone' });
-        // Underscore (commonly used with Backbone)
         if (deps['underscore']) {
             confidence += 15;
             sources.push({ type: 'package.json', field: 'dependencies.underscore' });
         }
-        // Lodash can be used as underscore replacement
         if (deps['lodash']) {
             confidence += 5;
             sources.push({ type: 'package.json', field: 'dependencies.lodash' });
         }
     }
-    // Marionette (Backbone framework)
     if (deps['backbone.marionette'] || deps['marionette']) {
         confidence += 10;
         sources.push({ type: 'package.json', field: 'dependencies.backbone.marionette' });
@@ -4406,18 +4347,15 @@ function emberDetector(projectPath, packageJson) {
     let confidence = 0;
     let version;
     const deps = collectAllDependencies(pkg);
-    // Ember source package
     if (deps['ember-source']) {
         confidence += 70;
         version = parseVersionString(deps['ember-source']);
         sources.push({ type: 'package.json', field: 'dependencies.ember-source' });
     }
-    // Ember CLI
     if (deps['ember-cli']) {
         confidence += 20;
         sources.push({ type: 'package.json', field: 'devDependencies.ember-cli' });
     }
-    // Ember Data
     if (deps['ember-data']) {
         confidence += 10;
         sources.push({ type: 'package.json', field: 'dependencies.ember-data' });
@@ -4448,18 +4386,15 @@ function jqueryDetector(projectPath, packageJson) {
     let confidence = 0;
     let version;
     const deps = collectAllDependencies(pkg);
-    // jQuery package
     if (deps['jquery']) {
         confidence += 80;
         version = parseVersionString(deps['jquery']);
         sources.push({ type: 'package.json', field: 'dependencies.jquery' });
     }
-    // jQuery UI
     if (deps['jquery-ui']) {
         confidence += 10;
         sources.push({ type: 'package.json', field: 'dependencies.jquery-ui' });
     }
-    // jQuery plugins
     if (deps['jquery-validation']) {
         confidence += 5;
         sources.push({ type: 'package.json', field: 'dependencies.jquery-validation' });
@@ -4606,7 +4541,6 @@ function prettierDetector(projectPath, packageJson) {
         confidence += 30;
         sources.push({ type: 'package.json', field: 'prettier' });
     }
-    // .prettierignore file
     if (exists(node_path.join(projectPath, '.prettierignore'))) {
         confidence += 10;
         sources.push({ type: 'config-file', path: '.prettierignore' });
@@ -4701,7 +4635,6 @@ function biomeDetector(projectPath, packageJson) {
     let configPath;
     let version;
     const deps = collectAllDependencies(pkg);
-    // @biomejs/biome package
     if (deps['@biomejs/biome']) {
         confidence += 70;
         version = parseVersionString(deps['@biomejs/biome']);
@@ -4988,7 +4921,7 @@ function npmWorkspacesDetector(workspacePath, packageJson) {
         sources.push({ type: 'lockfile', path: 'package-lock.json' });
     }
     if (exists(node_path.join(workspacePath, 'yarn.lock'))) {
-        return null; // Let yarn workspace detector handle this
+        return null;
     }
     if (confidence === 0) {
         return null;
@@ -5367,7 +5300,6 @@ function checkTsConfigStrict(projectPath) {
     if (!content)
         return undefined;
     try {
-        // Simple JSON parsing - doesn't handle comments but good enough for strict check
         const cleanContent = content.replace(/\/\*[\s\S]*?\*\/|\/\/.*/g, '');
         const parsed = parse(cleanContent);
         return parsed?.compilerOptions?.strict === true;
@@ -5390,19 +5322,16 @@ function typescriptDetector(projectPath, packageJson) {
     let configPath;
     let version;
     const deps = collectAllDependencies(pkg);
-    // TypeScript package
     if (deps['typescript']) {
         confidence += 50;
         version = parseVersionString(deps['typescript']);
         sources.push({ type: 'package.json', field: 'dependencies.typescript' });
     }
-    // tsconfig.json
     if (exists(node_path.join(projectPath, 'tsconfig.json'))) {
         confidence += 40;
         configPath = 'tsconfig.json';
         sources.push({ type: 'config-file', path: 'tsconfig.json' });
     }
-    // tsconfig.*.json variants
     const tsconfigVariants = ['tsconfig.build.json', 'tsconfig.lib.json', 'tsconfig.spec.json', 'tsconfig.app.json'];
     for (const variant of tsconfigVariants) {
         if (exists(node_path.join(projectPath, variant))) {
@@ -5411,7 +5340,6 @@ function typescriptDetector(projectPath, packageJson) {
             break;
         }
     }
-    // @types packages
     const typePackages = keys(deps).filter((d) => d.startsWith('@types/'));
     if (typePackages.length > 0) {
         confidence += 10;
@@ -5445,24 +5373,20 @@ function flowDetector(projectPath, packageJson) {
     let configPath;
     let version;
     const deps = collectAllDependencies(pkg);
-    // flow-bin package
     if (deps['flow-bin']) {
         confidence += 60;
         version = parseVersionString(deps['flow-bin']);
         sources.push({ type: 'package.json', field: 'dependencies.flow-bin' });
     }
-    // .flowconfig
     if (exists(node_path.join(projectPath, '.flowconfig'))) {
         confidence += 40;
         configPath = '.flowconfig';
         sources.push({ type: 'config-file', path: '.flowconfig' });
     }
-    // flow-typed directory
     if (exists(node_path.join(projectPath, 'flow-typed'))) {
         confidence += 10;
         sources.push({ type: 'directory', path: 'flow-typed/' });
     }
-    // @babel/preset-flow
     if (deps['@babel/preset-flow']) {
         confidence += 10;
         sources.push({ type: 'package.json', field: 'dependencies.@babel/preset-flow' });
@@ -5486,7 +5410,6 @@ function flowDetector(projectPath, packageJson) {
  * @returns `true` if the content contains JSDoc type annotations.
  */
 function hasJsDocTypes(content) {
-    // Check for JSDoc type annotations
     return (content.includes('@type {') ||
         content.includes('@param {') ||
         content.includes('@returns {') ||
@@ -5505,14 +5428,11 @@ function jsdocDetector(projectPath, packageJson) {
     const sources = [];
     let confidence = 0;
     const deps = collectAllDependencies(pkg);
-    // jsdoc package
     if (deps['jsdoc']) {
         confidence += 30;
         sources.push({ type: 'package.json', field: 'dependencies.jsdoc' });
     }
-    // typescript with checkJs (JSDoc type checking)
     if (deps['typescript']) {
-        // Check if checkJs is enabled in tsconfig
         const tsconfigPath = node_path.join(projectPath, 'tsconfig.json');
         const content = readFileIfExists(tsconfigPath);
         if (content) {
@@ -5529,12 +5449,10 @@ function jsdocDetector(projectPath, packageJson) {
             }
         }
     }
-    // Check for jsconfig.json (VS Code JS type checking)
     if (exists(node_path.join(projectPath, 'jsconfig.json'))) {
         confidence += 40;
         sources.push({ type: 'config-file', path: 'jsconfig.json' });
     }
-    // Sample check for JSDoc annotations in source files
     const srcDir = node_path.join(projectPath, 'src');
     if (exists(srcDir)) {
         try {
@@ -5616,8 +5534,6 @@ const allDetectors = {
 function isDetectAllOptions(value) {
     if (typeof value !== 'object' || value === null)
         return false;
-    // DetectAllOptions has skipCache or packageJson fields specifically
-    // PackageJson never has skipCache field
     return 'skipCache' in value || 'packageJson' in value;
 }
 /**
@@ -5649,9 +5565,7 @@ function isDetectAllOptions(value) {
  * ```
  */
 function detectAll(projectPath, packageJsonOrOptions) {
-    // Handle backward-compatible arguments
     const options = isDetectAllOptions(packageJsonOrOptions) ? packageJsonOrOptions : { packageJson: packageJsonOrOptions };
-    // Check cache first (unless skipCache is true)
     if (!options.skipCache) {
         const cached = detectAllCache.get(projectPath);
         if (cached) {
@@ -5702,7 +5616,6 @@ function detectAll(projectPath, packageJsonOrOptions) {
         legacyFrameworks: result.legacyFrameworks.map((f) => f.id),
         testingFrameworks: result.testingFrameworks.map((f) => f.id),
     });
-    // Cache the result
     detectAllCache.set(projectPath, result);
     return result;
 }
@@ -6119,7 +6032,6 @@ function detectNxVersion(workspacePath) {
     if (packageJson) {
         const nxVersion = packageJson.devDependencies?.['nx'] ?? packageJson.dependencies?.['nx'];
         if (nxVersion) {
-            // Strip semver range characters (^, ~, >=, etc.)
             return nxVersion.replace(/^[\^~>=<]+/, '');
         }
     }
@@ -6148,14 +6060,12 @@ function getNxWorkspaceInfo(workspacePath) {
     }
     const nxJson = readJsonFileIfExists(node_path.join(workspacePath, 'nx.json'));
     if (!nxJson) {
-        // Check for workspace.json as fallback (older NX)
         const workspaceJson = readJsonFileIfExists(node_path.join(workspacePath, 'workspace.json'));
         if (!workspaceJson) {
             nxLogger.debug('No nx.json or workspace.json found', { workspacePath });
             return null;
         }
         nxLogger.debug('Using legacy workspace.json', { workspacePath });
-        // Create minimal nx.json from workspace.json
         return {
             root: workspacePath,
             version: detectNxVersion(workspacePath),
@@ -6204,7 +6114,6 @@ function tryLoadDevkit() {
     }
     devkitLogger.debug('Attempting to load @nx/devkit');
     try {
-        // Dynamic require to avoid bundling
         const devkit = require('@nx/devkit');
         devkitLogger.debug('@nx/devkit loaded successfully');
         cachedResult = { available: true, devkit };
@@ -6301,7 +6210,6 @@ function readProjectJson(projectPath) {
  */
 function getProjectConfig(projectPath, workspacePath) {
     nxConfigLogger.debug('Getting project config', { projectPath, workspacePath });
-    // Try project.json first
     const projectJson = readProjectJson(projectPath);
     if (projectJson) {
         nxConfigLogger.debug('Using project.json config', { projectPath, name: projectJson.name });
@@ -6310,7 +6218,6 @@ function getProjectConfig(projectPath, workspacePath) {
             root: projectJson.root ?? node_path.relative(workspacePath, projectPath),
         };
     }
-    // Try to infer from package.json nx field
     const packageJson = readPackageJsonIfExists(projectPath);
     if (packageJson && typeof packageJson['nx'] === 'object') {
         nxConfigLogger.debug('Using package.json nx field', { projectPath, name: packageJson.name });
@@ -6339,13 +6246,11 @@ function scanForProjects(dirPath, workspacePath, projects, maxDepth, currentDept
     try {
         const entries = readDirectory(dirPath);
         for (const entry of entries) {
-            // Skip node_modules and hidden directories
             if (entry.name.startsWith('.') || entry.name === 'node_modules' || entry.name === 'dist') {
                 continue;
             }
             const fullPath = node_path.join(dirPath, entry.name);
             if (entry.isDirectory) {
-                // Check if this directory is an NX project
                 if (isNxProject(fullPath)) {
                     const config = getProjectConfig(fullPath, workspacePath);
                     if (config) {
@@ -6357,7 +6262,6 @@ function scanForProjects(dirPath, workspacePath, projects, maxDepth, currentDept
                         });
                     }
                 }
-                // Recursively scan subdirectories
                 scanForProjects(fullPath, workspacePath, projects, maxDepth, currentDepth + 1);
             }
         }
@@ -6375,12 +6279,10 @@ function scanForProjects(dirPath, workspacePath, projects, maxDepth, currentDept
  */
 function discoverNxProjects(workspacePath) {
     const projects = createMap();
-    // Check for workspace.json (older NX format)
     const workspaceJson = readJsonFileIfExists(node_path.join(workspacePath, 'workspace.json'));
     if (workspaceJson?.projects) {
         for (const [name, config] of entries(workspaceJson.projects)) {
             if (typeof config === 'string') {
-                // Path reference to project directory
                 const projectPath = node_path.join(workspacePath, config);
                 const projectConfig = getProjectConfig(projectPath, workspacePath);
                 if (projectConfig) {
@@ -6388,18 +6290,15 @@ function discoverNxProjects(workspacePath) {
                 }
             }
             else if (typeof config === 'object' && config !== null) {
-                // Inline config
                 projects.set(name, { name, ...config });
             }
         }
         return projects;
     }
-    // Scan for project.json files (newer NX format)
     const workspaceInfo = getNxWorkspaceInfo(workspacePath);
     const appsDir = workspaceInfo?.workspaceLayout.appsDir ?? 'apps';
     const libsDir = workspaceInfo?.workspaceLayout.libsDir ?? 'libs';
     const searchDirs = [appsDir, libsDir];
-    // Also check packages directory (common in some setups)
     if (exists(node_path.join(workspacePath, 'packages'))) {
         searchDirs.push('packages');
     }
@@ -6414,7 +6313,6 @@ function discoverNxProjects(workspacePath) {
             }
         }
     }
-    // Also check root-level projects (standalone projects in monorepo root)
     if (isNxProject(workspacePath)) {
         const config = readProjectJson(workspacePath);
         if (config) {
@@ -6447,10 +6345,8 @@ function buildSimpleProjectGraph(workspacePath, projects) {
             data: config,
         };
         dependencies[name] = [];
-        // Add implicit dependencies
         if (config.implicitDependencies) {
             for (const dep of config.implicitDependencies) {
-                // Skip negative dependencies (those starting with !)
                 if (!dep.startsWith('!')) {
                     dependencies[name].push({
                         target: dep,
@@ -6467,7 +6363,6 @@ function buildSimpleProjectGraph(workspacePath, projects) {
  * Known configuration file patterns organized by type.
  */
 const CONFIG_PATTERNS = {
-    // Package Management
     'package.json': {
         patterns: ['package.json'],
         format: 'json',
@@ -6494,14 +6389,12 @@ const CONFIG_PATTERNS = {
         description: 'NPM configuration',
         sensitive: true,
     },
-    // TypeScript
     tsconfig: {
         patterns: ['tsconfig.json', 'tsconfig.*.json'],
         format: 'jsonc',
         description: 'TypeScript configuration',
         canExtend: true,
     },
-    // Monorepo
     nx: {
         patterns: ['nx.json'],
         format: 'json',
@@ -6527,7 +6420,6 @@ const CONFIG_PATTERNS = {
         format: 'json',
         description: 'Lerna configuration',
     },
-    // Build Tools
     webpack: {
         patterns: ['webpack.config.js', 'webpack.config.ts', 'webpack.config.cjs', 'webpack.config.mjs'],
         format: 'js',
@@ -6558,7 +6450,6 @@ const CONFIG_PATTERNS = {
         format: 'json',
         description: 'SWC configuration',
     },
-    // Testing
     jest: {
         patterns: ['jest.config.js', 'jest.config.ts', 'jest.config.mjs'],
         description: 'Jest configuration',
@@ -6575,7 +6466,6 @@ const CONFIG_PATTERNS = {
         patterns: ['playwright.config.js', 'playwright.config.ts'],
         description: 'Playwright configuration',
     },
-    // Framework configs
     next: {
         patterns: ['next.config.js', 'next.config.mjs', 'next.config.ts'],
         format: 'js',
@@ -6601,7 +6491,6 @@ const CONFIG_PATTERNS = {
         format: 'js',
         description: 'Astro configuration',
     },
-    // Linting & Formatting
     eslint: {
         patterns: [
             'eslint.config.js',
@@ -6620,14 +6509,12 @@ const CONFIG_PATTERNS = {
         format: 'json',
         description: 'Prettier configuration',
     },
-    // Environment (sensitive)
     env: {
         patterns: ['.env', '.env.*', '*.env'],
         format: 'dotenv',
         description: 'Environment variables',
         sensitive: true,
     },
-    // Git
     '.gitignore': {
         patterns: ['.gitignore'],
         format: 'text',
@@ -6798,12 +6685,8 @@ function getConfigPaths(type) {
  *
  * @module @hyperfrontend/immutable-api-utils/built-in-copy/number
  */
-// Capture references at module initialization time
 const _parseInt = globalThis.parseInt;
 const _parseFloat = globalThis.parseFloat;
-// ============================================================================
-// Parsing
-// ============================================================================
 /**
  * (Safe copy) Parses a string and returns an integer.
  */
@@ -7158,11 +7041,9 @@ const analyzeLogger = createScopedLogger('project-scope:analyze');
  * @returns Detected workspace type
  */
 function detectWorkspaceType(projectPath) {
-    // Check for NX
     if (isNxWorkspace(projectPath) || findNxWorkspaceRoot(projectPath) !== null) {
         return 'nx';
     }
-    // Check for common monorepo markers
     if (exists(node_path.resolve(projectPath, 'turbo.json'))) {
         return 'turborepo';
     }
@@ -7172,7 +7053,6 @@ function detectWorkspaceType(projectPath) {
     if (exists(node_path.resolve(projectPath, 'pnpm-workspace.yaml'))) {
         return 'pnpm';
     }
-    // Check package.json for workspaces
     const pkg = readPackageJsonIfExists(projectPath);
     if (pkg?.workspaces) {
         if (exists(node_path.resolve(projectPath, 'yarn.lock'))) {
@@ -7181,7 +7061,7 @@ function detectWorkspaceType(projectPath) {
         if (exists(node_path.resolve(projectPath, 'package-lock.json'))) {
             return 'npm';
         }
-        return 'npm'; // default for workspaces
+        return 'npm';
     }
     return 'standalone';
 }
@@ -7193,15 +7073,12 @@ function detectWorkspaceType(projectPath) {
  * @returns True if the analysis should be performed
  */
 function shouldInclude(type, options) {
-    // If include list is specified, item must be in it
     if (options?.include && options.include.length > 0) {
         return options.include.includes(type);
     }
-    // If exclude list is specified, item must not be in it
     if (options?.exclude && options.exclude.length > 0) {
         return !options.exclude.includes(type);
     }
-    // Default: include everything
     return true;
 }
 /**
@@ -7250,28 +7127,22 @@ function normalizeConfigFormat(format) {
 function analyzeProject(projectPath, options) {
     const startTime = dateNow();
     const resolvedPath = node_path.resolve(projectPath);
-    // Set log level based on verbose option
     if (options?.verbose) {
         analyzeLogger.setLogLevel('debug');
     }
     analyzeLogger.debug('Starting project analysis', { path: resolvedPath, options });
-    // Read package.json once for reuse
     const packageJson = readPackageJsonIfExists(resolvedPath);
     analyzeLogger.debug('Package.json loaded', { found: !!packageJson, name: packageJson?.name });
-    // Detect project and workspace types
     const projectTypeDetection = detectProjectType(resolvedPath, {
         skipTechDetection: options?.depth === 'basic',
     });
     const projectType = projectTypeDetection.type;
     const workspaceType = detectWorkspaceType(resolvedPath);
     analyzeLogger.debug('Project type detected', { projectType, workspaceType });
-    // Get project name from package.json or directory name
     const projectName = packageJson?.name ?? node_path.basename(resolvedPath);
-    // Run all technology detectors
     const detections = shouldInclude('frameworks', options) || shouldInclude('buildTools', options) || shouldInclude('testing', options)
         ? detectAll(resolvedPath, packageJson ?? undefined)
         : null;
-    // Convert framework detections to FrameworkInfo
     const frameworks = shouldInclude('frameworks', options) && detections
         ? [
             ...detections.frontendFrameworks.map((d) => ({
@@ -7291,7 +7162,6 @@ function analyzeProject(projectPath, options) {
             })),
         ]
         : [];
-    // Convert build tool detections
     const buildTools = shouldInclude('buildTools', options) && detections
         ? detections.buildTools.map((d) => ({
             id: d.id,
@@ -7301,7 +7171,6 @@ function analyzeProject(projectPath, options) {
             confidence: d.confidence,
         }))
         : [];
-    // Convert testing framework detections
     const testingFrameworks = shouldInclude('testing', options) && detections
         ? detections.testingFrameworks.map((d) => ({
             id: d.id,
@@ -7312,7 +7181,6 @@ function analyzeProject(projectPath, options) {
             confidence: d.confidence,
         }))
         : [];
-    // Discover entry points
     const entryPoints = shouldInclude('entryPoints', options)
         ? discoverEntryPoints(resolvedPath).map((e) => ({
             path: e.path,
@@ -7320,7 +7188,6 @@ function analyzeProject(projectPath, options) {
             confidence: e.confidence,
         }))
         : [];
-    // Detect configuration files
     const configFiles = shouldInclude('configs', options)
         ? detectConfigs(resolvedPath).map((c) => ({
             path: c.path,
@@ -7329,7 +7196,6 @@ function analyzeProject(projectPath, options) {
             tool: c.type,
         }))
         : [];
-    // Get dependency summary
     let dependencies;
     if (shouldInclude('dependencies', options)) {
         const deps = getProjectDependencies(resolvedPath);
@@ -7350,7 +7216,6 @@ function analyzeProject(projectPath, options) {
             total: 0,
         };
     }
-    // Build metadata
     const metadata = {
         timestamp: createDate(),
         durationMs: dateNow() - startTime,
@@ -7425,15 +7290,12 @@ function formatWorkspaceType(type) {
  */
 function formatAnalysisText(result) {
     const lines = [];
-    // Header
     lines.push(`Project Analysis: ${result.name}`);
     lines.push('='.repeat(30));
     lines.push('');
-    // Basic info
     lines.push(`Type:           ${formatProjectType(result.projectType)}`);
     lines.push(`Workspace:      ${formatWorkspaceType(result.workspaceType)}`);
     lines.push('');
-    // Frameworks
     if (result.frameworks.length > 0) {
         lines.push('Frameworks:');
         for (const framework of result.frameworks) {
@@ -7447,7 +7309,6 @@ function formatAnalysisText(result) {
         }
         lines.push('');
     }
-    // Build tools
     if (result.buildTools.length > 0) {
         lines.push('Build Tools:');
         for (const tool of result.buildTools) {
@@ -7456,7 +7317,6 @@ function formatAnalysisText(result) {
         }
         lines.push('');
     }
-    // Testing
     if (result.testingFrameworks.length > 0) {
         lines.push('Testing:');
         for (const framework of result.testingFrameworks) {
@@ -7465,7 +7325,6 @@ function formatAnalysisText(result) {
         }
         lines.push('');
     }
-    // Entry points
     if (result.entryPoints.length > 0) {
         lines.push('Entry Points:');
         for (const entry of result.entryPoints.slice(0, 5)) {
@@ -7476,7 +7335,6 @@ function formatAnalysisText(result) {
         }
         lines.push('');
     }
-    // Configurations
     if (result.configFiles.length > 0) {
         lines.push('Configurations:');
         for (const config of result.configFiles.slice(0, 8)) {
@@ -7487,7 +7345,6 @@ function formatAnalysisText(result) {
         }
         lines.push('');
     }
-    // Dependencies summary
     lines.push('Dependencies:');
     lines.push(`  Production:    ${result.dependencies.production}`);
     lines.push(`  Development:   ${result.dependencies.development}`);
@@ -7574,7 +7431,7 @@ function parseAnalyzeArgs(args) {
             exclude: { type: 'string', short: 'e' },
         },
         allowPositionals: true,
-        strict: false, // Allow global options to pass through
+        strict: false,
     });
     const format = values.format;
     const depth = values.depth;
@@ -7633,7 +7490,6 @@ const analyzeCommandDef = {
     description: 'Analyze project structure and tech stack',
     execute(args, globalOptions) {
         const options = parseAnalyzeArgs(args);
-        // Global --json flag overrides format
         if (globalOptions.json) {
             options.format = 'json';
         }
@@ -7940,12 +7796,10 @@ Examples:
 function formatDependencyList(deps, maxItems = 20) {
     const lines = [];
     const depEntries = entries(deps);
-    // Sort alphabetically
     depEntries.sort((a, b) => a[0].localeCompare(b[0]));
     const displayCount = min(depEntries.length, maxItems);
     for (let i = 0; i < displayCount; i++) {
         const [name, version] = depEntries[i];
-        // Pad name to align versions
         const paddedName = name.padEnd(30);
         lines.push(`  ${paddedName} ${version}`);
     }
@@ -8188,13 +8042,10 @@ function buildTree(rootPath, walkEntries, options) {
         isDirectory: true,
         children: [],
     };
-    // Create a map for quick lookup
     const nodeMap = createMap();
     nodeMap.set('.', root);
-    // Sort entries by path for proper parent-child relationships
     const sortedEntries = [...walkEntries].sort((a, b) => a.relativePath.localeCompare(b.relativePath));
     for (const entry of sortedEntries) {
-        // Skip based on filters
         if (options.dirsOnly && !entry.isDirectory)
             continue;
         if (options.filesOnly && entry.isDirectory)
@@ -8205,7 +8056,6 @@ function buildTree(rootPath, walkEntries, options) {
             isDirectory: entry.isDirectory,
             children: [],
         };
-        // Add size/modified if requested
         if ((options.showSize || options.showModified) && entry.isFile) {
             const stats = getFileStat(entry.path);
             if (stats) {
@@ -8215,9 +8065,8 @@ function buildTree(rootPath, walkEntries, options) {
                     node.modified = stats.modified;
             }
         }
-        // Find parent
         const parts = entry.relativePath.split('/');
-        parts.pop(); // Remove current entry name
+        parts.pop();
         const parentPath = parts.join('/') || '.';
         const parent = nodeMap.get(parentPath);
         if (parent) {
@@ -8238,11 +8087,9 @@ function buildTree(rootPath, walkEntries, options) {
  */
 function renderTreeText(node, options, prefix = '', isLast = true) {
     const lines = [];
-    // Current line
     const connector = isLast ? '└── ' : '├── ';
     const dirMark = node.isDirectory ? '/' : '';
     let line = `${prefix}${connector}${node.name}${dirMark}`;
-    // Add size/modified
     const meta = [];
     if (options.showSize && node.size !== undefined) {
         meta.push(formatSize(node.size));
@@ -8254,10 +8101,8 @@ function renderTreeText(node, options, prefix = '', isLast = true) {
         line += `  [${meta.join(' ')}]`;
     }
     lines.push(line);
-    // Process children
     const childPrefix = prefix + (isLast ? '    ' : '│   ');
     const sortedChildren = [...node.children].sort((a, b) => {
-        // Directories first, then alphabetically
         if (a.isDirectory && !b.isDirectory)
             return -1;
         if (!a.isDirectory && b.isDirectory)
@@ -8281,9 +8126,7 @@ function renderTreeText(node, options, prefix = '', isLast = true) {
  */
 function formatTreeText(rootPath, tree, options) {
     const lines = [];
-    // Root line
     lines.push(node_path.basename(rootPath));
-    // Count stats
     let dirCount = 0;
     let fileCount = 0;
     /**
@@ -8302,7 +8145,6 @@ function formatTreeText(rootPath, tree, options) {
             countNodes(child);
         }
     }
-    // Render children of root
     const sortedChildren = [...tree.children].sort((a, b) => {
         if (a.isDirectory && !b.isDirectory)
             return -1;
@@ -8316,7 +8158,6 @@ function formatTreeText(rootPath, tree, options) {
         lines.push(...renderTreeText(child, options, '', isLast));
         countNodes(child);
     }
-    // Summary
     lines.push('');
     const dirText = dirCount === 1 ? '1 directory' : `${dirCount} directories`;
     const fileText = fileCount === 1 ? '1 file' : `${fileCount} files`;
@@ -8377,7 +8218,6 @@ function parseTreeArgs(args) {
 function treeCommand(options) {
     const rootPath = options.path ? node_path.resolve(options.path) : process.cwd();
     try {
-        // Collect entries via walk
         const walkEntries = [];
         walkDirectory(rootPath, (entry) => {
             walkEntries.push(entry);
@@ -8387,9 +8227,7 @@ function treeCommand(options) {
             ignorePatterns: options.ignore,
             includeHidden: false,
         });
-        // Build tree structure
         const tree = buildTree(rootPath, walkEntries, options);
-        // Format output
         let output;
         if (options.format === 'json') {
             output = formatTreeJson(tree);
@@ -8448,6 +8286,9 @@ Examples:
 
 /** Logger for CLI operations */
 const cliLogger = createScopedLogger('project-scope:cli');
+/** Output printer for user-facing CLI output (help, version, command results). */
+const output = createLogger(error, undefined, log);
+output.setLogLevel('log');
 /** Library version */
 const VERSION = '0.1.0';
 /**
@@ -8463,7 +8304,7 @@ const commands = {
  * Print general help information.
  */
 function printHelp() {
-    log(`
+    output.log(`
 project-scope <command> [options]
 
 A tool for analyzing JavaScript/TypeScript project structure and tech stack.
@@ -8495,7 +8336,7 @@ Examples:
  * Print CLI version.
  */
 function printVersion() {
-    log(`project-scope v${VERSION}`);
+    output.log(`project-scope v${VERSION}`);
 }
 /**
  * Parse global options from command line arguments.
@@ -8556,24 +8397,24 @@ function run(args) {
         setGlobalLogLevel('debug');
     }
     cliLogger.debug('CLI invoked', { args, globalOptions });
+    const commandName = args[0];
     if (globalOptions.version) {
         printVersion();
         return { exitCode: 0 };
     }
-    if (globalOptions.help && (args.length === 1 || !commands[args[0]])) {
+    if (globalOptions.help && (args.length === 1 || !commands[commandName])) {
         printHelp();
         return { exitCode: 0 };
     }
-    const commandName = args[0];
     const command = commands[commandName];
     if (!command) {
         cliLogger.warn('Unknown command requested', { commandName });
-        error(`Unknown command: ${commandName}`);
-        error('Run "project-scope --help" for usage information.');
+        output.error(`Unknown command: ${commandName}`);
+        output.error('Run "project-scope --help" for usage information.');
         return { exitCode: 1, error: `Unknown command: ${commandName}` };
     }
     if (globalOptions.help) {
-        log(command.getHelp());
+        output.log(command.getHelp());
         return { exitCode: 0 };
     }
     const commandArgs = args.slice(1);
@@ -8581,11 +8422,11 @@ function run(args) {
     const result = command.execute(commandArgs, globalOptions);
     cliLogger.debug('Command completed', { commandName, exitCode: result.exitCode });
     if (result.output) {
-        log(result.output);
+        output.log(result.output);
     }
     if (result.error) {
         cliLogger.error('Command error', { commandName, error: result.error });
-        error(result.error);
+        output.error(result.error);
     }
     return result;
 }
@@ -8621,25 +8462,22 @@ const BINARY_SIGNATURES = [
  */
 function detectEncodingInfo(buffer) {
     encodingLogger.debug('Detecting encoding info', { bufferSize: buffer.length });
-    // Check for UTF-8 BOM
     if (buffer.length >= 3) {
         if (buffer[0] === UTF8_BOM_BYTES[0] && buffer[1] === UTF8_BOM_BYTES[1] && buffer[2] === UTF8_BOM_BYTES[2]) {
             encodingLogger.debug('Detected UTF-8 BOM');
             return { type: 'text', encoding: 'utf-8', hasBom: true };
         }
     }
-    // Check for UTF-16 BOMs
     if (buffer.length >= 2) {
         if (buffer[0] === UTF16_BE_BOM_BYTES[0] && buffer[1] === UTF16_BE_BOM_BYTES[1]) {
             encodingLogger.debug('Detected UTF-16 BE BOM');
-            return { type: 'text', encoding: 'utf16le', hasBom: true }; // Node treats BE through utf16le
+            return { type: 'text', encoding: 'utf16le', hasBom: true };
         }
         if (buffer[0] === UTF16_LE_BOM_BYTES[0] && buffer[1] === UTF16_LE_BOM_BYTES[1]) {
             encodingLogger.debug('Detected UTF-16 LE BOM');
             return { type: 'text', encoding: 'utf16le', hasBom: true };
         }
     }
-    // Check for binary signatures
     for (const { signature, description } of BINARY_SIGNATURES) {
         if (buffer.length >= signature.length) {
             let matches = true;
@@ -8655,7 +8493,6 @@ function detectEncodingInfo(buffer) {
             }
         }
     }
-    // Check for null bytes (usually indicates binary)
     const sampleSize = min(buffer.length, 8000);
     for (let i = 0; i < sampleSize; i++) {
         if (buffer[i] === 0) {
@@ -8674,22 +8511,18 @@ function detectEncodingInfo(buffer) {
  */
 function detectEncoding(buffer) {
     if (buffer.length >= 3) {
-        // Check for UTF-8 BOM
         if (buffer[0] === UTF8_BOM_BYTES[0] && buffer[1] === UTF8_BOM_BYTES[1] && buffer[2] === UTF8_BOM_BYTES[2]) {
             return 'utf-8';
         }
     }
     if (buffer.length >= 2) {
-        // Check for UTF-16 LE BOM
         if (buffer[0] === UTF16_LE_BOM_BYTES[0] && buffer[1] === UTF16_LE_BOM_BYTES[1]) {
             return 'utf16le';
         }
-        // Check for UTF-16 BE BOM
         if (buffer[0] === UTF16_BE_BOM_BYTES[0] && buffer[1] === UTF16_BE_BOM_BYTES[1]) {
-            return 'utf16le'; // Node.js handles BE through utf16le
+            return 'utf16le';
         }
     }
-    // Default to UTF-8
     return 'utf-8';
 }
 /**
@@ -8759,10 +8592,8 @@ function bufferToString(content, encoding) {
         convertLogger.debug('Using provided encoding', { encoding });
         return content.toString(encoding);
     }
-    // Auto-detect and convert
     const info = detectEncodingInfo(content);
     if (info.type === 'text') {
-        // Remove BOM if present
         let offset = 0;
         if (info.hasBom) {
             offset = info.encoding === 'utf-8' ? 3 : 2;
@@ -8815,19 +8646,14 @@ function detectCaseSensitivity() {
     if (cachedCaseSensitive !== null) {
         return cachedCaseSensitive;
     }
-    // Quick check based on platform
     if (process.platform === 'win32') {
         cachedCaseSensitive = false;
         return false;
     }
-    // macOS is typically case-insensitive by default
     if (process.platform === 'darwin') {
-        // Could be case-sensitive HFS+/APFS, but assume insensitive by default
         cachedCaseSensitive = false;
         return false;
     }
-    // Test actual file system behavior for Linux and others
-    // Use mkdtempSync to create a secure temporary directory
     let secureTestDir = null;
     try {
         secureTestDir = node_fs.mkdtempSync(node_path.join(node_os.tmpdir(), 'case-sensitivity-test-'));
@@ -8838,12 +8664,9 @@ function detectCaseSensitivity() {
         node_fs.unlinkSync(testFile);
     }
     catch {
-        // Default to case-sensitive on Linux/Unix if test fails
-        // (win32 and darwin already returned early, so we're on a case-sensitive platform)
         cachedCaseSensitive = true;
     }
     finally {
-        // Clean up the secure temporary directory
         if (secureTestDir) {
             try {
                 node_fs.rmdirSync(secureTestDir);
@@ -8978,7 +8801,6 @@ function normalizeLineEndings(content, style = 'lf') {
     else {
         target = style === 'crlf' ? CRLF : LF;
     }
-    // First normalize all to LF, then convert to target
     const normalized = content.replace(/\r\n/g, LF).replace(/\r/g, LF);
     if (target === LF) {
         return normalized;
@@ -8993,7 +8815,6 @@ function normalizeLineEndings(content, style = 'lf') {
  */
 function detectLineEnding(content) {
     const hasCRLF = content.includes(CRLF);
-    // Check for LF that is NOT part of CRLF
     const hasLFOnly = content.includes('\n') && content.replace(/\r\n/g, '').includes('\n');
     if (hasCRLF && hasLFOnly)
         return 'mixed';
@@ -9471,7 +9292,6 @@ function createFsTree(root, options) {
             }
             const prefix = normalPath === '.' || normalPath === '' ? '' : normalPath + '/';
             for (const [changedPath, change] of _changes) {
-                // Handle root-level files
                 if (prefix === '') {
                     const childName = changedPath.split('/')[0];
                     if (change.type === 'DELETE' && !changedPath.includes('/')) {
@@ -9488,7 +9308,6 @@ function createFsTree(root, options) {
                 const relativePath = changedPath.slice(prefix.length);
                 const childName = relativePath.split('/')[0];
                 if (change.type === 'DELETE') {
-                    // Only remove if it's a direct child being deleted
                     if (!relativePath.includes('/')) {
                         childSet.delete(childName);
                     }
@@ -9574,12 +9393,10 @@ const factoryLogger = createScopedLogger('project-scope:vfs:factory');
 function createTree(root, options) {
     const normalizedRoot = normalizePath(root);
     factoryLogger.debug('createTree', { root: normalizedRoot });
-    // Validate root exists
     if (!exists(normalizedRoot)) {
         factoryLogger.warn('createTree failed: root does not exist', { root: normalizedRoot });
         throw createError(`Root directory does not exist: ${normalizedRoot}`);
     }
-    // Validate root is a directory
     if (!isDirectory(normalizedRoot)) {
         factoryLogger.warn('createTree failed: root is not a directory', { root: normalizedRoot });
         throw createError(`Root path is not a directory: ${normalizedRoot}`);
@@ -9626,7 +9443,6 @@ const vfsLogger = createScopedLogger('project-scope:vfs');
 function commitChanges(tree, options) {
     const changes = tree.listChanges();
     const appliedChanges = [];
-    // Set log level based on verbose option
     if (options?.verbose) {
         vfsLogger.setLogLevel('debug');
     }
@@ -9638,7 +9454,6 @@ function commitChanges(tree, options) {
         changes: [],
         dryRun: options?.dryRun ?? false,
     };
-    // Dry run - just count changes without writing
     if (options?.dryRun) {
         for (const change of changes) {
             switch (change.type) {
@@ -9656,7 +9471,6 @@ function commitChanges(tree, options) {
         result.changes = changes;
         return result;
     }
-    // Sort changes: deletes first (to free names), then creates, then updates
     const sortedChanges = [...changes].sort((a, b) => {
         const order = { DELETE: 0, CREATE: 1, UPDATE: 2 };
         return order[a.type] - order[b.type];
@@ -9669,11 +9483,9 @@ function commitChanges(tree, options) {
                 case 'UPDATE':
                     /* istanbul ignore if -- content is always defined for CREATE/UPDATE from tree.write() */
                     if (change.content !== undefined) {
-                        // Ensure directory exists
                         const dir = node_path.dirname(absPath);
                         ensureDir(dir);
                         node_fs.writeFileSync(absPath, change.content);
-                        // Apply permissions if specified
                         if (change.mode !== undefined) {
                             node_fs.chmodSync(absPath, change.mode);
                         }
@@ -9691,7 +9503,6 @@ function commitChanges(tree, options) {
                             node_fs.unlinkSync(absPath);
                         }
                         catch {
-                            // Try recursive delete for directories
                             node_fs.rmSync(absPath, { recursive: true });
                         }
                     }
@@ -9705,7 +9516,6 @@ function commitChanges(tree, options) {
             }
         }
         catch (error) {
-            // On error, throw with context
             vfsLogger.error('Commit failed', { path: change.path, type: change.type });
             const message = error instanceof Error
                 ? `Failed to ${change.type.toLowerCase()} ${change.path}: ${error.message}`
@@ -9714,7 +9524,6 @@ function commitChanges(tree, options) {
         }
     }
     result.changes = appliedChanges;
-    // Clear the tree's pending changes after successful commit
     tree.clearChanges();
     return result;
 }
@@ -9790,18 +9599,14 @@ function backtrackLcs(table, oldLines, newLines) {
  * @returns Filtered DiffLine array
  */
 function operationsToDiffLines(operations, contextLines) {
-    // Mark which lines should be included (changes + context)
     const include = new Array(operations.length).fill(false);
-    // First pass: mark all changes
     for (let i = 0; i < operations.length; i++) {
         if (operations[i].type !== 'same') {
-            // Mark this line and context around it
             for (let j = max(0, i - contextLines); j <= min(operations.length - 1, i + contextLines); j++) {
                 include[j] = true;
             }
         }
     }
-    // Second pass: convert to DiffLine
     const lines = [];
     let oldLineNum = 1;
     let newLineNum = 1;
@@ -9823,7 +9628,6 @@ function operationsToDiffLines(operations, contextLines) {
             }
         }
         else {
-            // Skip but update line numbers
             if (op.type === 'same') {
                 oldLineNum++;
                 newLineNum++;
@@ -9850,7 +9654,6 @@ function bufferToLines(content) {
     if (!content)
         return [];
     const text = content.toString('utf-8');
-    // Split by newline, keeping empty last line if present
     return text.split('\n');
 }
 /**
@@ -9877,9 +9680,7 @@ function generateDiff(change, options = {}) {
     diffLogger.debug('generateDiff', { path: change.path, type: change.type, contextLines });
     const oldLines = bufferToLines(change.originalContent);
     const newLines = bufferToLines(change.content);
-    // Handle edge cases
     if (change.type === 'CREATE') {
-        // All lines are additions
         const lines = newLines
             .filter((line) => line !== '' || newLines.indexOf(line) !== newLines.length - 1 || newLines.length === 1)
             .map((content, idx) => ({
@@ -9887,7 +9688,6 @@ function generateDiff(change, options = {}) {
             line: idx + 1,
             content,
         }));
-        // Filter out empty trailing line from split
         const filteredLines = lines.filter((l, i) => !(i === lines.length - 1 && l.content === '' && lines.length > 1));
         return {
             path: change.path,
@@ -9897,13 +9697,11 @@ function generateDiff(change, options = {}) {
         };
     }
     if (change.type === 'DELETE') {
-        // All lines are deletions
         const lines = oldLines.map((content, idx) => ({
             type: 'remove',
             line: idx + 1,
             content,
         }));
-        // Filter out empty trailing line from split
         const filteredLines = lines.filter((l, i) => !(i === lines.length - 1 && l.content === '' && lines.length > 1));
         return {
             path: change.path,
@@ -9912,7 +9710,6 @@ function generateDiff(change, options = {}) {
             deletions: filteredLines.length,
         };
     }
-    // UPDATE: compute actual diff
     const table = computeLcsTable(oldLines, newLines);
     const operations = backtrackLcs(table, oldLines, newLines);
     const lines = operationsToDiffLines(operations, contextLines);
@@ -9946,13 +9743,11 @@ function generateDiff(change, options = {}) {
  */
 function formatUnifiedDiff(diff) {
     const lines = [];
-    // Header
     lines.push(`--- a/${diff.path}`);
     lines.push(`+++ b/${diff.path}`);
     if (diff.lines.length === 0) {
         return lines.join('\n');
     }
-    // Group lines into hunks
     const hunks = [];
     const currentHunk = [];
     for (const line of diff.lines) {
@@ -9961,9 +9756,7 @@ function formatUnifiedDiff(diff) {
     if (currentHunk.length > 0) {
         hunks.push(currentHunk);
     }
-    // Output hunks
     for (const hunk of hunks) {
-        // Calculate hunk header
         const contextAndRemove = hunk.filter((l) => l.type === 'context' || l.type === 'remove');
         const contextAndAdd = hunk.filter((l) => l.type === 'context' || l.type === 'add');
         const oldStart = hunk.find((l) => l.type === 'context' || l.type === 'remove')?.line ?? 1;
@@ -9971,7 +9764,6 @@ function formatUnifiedDiff(diff) {
         const oldCount = contextAndRemove.length;
         const newCount = contextAndAdd.length;
         lines.push(`@@ -${oldStart},${oldCount} +${newStart},${newCount} @@`);
-        // Output lines
         for (const line of hunk) {
             const prefix = line.type === 'add' ? '+' : line.type === 'remove' ? '-' : ' ';
             lines.push(`${prefix}${line.content}`);