fix(skill-maker): require intake pause before routing

Add the explicit wait instruction back to the skill-maker intake so the router
pauses for user input instead of auto-routing immediately. Add a focused
regression test to keep the intake pause behavior covered.

Co-authored-by: Cursor <cursoragent@cursor.com>

Author: schultzp2020

skills/skill-maker/SKILL.md

@@ -15,6 +15,7 @@ What do you need to do?
 2. **Create a new skill** — Interview, draft, and review from scratch
 3. **Consolidate skills** — Merge multiple skills into fewer
 
+**Wait for response before proceeding.**
 </intake>
 
 <routing>

tests/conftest.py

@@ -19,6 +19,9 @@
 # Path to the overlay skill directory (markdown-only)
 OVERLAY_SKILL_DIR = PROJECT_ROOT / "skills" / "overlay"
 
+# Path to the skill-maker skill directory (markdown-only)
+SKILL_MAKER_DIR = PROJECT_ROOT / "skills" / "skill-maker"
+
 # Add orchestrator skill directory to path for testing
 if str(RHDH_SKILL_DIR) not in sys.path:
     sys.path.insert(0, str(RHDH_SKILL_DIR))
@@ -65,6 +68,12 @@ def overlay_skill_dir():
     return OVERLAY_SKILL_DIR
 
 
+@pytest.fixture
+def skill_maker_dir():
+    """Return the skill-maker directory path (skills/skill-maker)."""
+    return SKILL_MAKER_DIR
+
+
 @pytest.fixture
 def isolated_env(tmp_path, monkeypatch):
     """Create an isolated environment with temp directories.

tests/unit/test_skill_structure.py

@@ -165,6 +165,22 @@ def test_no_markdown_headings_in_xml_body(self, skill_md):
                     pytest.fail(f"Found markdown heading in <{section_name}>: {line.strip()}")
 
 
+class TestSkillMakerSkillMd:
+    """Test that skill-maker SKILL.md keeps router intake pause instructions."""
+
+    @pytest.fixture
+    def skill_md(self, skill_maker_dir):
+        """Load skill-maker SKILL.md content."""
+        skill_path = skill_maker_dir / "SKILL.md"
+        return skill_path.read_text(encoding="utf-8")
+
+    def test_intake_waits_for_user_response(self, skill_md):
+        """The intake section must tell the agent to wait before routing."""
+        match = re.search(r"<intake>(.*?)</intake>", skill_md, re.DOTALL)
+        assert match, "SKILL.md missing <intake> section"
+        assert "**Wait for response before proceeding.**" in match.group(1)
+
+
 class TestWorkflowStructure:
     """Test that workflow files have required structure."""