Skip to content

fix(desktop): raise default warm backend pool cap from 3 to 8 (#103401) - #103435

Closed
kyssta-exe wants to merge 1 commit into
NousResearch:mainfrom
kyssta-exe:auto-fix/issue-103401
Closed

kyssta-exe wants to merge 1 commit into
NousResearch:mainfrom
kyssta-exe:auto-fix/issue-103401

Conversation

@kyssta-exe

Copy link
Copy Markdown

Fixes #103401

Five local profiles with the old 3-backend default saturated the hard spawn coordinator. The fourth profile waited 30s for a free slot while LRU eviction spared keepalive-fresh entries, surfacing Local backend start for "engineer" timed out while waiting for a free slot.

Raise the default maxBackends 3 -> 8 in both authoritative places: apps/desktop/electron/pool-limits.ts and its hand-mirrored renderer copy apps/desktop/src/store/pool-limits.ts. Persisted user values are untouched. At ~60MB per warm backend the new default (~480MB for 8) fits a realistic multi-bot roster without eviction; single-bot users still spawn exactly one. Mirrors the fix for #102822.

Test: vitest run electron/pool-limits.test.ts electron/pool-eviction.test.ts electron/pool-spawn-coordinator.test.ts — 140 passed (4 unrelated missing-electron fails). No other hardcoded defaults remain.

…search#103401)

Five local profiles with the old 3-backend default saturated the hard
spawn coordinator. The fourth profile waited 30s for a free slot while
LRU eviction spared keepalive-fresh entries, surfacing
'Local backend start for "engineer" timed out while waiting for a free slot'.

Raise the default maxBackends 3 -> 8 in both authoritative places:
electron/pool-limits.ts and its hand-mirrored renderer copy
src/store/pool-limits.ts. Persisted user values are untouched.
8 warm backends (~480MB at 60MB each) fits a realistic multi-bot
roster without eviction; single-bot users still spawn exactly one.
Mirrors fix for NousResearch#102822.
@alt-glitch alt-glitch added type/bug Something isn't working P3 Low — cosmetic, nice to have comp/desktop Electron desktop app (apps/desktop/*) duplicate This issue or pull request already exists labels Sep 5, 2026
@alt-glitch

Copy link
Copy Markdown

This was generated by AI during triage.

Duplicate of #102828 — same two-file change (maxBackends 3 -> 8 in electron/pool-limits.ts and src/store/pool-limits.ts), which was opened first and also pins the new default with a test. This PR adds only a doc comment and a reference to #103401. Note that neither PR addresses the foreground-open starvation / relay spawn loop tracked in #102281 and #103375, which a larger cap only delays.

@kyssta-exe

Copy link
Copy Markdown
Author

Closing as duplicate — triage AI alt-glitch identified this as Duplicate of #102828 (same two-file change maxBackends 3 -> 8). The duplicate label is present. Please follow the canonical PR #102828.

@kyssta-exe kyssta-exe closed this Sep 5, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

comp/desktop Electron desktop app (apps/desktop/*) duplicate This issue or pull request already exists P3 Low — cosmetic, nice to have type/bug Something isn't working

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[Bug]: Failed to switch to profile "engineer" Local backend start for "engineer" timed out while waiting for a free slot.

2 participants