Respect public-dependency when documenting deps

Fixes #2025

When the public-dependency feature is enabled, cargo doc now only
documents direct dependencies and their public dependencies recursively.
This reduces documentation clutter by excluding transitive private
dependencies that users don't need to know about.

Behavior:
- Without public-dependency: all deps documented (backward compatible)
- With public-dependency enabled:
  * Direct dependencies always documented
  * Transitive deps only if marked public

This allows library authors to control their documentation surface by
marking which dependencies are part of their public API.

Author: Ojus Chugh

src/cargo/core/compiler/unit_dependencies.rs

@@ -637,6 +637,17 @@ fn compute_deps_doc(
     // built. If we're documenting *all* libraries, then we also depend on
     // the documentation of the library being built.
     let mut ret = Vec::new();
+    let is_root = state.ws.is_member(&unit.pkg);
+
+    // Check if public-dependency feature is enabled
+    let public_dep = state.gctx.cli_unstable().public_dependency
+        || unit
+            .pkg
+            .manifest()
+            .unstable_features()
+            .require(Feature::public_dependency())
+            .is_ok();
+
     for (id, deps) in state.deps(unit, unit_for) {
         let Some(dep_lib) = calc_artifact_deps(unit, unit_for, id, &deps, state, &mut ret)? else {
             continue;
@@ -657,7 +668,20 @@ fn compute_deps_doc(
             IS_NO_ARTIFACT_DEP,
         )?;
         ret.push(lib_unit_dep);
-        if dep_lib.documented() && state.intent.wants_deps_docs() {
+
+        // Decide whether to document this dependency.
+        // When public-dependency is enabled, only document:
+        // - Direct dependencies of workspace members
+        // - Public dependencies (recursively)
+        let should_doc = dep_lib.documented() && state.intent.wants_deps_docs() && {
+            if public_dep {
+                is_root || state.resolve().is_public_dep(unit.pkg.package_id(), id)
+            } else {
+                true
+            }
+        };
+
+        if should_doc {
             // Document this lib as well.
             let doc_unit_dep = new_unit_dep(
                 state,

tests/testsuite/doc_direct_deps.rs

@@ -60,21 +60,18 @@ fn doc_direct_deps_only() {
     assert!(p.root().join("target/doc/baz/index.html").is_file());
 }
 
-
 #[cargo_test(nightly, reason = "public-dependency feature is unstable")]
 fn doc_with_public_dependency_feature() {
     // foo -> bar (public) -> baz (public)
     // baz should be documented since bar marks it as public
-    
+
     Package::new("baz", "0.0.1")
         .file("src/lib.rs", "pub fn baz() {}")
         .publish();
 
     Package::new("bar", "0.0.1")
         .cargo_feature("public-dependency")
-        .add_dep(
-            cargo_test_support::registry::Dependency::new("baz", "0.0.1").public(true)
-        )
+        .add_dep(cargo_test_support::registry::Dependency::new("baz", "0.0.1").public(true))
         .file("src/lib.rs", "extern crate baz; pub fn bar() {}")
         .publish();
 
@@ -128,7 +125,7 @@ fn doc_with_public_dependency_feature() {
 fn doc_with_private_dependency() {
     // foo -> bar (private) -> baz
     // baz should NOT be documented (transitive private dep)
-    
+
     Package::new("baz", "0.0.1")
         .file("src/lib.rs", "pub fn baz() {}")
         .publish();
@@ -201,7 +198,10 @@ fn doc_mixed_public_private_deps() {
 
     Package::new("priv_dep_with_dep", "0.0.1")
         .dep("transitive", "0.0.1")
-        .file("src/lib.rs", "extern crate transitive; pub fn priv_dep_with_dep() {}")
+        .file(
+            "src/lib.rs",
+            "extern crate transitive; pub fn priv_dep_with_dep() {}",
+        )
         .publish();
 
     let p = project()
@@ -235,8 +235,12 @@ fn doc_mixed_public_private_deps() {
     assert!(p.root().join("target/doc/foo/index.html").is_file());
     assert!(p.root().join("target/doc/pub_dep/index.html").is_file());
     assert!(p.root().join("target/doc/priv_dep/index.html").is_file());
-    assert!(p.root().join("target/doc/priv_dep_with_dep/index.html").is_file());
-    
+    assert!(
+        p.root()
+            .join("target/doc/priv_dep_with_dep/index.html")
+            .is_file()
+    );
+
     // transitive should NOT be documented
     assert!(!p.root().join("target/doc/transitive/index.html").is_file());
 }