Add incremental build cache support and parallel linking

[js-9]

Changes

• write incremental build output to Library so it is cached together with the Library
• enable parallel linking for il2cpp test runs

Successful Workflow Run Link

Checklist

• Read the contribution guide and accept the code of conduct
• Docs (If new inputs or outputs have been added or changes to behavior that should be documented. Please make a PR
  in the documentation repo)
• Readme (updated or not needed)
• Tests (added, updated or not needed)

Related documentation PR at game-ci/documentation#522

Summary by CodeRabbit

• Chores
  • Updated Docker environment configuration for Windows systems with cache directory settings to ensure consistent behavior across deployments.

_{✏️ Tip: You can customize this high-level summary in your review settings.}

[github-actions]

[coderabbitai]

Walkthrough

A single environment variable BEE_CACHE_DIRECTORY has been added to the Windows Docker run command in src/model/docker.ts. This variable points to a cache directory path and requires no functional logic changes.

Changes

Cohort / File(s)	Summary
Docker Configuration src/model/docker.ts	Added --env BEE_CACHE_DIRECTORY=c:/github/workspace/Library/bee_cache to Windows Docker run command

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~2 minutes

• Single-line environment variable addition to a Docker command
• No control flow or logic changes
• Straightforward parameter configuration

Poem

🐰 A cache path in the Docker night,
Whispered softly to the Windows light,
BEE_CACHE_DIRECTORY finds its way,
In /github/workspace, safe to stay! 🍯

Pre-merge checks and finishing touches

❌ Failed checks (1 warning)

Check name	Status	Explanation	Resolution
Description check	⚠️ Warning	The pull request description is missing the required 'Related Issues' and 'Related PRs' sections, and lacks a workflow run link.	Add the missing 'Related Issues' section, add the source branch link to 'Related PRs', and provide a successful workflow run link from your own repository.

✅ Passed checks (2 passed)

Check name	Status	Explanation
Title check	✅ Passed	The title directly matches the PR objectives: it describes both main changes (incremental build cache support and parallel linking) that are implemented in the changeset.
Docstring Coverage	✅ Passed	No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

✨ Finishing touches

• 📝 Generate docstrings

🧪 Generate unit tests (beta)

• Create PR with unit tests
• Post copyable unit tests in a comment

---

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

• X
• Mastodon
• Reddit
• LinkedIn

_{Comment @coderabbitai help to get the list of available commands and usage tips.}

[js-9]

@timcassell Would you mind giving it a spin and reporting? I don't run il2cpp tests myself..a link to a successful run would be appreciated.

[coderabbitai]

Actionable comments posted: 1

🧹 Nitpick comments (1)

src/model/docker.ts (1)

140-140: Consider adding a comment explaining BEE_CACHE_DIRECTORY.

Adding an inline comment would help maintainers understand the purpose of this environment variable and its relationship to Unity's Bee build system and IL2CPP caching.

Apply this diff to add documentation:

                ${ImageEnvironmentFactory.getEnvVarString(parameters)} \
+                # Configure Unity Bee build cache for incremental IL2CPP builds
                --env BEE_CACHE_DIRECTORY=c:/github/workspace/Library/bee_cache \
                --env TEST_PLATFORMS="${testPlatforms}" \

📜 Review details

Configuration used: CodeRabbit UI

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between 61fd9aa and 6e94677.

⛔ Files ignored due to path filters (1)

• dist/test-standalone-scripts/Assets/Editor/UnityTestRunnerAction/PlayerBuildModifier.cs is excluded by !**/dist/**

📒 Files selected for processing (1)

• src/model/docker.ts (1 hunks)