chore: update deps, ensure close related updates are in final step

Author: DylanPiercey

.github/workflows/ci.yml

@@ -35,7 +35,7 @@ jobs:
     strategy:
       fail-fast: false
       matrix:
-        node: [18, 20]
+        node: [22]
     steps:
       - name: Checkout code
         uses: actions/checkout@v4

package-lock.json

@@ -40,35 +40,35 @@
     "test:update": "npm test -- --update"
   },
   "devDependencies": {
-    "@changesets/changelog-github": "^0.5.0",
-    "@changesets/cli": "^2.27.10",
-    "@eslint/js": "^9.15.0",
-    "@types/diff": "^6.0.0",
+    "@changesets/changelog-github": "^0.5.1",
+    "@changesets/cli": "^2.29.5",
+    "@eslint/js": "^9.30.1",
+    "@types/diff": "^8.0.0",
     "@types/diffable-html": "^5.0.2",
     "@types/istanbul-lib-instrument": "^1.7.7",
     "@types/mocha": "^10.0.10",
-    "@types/node": "^22.10.0",
+    "@types/node": "^22.16.0",
     "@types/prettier": "^3.0.0",
     "cross-env": "^7.0.3",
-    "diff": "^7.0.0",
+    "diff": "^8.0.2",
     "diffable-html": "^5.0.0",
-    "esbuild": "^0.24.0",
-    "eslint": "^9.15.0",
+    "esbuild": "^0.25.5",
+    "eslint": "^9.30.1",
     "eslint-formatter-unix": "^8.40.0",
     "eslint-plugin-simple-import-sort": "^12.1.1",
-    "fast-glob": "^3.3.2",
-    "globals": "^15.12.0",
+    "fast-glob": "^3.3.3",
+    "globals": "^16.3.0",
     "husky": "^9.1.7",
     "istanbul-lib-instrument": "^6.0.3",
-    "lint-staged": "^15.2.10",
-    "mocha": "^10.8.2",
+    "lint-staged": "^16.1.2",
+    "mocha": "^11.7.1",
     "mocha-snap": "^5.0.0",
     "nyc": "^17.1.0",
-    "playwright": "^1.49.0",
-    "prettier": "^3.4.1",
-    "prettier-plugin-packagejson": "^2.5.6",
-    "tsx": "^4.19.2",
-    "typescript": "^5.7.2",
-    "typescript-eslint": "^8.16.0"
+    "playwright": "^1.53.2",
+    "prettier": "^3.6.2",
+    "prettier-plugin-packagejson": "^2.5.18",
+    "tsx": "^4.20.3",
+    "typescript": "^5.8.3",
+    "typescript-eslint": "^8.35.1"
   }
 }

package.json

@@ -90,3 +90,28 @@ Embedded App.
 After blocking.
 ```
 
+# Step 1
+
+```html
+Embedded App.
+<script>
+  window.inlineScriptValues = []
+</script>
+<script src="/external.js?value=a">
+</script>
+<script>
+  inlineScriptValues.push(0, scriptValues.at(-1));
+</script>
+<script src="/external.js?value=b">
+</script>
+<script>
+  inlineScriptValues.push(1, scriptValues.at(-1));
+</script>
+<script src="/external.js?value=c">
+</script>
+<script>
+  inlineScriptValues.push(2, scriptValues.at(-1));
+</script>
+After blocking.
+```
+

src/__tests__/__snapshots__/blocking-scripts.expected.md

@@ -108,3 +108,34 @@ Embedded App.
 After blocking.
 ```
 
+# Step 1
+
+```html
+Embedded App.
+<script>
+  window.inlineScriptValues = []
+</script>
+<link
+  rel="stylesheet"
+  href="/external.css?color=rgb(255, 0, 0)"
+>
+<script>
+  inlineScriptValues.push(getComputedStyle(document.body).color);
+</script>
+<link
+  rel="stylesheet"
+  href="/external.css?color=rgb(0, 255, 0)"
+>
+<script>
+  inlineScriptValues.push(getComputedStyle(document.body).color);
+</script>
+<link
+  rel="stylesheet"
+  href="/external.css?color=rgb(0, 0, 255)"
+>
+<script>
+  inlineScriptValues.push(getComputedStyle(document.body).color);
+</script>
+After blocking.
+```
+

src/__tests__/__snapshots__/blocking-styles.expected.md

@@ -80,3 +80,22 @@ Embedded App.
 After blocking.
 ```
 
+# Step 1
+
+```html
+Embedded App.
+<script src="/external.js?value=a">
+</script>
+<script
+  src="/external.js?value=b"
+  crossorigin="use-credentials"
+>
+</script>
+<script
+  src="/external.js?value=c"
+  integrity="sha384-oqVuAfXRKap7fdgcCY5uykM6+R9GqQ8K/uxy9rx7HNQlGYl1kPzQho1wx4JwY8wC"
+>
+</script>
+After blocking.
+```
+

src/__tests__/__snapshots__/crossorigin-and-integrity-preloads.expected.md

@@ -22,3 +22,13 @@ Embedded App.
 After Script.
 ```
 
+# Step 3
+
+```html
+Embedded App.
+<script>
+  scriptValues = ["a", "b"];
+</script>
+After Script.
+```
+

src/__tests__/__snapshots__/inline-scripts.expected.md

@@ -95,3 +95,27 @@ Embedded App.
 After blocking.
 ```
 
+# Step 1
+
+```html
+Embedded App.
+<script src="/external.js?value=a">
+</script>
+<script
+  src="/external.js?value=b"
+  async
+>
+</script>
+<script
+  src="/external.js?value=c"
+  nomodule
+>
+</script>
+<script
+  src="/external.js?value=d"
+  type="module"
+>
+</script>
+After blocking.
+```
+

src/__tests__/__snapshots__/script-preloads.expected.md

@@ -31,14 +31,19 @@ export default (step?: Step[] | Step) => {
 
     let snapshot = "";
     let stepId = 0;
-    for (const step of steps) {
+    for (let i = 0; i < steps.length; i++) {
+      const step = steps[i];
       await waitForPendingRequests(
         page,
         typeof step === "function"
           ? step
           : () => page.evaluate((step) => writer.write(step), step),
       );
 
+      if (i === steps.length - 1) {
+        await page.evaluate(() => writer.close());
+      }
+
       await new Promise((r) => setTimeout(r, 500));
 
       if (changes.length) {
@@ -50,8 +55,6 @@ export default (step?: Step[] | Step) => {
       }
     }
 
-    await page.evaluate(() => writer.close());
-
     assert.ok(
       !changes.length,
       "Should not have any mutations after we've finished writing.",