The crawlPage viewport option will be passed directly to setViewport if it has a value

Author: coder-hxl

src/api.ts

@@ -432,10 +432,10 @@ async function pageSingleCrawlHandle(
   const { errorPageMap, browser } = extraConfig
 
   const page = await browser.newPage()
-  await page.setViewport({
-    width: detailTarget.viewport?.width ?? 1280,
-    height: detailTarget.viewport?.width ?? 1024
-  })
+
+  if (detailTarget.viewport) {
+    await page.setViewport(detailTarget.viewport)
+  }
 
   let response: HTTPResponse | null = null
   try {

src/types/api.ts

@@ -1,5 +1,5 @@
 import { IncomingHttpHeaders } from 'node:http'
-import { Browser, HTTPResponse, Page, Protocol } from 'puppeteer'
+import { Browser, HTTPResponse, Page, Protocol, Viewport } from 'puppeteer'
 
 import { AnyObject } from './common'
 
@@ -49,10 +49,7 @@ export interface CrawlPageDetailConfig extends CrawlCommonConfig {
   headers?: AnyObject | null
   cookies?: PageCookies | null
   priority?: number
-  viewport?: {
-    width?: number
-    height?: number
-  } | null
+  viewport?: Viewport | null
 }
 
 export interface CrawlDataDetailConfig extends CrawlCommonConfig {
@@ -80,10 +77,7 @@ export interface CrawlPageAdvancedConfig extends CrawlCommonConfig {
 
   headers?: AnyObject
   cookies?: PageCookies
-  viewport?: {
-    width?: number
-    height?: number
-  }
+  viewport?: Viewport
 
   onCrawlItemComplete?: (crawlPageSingleRes: CrawlPageSingleRes) => void
 }