Skip to content

fix(observability): sanitize LangSmith trace payloads - #25

Merged
ngoclam9415 merged 2 commits into
developfrom
fix/langsmith-trace-sanitizer
Jul 8, 2026
Merged

fix(observability): sanitize LangSmith trace payloads#25
ngoclam9415 merged 2 commits into
developfrom
fix/langsmith-trace-sanitizer

Conversation

@ngoclam9415

Copy link
Copy Markdown
Contributor

Summary

  • sanitize LangSmith trace inputs/outputs before SDK serialization so cyclic Dana timelines are logged as compact summaries
  • add regression coverage for cyclic Timeline payloads and JSON-safe trace processors
  • add a repo-local dana-cross-repo-debug skill for reproducing console-bug issues through tmux, backend API, and the Playground

Test plan

  • uv run ruff check dana/common/observable.py dana/common/trace_sanitizer.py tests/unit/test_observable_langsmith_dispatch.py
  • uv run python -m pytest tests/unit/test_observable_langsmith_dispatch.py -q
  • python -m json.tool .claude/skills/dana-cross-repo-debug/evals/evals.json
  • console-bug backend agent.ask repro returned HTTP 200 with no LangSmith recursion crash after local dana-runtime binding

@ngoclam9415
ngoclam9415 merged commit da6e5dc into develop Jul 8, 2026
1 check passed
@ngoclam9415
ngoclam9415 deleted the fix/langsmith-trace-sanitizer branch July 8, 2026 06:39
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.

1 participant