test(smoke): require field presence, use floors for catalog counts

Two review findings, both fair:

- `field_equals` treated a missing field as an explicit null, so a response that
  stopped emitting `all_terms_results` would have passed the boolean-query case.
  Presence is checked now: removing the field from the payload fails with
  "all_terms_results is missing from the response, expected null".
- pinning the Milano case at exactly 327 matching datasets broke this file's own
  rule that thresholds survive catalog drift — my own line from yesterday. Counts
  that track the catalog are floors now (`field_min`), while `null` and `0` stay
  equalities: they are behaviours, not counts.

Both defects still caught: removing the boolean skip fails with
"all_terms_results is 10, expected null". 16/16.

Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01MEpSWpAwuMaGkfnMpQdqK2

Author: aborruso

LOG.md

@@ -16,8 +16,12 @@ passed it. `returned_min` was added: a tool can report thousands of matches and
 an empty list, and those are different claims. Both cases now fail on the broken build and
 pass on the fixed one.
 
-Also added `field_equals`, an exact check on any payload field: the strict count had to be
-pinned at 1, at 0 and at null, which is three meanings of one field.
+Also added `field_equals`, an exact check on any payload field, and `field_min`, a floor.
+The split came from review: pinning the Milano case at exactly 327 matching datasets broke
+the file's own rule that thresholds survive catalog drift, so counts that track the catalog
+are floors now, while `null` and `0` stay equalities — they are behaviours, not counts.
+`field_equals` also requires the field to be present, or a response that stopped emitting
+`all_terms_results` would read as an explicit null and pass.
 
 16 cases, 16 passing.
 

scripts/README.md

@@ -115,7 +115,8 @@ Assertions available in a case's `expect`: `total_min` / `total_max` (catalog ma
 `returned_min` (rows actually handed back — not the same claim), `count_min` (rows in a
 listing tool), `first_matches` (regex on the first title), `margin_min` (how far the first
 result leads the second), `terms_equal` (the extracted query terms), `field_equals` (an
-exact value for any payload field, `null` included), `wrapped` / `not_wrapped` /
+exact value for any payload field, `null` included — the field must be present),
+`field_min` (a numeric floor, for values that track the catalog and drift), `wrapped` / `not_wrapped` /
 `effective_contains` (what reached Solr), `error_matches` (the case expects an error).
 
 Cases hit real portals, so the thresholds are loose enough to survive catalog drift and

scripts/smoke.mjs

@@ -153,11 +153,20 @@ function check(expect, payload) {
   // included. Written generic rather than per-field: the cases that needed it wanted
   // `all_terms_results` at 1, at 0 and at null, which is three meanings of one field.
   for (const [field, want] of Object.entries(expect.field_equals ?? {})) {
-    if (JSON.stringify(payload[field] ?? null) !== JSON.stringify(want))
-      fail.push(`${field} is ${JSON.stringify(payload[field] ?? null)}, expected ${JSON.stringify(want)}`);
+    // Presence is part of the check: a field that stopped being emitted must not read
+    // as an explicit null, or the boolean-query case would pass on a response that no
+    // longer carries `all_terms_results` at all.
+    if (!Object.hasOwn(payload, field))
+      fail.push(`${field} is missing from the response, expected ${JSON.stringify(want)}`);
+    else if (JSON.stringify(payload[field]) !== JSON.stringify(want))
+      fail.push(`${field} is ${JSON.stringify(payload[field])}, expected ${JSON.stringify(want)}`);
   }
   if (expect.terms_equal && JSON.stringify(payload.terms ?? null) !== JSON.stringify(expect.terms_equal))
     fail.push(`terms ${JSON.stringify(payload.terms)} differ from ${JSON.stringify(expect.terms_equal)}`);
+  for (const [field, min] of Object.entries(expect.field_min ?? {})) {
+    if (!(typeof payload[field] === "number" && payload[field] >= min))
+      fail.push(`${field} is ${JSON.stringify(payload[field])}, expected a number of at least ${min}`);
+  }
   if (expect.margin_min !== undefined) {
     const [a, b] = (payload.results ?? []).map((r) => r.score ?? 0);
     if (a === undefined || b === undefined || a - b < expect.margin_min)

tests/smoke/cases.json

@@ -136,7 +136,7 @@
           "comune",
           "lecce"
         ],
-        "field_equals": {
+        "field_min": {
           "all_terms_results": 1
         }
       }
@@ -208,10 +208,10 @@
           "aria",
           "milano"
         ],
-        "field_equals": {
-          "all_terms_results": 327
-        },
-        "margin_min": 0
+        "margin_min": 0,
+        "field_min": {
+          "all_terms_results": 163
+        }
       }
     },
     {