Skip to content

Scope the User Interface Test Build to Its Own Derived Data - #200

Merged
PSchmiedmayer merged 1 commit into
mainfrom
fix/scope-ui-test-derived-data
Aug 15, 2026
Merged

Scope the User Interface Test Build to Its Own Derived Data#200
PSchmiedmayer merged 1 commit into
mainfrom
fix/scope-ui-test-derived-data

Conversation

@PSchmiedmayer

Copy link
Copy Markdown
Collaborator

♻️ Current situation & Problem

The user interface test build writes to the shared derived data directory of the machine it runs on, which every other run on that self-hosted runner writes to as well. When two runs overlap, or when one is cancelled part way through a build, the next run reads a precompiled module that no longer matches the module map it was built from, and the build fails before compiling anything of ours:

error: file '.../DerivedData/MyHeartCounts-<hash>/Build/Products/Debug-iphonesimulator/FirebaseFirestoreInternal.framework/Modules/module.modulemap'
has been modified since the module file '.../SwiftExplicitPrecompiledModules/FirebaseFirestoreInternalWrapper-....pcm' was built
** TEST BUILD FAILED **

This is the only build in this repository that does not name a derived data path, so it is the only one exposed to it. It happened on the deployment run for the previous change, and re-running the same commit with no edit at all succeeded, which is the signature of a shared cache rather than anything in the project.

The same file also carries a job that was commented out when the user interface tests moved to the sharded build and run they use today. It has been dead since.

No related issue was identified.

⚙️ Release Notes

  • Give the user interface test build its own derived data directory, so concurrent and cancelled runs on the same machine can no longer leave a cache behind that fails the next one.
  • Remove the superseded user interface test job.

📚 Documentation

No documentation changes are required.

✅ Testing

  • The build the change touches is the one that produced the failure above, and it runs on every pull request, including this one
  • actionlint

Code of Conduct & Contributing Guidelines

By creating and submitting this pull request, you agree to follow our Code of Conduct and Contributing Guidelines:

Copilot AI lite review requested due to automatic review settings August 15, 2026 00:07

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Copilot was unable to review this pull request because the user who requested the review has reached their quota limit.

@coderabbitai

coderabbitai Bot commented Aug 15, 2026

Copy link
Copy Markdown

Important

Review available on request

  • 🔍 Trigger review

Reviews should be triggered manually for repositories with fewer than 10 stars. Select Trigger review above or comment @coderabbitai review to review the latest changes. For a full review, comment @coderabbitai full review.

⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro Plus

Run ID: b0f016cb-e10c-4141-a8da-ffb994375308


Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands.

@PSchmiedmayer
PSchmiedmayer merged commit 01a4955 into main Aug 15, 2026
27 checks passed
@github-project-automation github-project-automation Bot moved this from In Progress to Done in MyHeart Counts Project Planning Aug 15, 2026
@PSchmiedmayer
PSchmiedmayer deleted the fix/scope-ui-test-derived-data branch August 15, 2026 04:59
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants