[pepicrft]

What changed

ReferenceGenerator now generates the references of PBXFileSystemSynchronizedGroupBuildPhaseMembershipExceptionSet objects after the targets pass instead of while walking the groups. The PR also adds a regression test showing that the same project gets the same references when generated twice. The fixture round-trip test from #1200 now also checks that writing each fixture twice gives the same bytes; that check had to be left out of #1200 because of this bug.

Root cause

An exception set's reference is an MD5 of its identifiers, and those include its build phase's reference value. The generator fixes synchronized root groups and their exception sets while walking the main group, but build phases only get permanent references later, in generateTargetReferences. When the build phase was still temporary, the hash input contained its random TEMP_<UUID>. That made the exception set's reference, and every list pointing at it, different on every write.

It shows up whenever such an exception set and its build phase are new: a project built in code, or one written after invalidateUUIDs(). For example, writing Fixtures/SynchronizedRootGroups after invalidateUUIDs() gave a different reference for its build phase exception set on every run.

Approach

When the generator reaches a build phase membership exception set, it records the set along with the identifiers it would have used. Once all targets and their build phases have permanent references, it fixes them.

I preferred this to dropping the build phase reference from the identifiers. That would have changed the reference of every existing exception set, while deferring leaves the inputs unchanged whenever the build phase was already permanent. Build-file exception sets already depend on their target's reference, which is fixed before the groups are walked, so they were deterministic and are left alone.

Impact

• Projects that read and write existing .pbxproj files see no change: their build phases already have permanent references, so the identifiers are the same as before.
• Projects whose exception sets and build phases are new now get stable references instead of a different one on every write. That affects code that creates these objects and code that calls invalidateUUIDs().

Validation

• Rebased onto main after perf: speed up writing pbxproj files #1200. swift test: 384 XCTest and 103 Swift Testing tests pass.
• The strengthened round-trip test fails only on SynchronizedRootGroups without the fix and passes on all six fixtures with it.
• The new test_projectWithFilesystemSynchronizedRootGroupWithBuildPhaseException_generatesDeterministicIdentifiers fails without the fix and passes with it.
• A scratch check, not committed, wrote five fixtures five times each after invalidateUUIDs() and got identical bytes every time: SynchronizedRootGroups, iOS/Project, Xcode16/Test, AppWithExtensions and TargetWithCustomBuildRules. Before the fix, SynchronizedRootGroups differed on every run.
• mise run lint: SwiftFormat reports 0 files to format, and SwiftLint has no new warnings.

I found this while writing tests for #1200.