Skip to content

feat(telemetry): add JSON export writer#960

Open
EhabY wants to merge 1 commit into
feat/issue-903-export-telemetry-corefrom
feat/issue-903-export-telemetry-json-writer
Open

feat(telemetry): add JSON export writer#960
EhabY wants to merge 1 commit into
feat/issue-903-export-telemetry-corefrom
feat/issue-903-export-telemetry-json-writer

Conversation

@EhabY
Copy link
Copy Markdown
Collaborator

@EhabY EhabY commented May 17, 2026

Summary

  • add the JSON array telemetry export writer
  • preserve the stored snake_case JSONL event shape in the exported array
  • write through a unique temporary file before renaming into place

Refs #903.

Stack: 2 / 4. Base: #959. Next: #961.

Review follow-up

  • temp file names use randomUUID() to avoid same-process collisions
  • writer setup closes file handles if prefix writing fails
  • temp cleanup preserves the original export error
  • tests assert full stored event shape and empty-array output

Validation

  • pnpm test:extension ./test/unit/telemetry/export/range.test.ts ./test/unit/telemetry/export/files.test.ts ./test/unit/telemetry/export/writers.test.ts
  • pnpm typecheck
  • pnpm lint
  • pnpm format:check

Generated by Coder Agents.

@EhabY EhabY force-pushed the feat/issue-903-export-telemetry-json-writer branch from 199d2e7 to 3a29ac9 Compare May 17, 2026 19:42
@EhabY EhabY force-pushed the feat/issue-903-export-telemetry-core branch from 9d7d4ce to 3c02990 Compare May 17, 2026 19:42
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