atdd 0.7.2__py3-none-any.whl → 0.7.3__py3-none-any.whl

atdd/coach/schemas/config.schema.json

@@ -132,6 +132,32 @@
         }
       },
       "additionalProperties": false
+    },
+    "coach": {
+      "type": "object",
+      "description": "Coach agent validation settings",
+      "properties": {
+        "session_gate_command_prefixes": {
+          "type": "array",
+          "description": "Allowed prefixes for gate commands in session files",
+          "items": {"type": "string"},
+          "default": ["atdd", "pytest", "python", "npm", "supabase"],
+          "examples": [["atdd", "pytest", "npm run test"]]
+        },
+        "session_gate_command_exceptions": {
+          "type": "array",
+          "description": "Session files exempt from gate command prefix validation (by filename pattern)",
+          "items": {"type": "string"},
+          "default": [],
+          "examples": [["SESSION-00-*.md", "SESSION-01-legacy.md"]]
+        },
+        "archive_status_warnings": {
+          "type": "boolean",
+          "description": "Warn when COMPLETE sessions not in archive or archived sessions not COMPLETE",
+          "default": true
+        }
+      },
+      "additionalProperties": false
     }
   },
   "required": ["version", "release"],

atdd/coach/validators/test_session_archive_status.py

@@ -0,0 +1,339 @@
+"""
+Session archive status validation.
+
+Validates archive placement aligns with session status:
+1. COMPLETE sessions should be in archive/ (warning)
+2. Sessions in archive/ should be COMPLETE or OBSOLETE (warning)
+
+These are warnings, not failures, to allow migration flexibility.
+
+Convention: src/atdd/coach/conventions/session.convention.yaml
+Config: .atdd/config.yaml (coach.archive_status_warnings)
+
+Run: atdd validate coach
+"""
+import pytest
+import yaml
+import fnmatch
+from pathlib import Path
+from typing import Dict, List, Any, Optional
+
+from atdd.coach.utils.repo import find_repo_root
+from atdd.coach.utils.config import load_atdd_config
+
+
+# ============================================================================
+# Configuration
+# ============================================================================
+
+REPO_ROOT = find_repo_root()
+SESSIONS_DIR = REPO_ROOT / "atdd-sessions"
+ARCHIVE_DIR = SESSIONS_DIR / "archive"
+
+# Terminal statuses that should be archived
+ARCHIVABLE_STATUSES = {"COMPLETE", "OBSOLETE"}
+
+
+# ============================================================================
+# Fixtures
+# ============================================================================
+
+@pytest.fixture
+def config() -> Dict[str, Any]:
+    """Load ATDD configuration."""
+    return load_atdd_config(REPO_ROOT)
+
+
+@pytest.fixture
+def archive_warnings_enabled(config: Dict[str, Any]) -> bool:
+    """Check if archive status warnings are enabled."""
+    return config.get("coach", {}).get("archive_status_warnings", True)
+
+
+@pytest.fixture
+def session_files() -> List[Path]:
+    """Get all session files in main directory (not archive)."""
+    if not SESSIONS_DIR.exists():
+        return []
+
+    return sorted([
+        f for f in SESSIONS_DIR.glob("SESSION-*.md")
+        if f.name != "SESSION-TEMPLATE.md"
+    ])
+
+
+@pytest.fixture
+def archived_files() -> List[Path]:
+    """Get all session files in archive directory."""
+    if not ARCHIVE_DIR.exists():
+        return []
+
+    return sorted(ARCHIVE_DIR.glob("SESSION-*.md"))
+
+
+def parse_session_status(path: Path) -> Optional[str]:
+    """Parse status from session file frontmatter."""
+    content = path.read_text()
+
+    if not content.startswith("---"):
+        return None
+
+    parts = content.split("---", 2)
+    if len(parts) < 3:
+        return None
+
+    try:
+        frontmatter = yaml.safe_load(parts[1])
+        status = str(frontmatter.get("status", "")).upper()
+        # Handle status with extra info (e.g., "ACTIVE - working")
+        return status.split()[0] if status else None
+    except yaml.YAMLError:
+        return None
+
+
+# ============================================================================
+# Archive Status Validation Tests
+# ============================================================================
+
+@pytest.mark.coach
+def test_complete_sessions_should_be_archived(
+    session_files: List[Path],
+    archive_warnings_enabled: bool
+):
+    """
+    SPEC-COACH-ARCHIVE-001: COMPLETE sessions should be in archive/
+
+    Given: Session files in main directory
+    When: Checking status
+    Then: Sessions with COMPLETE/OBSOLETE status should be in archive/
+
+    Note: This is a warning, not a failure, to allow migration flexibility.
+    """
+    if not archive_warnings_enabled:
+        pytest.skip("Archive status warnings disabled in config")
+
+    if not session_files:
+        pytest.skip("No session files found")
+
+    should_archive = []
+
+    for path in session_files:
+        status = parse_session_status(path)
+        if status in ARCHIVABLE_STATUSES:
+            should_archive.append(f"{path.name} (status: {status})")
+
+    if should_archive:
+        print(
+            f"\n⚠️  Found {len(should_archive)} completed sessions not in archive/:\n" +
+            "\n".join(f"  - {s}" for s in should_archive) +
+            "\n\nConsider running 'atdd session archive <id>' to move them."
+        )
+
+
+@pytest.mark.coach
+def test_archived_sessions_should_be_complete(
+    archived_files: List[Path],
+    archive_warnings_enabled: bool
+):
+    """
+    SPEC-COACH-ARCHIVE-002: Archived sessions should have COMPLETE/OBSOLETE status
+
+    Given: Session files in archive/ directory
+    When: Checking status
+    Then: All archived sessions should have terminal status
+
+    Note: This is a warning, not a failure, to allow migration flexibility.
+    """
+    if not archive_warnings_enabled:
+        pytest.skip("Archive status warnings disabled in config")
+
+    if not archived_files:
+        pytest.skip("No archived session files found")
+
+    not_complete = []
+
+    for path in archived_files:
+        status = parse_session_status(path)
+        if status and status not in ARCHIVABLE_STATUSES:
+            not_complete.append(f"{path.name} (status: {status})")
+
+    if not_complete:
+        print(
+            f"\n⚠️  Found {len(not_complete)} archived sessions without terminal status:\n" +
+            "\n".join(f"  - {s}" for s in not_complete) +
+            "\n\nEither update status to COMPLETE/OBSOLETE or move back to atdd-sessions/."
+        )
+
+
+@pytest.mark.coach
+def test_archive_directory_structure():
+    """
+    SPEC-COACH-ARCHIVE-003: Archive directory exists if sessions directory exists
+
+    Given: atdd-sessions/ directory exists
+    When: Checking for archive/ subdirectory
+    Then: archive/ directory should exist (create if missing)
+    """
+    if not SESSIONS_DIR.exists():
+        pytest.skip("Sessions directory not found")
+
+    if not ARCHIVE_DIR.exists():
+        print(
+            f"\n⚠️  Archive directory does not exist: {ARCHIVE_DIR}\n"
+            "Consider creating it: mkdir -p atdd-sessions/archive/"
+        )
+
+
+# ============================================================================
+# Stale Session Detection
+# ============================================================================
+
+@pytest.mark.coach
+def test_stale_active_sessions(session_files: List[Path]):
+    """
+    SPEC-COACH-ARCHIVE-004: Detect potentially stale ACTIVE sessions
+
+    Given: Session files with ACTIVE status
+    When: Checking file modification time
+    Then: Warn about sessions not modified in 7+ days
+    """
+    if not session_files:
+        pytest.skip("No session files found")
+
+    from datetime import datetime, timedelta
+
+    stale_threshold = timedelta(days=7)
+    now = datetime.now()
+    stale = []
+
+    for path in session_files:
+        status = parse_session_status(path)
+        if status != "ACTIVE":
+            continue
+
+        mtime = datetime.fromtimestamp(path.stat().st_mtime)
+        age = now - mtime
+
+        if age > stale_threshold:
+            days = age.days
+            stale.append(f"{path.name} (last modified {days} days ago)")
+
+    if stale:
+        print(
+            f"\n⚠️  Found {len(stale)} potentially stale ACTIVE sessions:\n" +
+            "\n".join(f"  - {s}" for s in stale) +
+            "\n\nConsider updating status to BLOCKED, COMPLETE, or OBSOLETE."
+        )
+
+
+@pytest.mark.coach
+def test_blocked_sessions_have_reason(session_files: List[Path]):
+    """
+    SPEC-COACH-ARCHIVE-005: BLOCKED sessions should document reason
+
+    Given: Session files with BLOCKED status
+    When: Checking session log
+    Then: Most recent log entry should explain the blocker
+    """
+    if not session_files:
+        pytest.skip("No session files found")
+
+    import re
+
+    missing_reason = []
+
+    for path in session_files:
+        status = parse_session_status(path)
+        if status != "BLOCKED":
+            continue
+
+        content = path.read_text()
+
+        # Check for Blocked: section in session log
+        has_blocked_entry = bool(re.search(
+            r"\*\*Blocked:\*\*\s*\n\s*-\s*\S",
+            content
+        ))
+
+        # Or check frontmatter for blocker field
+        if content.startswith("---"):
+            parts = content.split("---", 2)
+            if len(parts) >= 3:
+                try:
+                    frontmatter = yaml.safe_load(parts[1])
+                    has_blocked_entry = has_blocked_entry or bool(
+                        frontmatter.get("blocker") or
+                        frontmatter.get("blocked_by") or
+                        frontmatter.get("blocked_reason")
+                    )
+                except yaml.YAMLError:
+                    pass
+
+        if not has_blocked_entry:
+            missing_reason.append(path.name)
+
+    if missing_reason:
+        print(
+            f"\n⚠️  Found {len(missing_reason)} BLOCKED sessions without documented reason:\n" +
+            "\n".join(f"  - {m}" for m in missing_reason) +
+            "\n\nAdd **Blocked:** entry to Session Log or 'blocker' field to frontmatter."
+        )
+
+
+# ============================================================================
+# Summary
+# ============================================================================
+
+@pytest.mark.coach
+def test_archive_status_summary(
+    session_files: List[Path],
+    archived_files: List[Path]
+):
+    """
+    Generate archive status summary.
+
+    This test always passes but prints a summary.
+    """
+    print("\n" + "=" * 60)
+    print("SESSION ARCHIVE STATUS SUMMARY")
+    print("=" * 60)
+
+    # Count by status in main directory
+    main_statuses: Dict[str, int] = {}
+    for path in session_files:
+        status = parse_session_status(path) or "UNKNOWN"
+        main_statuses[status] = main_statuses.get(status, 0) + 1
+
+    # Count by status in archive
+    archive_statuses: Dict[str, int] = {}
+    for path in archived_files:
+        status = parse_session_status(path) or "UNKNOWN"
+        archive_statuses[status] = archive_statuses.get(status, 0) + 1
+
+    print(f"\nMain directory ({len(session_files)} files):")
+    for status, count in sorted(main_statuses.items()):
+        marker = "⚠️ " if status in ARCHIVABLE_STATUSES else "   "
+        print(f"  {marker}{status}: {count}")
+
+    print(f"\nArchive directory ({len(archived_files)} files):")
+    for status, count in sorted(archive_statuses.items()):
+        marker = "⚠️ " if status not in ARCHIVABLE_STATUSES else "   "
+        print(f"  {marker}{status}: {count}")
+
+    # Summary advice
+    main_archivable = sum(
+        main_statuses.get(s, 0) for s in ARCHIVABLE_STATUSES
+    )
+    archive_active = sum(
+        archive_statuses.get(s, 0) for s in archive_statuses
+        if s not in ARCHIVABLE_STATUSES
+    )
+
+    if main_archivable > 0 or archive_active > 0:
+        print("\n⚠️  Recommendations:")
+        if main_archivable > 0:
+            print(f"   - Archive {main_archivable} completed session(s)")
+        if archive_active > 0:
+            print(f"   - Review {archive_active} non-terminal archived session(s)")
+
+    print("\n" + "=" * 60)

atdd/coach/validators/test_session_manifest_alignment.py

@@ -0,0 +1,387 @@
+"""
+Session-manifest alignment validation.
+
+Ensures session files and manifest entries are synchronized:
+1. Every manifest entry has a corresponding session file
+2. Every session file has a manifest entry
+3. Status in session frontmatter matches manifest status
+
+Convention: src/atdd/coach/conventions/session.convention.yaml
+Schema: src/atdd/coach/schemas/manifest.schema.json
+
+Run: atdd validate coach
+"""
+import pytest
+import yaml
+from pathlib import Path
+from typing import Dict, List, Any, Optional, Tuple
+
+from atdd.coach.utils.repo import find_repo_root
+
+
+# ============================================================================
+# Configuration
+# ============================================================================
+
+REPO_ROOT = find_repo_root()
+SESSIONS_DIR = REPO_ROOT / "atdd-sessions"
+ARCHIVE_DIR = SESSIONS_DIR / "archive"
+MANIFEST_FILE = REPO_ROOT / ".atdd" / "manifest.yaml"
+
+# Valid statuses from convention
+VALID_STATUSES = {"INIT", "PLANNED", "ACTIVE", "BLOCKED", "COMPLETE", "OBSOLETE", "UNKNOWN"}
+
+
+# ============================================================================
+# Fixtures
+# ============================================================================
+
+@pytest.fixture
+def manifest() -> Optional[Dict[str, Any]]:
+    """Load session manifest from .atdd/manifest.yaml."""
+    if not MANIFEST_FILE.exists():
+        return None
+
+    with open(MANIFEST_FILE) as f:
+        return yaml.safe_load(f)
+
+
+@pytest.fixture
+def session_files() -> List[Path]:
+    """Get all session files (excluding template, including archive)."""
+    if not SESSIONS_DIR.exists():
+        return []
+
+    files = []
+
+    # Main sessions directory
+    for f in SESSIONS_DIR.glob("SESSION-*.md"):
+        if f.name != "SESSION-TEMPLATE.md":
+            files.append(f)
+
+    # Archive directory
+    if ARCHIVE_DIR.exists():
+        for f in ARCHIVE_DIR.glob("SESSION-*.md"):
+            files.append(f)
+
+    return sorted(files)
+
+
+def parse_session_frontmatter(path: Path) -> Optional[Dict[str, Any]]:
+    """Parse YAML frontmatter from session file."""
+    content = path.read_text()
+
+    if not content.startswith("---"):
+        return None
+
+    parts = content.split("---", 2)
+    if len(parts) < 3:
+        return None
+
+    try:
+        return yaml.safe_load(parts[1])
+    except yaml.YAMLError:
+        return None
+
+
+def get_session_id_from_filename(filename: str) -> Optional[str]:
+    """Extract session ID from filename (e.g., SESSION-01-foo.md -> 01)."""
+    import re
+    match = re.match(r"SESSION-(\d{2})-", filename)
+    return match.group(1) if match else None
+
+
+# ============================================================================
+# Manifest Entry Validation Tests
+# ============================================================================
+
+@pytest.mark.coach
+def test_manifest_exists():
+    """
+    SPEC-COACH-MANIFEST-001: Manifest file exists
+
+    Given: Initialized ATDD project
+    When: Checking for manifest file
+    Then: .atdd/manifest.yaml exists
+    """
+    if not (REPO_ROOT / ".atdd").exists():
+        pytest.skip("ATDD not initialized (no .atdd/ directory)")
+
+    assert MANIFEST_FILE.exists(), \
+        f"Manifest file not found at {MANIFEST_FILE}. Run 'atdd init' first."
+
+
+@pytest.mark.coach
+def test_manifest_entries_have_session_files(manifest: Optional[Dict], session_files: List[Path]):
+    """
+    SPEC-COACH-MANIFEST-002: Every manifest entry has a corresponding session file
+
+    Given: Session manifest with entries
+    When: Checking for session files
+    Then: Each manifest entry has a matching file on disk
+    """
+    if manifest is None:
+        pytest.skip("No manifest file found")
+
+    sessions_dir = manifest.get("sessions_dir", "atdd-sessions")
+    missing = []
+
+    for entry in manifest.get("sessions", []):
+        file_path = entry.get("file", "")
+        session_id = entry.get("id", "")
+
+        # Resolve full path
+        full_path = REPO_ROOT / sessions_dir / file_path
+
+        if not full_path.exists():
+            missing.append(f"id={session_id}: {file_path}")
+
+    if missing:
+        pytest.fail(
+            f"Found {len(missing)} manifest entries without session files:\n" +
+            "\n".join(f"  - {m}" for m in missing)
+        )
+
+
+@pytest.mark.coach
+def test_session_files_have_manifest_entries(manifest: Optional[Dict], session_files: List[Path]):
+    """
+    SPEC-COACH-MANIFEST-003: Every session file has a manifest entry
+
+    Given: Session files in atdd-sessions/
+    When: Checking manifest entries
+    Then: Each session file has a corresponding manifest entry
+    """
+    if manifest is None:
+        pytest.skip("No manifest file found")
+
+    if not session_files:
+        pytest.skip("No session files found")
+
+    # Build set of manifest file paths
+    sessions_dir = manifest.get("sessions_dir", "atdd-sessions")
+    manifest_files = set()
+
+    for entry in manifest.get("sessions", []):
+        file_path = entry.get("file", "")
+        manifest_files.add(file_path)
+
+    # Check each session file
+    orphaned = []
+
+    for session_path in session_files:
+        # Compute relative path from sessions_dir
+        try:
+            rel_path = session_path.relative_to(REPO_ROOT / sessions_dir)
+            rel_str = str(rel_path)
+        except ValueError:
+            rel_str = session_path.name
+
+        if rel_str not in manifest_files:
+            orphaned.append(session_path.name)
+
+    if orphaned:
+        pytest.fail(
+            f"Found {len(orphaned)} session files without manifest entries:\n" +
+            "\n".join(f"  - {o}" for o in orphaned) +
+            "\n\nRun 'atdd session sync' to update manifest."
+        )
+
+
+@pytest.mark.coach
+def test_manifest_status_matches_session_frontmatter(manifest: Optional[Dict]):
+    """
+    SPEC-COACH-MANIFEST-004: Manifest status matches session frontmatter status
+
+    Given: Session manifest with status fields
+    When: Comparing to session file frontmatter
+    Then: Status values match (or warning on mismatch)
+    """
+    if manifest is None:
+        pytest.skip("No manifest file found")
+
+    sessions_dir = manifest.get("sessions_dir", "atdd-sessions")
+    mismatches = []
+
+    for entry in manifest.get("sessions", []):
+        file_path = entry.get("file", "")
+        session_id = entry.get("id", "")
+        manifest_status = entry.get("status", "UNKNOWN").upper()
+
+        # Load session file
+        full_path = REPO_ROOT / sessions_dir / file_path
+        if not full_path.exists():
+            continue
+
+        frontmatter = parse_session_frontmatter(full_path)
+        if frontmatter is None:
+            continue
+
+        # Compare status
+        file_status = str(frontmatter.get("status", "UNKNOWN")).upper()
+        # Handle status with extra info (e.g., "ACTIVE - working on X")
+        file_status_word = file_status.split()[0] if file_status else "UNKNOWN"
+
+        if file_status_word != manifest_status:
+            mismatches.append(
+                f"SESSION-{session_id}: manifest={manifest_status}, file={file_status_word}"
+            )
+
+    if mismatches:
+        # Warn but don't fail - mismatches may be intentional during transitions
+        print(
+            f"\n⚠️  Found {len(mismatches)} status mismatches between manifest and files:\n" +
+            "\n".join(f"  - {m}" for m in mismatches) +
+            "\n\nRun 'atdd session sync' to reconcile."
+        )
+
+
+@pytest.mark.coach
+def test_manifest_type_matches_session_frontmatter(manifest: Optional[Dict]):
+    """
+    SPEC-COACH-MANIFEST-005: Manifest type matches session frontmatter type
+
+    Given: Session manifest with type fields
+    When: Comparing to session file frontmatter
+    Then: Type values match
+    """
+    if manifest is None:
+        pytest.skip("No manifest file found")
+
+    sessions_dir = manifest.get("sessions_dir", "atdd-sessions")
+    mismatches = []
+
+    for entry in manifest.get("sessions", []):
+        file_path = entry.get("file", "")
+        session_id = entry.get("id", "")
+        manifest_type = entry.get("type", "unknown").lower()
+
+        # Load session file
+        full_path = REPO_ROOT / sessions_dir / file_path
+        if not full_path.exists():
+            continue
+
+        frontmatter = parse_session_frontmatter(full_path)
+        if frontmatter is None:
+            continue
+
+        # Compare type
+        file_type = str(frontmatter.get("type", "unknown")).lower()
+
+        if file_type != manifest_type:
+            mismatches.append(
+                f"SESSION-{session_id}: manifest={manifest_type}, file={file_type}"
+            )
+
+    if mismatches:
+        pytest.fail(
+            f"Found {len(mismatches)} type mismatches between manifest and files:\n" +
+            "\n".join(f"  - {m}" for m in mismatches)
+        )
+
+
+@pytest.mark.coach
+def test_manifest_ids_are_unique(manifest: Optional[Dict]):
+    """
+    SPEC-COACH-MANIFEST-006: Manifest session IDs are unique
+
+    Given: Session manifest
+    When: Checking session IDs
+    Then: No duplicate IDs exist
+    """
+    if manifest is None:
+        pytest.skip("No manifest file found")
+
+    ids = {}
+    duplicates = []
+
+    for entry in manifest.get("sessions", []):
+        session_id = entry.get("id", "")
+        file_path = entry.get("file", "")
+
+        if session_id in ids:
+            duplicates.append(f"id={session_id}: {ids[session_id]} AND {file_path}")
+        else:
+            ids[session_id] = file_path
+
+    if duplicates:
+        pytest.fail(
+            f"Found duplicate session IDs in manifest:\n" +
+            "\n".join(f"  - {d}" for d in duplicates)
+        )
+
+
+@pytest.mark.coach
+def test_manifest_file_ids_match_filename(manifest: Optional[Dict]):
+    """
+    SPEC-COACH-MANIFEST-007: Manifest entry ID matches filename
+
+    Given: Session manifest entries
+    When: Comparing ID to filename pattern
+    Then: Entry ID matches SESSION-{ID}-* in filename
+    """
+    if manifest is None:
+        pytest.skip("No manifest file found")
+
+    mismatches = []
+
+    for entry in manifest.get("sessions", []):
+        session_id = entry.get("id", "")
+        file_path = entry.get("file", "")
+
+        # Extract ID from filename
+        filename = Path(file_path).name
+        file_id = get_session_id_from_filename(filename)
+
+        if file_id and file_id != session_id:
+            mismatches.append(
+                f"id={session_id} but filename has id={file_id}: {file_path}"
+            )
+
+    if mismatches:
+        pytest.fail(
+            f"Found {len(mismatches)} ID mismatches between manifest and filenames:\n" +
+            "\n".join(f"  - {m}" for m in mismatches)
+        )
+
+
+# ============================================================================
+# Summary
+# ============================================================================
+
+@pytest.mark.coach
+def test_manifest_alignment_summary(manifest: Optional[Dict], session_files: List[Path]):
+    """
+    Generate a summary of manifest alignment.
+
+    This test always passes but prints a summary.
+    """
+    if manifest is None:
+        print("\n⚠️  No manifest file found. Run 'atdd init' to create one.")
+        return
+
+    manifest_count = len(manifest.get("sessions", []))
+    file_count = len(session_files)
+
+    print("\n" + "=" * 60)
+    print("SESSION MANIFEST ALIGNMENT SUMMARY")
+    print("=" * 60)
+    print(f"\nManifest entries: {manifest_count}")
+    print(f"Session files:    {file_count}")
+
+    if manifest_count == file_count:
+        print("\n✅ Manifest and files are aligned")
+    else:
+        print(f"\n⚠️  Difference: {abs(manifest_count - file_count)} entries")
+
+    # Status breakdown
+    statuses = {}
+    for entry in manifest.get("sessions", []):
+        status = entry.get("status", "UNKNOWN")
+        statuses[status] = statuses.get(status, 0) + 1
+
+    print("\nBy Status:")
+    for status, count in sorted(statuses.items()):
+        print(f"  {status}: {count}")
+
+    print("\n" + "=" * 60)

atdd/coach/validators/test_session_validation.py

@@ -21,6 +21,7 @@ from typing import List, Dict, Optional, Set, Tuple, Any
 import yaml
 
 from atdd.coach.utils.repo import find_repo_root
+from atdd.coach.utils.config import load_atdd_config
 
 # ============================================================================
 # Configuration
@@ -498,6 +499,166 @@ def test_session_has_gate_commands(active_session_files: List[Path]):
         pytest.fail(f"Missing gate commands:\n" + "\n".join(f"  - {m}" for m in missing))
 
 
+# ============================================================================
+# Gate Command Prefix Validation
+# ============================================================================
+
+# Default allowed command prefixes
+DEFAULT_GATE_COMMAND_PREFIXES = [
+    "atdd",
+    "pytest",
+    "python",
+    "npm",
+    "supabase",
+    "cd",  # Allow cd for directory changes before commands
+]
+
+
+@pytest.fixture
+def coach_config() -> Dict[str, Any]:
+    """Load coach configuration from .atdd/config.yaml."""
+    config = load_atdd_config(REPO_ROOT)
+    return config.get("coach", {})
+
+
+@pytest.fixture
+def gate_command_prefixes(coach_config: Dict[str, Any]) -> List[str]:
+    """Get allowed gate command prefixes from config."""
+    return coach_config.get(
+        "session_gate_command_prefixes",
+        DEFAULT_GATE_COMMAND_PREFIXES
+    )
+
+
+@pytest.fixture
+def gate_command_exceptions(coach_config: Dict[str, Any]) -> List[str]:
+    """Get session files exempt from gate command validation."""
+    return coach_config.get("session_gate_command_exceptions", [])
+
+
+def extract_gate_commands(body: str) -> List[str]:
+    """
+    Extract commands from code blocks in session body.
+
+    Returns list of command strings found in ```bash or ```shell blocks.
+    """
+    import re
+
+    commands = []
+
+    # Match code blocks with bash/shell language hint
+    code_block_pattern = re.compile(
+        r"```(?:bash|shell|sh)?\n(.*?)```",
+        re.DOTALL
+    )
+
+    for match in code_block_pattern.finditer(body):
+        block_content = match.group(1)
+        # Extract individual commands (non-empty lines)
+        for line in block_content.strip().split('\n'):
+            line = line.strip()
+            # Skip comments and empty lines
+            if line and not line.startswith('#'):
+                commands.append(line)
+
+    return commands
+
+
+def command_matches_prefix(command: str, prefixes: List[str]) -> bool:
+    """
+    Check if command starts with an allowed prefix.
+
+    Handles:
+    - Direct prefix match: "pytest tests/"
+    - Path prefix: "./atdd/atdd.py" matches "atdd"
+    - Chained commands: "cd foo && pytest" (each part checked)
+    """
+    import re
+
+    # Split on && and || to check each command part
+    parts = re.split(r'\s*(?:&&|\|\|)\s*', command)
+
+    for part in parts:
+        part = part.strip()
+        if not part:
+            continue
+
+        # Check direct prefix match
+        matched = False
+        for prefix in prefixes:
+            # Direct match
+            if part.startswith(prefix + " ") or part == prefix:
+                matched = True
+                break
+            # Path match (e.g., ./atdd/atdd.py matches "atdd")
+            if "/" in part:
+                cmd_name = part.split()[0].split("/")[-1]
+                # Remove .py extension
+                if cmd_name.endswith(".py"):
+                    cmd_name = cmd_name[:-3]
+                if cmd_name == prefix:
+                    matched = True
+                    break
+
+        if not matched:
+            return False
+
+    return True
+
+
+def test_session_gate_commands_use_canonical_prefixes(
+    active_session_files: List[Path],
+    gate_command_prefixes: List[str],
+    gate_command_exceptions: List[str]
+):
+    """
+    Test that gate commands use canonical command prefixes.
+
+    Given: Active session files with gate commands
+    When: Checking command prefixes in code blocks
+    Then: All commands start with allowed prefixes from config
+
+    Configurable via .atdd/config.yaml:
+        coach:
+          session_gate_command_prefixes: ["atdd", "pytest", "npm"]
+          session_gate_command_exceptions: ["SESSION-00-*.md"]
+    """
+    import fnmatch
+
+    violations = []
+
+    for f in active_session_files:
+        # Check if file is exempt
+        is_exempt = any(
+            fnmatch.fnmatch(f.name, pattern)
+            for pattern in gate_command_exceptions
+        )
+        if is_exempt:
+            continue
+
+        parsed = parse_session_file(f)
+        body = parsed["body"]
+
+        commands = extract_gate_commands(body)
+
+        for cmd in commands:
+            if not command_matches_prefix(cmd, gate_command_prefixes):
+                # Extract first word for clearer error message
+                first_word = cmd.split()[0] if cmd.split() else cmd
+                violations.append(
+                    f"{f.name}: '{first_word}...' not in allowed prefixes"
+                )
+
+    if violations:
+        pytest.fail(
+            f"Found {len(violations)} gate commands with non-canonical prefixes:\n" +
+            "\n".join(f"  - {v}" for v in violations[:20]) +
+            (f"\n  ... and {len(violations) - 20} more" if len(violations) > 20 else "") +
+            f"\n\nAllowed prefixes: {gate_command_prefixes}\n"
+            "Configure in .atdd/config.yaml: coach.session_gate_command_prefixes"
+        )
+
+
 # ============================================================================
 # Pre-Implementation Gate
 # ============================================================================

{atdd-0.7.2.dist-info → atdd-0.7.3.dist-info}/METADATA

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: atdd
-Version: 0.7.2
+Version: 0.7.3
 Summary: ATDD Platform - Acceptance Test Driven Development toolkit
 License: MIT
 Requires-Python: >=3.10

{atdd-0.7.2.dist-info → atdd-0.7.3.dist-info}/RECORD

@@ -26,7 +26,7 @@ atdd/coach/conventions/naming.convention.yaml,sha256=GHqymDducgMxcMmSUwoKE6xXGEv
 atdd/coach/conventions/session.convention.yaml,sha256=Z4hCFNjOizgDXTRVqNZ4VaG40TIFF31zDkMZdA-xrvI,26084
 atdd/coach/overlays/__init__.py,sha256=2lMiMSgfLJ3YHLpbzNI5B88AdQxiMEwjIfsWWb8t3To,123
 atdd/coach/overlays/claude.md,sha256=33mhpqhmsRhCtdWlU7cMXAJDsaVra9uBBK8URV8OtQA,101
-atdd/coach/schemas/config.schema.json,sha256=47cFGE5juBv9ewhtgrNir4b6I9imIIo8VjoD9yvASf4,4578
+atdd/coach/schemas/config.schema.json,sha256=OAxiJizlpmO-A_Mi-InSKpLr4AQ9t8u9lQBrPU81q2c,5605
 atdd/coach/schemas/manifest.schema.json,sha256=WO13-YF_FgH1awh96khCtk-112b6XSC24anlY3B7GjY,2885
 atdd/coach/templates/ATDD.md,sha256=JTSPm-J2Xr6XfPodDCxnVyrKoMf2eHF2rlFw8ft6RLQ,13277
 atdd/coach/templates/SESSION-TEMPLATE.md,sha256=uuvSR-OBVb1QrmU5HcS0iO9ohng0o2nVg-qaAe76HY4,9894
@@ -43,7 +43,9 @@ atdd/coach/validators/shared_fixtures.py,sha256=Ia3B2fUW-aKibwVPF6RnRemtu3R_Dfb-
 atdd/coach/validators/test_enrich_wagon_registry.py,sha256=WeTwYJqoNY6mEYc-QAvQo7YVagSOjaNKxB6Q6dpWqIM,6561
 atdd/coach/validators/test_registry.py,sha256=ffN70yA_1xxL3R8gdpGbY2M8dQXyuajIZhBZ-ylNiNs,17845
 atdd/coach/validators/test_release_versioning.py,sha256=B40DfbtrSGguPc537zXmjT75hhySfocWLzJWqOKZQcU,5678
-atdd/coach/validators/test_session_validation.py,sha256=0VszXtFwRTO04b5CxDPO3klk0VfiqlpdbNpshjMn-qU,39079
+atdd/coach/validators/test_session_archive_status.py,sha256=w-LHy-1oB2Dvv-SRF8L9TOzjdxHI3d90SiWaHXV-s8Y,10676
+atdd/coach/validators/test_session_manifest_alignment.py,sha256=5BHVIxY8ToU5A9gIg_jNqa3LyC2Ty6Br-Y5TlwzMDJg,11924
+atdd/coach/validators/test_session_validation.py,sha256=8qvW5lnCNlawzhLdP9vrE89synEscVLGuJnpS7WqYC4,43981
 atdd/coach/validators/test_traceability.py,sha256=qTyobt41VBiCr6xRN2C7BPtGYvk_2poVQIe814Blt8E,15977
 atdd/coach/validators/test_train_registry.py,sha256=YDnmC1MP4TfAMZzyldh0hfHMo7LY27DZ2GXmhQx7vds,6021
 atdd/coach/validators/test_update_feature_paths.py,sha256=zOKVDgEIpncSJwDh_shyyou5Pu-Ai7Z_XgF8zAbQVTA,4528
@@ -201,9 +203,9 @@ atdd/tester/validators/test_train_frontend_e2e.py,sha256=fpfUwTbAWzuqxbVKoaFw-ab
 atdd/tester/validators/test_train_frontend_python.py,sha256=KK2U3oNFWLyBK7YHC0fU7shR05k93gVcO762AI8Q3pw,9018
 atdd/tester/validators/test_typescript_test_naming.py,sha256=E-TyGv_GVlTfsbyuxrtv9sOWSZS_QcpH6rrJFbWoeeU,11280
 atdd/tester/validators/test_typescript_test_structure.py,sha256=eV89SD1RaKtchBZupqhnJmaruoROosf3LwB4Fwe4UJI,2612
-atdd-0.7.2.dist-info/licenses/LICENSE,sha256=OXLcl0T2SZ8Pmy2_dmlvKuetivmyPd5m1q-Gyd-zaYY,35149
-atdd-0.7.2.dist-info/METADATA,sha256=_Rvqn06owPOazCQlN34cO_5tPLWZipNsRU0yfUABKrU,8716
-atdd-0.7.2.dist-info/WHEEL,sha256=wUyA8OaulRlbfwMtmQsvNngGrxQHAvkKcvRmdizlJi0,92
-atdd-0.7.2.dist-info/entry_points.txt,sha256=-C3yrA1WQQfN3iuGmSzPapA5cKVBEYU5Q1HUffSJTbY,38
-atdd-0.7.2.dist-info/top_level.txt,sha256=VKkf6Uiyrm4RS6ULCGM-v8AzYN8K2yg8SMqwJLoO-xs,5
-atdd-0.7.2.dist-info/RECORD,,
+atdd-0.7.3.dist-info/licenses/LICENSE,sha256=OXLcl0T2SZ8Pmy2_dmlvKuetivmyPd5m1q-Gyd-zaYY,35149
+atdd-0.7.3.dist-info/METADATA,sha256=zGebjwsy2XRAV0Uh7ZzY8A0S-AtC5HvHtLzBrH0HTKM,8716
+atdd-0.7.3.dist-info/WHEEL,sha256=wUyA8OaulRlbfwMtmQsvNngGrxQHAvkKcvRmdizlJi0,92
+atdd-0.7.3.dist-info/entry_points.txt,sha256=-C3yrA1WQQfN3iuGmSzPapA5cKVBEYU5Q1HUffSJTbY,38
+atdd-0.7.3.dist-info/top_level.txt,sha256=VKkf6Uiyrm4RS6ULCGM-v8AzYN8K2yg8SMqwJLoO-xs,5
+atdd-0.7.3.dist-info/RECORD,,