chore: make localStorage optional when setting storage state

docs/src/api/class-apirequest.md

@@ -61,7 +61,7 @@ Methods like [`method: APIRequestContext.get`] take the base URL into considerat
     - `sameSite` <[SameSiteAttribute]<"Strict"|"Lax"|"None">>
   - `origins` <[Array]<[Object]>>
     - `origin` <[string]>
-    - `localStorage` <[Array]<[Object]>>
+    - `localStorage` ?<[Array]<[Object]>>
       - `name` <[string]>
       - `value` <[string]>
     - `indexedDB` ?<[Array]<[Object]>> indexedDB to set for context

docs/src/api/params.md

@@ -261,7 +261,7 @@ Specify environment variables that will be visible to the browser. Defaults to `
     - `sameSite` <[SameSiteAttribute]<"Strict"|"Lax"|"None">> sameSite flag
   - `origins` <[Array]<[Object]>>
     - `origin` <[string]>
-    - `localStorage` <[Array]<[Object]>> localStorage to set for context
+    - `localStorage` ?<[Array]<[Object]>> localStorage to set for context
       - `name` <[string]>
       - `value` <[string]>
     - `indexedDB` ?<[Array]<[Object]>> indexedDB to set for context

packages/playwright-core/src/protocol/validator.ts

@@ -165,7 +165,7 @@ scheme.IndexedDBDatabase = tObject({
 });
 scheme.SetOriginStorage = tObject({
   origin: tString,
-  localStorage: tArray(tType('NameValue')),
+  localStorage: tOptional(tArray(tType('NameValue'))),
   indexedDB: tOptional(tArray(tType('IndexedDBDatabase'))),
 });
 scheme.OriginStorage = tObject({

packages/playwright-core/types/types.d.ts

@@ -10103,7 +10103,7 @@ export interface Browser {
         /**
          * localStorage to set for context
          */
-        localStorage: Array<{
+        localStorage?: Array<{
           name: string;
 
           value: string;
@@ -17681,7 +17681,7 @@ export interface APIRequest {
       origins: Array<{
         origin: string;
 
-        localStorage: Array<{
+        localStorage?: Array<{
           name: string;
 
           value: string;
@@ -22410,7 +22410,7 @@ export interface BrowserContextOptions {
       /**
        * localStorage to set for context
        */
-      localStorage: Array<{
+      localStorage?: Array<{
         name: string;
 
         value: string;

packages/protocol/src/channels.d.ts

@@ -295,7 +295,7 @@ export type IndexedDBDatabase = {
 
 export type SetOriginStorage = {
   origin: string,
-  localStorage: NameValue[],
+  localStorage?: NameValue[],
   indexedDB?: IndexedDBDatabase[],
 };
 

packages/protocol/src/protocol.yml

@@ -263,7 +263,7 @@ SetOriginStorage:
   properties:
     origin: string
     localStorage:
-      type: array
+      type: array?
       items: NameValue
     indexedDB:
       type: array?