fix(claude): address review feedback on CLAUDE.md generation

- Introduce CONSTITUTION_REL_PATH constant in specify_cli and reuse it
  in ensure_constitution_from_template and ClaudeIntegration.ensure_claude_md
  so the path cannot drift between init scaffolding and integration setup.
- Make ensure_claude_md a classmethod that uses cls.context_file instead
  of hardcoding "CLAUDE.md".
- Expand CLAUDE.md body to list all core speckit workflow commands
  (constitution, specify, clarify, plan, tasks, analyze, checklist,
  implement) so Claude Code sees the full workflow.
- Strengthen tests to assert every section header and every command
  line via shared EXPECTED_CLAUDE_MD_SECTIONS / _COMMANDS constants.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>

Author: arun-gupta

src/specify_cli/__init__.py

@@ -97,6 +97,11 @@ def _build_ai_assistant_help() -> str:
 CLAUDE_LOCAL_PATH = Path.home() / ".claude" / "local" / "claude"
 CLAUDE_NPM_LOCAL_PATH = Path.home() / ".claude" / "local" / "node_modules" / ".bin" / "claude"
 
+# Relative path (from project root) to the authoritative constitution file.
+# Shared by init-time scaffolding and integration-specific context-file
+# generation so the two cannot drift.
+CONSTITUTION_REL_PATH = Path(".specify") / "memory" / "constitution.md"
+
 BANNER = """
 ███████╗██████╗ ███████╗ ██████╗██╗███████╗██╗   ██╗
 ██╔════╝██╔══██╗██╔════╝██╔════╝██║██╔════╝╚██╗ ██╔╝
@@ -753,7 +758,7 @@ def ensure_executable_scripts(project_path: Path, tracker: StepTracker | None =
 
 def ensure_constitution_from_template(project_path: Path, tracker: StepTracker | None = None) -> None:
     """Copy constitution template to memory if it doesn't exist (preserves existing constitution on reinitialization)."""
-    memory_constitution = project_path / ".specify" / "memory" / "constitution.md"
+    memory_constitution = project_path / CONSTITUTION_REL_PATH
     template_constitution = project_path / ".specify" / "templates" / "constitution-template.md"
 
     # If constitution already exists in memory, preserve it

src/specify_cli/integrations/claude/__init__.py

@@ -148,34 +148,44 @@ def _inject_frontmatter_flag(content: str, key: str, value: str = "true") -> str
             out.append(line)
         return "".join(out)
 
-    @staticmethod
-    def ensure_claude_md(project_root: Path) -> Path | None:
-        """Create a minimal root ``CLAUDE.md`` if missing.
+    @classmethod
+    def ensure_claude_md(cls, project_root: Path) -> Path | None:
+        """Create a minimal root context file (``CLAUDE.md``) if missing.
 
-        Claude Code expects ``CLAUDE.md`` at the project root; this file
-        acts as a bridge to ``.specify/memory/constitution.md``.
+        Claude Code expects ``context_file`` at the project root; this file
+        acts as a bridge to the constitution at ``CONSTITUTION_REL_PATH``.
         Returns the path if created, ``None`` otherwise.
         """
-        constitution = project_root / ".specify" / "memory" / "constitution.md"
-        claude_file = project_root / "CLAUDE.md"
-        if claude_file.exists() or not constitution.exists():
+        from specify_cli import CONSTITUTION_REL_PATH
+
+        if cls.context_file is None:
+            return None
+
+        constitution = project_root / CONSTITUTION_REL_PATH
+        context_file = project_root / cls.context_file
+        if context_file.exists() or not constitution.exists():
             return None
 
+        constitution_rel = CONSTITUTION_REL_PATH.as_posix()
         content = (
             "## Claude's Role\n"
-            "Read `.specify/memory/constitution.md` first. It is the authoritative source of truth for this project. "
+            f"Read `{constitution_rel}` first. It is the authoritative source of truth for this project. "
             "Everything in it is non-negotiable.\n\n"
             "## SpecKit Commands\n"
+            "- `/speckit.constitution` — establish or amend project principles\n"
             "- `/speckit.specify` — generate spec\n"
+            "- `/speckit.clarify` — ask structured de-risking questions (before `/speckit.plan`)\n"
             "- `/speckit.plan` — generate plan\n"
             "- `/speckit.tasks` — generate task list\n"
+            "- `/speckit.analyze` — cross-artifact consistency report (after `/speckit.tasks`)\n"
+            "- `/speckit.checklist` — generate quality checklists\n"
             "- `/speckit.implement` — execute plan\n\n"
             "## On Ambiguity\n"
             "If a spec is missing, incomplete, or conflicts with the constitution — stop and ask. "
             "Do not infer. Do not proceed.\n\n"
         )
-        claude_file.write_text(content, encoding="utf-8")
-        return claude_file
+        context_file.write_text(content, encoding="utf-8")
+        return context_file
 
     def setup(
         self,

tests/integrations/test_integration_claude.py

@@ -286,6 +286,23 @@ def test_claude_preset_creates_new_skill_without_commands_dir(self, tmp_path):
         assert "speckit-research" in metadata.get("registered_skills", [])
 
 
+EXPECTED_CLAUDE_MD_COMMANDS = (
+    "/speckit.constitution",
+    "/speckit.specify",
+    "/speckit.clarify",
+    "/speckit.plan",
+    "/speckit.tasks",
+    "/speckit.analyze",
+    "/speckit.checklist",
+    "/speckit.implement",
+)
+EXPECTED_CLAUDE_MD_SECTIONS = (
+    "## Claude's Role",
+    "## SpecKit Commands",
+    "## On Ambiguity",
+)
+
+
 class TestClaudeMdCreation:
     """Verify that CLAUDE.md is created during setup when constitution exists."""
 
@@ -303,6 +320,10 @@ def test_setup_creates_claude_md_when_constitution_exists(self, tmp_path):
         content = claude_md.read_text(encoding="utf-8")
         assert ".specify/memory/constitution.md" in content
         assert claude_md in created
+        for section in EXPECTED_CLAUDE_MD_SECTIONS:
+            assert section in content, f"missing section header: {section}"
+        for command in EXPECTED_CLAUDE_MD_COMMANDS:
+            assert f"`{command}`" in content, f"missing command: {command}"
 
     def test_setup_skips_claude_md_when_constitution_missing(self, tmp_path):
         integration = get_integration("claude")
@@ -354,6 +375,10 @@ def test_init_cli_creates_claude_md(self, tmp_path):
         assert claude_md.exists()
         content = claude_md.read_text(encoding="utf-8")
         assert ".specify/memory/constitution.md" in content
+        for section in EXPECTED_CLAUDE_MD_SECTIONS:
+            assert section in content, f"missing section header: {section}"
+        for command in EXPECTED_CLAUDE_MD_COMMANDS:
+            assert f"`{command}`" in content, f"missing command: {command}"
 
 
 class TestClaudeArgumentHints: