Filter out empty auxiliary groups

Sources/TuistGenerator/Generator/ProjectDescriptorGenerator.swift

@@ -145,6 +145,8 @@ final class ProjectDescriptorGenerator: ProjectDescriptorGenerating {
             graphTraverser: graphTraverser
         )
 
+        groups.removeEmptyAuxiliaryGroups()
+
         let xcodeProj = XcodeProj(workspace: workspace, pbxproj: pbxproj)
         return ProjectDescriptor(
             path: project.path,

Sources/TuistGenerator/Generator/ProjectGroups.swift

@@ -34,6 +34,17 @@ class ProjectGroups {
     private let pbxproj: PBXProj
     private let projectGroups: [String: PBXGroup]
 
+    private var auxiliaryGroups: [PBXGroup] {
+        // List of groups Xcode doesn't have by default if empty
+        //
+        // Note: The`Products` group is always included in the pbxproj but Xcode
+        // hides it in the UI if its empty.
+        [
+            frameworks,
+            cachedFrameworks,
+        ]
+    }
+
     // MARK: - Init
 
     private init(
@@ -67,6 +78,13 @@ class ProjectGroups {
         return group
     }
 
+    func removeEmptyAuxiliaryGroups() {
+        for emptyGroup in auxiliaryGroups.filter(\.children.isEmpty) {
+            sortedMain.children.removeAll(where: { $0 == emptyGroup })
+            pbxproj.delete(object: emptyGroup)
+        }
+    }
+
     static func generate(
         project: Project,
         pbxproj: PBXProj

Tests/TuistGeneratorTests/Generator/ProjectGroupsTests.swift

@@ -120,6 +120,49 @@ final class ProjectGroupsTests: XCTestCase {
         ])
     }
 
+    func test_removeEmptyAuxiliaryGroups_removesEmptyGroups() throws {
+        // Given
+        let project = Project.test()
+        subject = ProjectGroups.generate(
+            project: project,
+            pbxproj: pbxproj
+        )
+
+        // When
+        subject.removeEmptyAuxiliaryGroups()
+
+        // Then
+        let paths = subject.sortedMain.children.map(\.nameOrPath)
+        XCTAssertEqual(paths, [
+            "Project",
+            "Products",
+        ])
+    }
+
+    func test_removeEmptyAuxiliaryGroups_preservesNonEmptyGroups() throws {
+        // Given
+        let project = Project.test()
+        subject = ProjectGroups.generate(
+            project: project,
+            pbxproj: pbxproj
+        )
+
+        addFile(to: subject.frameworks)
+        addFile(to: subject.cachedFrameworks)
+
+        // When
+        subject.removeEmptyAuxiliaryGroups()
+
+        // Then
+        let paths = subject.sortedMain.children.map(\.nameOrPath)
+        XCTAssertEqual(paths, [
+            "Project",
+            "Frameworks",
+            "Cache",
+            "Products",
+        ])
+    }
+
     func test_targetFrameworks() throws {
         subject = ProjectGroups.generate(
             project: project,
@@ -213,4 +256,12 @@ final class ProjectGroupsTests: XCTestCase {
         XCTAssertNil(main.tabWidth)
         XCTAssertNil(main.wrapsLines)
     }
+
+    // MARK: - Helpers
+
+    private func addFile(to group: PBXGroup) {
+        let file = PBXFileReference()
+        pbxproj.add(object: file)
+        group.children.append(file)
+    }
 }