fix(contracts): decode ServerProviders forward-compatibly

[Brechard]

What Changed

ServerProviders (packages/contracts/src/server.ts) was Schema.Array(ServerProvider). This wraps it in ForwardCompatibleArray — the same helper #5055 introduced for ServerConfigIssues and availableEditors — so a provider element this build can't decode is dropped instead of failing the whole array.

Adds one test alongside the existing forward-compat tests in server.test.ts: an array with one decodable provider and one with an unknown status decodes to just the first.

Why

ServerProvider has several nested literal-union status fields: ServerProviderState, ServerProviderAuthStatus, ServerProviderVersionAdvisoryStatus, ServerProviderUpdateStatus. Adding a new member to any of them — which happens whenever provider status/update features grow — makes an older client's entire providers array fail to decode. Since providers lives inside ServerConfig, that failure isn't scoped to the provider list: ServerConfig decodes as one atomic unit, and server.getConfig is the client's initial-sync gate on every WebSocket connection (packages/client-runtime/src/rpc/session.ts). One unrecognized provider status takes down the whole config decode, which the client can't distinguish from a transport error, and it retries forever.

I hit this from a different angle (a downstream fork added editor ids that don't exist upstream, which failed availableEditors decode the same way #5055 already covers) and noticed ServerProviders was the one array left on the un-wrapped Schema.Array from that same PR. Since provider status enums are the field most likely to grow next, it seemed worth closing before it bites a real client version-skew case — an older mobile/desktop build against a server that shipped a new provider status.

UI Changes

None.

Checklist

• This PR is small and focused
• I explained what changed and why
• Tests pass locally (vp test run in packages/contracts, plus full workspace vp run -r typecheck)

---

Note

Low Risk
Contract decoding behavior change in contracts only; aligns with existing forward-compat patterns and reduces version-skew connection failures.

Overview
ServerProviders now uses ForwardCompatibleArray(ServerProvider) instead of a plain Schema.Array, matching availableEditors and ServerConfigIssues.

When the server sends a provider snapshot this build cannot decode (e.g. a new status literal), that element is dropped instead of failing the entire providers array and ServerConfig decode on initial WebSocket sync.

A test in server.test.ts asserts a mix of valid and unknown-status providers decodes to only the recognizable entry.

^{Reviewed by Cursor Bugbot for commit dc16ad8. Bugbot is set up for automated code reviews on this repo. Configure here.}

Note

Decode ServerProviders forward-compatibly by dropping unknown entries

Changes ServerProviders in server.ts from Schema.Array(ServerProvider) to ForwardCompatibleArray(ServerProvider). This makes decoding tolerant of unknown provider status kinds — entries that cannot be decoded are silently dropped instead of failing the entire array.

^{Macroscope summarized dc16ad8.}

[coderabbitai]

Important

Review skipped

Auto reviews are disabled on this repository. Please check the settings in the CodeRabbit UI or the .coderabbit.yaml file in this repository. To trigger a single review, invoke the @coderabbitai review command.

⚙️ Run configuration

Configuration used: Repository UI

Review profile: CHILL

Plan: Pro Plus

Run ID: 2afba617-68a3-414d-9adc-e83d447ddcb6

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

Use the checkbox below for a quick retry:

• 🔍 Trigger review

---

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

[macroscopeapp]

Approvability

Verdict: Approved dc16ad8

This is a straightforward bug fix applying an existing forward-compatibility pattern (ForwardCompatibleArray) already used elsewhere in this codebase. The change has limited scope, clear intent, and includes appropriate test coverage.

^{You can customize Macroscope's approvability policy. Learn more.}