Skip to content

Split real-world job consolidation report DTOs#284

Merged
yvette-carlisle merged 1 commit into
mainfrom
xy/split-job-report-consolidation
Jun 29, 2026
Merged

Split real-world job consolidation report DTOs#284
yvette-carlisle merged 1 commit into
mainfrom
xy/split-job-report-consolidation

Conversation

@yvette-carlisle

Copy link
Copy Markdown
Member

Summary

  • Split real-world job benchmark consolidation report DTOs into consolidation_reports.rs.
  • Kept job_reports.rs as the parent report model surface with the same re-exported type names.
  • Preserved derives, field names, field order, field types, serde behavior, and report generation logic.

Validation

  • cargo make fmt-rust
  • cargo make check-rust
  • cargo make fmt-rust-check
  • git diff --check
  • cargo make lint-vstyle
  • cargo make smoke-real-world-job-report
  • cargo make lint-rust
  • cargo make test-rust (379 passed, 92 skipped)
  • cargo make check (379 passed, 92 skipped)

Review

  • Scout selected consolidation report DTOs as a strong production module boundary.
  • Skeptic passed with no findings after staged diff review.

…ation report DTOs after strict validation.","authority":"manual"}
@yvette-carlisle yvette-carlisle merged commit cc7bd7f into main Jun 29, 2026
12 checks passed
@yvette-carlisle yvette-carlisle deleted the xy/split-job-report-consolidation branch June 29, 2026 23:20
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