refactor(install): split helper modules#698
Conversation
Greptile SummaryThis PR is a pure structural refactor: four helper clusters that had accumulated at the bottom of
Confidence Score: 5/5Safe to merge — this is a mechanical code-motion refactor with no logic changes. Every moved function is a verbatim copy of its original; the one path-depth adjustment (super::super in summary.rs) resolves to the same target; cargo fmt/check/clippy/test are all listed as passing. No logic, behavior, or public API is altered. No files require special attention. Important Files Changed
Reviews (1): Last reviewed commit: "refactor(install): split helper modules" | Re-trigger Greptile |
What changed
commands/install/mod.rsinto focused modules:lockfile_dir.rsfor lockfile-dir importer remapping.summary.rsfor human install summary output.sweep.rsfor.aubeinvalidation and orphan cleanup.unreviewed_builds.rsfor skipped-build warning replay and emission.UnreviewedBuildvisibility only within the install module tree so the new warning module can reuse the existing lifecycle type.Why
install/mod.rswas carrying unrelated helper groups after the main install pipeline. Moving these groups into modules makes the install command easier to scan while preserving behavior.Validation
cargo fmt --checkcargo check -p aubecargo clippy --all-targets -- -D warningscargo test -p aube commands::installNote
Low Risk
Low risk refactor that primarily moves helper functions into new modules and adjusts visibility/imports; behavior should remain the same aside from minor risk of missed wiring or path/visibility mistakes.
Overview
Refactors
commands/install/mod.rsby extracting several post-pipeline helper clusters into focused modules:lockfile_dir(importer-key remapping + foreign-importer guard for--lockfile-dir),summary(human-readable install output helpers),sweep(.aubecache invalidation/orphan cleanup), andunreviewed_builds(replay + emission of skipped-build warnings).Updates call sites to use the new modules and widens
UnreviewedBuildvisibility topub(in crate::commands::install)so the newunreviewed_buildsmodule can reuse the existing lifecycle type.Reviewed by Cursor Bugbot for commit 87f03ef. Bugbot is set up for automated code reviews on this repo. Configure here.