fix: address integration catalog review feedback

Author: Adr1an04

src/specify_cli/__init__.py

@@ -2757,7 +2757,10 @@ def integration_catalog_list():
 
 @integration_catalog_app.command("add")
 def integration_catalog_add(
-    url: str = typer.Argument(..., help="Catalog URL to add (must use HTTPS)"),
+    url: str = typer.Argument(
+        ...,
+        help="Catalog URL to add (HTTPS required, except http://localhost for local testing)",
+    ),
     name: Optional[str] = typer.Option(None, "--name", help="Catalog name"),
 ):
     """Add an integration catalog source to the project config."""

src/specify_cli/integrations/catalog.py

@@ -153,12 +153,18 @@ def _load_catalog_config(
                 raise IntegrationValidationError(
                     f"Invalid catalog URL in {config_path} at index {idx}: {exc}"
                 ) from exc
+            raw_priority = item.get("priority", idx + 1)
+            if isinstance(raw_priority, bool):
+                raise IntegrationValidationError(
+                    f"Invalid priority for catalog '{item.get('name', idx + 1)}': "
+                    f"expected integer, got {raw_priority!r}"
+                )
             try:
-                priority = int(item.get("priority", idx + 1))
+                priority = int(raw_priority)
             except (TypeError, ValueError):
                 raise IntegrationValidationError(
                     f"Invalid priority for catalog '{item.get('name', idx + 1)}': "
-                    f"expected integer, got {item.get('priority')!r}"
+                    f"expected integer, got {raw_priority!r}"
                 )
             raw_install = item.get("install_allowed", False)
             if isinstance(raw_install, str):

tests/integrations/test_integration_catalog.py

@@ -972,6 +972,29 @@ def test_load_catalog_config_raises_validation_error_for_invalid_yaml(
         with pytest.raises(IntegrationValidationError, match="Failed to read catalog config"):
             cat.get_active_catalogs()
 
+    def test_load_catalog_config_rejects_boolean_priority(self, tmp_path, monkeypatch):
+        self._isolate(tmp_path, monkeypatch)
+        cfg_path = tmp_path / ".specify" / "integration-catalogs.yml"
+        cfg_path.parent.mkdir(parents=True, exist_ok=True)
+        cfg_path.write_text(
+            yaml.dump(
+                {
+                    "catalogs": [
+                        {
+                            "url": "https://a.example.com/catalog.json",
+                            "name": "a",
+                            "priority": True,
+                        }
+                    ]
+                }
+            ),
+            encoding="utf-8",
+        )
+
+        cat = IntegrationCatalog(tmp_path)
+        with pytest.raises(IntegrationValidationError, match="Invalid priority|expected integer"):
+            cat.get_active_catalogs()
+
     def test_remove_catalog_uses_display_order_with_explicit_priorities(
         self, tmp_path, monkeypatch
     ):