fix: handle absolute paths in findMatchingFilesFallback method

- Added isAbsolute check to handle both absolute and relative search paths
- Fixed test to use relative path like the actual code does
- Should resolve the fallback method returning 0 files issue

Author: M-Chris

src/core/modules/auto-discovery.ts

@@ -1,6 +1,6 @@
 // Auto-discovery system for Moro modules
 import { readdirSync, statSync } from 'fs';
-import { join, extname, relative } from 'path';
+import { join, extname, relative, isAbsolute } from 'path';
 import { ModuleConfig } from '../../types/module';
 import { DiscoveryOptions } from '../../types/discovery';
 import { ModuleDefaultsConfig } from '../../types/config';
@@ -431,8 +431,8 @@ export class ModuleDiscovery {
       recursive: true,
     } as ModuleDefaultsConfig['autoDiscovery'];
 
-    // Resolve the full search path
-    const fullSearchPath = join(this.baseDir, searchPath);
+    // Handle both absolute and relative paths
+    const fullSearchPath = isAbsolute(searchPath) ? searchPath : join(this.baseDir, searchPath);
     console.error(`FALLBACK: Full search path: ${fullSearchPath}`);
 
     // Check if search path exists

tests/unit/core/auto-discovery.test.ts

@@ -107,14 +107,14 @@ describe('Enhanced Module Auto-Discovery System', () => {
         let modules: any[] = [];
         let discoveryError: any = null;
         let debugInfo: string[] = [];
-        
+
         try {
           debugInfo.push('=== TESTING CORE LOGIC DIRECTLY ===');
-          
+
           // Test the individual methods to see where it fails
           const testSearchPath = join(tempDir, 'modules');
           debugInfo.push(`Testing search path: ${testSearchPath}`);
-          
+
           // Check if the ModuleDiscovery instance can access the path
           const { access } = await import('fs/promises');
           try {
@@ -123,40 +123,40 @@ describe('Enhanced Module Auto-Discovery System', () => {
           } catch (e) {
             debugInfo.push(`Search path accessible: NO - ${e}`);
           }
-          
+
           // Test the glob detection logic directly
           const isCI = process.env.CI === 'true' || process.env.GITHUB_ACTIONS === 'true';
           const nodeVersion = process.version;
           debugInfo.push(`CI detected: ${isCI}, Node: ${nodeVersion}`);
-          
-          // DIRECTLY TEST THE FALLBACK METHOD
+
+          // DIRECTLY TEST THE FALLBACK METHOD (use relative path like the real code)
           const fallbackFiles = await (discovery as any).findMatchingFilesFallback(
-            testSearchPath,
+            'modules', // Use relative path, not absolute
             config.patterns,
             config.ignorePatterns,
             config.maxDepth
           );
           debugInfo.push(`Fallback found ${fallbackFiles.length} files: ${JSON.stringify(fallbackFiles)}`);
-          
+
           // DIRECTLY TEST PATTERN MATCHING
           const testPaths = ['modules/users/index.ts', 'modules/orders/index.ts'];
           const testPatterns = ['**/index.{ts,js}', '**/*.module.{ts,js}'];
-          
+
           for (const testPath of testPaths) {
             for (const testPattern of testPatterns) {
               const matches = (discovery as any).matchesSimplePattern(testPath, testPattern);
               debugInfo.push(`Pattern test: "${testPath}" vs "${testPattern}" = ${matches}`);
             }
           }
-          
+
           // Now call the discovery method
           modules = await discovery.discoverModulesAdvanced(config);
           debugInfo.push('=== DISCOVERY COMPLETED ===');
         } catch (error) {
           debugInfo.push(`=== ERROR === ${error}`);
           discoveryError = error;
         }
-        
+
         debugInfo.push(`Final modules count: ${modules.length}`);
         debugInfo.push(`Final modules: ${JSON.stringify(modules.map(m => ({ name: m.name, version: m.version })))}`);
         debugInfo.push(`Discovery error: ${discoveryError}`);