Skip to content

Add Claude CLI observability logs#76

Merged
laotoutou merged 2 commits intomainfrom
feat/claude-cli-observability
Apr 28, 2026
Merged

Add Claude CLI observability logs#76
laotoutou merged 2 commits intomainfrom
feat/claude-cli-observability

Conversation

@laotoutou
Copy link
Copy Markdown
Contributor

@laotoutou laotoutou commented Apr 28, 2026

Summary

  • add wrapper-level observability around Claude CLI process spawn, first stdin write, first stdout byte, and first stdout line
  • add Codex app-server observability around process spawn, first stdin write, first stdout/stderr lines, initialize, thread, turn, first notification, first codex event, and first user-visible event
  • keep vendor process instrumentation in wrapper-owned code without patching vendored dependencies
  • add focused tests covering the new structured logs

Testing

  • node --test test/claude-runtime.test.js
  • node --test test/codex-runtime.test.js
  • node --test

@laotoutou laotoutou merged commit 14cf2e0 into main Apr 28, 2026
4 checks passed
@laotoutou laotoutou deleted the feat/claude-cli-observability branch April 28, 2026 12:00
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