chore: Simplify shared Roslyn worker internals

[hatayama]

Summary

• Separate shared Roslyn worker wire framing and template rendering from process lifecycle orchestration.
• Keep dynamic-code compilation behavior and worker protocol bytes unchanged.

User Impact

• Execute-dynamic-code compilation, retry, timeout, cancellation, and fallback behavior remain unchanged.
• The smaller worker host makes later lifecycle and assembly-build refactors easier to review independently.

Changes

• Add SharedRoslynCompilerWorkerProtocol to own request encoding, response header parsing, diagnostic framing, protocol read timeout, quit marker, and worker program template token replacement.
• Retarget existing command/template tests directly to the new owner and remove six host test wrappers.
• Add characterization coverage for valid response headers, invalid prefixes, and non-numeric exit codes before extraction.
• Reduce SharedRoslynCompilerWorkerHost from 988 to 865 lines while leaving retry, process lifecycle, assembly build, and shutdown state in the host.
• The only non-move production change is passing the current worker StandardOutput reader into ReadDiagnosticLines; all compile calls run under SharedCompilerWorkerLock, and Process.StandardOutput returns the same reader instance used for the response header.
• Preserve the existing double Path.GetFullPath application, blocking read behavior, strings, markers, timeout values, and out int exitCode contract.

Verification

• dist/darwin-arm64/uloop compile --project-path "$(git rev-parse --show-toplevel)" (0 errors, 0 warnings)
• SharedRoslynCompilerWorkerHostTests (10 passed)
• ExternalCompilerPathResolverTests (15 passed, including worker-build fallback)
• StaticFacadeStateGuardTests (20 passed)
• Confirmed that removed host wrappers have zero references and the protocol has no reverse dependency on the host.
• No IPC wire shape, CLI/package protocol version, or release input changes.

[coderabbitai]

Warning

Review limit reached

@hatayama, you've reached your PR review limit, so we couldn't start this review.

Next review available in: 17 minutes

Enable usage-based reviews in Billing to review now. Otherwise, wait until the next included review is available.
You're only billed for reviews past your plan's rate limits ($0.25/file).

How can I continue?

After more reviews become available, a review can be triggered using the @coderabbitai review command as a PR comment. Alternatively, push new commits to this PR.

To avoid repeated limits, reduce automatic review volume by pausing incremental auto-reviews earlier, using label-based review opt-in, excluding WIP or generated PR titles, or requesting reviews manually when the PR is ready. If your team needs uninterrupted high-volume reviews, an organization admin can enable usage-based reviews.

How do review limits work?

CodeRabbit enforces per-developer PR review limits for each organization. Most developers receive the normal plan review availability.

For paid Pro and Pro+ PR reviews, CodeRabbit uses adaptive limits for sustained high-volume activity. When a developer's recent PR review activity reaches the 95th percentile or higher among CodeRabbit users, additional reviews become available more gradually as earlier reviews age out of the rolling window.

Please refer docs for additional details.

Review details

⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Pro

Run ID: a00b5f31-916f-4453-a6f3-ee2f9c905f7c

📥 Commits

Reviewing files that changed from the base of the PR and between 829e84f and 48d8191.

⛔ Files ignored due to path filters (1)

• Packages/src/Editor/FirstPartyTools/ExecuteDynamicCode/DynamicCompilation/SharedRoslynCompilerWorkerProtocol.cs.meta is excluded by none and included by none

📒 Files selected for processing (3)

• Assets/Tests/Editor/DynamicCodeToolTests/SharedRoslynCompilerWorkerHostTests.cs
• Packages/src/Editor/FirstPartyTools/ExecuteDynamicCode/DynamicCompilation/SharedRoslynCompilerWorkerHost.cs
• Packages/src/Editor/FirstPartyTools/ExecuteDynamicCode/DynamicCompilation/SharedRoslynCompilerWorkerProtocol.cs

✨ Finishing Touches

🧪 Generate unit tests (beta)

• Create PR with unit tests
• Commit unit tests in branch refactor/hatayama/extract-shared-roslyn-worker-protocol

---

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.}

[cubic-dev-ai]

No issues found across 4 files

_{You’re at about 92% of the monthly reviewed-line limit. You may want to disable incremental reviews to conserve quota. Reviews will continue until that limit is exceeded. If you need help avoiding interruptions, please contact contact@cubic.dev.}

_{Re-trigger cubic}