Skip to content

agent: Serve a telemetry snapshot through agentctl telemetry --json - #249

Merged
lbacik merged 4 commits into
mainfrom
agent/issue-240
Oct 14, 2026
Merged

lbacik merged 4 commits into
mainfrom
agent/issue-240

Conversation

@lbacik

@lbacik lbacik commented Oct 14, 2026

Copy link
Copy Markdown
Owner

Closes #240

Summary

Claude SDK completed the implementation workflow.

Evidence

  • Check: exit code 0
  • Reviews performed: 1
  • Review repair cycles: 0/2

Generated by simple-coding-agent

Attempt-Id: 2026-10-14T12:03:00.200226Z
…telemetry print, drop unused identity surface

- telemetry_control_snapshot returns a deepcopy so served snapshots never
  alias the cached control section.
- agentctl telemetry print moved inside the try so no traceback can escape.
- Drop unused TelemetryReporter.identity property.

Co-Authored-By: Claude Code <noreply@anthropic.com>
…rd, stronger shape test

- TelemetryReporter.snapshot reads the control section before consuming a
  sequence number, so failed builds do not consume one.
- _reply_telemetry answers snapshot_failed (with rate-limited
  telemetry_failed event) when the snapshot is not JSON-serialisable,
  instead of escaping to the generic handler path.
- Shape test pins the intentional tracer divergences from the populated
  example; new tests cover the sequence gap and the unserialisable reply.

Co-Authored-By: Claude Code <noreply@anthropic.com>
@lbacik
lbacik enabled auto-merge October 14, 2026 12:29
@lbacik
lbacik merged commit 11f6b9b into main Oct 14, 2026
4 checks passed
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.

Serve a telemetry snapshot through agentctl telemetry --json

2 participants