fix(langsmith): align get_run/update_run outputs and narrow feedback score type

- get_run now also outputs runId (alias of id) so workflows can read
  the run identifier consistently across all operations on the block
- update_run now parses and surfaces the response message instead of
  discarding the body entirely, matching create_run's pattern
- narrow LangsmithCreateFeedbackParams.score to number — the tool
  param is declared as JSON-schema 'number' and the block's parseScore
  never produces a boolean, so the wider type was dead and misleading

Author: waleedlatif1

apps/sim/tools/langsmith/create_feedback.ts

@@ -37,7 +37,7 @@ export const langsmithCreateFeedbackTool: ToolConfig<
       type: 'number',
       required: false,
       visibility: 'user-or-llm',
-      description: 'Numeric or boolean score for the feedback metric',
+      description: 'Numeric score for the feedback metric',
     },
     value: {
       type: 'string',
@@ -102,7 +102,7 @@ export const langsmithCreateFeedbackTool: ToolConfig<
         id: data.id as string,
         key: data.key as string,
         runId: (data.run_id as string) ?? null,
-        score: (data.score as number | boolean) ?? null,
+        score: (data.score as number) ?? null,
         value: (data.value as string | number | boolean) ?? null,
         comment: (data.comment as string) ?? null,
         createdAt: (data.created_at as string) ?? null,

apps/sim/tools/langsmith/get_run.ts

@@ -39,6 +39,7 @@ export const langsmithGetRunTool: ToolConfig<LangsmithGetRunParams, LangsmithGet
       success: true,
       output: {
         id: data.id as string,
+        runId: data.id as string,
         name: data.name as string,
         runType: data.run_type as string,
         status: (data.status as string) ?? null,
@@ -58,6 +59,10 @@ export const langsmithGetRunTool: ToolConfig<LangsmithGetRunParams, LangsmithGet
   },
   outputs: {
     id: { type: 'string', description: 'Run ID' },
+    runId: {
+      type: 'string',
+      description: 'Run ID (alias of id, for consistency with other operations)',
+    },
     name: { type: 'string', description: 'Run name' },
     runType: {
       type: 'string',

apps/sim/tools/langsmith/types.ts

@@ -74,6 +74,7 @@ export interface LangsmithUpdateRunResponse extends ToolResponse {
   output: {
     accepted: boolean
     runId: string
+    message: string | null
   }
 }
 
@@ -85,6 +86,7 @@ export interface LangsmithGetRunParams {
 export interface LangsmithGetRunResponse extends ToolResponse {
   output: {
     id: string
+    runId: string
     name: string
     runType: string
     status: string | null
@@ -108,7 +110,7 @@ export interface LangsmithCreateFeedbackParams {
   apiKey: string
   runId: string
   key: string
-  score?: number | boolean
+  score?: number
   value?: string
   comment?: string
   correction?: Record<string, unknown>
@@ -120,7 +122,7 @@ export interface LangsmithCreateFeedbackResponse extends ToolResponse {
     id: string
     key: string
     runId: string | null
-    score: number | boolean | null
+    score: number | null
     value: string | number | boolean | null
     comment: string | null
     createdAt: string | null

apps/sim/tools/langsmith/update_run.ts

@@ -100,11 +100,23 @@ export const langsmithUpdateRunTool: ToolConfig<
       throw new Error(`LangSmith update run failed (${response.status}): ${errorText}`)
     }
 
+    const responseText = await response.text()
+    let message: string | null = null
+    if (responseText) {
+      try {
+        const data = JSON.parse(responseText) as Record<string, unknown>
+        message = typeof data.message === 'string' ? data.message : null
+      } catch {
+        // Response body isn't JSON (e.g. empty object or plain text) — no message to surface
+      }
+    }
+
     return {
       success: true,
       output: {
         accepted: true,
         runId: params?.runId.trim() ?? '',
+        message,
       },
     }
   },
@@ -117,5 +129,10 @@ export const langsmithUpdateRunTool: ToolConfig<
       type: 'string',
       description: 'ID of the run that was updated',
     },
+    message: {
+      type: 'string',
+      description: 'Response message from LangSmith, if provided',
+      optional: true,
+    },
   },
 }