Skip to content

Batch Rust module split refactor#312

Merged
yvette-carlisle merged 6 commits into
mainfrom
xy/module-batch-8
Jun 30, 2026
Merged

Batch Rust module split refactor#312
yvette-carlisle merged 6 commits into
mainfrom
xy/module-batch-8

Conversation

@yvette-carlisle

Copy link
Copy Markdown
Member

Summary

  • Split rerank provider implementation into focused local/noise/response/test modules behind the existing facade.
  • Split real_world_live_adapter ELF runtime, domain materializers, knowledge adapter, and QMD runtime into normal Rust modules with explicit visibility boundaries.
  • Split the docs extension acceptance lifecycle tests into a dedicated child module while keeping shared fixtures in the parent.

Validation

  • RUSTC=/run/current-system/sw/bin/rustc cargo make fmt-rust && RUSTC=/run/current-system/sw/bin/rustc cargo make check
  • Final segment result: 403 tests run: 403 passed, 92 skipped
  • git diff --check origin/main...HEAD

Batch Size

  • 23 files changed, 1780 insertions(+), 1647 deletions(-)

…1\",\"summary\":\"Split rerank provider modules\",\"authority\":\"manual\"}","authority":"manual"}
…1\",\"summary\":\"Split live adapter domain materializers\",\"authority\":\"manual\"}","authority":"manual"}
…1\",\"summary\":\"Split live adapter ELF runtime modules\",\"authority\":\"manual\"}","authority":"manual"}
…1\",\"summary\":\"Split live adapter knowledge artifact modules\",\"authority\":\"manual\"}","authority":"manual"}
…1\",\"summary\":\"Split live adapter QMD runtime modules\",\"authority\":\"manual\"}","authority":"manual"}
…1\",\"summary\":\"Split docs extension acceptance lifecycle tests\",\"authority\":\"manual\"}","authority":"manual"}
@yvette-carlisle yvette-carlisle merged commit 0749342 into main Jun 30, 2026
12 checks passed
@yvette-carlisle yvette-carlisle deleted the xy/module-batch-8 branch June 30, 2026 17:46
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