Merge pull request #155 from tech-sushant/rca-fix

refactor: change testId type from string to number across RCA

Author: gaurav-singh-9227

src/tools/rca-agent-utils/constants.ts

@@ -3,10 +3,10 @@ import { TestStatus } from "./types.js";
 
 export const FETCH_RCA_PARAMS = {
   testId: z
-    .array(z.string())
+    .array(z.number().int())
     .max(3)
     .describe(
-      "Array of test IDs to fetch RCA data for (maximum 3 IDs). If not provided, use the listTestIds tool get all failed testcases. If more than 3 IDs are provided, only the first 3 will be processed.",
+      "Array of integer test IDs to fetch RCA data for (maximum 3 IDs). These must be numeric test IDs, not session IDs or strings. If not provided, use the listTestIds tool to get all failed testcases. If more than 3 IDs are provided, only the first 3 will be processed.",
     ),
 };
 

src/tools/rca-agent-utils/rca-data.ts

@@ -97,11 +97,11 @@ async function updateProgress(
 }
 
 async function fetchInitialRCA(
-  testId: string,
+  testId: number,
   headers: Record<string, string>,
   baseUrl: string,
 ): Promise<RCATestCase> {
-  const url = baseUrl.replace("{testId}", testId);
+  const url = baseUrl.replace("{testId}", testId.toString());
 
   try {
     const response = await fetch(url, { headers });
@@ -179,7 +179,7 @@ async function pollRCAResults(
       await Promise.allSettled(
         inProgressCases.map(async (tc) => {
           try {
-            const pollUrl = baseUrl.replace("{testId}", tc.id);
+            const pollUrl = baseUrl.replace("{testId}", tc.id.toString());
             const response = await fetch(pollUrl, { headers });
             if (!response.ok) {
               const errorText = await response.text();
@@ -240,7 +240,7 @@ async function pollRCAResults(
 }
 
 export async function getRCAData(
-  testIds: string[],
+  testIds: number[],
   authString: string,
   context?: ScanProgressContext,
 ): Promise<RCAResponse> {

src/tools/rca-agent-utils/types.ts

@@ -21,7 +21,7 @@ export interface TestRun {
 }
 
 export interface FailedTestInfo {
-  test_id: string;
+  test_id: number;
   test_name: string;
 }
 
@@ -39,8 +39,8 @@ export enum RCAState {
 }
 
 export interface RCATestCase {
-  id: string;
-  testRunId: string;
+  id: number;
+  testRunId: number;
   state: RCAState;
   rcaData?: any;
 }

src/tools/rca-agent.ts

@@ -60,7 +60,7 @@ export async function getBuildIdTool(
 
 // Tool function that fetches RCA data
 export async function fetchRCADataTool(
-  args: { testId: string[] },
+  args: { testId: number[] },
   config: BrowserStackConfig,
 ): Promise<CallToolResult> {
   try {