The latest updates on your projects. Learn more about Vercel for GitHub.

1 Skipped Deployment

Project	Deployment	Actions	Updated (UTC)
docs	Skipped		Apr 29, 2026 10:14pm

PR Summary

Medium Risk
Touches trace rendering and shared formatting logic (names/icons/cost units), so UI regressions or mismatched provider/tool mappings are possible, but changes are contained to observability surfaces.

Overview
Trace UI now centralizes shared span helpers in a new logs/components/log-details/utils.ts, and both TraceSpans and TraceView are refactored to consume these utilities (time parsing, error propagation checks, iteration detection, token/TTFT/TPS formatting, display-name resolution, and block icon/color lookup).

Tool spans are now displayed and matched using normalized tool IDs (e.g. knowledge_search_<uuid> -> knowledge_search), and icon/color resolution also uses the normalized tool name so keyed tools map to the correct block appearance.

Cost formatting in trace surfaces is changed to credits via dollarsToCredits, and the shared normalizeToolId logic is extracted to tools/normalize.ts and re-exported from tools/index.ts for reuse in client code.

^{Reviewed by Cursor Bugbot for commit 4c07eb9. Configure here.}

Greptile Summary

This PR consolidates duplicate utility functions from trace-view.tsx and trace-spans.tsx into a new shared utils.ts, extracts normalizeToolId into a client-safe tools/normalize.ts, and updates both trace components to normalize UUID-suffixed tool span names and display costs in credits instead of raw dollar amounts.

Confidence Score: 5/5

Safe to merge — only a P2 style nit on import placement; all logic, normalization, and credit conversion are correct.

No P0 or P1 findings. The single finding is a mid-file import statement in tools/index.ts that Biome's organizeImports would normally reorder — it has no runtime impact. All previously flagged issues (locale on toLocaleString, sub-credit visibility, dollar/credit inconsistency between views) have been addressed.

apps/sim/tools/index.ts — import statement should be moved to the top of the file.

Important Files Changed

Flowchart

%%{init: {'theme': 'neutral'}}%%
flowchart TD
    A[TraceSpan with name e.g. knowledge_search_uuid] --> B{span.type === 'tool'?}
    B -- Yes --> C[normalizeToolId from tools/normalize.ts]
    C --> D[Stripped name e.g. knowledge_search]
    B -- No --> E[span.name unchanged]
    D --> F[getDisplayName in utils.ts]
    E --> F
    F --> G[trace-view.tsx: tree row label + tooltip + detail pane]
    F --> H[trace-spans.tsx: span node label]
    F --> I[spanMatchesQuery: search matches normalized name]
    J[span.cost.total in dollars] --> K[formatCostAmount in utils.ts]
    K --> L[dollarsToCredits: Math.round x 200]
    L -- result gt 0 --> M[N credits]
    L -- result eq 0 --> N[less than 1 credit]
    M --> O[trace-view.tsx: header summary + detail pane Cost row]
    N --> O
    M --> P[trace-spans.tsx: span Cost row]
    N --> P

_{Reviews (5): Last reviewed commit: "fix(trace-spans): simplify formatCostSum..." | Re-trigger Greptile}

@greptile

@cursor review

@greptile

@cursor review

@greptile

@cursor review

@greptile

@cursor review