atdd 0.6.1__tar.gz → 0.7.1__tar.gz

{atdd-0.6.1/src/atdd.egg-info → atdd-0.7.1}/PKG-INFO

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

{atdd-0.6.1 → atdd-0.7.1}/pyproject.toml

@@ -4,7 +4,7 @@ build-backend = "setuptools.build_meta"
 
 [project]
 name = "atdd"
-version = "0.6.1"
+version = "0.7.1"
 description = "ATDD Platform - Acceptance Test Driven Development toolkit"
 requires-python = ">=3.10"
 readme = "README.md"
@@ -39,4 +39,6 @@ markers = [
     "tester: marks tests as tester validators",
     "platform: marks tests as platform validators",
     "security: marks tests as security validators",
+    "locale: marks tests as localization validators (Localization Manifest Spec v1)",
+    "coder: marks tests as coder phase validators",
 ]

{atdd-0.6.1 → atdd-0.7.1}/src/atdd/coach/commands/inventory.py

@@ -148,9 +148,16 @@ class RepositoryInventory:
                 by_theme[theme] += 1
 
             # Gap analysis
-            expectations = train.get("expectations", {})
-            test_fields = train.get("test", {})
-            code_fields = train.get("code", {})
+            expectations = train.get("expectations")
+            test_fields = train.get("test")
+            code_fields = train.get("code")
+
+            if not isinstance(expectations, dict):
+                expectations = {}
+            if test_fields is None:
+                test_fields = {}
+            if code_fields is None:
+                code_fields = {}
 
             # Normalize test/code to dict form
             if isinstance(test_fields, str):

{atdd-0.6.1 → atdd-0.7.1}/src/atdd/coach/schemas/config.schema.json

@@ -121,6 +121,17 @@
         }
       },
       "additionalProperties": false
+    },
+    "localization": {
+      "type": "object",
+      "description": "Localization validation settings (Localization Manifest Spec v1)",
+      "properties": {
+        "manifest": {
+          "type": "string",
+          "description": "Path to localization manifest.json relative to repo root (e.g., web/locales/manifest.json)"
+        }
+      },
+      "additionalProperties": false
     }
   },
   "required": ["version", "release"],

atdd-0.7.1/src/atdd/coach/utils/locale_phase.py

@@ -0,0 +1,97 @@
+"""
+Localization Manifest Spec v1 Rollout Phase Controller.
+
+Manages the phased rollout of localization validation rules:
+- Phase 1 (WARNINGS_ONLY): All validators emit warnings only
+- Phase 2 (TESTER_ENFORCEMENT): Tester phase validators (LOCALE-TEST-*) strict
+- Phase 3 (FULL_ENFORCEMENT): All validators including Coder (LOCALE-CODE-*) strict
+
+Usage in validators:
+    from atdd.coach.utils.locale_phase import LocalePhase, should_enforce_locale
+
+    if should_enforce_locale(LocalePhase.TESTER_ENFORCEMENT):
+        assert condition, "Error message"
+    else:
+        if not condition:
+            emit_locale_warning("LOCALE-TEST-1.1", "Warning message", LocalePhase.TESTER_ENFORCEMENT)
+"""
+
+from enum import IntEnum
+from typing import Optional
+import warnings
+
+
+class LocalePhase(IntEnum):
+    """
+    Rollout phases for Localization Manifest Spec v1.
+
+    Phases are ordered by strictness level:
+    - WARNINGS_ONLY (1): All new validators emit warnings, no assertions
+    - TESTER_ENFORCEMENT (2): Tester phase validators (LOCALE-TEST-*) strict
+    - FULL_ENFORCEMENT (3): All validators including Coder (LOCALE-CODE-*) strict
+    """
+    WARNINGS_ONLY = 1
+    TESTER_ENFORCEMENT = 2
+    FULL_ENFORCEMENT = 3
+
+
+# Current rollout phase - update this to advance through phases
+CURRENT_LOCALE_PHASE = LocalePhase.WARNINGS_ONLY
+
+
+def should_enforce_locale(validator_phase: LocalePhase) -> bool:
+    """
+    Check if a locale validator should enforce strict mode.
+
+    Args:
+        validator_phase: The phase at which this validator becomes strict
+
+    Returns:
+        True if current phase >= validator_phase (should enforce)
+        False if current phase < validator_phase (should warn only)
+
+    Example:
+        # This validator becomes strict in Phase 2
+        if should_enforce_locale(LocalePhase.TESTER_ENFORCEMENT):
+            assert all_keys_match, "Keys must match reference locale"
+        else:
+            if not all_keys_match:
+                emit_locale_warning("LOCALE-TEST-1.4", "Keys don't match", LocalePhase.TESTER_ENFORCEMENT)
+    """
+    return CURRENT_LOCALE_PHASE >= validator_phase
+
+
+def get_current_locale_phase() -> LocalePhase:
+    """Get the current locale rollout phase."""
+    return CURRENT_LOCALE_PHASE
+
+
+def get_locale_phase_name(phase: Optional[LocalePhase] = None) -> str:
+    """Get human-readable name for a locale phase."""
+    phase = phase or CURRENT_LOCALE_PHASE
+    return {
+        LocalePhase.WARNINGS_ONLY: "Phase 1: Warnings Only",
+        LocalePhase.TESTER_ENFORCEMENT: "Phase 2: Tester Enforcement",
+        LocalePhase.FULL_ENFORCEMENT: "Phase 3: Full Enforcement",
+    }.get(phase, "Unknown Phase")
+
+
+def emit_locale_warning(
+    spec_id: str,
+    message: str,
+    validator_phase: LocalePhase = LocalePhase.TESTER_ENFORCEMENT
+) -> None:
+    """
+    Emit a locale validation warning with phase context.
+
+    Args:
+        spec_id: The SPEC ID (e.g., "LOCALE-TEST-1.1")
+        message: The warning message
+        validator_phase: Phase when this becomes an error
+    """
+    phase_name = get_locale_phase_name(validator_phase)
+    warnings.warn(
+        f"[{spec_id}] {message} (will become error in {phase_name})",
+        category=UserWarning,
+        stacklevel=3
+    )

{atdd-0.6.1 → atdd-0.7.1}/src/atdd/coach/validators/shared_fixtures.py

@@ -599,3 +599,52 @@ def wagon_to_train_mapping(train_files: List[Tuple[Path, Dict]]) -> Dict[str, Li
                 mapping[wagon_slug].append(train_id)
 
     return mapping
+
+
+# ============================================================================
+# LOCALIZATION FIXTURES (Localization Manifest Spec v1)
+# ============================================================================
+
+
+@pytest.fixture(scope="module")
+def locale_manifest_path(atdd_config: Dict[str, Any]) -> Optional[Path]:
+    """
+    Get path to localization manifest file from config.
+
+    Returns:
+        Path to manifest file, or None if localization not configured
+    """
+    manifest_rel = atdd_config.get("localization", {}).get("manifest")
+    if not manifest_rel:
+        return None
+    return REPO_ROOT / manifest_rel
+
+
+@pytest.fixture(scope="module")
+def locale_manifest(locale_manifest_path: Optional[Path]) -> Optional[Dict[str, Any]]:
+    """
+    Load localization manifest from configured path.
+
+    Returns:
+        Manifest dict with reference, locales, namespaces, or None if not configured
+    """
+    if locale_manifest_path is None:
+        return None
+    if not locale_manifest_path.exists():
+        return None
+
+    with open(locale_manifest_path) as f:
+        return json.load(f)
+
+
+@pytest.fixture(scope="module")
+def locales_dir(locale_manifest_path: Optional[Path]) -> Optional[Path]:
+    """
+    Get locales directory (parent of manifest file).
+
+    Returns:
+        Path to locales directory, or None if not configured
+    """
+    if locale_manifest_path is None:
+        return None
+    return locale_manifest_path.parent

atdd-0.7.1/src/atdd/coder/validators/test_i18n_runtime.py

@@ -0,0 +1,171 @@
+"""
+i18n runtime validation (Localization Manifest Spec v1).
+
+Validates that runtime code uses the centralized locale manifest:
+- LOCALE-CODE-2.1: i18nConfig.ts imports from manifest (not hardcoded arrays)
+- LOCALE-CODE-2.2: LanguageSwitcher uses shared SUPPORTED_LOCALES
+"""
+
+import re
+import pytest
+from pathlib import Path
+from typing import Optional
+
+from atdd.coach.utils.locale_phase import (
+    LocalePhase,
+    should_enforce_locale,
+    emit_locale_warning,
+)
+from atdd.coach.utils.repo import find_repo_root
+
+# Path constants
+REPO_ROOT = find_repo_root()
+WEB_DIR = REPO_ROOT / "web"
+
+
+def _find_file(base_dir: Path, *possible_paths: str) -> Optional[Path]:
+    """Find first existing file from list of possible paths."""
+    for rel_path in possible_paths:
+        full_path = base_dir / rel_path
+        if full_path.exists():
+            return full_path
+    return None
+
+
+def _read_file_content(path: Path) -> Optional[str]:
+    """Read file content, return None on error."""
+    try:
+        return path.read_text()
+    except Exception:
+        return None
+
+
+@pytest.mark.locale
+@pytest.mark.coder
+def test_i18n_config_uses_manifest(locale_manifest, locale_manifest_path):
+    """
+    LOCALE-CODE-2.1: i18nConfig.ts imports from manifest (not hardcoded arrays)
+
+    Given: Web application with i18n configuration
+    When: Checking i18nConfig.ts or i18n.ts
+    Then: Configuration imports locales from manifest or shared constant
+          NOT hardcoded locale arrays like ['en', 'es', 'fr']
+    """
+    if locale_manifest is None:
+        pytest.skip("Localization not configured")
+
+    i18n_config = _find_file(
+        WEB_DIR,
+        "src/i18nConfig.ts",
+        "src/i18n/config.ts",
+        "src/i18n.ts",
+        "src/lib/i18n.ts",
+        "src/config/i18n.ts",
+    )
+
+    if i18n_config is None:
+        pytest.skip("No i18n config file found in web/src/")
+
+    content = _read_file_content(i18n_config)
+    if content is None:
+        pytest.skip(f"Cannot read {i18n_config}")
+
+    hardcoded_array_pattern = re.compile(
+        r"(?:locales|supportedLocales|SUPPORTED_LOCALES|languages)\s*[=:]\s*\[\s*['\"][a-z]{2}",
+        re.IGNORECASE
+    )
+
+    if hardcoded_array_pattern.search(content):
+        manifest_import_patterns = [
+            r"from\s+['\"].*manifest",
+            r"import.*manifest",
+            r"require\s*\(\s*['\"].*manifest",
+            r"SUPPORTED_LOCALES",
+            r"getSupportedLocales",
+        ]
+
+        has_manifest_usage = any(
+            re.search(pattern, content, re.IGNORECASE)
+            for pattern in manifest_import_patterns
+        )
+
+        if not has_manifest_usage:
+            msg = (
+                f"i18n config has hardcoded locale array: {i18n_config.relative_to(REPO_ROOT)}\n"
+                f"  Should import from manifest.json or use shared SUPPORTED_LOCALES constant"
+            )
+            if should_enforce_locale(LocalePhase.FULL_ENFORCEMENT):
+                pytest.fail(msg)
+            else:
+                emit_locale_warning("LOCALE-CODE-2.1", msg, LocalePhase.FULL_ENFORCEMENT)
+                pytest.skip(msg)
+
+
+@pytest.mark.locale
+@pytest.mark.coder
+def test_language_switcher_uses_shared_locales(locale_manifest, locale_manifest_path):
+    """
+    LOCALE-CODE-2.2: LanguageSwitcher uses shared SUPPORTED_LOCALES
+
+    Given: Web application with language switcher component
+    When: Checking LanguageSwitcher component
+    Then: Component imports locales from shared constant or manifest
+          NOT hardcoded locale arrays
+    """
+    if locale_manifest is None:
+        pytest.skip("Localization not configured")
+
+    switcher_patterns = [
+        "src/components/LanguageSwitcher.tsx",
+        "src/components/LocaleSwitcher.tsx",
+        "src/components/ui/LanguageSwitcher.tsx",
+        "src/components/common/LanguageSwitcher.tsx",
+        "src/features/i18n/LanguageSwitcher.tsx",
+    ]
+
+    switcher_file = _find_file(WEB_DIR, *switcher_patterns)
+
+    if switcher_file is None:
+        switcher_files = list(WEB_DIR.rglob("*[Ll]anguage*[Ss]witcher*.tsx"))
+        if not switcher_files:
+            switcher_files = list(WEB_DIR.rglob("*[Ll]ocale*[Ss]witcher*.tsx"))
+        if switcher_files:
+            switcher_file = switcher_files[0]
+
+    if switcher_file is None:
+        pytest.skip("No LanguageSwitcher component found")
+
+    content = _read_file_content(switcher_file)
+    if content is None:
+        pytest.skip(f"Cannot read {switcher_file}")
+
+    hardcoded_array_pattern = re.compile(
+        r"(?:locales|languages|options)\s*[=:]\s*\[\s*(?:\{[^}]*locale[^}]*['\"][a-z]{2}|['\"][a-z]{2})",
+        re.IGNORECASE
+    )
+
+    if hardcoded_array_pattern.search(content):
+        shared_patterns = [
+            r"SUPPORTED_LOCALES",
+            r"getSupportedLocales",
+            r"from\s+['\"].*manifest",
+            r"from\s+['\"].*i18n",
+            r"from\s+['\"].*config",
+            r"useLocales",
+        ]
+
+        has_shared_usage = any(
+            re.search(pattern, content, re.IGNORECASE)
+            for pattern in shared_patterns
+        )
+
+        if not has_shared_usage:
+            msg = (
+                f"LanguageSwitcher has hardcoded locale array: {switcher_file.relative_to(REPO_ROOT)}\n"
+                f"  Should import from shared SUPPORTED_LOCALES or manifest"
+            )
+            if should_enforce_locale(LocalePhase.FULL_ENFORCEMENT):
+                pytest.fail(msg)
+            else:
+                emit_locale_warning("LOCALE-CODE-2.2", msg, LocalePhase.FULL_ENFORCEMENT)
+                pytest.skip(msg)

atdd-0.7.1/src/atdd/tester/schemas/locale_manifest.schema.json

@@ -0,0 +1,53 @@
+{
+  "$schema": "http://json-schema.org/draft-07/schema#",
+  "$id": "https://atdd.dev/schemas/tester/locale_manifest.schema.json",
+  "title": "Locale Manifest",
+  "description": "Schema for localization manifest.json - single source of truth for supported locales and namespaces (Localization Manifest Spec v1)",
+  "type": "object",
+  "required": ["reference", "locales", "namespaces"],
+  "properties": {
+    "reference": {
+      "type": "string",
+      "pattern": "^[a-z]{2}(-[A-Z]{2})?$",
+      "description": "Reference locale code (e.g., 'en', 'en-US'). All other locales must match this locale's keys."
+    },
+    "locales": {
+      "type": "array",
+      "minItems": 1,
+      "items": {
+        "type": "string",
+        "pattern": "^[a-z]{2}(-[A-Z]{2})?$"
+      },
+      "uniqueItems": true,
+      "description": "List of supported locale codes. Must include the reference locale."
+    },
+    "namespaces": {
+      "type": "array",
+      "minItems": 1,
+      "items": {
+        "type": "string",
+        "pattern": "^[a-z][a-z0-9_-]*$"
+      },
+      "uniqueItems": true,
+      "description": "Required namespaces that must exist for all locales (e.g., 'common', 'ui', 'errors')"
+    },
+    "optional_namespaces": {
+      "type": "array",
+      "items": {
+        "type": "string",
+        "pattern": "^[a-z][a-z0-9_-]*$"
+      },
+      "uniqueItems": true,
+      "description": "Optional namespaces that may not exist for all locales. If present, must match reference keys."
+    }
+  },
+  "additionalProperties": false,
+  "examples": [
+    {
+      "reference": "en",
+      "locales": ["en", "es", "fr", "de", "ja"],
+      "namespaces": ["common", "ui", "errors"],
+      "optional_namespaces": ["landing", "marketing"]
+    }
+  ]
+}