@artymclabin/qa-review 0.3.10 → 0.3.12

package/dist/client/QAReviewOverlay.js

@@ -882,8 +882,8 @@ export function QAReviewOverlay({ items, target, gateParam, stateUrl, submitUrl,
                         }) })), _jsxs("div", { className: "qar-finish-actions", children: [submitUrl && (_jsxs(_Fragment, { children: [_jsxs("button", { type: "button", onClick: saveToDb, disabled: save.status === "saving" || save.status === "ok", className: "qar-btn-primary", "data-qatip": "Store a snapshot of this whole run (verdicts are already saved individually)", children: [_jsx(Database, { size: 16, "aria-hidden": true }), save.status === "saving"
                                                 ? "Saving…"
                                                 : save.status === "ok"
-                                                    ? "Saved to database ✓"
-                                                    : "Save review to database"] }), save.status === "error" && _jsx("p", { className: "qar-warn", children: save.message })] })), _jsxs("button", { type: "button", onClick: copyResults, className: "qar-btn-outline-accent", "data-qatip": "Copy all verdicts of this page as JSON", children: [_jsx(ClipboardCopy, { size: 16, "aria-hidden": true }), copied ? "Copied!" : "Copy JSON"] }), roundTotal > 0 && (_jsx("button", { type: "button", onClick: () => {
+                                                    ? "Snapshot saved ✓"
+                                                    : "Save session snapshot (optional)"] }), save.status !== "ok" && (_jsx("p", { className: "qar-finish-autosaved", style: { marginTop: 4 }, children: "Your verdicts above are already saved - this only logs a snapshot of the whole run." })), save.status === "error" && _jsx("p", { className: "qar-warn", children: save.message })] })), _jsxs("button", { type: "button", onClick: copyResults, className: "qar-btn-outline-accent", "data-qatip": "Copy all verdicts of this page as JSON", children: [_jsx(ClipboardCopy, { size: 16, "aria-hidden": true }), copied ? "Copied!" : "Copy JSON"] }), roundTotal > 0 && (_jsx("button", { type: "button", onClick: () => {
                                     setFinished(false);
                                     // 🚨 THE LAST ITEM, NOT THE FIRST (Arty 2026-08-09: "Clicking
                                     // on 'Back to review' in the QA interactive panel should go to

package/dist/client/fingerprint.d.ts

@@ -5,9 +5,24 @@ export declare function normalizeText(text: string): string;
 export declare function fingerprintText(text: string): string;
 /**
  * Fingerprint the CURRENT content of an item: anchored items hash the
- * element's rendered innerText; off-DOM task items hash their question text
- * (sub, falling back to title). Returns null when the anchor is not present
- * (e.g. other-viewport element) - no judgement is made then.
+ * element's textContent; off-DOM task items hash their question text (sub,
+ * falling back to title). Returns null when the anchor is not present (e.g.
+ * other-viewport element) - no judgement is made then.
+ *
+ * 🚨 textContent, NEVER innerText (Arty 2026-09-24, GameReady fundraise FAQ).
+ * innerText is spec'd to approximate what a user currently SEES, so it
+ * excludes anything not laid out - which includes the body of a closed
+ * native `<details>`. Every accordion-style FAQ defaults to collapsed, so an
+ * anchor on the answer paragraph inside it fingerprinted ONLY the always-
+ * visible summary/question text forever, regardless of how the answer was
+ * edited. "NOT ALTERED since your rejection" then showed on an item that
+ * had, in fact, been fixed the same day it was rejected - the exact
+ * regression this poka-yoke exists to prevent, silently defeated by the one
+ * DOM shape (collapsed accordion) FAQ content overwhelmingly uses. textContent
+ * does not care about layout/visibility, which is what "did the words
+ * change" actually needs - this was invisible in tests because jsdom does
+ * not implement `<details>` collapse or innerText's visibility semantics at
+ * all, so the bug could only ever surface against a real rendered page.
  */
 export declare function itemFingerprint(item: QAReviewItem, doc: Document): string | null;
 export type FingerprintStatus = "unchanged" | "changed";

package/dist/client/fingerprint.js

@@ -16,9 +16,24 @@ export function fingerprintText(text) {
 }
 /**
  * Fingerprint the CURRENT content of an item: anchored items hash the
- * element's rendered innerText; off-DOM task items hash their question text
- * (sub, falling back to title). Returns null when the anchor is not present
- * (e.g. other-viewport element) - no judgement is made then.
+ * element's textContent; off-DOM task items hash their question text (sub,
+ * falling back to title). Returns null when the anchor is not present (e.g.
+ * other-viewport element) - no judgement is made then.
+ *
+ * 🚨 textContent, NEVER innerText (Arty 2026-09-24, GameReady fundraise FAQ).
+ * innerText is spec'd to approximate what a user currently SEES, so it
+ * excludes anything not laid out - which includes the body of a closed
+ * native `<details>`. Every accordion-style FAQ defaults to collapsed, so an
+ * anchor on the answer paragraph inside it fingerprinted ONLY the always-
+ * visible summary/question text forever, regardless of how the answer was
+ * edited. "NOT ALTERED since your rejection" then showed on an item that
+ * had, in fact, been fixed the same day it was rejected - the exact
+ * regression this poka-yoke exists to prevent, silently defeated by the one
+ * DOM shape (collapsed accordion) FAQ content overwhelmingly uses. textContent
+ * does not care about layout/visibility, which is what "did the words
+ * change" actually needs - this was invisible in tests because jsdom does
+ * not implement `<details>` collapse or innerText's visibility semantics at
+ * all, so the bug could only ever surface against a real rendered page.
  */
 export function itemFingerprint(item, doc) {
     if (!item.selector)
@@ -26,7 +41,7 @@ export function itemFingerprint(item, doc) {
     const el = doc.querySelector(item.selector);
     if (!el)
         return null;
-    return fingerprintText(el.innerText ?? el.textContent ?? "");
+    return fingerprintText(el.textContent ?? "");
 }
 /**
  * Compare the fingerprint STORED with the last verdict against the current

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@artymclabin/qa-review",
-  "version": "0.3.10",
+  "version": "0.3.12",
   "description": "Interactive on-page QA review overlay (spotlight walkthrough, approve/reject verdict ledger) with a framework-agnostic server handler factory and a self-provisioning Postgres store.",
   "license": "MIT",
   "repository": {