feat(settings): warn that keep-awake cannot survive a closed laptop lid

[robbie-c]

Problem

The "Keep awake while agents work" setting uses Electron's powerSaveBlocker("prevent-app-suspension"), which only prevents idle system sleep. Closing a laptop lid forces sleep on macOS (and by default on Windows) — no user-space API can block that. Users reasonably expect the toggle to keep an agent running with the lid closed, and it can't.

Change

• Add hasBuiltInBattery() to the IPowerManager platform interface, implemented in the Electron adapter:
  • macOS: ioreg -rc AppleSmartBattery
  • Windows: Get-CimInstance Win32_Battery (covers Windows laptops, which also sleep on lid close by default)
  • Linux: /sys/class/power_supply/BAT*
  • Result is cached; detection failure falls back to false (no caveat shown).
• Expose it via SleepService and a new sleep.hasBuiltInBattery host-router query.
• On devices with a built-in battery, the setting description now reads: "Prevent your computer from going to sleep on its own while the agent is running a task. Closing the lid will still put it to sleep."

Testing

• pnpm --filter typecheck green for platform, core, host-router, ui, workspace-server, code.
• Core unit tests pass; IPowerManager test fakes updated.
• Verified ioreg -rc AppleSmartBattery detection on a MacBook.

Related: recovery of turns interrupted by sleep is addressed separately.

---

Created with PostHog Code

[trunk-io]

Merging to main in this repository is managed by Trunk.

• To merge this pull request, check the box to the left or comment /trunk merge below.

After your PR is submitted to the merge queue, this comment will be automatically updated with its status. If the PR fails, failure details will also be posted here

[github-actions]

React Doctor found no issues in the changed files. 🎉

_{Reviewed by React Doctor for commit ed0ec9b.}

[greptile-apps]

Comments Outside Diff (1)

1. packages/core/src/sleep/sleep.test.ts, line 37-117 (link)

   Missing test for SleepService.hasBuiltInBattery

   The mock is wired up correctly, but the describe("SleepService") block has no test case that calls service.hasBuiltInBattery() and asserts it delegates to powerManager.hasBuiltInBattery. Given the team's preference for comprehensive tests, a test like it("delegates hasBuiltInBattery to the power manager", ...) would close the gap.

_{Reviews (1): Last reviewed commit: "feat(settings): warn that keep-awake can..." | Re-trigger Greptile}

[greptile-apps]

The base description text is duplicated across both branches of the ternary — the only difference is the extra sentence appended for laptops. Extracting the shared portion satisfies the OnceAndOnlyOnce simplicity rule and makes future copy changes a single-site edit.

Suggested change

	description={
	hasBuiltInBattery
	? "Prevent your computer from going to sleep on its own while the agent is running a task. Closing the lid will still put it to sleep."
	: "Prevent your computer from going to sleep on its own while the agent is running a task"
	}
	description={`Prevent your computer from going to sleep on its own while the agent is running a task.${hasBuiltInBattery ? " Closing the lid will still put it to sleep." : ""}`}

Note: If this suggestion doesn't match your team's coding style, reply to this and let me know. I'll remember it for next time!

[robbie-c]

I prefer it the original way

[stamphog]

Gates denied this PR: it's classified as a feature spanning two areas (platform interface change + UI + host adapter + router), which requires human review. The deny-list also flagged "auth" due to a touched test file. A human reviewer needs to approve before this can be auto-merged.

[stamphog]

Gates denied this PR: it spans two areas (platform interface, host adapter, router, and UI) and triggered the auth deny-list due to a touched test file. Both conditions require a human reviewer to approve before auto-merge.