Repository navigation
feat: TranslatePsy in the main send-check flow - #29
salazarsebas wants to merge 1 commit into
Conversation
|
Important Review skippedAuto reviews are disabled on base/target branches other than the default branch. Please check the settings in the CodeRabbit UI or the ⚙️ Run configurationConfiguration used: defaults Review profile: CHILL Plan: Advanced Run ID: You can disable this status message by setting the Use the checkbox below for a quick retry:
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. Comment |
# Conflicts: # apps/web/src/App.tsx
|
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. |
Summary
Implements issue #3. Pasted chat goes through
TranslateAndAnalyzeMessagebeforeAnalyzeSendIntent, so translation is in the critical path, not a side feature.QvacTranslateAdapter.detectLanguageandtranslateboth call an injected QVAC client (createQvacTranslationClientuses SDKcompletion()for detect andtranslate()for the actual translation).@qvac/sdk. Without a client, detect returnsenso the web demo still analyzes the original paste instead of throwing.AnalyzeSendIntentaccepts optionaltextMatchesfrom the translate use case so the model is not run twice on the same text.@custos/adapters-qvac/runtime(barrel: completion + translation factories).translate(), which covers the ES/EN/ZH demo. TranslatePsy-AfriSLM isAFRICAN_4B_TRANSLATION_Q4_K_Min the same registry if the demo language set is African.Stacked on #28 (
feat/qvac-scam-inference). Merge that first, then retarget this PR tomain.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 resolveApp.tsxonce.Test plan
pnpm --filter @custos/core test(includes TranslateAndAnalyzeMessage + precomputed matches)pnpm --filter @custos/adapters-qvac testpnpm run build/typecheck/ web buildcreateQvacTranslationClient, paste Spanish scam chat, confirm on-device translation then detectionCloses #3