cargo-rail: fixes to nextest integration and watch mode via bacon

Author: loadingalias

src/commands/test.rs

@@ -29,8 +29,8 @@ pub fn run_test(ctx: &WorkspaceContext, config: TestConfig) -> RailResult<()> {
     detect_default_base_ref(ctx.git.git())?
   };
 
-  // Analyze changes since the base ref
-  let impact = analyzer.analyze_changes(&base_ref, "HEAD")?;
+  // Analyze changes since the base ref (to working tree)
+  let impact = analyzer.analyze_changes(&base_ref, None)?;
 
   // Get minimal test set
   let test_targets = impact.minimal_test_set();

src/sync/engine.rs

@@ -747,7 +747,7 @@ impl<'a> SyncEngine<'a> {
     let analyzer = ChangeImpact::new(self.ctx);
 
     // Check if this specific crate has changes
-    if let Some(impact) = analyzer.analyze_crate_changes(&self.config.crate_name, from, "HEAD")? {
+    if let Some(impact) = analyzer.analyze_crate_changes(&self.config.crate_name, from, Some("HEAD"))? {
       // Filter out commits from remote
       let crate_path = &self.config.crate_paths[0];
       let new_commits = self

src/workspace/change_analyzer.rs

@@ -119,9 +119,11 @@ impl<'a> ChangeImpact<'a> {
   }
 
   /// Analyze changes between two git refs with full graph awareness
-  pub fn analyze_changes(&self, from: &str, to: &str) -> RailResult<ImpactReport> {
+  ///
+  /// If `to` is None, compares against the working tree (uncommitted changes).
+  pub fn analyze_changes(&self, from: &str, to: Option<&str>) -> RailResult<ImpactReport> {
     // 1. Get changed files from git
-    let changed_files = self.ctx.git.git().get_changed_files_between(from, Some(to))?;
+    let changed_files = self.ctx.git.git().get_changed_files_between(from, to)?;
 
     // 2. Categorize changes by file type using new classification system
     let categories = self.categorize_changes(&changed_files);
@@ -152,7 +154,12 @@ impl<'a> ChangeImpact<'a> {
   }
 
   /// Analyze changes for a specific crate only
-  pub fn analyze_crate_changes(&self, crate_name: &str, from: &str, to: &str) -> RailResult<Option<ImpactReport>> {
+  pub fn analyze_crate_changes(
+    &self,
+    crate_name: &str,
+    from: &str,
+    to: Option<&str>,
+  ) -> RailResult<Option<ImpactReport>> {
     let full_report = self.analyze_changes(from, to)?;
 
     // Filter to only this crate