fix(gatsby): account for edge case when payload of DELETE_NODE is undefined

packages/gatsby/src/redux/reducers/__tests__/queries.ts

@@ -826,6 +826,13 @@ describe(`create node`, () => {
 })
 
 describe(`delete node`, () => {
+  it(`accounts for undefined payload`, () => {
+    const runDeleteNode = (): void => {
+      reducer(state, { type: `DELETE_NODE`, payload: undefined })
+    }
+    expect(runDeleteNode).not.toThrow(`Cannot read property 'id' of undefined`)
+  })
+
   it.todo(`marks page query as dirty when it has this node as a dependency`)
   it.todo(`marks static query as dirty when it has this node as a dependency`)
   it.todo(

packages/gatsby/src/redux/reducers/queries.ts

@@ -170,6 +170,9 @@ export function queriesReducer(
     case `CREATE_NODE`:
     case `DELETE_NODE`: {
       const node = action.payload
+      if (!node) {
+        return state
+      }
       const queriesByNode = state.byNode.get(node.id) ?? []
       const queriesByConnection =
         state.byConnection.get(node.internal.type) ?? []

packages/gatsby/src/redux/types.ts

@@ -737,7 +737,8 @@ export interface IAddChildNodeToParentNodeAction {
 
 export interface IDeleteNodeAction {
   type: `DELETE_NODE`
-  payload: IGatsbyNode
+  // FIXME: figure out why payload can be undefined here
+  payload: IGatsbyNode | void
 }
 
 export interface IDeleteNodesAction {