Skip to content

Split real-world benchmark report contracts#294

Merged
yvette-carlisle merged 4 commits into
mainfrom
xy/report-contract-modules
Jun 30, 2026
Merged

Split real-world benchmark report contracts#294
yvette-carlisle merged 4 commits into
mainfrom
xy/report-contract-modules

Conversation

@yvette-carlisle

Copy link
Copy Markdown
Member

Summary

  • Split job_reports.rs into a facade plus focused job report contract modules.
  • Split summary_reports.rs into core, domain, and suite report contract modules.
  • Split external_adapter_reports.rs into detail, manifest, misc, and summary report contract modules.

Validation

  • cargo make fmt-rust-check
  • cargo make check-rust
  • cargo make test-rust (403 passed, 92 skipped)

Review Notes

  • Batched as one PR to avoid repeated remote CI waits for adjacent low-risk report-contract splits.
  • Batch-level skeptic passed with notes: effective facade surface is preserved; moved internal fields use pub(crate) inside private child modules and facades re-export the same contract names with pub(super).

@yvette-carlisle yvette-carlisle merged commit 8e80d83 into main Jun 30, 2026
12 checks passed
@yvette-carlisle yvette-carlisle deleted the xy/report-contract-modules branch June 30, 2026 05:14
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