fix: address fourth round of PR review feedback

- Remove unused _write_catalog helper from test file
- Fix comment: tests use monkeypatched urlopen, not file:// URLs
- Wrap cache unlink calls in OSError handler
- Add explicit encoding='utf-8' to all cache read_text/write_text calls
- Restore packaging.version.Version for descriptor version validation
  to align with extension/preset validators
- Add missing goose entry to integrations/catalog.json

Author: mnriem

integrations/catalog.json

@@ -245,6 +245,15 @@
       "author": "spec-kit-core",
       "repository": "https://github.com/github/spec-kit",
       "tags": ["generic"]
+    },
+    "goose": {
+      "id": "goose",
+      "name": "Goose",
+      "version": "1.0.0",
+      "description": "Goose CLI integration with YAML recipe format",
+      "author": "spec-kit-core",
+      "repository": "https://github.com/github/spec-kit",
+      "tags": ["cli"]
     }
   }
 }

src/specify_cli/integrations/catalog.py

@@ -19,6 +19,7 @@
 from typing import Any, Dict, List, Optional
 
 import yaml
+from packaging import version as pkg_version
 
 
 # ---------------------------------------------------------------------------
@@ -239,17 +240,20 @@ def _fetch_single_catalog(
 
         if not force_refresh and cache_file.exists() and cache_meta.exists():
             try:
-                meta = json.loads(cache_meta.read_text())
+                meta = json.loads(cache_meta.read_text(encoding="utf-8"))
                 cached_at = datetime.fromisoformat(meta.get("cached_at", ""))
                 if cached_at.tzinfo is None:
                     cached_at = cached_at.replace(tzinfo=timezone.utc)
                 age = (datetime.now(timezone.utc) - cached_at).total_seconds()
                 if age < self.CACHE_DURATION:
-                    return json.loads(cache_file.read_text())
+                    return json.loads(cache_file.read_text(encoding="utf-8"))
             except (json.JSONDecodeError, ValueError, KeyError, TypeError):
                 # Cache is invalid or stale metadata; delete and refetch from source.
-                cache_file.unlink(missing_ok=True)
-                cache_meta.unlink(missing_ok=True)
+                try:
+                    cache_file.unlink(missing_ok=True)
+                    cache_meta.unlink(missing_ok=True)
+                except OSError:
+                    pass
 
         try:
             with urllib.request.urlopen(entry.url, timeout=10) as resp:
@@ -273,15 +277,16 @@ def _fetch_single_catalog(
 
             try:
                 self.cache_dir.mkdir(parents=True, exist_ok=True)
-                cache_file.write_text(json.dumps(catalog_data, indent=2))
+                cache_file.write_text(json.dumps(catalog_data, indent=2), encoding="utf-8")
                 cache_meta.write_text(
                     json.dumps(
                         {
                             "cached_at": datetime.now(timezone.utc).isoformat(),
                             "catalog_url": entry.url,
                         },
                         indent=2,
-                    )
+                    ),
+                    encoding="utf-8",
                 )
             except OSError:
                 pass  # Cache is best-effort; proceed with fetched data
@@ -473,9 +478,11 @@ def _validate(self) -> None:
                 "must be lowercase alphanumeric with hyphens only"
             )
 
-        if not re.match(r"^\d+\.\d+\.\d+$", integ["version"]):
+        try:
+            pkg_version.Version(integ["version"])
+        except pkg_version.InvalidVersion:
             raise IntegrationDescriptorError(
-                f"Invalid version '{integ['version']}': must use semantic versioning (e.g., 1.0.0)"
+                f"Invalid version '{integ['version']}'"
             )
 
         requires = self.data["requires"]

tests/integrations/test_integration_catalog.py

@@ -118,20 +118,10 @@ def test_empty_config_raises(self, tmp_path):
 
 
 # ---------------------------------------------------------------------------
-# IntegrationCatalog — fetch & search (using local file:// catalog)
+# IntegrationCatalog — fetch & search (using monkeypatched urlopen responses)
 # ---------------------------------------------------------------------------
 
 
-def _write_catalog(path: Path, integrations: dict) -> None:
-    """Helper: write a catalog JSON file."""
-    path.parent.mkdir(parents=True, exist_ok=True)
-    path.write_text(json.dumps({
-        "schema_version": "1.0",
-        "updated_at": "2026-01-01T00:00:00Z",
-        "integrations": integrations,
-    }, indent=2))
-
-
 class TestCatalogFetch:
     """Tests that use a local HTTP server stub via monkeypatch."""