wip: support outputs for develop feature

Author: baszalmstra

crates/pixi/tests/integration_rust/common/package_database.rs

@@ -9,7 +9,7 @@ use itertools::Itertools;
 use miette::IntoDiagnostic;
 use rattler_conda_types::{
     ChannelInfo, PackageName, PackageRecord, PackageUrl, Platform, RepoData, VersionWithSource,
-    package::ArchiveType,
+    package::{ArchiveType, RunExportsJson},
 };
 use std::{collections::HashSet, path::Path};
 use tempfile::TempDir;
@@ -294,7 +294,7 @@ impl PackageBuilder {
                 track_features: vec![],
                 version: self.version,
                 purls: self.purls,
-                run_exports: None,
+                run_exports: Some(RunExportsJson::default()),
                 python_site_packages_path: None,
                 experimental_extra_depends: Default::default(),
             },

crates/pixi/tests/integration_rust/develop_dependencies_tests.rs

@@ -62,8 +62,7 @@ async fn test_develop_dependencies_basic() {
     setup_tracing();
 
     // Create a package database with common dependencies
-    let mut package_database = create_test_package_database();
-    package_database.add_package(Package::build("empty-backend", "0.1.0").finish());
+    let package_database = create_test_package_database();
 
     // Convert to channel
     let channel = package_database.into_channel().await.unwrap();
@@ -156,8 +155,7 @@ async fn test_develop_dependencies_with_source_dependencies() {
     setup_tracing();
 
     // Create a package database
-    let mut package_database = create_test_package_database();
-    package_database.add_package(Package::build("empty-backend", "0.1.0").finish());
+    let package_database = create_test_package_database();
 
     let channel = package_database.into_channel().await.unwrap();
 
@@ -273,8 +271,7 @@ package-a = {{ path = "./package-a" }}
 async fn test_develop_dependencies_with_cross_references() {
     setup_tracing();
 
-    let mut package_database = create_test_package_database();
-    package_database.add_package(Package::build("empty-backend", "0.1.0").finish());
+    let package_database = create_test_package_database();
 
     let channel = package_database.into_channel().await.unwrap();
 
@@ -379,8 +376,7 @@ package-y = {{ path = "{}" }}
 async fn test_develop_dependencies_in_features() {
     setup_tracing();
 
-    let mut package_database = create_test_package_database();
-    package_database.add_package(Package::build("empty-backend", "0.1.0").finish());
+    let package_database = create_test_package_database();
 
     let channel = package_database.into_channel().await.unwrap();
 
@@ -445,13 +441,127 @@ feature-package = {{ path = "./feature-package" }}
     );
 }
 
+/// Test that a source package can be listed both in [develop] and in dependencies
+/// without causing conflicts (the package is essentially included twice, once as a develop dep
+/// and once as a regular source dep)
+#[tokio::test]
+async fn test_develop_and_regular_dependency_same_package() {
+    setup_tracing();
+
+    let package_database = create_test_package_database();
+
+    let channel = package_database.into_channel().await.unwrap();
+
+    let backend_override = BackendOverride::from_memory(PassthroughBackend::instantiator());
+    let pixi = PixiControl::new()
+        .unwrap()
+        .with_backend_override(backend_override);
+
+    // Create a shared package that will be both a develop dependency and a regular dependency
+    let shared_package_path = create_source_package(
+        pixi.workspace_path(),
+        "shared-package",
+        "1.0.0",
+        r#"
+[package.host-dependencies]
+python = ">=3.8"
+"#,
+    );
+
+    // Create another package that depends on shared-package as a regular source dependency
+    let _dependent_package = create_source_package(
+        pixi.workspace_path(),
+        "dependent-package",
+        "1.0.0",
+        &format!(
+            r#"
+[package.run-dependencies]
+shared-package = {{ path = "{}" }}
+numpy = ">=1.0"
+"#,
+            shared_package_path.to_string_lossy().replace('\\', "\\\\")
+        ),
+    );
+
+    // Create a manifest that:
+    // 1. Lists shared-package as a develop dependency
+    // 2. Lists dependent-package as a regular source dependency
+    // This means shared-package appears both as a develop dep and as a transitive source dep
+    let manifest_content = format!(
+        r#"
+[workspace]
+channels = ["{}"]
+platforms = ["{}"]
+preview = ["pixi-build"]
+
+[dependencies]
+dependent-package = {{ path = "./dependent-package" }}
+
+[develop]
+shared-package = {{ path = "{}" }}
+"#,
+        channel.url(),
+        Platform::current(),
+        shared_package_path.to_string_lossy().replace('\\', "\\\\")
+    );
+
+    fs::write(pixi.manifest_path(), manifest_content).unwrap();
+
+    // Update the lock-file - this should work without conflicts
+    let lock_file = pixi.update_lock_file().await.unwrap();
+
+    // Verify that python is in the lock-file (from shared-package's dependencies)
+    assert!(
+        lock_file.contains_conda_package(
+            consts::DEFAULT_ENVIRONMENT_NAME,
+            Platform::current(),
+            "python",
+        ),
+        "python should be in the lock-file (run dependency of shared-package)"
+    );
+
+    // Verify that numpy is in the lock-file (from dependent-package's dependencies)
+    assert!(
+        lock_file.contains_conda_package(
+            consts::DEFAULT_ENVIRONMENT_NAME,
+            Platform::current(),
+            "numpy",
+        ),
+        "numpy should be in the lock-file (run dependency of dependent-package)"
+    );
+
+    // Verify that dependent-package IS in the lock-file (it's a regular source dependency)
+    assert!(
+        lock_file.contains_conda_package(
+            consts::DEFAULT_ENVIRONMENT_NAME,
+            Platform::current(),
+            "dependent-package",
+        ),
+        "dependent-package SHOULD be in the lock-file (it's a regular source dependency)"
+    );
+
+    // Key assertion: shared-package WILL appear in the lock-file as a built package
+    // because it's a source dependency of dependent-package.
+    // The fact that it's also in [develop] doesn't prevent it from being built when
+    // it's needed as a dependency of another package.
+    // This is correct behavior - [develop] means "install my dependencies without building me",
+    // but if another package needs it built, it will be built.
+    assert!(
+        lock_file.contains_conda_package(
+            consts::DEFAULT_ENVIRONMENT_NAME,
+            Platform::current(),
+            "shared-package",
+        ),
+        "shared-package SHOULD be in the lock-file (it's built as a source dependency of dependent-package)"
+    );
+}
+
 /// Test that platform-specific develop dependencies work correctly
 #[tokio::test]
 async fn test_develop_dependencies_platform_specific() {
     setup_tracing();
 
-    let mut package_database = create_test_package_database();
-    package_database.add_package(Package::build("empty-backend", "0.1.0").finish());
+    let package_database = create_test_package_database();
 
     let channel = package_database.into_channel().await.unwrap();
 

crates/pixi_command_dispatcher/src/command_dispatcher/mod.rs

@@ -505,81 +505,7 @@ impl CommandDispatcher {
         self.execute_task(spec).await
     }
 
-    /// Returns the dependencies (build, host, and run) for a specific output
-    /// from a source package.
     ///
-    /// This method retrieves metadata from the build backend for the given
-    /// source and extracts the dependencies for the specified output name.
-    /// The dependencies are returned in their raw form as specified by the
-    /// backend, including both regular dependencies and constraints.
-    ///
-    /// This is useful when you want to install the dependencies of a source
-    /// package without actually building it - for instance, to set up a
-    /// development environment or to inspect what would be needed to build
-    /// the package.
-    ///
-    /// # Requirements
-    ///
-    /// - The build backend must support the `conda/outputs` procedure (API v1+)
-    /// - The specified output must exist in the source package
-    pub async fn get_output_dependencies(
-        &self,
-        spec: crate::GetOutputDependenciesSpec,
-    ) -> Result<crate::OutputDependencies, CommandDispatcherError<crate::GetOutputDependenciesError>>
-    {
-        spec.request(self.clone()).await
-    }
-
-    /// Expands a list of dev sources into their dependencies.
-    ///
-    /// Dev sources are source packages whose dependencies should be installed
-    /// without building the packages themselves. This is particularly useful
-    /// for development environments where you want to work on a package while
-    /// having all its dependencies available.
-    ///
-    /// For each dev source, this method:
-    /// 1. Checks out the source code
-    /// 2. Extracts all dependencies (build, host, and run) and constraints
-    /// 3. Filters out dependencies that are themselves dev sources (to avoid
-    ///    duplication)
-    ///
-    /// The result can then be merged into a `PixiEnvironmentSpec` and passed to
-    /// `solve_pixi_environment` to create an environment with all the dependencies
-    /// but without the dev source packages themselves.
-    ///
-    /// # Example
-    ///
-    /// ```ignore
-    /// // Expand dev sources
-    /// let expanded = command_dispatcher.expand_dev_sources(spec).await?;
-    ///
-    /// // Merge into environment spec
-    /// let mut env_spec = PixiEnvironmentSpec::default();
-    /// env_spec.dependencies.extend(expanded.dependencies);
-    /// env_spec.constraints.extend(expanded.constraints);
-    ///
-    /// // Solve the environment
-    /// let solved = command_dispatcher.solve_pixi_environment(env_spec).await?;
-    /// ```
-    ///
-    /// # Use case
-    ///
-    /// You're developing two packages (`pkgA` and `pkgB`) where `pkgA` depends
-    /// on `pkgB`. Instead of building both packages, you can use
-    /// `expand_dev_sources` to:
-    /// 1. Get the dependencies of both `pkgA` and `pkgB`
-    /// 2. Remove `pkgB` from `pkgA`'s dependencies (since it's also a dev source)
-    /// 3. Create an environment with just the external dependencies
-    ///
-    /// This allows you to work on the source code of both packages directly.
-    pub async fn expand_dev_sources(
-        &self,
-        spec: crate::ExpandDevSourcesSpec,
-    ) -> Result<crate::ExpandedDevSources, CommandDispatcherError<crate::ExpandDevSourcesError>>
-    {
-        spec.request(self.clone()).await
-    }
-
     /// Calls into a pixi build backend to perform a source build.
     pub(crate) async fn backend_source_build(
         &self,