test(desktop): expect JetBrains IDEs among remote editors

[juliusmarminge]

Main CI fails on apps/desktop/src/ipc/methods/window.test.ts > "finds remote editors installed without PATH launchers". #17271 gave JetBrains IDEs a remote (jetbrains://gateway) scheme, so probeRemoteEditors now correctly detects an installed WebStorm, but the test still asserted it was excluded.

Flip the assertion to expect WebStorm. vp test run src/ipc/methods/window.test.ts passes (9/9).

Opus 5.5 via Claude Code in T3 Code.

🤖 Generated with Claude Code

---

[macroscopeapp]

Approvability

Verdict: Approved at 75adc10

Macroscope's review found this PR approvable — The PR changes only a single ignored desktop test assertion, so its impact is limited to test expectations and does not modify shipped product behavior. It also introduces no product-default or static-analysis suppression changes.

Notes:

• Diff unchanged. Approvability was decided on eligibility alone.

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

Dismissing prior approval to re-evaluate 75adc10

[coderabbitai]

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info

⚙️ Run configuration

• Configuration used: Path: .coderabbit.config.ts
• Review profile: CHILL
• Plan: Team
• Run ID: 342e4ea8-b362-4d26-98ca-ceb440d2c60e

📥 Commits

Reviewing files that changed from the base of the PR and between df616cc and 75adc10.

📒 Files selected for processing (1)

• apps/desktop/src/ipc/methods/window.test.ts

Included review availability: This review used your included allowance. Your plan provides up to 10 included reviews per hour; 5 remain after this review.

---

📝 Walkthrough

Walkthrough

The remote-editor discovery test now expects probeRemoteEditors to detect WebStorm installed in its application bundle.

Changes

Remote editor discovery

Layer / File(s)	Summary
WebStorm discovery assertion apps/desktop/src/ipc/methods/window.test.ts	The test now expects the discovered editors to include webstorm.

Priority: ⬇️ Low

Estimated code review effort: 1 (Trivial) | ~3 minutes

Change: Other

Merge Risk: ⚪ Minimal · up to 75adc

This test expectation matches the WebStorm discovery behavior represented by its fixture. No actionable merge-blocking risk is evident.

🚥 Pre-merge checks | ✅ 4

✅ Passed checks (4 passed)

Check name	Status	Explanation
Description check	✅ Passed	The description clearly explains the failing assertion, the related behavior change from #17271, the code change, and the focused test result. It does not use the template headings or provide a direct…
Title check	✅ Passed	The title is concise, specific, and accurately describes the test change to expect JetBrains IDEs among remote editors.
Linked Issues check	✅ Passed	Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check	✅ Passed	Check skipped because no linked issues were found for this pull request.

✨ Finishing Touches

📝 Generate docstrings

• Commit to this branch
• Create a new PR

🧪 Generate unit tests (beta)

• Commit to this branch
• Create a new PR

• Autopilot · Keep fixing CodeRabbit findings and required CI, and resolving merge conflicts

---

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

[github-actions]

Thread transfer impact

✅ Thread transfer remains within every enforced ceiling.

ℹ️ The exact PR base did not have a successful artifact. Baseline uses the latest successful main measurement shown below.

Provider	Metric	Main baseline	This PR	Impact	PR ceiling
Codex	Total thread wire	5.0 KiB	5.0 KiB	0 B (0.0%)	6.8 KiB	✅
Codex	Thread snapshot wire	3.8 KiB	3.8 KiB	0 B (0.0%)	4.9 KiB	✅
Codex	Live turn WebSocket wire	1.2 KiB	1.2 KiB	0 B (0.0%)	2.0 KiB	✅
Codex	Live turn WebSocket decoded	20.9 KiB	20.9 KiB	0 B (0.0%)	29.3 KiB	✅
Codex	Live turn messages	2	2	0 (0.0%)	8	✅
Claude	Total thread wire	5.0 KiB	5.0 KiB	+24 B (+0.5%)	6.8 KiB	✅
Claude	Thread snapshot wire	3.8 KiB	3.8 KiB	0 B (0.0%)	4.9 KiB	✅
Claude	Live turn WebSocket wire	1.2 KiB	1.2 KiB	+24 B (+2.0%)	2.0 KiB	✅
Claude	Live turn WebSocket decoded	21.2 KiB	21.2 KiB	+41 B (+0.2%)	29.3 KiB	✅
Claude	Live turn messages	1	2	+1 (+100.0%)	8	✅

Baseline: fd25c42 · PR result: 75adc10 · 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: 108.5 KiB
• Claude decoded thread snapshot: 108.8 KiB

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