Skip to content

feat: TranslatePsy in the main send-check flow - #29

Closed
salazarsebas wants to merge 1 commit into
feat/qvac-scam-inferencefrom
feat/translate-psy
Closed

salazarsebas wants to merge 1 commit into
feat/qvac-scam-inferencefrom
feat/translate-psy

Conversation

@salazarsebas

Copy link
Copy Markdown
Contributor

Summary

Implements issue #3. Pasted chat goes through TranslateAndAnalyzeMessage before AnalyzeSendIntent, so translation is in the critical path, not a side feature.

  • QvacTranslateAdapter.detectLanguage and translate both call an injected QVAC client (createQvacTranslationClient uses SDK completion() for detect and translate() for the actual translation).
  • Vite cannot load @qvac/sdk. Without a client, detect returns en so the web demo still analyzes the original paste instead of throwing.
  • AnalyzeSendIntent accepts optional textMatches from the translate use case so the model is not run twice on the same text.
  • Runtime export is now @custos/adapters-qvac/runtime (barrel: completion + translation factories).
  • Default translation model is Llama 3.2 1B Instruct via QVAC translate(), which covers the ES/EN/ZH demo. TranslatePsy-AfriSLM is AFRICAN_4B_TRANSLATION_Q4_K_M in the same registry if the demo language set is African.

Stacked on #28 (feat/qvac-scam-inference). Merge that first, then retarget this PR to main.

Also conflicts with #26/#27 on App.tsx (those add network detection and risk UI). Merge order: #28, then this, then rebase #26/#27 or the reverse and resolve App.tsx once.

Test plan

  • pnpm --filter @custos/core test (includes TranslateAndAnalyzeMessage + precomputed matches)
  • pnpm --filter @custos/adapters-qvac test
  • pnpm run build / typecheck / web build
  • On QVAC hardware: wire createQvacTranslationClient, paste Spanish scam chat, confirm on-device translation then detection

Closes #3

@coderabbitai

coderabbitai Bot commented Sep 10, 2026

Copy link
Copy Markdown

Important

Review skipped

Auto reviews are disabled on base/target branches other than the default branch.

Please check the settings in the CodeRabbit UI or the .coderabbit.yaml file in this repository. To trigger a single review, invoke the @coderabbitai review command.

⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Advanced

Run ID: 8bc0416a-7fd5-439c-b9bb-e1661b2e6e7c

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

Use the checkbox below for a quick retry:

  • 🔍 Trigger review

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands.

KevinMB0220 added a commit that referenced this pull request Sep 10, 2026
# Conflicts:
#	apps/web/src/App.tsx
@KevinMB0220

Copy link
Copy Markdown
Contributor

Merged into main together with the rest of the QVAC/App.tsx stack (#27, #28, #31) via integration commits — GitHub couldn't auto-link this one since it doesn't fast-forward cleanly against its original base, but all of this branch's changes (TranslatePsy wiring in the send-check flow) are included in main as of 75a6be8. Closing.

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.

2 participants