fix: simulateTransaction accounts items can be null (#23229)

* fix: simulated accounts can be null

* Use Missing rather than token program id

Co-authored-by: Arrowana <8245419+Arrowana@users.noreply.github.com>

web3.js/src/connection.ts

@@ -469,7 +469,7 @@ export type SimulatedTransactionAccountInfo = {
 export type SimulatedTransactionResponse = {
   err: TransactionError | string | null;
   logs: Array<string> | null;
-  accounts?: SimulatedTransactionAccountInfo[] | null;
+  accounts?: (SimulatedTransactionAccountInfo | null)[] | null;
   unitsConsumed?: number;
 };
 
@@ -480,13 +480,15 @@ const SimulatedTransactionResponseStruct = jsonRpcResultAndContext(
     accounts: optional(
       nullable(
         array(
-          pick({
+          nullable(
-            executable: boolean(),
+            pick({
-            owner: string(),
+              executable: boolean(),
-            lamports: number(),
+              owner: string(),
-            data: array(string()),
+              lamports: number(),
-            rentEpoch: optional(number()),
+              data: array(string()),
-          }),
+              rentEpoch: optional(number()),
+            }),
+          ),
         ),
       ),
     ),

web3.js/test/connection.test.ts

@@ -3044,10 +3044,16 @@ describe('Connection', () => {
       const results2 = await connection.simulateTransaction(
         message,
         [account1],
-        [account1.publicKey],
+        [
+          account1.publicKey,
+          new PublicKey('Missing111111111111111111111111111111111111'),
+        ],
       );
 
-      expect(results2.value.accounts).lengthOf(1);
+      expect(results2.value.accounts).lengthOf(2);
+      if (results2.value.accounts) {
+        expect(results2.value.accounts[1]).to.be.null;
+      }
     }).timeout(10000);
 
     it('transaction', async () => {