FEATURE: Localization fallbacks (server-side)

[akshayutture-augment]

Test 9

---

Replicated from ai-code-review-evaluation/discourse-greptile#9

[greptile-apps]

Your free trial has ended. If you'd like to continue receiving code reviews, you can add a payment method here.

[akshayutture-augment]

@greptileai

[greptile-apps]

Greptile Overview

Confidence Score: 4/5

• This PR is safe to merge with low risk
• The implementation is well-structured and follows Discourse patterns. The custom FallbackLocaleList class properly handles dynamic site settings, and the initialization order is correct (loads after freedom patches and site settings due to alphabetical ordering). The lazy-loading approach in the [] method avoids initialization timing issues. Minor confidence reduction due to lack of visible test coverage in the changeset.
• No files require special attention

Important Files Changed

File Analysis

Filename	Score	Overview
config/initializers/i18n.rb	4/5	Added custom FallbackLocaleList class to handle locale fallback order (user locale → site default → English) and integrated pluralization support
lib/freedom_patches/translate_accelerator.rb	5/5	Added ensure_loaded! method to support explicit locale preloading for fallback chain
app/controllers/application_controller.rb	5/5	Added call to I18n.fallbacks.ensure_loaded! in set_locale to preload fallback locales

Sequence Diagram

sequenceDiagram
    participant Controller as ApplicationController
    participant I18n as I18n Module
    participant Fallbacks as FallbackLocaleList
    participant Accelerator as TranslateAccelerator
    participant Backend as I18n::Backend
    
    Controller->>Controller: set_locale()
    Controller->>I18n: Set locale (user or site default)
    Controller->>Fallbacks: ensure_loaded!()
    Fallbacks->>Fallbacks: self[I18n.locale]
    Note over Fallbacks: Returns [user_locale, site_default, :en]
    loop For each locale in fallback chain
        Fallbacks->>I18n: ensure_loaded!(locale)
        I18n->>Accelerator: load_locale(locale)
        Note over Accelerator: Check if already loaded
        alt Not yet loaded
            Accelerator->>Backend: load_translations(locale files)
            Accelerator->>Accelerator: Add to @loaded_locales
        end
    end
    
    Note over Controller,Backend: Later during request...
    Controller->>I18n: translate(key)
    I18n->>Accelerator: Check cache
    alt Cache miss
        I18n->>Backend: Lookup with fallbacks
        Note over Backend: Tries user locale → site default → :en
    end

Loading

[greptile-apps]

_{7 files reviewed, no comments}

_{Edit Code Review Agent Settings | Greptile}