Skip to content

Use Selenium’s default session request timeout - #55

Merged
raghubetina merged 1 commit into
mainfrom
codex/selenium-default-session-timeout
Sep 24, 2026
Merged

raghubetina merged 1 commit into
mainfrom
codex/selenium-default-session-timeout

Conversation

@raghubetina

@raghubetina raghubetina commented Sep 24, 2026 •

Copy link
Copy Markdown
Contributor

Remove Drawing Board’s 30-second Selenium session-request override to follow the approved upstream-default behavior. The server queue default is 300 seconds; generated applications retain a separate 60-second Ruby HTTP read timeout. No warm-up scripts or retries are added.

script/check, Compose validation and final-head hosted contract CI pass. Claude Opus 5.5 independently reviewed this companion and Core #119; the timeout explanation is corrected. The observed Codespaces failure and remaining provider qualification stay under Service #729. New root-compiled apps require the separate Service Core adoption.

The 30-second cutoff rejected a Codespaces browser that started after
31.7 seconds. Let Selenium's own deadline accommodate slow starts
instead of adding warm-up or retry machinery.

Provider qualification remains tracked in firstdraft/firstdraft#729.
@raghubetina
raghubetina force-pushed the codex/selenium-default-session-timeout branch from 83ffe99 to 7be76f3 Compare September 24, 2026 15:52
@raghubetina
raghubetina marked this pull request as ready for review September 24, 2026 16:06
@raghubetina
raghubetina merged commit 6044f36 into main Sep 24, 2026
1 check 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