moai-adk 0.9.0__py3-none-any.whl → 0.15.1__py3-none-any.whl

moai_adk/templates/.claude/hooks/alfred/session_end__cleanup.py

@@ -1,5 +1,5 @@
 #!/usr/bin/env python3
-# @CODE:HOOKS-CLARITY-001 | SPEC: Individual hook files for better UX
+# @CODE:HOOKS-CLARITY-CLEAN | SPEC: Individual hook files for better UX
 """SessionEnd Hook: Session Cleanup and Finalization
 
 Claude Code Event: SessionEnd
@@ -10,11 +10,13 @@ Output: Continue execution (currently a stub for future enhancements)
 """
 
 import json
-import signal
 import sys
 from pathlib import Path
 from typing import Any
 
+from utils.timeout import CrossPlatformTimeout
+from utils.timeout import TimeoutError as PlatformTimeoutError
+
 # Setup import path for shared modules
 HOOKS_DIR = Path(__file__).parent
 SHARED_DIR = HOOKS_DIR / "shared"
@@ -24,16 +26,6 @@ if str(SHARED_DIR) not in sys.path:
 from handlers import handle_session_end
 
 
-class HookTimeoutError(Exception):
-    """Hook execution timeout exception"""
-    pass
-
-
-def _timeout_handler(signum, frame):
-    """Signal handler for 5-second timeout"""
-    raise HookTimeoutError("Hook execution exceeded 5-second timeout")
-
-
 def main() -> None:
     """Main entry point for SessionEnd hook
 
@@ -48,8 +40,8 @@ def main() -> None:
         1: Error (timeout, JSON parse failure, handler exception)
     """
     # Set 5-second timeout
-    signal.signal(signal.SIGALRM, _timeout_handler)
-    signal.alarm(5)
+    timeout = CrossPlatformTimeout(5)
+    timeout.start()
 
     try:
         # Read JSON payload from stdin
@@ -63,11 +55,11 @@ def main() -> None:
         print(json.dumps(result.to_dict()))
         sys.exit(0)
 
-    except HookTimeoutError:
+    except PlatformTimeoutError:
         # Timeout - return minimal valid response
         timeout_response: dict[str, Any] = {
             "continue": True,
-            "systemMessage": "⚠️ SessionEnd cleanup timeout - session ending anyway"
+            "systemMessage": "⚠️ SessionEnd cleanup timeout - session ending anyway",
         }
         print(json.dumps(timeout_response))
         print("SessionEnd hook timeout after 5 seconds", file=sys.stderr)
@@ -77,7 +69,7 @@ def main() -> None:
         # JSON parse error
         error_response: dict[str, Any] = {
             "continue": True,
-            "hookSpecificOutput": {"error": f"JSON parse error: {e}"}
+            "hookSpecificOutput": {"error": f"JSON parse error: {e}"},
         }
         print(json.dumps(error_response))
         print(f"SessionEnd JSON parse error: {e}", file=sys.stderr)
@@ -87,7 +79,7 @@ def main() -> None:
         # Unexpected error
         error_response: dict[str, Any] = {
             "continue": True,
-            "hookSpecificOutput": {"error": f"SessionEnd error: {e}"}
+            "hookSpecificOutput": {"error": f"SessionEnd error: {e}"},
         }
         print(json.dumps(error_response))
         print(f"SessionEnd unexpected error: {e}", file=sys.stderr)
@@ -95,7 +87,7 @@ def main() -> None:
 
     finally:
         # Always cancel alarm
-        signal.alarm(0)
+        timeout.cancel()
 
 
 if __name__ == "__main__":

moai_adk/templates/.claude/hooks/alfred/session_start__show_project_info.py

@@ -10,11 +10,13 @@ Output: System message with formatted project summary
 """
 
 import json
-import signal
 import sys
 from pathlib import Path
 from typing import Any
 
+from utils.timeout import CrossPlatformTimeout
+from utils.timeout import TimeoutError as PlatformTimeoutError
+
 # Setup import path for shared modules
 HOOKS_DIR = Path(__file__).parent
 SHARED_DIR = HOOKS_DIR / "shared"
@@ -24,16 +26,6 @@ if str(SHARED_DIR) not in sys.path:
 from handlers import handle_session_start
 
 
-class HookTimeoutError(Exception):
-    """Hook execution timeout exception"""
-    pass
-
-
-def _timeout_handler(signum, frame):
-    """Signal handler for 5-second timeout"""
-    raise HookTimeoutError("Hook execution exceeded 5-second timeout")
-
-
 def main() -> None:
     """Main entry point for SessionStart hook
 
@@ -48,8 +40,8 @@ def main() -> None:
         1: Error (timeout, JSON parse failure, handler exception)
     """
     # Set 5-second timeout
-    signal.signal(signal.SIGALRM, _timeout_handler)
-    signal.alarm(5)
+    timeout = CrossPlatformTimeout(5)
+    timeout.start()
 
     try:
         # Read JSON payload from stdin
@@ -63,11 +55,11 @@ def main() -> None:
         print(json.dumps(result.to_dict()))
         sys.exit(0)
 
-    except HookTimeoutError:
+    except PlatformTimeoutError:
         # Timeout - return minimal valid response
         timeout_response: dict[str, Any] = {
             "continue": True,
-            "systemMessage": "⚠️ Session start timeout - continuing without project info"
+            "systemMessage": "⚠️ Session start timeout - continuing without project info",
         }
         print(json.dumps(timeout_response))
         print("SessionStart hook timeout after 5 seconds", file=sys.stderr)
@@ -77,7 +69,7 @@ def main() -> None:
         # JSON parse error
         error_response: dict[str, Any] = {
             "continue": True,
-            "hookSpecificOutput": {"error": f"JSON parse error: {e}"}
+            "hookSpecificOutput": {"error": f"JSON parse error: {e}"},
         }
         print(json.dumps(error_response))
         print(f"SessionStart JSON parse error: {e}", file=sys.stderr)
@@ -87,7 +79,7 @@ def main() -> None:
         # Unexpected error
         error_response: dict[str, Any] = {
             "continue": True,
-            "hookSpecificOutput": {"error": f"SessionStart error: {e}"}
+            "hookSpecificOutput": {"error": f"SessionStart error: {e}"},
         }
         print(json.dumps(error_response))
         print(f"SessionStart unexpected error: {e}", file=sys.stderr)
@@ -95,7 +87,7 @@ def main() -> None:
 
     finally:
         # Always cancel alarm
-        signal.alarm(0)
+        timeout.cancel()
 
 
 if __name__ == "__main__":

moai_adk/templates/.claude/hooks/alfred/shared/core/__init__.py

@@ -156,8 +156,8 @@ class HookResult:
             "continue": self.continue_execution,
             "hookSpecificOutput": {
                 "hookEventName": "UserPromptSubmit",
-                "additionalContext": context_str
-            }
+                "additionalContext": context_str,
+            },
         }
 
 

moai_adk/templates/.claude/hooks/alfred/shared/core/checkpoint.py

@@ -56,7 +56,7 @@ def detect_risky_operation(tool_name: str, tool_args: dict[str, Any], cwd: str)
 
     Risky Operations:
         - Bash tool: rm -rf, git merge, git reset --hard, git rebase, script execution
-        - Edit/Write tool: CLAUDE.md, config.json, .moai/memory/*.md
+        - Edit/Write tool: CLAUDE.md, config.json, .claude/skills/*.md
         - MultiEdit tool: Edit ≥10 items File simultaneously
         - Script execution: Python, Node, Java, Go, Rust, Dart, Swift, Kotlin, Shell scripts
 
@@ -98,8 +98,8 @@ def detect_risky_operation(tool_name: str, tool_args: dict[str, Any], cwd: str)
         critical_files = [
             "CLAUDE.md",
             "config.json",
-            ".moai/memory/development-guide.md",
-            ".moai/memory/spec-metadata.md",
+            ".claude/skills/moai-alfred-dev-guide/reference.md",
+            ".claude/skills/moai-alfred-spec-metadata-extended/reference.md",
             ".moai/config.json",
         ]
 

moai_adk/templates/.claude/hooks/alfred/shared/core/context.py

@@ -22,13 +22,13 @@ def get_jit_context(prompt: str, cwd: str) -> list[str]:
         If there is no matching pattern or file, an empty list []
 
     Patterns:
-        - "/alfred:1-plan" → .moai/memory/spec-metadata.md
-        - "/alfred:2-run" → .moai/memory/development-guide.md
+        - "/alfred:1-plan" → .claude/skills/moai-alfred-spec-metadata-extended/reference.md
+        - "/alfred:2-run" → .claude/skills/moai-alfred-dev-guide/reference.md
         - "test" → tests/ (if directory exists)
 
     Examples:
         >>> get_jit_context("/alfred:1-plan", "/project")
-        ['.moai/memory/spec-metadata.md']
+        ['.claude/skills/moai-alfred-spec-metadata-extended/reference.md']
         >>> get_jit_context("implement test", "/project")
         ['tests/']
         >>> get_jit_context("unknown", "/project")
@@ -49,8 +49,8 @@ def get_jit_context(prompt: str, cwd: str) -> list[str]:
 
     # Pattern matching
     patterns = {
-        "/alfred:1-plan": [".moai/memory/spec-metadata.md"],
-        "/alfred:2-run": [".moai/memory/development-guide.md"],
+        "/alfred:1-plan": [".claude/skills/moai-alfred-spec-metadata-extended/reference.md"],
+        "/alfred:2-run": [".claude/skills/moai-alfred-dev-guide/reference.md"],
         "test": ["tests/"],
     }
 

moai_adk/templates/.claude/hooks/alfred/shared/core/project.py

@@ -70,6 +70,7 @@ def find_project_root(start_path: str | Path = ".") -> Path:
 
 class TimeoutError(Exception):
     """Signal-based timeout exception"""
+
     pass
 
 
@@ -86,6 +87,7 @@ def timeout_handler(seconds: int):
     Raises:
         TimeoutError: If operation exceeds timeout
     """
+
     def _handle_timeout(signum, frame):
         raise TimeoutError(f"Operation timed out after {seconds} seconds")
 
@@ -427,19 +429,10 @@ def get_version_check_config(cwd: str) -> dict[str, Any]:
         - REFACTOR: Add validation and error handling
     """
     # TTL mapping by frequency
-    TTL_BY_FREQUENCY = {
-        "always": 0,
-        "daily": 24,
-        "weekly": 168,
-        "never": float('inf')
-    }
+    ttl_by_frequency = {"always": 0, "daily": 24, "weekly": 168, "never": float("inf")}
 
     # Default configuration
-    defaults = {
-        "enabled": True,
-        "frequency": "daily",
-        "cache_ttl_hours": 24
-    }
+    defaults = {"enabled": True, "frequency": "daily", "cache_ttl_hours": 24}
 
     # Find project root to ensure we read config from correct location
     project_root = find_project_root(cwd)
@@ -461,21 +454,17 @@ def get_version_check_config(cwd: str) -> dict[str, Any]:
         frequency = moai_config.get("update_check_frequency", defaults["frequency"])
 
         # Validate frequency
-        if frequency not in TTL_BY_FREQUENCY:
+        if frequency not in ttl_by_frequency:
             frequency = defaults["frequency"]
 
         # Calculate TTL from frequency
-        cache_ttl_hours = TTL_BY_FREQUENCY[frequency]
+        cache_ttl_hours = ttl_by_frequency[frequency]
 
         # Allow explicit cache_ttl_hours override
         if "cache_ttl_hours" in version_check_config:
             cache_ttl_hours = version_check_config["cache_ttl_hours"]
 
-        return {
-            "enabled": enabled,
-            "frequency": frequency,
-            "cache_ttl_hours": cache_ttl_hours
-        }
+        return {"enabled": enabled, "frequency": frequency, "cache_ttl_hours": cache_ttl_hours}
 
     except (OSError, json.JSONDecodeError, KeyError):
         # Config read or parse error - return defaults
@@ -570,36 +559,42 @@ def is_major_version_change(current: str, latest: str) -> bool:
 def get_package_version_info(cwd: str = ".") -> dict[str, Any]:
     """Check MoAI-ADK current and latest version with caching and offline support.
 
+    ⭐ CRITICAL GUARANTEE: This function ALWAYS returns the current installed version.
+    Network failures, cache issues, and timeouts NEVER result in "unknown" version.
+
     Execution flow:
-    1. Try to load from cache (< 50ms)
-    2. If cache invalid, check network
-    3. If network available, query PyPI
-    4. If network unavailable, return current version only
-    5. Save result to cache for next time
+    1. Get current installed version (ALWAYS succeeds) ← CRITICAL
+    2. Build minimal result with current version
+    3. Try to load from cache (< 50ms) - optional enhancement
+    4. If cache valid, return cached latest info
+    5. If cache invalid/miss, optionally query PyPI - optional enhancement
+    6. Save result to cache for next time - optional
 
     Args:
         cwd: Project root directory (for cache location)
 
     Returns:
         dict with keys:
-            - "current": Current installed version
-            - "latest": Latest version available on PyPI
+            - "current": Current installed version (ALWAYS valid, never empty)
+            - "latest": Latest version available on PyPI (may be "unknown")
             - "update_available": Boolean indicating if update is available
             - "upgrade_command": Recommended upgrade command (if update available)
-            - "release_notes_url": URL to release notes (Phase 3)
-            - "is_major_update": Boolean indicating major version change (Phase 3)
+            - "release_notes_url": URL to release notes
+            - "is_major_update": Boolean indicating major version change
 
-    Note:
-        - Cache hit (< 24 hours): Returns in ~20ms, no network access
-        - Cache miss + online: Query PyPI (1s timeout), cache result
-        - Cache miss + offline: Return current version only (~100ms)
-        - Offline + cached: Return from cache in ~20ms
+    Guarantees:
+        - Cache hit (< 24 hours): Returns in ~20ms, no network access ✓
+        - Cache miss + online: Query PyPI (1s timeout), cache result ✓
+        - Cache miss + offline: Return current version only (~100ms) ✓
+        - Network timeout: Returns current + "unknown" latest (~50ms) ✓
+        - Any exception: Always returns current version ✓
 
     TDD History:
         - RED: 5 test scenarios (network detection, cache integration, offline mode)
         - GREEN: Integrate VersionCache with network detection
         - REFACTOR: Extract cache directory constant, improve error handling
         - Phase 3: Add release_notes_url and is_major_update fields (@CODE:VERSION-INTEGRATE-FIELDS-001)
+        - Phase 4: CRITICAL FIX - Always guarantee current version return (@CODE:VERSION-ALWAYS-VALID-001)
     """
     import importlib.util
     import urllib.error
@@ -613,13 +608,13 @@ def get_package_version_info(cwd: str = ".") -> dict[str, Any]:
         if spec and spec.loader:
             version_cache_module = importlib.util.module_from_spec(spec)
             spec.loader.exec_module(version_cache_module)
-            VersionCache = version_cache_module.VersionCache
+            version_cache_class = version_cache_module.VersionCache
         else:
             # Skip caching if module can't be loaded
-            VersionCache = None
+            version_cache_class = None
     except (ImportError, OSError):
         # Graceful degradation: skip caching on import errors
-        VersionCache = None
+        version_cache_class = None
 
     # 1. Find project root (ensure cache is always in correct location)
     # This prevents creating .moai/cache in wrong locations when hooks run
@@ -628,7 +623,7 @@ def get_package_version_info(cwd: str = ".") -> dict[str, Any]:
 
     # 2. Initialize cache (skip if VersionCache couldn't be imported)
     cache_dir = project_root / CACHE_DIR_NAME
-    version_cache = VersionCache(cache_dir) if VersionCache else None
+    version_cache = version_cache_class(cache_dir) if version_cache_class else None
 
     # 2. Get current installed version first (needed for cache validation)
     current_version = "unknown"
@@ -641,7 +636,7 @@ def get_package_version_info(cwd: str = ".") -> dict[str, Any]:
             "current": "dev",
             "latest": "unknown",
             "update_available": False,
-            "upgrade_command": ""
+            "upgrade_command": "",
         }
 
     # 3. Try to load from cache (fast path with version validation)
@@ -664,7 +659,7 @@ def get_package_version_info(cwd: str = ".") -> dict[str, Any]:
         "current": current_version,
         "latest": "unknown",
         "update_available": False,
-        "upgrade_command": ""
+        "upgrade_command": "",
     }
 
     # 5. Check if version check is enabled in config
@@ -695,7 +690,9 @@ def get_package_version_info(cwd: str = ".") -> dict[str, Any]:
                     release_url = project_urls.get("Changelog", "")
                     if not release_url:
                         # Fallback to GitHub releases URL pattern
-                        release_url = f"https://github.com/modu-ai/moai-adk/releases/tag/v{result['latest']}"
+                        release_url = (
+                            f"https://github.com/modu-ai/moai-adk/releases/tag/v{result['latest']}"
+                        )
                     result["release_notes_url"] = release_url
                 except (KeyError, AttributeError, TypeError):
                     result["release_notes_url"] = None
@@ -719,10 +716,12 @@ def get_package_version_info(cwd: str = ".") -> dict[str, Any]:
 
             if latest_parts > current_parts:
                 result["update_available"] = True
-                result["upgrade_command"] = f"uv pip install --upgrade moai-adk>={result['latest']}"
+                result["upgrade_command"] = "uv tool upgrade moai-adk"
 
                 # Detect major version change
-                result["is_major_update"] = is_major_version_change(result["current"], result["latest"])
+                result["is_major_update"] = is_major_version_change(
+                    result["current"], result["latest"]
+                )
             else:
                 result["is_major_update"] = False
         except (ValueError, AttributeError):

moai_adk/templates/.claude/hooks/alfred/shared/core/tags.py

@@ -38,9 +38,22 @@ from pathlib import Path
 from typing import Iterable, List, Optional
 
 DEFAULT_CODE_EXTS = (
-    ".py", ".ts", ".tsx", ".js", ".jsx", ".go", ".rs",
-    ".java", ".kt", ".rb", ".php", ".c", ".cpp", ".cs",
-    ".swift", ".scala"
+    ".py",
+    ".ts",
+    ".tsx",
+    ".js",
+    ".jsx",
+    ".go",
+    ".rs",
+    ".java",
+    ".kt",
+    ".rb",
+    ".php",
+    ".c",
+    ".cpp",
+    ".cs",
+    ".swift",
+    ".scala",
 )
 
 
@@ -88,35 +101,49 @@ def _load_rules(cwd: str) -> List[Rule]:
 
     # Defaults (ordered)
     return [
-        Rule(
-            include=[".moai/specs/**", "**/SPEC-*/spec.md"],
-            expect="@SPEC:",
-            exclude=[]
-        ),
+        Rule(include=[".moai/specs/**", "**/SPEC-*/spec.md"], expect="@SPEC:", exclude=[]),
         Rule(
             include=[
-                "**/*_test.py", "**/test_*.py", "**/*.test.ts",
-                "**/*.test.tsx", "**/*.test.js", "**/*.test.jsx",
-                "**/*.test.go", "**/*.test.rs", "**/*.spec.ts",
-                "**/*.spec.tsx", "tests/**"
+                "**/*_test.py",
+                "**/test_*.py",
+                "**/*.test.ts",
+                "**/*.test.tsx",
+                "**/*.test.js",
+                "**/*.test.jsx",
+                "**/*.test.go",
+                "**/*.test.rs",
+                "**/*.spec.ts",
+                "**/*.spec.tsx",
+                "tests/**",
             ],
             expect="@TEST:",
-            exclude=[".claude/**"]
+            exclude=[".claude/**"],
         ),
         Rule(
             include=["docs/**/*.md", "**/README.md", "**/*.api.md"],
             expect="@DOC:",
-            exclude=[".claude/**"]
+            exclude=[".claude/**"],
         ),
         Rule(
             include=["**/*"],
             expect="@CODE:",
             exclude=[
-                "tests/**", "docs/**", ".moai/**", ".claude/**",
-                "**/*.md", "**/*.json", "**/*.yml", "**/*.yaml",
-                "**/*.toml", "**/*.lock", "**/*.svg", "**/*.png",
-                "**/*.jpg", "**/*.jpeg", "**/*.gif"
-            ]
+                "tests/**",
+                "docs/**",
+                ".moai/**",
+                ".claude/**",
+                "**/*.md",
+                "**/*.json",
+                "**/*.yml",
+                "**/*.yaml",
+                "**/*.toml",
+                "**/*.lock",
+                "**/*.svg",
+                "**/*.png",
+                "**/*.jpg",
+                "**/*.jpeg",
+                "**/*.gif",
+            ],
         ),
     ]
 
@@ -147,17 +174,22 @@ def _iter_recent_changes(cwd: str) -> Iterable[Path]:
         # Staged files
         r1 = subprocess.run(
             ["git", "diff", "--name-only", "--cached"],
-            cwd=cwd, capture_output=True, text=True, timeout=1
+            cwd=cwd,
+            capture_output=True,
+            text=True,
+            timeout=1,
         )
         # Modified (unstaged) tracked files
         r2 = subprocess.run(
-            ["git", "ls-files", "-m"],
-            cwd=cwd, capture_output=True, text=True, timeout=1
+            ["git", "ls-files", "-m"], cwd=cwd, capture_output=True, text=True, timeout=1
         )
         # Untracked (other) files respecting .gitignore
         r3 = subprocess.run(
             ["git", "ls-files", "-o", "--exclude-standard"],
-            cwd=cwd, capture_output=True, text=True, timeout=1
+            cwd=cwd,
+            capture_output=True,
+            text=True,
+            timeout=1,
         )
         names = set()
         if r1.returncode == 0:

moai_adk/templates/.claude/hooks/alfred/shared/core/version_cache.py

@@ -86,7 +86,7 @@ class VersionCache:
             return False
 
         try:
-            with open(self.cache_file, 'r') as f:
+            with open(self.cache_file, "r") as f:
                 data = json.load(f)
 
             age_hours = self._calculate_age_hours(data["last_check"])
@@ -112,7 +112,7 @@ class VersionCache:
             return None
 
         try:
-            with open(self.cache_file, 'r') as f:
+            with open(self.cache_file, "r") as f:
                 return json.load(f)
         except (json.JSONDecodeError, OSError):
             # Graceful degradation on read errors
@@ -144,7 +144,7 @@ class VersionCache:
                 version_info["last_check"] = datetime.now(timezone.utc).isoformat()
 
             # Write to cache file
-            with open(self.cache_file, 'w') as f:
+            with open(self.cache_file, "w") as f:
                 json.dump(version_info, f, indent=2)
 
             return True
@@ -186,7 +186,7 @@ class VersionCache:
             return 0.0
 
         try:
-            with open(self.cache_file, 'r') as f:
+            with open(self.cache_file, "r") as f:
                 data = json.load(f)
 
             return self._calculate_age_hours(data["last_check"])