Skip to content

Add bounded warm Pi provider workers#103

Merged
glittercowboy merged 10 commits intomainfrom
codex/warm-pi-provider-processes-spec
Apr 29, 2026
Merged

Add bounded warm Pi provider workers#103
glittercowboy merged 10 commits intomainfrom
codex/warm-pi-provider-processes-spec

Conversation

@glittercowboy
Copy link
Copy Markdown
Contributor

@glittercowboy glittercowboy commented Apr 29, 2026

Summary

  • Adds warm Pi worker config, identity keys, reusable worker process ownership, and bounded worker eviction.
  • Routes opted-in Pi tasks through warm workers with status snapshots and socket API visibility.
  • Adds warm lifecycle coverage for daemon Pi reuse, Codex AppServer provider reuse, and Claude SDK query pump reuse.

Test Plan

  • go test ./...
  • cd internal/pi/extension && npm test
  • go build -o gsd-cloud .
  • GSD_WARM_PI_WORKERS=1 go test ./tests/e2e -run TestDaemonWarmPiWorkerReusesProcessAcrossTasks -count=1
  • ps -axo pid,ppid,stat,command | rg '[f]ake-pi-warm|[p]i -p --mode rpc'

Merge Order

  • Single-repo daemon change. No upstream dependency.

Post-Merge Actions

  • No tag or daemon release is required by this PR alone.

Summary by CodeRabbit

Release Notes

  • New Features

    • Added warm Pi process reuse: long-lived Pi processes now persist between tasks for improved performance.
    • Added /workers API endpoint to monitor warm worker state and lifecycle.
    • Added configuration for warm worker idle timeout and capacity limits.
    • Added worker eviction policies based on TTL, LRU, and memory pressure.
  • Documentation

    • Added implementation plan and design specification for warm Pi provider process lifecycle management.

@coderabbitai
Copy link
Copy Markdown

coderabbitai Bot commented Apr 29, 2026

Caution

Review failed

Pull request was closed or merged during review

📝 Walkthrough
🚥 Pre-merge checks | ✅ 4 | ❌ 1

❌ Failed checks (1 warning)

Check name Status Explanation Resolution
Docstring Coverage ⚠️ Warning Docstring coverage is 5.77% which is insufficient. The required threshold is 80.00%. Write docstrings for the functions missing them to satisfy the coverage threshold.
✅ Passed checks (4 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title 'Add bounded warm Pi provider workers' directly and clearly describes the main change in the PR, which introduces warm worker pooling with lifecycle management.
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.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing Touches
📝 Generate docstrings
  • Create stacked PR
  • Commit on current branch
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch codex/warm-pi-provider-processes-spec

Review rate limit: 3/10 reviews remaining, refill in 37 minutes and 13 seconds.

Comment @coderabbitai help to get the list of available commands and usage tips.

@glittercowboy glittercowboy merged commit 9d273ce into main Apr 29, 2026
1 of 2 checks passed
@glittercowboy glittercowboy deleted the codex/warm-pi-provider-processes-spec branch April 29, 2026 22:13
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.

1 participant