fix(telemetry): fix endToolSpan status on success — toolCalls is empty in finally

Root cause: checkAndNotifyCompletion clears this.toolCalls before the
finally block in executeSingleToolCall runs, so the tc lookup always
returns undefined.

Fix: set OK status explicitly in _executeToolCallBody's success path
via safeSetStatus(span, OK), and call endToolSpan() without metadata
in finally (just ends the span, preserves pre-set status from any path).

Author: doudouOUC

packages/core/src/core/coreToolScheduler.ts

@@ -1868,12 +1868,7 @@ export class CoreToolScheduler {
         this._executeToolCallBody(scheduledCall, signal, toolSpan),
       );
     } finally {
-      const tc = this.toolCalls.find((c) => c.request.callId === callId);
-      if (tc?.status === 'success') {
-        endToolSpan(toolSpan, { success: true });
-      } else {
-        endToolSpan(toolSpan);
-      }
+      endToolSpan(toolSpan);
     }
   }
 
@@ -2170,6 +2165,7 @@ export class CoreToolScheduler {
             : {}),
         };
         this.setStatusInternal(callId, 'success', successResponse);
+        safeSetStatus(span, { code: SpanStatusCode.OK });
       } else {
         // It is a failure
         // PostToolUseFailure Hook