Skip to content

Increase default sandbox timeout to 30 minutes - #59

Merged
skrishnan22 merged 1 commit into
mainfrom
chore/default-sandbox-timeout-30m
Sep 5, 2026
Merged

skrishnan22 merged 1 commit into
mainfrom
chore/default-sandbox-timeout-30m

Conversation

@skrishnan22

Copy link
Copy Markdown
Owner

Summary

  • Bump DEFAULT_CONFIG.max_time from 15m to 30m so new sessions get a 30-minute sandbox timeout by default.
  • Applies to CLI config defaults, session directory fallbacks, and orchestrator session creation when no explicit max_time is provided.

Test plan

  • pnpm --filter @codevil/shared test
  • Start a session without --max-time and confirm max_time / max_session_time is 30m

Made with Cursor

Longer-running agent sessions were hitting the 15-minute default before completion; 30m gives more headroom without requiring per-session overrides.

Co-authored-by: Cursor <cursoragent@cursor.com>
@greptile-apps

greptile-apps Bot commented Sep 5, 2026

Copy link
Copy Markdown

Greptile Summary

This PR increases the shared default sandbox session timeout from 15 minutes to 30 minutes.

  • New CLI configurations inherit the 30-minute default.
  • Session-directory and orchestrator fallback paths receive the same value.
  • Explicit timeout settings remain unaffected.

Confidence Score: 5/5

The PR appears safe to merge with no identified correctness, security, or repository-rule issues.

The new duration is accepted by existing parsing and validation, reaches all relevant default session-creation paths, and preserves explicit user overrides.

Important Files Changed

Filename Overview
packages/shared/src/config.ts Updates DEFAULT_CONFIG.max_time to 30m, which is supported and consistently consumed by downstream session-creation paths.

Reviews (1): Last reviewed commit: "Increase default sandbox session timeout..." | Re-trigger Greptile

@skrishnan22
skrishnan22 merged commit 3286b69 into main Sep 5, 2026
4 checks passed
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