Add note about using the explicit bounds over the preferences

Author: konstin

crates/uv-workspace/src/pyproject_mut.rs

@@ -1,9 +1,9 @@
+use itertools::Itertools;
+use serde::{Deserialize, Serialize};
+use std::fmt::{Display, Formatter};
 use std::path::Path;
 use std::str::FromStr;
 use std::{fmt, iter, mem};
-
-use itertools::Itertools;
-use serde::{Deserialize, Serialize};
 use thiserror::Error;
 use toml_edit::{
     Array, ArrayOfTables, DocumentMut, Formatted, Item, RawString, Table, TomlError, Value,
@@ -112,6 +112,17 @@ pub enum AddBoundsKind {
     Exact,
 }
 
+impl Display for AddBoundsKind {
+    fn fmt(&self, f: &mut Formatter<'_>) -> fmt::Result {
+        match self {
+            Self::Lower => write!(f, "lower"),
+            Self::Major => write!(f, "major"),
+            Self::Minor => write!(f, "minor"),
+            Self::Exact => write!(f, "exact"),
+        }
+    }
+}
+
 impl AddBoundsKind {
     fn specifiers(self, version: Version) -> VersionSpecifiers {
         // Nomenclature: "major" is the most significant component of the version, "minor" is the
@@ -680,7 +691,7 @@ impl PyProjectTomlMut {
         dependency_type: &DependencyType,
         index: usize,
         version: Version,
-        bound_kind: &AddBoundsKind,
+        bound_kind: AddBoundsKind,
     ) -> Result<(), Error> {
         let group = match dependency_type {
             DependencyType::Production => self.dependencies_array()?,

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

@@ -527,7 +527,7 @@ pub(crate) async fn add(
         locked,
         &dependency_type,
         raw,
-        &bounds.unwrap_or_default(),
+        bounds,
         constraints,
         &settings,
         &network_settings,
@@ -752,7 +752,7 @@ async fn lock_and_sync(
     locked: bool,
     dependency_type: &DependencyType,
     raw: bool,
-    bound_kind: &AddBoundsKind,
+    bound_kind: Option<AddBoundsKind>,
     constraints: Vec<NameRequirementSpecification>,
     settings: &ResolverInstallerSettings,
     network_settings: &NetworkSettings,
@@ -828,6 +828,15 @@ async fn lock_and_sync(
                 None => true,
             };
             if !is_empty {
+                if let Some(bound_kind) = bound_kind {
+                    writeln!(
+                        printer.stderr(),
+                        "{} Using explicit requirement `{}` over bounds preference `{}`",
+                        "note:".bold(),
+                        edit.requirement,
+                        bound_kind
+                    )?;
+                }
                 continue;
             }
 
@@ -840,7 +849,12 @@ async fn lock_and_sync(
             // For example, convert `1.2.3+local` to `1.2.3`.
             let minimum = (*minimum).clone().without_local();
 
-            toml.set_dependency_bound(&edit.dependency_type, *index, minimum, bound_kind)?;
+            toml.set_dependency_bound(
+                &edit.dependency_type,
+                *index,
+                minimum,
+                bound_kind.unwrap_or_default(),
+            )?;
 
             modified = true;
         }

crates/uv/tests/it/edit.rs

@@ -11727,7 +11727,7 @@ fn add_optional_normalize() -> Result<()> {
     Ok(())
 }
 
-/// Add a PyPI requirement with the given bounds.
+/// Test `uv add` with different kinds of bounds and constraints.
 #[test]
 fn add_bounds() -> Result<()> {
     let context = TestContext::new("3.12");
@@ -11905,3 +11905,53 @@ fn add_bounds() -> Result<()> {
 
     Ok(())
 }
+
+/// Hint that we're using an explicit bound over the preferred bounds.
+#[test]
+fn add_bounds_requirement_over_bounds_kind() -> Result<()> {
+    let context = TestContext::new("3.12");
+
+    // Set bounds in `uv.toml`
+    let uv_toml = context.temp_dir.child("uv.toml");
+    uv_toml.write_str(indoc! {r#"
+        add-bounds = "exact"
+    "#})?;
+    let pyproject_toml = context.temp_dir.child("pyproject.toml");
+    pyproject_toml.write_str(indoc! {r#"
+        [project]
+        name = "project"
+        version = "0.1.0"
+        requires-python = ">=3.12"
+    "#})?;
+
+    uv_snapshot!(context.filters(), context.add().arg("anyio==4.2").arg("idna").arg("--bounds").arg("minor").arg("--preview"), @r"
+    success: true
+    exit_code: 0
+    ----- stdout -----
+
+    ----- stderr -----
+    Resolved 4 packages in [TIME]
+    note: Using explicit requirement `anyio==4.2` over bounds preference `minor`
+    Prepared 3 packages in [TIME]
+    Installed 3 packages in [TIME]
+     + anyio==4.2.0
+     + idna==3.6
+     + sniffio==1.3.1
+    ");
+
+    let pyproject_toml = context.read("pyproject.toml");
+    assert_snapshot!(
+        pyproject_toml, @r#"
+    [project]
+    name = "project"
+    version = "0.1.0"
+    requires-python = ">=3.12"
+    dependencies = [
+        "anyio==4.2",
+        "idna>=3.6,<3.7",
+    ]
+    "#
+    );
+
+    Ok(())
+}