refactor

crates/uv/src/cli.rs

@@ -1954,7 +1954,7 @@ pub(crate) struct AddArgs {
 pub(crate) struct RemoveArgs {
     /// The names of the packages to remove (e.g., `flask`).
     #[arg(required = true)]
-    pub(crate) requirements: Vec<String>,
+    pub(crate) requirements: Vec<PackageName>,
 
     /// The Python interpreter into which packages should be installed.
     ///

crates/uv/src/commands/project/remove.rs

@@ -14,7 +14,7 @@ use crate::printer::Printer;
 /// Remove one or more packages from the project requirements.
 #[allow(clippy::too_many_arguments)]
 pub(crate) async fn remove(
-    requirements: Vec<String>,
+    requirements: Vec<PackageName>,
     python: Option<String>,
     preview: PreviewMode,
     cache: &Cache,
@@ -29,7 +29,6 @@ pub(crate) async fn remove(
 
     let mut pyproject = PyProjectTomlMut::from_toml(project.current_project().pyproject_toml())?;
     for req in requirements {
-        let req = PackageName::new(req)?;
         if pyproject.remove_dependency(&req)?.is_empty() {
             anyhow::bail!(
                 "The dependency `{}` could not be found in `dependencies`",

crates/uv/src/settings.rs

@@ -415,7 +415,7 @@ impl AddSettings {
 #[allow(clippy::struct_excessive_bools, dead_code)]
 #[derive(Debug, Clone)]
 pub(crate) struct RemoveSettings {
-    pub(crate) requirements: Vec<String>,
+    pub(crate) requirements: Vec<PackageName>,
     pub(crate) python: Option<String>,
 }