fix(server): stop probing Cursor until enabled

[t3dotgg]

Cursor was turned back on by #7089 after #7459 made optional providers opt-in. Every installation then tried to launch cursor-agent at startup and during provider health checks, although the installation guide says Cursor is off by default.

Restore the Cursor opt-in default and verify that untouched installations skip Cursor probes. Saved Cursor opt-ins still work.

Made by GPT-5.6 Sol through the Codex harness.

---

Note

Low Risk
Default-only configuration change with regression tests; explicit Cursor opt-in paths are preserved and probing already short-circuits when disabled.

Overview
Cursor is off by default again, matching Grok/OpenCode and the documented install behavior, so fresh or default server settings no longer spawn cursor-agent during startup or health checks.

The change is in CursorSettings: the schema decoding default for enabled goes from true to false, which flows through defaultEnabledForDriver and DEFAULT_SERVER_SETTINGS. Explicit opt-in is unchanged—settings that set providers.cursor.enabled: true (and matching instance flags) still decode as enabled.

Tests were updated to assert the new defaults, cover preserved opt-in, and confirm the provider registry leaves Cursor disabled and does not probe when only other providers are toggled off (without forcing cursor.enabled: false in the fixture).

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

Note

Set CursorSettings.enabled default to false

• Changes the enabled default in CursorSettings from true to false via Schema.withDecodingDefault(Effect.succeed(false)) in settings.ts, so Cursor is off unless the user opts in.
• Updates tests in settings.test.ts and ProviderRegistry.test.ts to assert Cursor is disabled by default and that probing is skipped under default settings.
• Risk: existing configs that relied on Cursor being enabled by default must now set providers.cursor.enabled = true explicitly.

^{Macroscope summarized d4ab1c7.}

[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: 82cd7bc7-6947-4eca-b74e-a4b1cd2343b6

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

[github-actions]

Thread transfer impact

✅ Thread transfer remains within every enforced ceiling.

Provider	Metric	Main baseline	This PR	Impact	PR ceiling
Codex	Total thread wire	13.3 KiB	13.3 KiB	+9 B (+0.1%)	15.1 KiB	✅
Codex	Thread snapshot wire	6.9 KiB	6.9 KiB	+6 B (+0.1%)	7.3 KiB	✅
Codex	Live turn WebSocket wire	6.4 KiB	6.4 KiB	+3 B (+0.0%)	7.8 KiB	✅
Codex	Live turn WebSocket decoded	55.6 KiB	55.6 KiB	0 B (0.0%)	66.4 KiB	✅
Codex	Live turn messages	10	10	0 (0.0%)	21	✅
Claude	Total thread wire	13.3 KiB	13.3 KiB	+10 B (+0.1%)	15.1 KiB	✅
Claude	Thread snapshot wire	6.9 KiB	6.9 KiB	−1 B (−0.0%)	7.3 KiB	✅
Claude	Live turn WebSocket wire	6.5 KiB	6.5 KiB	+11 B (+0.2%)	7.8 KiB	✅
Claude	Live turn WebSocket decoded	56.4 KiB	56.4 KiB	0 B (0.0%)	66.4 KiB	✅
Claude	Live turn messages	11	11	0 (0.0%)	21	✅

Baseline: afc8342 · PR result: d4ab1c7 · Source CI: success

Scenario and decoded snapshot size

10 historical turns, 5 command tools per turn, 878.9 KiB retained MCP result per historical turn, and a 1.05 MiB retained result in the measured turn.

• Codex decoded thread snapshot: 109.4 KiB
• Claude decoded thread snapshot: 110.1 KiB

Updated in place by a trusted workflow. PR artifacts are strictly validated and never executed.

[macroscopeapp]

Approvability

Verdict: Approved at d4ab1c7

Macroscope's review found this PR approvable — This restores Cursor's documented opt-in default, preventing unnecessary cursor-agent probes for untouched installations while preserving explicit opt-ins. The change is isolated to one setting default with focused regression coverage and no schema or deployment impact.

^{You can add or adjust custom eligibility rules. Learn more.}