fix(workflow): address Copilot review feedback on #2421

- Centralize the integration.json path as a module-level INTEGRATION_JSON
  constant in workflows/engine.py (mirrors specify_cli.INTEGRATION_JSON;
  cannot be imported directly without a circular dependency).
- Catch UnicodeDecodeError alongside JSONDecodeError/OSError so a
  non-UTF8 integration.json falls back to the literal default instead
  of crashing the workflow engine. Adds a regression test.
- Drop the narrow requires.integrations.any allowlist in the speckit
  workflow YAML; the four core commands (specify/plan/tasks/implement)
  are provided by every integration, so the previous list was always
  inaccurate (e.g. opencode, codex, etc. were excluded).

Author: Quratulain-bilal

src/specify_cli/workflows/engine.py

@@ -22,6 +22,13 @@
 from .base import RunStatus, StepContext, StepResult, StepStatus
 
 
+# Path to the file recording the project's initialized integration. Kept in
+# sync with ``specify_cli.INTEGRATION_JSON``; the workflows package cannot
+# import that constant directly without creating a circular import on the
+# CLI module.
+INTEGRATION_JSON = ".specify/integration.json"
+
+
 # -- Workflow Definition --------------------------------------------------
 
 
@@ -741,12 +748,12 @@ def _load_project_integration(self) -> str | None:
         Returns None when the file is missing or malformed; callers are
         expected to fall back to a literal default.
         """
-        path = self.project_root / ".specify" / "integration.json"
+        path = self.project_root / INTEGRATION_JSON
         if not path.is_file():
             return None
         try:
             data = json.loads(path.read_text(encoding="utf-8"))
-        except (json.JSONDecodeError, OSError):
+        except (json.JSONDecodeError, OSError, UnicodeDecodeError):
             return None
         if not isinstance(data, dict):
             return None

tests/test_workflows.py

@@ -1593,6 +1593,31 @@ def test_integration_auto_ignores_malformed_integration_json(self, project_dir):
         resolved = engine._resolve_inputs(definition, {})
         assert resolved["integration"] == "auto"
 
+    def test_integration_auto_ignores_non_utf8_integration_json(self, project_dir):
+        """A non-UTF8 integration.json must not crash — fall back to the literal default."""
+        from specify_cli.workflows.engine import WorkflowEngine, WorkflowDefinition
+
+        specify_dir = project_dir / ".specify"
+        specify_dir.mkdir(parents=True, exist_ok=True)
+        # 0xFF is invalid as the leading byte of a UTF-8 sequence, so
+        # ``Path.read_text(encoding="utf-8")`` raises UnicodeDecodeError.
+        (specify_dir / "integration.json").write_bytes(b"\xff\xfe\x00\x00")
+
+        definition = WorkflowDefinition.from_string("""
+schema_version: "1.0"
+workflow:
+  id: "auto-non-utf8"
+  name: "Auto Non UTF-8"
+  version: "1.0.0"
+inputs:
+  integration:
+    type: string
+    default: "auto"
+""")
+        engine = WorkflowEngine(project_dir)
+        resolved = engine._resolve_inputs(definition, {})
+        assert resolved["integration"] == "auto"
+
 
 # ===== State Persistence Tests =====
 

workflows/speckit/workflow.yml

@@ -8,8 +8,9 @@ workflow:
 
 requires:
   speckit_version: ">=0.7.2"
-  integrations:
-    any: ["copilot", "claude", "gemini"]
+  # The four commands used below (specify, plan, tasks, implement) are core
+  # spec-kit commands provided by every integration, so this workflow runs
+  # against whichever integration the project was initialized with.
 
 inputs:
   spec: