fix(workflows): address remaining PR #2408 review findings

- engine.py: strip whitespace from integration value in
  _resolve_integration_auto(); whitespace-only strings like '   '
  now fall back to 'copilot' instead of being returned as-is
- __init__.py: define INTEGRATION_JSON locally instead of importing
  from workflows.constants; avoids executing the workflows package
  __init__ (which registers all step types) during CLI startup for
  commands that don't use workflows
- tests: add test for whitespace-only integration value fallback

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>

Author: Markus

src/specify_cli/__init__.py

@@ -57,7 +57,9 @@
 # For cross-platform keyboard input
 import readchar
 
-from .workflows.constants import INTEGRATION_JSON
+# Keep this local so importing specify_cli does not import the workflows package
+# (whose package initialization registers steps and imports all workflow modules).
+INTEGRATION_JSON = ".specify/integration.json"
 
 GITHUB_API_LATEST = "https://api.github.com/repos/github/spec-kit/releases/latest"
 

src/specify_cli/workflows/engine.py

@@ -760,8 +760,10 @@ def _resolve_integration_auto(self) -> str:
             return self._AUTO_FALLBACK
         if isinstance(data, dict):
             value = data.get("integration")
-            if isinstance(value, str) and value:
-                return value
+            if isinstance(value, str):
+                normalized_value = value.strip()
+                if normalized_value:
+                    return normalized_value
         return self._AUTO_FALLBACK
 
     @staticmethod

tests/test_workflows.py

@@ -2076,3 +2076,22 @@ def test_resolve_inputs_auto_oserror_falls_back_to_copilot(self, project_dir):
 
         step_output = state.step_results["specify"]["output"]
         assert step_output["integration"] == "copilot"
+
+    def test_resolve_inputs_auto_whitespace_only_falls_back_to_copilot(self, project_dir):
+        """When integration.json has a whitespace-only value, 'auto' falls back to 'copilot'."""
+        from unittest.mock import patch
+        from specify_cli.workflows.engine import WorkflowEngine, WorkflowDefinition
+
+        int_json = project_dir / ".specify" / "integration.json"
+        int_json.write_text(json.dumps({"integration": "   "}), encoding="utf-8")
+
+        definition = WorkflowDefinition.from_string(self._make_workflow_yaml())
+        engine = WorkflowEngine(project_dir)
+
+        with patch(
+            "specify_cli.workflows.steps.command.shutil.which", return_value=None
+        ):
+            state = engine.execute(definition)
+
+        step_output = state.step_results["specify"]["output"]
+        assert step_output["integration"] == "copilot"