Review feedback

Author: tunetheweb

src/types.ts

@@ -58,9 +58,11 @@ interface PerformanceEntryMap {
 
 // Update built-in types to be more accurate.
 declare global {
-  // https://wicg.github.io/nav-speculation/prerendering.html#document-prerendering
   interface Document {
-    prerendering?: boolean
+    // https://wicg.github.io/nav-speculation/prerendering.html#document-prerendering
+    prerendering?: boolean;
+    // https://github.com/WICG/page-lifecycle/blob/main/README.md
+    wasDiscarded?: boolean;
   }
 
   interface Performance {

src/types/base.ts

@@ -105,12 +105,3 @@ export interface ReportOpts {
  *   loading. This is equivalent to the corresponding `readyState` value.
  */
 export type LoadState = 'loading' | 'dom-interactive' | 'dom-content-loaded' | 'complete';
-
-/**
- * Extend the document with the new wasDiscarded boolean which is not supported
- * in typescript yet
- * https://github.com/WICG/page-lifecycle/blob/main/README.md
- */ 
-declare global {
-  interface Document { wasDiscarded?: boolean; }
-}

test/views/layout.njk

@@ -105,7 +105,6 @@
        */
       self.__stubWasDiscarded = () => {
         return new Promise((resolve) => {
-          const navEntry = performance.getEntriesByType('navigation')[0];
           // Only stub if the page isn't actually discarded.
           if (!document.wasDiscarded) {
             Object.defineProperty(document, 'wasDiscarded', {