Skip to content

Retry session create with exp timeout - #1308

Closed
sonic16x wants to merge 3 commits into
masterfrom
exp-session-create
Closed

Retry session create with exp timeout#1308
sonic16x wants to merge 3 commits into
masterfrom
exp-session-create

Conversation

@sonic16x

@sonic16x sonic16x commented Aug 7, 2026

Copy link
Copy Markdown
Contributor
Screenshot 2026-08-07 at 15 02 05

@github-actions

github-actions Bot commented Aug 7, 2026

Copy link
Copy Markdown

✅ Testplane E2E run succeed

Report

@github-actions

github-actions Bot commented Aug 7, 2026

Copy link
Copy Markdown

✅ Testplane browser-env run succeed

Report

@sonic16x
sonic16x force-pushed the exp-session-create branch 3 times, most recently from a3891d9 to 599cda3 Compare August 7, 2026 06:58
@pkg-pr-new

pkg-pr-new Bot commented Aug 7, 2026

Copy link
Copy Markdown

Open in StackBlitz

npm i https://pkg.pr.new/gemini-testing/testplane@1308

commit: 6203503

@sonic16x
sonic16x force-pushed the exp-session-create branch from 599cda3 to 460ea1d Compare August 7, 2026 08:01
@sonic16x
sonic16x requested a review from KuznetsovRoman August 7, 2026 08:13
Comment thread src/browser/new-browser.ts Outdated
import { Config } from "../config";
import { BrowserConfig } from "../config/browser-config";
import { BrowserName, type W3CBrowserName } from "./types";
import { exponentiallyWait } from "../ws-connection/utils";

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Browser should not directly depend on ws-connection utils module

If you want to use "exponentiallyWait" here, its better to pull "exponentiallyWait" to generic utils, which will be used both in ws-connection and browser

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Fixed

@sonic16x sonic16x closed this Aug 10, 2026
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.

2 participants