fix: use strip_prefix instead of manual slicing

echobt · echobt · commit 686cc18b2420 · 2026-02-04T16:37:10.000Z
diff --git a/src/cortex-cli/src/utils/paths.rs b/src/cortex-cli/src/utils/paths.rs
@@ -39,10 +39,10 @@ pub fn expand_tilde(path: &str) -> String {
         if let Some(home) = dirs::home_dir() {
             return home.to_string_lossy().to_string();
         }
-    } else if path.starts_with("~/") {
+    } else if let Some(suffix) = path.strip_prefix("~/") {
         // Handle "~/" prefix - expand to home directory + rest of path
         if let Some(home) = dirs::home_dir() {
-            return home.join(&path[2..]).to_string_lossy().to_string();
+            return home.join(suffix).to_string_lossy().to_string();
         }
     }
     path.to_string()

Original file line number	Diff line number	Diff line change
`@@ -39,10 +39,10 @@ pub fn expand_tilde(path: &str) -> String {`
`39`	`39`	`if let Some(home) = dirs::home_dir() {`
`40`	`40`	`return home.to_string_lossy().to_string();`
`41`	`41`	`}`
`42`		`- } else if path.starts_with("~/") {`
	`42`	`+ } else if let Some(suffix) = path.strip_prefix("~/") {`
`43`	`43`	`// Handle "~/" prefix - expand to home directory + rest of path`
`44`	`44`	`if let Some(home) = dirs::home_dir() {`
`45`		`- return home.join(&path[2..]).to_string_lossy().to_string();`
	`45`	`+ return home.join(suffix).to_string_lossy().to_string();`
`46`	`46`	`}`
`47`	`47`	`}`
`48`	`48`	`path.to_string()`