fix(cli): resolve derived data path from DERIVED_DATA_DIR env in inspect commands (tuist#9396)

Co-authored-by: Claude Opus 4.6 <noreply@anthropic.com>

Author: fortmarek

cli/Sources/TuistKit/Services/Inspect/InspectBuildCommandService.swift

@@ -110,12 +110,14 @@ struct InspectBuildCommandService {
         Logger.current.debug("Inspect build: base path resolved to \(basePath.pathString)")
         let projectPath = try await xcodeProjectOrWorkspacePathLocator.locate(from: basePath)
         Logger.current.debug("Inspect build: project path resolved to \(projectPath.pathString)")
-        let currentWorkingDirectory = try await Environment.current.currentWorkingDirectory()
-        var projectDerivedDataDirectory: AbsolutePath! = try derivedDataPath.map { try AbsolutePath(
-            validating: $0,
-            relativeTo: currentWorkingDirectory
-        ) }
-        if projectDerivedDataDirectory == nil {
+        let projectDerivedDataDirectory: AbsolutePath
+        if let derivedDataPath {
+            let currentWorkingDirectory = try await Environment.current.currentWorkingDirectory()
+            projectDerivedDataDirectory = try AbsolutePath(
+                validating: derivedDataPath,
+                relativeTo: currentWorkingDirectory
+            )
+        } else {
             projectDerivedDataDirectory = try await derivedDataLocator.locate(for: projectPath)
         }
         Logger.current.debug("Inspect build: derived data directory resolved to \(projectDerivedDataDirectory.pathString)")

cli/Sources/TuistKit/Services/Inspect/InspectTestCommandService.swift

@@ -133,13 +133,14 @@ struct InspectTestCommandService {
 
         let projectPath = try await xcodeProjectOrWorkspacePathLocator.locate(from: basePath)
         Logger.current.debug("Inspect test: project path resolved to \(projectPath.pathString)")
-        let projectDerivedDataDirectory = if let derivedDataPath {
-            try AbsolutePath(
+        let projectDerivedDataDirectory: AbsolutePath
+        if let derivedDataPath {
+            projectDerivedDataDirectory = try AbsolutePath(
                 validating: derivedDataPath,
                 relativeTo: currentWorkingDirectory
             )
         } else {
-            try await derivedDataLocator.locate(for: projectPath)
+            projectDerivedDataDirectory = try await derivedDataLocator.locate(for: projectPath)
         }
         Logger.current.debug("Inspect test: derived data directory resolved to \(projectDerivedDataDirectory.pathString)")
 

cli/Sources/TuistSupport/Utils/DerivedDataLocator.swift

@@ -17,6 +17,13 @@ public final class DerivedDataLocator: DerivedDataLocating {
     public func locate(
         for projectPath: AbsolutePath
     ) async throws -> AbsolutePath {
+        if let derivedDataDir = Environment.current.variables["DERIVED_DATA_DIR"] {
+            let derivedDataDirPath = try AbsolutePath(validating: derivedDataDir)
+            let defaultDerivedDataDirectory = try await Environment.current.derivedDataDirectory()
+            if derivedDataDirPath != defaultDerivedDataDirectory {
+                return derivedDataDirPath
+            }
+        }
         let hash = try XcodeProjectPathHasher.hashString(for: projectPath.pathString)
         return try await Environment.current.derivedDataDirectory()
             .appending(component: "\(projectPath.basenameWithoutExt)-\(hash)")

cli/Tests/TuistSupportTests/Utils/DerivedDataLocatorTests.swift

@@ -0,0 +1,56 @@
+import FileSystem
+import FileSystemTesting
+import Foundation
+import Path
+import Testing
+import TuistEnvironment
+import TuistTesting
+
+@testable import TuistSupport
+
+struct DerivedDataLocatorTests {
+    private let subject = DerivedDataLocator()
+
+    @Test(.inTemporaryDirectory, .withMockedEnvironment())
+    func locate_uses_DERIVED_DATA_DIR_when_different_from_default() async throws {
+        let temporaryDirectory = try #require(FileSystem.temporaryTestDirectory)
+        let projectPath = temporaryDirectory.appending(component: "App.xcodeproj")
+
+        let mockedEnvironment = try #require(Environment.mocked)
+        let customDerivedDataPath = temporaryDirectory.appending(component: "custom-derived-data")
+        mockedEnvironment.variables["DERIVED_DATA_DIR"] = customDerivedDataPath.pathString
+
+        let result = try await subject.locate(for: projectPath)
+
+        #expect(result == customDerivedDataPath)
+    }
+
+    @Test(.inTemporaryDirectory, .withMockedEnvironment())
+    func locate_ignores_DERIVED_DATA_DIR_when_equal_to_default() async throws {
+        let temporaryDirectory = try #require(FileSystem.temporaryTestDirectory)
+        let projectPath = temporaryDirectory.appending(component: "App.xcodeproj")
+
+        let mockedEnvironment = try #require(Environment.mocked)
+        let defaultDerivedDataDirectory = try await Environment.current.derivedDataDirectory()
+        mockedEnvironment.variables["DERIVED_DATA_DIR"] = defaultDerivedDataDirectory.pathString
+
+        let result = try await subject.locate(for: projectPath)
+
+        #expect(result != defaultDerivedDataDirectory)
+        #expect(result.parentDirectory == defaultDerivedDataDirectory)
+        #expect(result.basename.hasPrefix("App-"))
+    }
+
+    @Test(.inTemporaryDirectory, .withMockedEnvironment())
+    func locate_falls_back_to_hash_based_path() async throws {
+        let temporaryDirectory = try #require(FileSystem.temporaryTestDirectory)
+        let projectPath = temporaryDirectory.appending(component: "App.xcodeproj")
+
+        let defaultDerivedDataDirectory = try await Environment.current.derivedDataDirectory()
+
+        let result = try await subject.locate(for: projectPath)
+
+        #expect(result.parentDirectory == defaultDerivedDataDirectory)
+        #expect(result.basename.hasPrefix("App-"))
+    }
+}