fix get_pr_comments to correctly resolve path now that we use amp sdk

Author: sayansisodiya

src/config.ts

@@ -1,7 +1,9 @@
-import { readFileSync } from 'fs';
+import { existsSync, readFileSync } from 'fs';
 import * as yaml from 'js-yaml';
 import { config } from 'dotenv';
 import { z } from 'zod';
+import path from 'path';
+import { fileURLToPath } from 'url';
 
 // Load environment variables
 config();
@@ -42,8 +44,23 @@ class ConfigLoader {
   private static instance: ConfigLoader;
   private config: Config;
 
+  private resolveConfigPath(): string {
+    const candidates: (string | undefined)[] = [
+      process.env.CONFIG_PATH,
+      process.env.GITHUB_APP_CWD && path.join(process.env.GITHUB_APP_CWD, 'config.yml'),
+      path.resolve(path.dirname(fileURLToPath(import.meta.url)), '..', 'config.yml'),
+      path.resolve(process.cwd(), 'config.yml'),
+    ];
+    
+    const found = candidates.filter(Boolean).find(p => existsSync(p as string)) as string | undefined;
+    if (!found) {
+      throw new Error(`config.yml not found. Searched: ${candidates.filter(Boolean).join(', ')}`);
+    }
+    return found;
+  }
+
   private constructor() {
-    const configFile = readFileSync('config.yml', 'utf8');
+    const configFile = readFileSync(this.resolveConfigPath(), 'utf8');
     const rawConfig = yaml.load(configFile);
     const processedConfig = this.processEnvVars(rawConfig);
     this.config = ConfigSchema.parse(processedConfig);

toolbox/get-pr-comments.js

@@ -2,11 +2,24 @@
 
 import fs from 'fs';
 import path from 'path';
-import { pathToFileURL } from 'url';
+import { fileURLToPath, pathToFileURL } from 'url';
 
-// Resolve compiled code from project root → dist/
-const rootDir = path.resolve(process.cwd());
-const distImport = async (rel) => import(pathToFileURL(path.join(rootDir, 'dist', rel)).href);
+// Resolve compiled code relative to script location (not cwd)
+const __filename = fileURLToPath(import.meta.url);
+const __dirname = path.dirname(__filename);
+
+const candidateDistDirs = [
+  path.resolve(__dirname, '..'),          // when script is in dist/toolbox
+  path.resolve(__dirname, '..', 'dist'),  // when script is in toolbox/
+  path.resolve(process.cwd(), 'dist'),    // fallback
+];
+
+const distDir = candidateDistDirs.find(d => fs.existsSync(path.join(d, 'config.js')));
+if (!distDir) {
+  throw new Error(`Cannot locate dist/ (searched: ${candidateDistDirs.join(', ')})`);
+}
+
+const distImport = async (rel) => import(pathToFileURL(path.join(distDir, rel)).href);
 
 const action = process.env.TOOLBOX_ACTION;