playwright-mutation-gate 0.3.1 → 0.3.3

package/README.md

@@ -231,19 +231,22 @@ await page.route('**/*', async (route) => {
 });
 ```
 
+The real preamble also skips binary responses (images, fonts, media) instead of
+decoding them as text, and serves any response with no match untouched.
+
 ### A worked example
 
 A test that reads fine on review but checks the wrong layer:
 
 ```ts
-// The confirmation greeting is server-rendered, then re-rendered on the client
-// from a data-name attribute. This assertion checks the client render.
+// The confirmation greeting is drawn on the client from a name cached at checkout
+// (localStorage), not from this response. So the assertion checks the cache.
 await expect(page.getByTestId('receipt-greeting')).toHaveText('Thanks, Ada!'); // @primary-assert
 ```
 
 Inversion kills it (flip the text and the test goes red), so it clears the
 standard gate. Behavior mutation corrupts `Thanks, Ada!` in the response; the
-client rebuilds it from `data-name`, the assertion stays green:
+client rebuilds it from the cached name, the assertion stays green:
 
 ```text
 tests/receipt.spec.ts

package/dist/behavior.js

@@ -130,16 +130,24 @@ function parseDestructuredFixtures(params) {
 }
 /**
  * Build the route-interception preamble injected at the top of the test body.
- * It fetches each response, replaces every occurrence of the asserted value in
- * the headers and body with a sentinel, and records via PMG_HIT_FILE whether
- * any occurrence was found. `await import('node:fs')` keeps the preamble
- * self-contained, with no top-level import that could clash with the spec.
+ * It fetches each text response, replaces every occurrence of the asserted value
+ * in the headers and body with a sentinel, and records via PMG_HIT_FILE whether
+ * any occurrence was found. Binary resources (images, media, fonts) are passed
+ * straight through: decoding them as text to search for a string would corrupt
+ * them on the way back out, and the asserted value never lives there. A response
+ * with no match is served untouched rather than round-tripped through `text()`.
+ * `await import('node:fs')` keeps the preamble self-contained, with no top-level
+ * import that could clash with the spec.
  */
 function buildPreamble(value) {
     const needle = JSON.stringify(value);
     const replace = JSON.stringify(REPLACE);
     return [
         `  await page.route('**/*', async (route) => {`,
+        `    const resourceType = route.request().resourceType();`,
+        `    if (resourceType === 'image' || resourceType === 'media' || resourceType === 'font') {`,
+        `      return route.continue();`,
+        `    }`,
         `    const resp = await route.fetch();`,
         `    const headers = resp.headers();`,
         `    const NEEDLE = ${needle};`,
@@ -157,7 +165,10 @@ function buildPreamble(value) {
         `      body = body.split(NEEDLE).join(REPLACE);`,
         `      hit = true;`,
         `    }`,
-        `    if (hit && process.env.PMG_HIT_FILE) {`,
+        `    if (!hit) {`,
+        `      return route.fulfill({ response: resp });`,
+        `    }`,
+        `    if (process.env.PMG_HIT_FILE) {`,
         `      const fs = await import('node:fs');`,
         `      fs.appendFileSync(process.env.PMG_HIT_FILE, '1');`,
         `    }`,

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "playwright-mutation-gate",
-  "version": "0.3.1",
+  "version": "0.3.3",
   "description": "Mutation testing for Playwright assertions: invert each test's primary assertion and fail the run if the test stays green.",
   "keywords": [
     "playwright",