[T3339] FIX: gate B2S letter translation on detected content language

[Danielgergely]

• FIX: Trust the letter's detected language over the field-office stamp; fail safe to translation when it can't be determined

[gemini-code-assist]

Caution

The consumer version of Gemini Code Assist on GitHub has been sunset. All code review activity has officially ceased.

[greptile-apps]

Confidence Score: 5/5

The PR appears safe to merge, with undetectable content consistently falling back to translation or manual handling.

The new helper uses the most processed available letter text, while both callers explicitly reject an empty detection result.

Files Needing Attention: Module dependencies guarantee the helper is available, and no incompatible consumers or overrides were identified.

T-Rex Logs

What T-Rex did

• Before capture, stamp-based decisions incorrectly queued readable English and allowed French, blank, and undetectable content.
• After capture, language detection correctly identified English and French and produced publish/queue plus auto-send decisions that match all acceptance expectations (acceptance_failures=0).
• The blocker capture recorded the exact failed Odoo and PostgreSQL availability commands.
• Reviewed the related artifacts to verify pre- and post-capture behavior and the blocker command evidence.

_{Ran code and verified through T-Rex}

Important Files Changed

Filename	Overview
partner_communication_compassion/models/correspondence.py	Replaces metadata-based auto-send eligibility with a fail-safe detected-content-language check.
sbc_compassion/models/correspondence.py	Adds the shared content normalization and language-detection helper used by both gates.
sbc_translation/models/correspondence.py	Uses detected content language to decide whether a letter can bypass translation.

Flowchart

%%{init: {'theme': 'neutral'}}%%
flowchart TD
    A[Select translated, English, or original text] --> B[Normalize text and detect language]
    B --> C{Language detected and supported?}
    C -->|Yes| D[Continue publication or auto-send]
    C -->|No| E[Route to translation or manual delivery]

Loading

_{Reviews (1): Last reviewed commit: "[T3339] FIX: gate B2S letter translation..." | Re-trigger Greptile}