Skip to content

fix: enforce OFF full-text provider contract - #18

Merged
maxjustships merged 1 commit into
mainfrom
feat/off-fulltext-contract
Jul 20, 2026
Merged

maxjustships merged 1 commit into
mainfrom
feat/off-fulltext-contract

Conversation

@maxjustships

Copy link
Copy Markdown
Owner

Summary

  • route OFF free-text resolution exclusively through the documented v1 CGI contract; v2 is retained only for product/barcode reachability
  • surface separate OFF product/barcode and full-text readiness in setup and nomnom doctor --json
  • add replayable endpoint, parameter, semantic-query, retry/Retry-After, no-fallback, and confidence guardrail coverage

Validation

  • PYTHONPATH=. pytest -q (126 passed)
  • ruff check .

Closes #17

@maxjustships
maxjustships merged commit d116206 into main Jul 20, 2026
4 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

OFF free-text resolution is broken: v2 ignores search_terms and returns unrelated catalog rows

1 participant