Skip to content

fix: preserve negated ticket intent in audio routing#7

Open
prasithg wants to merge 1 commit into
mainfrom
hermes/nightly-negated-ticket-audio-2026-07-13
Open

fix: preserve negated ticket intent in audio routing#7
prasithg wants to merge 1 commit into
mainfrom
hermes/nightly-negated-ticket-audio-2026-07-13

Conversation

@prasithg

Copy link
Copy Markdown
Owner

Summary

  • preserve explicit ticket negation before evaluating affirmative acquisition
  • keep later positive purchase clauses behind human approval
  • route no-buy lookup read-only, abandonment to no-op, and quoted family messages back to the normal local confirmation/no-send path
  • add one diversity-reviewed metadata-only audio fixture and refresh UTC release evidence

Audio/data evidence

  • 3 clearly synthetic Daniel TTS clips; 6 local Whisper tiny/base passes
  • before: 0/3 bounded-oracle routes; after: 3/3
  • accepted: 1 cross-family ticket-negation message fixture (diversity score 0.333, accept_review)
  • rejected in Operations: 2 overlapping contrasts (ticket lookup; no-context cancel)
  • repo audio Autodata: 36/36 accepted, 6 held, 1 rejection-ledger row, 27 hard negatives/no-action, 0 unsafe
  • raw audio/local paths committed: 0

Safety

  • no checkout or purchase path
  • family message remains local and confirmation-gated; external_send=false
  • later affirmative ticket purchase still requires human approval
  • no private family data and no clinical claim

Verification

  • backend/.venv/bin/python -m pytest backend/tests/test_textloop.py backend/tests/test_nbest_repair.py backend/tests/test_audio_autodata_evaluator.py backend/tests/test_audio_autodata_promoter.py -q — 82 passed
  • TZ=UTC make eval-audio-autodata — 36/36, 0 unsafe, PASS
  • TZ=UTC make eval-release-readiness — PASS, 4/4 claims, 17 assertions
  • TZ=UTC make test — 631 passed, 2 pre-existing dependency warnings
  • git diff --check — passed

Review note

Local independent Claude review could not run because the CLI was not authenticated. Please review negation clause scoping and the transition from purchase hold to a local confirmation-gated message before merge.

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.

1 participant