fix(cmux-next): VNC reconnect test helpers are nonisolated (CmuxNextAppTests compile red)
#26557 landed RemoteVNCReconnectTests (carried from #26697). Under the CmuxNext test target's .defaultIsolation(MainActor.self), the private AttemptCounter is main-actor isolated, so the @Sendable transport factory cannot call increment():
RemoteViewVNCTests.swift:130:22: error: call to main actor-isolated instance method 'increment()' in a synchronous nonisolated context
CmuxNextAppTests does not compile on feat-cmux-next at 0ce3c31ee53, so every CmuxNext suite run fails at build. This marks the two lock-guarded test helpers nonisolated, as #27811 already writes them (that PR now conflicts with base). Two lines, tests only.
Observed / Not run
- Fleet focused run at feat-cmux-next 57b38a2-based head: test target fails to compile with the error above.
- Fleet focused run at this head (RemoteVNCReconnectTests, RemoteVNCMappingTests, PaneTabBarControlsTests, NewTabMenuTriggerTests): result posted in a comment below.
Changelog
none
Agent session: agent session
š¤ Generated with Claude Code
![View with [code]smith](https://pr-comments-assets.blacksmith.sh/codesmith/view-with-codesmith-dark-v3-3x.png)
![Autofix with [code]smith](https://pr-comments-assets.blacksmith.sh/codesmith/autofix-with-codesmith-dark-3x.png)
Need help on this PR? Tag @codesmith-bot with what you need.
Summary by cubic
Fixes the CmuxNextAppTests compile failure on feat-cmux-next by marking two VNC reconnect test helpers as nonisolated.
Under the test target's MainActor default isolation, the @Sendable transport factory could not call AttemptCounter.increment(), breaking the build for every CmuxNext suite run. The lock-guarded CountingRFBTransport and AttemptCounter helpers are now nonisolated; test behavior is unchanged.