paginated stdout/stderr in ExecutionResult #670
Merged
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
sid-rl
force-pushed
the
siddarth/python-sdk
branch
from
94c9d22 to
6d3e819
Compare
sid-rl
force-pushed
the
siddarth/python-stdout-stderr
branch
from
b102716 to
eaeffaf
Compare
sid-rl
force-pushed
the
siddarth/python-stdout-stderr
branch
from
eaeffaf to
69be793
Compare
dines-rl
approved these changes
Nov 18, 2025
sid-rl
added a commit
that referenced
this pull request
Nov 19, 2025
* Increase pytest workers from 2 to 10 * Reduce pytest workers from 10 to 5 * initial llm work * added await_suspended methods to base api * added await_completed methods to base api * unit and smoke tests for base api changes * added snapshot_disk methods to devbox classes * big refactor (moved client classes to _sync.py and _async.py, and moved shared methods/types to _helpers.py) * linting, formattting, and type checking changes * lint fixes * circular import fixes * circular import fixes (async) * lint fix * unit tests * unit test refactoring * async client accepts new blueprint create_and_await_build_complete parameters * added file system mounts to devbox creation parameters and cleaned up default parameters (matches base api) * fixed snapshot list, cleaned up getattr * abandon trying to support other upload file types for devbox file uploads (sticks to base api FileType) * smoke tests * fixed tests to expect updated parameter/member names * lint fixes * docs + examples * add missing mount parameters * lint fixes * unpack TypedDict params directly instead of explicitly declaring them * corrected examples * cleaned up underscore prefixes: renamed sync and async modules, and added protocol interface for devbox cmd/file/net * update docs to correct await async devboxes and not use async callbacks (not supported) * added module-level docstrings * formatting changes * fixed unit test imports * fixed expected status for snapshot delete smoketests * Add coverage files to .gitignore and remove .coverage from tracking * increased timeout for snapshot tests * clean up default value for max_retries * remove examples (for now). will upload verified examples later * python sdk manual types (and small fixes/cleanups) * fixed execution result stdout/stderr * unit tests * end to end tests * uv.lock version update * add maintainence comments for manually edited types files * paginated stdout/stderr in ExecutionResult (#670) * added pagination logic to stdout/stderr * unit test adjustments * added smoke tests and todos for fixing and testing output line counting logic * increased smoke test timeouts * scoped docstrings for public modules, classes and methods * formatting fixes * rename ExecutionResult .raw to .result * declare terminal states for devboxes.await_suspended * visual separation of sync vs async devbox protocols, examples for async execution interface * update maintenance instructions for DevboxCreateParams * change manager class suffixes from Client to Ops * remove unnecessary tests from blueprint and snapshot smoke tests * lint fixes * increase timeout for async sdk client smoke tests * updated sdk readme
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Add methods to take advantage of stdout/stderr functionality and prevent truncated output.