Skip to content

style: apply rustfmt across current workspace - #451

Merged
aaltshuler merged 1 commit into
mainfrom
codex/rustfmt-clean-slate
Aug 6, 2026
Merged

style: apply rustfmt across current workspace#451
aaltshuler merged 1 commit into
mainfrom
codex/rustfmt-clean-slate

Conversation

@aaltshuler

@aaltshuler aaltshuler commented Aug 6, 2026

Copy link
Copy Markdown
Collaborator

What and why

Supersedes #448 with a clean-slate formatter sweep generated from current main, after removal of the RFC-026/WAL implementation and the #450/#447 query merges.

This is the verbatim output of cargo fmt --all: 42 tracked Rust files, +4,873/-4,309 lines. It changes only layout and carries no semantic, dependency, generated-file, documentation, or persisted-format change.

Compared with #448, the replacement drops 28 obsolete files from the diff, including all deleted streaming/WAL code.

Verification

  • cargo fmt --all --check
  • git diff --check
  • bash scripts/check-agents-md.sh
  • Independent reproduction from a pristine origin/main archive produced the same 42 files with zero byte mismatches.
  • cargo test --workspace --locked --no-fail-fast: every non-spawn target passed. The external shared target directory prevented two CLI suites from discovering omnigraph-server; rerunning those exact suites with explicit built binary paths passed: parity 13/13 and system-local 27 passed, 1 credentialed test ignored.

Clippy cleanup and a permanent fmt/clippy CI gate remain separate follow-ups.

Greptile Summary

This PR applies cargo fmt --all across 42 Rust files without changing program semantics, public interfaces, dependencies, or persisted formats.

  • Reformats CLI helpers, configuration code, and associated integration tests.
  • Reformats cluster orchestration tests and failpoint coverage.
  • Reformats server query code and authorization, boot, multi-graph, and stored-query tests.
  • Reformats core query and embedding code plus engine tests, examples, and benchmarks.

Confidence Score: 5/5

The PR appears safe to merge because the changes are confined to rustfmt-generated layout and whitespace updates.

No changed expression, control-flow decision, public contract, security boundary, dependency, or persisted representation was identified; the formatter sweep preserves the existing Rust syntax and behavior.

Important Files Changed

Filename Overview
crates/omnigraph-cli/src/helpers.rs Rustfmt-only wrapping and whitespace changes preserve CLI URL construction, scope resolution, and query validation behavior.
crates/omnigraph-cluster/src/tests.rs Large indentation and line-wrapping cleanup preserves the cluster orchestration test logic and assertions.
crates/omnigraph-server/src/queries.rs Formatting-only changes preserve stored-query registry behavior.
crates/omnigraph/src/embedding.rs Formatting-only changes preserve embedding configuration, request, retry, and validation logic.
crates/omnigraph/src/exec/query.rs Formatting-only changes preserve query execution, traversal, search, and projection behavior.

Reviews (1): Last reviewed commit: "style: apply rustfmt across current work..." | Re-trigger Greptile

@aaltshuler
aaltshuler merged commit ad3da41 into main Aug 6, 2026
9 checks passed
@aaltshuler
aaltshuler deleted the codex/rustfmt-clean-slate branch August 6, 2026 23:07
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