Take show_untracked into account

Author: cruessler

asyncgit/src/sync/config.rs

@@ -56,6 +56,13 @@ pub fn untracked_files_config_repo(
 		}
 	}
 
+	// This does not reflect how git works according to its docs that say: "If this variable is not
+	// specified, it defaults to `normal`."
+	//
+	// https://git-scm.com/docs/git-config#Documentation/git-config.txt-statusshowUntrackedFiles
+	//
+	// Note that this might become less relevant over time as more code gets migrated to `gitoxide`
+	// because `gitoxide` respects `status.showUntrackedFiles` by default.
 	Ok(ShowUntrackedFilesConfig::All)
 }
 

asyncgit/src/sync/status.rs

@@ -159,24 +159,37 @@ pub fn is_workdir_clean(
 	Ok(statuses.is_empty())
 }
 
+impl From<ShowUntrackedFilesConfig> for gix::status::UntrackedFiles {
+	fn from(value: ShowUntrackedFilesConfig) -> Self {
+		use gix::status::UntrackedFiles;
+
+		match value {
+			ShowUntrackedFilesConfig::All => UntrackedFiles::Files,
+			ShowUntrackedFilesConfig::Normal => {
+				UntrackedFiles::Collapsed
+			}
+			ShowUntrackedFilesConfig::No => UntrackedFiles::None,
+		}
+	}
+}
+
 /// guarantees sorting
 pub fn get_status(
 	repo_path: &RepoPath,
 	status_type: StatusType,
-	_show_untracked: Option<ShowUntrackedFilesConfig>,
+	show_untracked: Option<ShowUntrackedFilesConfig>,
 ) -> Result<Vec<StatusItem>> {
 	scope_time!("get_status");
 
-	// TODO: take parameter `show_untracked` into account, trying to replicate the existing
-	// behaviour.
-
 	let repo: gix::Repository =
 				gix::ThreadSafeRepository::discover_with_environment_overrides(repo_path.gitpath())
 						.map(Into::into)?;
 
-	let status = repo
-		.status(gix::progress::Discard)?
-		.untracked_files(gix::status::UntrackedFiles::Files);
+	let mut status = repo.status(gix::progress::Discard)?;
+
+	if let Some(config) = show_untracked {
+		status = status.untracked_files(config.into());
+	}
 
 	let mut res = Vec::new();