Skip to content

feat: Add a settle_timeout_seconds option to wait for Simulator background daemons - #595

Merged
dotdoom merged 1 commit into
futureware-tech:mainfrom
mykola-mokhnach:settle-timeout-seconds
Aug 16, 2026
Merged

feat: Add a settle_timeout_seconds option to wait for Simulator background daemons#595
dotdoom merged 1 commit into
futureware-tech:mainfrom
mykola-mokhnach:settle-timeout-seconds

Conversation

@mykola-mokhnach

Copy link
Copy Markdown
Contributor

Downstream CI found that simctl bootstatus only confirms SpringBoard is reachable, while launchd_sim keeps spawning background daemons for another 30-90s, starving real work of CPU. This adds an opt-in poll on launchd_sim's direct children's aggregate %CPU, bounded by a mandatory positive timeout so it can never hang a job. Unset (default) or 0 preserves current behavior exactly, with nothing logged.

@mykola-mokhnach mykola-mokhnach changed the title Add a settle_timeout_seconds option to wait for Simulator background daemons feat: Add a settle_timeout_seconds option to wait for Simulator background daemons Aug 15, 2026
…round daemons

Downstream CI found that simctl bootstatus only confirms SpringBoard is
reachable, while launchd_sim keeps spawning background daemons for another
30-90s, starving real work of CPU. This adds an opt-in poll on launchd_sim's
direct children's aggregate %CPU, bounded by a mandatory positive timeout so
it can never hang a job. Unset (default) or 0 preserves current behavior
exactly, with nothing logged.

Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>

@dotdoom dotdoom left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thank you!

@dotdoom
dotdoom merged commit 61c31f6 into futureware-tech:main Aug 16, 2026
2 checks passed
@dotdoom

dotdoom commented Aug 16, 2026

Copy link
Copy Markdown
Member

FTR verified dist built correctly

% nix shell nixpkgs#nodejs_22
% rm -rf dist
% npm run package

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants