verifaied 0.23.0.dev46__tar.gz → 0.23.1.dev48__tar.gz

{verifaied-0.23.0.dev46 → verifaied-0.23.1.dev48}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.5
 Name: verifaied
-Version: 0.23.0.dev46
+Version: 0.23.1.dev48
 Summary: Find what's untested in your code — locally, no account required
 Project-URL: Homepage, https://pypi.org/project/verifaied/
 Author: Kyle Richards

{verifaied-0.23.0.dev46 → verifaied-0.23.1.dev48}/pyproject.toml

@@ -1,6 +1,6 @@
 [project]
 name = "verifaied"
-version = "0.23.0.dev46"
+version = "0.23.1.dev48"
 description = "Find what's untested in your code — locally, no account required"
 readme = "README.md"
 requires-python = ">=3.10"

{verifaied-0.23.0.dev46 → verifaied-0.23.1.dev48}/src/verifaied/recorder_driver.js

@@ -169,6 +169,7 @@ const {
   compactSnapshot,
   shouldAutoRead,
   shortUrl,
+  isOwnRequest,
   noteProblem,
   problemsSince,
   AUTO_USER_QUIET_MS,
@@ -443,9 +444,13 @@ async function main() {
     return lines;
   };
   page.on("console", (message) => {
-    if (message.type() === "error") {
-      noteProblem(problems, `console error: ${message.text()}`);
-    }
+    if (message.type() !== "error") return;
+    // A refused request also arrives here as a resource-load error, which
+    // names the URL in its location and the verb nowhere — the coverage
+    // reporter's own post again, see `isOwnRequest`.
+    const where = message.location() || {};
+    if (isOwnRequest(where.url, undefined)) return;
+    noteProblem(problems, `console error: ${message.text()}`);
   });
   page.on("pageerror", (error) => {
     noteProblem(
@@ -458,6 +463,8 @@ async function main() {
     // A cancelled navigation or a request the page abandoned is not the
     // application failing at anything.
     if (failure === "net::ERR_ABORTED") return;
+    // The coverage reporter's own post — see `isOwnRequest`.
+    if (isOwnRequest(request.url(), request.method())) return;
     noteProblem(
       problems,
       `request failed: ${request.method()} ${shortUrl(request.url())} (${failure})`,
@@ -467,6 +474,9 @@ async function main() {
     const kind = response.request().resourceType();
     if (kind !== "document" && kind !== "xhr" && kind !== "fetch") return;
     if (response.status() < 400) return;
+    // The coverage reporter's own post — see `isOwnRequest`.
+    const request = response.request();
+    if (isOwnRequest(request.url(), request.method())) return;
     noteProblem(
       problems,
       `${response.request().method()} ${shortUrl(response.url())} → ${response.status()}`,
@@ -658,6 +668,9 @@ async function main() {
     const onRequest = (request) => {
       const kind = request.resourceType();
       if (kind === "eventsource" || kind === "websocket") return;
+      // The coverage reporter's own post — see `isOwnRequest`. It fires
+      // on a timer, so an action must never be held waiting for one.
+      if (isOwnRequest(request.url(), request.method())) return;
       inflight.set(request, Date.now());
     };
     const onDone = (request) => {

{verifaied-0.23.0.dev46 → verifaied-0.23.1.dev48}/src/verifaied/recorder_refs.js

@@ -254,6 +254,30 @@ function shortUrl(url) {
   }
 }
 
+/**
+ * Whether a request is the coverage reporter's own post — verifAIed's
+ * furniture, not the application's. The reporter every instrumented app
+ * carries POSTs `window.__coverage__` to `http://127.0.0.1:<port>/coverage`
+ * on a timer, fire-and-forget, and a recording starts no collector, so
+ * during one every post is refused. That is not the application failing
+ * at anything, and reporting it would have the assistant chase a
+ * "backend issue" on every step.
+ *
+ * The method may be unknown, and then it is simply not asked about: the
+ * same refused post also arrives as a `Failed to load resource` console
+ * error, which names the URL in its location and the verb nowhere.
+ */
+function isOwnRequest(url, method) {
+  const verb = String(method == null ? "" : method);
+  if (verb !== "" && verb.toUpperCase() !== "POST") return false;
+  try {
+    const parsed = new URL(String(url));
+    return parsed.hostname === "127.0.0.1" && parsed.pathname === "/coverage";
+  } catch {
+    return false;
+  }
+}
+
 /**
  * Record one problem. A repeat of the last one collapses into a count —
  * a render loop fires the same error hundreds of times and would
@@ -305,6 +329,7 @@ module.exports = {
   compactSnapshot,
   shouldAutoRead,
   shortUrl,
+  isOwnRequest,
   noteProblem,
   problemsSince,
   AUTO_USER_QUIET_MS,

{verifaied-0.23.0.dev46 → verifaied-0.23.1.dev48}/tests/test_testcmd.py

@@ -12454,6 +12454,18 @@ def test_the_refs_module_finds_compares_and_rewrites():
         refs.shortUrl("http://localhost:3000/api/save?x=1#frag"),
         refs.shortUrl("not a url"),
       ],
+      ownRequest: [
+        refs.isOwnRequest("http://127.0.0.1:4571/coverage", "POST"),
+        refs.isOwnRequest("http://127.0.0.1:4620/coverage", "post"),
+        refs.isOwnRequest("http://localhost:3001/coverage", "POST"),
+        refs.isOwnRequest("http://127.0.0.1:4571/coverage", "GET"),
+        refs.isOwnRequest("http://127.0.0.1:4571/api/coverage", "POST"),
+        refs.isOwnRequest("not a url", "POST"),
+        refs.isOwnRequest("http://127.0.0.1:4571/coverage", undefined),
+        refs.isOwnRequest("http://localhost:3001/coverage", undefined),
+        refs.isOwnRequest("http://127.0.0.1:4571/api/coverage", undefined),
+        refs.isOwnRequest("not a url", undefined),
+      ],
       compacted: refs.compactSnapshot([
         "- generic [active] [ref=e1]:",
         "  - generic [ref=e2] [cursor=pointer]:",
@@ -12529,6 +12541,25 @@ def test_the_refs_module_finds_compares_and_rewrites():
     assert problems["empty"] == []
     assert problems["ringSize"] == 250
     assert answer["shortUrl"] == ["/api/save?x=1", "not a url"]
+    # Only the reporter's exact shape is verifAIed's furniture: any port
+    # on 127.0.0.1, path /coverage, POST in any case. A `/coverage` route
+    # on localhost is the application's own host and must stay reported,
+    # as must a GET, another path, and anything that is not a URL. The
+    # last four are the console listener's question, where the verb is
+    # unknown: an unsupplied method is not asked about, and the URL alone
+    # still has to be the reporter's.
+    assert answer["ownRequest"] == [
+        True,
+        True,
+        False,
+        False,
+        False,
+        False,
+        True,
+        False,
+        False,
+        False,
+    ]
     # Layout boxes go and their children move up a level; the active
     # root, a clickable container, and every leaf with text stay.
     assert answer["compacted"].split("\n") == [
@@ -13062,6 +13093,23 @@ def test_the_driver_watches_the_page_rather_than_driving_the_recorder():
     assert 'page.ariaSnapshot({ mode: "ai" })' in driver
 
 
+def test_the_driver_never_reports_the_coverage_reporter_as_a_problem():
+    """The reporter an instrumented dev server injects posts coverage to a
+    collector a recording never starts, so every post is refused. Four
+    call sites keep that out of the assistant's view: the three problem
+    listeners — `requestfailed`, a >=400 `response` (so a collector
+    answering 5xx is not billed to the application either) and `console`,
+    since Chromium reports every refused request as a resource-load error
+    naming the URL in its location and the verb nowhere — and
+    `watchRequests`, so a timer's post never holds an action's settle."""
+    driver = _code_only(DRIVER_PATH.read_text())
+    assert driver.count("isOwnRequest(request.url(), request.method())") == 3
+    assert "isOwnRequest(where.url, undefined)" in driver
+    # And it comes from the refs module, beside the rest of the pure half.
+    destructure = driver.split('require(path.join(__dirname, "recorder_refs.js"))')[0]
+    assert "isOwnRequest," in destructure
+
+
 def test_the_driver_ships_beside_every_module_it_requires():
     """All four files live in the same Python package, so the wheel that
     carries one carries the rest."""

{verifaied-0.23.0.dev46 → verifaied-0.23.1.dev48}/uv.lock

@@ -442,7 +442,7 @@ wheels = [
 
 [[package]]
 name = "verifaied"
-version = "0.23.0"
+version = "0.23.1"
 source = { editable = "." }
 dependencies = [
     { name = "httpx" },