fix(agent-loader): ignore node_modules and hidden folders in agent discovery and fix integration test timeouts

Author: Varun-S10

dev/src/utils/agent_loader.ts

@@ -492,6 +492,12 @@ export class AgentLoader {
         }
 
         if (fileOrDir.isDirectory) {
+          if (
+            fileOrDir.name === 'node_modules' ||
+            fileOrDir.name.startsWith('.')
+          ) {
+            return;
+          }
           return this.loadAgentFromDirectory(fileOrDir);
         }
       }),

tests/integration/agent_loader/agent_dirname_test.ts

@@ -13,7 +13,8 @@ import {sendInput} from '../test_case_utils.js';
 
 const execAsync = promisify(exec);
 const dirname = process.cwd();
-const TEST_EXECUTION_TIMEOUT = 40000;
+const TEST_EXECUTION_TIMEOUT = 60000;
+const HOOK_TIMEOUT = 120000;
 
 describe.each(['__dirname', '__filename', 'import_meta_url'])(
   'Agent with %s',
@@ -26,7 +27,7 @@ describe.each(['__dirname', '__filename', 'import_meta_url'])(
 
     beforeAll(async () => {
       await execAsync('npm install', {cwd: projectPath});
-    }, TEST_EXECUTION_TIMEOUT);
+    }, HOOK_TIMEOUT);
 
     it(
       'should run agent and load params from file nearby via package.json script',
@@ -56,6 +57,6 @@ describe.each(['__dirname', '__filename', 'import_meta_url'])(
       await fs
         .unlink(path.join(projectPath, 'package-lock.json'))
         .catch(() => {});
-    }, TEST_EXECUTION_TIMEOUT);
+    }, HOOK_TIMEOUT);
   },
 );

tests/integration/app_loader/app_loader_test.ts

@@ -15,7 +15,8 @@ import {sendInput} from '../test_case_utils.js';
 
 const execAsync = promisify(exec);
 const dirname = process.cwd();
-const TEST_EXECUTION_TIMEOUT = 40000;
+const TEST_EXECUTION_TIMEOUT = 60000;
+const HOOK_TIMEOUT = 120000;
 
 describe('App loader CLI integration', () => {
   describe.each(['app_ts', 'app_js', 'app_default'])(
@@ -29,7 +30,7 @@ describe('App loader CLI integration', () => {
 
       beforeAll(async () => {
         await execAsync('npm install', {cwd: projectPath});
-      }, TEST_EXECUTION_TIMEOUT);
+      }, HOOK_TIMEOUT);
 
       it(
         'should run app via package.json start script and get responses',
@@ -62,7 +63,7 @@ describe('App loader CLI integration', () => {
         await fs
           .unlink(path.join(projectPath, 'package-lock.json'))
           .catch(() => {});
-      }, TEST_EXECUTION_TIMEOUT);
+      }, HOOK_TIMEOUT);
     },
   );
 });
@@ -77,7 +78,8 @@ describe('AgentLoader discovery and loading integration', () => {
   beforeAll(async () => {
     await execAsync('npm install', {cwd: projectPath});
     loader = new AgentLoader(projectPath);
-  }, TEST_EXECUTION_TIMEOUT);
+    await loader.preloadAgents();
+  }, HOOK_TIMEOUT);
 
   it(
     'should discover apps vs agents across directories and standalone files',
@@ -138,5 +140,5 @@ describe('AgentLoader discovery and loading integration', () => {
     await fs
       .unlink(path.join(projectPath, 'package-lock.json'))
       .catch(() => {});
-  }, TEST_EXECUTION_TIMEOUT);
+  }, HOOK_TIMEOUT);
 });