Merge branch 'chore/add-empty-context-detection' into 'main'

Add empty context detection in i18n:validate job

See merge request web/clients!25664

Author: MargeBot

packages/activation/src/components/Modals/ProductSelectionModal/ProductItem.tsx

@@ -9,15 +9,15 @@ import { IcUsers } from '@proton/icons/icons/IcUsers';
 
 const productMap = {
     [ImportType.MAIL]: {
-        getLabel: () => c('').t`Emails`,
+        getLabel: () => c('Label').t`Emails`,
         icon: <IcEnvelope className="shrink-0" />,
     },
     [ImportType.CALENDAR]: {
-        getLabel: () => c('').t`Calendars`,
+        getLabel: () => c('Label').t`Calendars`,
         icon: <IcCalendarGrid className="shrink-0" />,
     },
     [ImportType.CONTACTS]: {
-        getLabel: () => c('').t`Contacts`,
+        getLabel: () => c('Label').t`Contacts`,
         icon: <IcUsers className="shrink-0" />,
     },
 };

packages/activation/src/oles/components/MigrationSetup/StepFinal.tsx

@@ -119,7 +119,12 @@ const StepFinal: FC<StepComponentProps> = ({ model: migrationConfiguration }) =>
                 {c('BOSS')
                     .t`You're almost done, you need to configure your domain to receive your emails directly on ${BRAND_NAME}. Once confirmed, your team will stop receiving new emails on Gmail and the migration will be completed.`}
             </p>
-            <LazyLottie style={{ padding: '0 10rem' }} autoPlay getAnimationData={() => import('../../animations/providerSwitch.json')} loop={true} />
+            <LazyLottie
+                style={{ padding: '0 10rem' }}
+                autoPlay
+                getAnimationData={() => import('../../animations/providerSwitch.json')}
+                loop={true}
+            />
             <p className="color-weak">{c('BOSS')
                 .t`Copy the below code and paste it in the DNS section of your domain host.`}</p>
             <DNSGroupRecords group={group} />
@@ -131,7 +136,7 @@ const StepFinal: FC<StepComponentProps> = ({ model: migrationConfiguration }) =>
             >
                 <div className="px-2">
                     <p className="m-0">{c('Label').t`Confirm MX records added`}</p>
-                    <span className="color-weak text-sm">{c('').t`Required to proceed to the next step`}</span>
+                    <span className="color-weak text-sm">{c('Info').t`Required to proceed to the next step`}</span>
                 </div>
             </Checkbox>
         </div>

packages/i18n/scripts/linter.mjs

@@ -132,6 +132,16 @@ async function* errorIterator(source = 'src', options = { isVerbose: false }) {
         if (errorsNumbers.match) {
             yield* errorsNumbers.errors(file);
         }
+
+        // Matches an empty (or whitespace-only) context: c(''), c(""), c(``)
+        const errorEmptyContext = testRule(
+            /c\(\s*(?:\x27\s*\x27|\x22\s*\x22|\x60\s*\x60)\s*\)\./g,
+            content,
+            'emptyContext'
+        );
+        if (errorEmptyContext.match) {
+            yield* errorEmptyContext.errors(file);
+        }
     }
 }
 
@@ -181,6 +191,15 @@ function formatErrors(error) {
 `;
     }
 
+    if (error.type === 'emptyContext') {
+        return `🚨 [Error] ${error.file}:${error.index}
+    match: ${error.match}
+     line: ${error.line}
+      fix: The translation context must not be empty.
+           Use a meaningful context - c(\x27<context>\x27).t\`<string>\`
+`;
+    }
+
     if (error.type === 'numbers') {
         return `🚨 [Error] ${error.file}:${error.index}
     match: ${error.match}