fix(worktrees): always dirty-check before removal, even for non-utf8 paths

remove_worktree only ran the uncommitted/untracked-changes guard when
the worktree path was valid UTF-8 (via path().to_str()). For a non-utf8
path the check was silently skipped and the working directory was pruned
regardless of its contents — a data-loss window. RepoPath already
implements From<PathBuf>, so pass the path directly and drop the to_str
hop; behavior is identical for utf8 paths and the guard now always runs.

Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>

Author: stevenpollack

asyncgit/src/sync/worktree.rs

@@ -214,14 +214,12 @@ pub fn remove_worktree(
 	// changes (libgit2 does not refuse this itself). Only checkable
 	// when the working dir still exists.
 	if worktree.validate().is_ok() {
-		if let Some(path) = worktree.path().to_str() {
-			let wt_path: RepoPath = path.into();
-			if !is_workdir_clean(&wt_path, None)? {
-				return Err(Error::Generic(
-					"worktree has uncommitted changes; commit or discard them first"
-						.to_string(),
-				));
-			}
+		let wt_path: RepoPath = worktree.path().to_path_buf().into();
+		if !is_workdir_clean(&wt_path, None)? {
+			return Err(Error::Generic(
+				"worktree has uncommitted changes; commit or discard them first"
+					.to_string(),
+			));
 		}
 	}