Skip to content

Apply the preparation contract to bulk and mailbox scoring - #1039

Merged
Userunknown84 merged 1 commit into
Userunknown84:mainfrom
pavsoss:feat/1037-bulk-scan-parity
Aug 1, 2026
Merged

Apply the preparation contract to bulk and mailbox scoring#1039
Userunknown84 merged 1 commit into
Userunknown84:mainfrom
pavsoss:feat/1037-bulk-scan-parity

Conversation

@pavsoss

@pavsoss pavsoss commented Aug 1, 2026

Copy link
Copy Markdown
Contributor

Summary

Bulk scoring and mailbox scanning each built their own model input and skipped text preparation entirely, so a classification depended on which entry point a message arrived through. The same text could be labelled one way via the prediction endpoint and another way via a file upload or an inbox scan.

Part of #1037.

Changes

Both paths now prepare text the same way the prediction path does, so identical input produces identical results regardless of entry point. Bulk results still echo each row exactly as uploaded, so a caller can match a result back to its input line even though a canonicalised form was scored.

Test plan

  • Recording doubles capture what actually reaches the vectorizer, confirming bulk rows and scanned messages are scored in prepared form.
  • Obfuscated and plain spellings of the same message are shown to reduce to one scored string.
  • Bulk responses are checked to echo the original row text.

Depends on

Stacked on #1038. Its commit is included here, so this diff shrinks to just the bulk and mailbox changes once #1038 merges.

Bulk rows and scanned inbox messages went to the vectorizer unprepared, so the same message could be classified differently depending on whether it arrived through the prediction endpoint, a file upload or a mailbox scan.
@vercel

vercel Bot commented Aug 1, 2026

Copy link
Copy Markdown

@pavsoss is attempting to deploy a commit to the Aditya Sharma's projects Team on Vercel.

A member of the Team first needs to authorize it.

@github-actions github-actions Bot added the SSoC26 label Aug 1, 2026
@Userunknown84
Userunknown84 merged commit c65dbd6 into Userunknown84:main Aug 1, 2026
4 of 12 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants