-
Notifications
You must be signed in to change notification settings - Fork 10.8k
Sync docs: process readiness (sandbox-sdk#273) #26915
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
|
This pull request requires reviews from CODEOWNERS as it changes files that match the following patterns:
|
2ef6eb7 to
cead379
Compare
cead379 to
b5e15ee
Compare
|
Added comprehensive Processes API reference documentation: ✅ New dedicated API page (
✅ Updated API index to include the new Processes card This complements the existing updates to the Commands API and background-processes guide, providing a complete reference for the process readiness feature from sandbox-sdk#273. |
|
@claude this is too many changes, look at original PR, make changes minimal and clear |
650095e to
ceaa95e
Compare
|
I've updated this PR with corrected documentation. The previous version documented APIs that don't exist in the actual implementation ( The actual API from PR #273 is:
I've now documented these correctly and updated examples throughout:
All examples now use the actual API methods. |
47026f0 to
4252b9b
Compare
46d2f21 to
bb78905
Compare
Co-authored-by: katereznykova <[email protected]>
Summary
Documents the new process readiness detection feature added in cloudflare/sandbox-sdk#273.
This PR adds documentation for two new methods on the
Processobject:process.waitForPort(port, options?)- Wait for a process to start listening on a portprocess.waitForLog(pattern, timeout?)- Wait for a log pattern to appear in process outputChanges
Commands API Reference (
/sandbox/api/commands/)Updated
startProcess()return type documentation to showProcessobject with all available methodsAdded complete reference documentation for
process.waitForPort():Added complete reference documentation for
process.waitForLog():Background Processes Guide (
/sandbox/guides/background-processes/)waitForPort()instead of manual log streamingNew Process Readiness Guide (
/sandbox/guides/process-readiness/)waitForPort()modes (HTTP and TCP)waitForLog()Test plan
Related resources
🤖 Generated with Claude Code