🛑 Prevent overwriting identical workspace data

Sources/XcodeProj/Project/XcodeProj.swift

@@ -130,7 +130,6 @@ extension XcodeProj: Writable {
     /// - Parameter override: if workspace should be overridden. Default is true.
     ///   If false will throw error if workspace already exists at the given path.
     public func writeWorkspace(path: Path, override: Bool = true) throws {
-        guard let p = path.glob("*.xcworkspacedata").first, workspace.data != (try? XCWorkspaceData(path: p)) else { return }
         try workspace.write(path: XcodeProj.workspacePath(path), override: override)
     }
 

Sources/XcodeProj/Workspace/XCWorkspace.swift

@@ -53,6 +53,15 @@ public final class XCWorkspace: Writable, Equatable {
     public func write(path: Path, override: Bool = true) throws {
         let dataPath = path + "contents.xcworkspacedata"
         if override, dataPath.exists {
+            if let existingContent: String = try? dataPath.read(),
+               existingContent == data.rawContents() {
+                // Raw data matches what's on disk
+                // there's no need to overwrite the contents
+                // this mitigates Xcode forcing users to
+                // close and re-open projects/workspaces
+                // on regneration.
+                return
+            }
             try dataPath.delete()
         }
         try dataPath.mkpath()

Sources/XcodeProj/Workspace/XCWorkspaceData.swift

@@ -36,20 +36,24 @@ extension XCWorkspaceData: Writable {
 
         self.init(children: children)
     }
-
-    // MARK: - <Writable>
-
-    public func write(path: Path, override: Bool = true) throws {
+
+    func rawContents() -> String {
         let document = AEXMLDocument()
         let workspace = document.addChild(name: "Workspace", value: nil, attributes: ["version": "1.0"])
         _ = children
             .map { $0.xmlElement() }
             .map(workspace.addChild)
+        return document.xmlXcodeFormat
+    }
 
+    // MARK: - <Writable>
+
+    public func write(path: Path, override: Bool = true) throws {
+        let rawXml = rawContents()
         if override, path.exists {
             try path.delete()
         }
-        try path.write(document.xmlXcodeFormat)
+        try path.write(rawXml)
     }
 }