@jsenv/core 27.0.0-alpha.93 → 27.0.0

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@jsenv/core",
-  "version": "27.0.0-alpha.93",
+  "version": "27.0.0",
   "description": "Tool to develop, test and build js projects",
   "license": "MIT",
   "author": {
@@ -61,7 +61,7 @@
   },
   "dependencies": {
     "@babel/plugin-proposal-dynamic-import": "7.16.7",
-    "@babel/plugin-transform-modules-systemjs": "7.18.5",
+    "@babel/plugin-transform-modules-systemjs": "7.18.6",
     "@babel/plugin-transform-modules-umd": "7.18.0",
     "@c88/v8-coverage": "0.1.1",
     "@financial-times/polyfill-useragent-normaliser": "2.0.1",
@@ -71,13 +71,13 @@
     "@jsenv/importmap": "1.2.1",
     "@jsenv/integrity": "0.0.1",
     "@jsenv/node-esm-resolution": "0.1.0",
-    "@jsenv/server": "12.7.3",
+    "@jsenv/server": "12.7.4",
     "@jsenv/uneval": "1.6.0",
     "@jsenv/url-meta": "7.0.0",
     "@jsenv/urls": "1.2.6",
     "@jsenv/utils": "2.0.1",
     "@jsenv/babel-plugins": "1.0.5",
-    "@jsenv/log": "2.1.0",
+    "@jsenv/log": "3.0.2",
     "@jsenv/sourcemap": "1.0.1",
     "acorn-import-assertions": "1.8.0",
     "cuid": "2.1.8",
@@ -99,15 +99,15 @@
     "@babel/plugin-syntax-import-assertions": "7.17.12",
     "@jsenv/assert": "2.6.0",
     "@jsenv/eslint-config": "16.0.9",
-    "@jsenv/file-size-impact": "13.0.0",
+    "@jsenv/file-size-impact": "13.0.1",
     "@jsenv/https-local": "2.1.0",
-    "@jsenv/package-workspace": "0.3.0",
-    "@jsenv/performance-impact": "3.0.0",
-    "eslint": "8.18.0",
+    "@jsenv/package-workspace": "0.4.0",
+    "@jsenv/performance-impact": "3.0.1",
+    "eslint": "8.19.0",
     "eslint-plugin-html": "6.2.0",
     "eslint-plugin-import": "2.26.0",
     "eslint-plugin-react": "7.30.1",
-    "playwright": "1.22.2",
+    "playwright": "1.23.1",
     "prettier": "2.7.1"
   }
-}
+}

package/src/build/build.js

@@ -988,7 +988,7 @@ const applyUrlVersioning = async ({
       // - ignored files:
       //   we don't know their content
       // - unused files without reference
-      //   File updated such as style.css -> style.css.js or file.js->file.es5.js
+      //   File updated such as style.css -> style.css.js or file.js->file.nomodule.js
       //   Are used at some point just to be discarded later because they need to be converted
       //   There is no need to version them and we could not because the file have been ignored
       //   so their content is unknown

package/src/build/start_build_server.js

@@ -56,8 +56,8 @@ export const startBuildServer = async ({
     "./package.json": true,
     "./jsenv.config.mjs": true,
   },
+  buildServerAutoreload = false,
   buildServerMainFile = getCallerPosition().url,
-  buildServerAutoreload = true,
   cooldownBetweenFileEvents,
 }) => {
   const logger = createLogger({ logLevel })
@@ -96,54 +96,56 @@ export const startBuildServer = async ({
     port = await findFreePort(port, { signal: operation.signal })
   }
 
-  const reloadableWorker = createReloadableWorker(buildServerMainFile)
-  if (buildServerAutoreload && reloadableWorker.isPrimary) {
-    const buildServerFileChangeCallback = ({ relativeUrl, event }) => {
-      const url = new URL(relativeUrl, rootDirectoryUrl).href
-      if (buildServerAutoreload) {
+  let reloadableWorker
+  if (buildServerAutoreload) {
+    reloadableWorker = createReloadableWorker(buildServerMainFile)
+    if (reloadableWorker.isPrimary) {
+      const buildServerFileChangeCallback = ({ relativeUrl, event }) => {
+        const url = new URL(relativeUrl, rootDirectoryUrl).href
         logger.info(`file ${event} ${url} -> restarting server...`)
         reloadableWorker.reload()
       }
-    }
-    const stopWatchingBuildServerFiles = registerDirectoryLifecycle(
-      rootDirectoryUrl,
-      {
-        watchPatterns: {
-          [buildServerMainFile]: true,
-          ...buildServerFiles,
-        },
-        cooldownBetweenFileEvents,
-        keepProcessAlive: false,
-        recursive: true,
-        added: ({ relativeUrl }) => {
-          buildServerFileChangeCallback({ relativeUrl, event: "added" })
-        },
-        updated: ({ relativeUrl }) => {
-          buildServerFileChangeCallback({ relativeUrl, event: "modified" })
-        },
-        removed: ({ relativeUrl }) => {
-          buildServerFileChangeCallback({ relativeUrl, event: "removed" })
+      const stopWatchingBuildServerFiles = registerDirectoryLifecycle(
+        rootDirectoryUrl,
+        {
+          watchPatterns: {
+            ...buildServerFiles,
+            [buildServerMainFile]: true,
+            ".jsenv/": false,
+          },
+          cooldownBetweenFileEvents,
+          keepProcessAlive: false,
+          recursive: true,
+          added: ({ relativeUrl }) => {
+            buildServerFileChangeCallback({ relativeUrl, event: "added" })
+          },
+          updated: ({ relativeUrl }) => {
+            buildServerFileChangeCallback({ relativeUrl, event: "modified" })
+          },
+          removed: ({ relativeUrl }) => {
+            buildServerFileChangeCallback({ relativeUrl, event: "removed" })
+          },
         },
-      },
-    )
-    operation.addAbortCallback(() => {
-      stopWatchingBuildServerFiles()
-      reloadableWorker.terminate()
-    })
-    const worker = await reloadableWorker.load()
-    const messagePromise = new Promise((resolve) => {
-      worker.once("message", resolve)
-    })
-    await messagePromise
-    // if (!keepProcessAlive) {
-    //   worker.unref()
-    // }
-    return {
-      origin: `${protocol}://127.0.0.1:${port}`,
-      stop: () => {
+      )
+      operation.addAbortCallback(() => {
         stopWatchingBuildServerFiles()
         reloadableWorker.terminate()
-      },
+      })
+      const worker = await reloadableWorker.load()
+      const messagePromise = new Promise((resolve) => {
+        worker.once("message", resolve)
+      })
+      await messagePromise
+      // if (!keepProcessAlive) {
+      //   worker.unref()
+      // }
+      return {
+        origin: `${protocol}://127.0.0.1:${port}`,
+        stop: () => {
+          stopWatchingBuildServerFiles()
+          reloadableWorker.terminate()
+        },
+      }
     }
   }
 
@@ -195,7 +197,7 @@ export const startBuildServer = async ({
     logger.info(`- ${server.origins[key]}`)
   })
   logger.info(``)
-  if (reloadableWorker.isWorker) {
+  if (reloadableWorker && reloadableWorker.isWorker) {
     parentPort.postMessage(server.origin)
   }
   return {

package/src/dev/plugins/toolbar/client/jsenv_logo.svg

@@ -4,53 +4,51 @@
   viewBox="0 0 430 430"
   enable-background="new 0 0 430 430"
 >
-  <g>
-    <g>
-      <path
-        id="j-letter"
-        fill="#F7931E"
-        d="M78.879,272.229c3.381-0.441,7.056-1.176,9.408-3.822c2.793-2.939,3.234-8.086,3.234-19.111v-63.064
+  <g id="letters">
+    <path
+      id="j"
+      fill="#F7931E"
+      d="M78.879,272.229c3.381-0.441,7.056-1.176,9.408-3.822c2.793-2.939,3.234-8.086,3.234-19.111v-63.064
           h19.404v66.74c0,16.023-3.381,23.52-9.555,28.371c-5.292,4.264-14.112,6.174-21.021,6.174L78.879,272.229z M111.072,167.121
           c0,6.174-4.41,10.438-10.143,10.438c-5.292,0-9.702-4.263-9.555-10.438c-0.147-6.468,4.41-10.437,9.849-10.437
           S111.072,160.653,111.072,167.121z"
-      />
-      <path
-        id="s-letter"
-        fill="#F7931E"
-        d="M123.714,240.77c2.793,1.764,8.085,3.822,12.936,3.822c4.998,0,7.791-2.205,7.791-6.469
+    />
+    <path
+      id="s"
+      fill="#F7931E"
+      d="M123.714,240.77c2.793,1.764,8.085,3.822,12.936,3.822c4.998,0,7.791-2.205,7.791-6.469
           c0-3.675-2.058-6.027-8.379-9.702c-11.76-6.615-14.553-14.112-14.553-20.727c0-12.937,9.261-22.639,23.667-22.639
           c6.027,0,10.731,1.47,14.553,3.234l-3.381,14.112c-2.793-1.617-6.174-2.94-9.996-2.94c-4.851,0-7.203,2.94-7.203,6.174
           c0,3.234,1.617,4.998,8.379,8.967c10.584,6.027,14.553,13.524,14.553,21.756c0,14.699-10.584,22.639-25.432,22.639
           c-6.027,0-13.083-1.617-16.464-4.117L123.714,240.77z"
-      />
-      <path
-        id="e-letter"
-        fill="#F7931E"
-        d="M204.414,223.57c0.148,21.756,8.674,26.461,16.906,26.461c4.852,0,8.967-1.176,11.76-2.793l1.764,8.379
+    />
+    <path
+      id="e"
+      fill="#F7931E"
+      d="M204.414,223.57c0.148,21.756,8.674,26.461,16.906,26.461c4.852,0,8.967-1.176,11.76-2.793l1.764,8.379
           c-3.969,2.205-9.996,3.381-15.582,3.381c-17.052,0-25.872-13.965-25.872-35.722c0-23.08,9.702-37.486,23.961-37.486
           c14.553,0,20.285,15.289,20.285,31.459c0,2.646,0,4.557-0.146,6.321H204.414z M226.76,215.191
           c0.293-14.259-5.145-20.581-10.584-20.581c-7.352,0-11.174,10.878-11.613,20.581H226.76z"
-      />
-      <path
-        id="n-letter"
-        fill="#F7931E"
-        d="M249.836,205.342c0-8.379-0.441-12.643-0.588-18.375h9.85l0.588,8.526h0.293
+    />
+    <path
+      id="n"
+      fill="#F7931E"
+      d="M249.836,205.342c0-8.379-0.441-12.643-0.588-18.375h9.85l0.588,8.526h0.293
           c3.088-5.733,8.82-9.702,16.318-9.702c9.85,0,17.199,7.35,17.199,24.55v47.775h-11.318V212.25c0-8.379-1.617-16.905-9.85-16.905
           c-4.705,0-9.262,3.969-10.732,11.613c-0.293,1.764-0.439,3.969-0.439,6.321v44.835h-11.32V205.342z"
-      />
-      <path
-        id="v-letter"
-        fill="#F7931E"
-        d="M313.783,186.966l8.967,39.985c1.029,4.851,2.205,11.172,3.088,17.346h0.441
+    />
+    <path
+      id="v"
+      fill="#F7931E"
+      d="M313.783,186.966l8.967,39.985c1.029,4.851,2.205,11.172,3.088,17.346h0.441
           c0.734-5.879,1.91-11.907,2.939-17.346l8.82-39.985h11.172l-18.523,71.149h-10.143l-18.521-71.149H313.783z"
-      />
-    </g>
-    <g>
-      <g>
-        <path
-          id="arc1"
-          fill="#F7931E"
-          d="M32.118,204.979c0,0,0.105,1.801,0.302,5.179c0.124,3.4,0.638,8.29,1.434,14.48
+    />
+  </g>
+  <g id="arcs">
+    <path
+      id="arc1"
+      fill="#F7931E"
+      d="M32.118,204.979c0,0,0.105,1.801,0.302,5.179c0.124,3.4,0.638,8.29,1.434,14.48
             c0.249,1.546,0.511,3.174,0.786,4.882c0.137,0.853,0.278,1.725,0.421,2.617c0.154,0.89,0.376,1.789,0.558,2.712
             c0.384,1.843,0.783,3.761,1.197,5.75c0.444,1.981,1.027,4.008,1.56,6.112c2.274,8.378,5.328,17.708,9.758,27.396
             c4.406,9.688,10.04,19.776,17.062,29.673c7.003,9.901,15.393,19.609,25.102,28.499c9.702,8.888,20.652,17.025,32.581,23.85
@@ -74,13 +72,11 @@
             c-12.467-7.743-23.632-16.999-33.41-26.861c-9.758-9.893-18.135-20.455-24.833-31.222c-6.743-10.739-11.988-21.536-15.97-31.794
             c-4.006-10.258-6.601-20.021-8.441-28.721c-1.917-8.698-2.688-16.429-3.245-22.764c-0.396-6.374-0.584-11.386-0.485-14.752
             c0.026-3.383,0.04-5.188,0.04-5.188c0.003-0.47,0.387-0.848,0.857-0.844C31.725,204.181,32.091,204.536,32.118,204.979z"
-        />
-      </g>
-      <g>
-        <path
-          id="arc2"
-          fill="#F7931E"
-          d="M393.625,225.021c0,0-0.104-1.731-0.297-4.979c-0.049-0.812-0.103-1.718-0.162-2.716
+    />
+    <path
+      id="arc2"
+      fill="#F7931E"
+      d="M393.625,225.021c0,0-0.104-1.731-0.297-4.979c-0.049-0.812-0.103-1.718-0.162-2.716
             c-0.029-0.499-0.061-1.021-0.093-1.565c-0.059-0.538-0.119-1.097-0.182-1.679c-0.269-2.323-0.534-4.99-0.906-7.966
             c-0.241-1.486-0.496-3.051-0.763-4.693c-0.133-0.82-0.269-1.658-0.408-2.516c-0.126-0.859-0.286-1.731-0.487-2.616
             c-0.778-3.54-1.458-7.393-2.614-11.413c-0.266-1.009-0.537-2.034-0.812-3.075c-0.264-1.043-0.553-2.096-0.908-3.147
@@ -119,26 +115,26 @@
             c0.173,3.065,0.257,5.808,0.365,8.183c0.022,0.594,0.045,1.165,0.066,1.714c-0.006,0.545-0.01,1.068-0.015,1.567
             c-0.009,1-0.017,1.908-0.024,2.721c-0.029,3.252-0.045,4.987-0.045,4.987c-0.004,0.47-0.389,0.847-0.857,0.843
             C394.018,225.817,393.652,225.463,393.625,225.021z"
-        />
-      </g>
-      <circle
-        id="circle2"
-        fill="#F7931E"
-        stroke-width="9"
-        stroke-miterlimit="10"
-        cx="70.536"
-        cy="110.694"
-        r="36.011"
-      />
-      <circle
-        id="circle1"
-        fill="#F7931E"
-        stroke-width="9"
-        stroke-miterlimit="10"
-        cx="362.721"
-        cy="309.694"
-        r="36.012"
-      />
-    </g>
+    />
+  </g>
+  <g id="circles">
+    <circle
+      id="circle2"
+      fill="#F7931E"
+      stroke-width="9"
+      stroke-miterlimit="10"
+      cx="70.536"
+      cy="110.694"
+      r="36.011"
+    />
+    <circle
+      id="circle1"
+      fill="#F7931E"
+      stroke-width="9"
+      stroke-miterlimit="10"
+      cx="362.721"
+      cy="309.694"
+      r="36.012"
+    />
   </g>
-</svg>
+</svg>

package/src/dev/start_dev_server.js

@@ -31,19 +31,21 @@ export const startDevServer = async ({
   certificate,
   privateKey,
   keepProcessAlive = true,
+  serverPlugins,
+
   rootDirectoryUrl,
+  clientFiles = {
+    "./src/": true,
+    "./tests/": true,
+  },
   devServerFiles = {
     "./package.json": true,
     "./jsenv.config.mjs": true,
   },
+  clientAutoreload = true,
+  devServerAutoreload = false,
   devServerMainFile = getCallerPosition().url,
-  devServerAutoreload = true,
-  clientFiles = {
-    "./src/": true,
-    "./tests/": true,
-  },
   cooldownBetweenFileEvents,
-  clientAutoreload = true,
 
   sourcemaps = "inline",
   // default runtimeCompat assume dev server will be request by recent browsers
@@ -92,53 +94,57 @@ export const startDevServer = async ({
     port = await findFreePort(port, { signal: operation.signal })
   }
 
-  const reloadableWorker = createReloadableWorker(devServerMainFile)
-  if (devServerAutoreload && reloadableWorker.isPrimary) {
-    const devServerFileChangeCallback = ({ relativeUrl, event }) => {
-      const url = new URL(relativeUrl, rootDirectoryUrl).href
-      logger.info(`file ${event} ${url} -> restarting server...`)
-      reloadableWorker.reload()
-    }
-    const stopWatchingDevServerFiles = registerDirectoryLifecycle(
-      rootDirectoryUrl,
-      {
-        watchPatterns: {
-          [devServerMainFile]: true,
-          ...devServerFiles,
-        },
-        cooldownBetweenFileEvents,
-        keepProcessAlive: false,
-        recursive: true,
-        added: ({ relativeUrl }) => {
-          devServerFileChangeCallback({ relativeUrl, event: "added" })
-        },
-        updated: ({ relativeUrl }) => {
-          devServerFileChangeCallback({ relativeUrl, event: "modified" })
-        },
-        removed: ({ relativeUrl }) => {
-          devServerFileChangeCallback({ relativeUrl, event: "removed" })
+  let reloadableWorker
+  if (devServerAutoreload) {
+    reloadableWorker = createReloadableWorker(devServerMainFile)
+    if (reloadableWorker.isPrimary) {
+      const devServerFileChangeCallback = ({ relativeUrl, event }) => {
+        const url = new URL(relativeUrl, rootDirectoryUrl).href
+        logger.info(`file ${event} ${url} -> restarting server...`)
+        reloadableWorker.reload()
+      }
+      const stopWatchingDevServerFiles = registerDirectoryLifecycle(
+        rootDirectoryUrl,
+        {
+          watchPatterns: {
+            ...devServerFiles.include,
+            [devServerMainFile]: true,
+            ".jsenv/": false,
+          },
+          cooldownBetweenFileEvents,
+          keepProcessAlive: false,
+          recursive: true,
+          added: ({ relativeUrl }) => {
+            devServerFileChangeCallback({ relativeUrl, event: "added" })
+          },
+          updated: ({ relativeUrl }) => {
+            devServerFileChangeCallback({ relativeUrl, event: "modified" })
+          },
+          removed: ({ relativeUrl }) => {
+            devServerFileChangeCallback({ relativeUrl, event: "removed" })
+          },
         },
-      },
-    )
-    operation.addAbortCallback(() => {
-      stopWatchingDevServerFiles()
-      reloadableWorker.terminate()
-    })
-
-    const worker = await reloadableWorker.load()
-    const messagePromise = new Promise((resolve) => {
-      worker.once("message", resolve)
-    })
-    await messagePromise
-    // if (!keepProcessAlive) {
-    //   worker.unref()
-    // }
-    return {
-      origin: `${protocol}://127.0.0.1:${port}`,
-      stop: () => {
+      )
+      operation.addAbortCallback(() => {
         stopWatchingDevServerFiles()
         reloadableWorker.terminate()
-      },
+      })
+
+      const worker = await reloadableWorker.load()
+      const messagePromise = new Promise((resolve) => {
+        worker.once("message", resolve)
+      })
+      await messagePromise
+      // if (!keepProcessAlive) {
+      //   worker.unref()
+      // }
+      return {
+        origin: `${protocol}://127.0.0.1:${port}`,
+        stop: () => {
+          stopWatchingDevServerFiles()
+          reloadableWorker.terminate()
+        },
+      }
     }
   }
 
@@ -155,7 +161,10 @@ export const startDevServer = async ({
     })
   }
   const stopWatchingClientFiles = registerDirectoryLifecycle(rootDirectoryUrl, {
-    watchPatterns: clientFiles,
+    watchPatterns: {
+      ...clientFiles,
+      ".jsenv/": false,
+    },
     cooldownBetweenFileEvents,
     keepProcessAlive: false,
     recursive: true,
@@ -218,6 +227,7 @@ export const startDevServer = async ({
     urlGraph,
     kitchen,
     scenario: "dev",
+    serverPlugins,
   })
   startDevServerTask.done()
   logger.info(``)
@@ -230,7 +240,7 @@ export const startDevServer = async ({
       kitchen.pluginController.callHooks("destroy", {})
     }
   })
-  if (reloadableWorker.isWorker) {
+  if (reloadableWorker && reloadableWorker.isWorker) {
     parentPort.postMessage(server.origin)
   }
   return {

package/src/omega/kitchen.js

@@ -64,6 +64,7 @@ export const createKitchen = ({
       return RUNTIME_COMPAT.isSupported(runtimeCompat, feature)
     },
   }
+  pluginController.callHooks("init", kitchenContext)
   const createReference = ({
     data = {},
     node,

package/src/omega/server/file_service.js

@@ -61,7 +61,13 @@ export const createFileService = ({
     const urlInfo = urlGraph.reuseOrCreateUrlInfo(reference.url)
 
     const ifNoneMatch = request.headers["if-none-match"]
-    if (ifNoneMatch && urlInfo.contentEtag === ifNoneMatch) {
+    if (
+      ifNoneMatch &&
+      urlInfo.contentEtag === ifNoneMatch &&
+      // - isValid is true by default
+      // - isValid can be overriden by plugins such as cjs_to_esm
+      urlInfo.isValid()
+    ) {
       return {
         status: 304,
         headers: {
@@ -83,6 +89,7 @@ export const createFileService = ({
         urlInfo.originalContent = null
         urlInfo.type = null
         urlInfo.subtype = null
+        urlInfo.dependsOnPackageJson = false
         urlInfo.timing = {}
       }
       const { runtimeName, runtimeVersion } = parseUserAgentHeader(

package/src/omega/url_graph.js

@@ -121,33 +121,48 @@ export const createUrlGraph = ({
   }
 
   if (clientFileChangeCallbackList) {
-    const updateModifiedTimestamp = (urlInfo, modifiedTimestamp) => {
-      const seen = []
-      const iterate = (urlInfo) => {
-        if (seen.includes(urlInfo.url)) {
-          return
-        }
-        seen.push(urlInfo.url)
-        urlInfo.modifiedTimestamp = modifiedTimestamp
-        urlInfo.dependents.forEach((dependentUrl) => {
-          const dependentUrlInfo = urlInfos[dependentUrl]
-          const { hotAcceptDependencies = [] } = dependentUrlInfo.data
-          if (!hotAcceptDependencies.includes(urlInfo.url)) {
-            iterate(dependentUrlInfo)
-          }
-        })
-      }
-      iterate(urlInfo)
-    }
     clientFileChangeCallbackList.push(({ url }) => {
       const urlInfo = urlInfos[url]
       if (urlInfo) {
-        updateModifiedTimestamp(urlInfo, Date.now())
-        urlInfo.contentEtag = null
+        considerModified(urlInfo, Date.now())
       }
     })
   }
 
+  const considerModified = (urlInfo, modifiedTimestamp = Date.now()) => {
+    const seen = []
+    const iterate = (urlInfo) => {
+      if (seen.includes(urlInfo.url)) {
+        return
+      }
+      seen.push(urlInfo.url)
+      urlInfo.modifiedTimestamp = modifiedTimestamp
+      urlInfo.contentEtag = undefined
+      urlInfo.dependents.forEach((dependentUrl) => {
+        const dependentUrlInfo = urlInfos[dependentUrl]
+        const { hotAcceptDependencies = [] } = dependentUrlInfo.data
+        if (!hotAcceptDependencies.includes(urlInfo.url)) {
+          iterate(dependentUrlInfo)
+        }
+      })
+    }
+    iterate(urlInfo)
+  }
+
+  const getRelatedUrlInfos = (url) => {
+    const urlInfosUntilNotInline = []
+    const parentUrlInfo = getUrlInfo(url)
+    if (parentUrlInfo) {
+      urlInfosUntilNotInline.push(parentUrlInfo)
+      if (parentUrlInfo.inlineUrlSite) {
+        urlInfosUntilNotInline.push(
+          ...getRelatedUrlInfos(parentUrlInfo.inlineUrlSite.url),
+        )
+      }
+    }
+    return urlInfosUntilNotInline
+  }
+
   return {
     urlInfos,
     reuseOrCreateUrlInfo,
@@ -156,6 +171,8 @@ export const createUrlGraph = ({
     inferReference,
     findDependent,
     updateReferences,
+    considerModified,
+    getRelatedUrlInfos,
 
     toJSON: (rootDirectoryUrl) => {
       const data = {}
@@ -176,6 +193,9 @@ export const createUrlGraph = ({
 const createUrlInfo = (url) => {
   return {
     modifiedTimestamp: 0,
+    contentEtag: null,
+    dependsOnPackageJson: false,
+    isValid,
     data: {}, // plugins can put whatever they want here
     references: [],
     dependencies: new Set(),
@@ -192,10 +212,12 @@ const createUrlInfo = (url) => {
     shouldHandle: undefined,
     originalContent: undefined,
     content: undefined,
-    contentEtag: null,
+
     sourcemap: null,
     sourcemapReference: null,
     timing: {},
     headers: {},
   }
 }
+
+const isValid = () => true