A file-based task manager
Trim fzf::select output before parsing
fzf appends a trailing newline to its stdout. The String FromStr impl
is just a clone, so picked-string callers (`pick_namespace`,
`PropAction::Set` value/key picker) saw "noah\n" and forwarded that
through to validate_namespace, which rightly rejected the embedded
newline.
Also affected non-string callers in principle (Id from "tsk-1\n"
parses the int part as "1\n", which fails) but no live caller hit
that yet.
Trim once in fzf::select so every caller benefits.
Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
+4
-1
+4
-1
src/fzf.rs
+4
-1
src/fzf.rs
···
34
34
if output.stdout.is_empty() {
35
35
Ok(None)
36
36
} else {
37
-
Ok(Some(String::from_utf8(output.stdout)?.parse()?))
37
+
// fzf appends a trailing newline; strip it so the FromStr impls
38
+
// (Id, String, usize, etc.) all work.
39
+
let raw = String::from_utf8(output.stdout)?;
40
+
Ok(Some(raw.trim().parse()?))
38
41
}
39
42
}