Skip to content

Commit eb23b52

Browse files
committed
chore(docs): drop MIN_API_HITS threshold from scraper smoke check
Keep the post-index curl as informational logging, but don't fail the run on api-nr count. Hard-coded thresholds don't auto-adjust as aztec-nr content size changes, and the count is more useful as a debugging signal than a CI gate.
1 parent 38f180b commit eb23b52

1 file changed

Lines changed: 3 additions & 12 deletions

File tree

.github/workflows/docs-typesense.yml

Lines changed: 3 additions & 12 deletions
Original file line numberDiff line numberDiff line change
@@ -34,12 +34,6 @@ jobs:
3434
# regression (which happened with #22861 dropping the index from
3535
# ~12k to 48 records) into a loud CI failure.
3636
MIN_HITS: "5000"
37-
# Fail the run if fewer than this many api-nr records are visible in
38-
# the live index after reindex. #23058 indexed ~12k non-api records
39-
# and silently dropped every aztec-nr-api record (rejected by the
40-
# collection schema) without tripping MIN_HITS, so this guard catches
41-
# the api-nr-specific regression directly.
42-
MIN_API_HITS: "1000"
4337
TYPESENSE_API_KEY: ${{ secrets.TYPESENSE_API_KEY }}
4438
TYPESENSE_HOST: ${{ secrets.TYPESENSE_HOST }}
4539
run: |
@@ -64,11 +58,12 @@ jobs:
6458
exit 1
6559
fi
6660
67-
# Verify api-nr records are searchable in the live index. The
61+
# Log how many api-nr records are visible in the live index. The
6862
# docusaurus theme always prepends `default` to its contextual
6963
# docusaurus_tag filter, and no docusaurus page is stamped with
7064
# `default` (each carries its plugin-context tag instead), so this
7165
# facet count is effectively the count of indexed api-nr records.
66+
# Informational only: the count varies with aztec-nr content size.
7267
api_hits=$(curl -fsS \
7368
"https://$TYPESENSE_HOST/collections/aztec-docs/documents/search" \
7469
-H "X-TYPESENSE-API-KEY: $TYPESENSE_API_KEY" \
@@ -78,8 +73,4 @@ jobs:
7873
--data-urlencode "filter_by=docusaurus_tag:=[default]&&language:=en" \
7974
--data-urlencode "per_page=1" \
8075
| jq -r '.found')
81-
echo "api-nr records visible under docusaurus_tag:=[default]: $api_hits (threshold: $MIN_API_HITS)"
82-
if [ "$api_hits" -lt "$MIN_API_HITS" ]; then
83-
echo "::error::Only $api_hits api-nr records visible (expected at least $MIN_API_HITS). Aztec.nr API search is likely broken."
84-
exit 1
85-
fi
76+
echo "api-nr records visible under docusaurus_tag:=[default]: $api_hits"

0 commit comments

Comments
 (0)