fix: accept numeric string catalog priorities

Author: Adr1an04

src/specify_cli/integrations/catalog.py

@@ -513,13 +513,21 @@ def add_catalog(self, url: str, name: Optional[str] = None) -> None:
                 )
             if "priority" in cat:
                 raw_priority = cat.get("priority")
-                if isinstance(raw_priority, bool) or not isinstance(raw_priority, int):
+                if isinstance(raw_priority, bool):
                     raise IntegrationValidationError(
                         f"Invalid catalog entry at index {idx}: "
                         f"'priority' must be an integer, got "
                         f"{type(raw_priority).__name__}."
                     )
-                existing_priorities.append(raw_priority)
+                try:
+                    normalized_priority = int(raw_priority)
+                except (TypeError, ValueError):
+                    raise IntegrationValidationError(
+                        f"Invalid catalog entry at index {idx}: "
+                        f"'priority' must be an integer, got "
+                        f"{type(raw_priority).__name__}."
+                    ) from None
+                existing_priorities.append(normalized_priority)
             else:
                 # Match `_load_catalog_config()`'s defaulting rule so the new
                 # entry still sorts after implicit-priority siblings.

tests/integrations/test_integration_catalog.py

@@ -799,6 +799,30 @@ def test_add_catalog_rejects_non_integer_priority(self, tmp_path, monkeypatch):
         with pytest.raises(IntegrationValidationError, match="'priority' must be an integer"):
             cat.add_catalog("https://b.example.com/catalog.json")
 
+    def test_add_catalog_accepts_numeric_string_priority(self, tmp_path, monkeypatch):
+        self._isolate(tmp_path, monkeypatch)
+        cfg_path = tmp_path / ".specify" / "integration-catalogs.yml"
+        cfg_path.write_text(
+            yaml.dump(
+                {
+                    "catalogs": [
+                        {
+                            "url": "https://a.example.com/catalog.json",
+                            "name": "a",
+                            "priority": "10",
+                        }
+                    ]
+                }
+            ),
+            encoding="utf-8",
+        )
+        cat = IntegrationCatalog(tmp_path)
+        cat.add_catalog("https://b.example.com/catalog.json", name="b")
+
+        data = yaml.safe_load(cfg_path.read_text(encoding="utf-8"))
+        assert data["catalogs"][-1]["name"] == "b"
+        assert data["catalogs"][-1]["priority"] == 11
+
     def test_add_catalog_rejects_existing_entry_with_bad_url(self, tmp_path, monkeypatch):
         """A sibling entry with an http:// URL should block a new add."""
         self._isolate(tmp_path, monkeypatch)