improved language about keepAlive timeout#644
Conversation
- Updated Sandboxes/Processes.mdx Mintlify-Source: dashboard-editor
There was a problem hiding this comment.
Needs attention — 1 issue in 1 file
Documentation content is a clear improvement. The only issue is a missing trailing newline at end of file, which can cause minor git/tooling annoyances.
Prompt for AI agents (all issues)
Check if these issues are valid — if so, understand the root cause of each and fix them.
<assessment>
Documentation content is a clear improvement. The only issue is a missing trailing newline at end of file, which can cause minor git/tooling annoyances.
</assessment>
<file name="Sandboxes/Processes.mdx">
<issue location="Sandboxes/Processes.mdx:420">
File is missing a trailing newline. Most tools (including `git diff`) flag this, and POSIX defines a text file as ending with a newline. Add one to avoid noise in future diffs.
</issue>
</file>
Tag @mendral-app with feedback or questions. View session
| - `failed` | ||
| - `killed` | ||
| - `stopped` | ||
| - `stopped` No newline at end of file |
There was a problem hiding this comment.
maintainability (P3): File is missing a trailing newline. Most tools (including git diff) flag this, and POSIX defines a text file as ending with a newline. Add one to avoid noise in future diffs.
Suggested change
| - `stopped` | |
| - `stopped` | |
Prompt for AI agents
Check if this issue is valid — if so, understand the root cause and fix it. At Sandboxes/Processes.mdx, line 420:
<issue>
File is missing a trailing newline. Most tools (including `git diff`) flag this, and POSIX defines a text file as ending with a newline. Add one to avoid noise in future diffs.
</issue>
🔄 Interaction Flow —
|
| Scenario | Process auto-kill | Sandbox standby |
|---|---|---|
No keepAlive |
Never (runs to completion) | Normal inactivity rules apply |
keepAlive: true, default timeout |
After 600s | Prevented while process runs |
keepAlive: true, timeout: 0 |
Never | Prevented while process runs |
keepAlive: true, timeout: N |
After N seconds | Prevented while process runs |
The key clarification in this PR: timeout without keepAlive only bounds the API wait duration — it does not terminate the process.
Note
Posted by PR Sequence Diagram · Tag @mendral-app with feedback.
🧪 Testing GuideWhat this PR addressesThis PR improves the documentation for Blaxel sandbox process execution, specifically clarifying the semantics of Steps to verify the changesSince this is a documentation-only change to
What to verify (expected behavior)
Note Posted by PR Testing Guide · Tag @mendral-app with feedback. |
|
📋 Created Linear issue ENG-3457 — status: In Progress
Auto-created because no Linear reference was found in the PR title, description, or branch name. Note Posted by Linear Issue Enforcer · Tag @mendral-app with feedback. |
Mintlify
0 threads from 0 users in Mintlify
Note
Documentation improvements to the Sandboxes/Processes page: clarifies
keepAlivevstimeoutsemantics, adds a summary table for their interaction, reformats JSX components for readability, and updates code highlight syntax.Written by Mendral for commit 78e8462.