@piwitests/reporter 0.39.0 → 0.41.0

package/dist/internal/capture/locator-healing.js

@@ -259,8 +259,9 @@ function isAutoGenerated(value) {
   }
   if (/^(emotion-|styled-|css-|sc-)/.test(value)) return true;
   if (value.startsWith("ng-")) return true;
-  if (/^(radix-|headlessui-|mui-|mantine-|chakra-)/i.test(value)) return true;
+  if (/^(radix-|headlessui-|mui-|mantine-|chakra-|reka-|bits-|downshift-|react-aria|el-id-)/i.test(value)) return true;
   if (/^:r[0-9a-z]+:$/i.test(value) || /^«r[0-9a-z]+»$/i.test(value)) return true;
+  if (/(^|[-_:])v-\d+(-\d+)+$/.test(value) || /(^|[-_:])v-\d+(-\d+)+[-_:]/.test(value)) return true;
   return false;
 }
 var TEST_DATA_ATTRS = /* @__PURE__ */ new Set(["data-test", "data-test-id", "data-qa", "data-qa-id", "data-cy", "data-e2e"]);
@@ -477,9 +478,15 @@ function generateAlternatives(attrs) {
 }
 function approximateAccessibleName(attrs) {
   const a = attrs.attributes;
+  const labelText = attrs.labelText || null;
+  if (a["aria-labelledby"] && labelText) return labelText;
   const ariaLabel = a["aria-label"];
   if (ariaLabel) return ariaLabel;
-  if (attrs.textContent) return attrs.textContent;
+  if (["input", "select", "textarea"].includes(attrs.tagName)) {
+    if (labelText) return labelText;
+  } else if (attrs.textContent) {
+    return attrs.textContent;
+  }
   const title = a["title"];
   if (title) return title;
   const placeholder = a["placeholder"];

package/dist/internal/capture/pick-on-failure.js

@@ -472,6 +472,11 @@ function installPickerOverlay(arg) {
     placeLabel(r);
     foot.textContent = "click to pick \xB7 \u2191 parent \xB7 \u2193 child \xB7 Esc skip";
   };
+  const redescribe = () => {
+    labeled = null;
+    if (current()) refresh();
+  };
+  g.__piwiRedescribe = redescribe;
   const stop = (e) => {
     e.preventDefault();
     e.stopImmediatePropagation();
@@ -526,6 +531,7 @@ function installPickerOverlay(arg) {
     highlight.remove();
     label.remove();
     banner.remove();
+    if (g.__piwiRedescribe === redescribe) delete g.__piwiRedescribe;
   };
   const reportPicked = (el) => {
     removePickingListeners();
@@ -1056,8 +1062,9 @@ function isAutoGenerated(value) {
   }
   if (/^(emotion-|styled-|css-|sc-)/.test(value)) return true;
   if (value.startsWith("ng-")) return true;
-  if (/^(radix-|headlessui-|mui-|mantine-|chakra-)/i.test(value)) return true;
+  if (/^(radix-|headlessui-|mui-|mantine-|chakra-|reka-|bits-|downshift-|react-aria|el-id-)/i.test(value)) return true;
   if (/^:r[0-9a-z]+:$/i.test(value) || /^«r[0-9a-z]+»$/i.test(value)) return true;
+  if (/(^|[-_:])v-\d+(-\d+)+$/.test(value) || /(^|[-_:])v-\d+(-\d+)+[-_:]/.test(value)) return true;
   return false;
 }
 var TEST_DATA_ATTRS = /* @__PURE__ */ new Set(["data-test", "data-test-id", "data-qa", "data-qa-id", "data-cy", "data-e2e"]);
@@ -1274,9 +1281,15 @@ function generateAlternatives(attrs) {
 }
 function approximateAccessibleName(attrs) {
   const a = attrs.attributes;
+  const labelText = attrs.labelText || null;
+  if (a["aria-labelledby"] && labelText) return labelText;
   const ariaLabel = a["aria-label"];
   if (ariaLabel) return ariaLabel;
-  if (attrs.textContent) return attrs.textContent;
+  if (["input", "select", "textarea"].includes(attrs.tagName)) {
+    if (labelText) return labelText;
+  } else if (attrs.textContent) {
+    return attrs.textContent;
+  }
   const title = a["title"];
   if (title) return title;
   const placeholder = a["placeholder"];

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@piwitests/reporter",
-  "version": "0.39.0",
+  "version": "0.41.0",
   "description": "Playwright reporter that streams results, traces and HTML reports to a Piwi Dashboard instance",
   "url": "https://github.com/PiwiTests/platform",
   "homepage": "https://piwitests.dev",

package/templates/skills/fix-a-reported-bug/SKILL.md

@@ -0,0 +1,47 @@
+---
+name: fix-a-reported-bug
+description: Fix a bug someone reported from Piwi Picker, starting from its failing test. Use when the user names a Piwi bug report ("bug #37"), asks to "fix the reported bug", "reproduce this bug report", or pastes a link to /bug-reports/<id>.
+---
+
+# Fix a reported bug
+
+A bug report sent from Piwi Picker is already a test: the steps someone took, the assertion that says what the page
+should show, and the value it showed instead. This skill turns it into a committed failing test, reproduces the bug,
+fixes it, and proves the fix with that same test.
+
+## How you reach Piwi
+
+Prefer the **Piwi MCP server** if it is connected: `get_bug_report` (the steps in words, what was expected, the
+evidence, the reproductions, the tests that visit the page) and `render_steps` with `bugReportId` (the failing test).
+Without it, the CLI does the same with the dashboard's URL and an API key: `npx @piwitests/reporter bug <id>`.
+
+## Steps
+
+1. **Read the report.** `get_bug_report { id }`. Note the page, each expected result with the value the page showed,
+   the reporter's note, the failed requests and console errors. The evidence often names the cause (a `POST … 500`).
+
+2. **Write the failing test.** `npx @piwitests/reporter bug <id> --write` writes the spec to the project's bugs folder
+   (`tests/bugs` unless the project names another) and runs it once. It is marked `test.fail()` and annotated
+   `piwi:bug <id>`, so it passes while the bug exists. Exit code 0 means the bug reproduces here. Exit code 1 means it
+   does not: read the output. A step that finds no element means the page or the data differ here (a user, a cart, a
+   flag the report assumed): make the test's setup provide them, in the project's style, rather than deleting steps.
+
+3. **Look at the tests that already visit the page** (`missedBy` in `get_bug_report`). If one of them reaches the
+   element, the missing piece is often one assertion in that test; say so in your report, and keep the bug's own spec
+   as the regression test.
+
+4. **Fix the bug** in the application code. Keep the change to what the evidence and the expected result point at.
+
+5. **Prove the fix.** Remove the `test.fail()` line from the bug's spec and run it: it must pass. Run the tests that
+   visit the page too (`npx playwright test <their files>`), so the fix breaks nothing around it.
+
+6. **Report.** The cause, the fix, the spec file, and the two runs. Once the spec runs in CI without `test.fail()`, Piwi
+   closes the report on its own; before that, a run where the spec still carries `test.fail()` and passes marks it
+   "looks fixed".
+
+## Guardrails
+
+- Never change the expected value in the spec to make it pass: it is what the reporter said the page should show. If
+  you believe the reporter was wrong, stop and ask.
+- Keep `piwi:bug <id>` on the test: it is how the report follows the fix.
+- One bug, one spec. Do not fold the reproduction into an unrelated test.

package/templates/skills/run-the-right-tests/SKILL.md

@@ -9,10 +9,22 @@ Running the whole suite for every change is slow, and hand-maintained grep patte
 
 ## How you reach Piwi
 
-Prefer the **Piwi MCP server** if it is connected (`list_selections`, `resolve_selection`, `preview_selection`). Otherwise use the reporter CLI (`npx @piwitests/reporter select|run`) with `PIWI_DASHBOARD_URL` / `PIWI_API_KEY` / `PIWI_PROJECT_NAME` set, or the dashboard's **Selections** tab.
+Prefer the **Piwi MCP server** if it is connected (`list_selections`, `resolve_selection`, `preview_selection`,
+`predict_locator_breaks`). Otherwise use the reporter CLI (`npx @piwitests/reporter select|run|preflight`) with `PIWI_DASHBOARD_URL` / `PIWI_API_KEY` / `PIWI_PROJECT_NAME` set, or the dashboard's **Selections** tab.
 
 ## Steps
 
+0. **After a UI change, check the locators it breaks first.** Renaming a label, an `aria-label`, a test id, a
+   placeholder or a translation breaks every test that finds the element by it. Run
+   ```
+   npx @piwitests/reporter preflight --fix
+   ```
+   It diffs the working tree against `HEAD`, lists the locators the change breaks with their tests, and rewrites the
+   ones whose call site holds the string (keeping the quotes). Edit the ones it lists as "edit by hand" yourself, then
+   run the tests it names with `npx @piwitests/reporter preflight --run`. Over MCP, `predict_locator_breaks` takes
+   your `git diff` and returns the same breaks and edits. A prediction is not a failure: say which breaks you fixed and
+   which you left, and why.
+
 1. **See what selections exist.** Call `list_selections` for the project. Every project has the built-ins `failed` (tests whose latest run failed) and `quarantine-free`, plus any the team saved (often `smoke`). Pick the one that matches the intent:
    - About to make a broad change, or want a quick confidence check → `smoke` (or `preview_selection` with `{ "include": [{ "tags": ["smoke"] }] }`).
    - Iterating on a fix → `failed`, or a `{ "include": [{ "failedInLastRuns": 5 }] }` preview.