Repository navigation
feat(admin): trace performance, grouped timeline and report exports - #629
Merged
Merged
Conversation
Contributor
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
MrParamecium
changed the title
- Poll the trace envelope before events (then events + performance in parallel), so a run that finishes between reads cannot stop polling with its last flush and terminal event unread; re-render only on change and restore focus. - Trading log: link fills through the execution event's decision_id and the per-action reasoning_summaries; show the latest 50 newest-first with the true count. - Backtest report: fetch once per trace+status, so restoring `open` on each re-render no longer refetches; read llm_model (agent_runs has no model_name). - Performance: report the run's stored total_return / max_drawdown instead of recomputing against the first equity mark. - Export: status from the events inside the boundary (failed != incomplete); word-based key redaction (token counts stay visible); Markdown-escape agent text; linear-time email/newline handling. - Null-guard renderTraceDetail; tokenise colours, drop dead timeline rules, bound the sticky rails; restore the rejected-query-name scan on all read modules. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
This branch was successfully deployed
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Admin trace details now connect trading outcomes to recorded decisions: an exact-run equity curve links trades to collapsible decision/step groups, with reasoning summaries and optional raw JSON. Expanded groups survive polling and failed groups open by default.
Administrators can inspect stored metrics and configuration for the trace’s exact backtest run across owners through an admin-only endpoint. Run configuration explicitly labels missing values. JSON and Markdown downloads include every recorded event page up to a fixed sequence cutoff, redact sensitive key fields and common credential patterns, and include artifact references without bundling attachments.
Validation: 349 admin/composition tests passed; 7 trace frontend tests passed again after the final summary adjustment. Tests cover cross-owner admin reads, unauthenticated rejection, 205-event exports, concurrent appends, download headers, invalid formats, and grouping. Synthetic browser preview verified trade-to-decision expansion and read-only backtest disclosure. Local 9017 service updated. Production Neon and a real end-to-end backtest were not exercised locally; export redaction is pattern-based.