feat: Handle references for exception sets in Synchronized Groups

[mirkokg]

Generate permanent references for PBXFileSystemSynchronizedExceptionSet objects, including build file and build phase exception sets. Updates tests to verify that references for exception sets are correctly converted to permanent values.

Resolves #934

Short description 📝

When creating projects with PBXFileSystemSynchronizedRootGroup objects that contain exception sets (PBXFileSystemSynchronizedBuildFileExceptionSet or PBXFileSystemSynchronizedGroupBuildPhaseMembershipExceptionSet), the exception set IDs were being written with TEMP_ prefixes instead of proper deterministic IDs.

Solution 📦

Extended the ReferenceGenerator to process exception sets when generating references for synchronized root groups. Added a new generateExceptionSetReferences method that:

1. Iterates through all exceptions in a PBXFileSystemSynchronizedRootGroup
2. Generates deterministic IDs based on the exception type:

• For PBXFileSystemSynchronizedBuildFileExceptionSet: uses the target reference as part of the identifier
• For PBXFileSystemSynchronizedGroupBuildPhaseMembershipExceptionSet: uses the build phase reference as part of the identifier

This follows the same pattern used for other object types in the reference generator.

Implementation 👩‍💻👨‍💻

• Identified that PBXFileSystemSynchronizedRootGroup.exceptions were not being processed in ReferenceGenerator
• Modified generateSynchronizedRootGroupReferences() to iterate through exception sets after fixing the root group reference
• Added generateExceptionSetReferences() method to handle reference generation for exception set objects
• Added unit tests to verify the functionality
• Verified all tests pass with the changes
• Confirmed no TEMP_ prefixes appear in generated project files

[mirkokg]

@fortmarek @yonaskolb @pepicrft can someone please help with reviewing this? I would really appreciate it.

[yonaskolb]

Looks good to me, thanks @mirkokg!

[yonaskolb]

I'm not familiar with the CI workflows in this repo anymore but looks like there are just some formatting errors related to trailing spaces.

[mirkokg]

@yonaskolb I removed extra spaces. Thanks!

[pepicrft]

Thanks @mirkokg. We can merge it once CI is green.

[dosubot]

Related Documentation

Checked 1 published document(s) in 1 knowledge base(s). No updates required.

^{How did I do? Any feedback?}  

[pepicrft]

@mirkokg the changes have been released in 9.7.0

[mirkokg]

Thanks!