Skip to content

perf(reports): remove codec processes from collection - #1378

Merged
Chemaclass merged 1 commit into
mainfrom
feat/1371-report-transport
Oct 2, 2026
Merged

Chemaclass merged 1 commit into
mainfrom
feat/1371-report-transport

Conversation

@Chemaclass

Copy link
Copy Markdown
Member

🤔 Background

Related #1371

Parallel reports launch codec processes for each row's text fields. Collect complete worker records without those processes while preserving public formats and completion order.

💡 Changes

  • Publish separate lossless worker records and replay them once in completion order, including parent and file hook failures.
  • Preserve arbitrary text and large output, avoid record collisions, and clean all records with the run.
  • Cover concurrent publication, incomplete records, lifecycle failures, all report formats and zero codec calls. Update the changelog.
  • Source, standalone, static and formatting gates pass. An isolated 100-row collector takes 46 ms instead of 1,185 ms and removes 400 base64 plus 200 tr calls; 512 KiB output takes 99 ms instead of 215 ms. Medians use five alternating pairs against main at f33a9ac on macOS Bash 3.2.

@Chemaclass Chemaclass added the enhancement New feature or request label Oct 2, 2026
@Chemaclass Chemaclass self-assigned this Oct 2, 2026
@Chemaclass
Chemaclass merged commit 5731afa into main Oct 2, 2026
37 checks passed
@Chemaclass
Chemaclass deleted the feat/1371-report-transport branch October 2, 2026 15:08
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant