fix(dashboard): estado 'cancelled' en color neutral, no verde de éxito#31
Merged
Merged
Conversation
…e de éxito Desde que el colector deriva el estado 'cancelled' (#28), varias píldoras de estado del dashboard seguían tratando el estado como binario (error → rojo, resto → verde esmeralda). Eso pintaba una traza/span *cancelada* con el mismo verde de éxito que una 'ok', engañando a simple vista en la vista de sesión (preview de traza), en el detalle de span y en Tools. Se centraliza el mapeo estado→tono en lib/status.ts (ok/error/neutral) y se usa en SessionDetail, ToolsView y TraceDetail. 'cancelled' ahora se muestra en gris neutral. Test unitario nuevo (apps añadido al include de vitest). Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
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
1 participant
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.
Bug
Desde que el colector derivó el estado
cancelled(#28), el dashboard siguió tratando el estado de traza/span como binario en varias píldoras de estado:Resultado: una traza o span cancelada se pintaba con el mismo verde de éxito que una
ok— engañoso a simple vista. Sitios afectados:SessionDetail.tsx→TracePreview: la píldora de estado al hacer hover sobre una traza en el timeline de la sesión mostrabacancelleden verde de éxito (el más visible).TraceDetail.tsx→SpanDrawer: el campo Status dabatone='ok'(verde) a spans cancelados.ToolsView.tsx: la píldora de estado de tool-calls.TraceListya lo manejaba bien (acento gris paracancelled); el resto se quedó atrás.Fix
apps/dashboard/src/lib/status.ts: única fuente de verdad estado→tono (ok/error/neutral) constatusTone()ystatusBadgeClass().cancelled→ gris neutral.SessionDetail,ToolsViewyTraceDetail.RunRow/RunDetail(sus estados sonRunStatus/'ok'|'error', conrunningpropio — no aplicacancelled).Tests
apps/dashboard/src/lib/status.test.ts(7 casos, incluye regresión decancelled).apps/**/src/**/*.test.tsañadido alincludede Vitest (el dashboard no tenía cobertura). Entornonode, helper puro, sin infra de React.scripts/traction.test.mjs(fallo preexistente solo-Windows por BOM/CRLF; verde en CI Linux).typecheckybuildverdes.🤖 Generated with Claude Code