[OrcaWin]

	Files	Added	Deleted	Net
Test	0	0	0	0
Prod	2	$\color{#1a7f37}{\Huge{\mathbf{+}}}$​11	$\color{#cf222e}{\Huge{\mathbf{−}}}$​23	$\color{#cf222e}{\Huge{\mathbf{−}}}$​12

ELI5

A CI lint script that blocks new synchronous program launches crashed on startup, so every PR that merges main fails two required checks. This points the script at the shared library its helper functions moved to, and catches up the list of allowed launches that went stale while the check couldn't run.

What Changed

• config/scripts/check-main-sync-spawn-ratchet.mjs (ci: stop new main-process code from freezing Orca with synchronous launches #27756) imported diffCounts, formatBaseline, parseBaseline and parseBaselineNotes from check-focus-read-ratchet.mjs. chore(ratchet): one ratchet baseline library #27788 had already moved those helpers into ratchet-baseline.mjs, three of them under new names, so the script threw SyntaxError: Export named 'diffCounts' not found and oxlint reported four missing imports.
• The script now imports diffCounts, parseCountBaseline, pruneCountBaseline and totalCount from ratchet-baseline.mjs. Its hand-written --prune and total duplicated pruneCountBaseline and totalCount, so they now call the library, the same way the focus-read and layout-writer ratchets do.
• config/main-sync-spawn-baseline.txt:
  • It adds src/main/daemon/pty-process-ceiling-runtime.ts and src/main/daemon/pty-process-ceiling.ts (1 each). Both run Linux-only systemctl in the terminal daemon process. They landed in feat(daemon): opt-in OS ceiling on terminal processes, reported per host #27769 before ci: stop new main-process code from freezing Orca with synchronous launches #27756's baseline was recorded, and the broken script never got to flag them.
  • It drops src/main/runtime/mobile-notification-dismissal-store.ts, whose launch is gone (--prune).

Why

The import fix alone leaves CI red: once the script runs, it reports the two missing daemon entries and the stale one. Adding the daemon entries records code that was already on main when the ratchet landed, which the baseline header allows for daemon-process files. Moving those calls off runProcessSync belongs in its own change.

Linked Issue

Fixes the main breakage from #27756 + #27788 ("Check main-process sync launches" and "Enforce focused code-quality plugins" failing).

Visual Proof

N/A: CI tooling only.

Testing

• node config/scripts/check-main-sync-spawn-ratchet.mjs: crashed before the change; now passes, 119 uses in 81 files.

• --prune rewrites the baseline with no further change.

• check-main-sync-spawn-ratchet.test.mjs and ratchet-baseline.test.mjs: 63/63 pass.

• oxlint with config/oxlint-code-quality-native-plugins.json --deny-warnings is clean (it reported the 4 missing imports before), as are anti-slop and oxfmt.

• I manually tested these changes locally

• Automated tests added/updated (existing ratchet tests cover the library calls)

Checklist

• This PR is small and focused
• I explained what changed and why
• Before/after screenshots or videos attached for UI changes, or N/A with reason
• Self-reviewed for correctness, security, and performance
• Cross-platform, SSH/remote, and path/shortcut impact considered: CI script and baseline only

[coderabbitai]

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info

⚙️ Run configuration

• Configuration used: Repository UI
• Review profile: CHILL
• Plan: Advanced
• Run ID: 070ffc98-2f9e-4063-8e50-c032ca8e58f3

---

📥 Commits

Reviewing files that changed from the base of the PR and between a940929 and 66f536c.

---

📒 Files selected for processing (2)

• config/main-sync-spawn-baseline.txt
• config/scripts/check-main-sync-spawn-ratchet.mjs

---

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

---

---

📝 Walkthrough

Walkthrough

The sync-spawn baseline adds one launch count for each of two PTY process-ceiling files and removes the count for the mobile notification dismissal store. The checker now uses shared helpers to parse and total baseline counts and to prune the baseline.

---

Priority: ⬇️ Low

Merge Risk: ⚪ Minimal · up to 66f53

No actionable issue remains from the reviewed changes; the PR appears ready to merge subject to normal checks.

Pre-merge checks · 4 · 1

Failed checks (1 warning)

Check name	Status	Explanation	Resolution
Docstring Coverage		Docstring coverage is 0.00% which is insufficient. The required threshold is 80.00%. Docstring coverage is scoped to functions touched by this diff. Analyzed 3 functions across 1 files. (1 skipped: 1 …	Write docstrings for the functions missing them to satisfy the coverage threshold.

Passed checks (4 passed)

Check name	Status	Explanation
Title check		The title clearly identifies the primary change: updating the main sync-spawn ratchet to use the shared ratchet library. It is concise and specific.
Description check		The description is complete and directly explains the problem, changes, rationale, testing, visual-proof exception, and scope. It omits some template sections and does not provide a direct linked-issu…
Linked Issues check		Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check		Check skipped because no linked issues were found for this pull request.

---

Full details: Docstring Coverage

Explanation

Docstring coverage is 0.00% which is insufficient. The required threshold is 80.00%. Docstring coverage is scoped to functions touched by this diff. Analyzed 3 functions across 1 files. (1 skipped: 1 unsupported.)

---

• Fix all pre-merge checks with AI

✨ Finishing Touches 💡 1

📝 Generate docstrings 💡

• Commit to this branch
• Create a new PR

---

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

---

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