Move _re/_sys imports out of loop and _RESERVED_STEP_IDS to module level

Author: Copilot

src/specify_cli/__init__.py

@@ -5933,6 +5933,11 @@ def workflow_step_list():
         )
 
 
+# IDs that map to internal names used under .specify/workflows/steps/ and must
+# not be used as custom step IDs (dotfile check is done separately at runtime).
+_RESERVED_STEP_IDS: frozenset[str] = frozenset({".cache", "step-registry.json"})
+
+
 @workflow_step_app.command("add")
 def workflow_step_add(
     step_id: str = typer.Argument(..., help="Step type ID from catalog"),
@@ -6033,7 +6038,6 @@ def _safe_fetch(url: str) -> bytes:
     # Reject IDs that collide with internal names used under steps_base_dir
     # (dotfiles, the cache dir, and the registry filename) to prevent
     # corrupting caching or registry persistence.
-    _RESERVED_STEP_IDS = {".cache", "step-registry.json"}
     if step_id.startswith(".") or step_id in _RESERVED_STEP_IDS:
         console.print(
             f"[red]Error:[/red] '{step_id}' is a reserved name and cannot be used as a step ID"

src/specify_cli/workflows/__init__.py

@@ -81,6 +81,8 @@ def load_custom_steps(project_root: Path) -> list[str]:
     Silently skips packages that fail to import or validate.
     """
     import importlib.util as _importlib_util
+    import re as _re
+    import sys as _sys
 
     steps_dir = Path(project_root) / ".specify" / "workflows" / "steps"
     if not steps_dir.is_dir():
@@ -108,9 +110,6 @@ def load_custom_steps(project_root: Path) -> list[str]:
             if type_key in STEP_REGISTRY:
                 continue
 
-            import re as _re
-            import sys as _sys
-
             # Sanitize type_key so the synthetic module name is a valid identifier
             # (e.g. "test-custom" → "_speckit_custom_step_test_custom").
             safe_key = _re.sub(r"[^A-Za-z0-9_]", "_", type_key)