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 self-assigned this May 17, 2026
@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
@EhabY EhabY force-pushed the feat/issue-903-export-telemetry-json-writer branch from 3a29ac9 to eb776a7 Compare May 18, 2026 17:03
@EhabY EhabY force-pushed the feat/issue-903-export-telemetry-core branch from 3c02990 to 350d662 Compare May 18, 2026 17:03
@EhabY EhabY force-pushed the feat/issue-903-export-telemetry-json-writer branch from eb776a7 to ba095a9 Compare May 18, 2026 19:05
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