donobu 5.39.1 → 5.40.0

package/dist/esm/reporter/render.js

@@ -64,14 +64,14 @@ function ansiToHtml(str) {
  * Read a few lines of source around `targetLine` (1-based) and return an HTML
  * snippet block, or null if the file is unreadable.
  */
-function readSourceSnippet(file, targetLine) {
+function readSourceSnippet(file, targetLine, wrapperClass = 'native-step-snippet') {
     try {
         const source = (0, fs_1.readFileSync)(file, 'utf8');
         const lines = source.split('\n');
         const context = 2;
         const start = Math.max(0, targetLine - 1 - context);
         const end = Math.min(lines.length - 1, targetLine - 1 + context);
-        let html = '<div class="native-step-snippet">';
+        let html = `<div class="${wrapperClass}">`;
         for (let i = start; i <= end; i++) {
             const lineNum = i + 1;
             const isTarget = lineNum === targetLine;
@@ -1515,6 +1515,26 @@ function renderHtml(report, triage, outputDir) {
         const lastResult = test.results.at(-1);
         const displayFileName = test.file.split('/').pop() ?? test.file;
         let detailsHtml = '';
+        // 0. Skip reason — surface the description and call site from
+        //    test.skip()/test.fixme() so a skipped test explains itself.
+        if (test.status === 'skipped') {
+            const skipAnnotation = test.annotations.find((a) => a.type === 'skip' || a.type === 'fixme');
+            const label = skipAnnotation?.type === 'fixme' ? 'Fixme reason' : 'Skip reason';
+            const description = skipAnnotation?.description?.trim();
+            const body = description
+                ? esc(description)
+                : 'No description was provided when this test was skipped. The reason may be implicit in the surrounding test code or condition.';
+            let locationHtml = '';
+            let snippetHtml = '';
+            const loc = skipAnnotation?.location;
+            if (loc?.file) {
+                const fileBase = (0, path_1.basename)(loc.file);
+                locationHtml = `<div class="skip-location">at <span class="skip-location-ref">${esc(fileBase)}:${loc.line}</span></div>`;
+                snippetHtml =
+                    readSourceSnippet(loc.file, loc.line, 'source-snippet') ?? '';
+            }
+            detailsHtml += `<div class="detail-section"><div class="detail-label">${label}</div><div class="detail-objective">${body}</div>${locationHtml}${snippetHtml}</div>`;
+        }
         // 1. AI analysis — why it failed (the headline)
         if (test.plan) {
             detailsHtml += renderFailureSummary(test.plan);
@@ -1824,6 +1844,11 @@ details.native-step>summary::-webkit-details-marker{display:none}
 details.native-step[open]>summary .native-step-chevron{transform:rotate(90deg)}
 .native-step-error{font-size:11px;font-family:var(--mono);padding:4px 0 2px 44px;margin:0;white-space:pre-wrap;word-break:break-word;color:var(--text-muted)}
 .native-step-snippet{font-size:11px;font-family:var(--mono);margin:4px 0 2px 44px;overflow:hidden}
+.skip-location{font-size:11px;color:var(--text-muted);margin-top:8px;font-family:var(--mono)}
+.skip-location-ref{color:var(--text)}
+.source-snippet{font-size:11px;font-family:var(--mono);margin:6px 0 0 0;overflow:hidden;border:1px solid var(--border-subtle);border-radius:var(--radius);padding:6px 0;background:var(--bg)}
+.source-snippet .snippet-line--target{background:rgba(255,127,58,.10)}
+.source-snippet .snippet-line--target .snippet-linenum{color:var(--orange,#FF7F3A)}
 .native-step-children{display:flex;flex-direction:column;margin:4px 0 0 10px;border-left:1px solid var(--border-subtle);padding-left:8px}
 .native-step-children>.filmstrip-step{padding-left:8px}
 

package/dist/reporter/render.js

@@ -64,14 +64,14 @@ function ansiToHtml(str) {
  * Read a few lines of source around `targetLine` (1-based) and return an HTML
  * snippet block, or null if the file is unreadable.
  */
-function readSourceSnippet(file, targetLine) {
+function readSourceSnippet(file, targetLine, wrapperClass = 'native-step-snippet') {
     try {
         const source = (0, fs_1.readFileSync)(file, 'utf8');
         const lines = source.split('\n');
         const context = 2;
         const start = Math.max(0, targetLine - 1 - context);
         const end = Math.min(lines.length - 1, targetLine - 1 + context);
-        let html = '<div class="native-step-snippet">';
+        let html = `<div class="${wrapperClass}">`;
         for (let i = start; i <= end; i++) {
             const lineNum = i + 1;
             const isTarget = lineNum === targetLine;
@@ -1515,6 +1515,26 @@ function renderHtml(report, triage, outputDir) {
         const lastResult = test.results.at(-1);
         const displayFileName = test.file.split('/').pop() ?? test.file;
         let detailsHtml = '';
+        // 0. Skip reason — surface the description and call site from
+        //    test.skip()/test.fixme() so a skipped test explains itself.
+        if (test.status === 'skipped') {
+            const skipAnnotation = test.annotations.find((a) => a.type === 'skip' || a.type === 'fixme');
+            const label = skipAnnotation?.type === 'fixme' ? 'Fixme reason' : 'Skip reason';
+            const description = skipAnnotation?.description?.trim();
+            const body = description
+                ? esc(description)
+                : 'No description was provided when this test was skipped. The reason may be implicit in the surrounding test code or condition.';
+            let locationHtml = '';
+            let snippetHtml = '';
+            const loc = skipAnnotation?.location;
+            if (loc?.file) {
+                const fileBase = (0, path_1.basename)(loc.file);
+                locationHtml = `<div class="skip-location">at <span class="skip-location-ref">${esc(fileBase)}:${loc.line}</span></div>`;
+                snippetHtml =
+                    readSourceSnippet(loc.file, loc.line, 'source-snippet') ?? '';
+            }
+            detailsHtml += `<div class="detail-section"><div class="detail-label">${label}</div><div class="detail-objective">${body}</div>${locationHtml}${snippetHtml}</div>`;
+        }
         // 1. AI analysis — why it failed (the headline)
         if (test.plan) {
             detailsHtml += renderFailureSummary(test.plan);
@@ -1824,6 +1844,11 @@ details.native-step>summary::-webkit-details-marker{display:none}
 details.native-step[open]>summary .native-step-chevron{transform:rotate(90deg)}
 .native-step-error{font-size:11px;font-family:var(--mono);padding:4px 0 2px 44px;margin:0;white-space:pre-wrap;word-break:break-word;color:var(--text-muted)}
 .native-step-snippet{font-size:11px;font-family:var(--mono);margin:4px 0 2px 44px;overflow:hidden}
+.skip-location{font-size:11px;color:var(--text-muted);margin-top:8px;font-family:var(--mono)}
+.skip-location-ref{color:var(--text)}
+.source-snippet{font-size:11px;font-family:var(--mono);margin:6px 0 0 0;overflow:hidden;border:1px solid var(--border-subtle);border-radius:var(--radius);padding:6px 0;background:var(--bg)}
+.source-snippet .snippet-line--target{background:rgba(255,127,58,.10)}
+.source-snippet .snippet-line--target .snippet-linenum{color:var(--orange,#FF7F3A)}
 .native-step-children{display:flex;flex-direction:column;margin:4px 0 0 10px;border-left:1px solid var(--border-subtle);padding-left:8px}
 .native-step-children>.filmstrip-step{padding-left:8px}
 

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "donobu",
-  "version": "5.39.1",
+  "version": "5.40.0",
   "description": "Create browser automations with an LLM agent and replay them as Playwright scripts.",
   "main": "dist/main.js",
   "module": "dist/esm/main.js",