Skip to content

Commit d14397e

Browse files
committed
Update default browser viewport size to 900x600
- Modify default viewport size in BrowserSession, system tests, and BrowserSession tests - Adjust viewport size from 1280x800 to 900x600 consistently across files
1 parent 200e9fc commit d14397e

File tree

3 files changed

+8
-6
lines changed

3 files changed

+8
-6
lines changed

src/core/prompts/__tests__/system.test.ts

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -191,7 +191,7 @@ describe("SYSTEM_PROMPT", () => {
191191
true, // supportsComputerUse
192192
undefined, // mcpHub
193193
undefined, // diffStrategy
194-
"1280x800", // browserViewportSize
194+
"900x600", // browserViewportSize
195195
defaultModeSlug, // mode
196196
undefined, // customModePrompts
197197
undefined, // customModes,

src/services/browser/BrowserSession.ts

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -21,7 +21,7 @@ export class BrowserSession {
2121
private currentMousePosition?: string
2222
private sessionTimeout?: NodeJS.Timeout
2323
private readonly SESSION_TIMEOUT = 30 * 60 * 1000 // 30 minutes
24-
private viewportSize: string = "1280x800" // Default viewport size
24+
private viewportSize: string = "900x600" // Default viewport size
2525

2626
constructor(context: vscode.ExtensionContext) {
2727
this.context = context

src/services/browser/__tests__/BrowserSession.test.ts

Lines changed: 6 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -41,6 +41,8 @@ describe("BrowserSession", () => {
4141
mockPage = {
4242
goto: jest.fn().mockResolvedValue(undefined),
4343
setViewport: jest.fn().mockResolvedValue(undefined),
44+
setExtraHTTPHeaders: jest.fn().mockResolvedValue(undefined),
45+
setDefaultNavigationTimeout: jest.fn().mockResolvedValue(undefined),
4446
screenshot: jest.fn().mockResolvedValue(Buffer.from("mock-screenshot-base64")),
4547
mouse: {
4648
click: jest.fn(),
@@ -96,7 +98,7 @@ describe("BrowserSession", () => {
9698
await Promise.all([
9799
mockPuppeteer.launch(),
98100
mockBrowser.newPage(),
99-
mockPage.setViewport({ width: 900, height: 600 } as Viewport),
101+
mockPage.setViewport({ width: 1280, height: 800 } as Viewport),
100102
])
101103

102104
// Verify browser was launched and page was created
@@ -143,8 +145,8 @@ describe("BrowserSession", () => {
143145
expect(mockPage.goto).toHaveBeenCalledWith(
144146
"http://test.com",
145147
expect.objectContaining({
146-
timeout: 7_000,
147-
waitUntil: ["domcontentloaded", "networkidle2"],
148+
timeout: 15_000,
149+
waitUntil: "domcontentloaded",
148150
}),
149151
)
150152
})
@@ -244,7 +246,7 @@ describe("BrowserSession", () => {
244246
})
245247

246248
it("should get current viewport size", () => {
247-
expect(browserSession.getViewportSize()).toBe("1280x800") // Default size
249+
expect(browserSession.getViewportSize()).toBe("900x600") // Default size
248250
})
249251
})
250252

0 commit comments

Comments
 (0)