Skip to content

tidy: group CLI modules into llm/github/ast + refresh README - #14

Merged
enekos merged 1 commit into
masterfrom
tidy-cli-and-readme
Jun 16, 2026
Merged

enekos merged 1 commit into
masterfrom
tidy-cli-and-readme

Conversation

@enekos

@enekos enekos commented Jun 16, 2026

Copy link
Copy Markdown
Owner

Housekeeping pass on the aatxe CLI crate + the README. No public/crate-API changes.

CLI crate reorganized

The binary's flat src/ had 11 loose top-level modules mixing concerns. Grouped into three documented modules (git tracked every move as a rename, so blame/history survives):

crates/aatxe/src/
  llm/      claude_code · pi_proxy · subprocess_llm · stub_client · gemini_http
  github/   gh_diff · github_http
  ast/      ast_scope · ast_import_extractor
  commands/  adapter/  cli.rs  curator.rs  main.rs
  • main.rs drops from 10 mod lines to 3; each group has a mod.rs documenting its members.
  • All cross-references rewritten to the new paths (crate::llm::…, crate::github::…, crate::ast::…).
  • The six library crates (aatxe-core, aatxe-bench, aatxe-ast, aatxe-council, aatxe-learn, aatxe-evals, aatxe-ui) are untouched — nothing on crates.io changes.

README

  • Replaced the misaligned ASCII hero diagram with a Mermaid flowchart — GitHub renders it natively, so it can't drift out of alignment again.
  • Stronger opening: value prop + the three-signal trust argument + the council dogfood baseline up front; mythology demoted to a closing aside.
  • Fixed stale facts: dropped the outdated "52 tests" claim, corrected the GH client from reqwest → ureq, and listed all 7 crates in Workspace layout (was 4) with the new CLI-internal grouping.

Verification

cargo fmt --check + cargo clippy --all-targets -- -D warnings clean; full cargo test --workspace passes (0 failures).

Reorganize the aatxe CLI crate's flat src/ — 11 loose top-level modules
become three documented groups (renames preserve history):

  llm/      claude_code, pi_proxy, subprocess_llm, stub_client, gemini_http
  github/   gh_diff, github_http
  ast/      ast_scope, ast_import_extractor

main.rs drops from 10 mod lines to 3; cross-references rewritten to the
new paths. The six library/published crates (aatxe-core, aatxe-bench, …)
are untouched, so nothing on crates.io changes.

README:
- Replace the misaligned ASCII hero diagram with a Mermaid flowchart
  (GitHub renders it natively — can't drift out of alignment).
- Stronger opening: value prop + three-signal trust argument + council
  dogfood baseline up front; mythology demoted to a closing aside.
- Fix stale facts: drop the outdated "52 tests" claim, correct the GH
  client from reqwest to ureq, and list all 7 crates in Workspace layout
  (was 4) with the new CLI-internal grouping.

cargo fmt --check + clippy -D warnings clean; full workspace test suite
passes.
@enekos
enekos merged commit 84b1a30 into master Jun 16, 2026
5 of 6 checks passed
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