Skip to content

Modularize GraphRAG smoke runner#338

Merged
yvette-carlisle merged 2 commits into
mainfrom
xy/module-batch-32
Jul 1, 2026
Merged

Modularize GraphRAG smoke runner#338
yvette-carlisle merged 2 commits into
mainfrom
xy/module-batch-32

Conversation

@yvette-carlisle

Copy link
Copy Markdown
Member

Summary

  • Keep scripts/graphrag-docker-smoke.py as the command-compatible entrypoint.
  • Split the 1470-line GraphRAG smoke runner into focused Python modules under scripts/graphrag_smoke/: context, models, common helpers, benchmark scoring, corpus generation, runtime commands, table mapping, generated artifacts, and runner orchestration.
  • Preserve generated manifest/fixture/report behavior while reducing the largest module to 474 lines and the entrypoint to 10 lines.

Batch size

  • 11 files changed, 1539 insertions, 1461 deletions, 3000 total changed lines.
  • Original runner reduced from 1470 lines to 10 lines.

Validation

  • python3 -m py_compile scripts/graphrag-docker-smoke.py scripts/graphrag_smoke/*.py
  • no-live smoke: ELF_GRAPHRAG_SMOKE_RUN=0 ELF_GRAPHRAG_SMOKE_ALLOW_HOST=1 ... scripts/graphrag-docker-smoke.py
  • RUSTC=/run/current-system/sw/bin/rustc cargo nextest run -p elf-eval --test real_world_job_benchmark graph_rag
  • RUSTC=/run/current-system/sw/bin/rustc cargo make fmt-rust
  • RUSTC=/run/current-system/sw/bin/rustc cargo make check (403 passed, 92 skipped)

Comment thread scripts/graphrag_smoke/runner.py Fixed
@yvette-carlisle yvette-carlisle merged commit c480e5c into main Jul 1, 2026
12 checks passed
@yvette-carlisle yvette-carlisle deleted the xy/module-batch-32 branch July 1, 2026 09:36
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