fix(telemetry): fix tool call status logic and remove unused import

- Fix tool call status: now properly detects error state
- Remove unused Bus import from collector.ts

Author: Kenny-Heitritter

packages/opencode/src/telemetry/collector.ts

@@ -6,7 +6,6 @@
  */
 
 import { Log } from "../util/log"
-import { Bus } from "../bus"
 import { Config } from "../config/config"
 import { createSanitizer, hashFilePath, getFileExtension } from "./sanitizer"
 import { createSignalTracker, type SignalTracker } from "./signals"

packages/opencode/src/telemetry/integration.ts

@@ -152,17 +152,18 @@ function subscribeToEvents(state: TelemetryState): void {
     Bus.subscribe(MessageV2.Event.PartUpdated, (event) => {
       const { part } = event.properties
 
-      // Handle tool calls
-      if (part.type === "tool" && part.state === "completed") {
+      // Handle completed tool calls
+      if (part.type === "tool" && (part.state === "completed" || part.state === "error")) {
         const duration = part.time?.end && part.time?.start ? part.time.end - part.time.start : 0
+        const status = part.state === "completed" ? "success" : "error"
 
         collector.recordToolCall(
           part.tool,
-          part.state === "completed" ? "success" : "error",
+          status,
           duration,
           part.input ? JSON.stringify(part.input).length : undefined,
           part.output ? part.output.length : undefined,
-          undefined, // errorType - would need to parse from output
+          status === "error" ? "tool_error" : undefined,
         )
       }