Skip to content

Fix Cloud Browser trace agent type formatting#110

Open
xTRam1 wants to merge 1 commit into
mainfrom
codex/sdk-cloud-trace-agent-type-fix-main
Open

Fix Cloud Browser trace agent type formatting#110
xTRam1 wants to merge 1 commit into
mainfrom
codex/sdk-cloud-trace-agent-type-fix-main

Conversation

@xTRam1
Copy link
Copy Markdown
Contributor

@xTRam1 xTRam1 commented May 7, 2026

Summary

  • emit string agent_type labels for built-in Pyodide window.agent() trace events
  • make narada-core trace parsing tolerate historical numeric agent_type values
  • add Cloud Browser trace tests covering built-in and nested child action traces
  • bump package versions so the fix can be published and consumed by frontend/Pyodide runtime

Why

Deep Cloud Browser nested APA trace runs can fail when normal window.agent() trace events carry numeric enum values such as 2, while the trace parser expects strings. The E2E harness captured this as a real product failure in the deep Cloud Browser lane.

Validation

  • uv run pytest packages/narada-pyodide/tests/test_cloud_browser.py packages/narada/tests/test_input_variables.py -q
  • uv run ruff check packages/narada-core/src/narada_core/tracing/model.py packages/narada-pyodide/src/narada/window.py packages/narada-pyodide/tests/test_cloud_browser.py
  • python -m py_compile packages/narada-core/src/narada_core/tracing/model.py packages/narada-pyodide/src/narada/window.py packages/narada-pyodide/tests/test_cloud_browser.py
  • git diff --check

@xTRam1 xTRam1 changed the title [codex] Fix Cloud Browser trace agent type formatting Fix Cloud Browser trace agent type formatting May 7, 2026
@xTRam1 xTRam1 marked this pull request as ready for review May 7, 2026 22:43
@xTRam1 xTRam1 requested a review from zizhengtai May 7, 2026 22:49
@zizhengtai
Copy link
Copy Markdown
Contributor

Probably easier to redo this PR because of the merge conflicts

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants