fix(cli): skip tolgee sync for unmapped catalogs

Author: WendellXY

langcodec-cli/src/tolgee.rs

@@ -197,7 +197,15 @@ pub fn prefill_translate_from_tolgee(
     }
 
     let project = load_tolgee_project(settings.config.as_deref())?;
-    let mapping = resolve_mapping_for_catalog(&project, local_catalog_path)?;
+    let Some(mapping) = find_mapping_for_catalog(&project, local_catalog_path)? else {
+        if settings.namespaces.is_empty() {
+            return Ok(None);
+        }
+        return Err(format!(
+            "Catalog '{}' is not configured in Tolgee push.files",
+            local_catalog_path
+        ));
+    };
 
     if !settings.namespaces.is_empty()
         && !settings
@@ -523,22 +531,16 @@ fn select_mappings(
     Ok(selected)
 }
 
-fn resolve_mapping_for_catalog(
+fn find_mapping_for_catalog(
     project: &TolgeeProject,
     local_catalog_path: &str,
-) -> Result<TolgeeMappedFile, String> {
+) -> Result<Option<TolgeeMappedFile>, String> {
     let resolved = absolute_from_current_dir(local_catalog_path)?;
-    project
+    Ok(project
         .mappings
         .iter()
         .find(|mapping| mapping.absolute_path == resolved)
-        .cloned()
-        .ok_or_else(|| {
-            format!(
-                "Catalog '{}' is not configured in Tolgee push.files",
-                local_catalog_path
-            )
-        })
+        .cloned())
 }
 
 fn discover_tolgee_cli(project_root: &Path) -> Result<TolgeeCliInvocation, String> {

langcodec-cli/src/translate.rs

@@ -2772,6 +2772,56 @@ target = "translated.xcstrings"
         assert!(captured.contains("\"Core\""));
     }
 
+    #[test]
+    fn tolgee_translate_ignores_unmapped_catalogs_without_namespace_filter() {
+        let temp_dir = TempDir::new().unwrap();
+        let project_root = temp_dir.path();
+        let source = project_root.join("ModuleExport.xcstrings");
+
+        fs::write(
+            &source,
+            r#"{
+  "sourceLanguage" : "en",
+  "version" : "1.0",
+  "strings" : {
+    "welcome" : {
+      "localizations" : {
+        "en" : {
+          "stringUnit" : {
+            "state" : "translated",
+            "value" : "Welcome"
+          }
+        },
+        "fr" : {
+          "stringUnit" : {
+            "state" : "translated",
+            "value" : "Bienvenue"
+          }
+        }
+      }
+    }
+  }
+}"#,
+        )
+        .unwrap();
+
+        let tolgee_config = write_translate_tolgee_config(project_root);
+        let mut options = base_options(&source, None);
+        options.target_langs = vec!["fr".to_string()];
+        options.provider = None;
+        options.model = None;
+        options.use_tolgee = true;
+        options.tolgee_config = Some(tolgee_config.to_string_lossy().to_string());
+
+        let prepared = prepare_translation(&options).unwrap();
+        assert!(prepared.tolgee_context.is_none());
+        assert!(prepared.jobs.is_empty());
+
+        let outcome = run_prepared_translation(prepared, None).unwrap();
+        assert_eq!(outcome.translated, 0);
+        assert_eq!(outcome.failed, 0);
+    }
+
     #[test]
     fn falls_back_to_xcstrings_key_when_source_locale_entry_is_missing() {
         let temp_dir = TempDir::new().unwrap();