Skip to content

Trace drawer: selected span and active span id copy into each other (pre-existing oscillation) #6714

Description

@mmabrouk

Found during the Codex xhigh review of #6711 (the Use API drawer render loop, #6708).

Codex checked whether any other component pairs two selection states that reconcile each other through effects. It found one: web/packages/agenta-observability-ui/src/traceDrawer/TraceDrawerContent.tsx keeps a local selected span and copies it to and from the global active span id. Codex reproduced a B/A/B/A alternation between the two.

This is separate code and separate atoms from #6708, so #6711 leaves it alone. It is pre-existing and not a v0.115.3 regression.

Suggested fix, same shape as #6711: keep one selection state (the active span id) and derive the local view from it, instead of two states that write each other.

Not a release blocker. Filed as a follow-up from the v0.115.3 release run.

Activity

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Metadata

Assignees

No one assigned

    Labels

    No labels
    No labels

    Type

    No type

    Projects

    No projects

      Milestone

      No milestone

      Relationships

      None yet

      Development

      No branches or pull requests

      Issue actions