tewip-ai 0.70.9 → 0.70.11

package/dist/engine/design/frame.js

@@ -62,12 +62,25 @@ export function impliedLandmark(layerName) {
     return null;
 }
 const PLACEHOLDER = /^(?:lorem ipsum|placeholder|label|text|body|paragraph|xxx+|\.\.\.|…)$/i;
-/** Text worth checking for: real words a person would notice missing. */
+/**
+ * Text worth checking for: real words a person would notice missing - **or a number**.
+ *
+ * This required two consecutive letters, which filters the noise it was written for (a lone glyph,
+ * an ellipsis, "Label") and also silently discarded every number in the design. Field-tested: a
+ * checkout frame specifying **£54.99** against a page serving **£59.99** produced a byte-identical
+ * report - "5 check(s) from the design, 3 met, 2 different" - because the price was never one of the
+ * five. A wrong price on a checkout screen is the drift a design review exists to catch, and it was
+ * the one kind that could not be seen.
+ *
+ * A run of two or more digits now counts, so a price, a total, a quantity or a rate becomes a check.
+ * A single character still does not, nor does punctuation on its own: `3`, `—` and `...` are noise,
+ * `£54.99` and `20%` are the design speaking.
+ */
 function meaningful(text) {
     const t = text.trim();
     if (t.length < 2 || t.length > 120)
         return false;
-    if (!/[A-Za-z]{2}/.test(t))
+    if (!/[A-Za-z]{2}/.test(t) && !/\d{2}/.test(t))
         return false;
     return !PLACEHOLDER.test(t);
 }

package/dist/engine/watch/watch-command.js

@@ -19,6 +19,7 @@ var __rewriteRelativeImportExtension = (this && this.__rewriteRelativeImportExte
 import { existsSync, readFileSync, readdirSync, statSync, watch } from 'node:fs';
 import { join, resolve } from 'node:path';
 import { spawnSync } from 'node:child_process';
+import { detectFrameworks } from '../quality/frameworks.js';
 /** Where test runners in this project put their reports, in the order they are usually found. */
 export const REPORT_PLACES = [
     'blob-report',
@@ -103,6 +104,40 @@ const SKIP_DIR = /(?:^|[\\/])(?:node_modules|\.git|\.tewip|dist|build|target|cov
  * Watches the project and explains failures as they appear. Returns when `once` is set; otherwise
  * it runs until the person stops it, which is what a terminal beside your editor is for.
  */
+/**
+ * What to change so the next run can be explained - named for the runner this project actually uses.
+ *
+ * The advice was a fixed list of three examples: pytest, jest, dotnet. Field-tested on a project
+ * using **`node --test`**, Node's own built-in runner, the suite failed with a perfectly clear
+ * assertion - `26 !== 25`, the test named, the line numbered - and the answer was a list that did
+ * not mention the runner failing in front of it. Telling somebody to configure jest when they are
+ * not using jest is the kind of advice that gets a tool closed.
+ *
+ * The frameworks this project has are already detectable, so the first line is the one command that
+ * will work here, and the examples stay underneath for anyone whose runner was not found.
+ */
+export function howToWriteAReport(root) {
+    const JUNIT = {
+        node: 'node --test --test-reporter=junit --test-reporter-destination=report.xml',
+        jest: 'npx jest --reporters=default --reporters=jest-junit',
+        vitest: 'npx vitest run --reporter=junit --outputFile=report.xml',
+        mocha: 'npx mocha --reporter mocha-junit-reporter',
+        pytest: 'pytest --junitxml=report.xml',
+        go: 'gotestsum --junitfile report.xml',
+        dotnet: 'dotnet test --logger trx',
+        maven: 'mvn test   (Surefire already writes XML to target/surefire-reports)',
+        gradle: './gradlew test   (already writes XML to build/test-results)',
+        playwright: 'npx playwright test --reporter=blob',
+    };
+    const here = detectFrameworks(root)
+        .map((f) => JUNIT[f.id])
+        .filter((line, i, all) => Boolean(line) && all.indexOf(line) === i);
+    const head = 'The tests failed but wrote no report Tewip can read.';
+    if (here.length === 0) {
+        return `${head} Make the runner write JUnit XML (for example: pytest --junitxml=report.xml, jest --reporters=jest-junit, dotnet test --logger trx).`;
+    }
+    return [`${head} For the runner${here.length === 1 ? '' : 's'} this project has, that is:`, ...here.map((line) => `  ${line}`)].join(String.fromCharCode(10));
+}
 export async function runWatch(cmd) {
     const root = resolve(cmd.root);
     const explain = cmd.explain ?? explainReport;
@@ -124,9 +159,7 @@ export async function runWatch(cmd) {
             }
             const found = cmd.report ? { path: resolve(root, cmd.report), at: Date.now() } : newestReport(root);
             if (!found || !existsSync(found.path)) {
-                console.log(cmd.cmd
-                    ? 'The tests failed but wrote no report Tewip can read. Make the runner write JUnit XML (for example: pytest --junitxml=report.xml, jest --reporters=jest-junit, dotnet test --logger trx).'
-                    : 'No test report here yet. Run your tests, or use --cmd "npm test" to let Tewip run them.');
+                console.log(cmd.cmd ? howToWriteAReport(root) : 'No test report here yet. Run your tests, or use --cmd "npm test" to let Tewip run them.');
                 return;
             }
             if (!cmd.cmd && found.at === seen)

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "tewip-ai",
-  "version": "0.70.9",
+  "version": "0.70.11",
   "mcpName": "io.github.hmamut39/tewip",
   "description": "Tewip finds why each failing test failed - real bug, flaky test, environment, test data or changed selector - for any stack, from the command line, CI, or your editor.",
   "keywords": [