@imwz/wp-pattern-sentinel 0.3.0 → 1.0.1

package/README.md

@@ -192,6 +192,18 @@ Commit `.sentinel-cache.json` to track validated state across sessions. Add it t
 
 When any pattern fails, sentinel automatically writes a `sentinel-<timestamp>.log.json` file in the current working directory and prints the path after the summary. This preserves error details for later inspection without needing to re-run. Use `--log` to write the file even on a fully-passing run.
 
+Failing results include a `savedContent` field — the editor's serialized output — so you can diff it directly against the source file:
+
+```bash
+node -e "
+  const log = JSON.parse(require('fs').readFileSync('sentinel-*.log.json'));
+  const r = log.results.find(r => !r.passed);
+  console.log(r.savedContent);
+" | diff - patterns/my-pattern.php
+```
+
+`block_validation` errors also surface Gutenberg's human-readable issue messages (e.g. `"Expected attribute 'class' of value '…' but got '…'"`), so you no longer need to open the browser console to identify what failed.
+
 ## npm publish
 
 When ready to publish:

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@imwz/wp-pattern-sentinel",
-  "version": "0.3.0",
+  "version": "1.0.1",
   "description": "Browser-based WordPress block pattern validator using Playwright",
   "type": "module",
   "engines": {

package/src/main.js

@@ -210,7 +210,11 @@ async function validatePatternFile(patternPath, options, context) {
     if (blockErrors.length > 0) {
       saveResult.errors.push(...blockErrors.map(e => ({
         type:    'block_validation',
-        message: `${e.blockName} (${e.blockId}): ${e.error}`,
+        message: `${e.blockName} (${e.blockId}): ${e.error}${
+          e.validationIssues?.length
+            ? '\n       Issues: ' + e.validationIssues.join('\n       ')
+            : ''
+        }`,
       })));
     }
 
@@ -266,6 +270,7 @@ async function writeLogFile(results) {
         duration: r.duration,
         errors:   r.errors,
         warnings: r.warnings,
+        ...(r.passed ? {} : { savedContent: r.savedContent ?? null }),
       })),
     };
     await fs.promises.writeFile(logPath, JSON.stringify(payload, null, 2));

package/src/validation.js

@@ -8,7 +8,18 @@ export async function checkBlockValidation(page) {
     return await page.evaluate(() => {
       const walk = blocks => blocks.flatMap(block => [
         ...(block.isValid === false
-          ? [{ blockId: block.clientId, blockName: block.name, error: 'Block validation failed' }]
+          ? [{
+              blockId:          block.clientId,
+              blockName:        block.name,
+              error:            'Block validation failed',
+              validationIssues: (block.validationIssues ?? []).map(issue => {
+                try {
+                  return (issue.args ?? [])
+                    .map(a => (typeof a === 'string' ? a : JSON.stringify(a)))
+                    .join(' ');
+                } catch { return 'unknown issue'; }
+              }),
+            }]
           : []),
         ...walk(block.innerBlocks ?? []),
       ]);
@@ -20,6 +31,15 @@ export async function checkBlockValidation(page) {
   }
 }
 
+/**
+ * Strip the `ref` key WordPress injects into wp:navigation block comments when
+ * a pattern is first loaded in the editor. All FSE themes ship navigation
+ * patterns without a ref; WordPress assigns one automatically and it must not
+ * be treated as a content mismatch.
+ */
+const stripNavRef = str =>
+  str.replace(/(<!-- wp:navigation \{)"ref":\d+,\s*/g, '$1');
+
 /**
  * Compare the editor's serialized output against the original source.
  * Whitespace-normalizes both sides before diffing to avoid false positives
@@ -35,12 +55,12 @@ export async function compareContent(page, originalContent) {
     result.savedContent = savedContent;
 
     const normalize = str => str.replace(/\s+/g, ' ').trim();
-    if (normalize(savedContent) === normalize(originalContent)) return result;
+    if (normalize(stripNavRef(savedContent)) === normalize(stripNavRef(originalContent))) return result;
 
     result.matches = false;
 
-    const origLines  = originalContent.split('\n').map(l => l.trim()).filter(Boolean);
-    const savedLines = savedContent.split('\n').map(l => l.trim()).filter(Boolean);
+    const origLines  = stripNavRef(originalContent).split('\n').map(l => l.trim()).filter(Boolean);
+    const savedLines = stripNavRef(savedContent).split('\n').map(l => l.trim()).filter(Boolean);
 
     const removed = origLines.filter(l => !savedLines.includes(l)).slice(0, 5);
     const added   = savedLines.filter(l => !origLines.includes(l)).slice(0, 5);