prizmkit 1.0.13 → 1.0.14

package/bundled/dev-pipeline/run-bugfix.sh

@@ -17,9 +17,14 @@ set -euo pipefail
 #   MAX_RETRIES           Max retries per bug (default: 3)
 #   SESSION_TIMEOUT       Session timeout in seconds (default: 0 = no limit)
 #   AI_CLI                AI CLI command name (auto-detected: cbc or claude)
+#   CODEBUDDY_CLI         Legacy alias for AI_CLI (deprecated, use AI_CLI instead)
+#   PRIZMKIT_PLATFORM     Force platform: 'codebuddy' or 'claude' (auto-detected)
 #   VERBOSE               Set to 1 to enable --verbose on AI CLI
 #   HEARTBEAT_INTERVAL    Heartbeat log interval in seconds (default: 30)
 #   HEARTBEAT_STALE_THRESHOLD  Heartbeat stale threshold in seconds (default: 600)
+#   LOG_CLEANUP_ENABLED   Run periodic log cleanup (default: 1)
+#   LOG_RETENTION_DAYS    Delete logs older than N days (default: 14)
+#   LOG_MAX_TOTAL_MB      Keep total logs under N MB via oldest-first cleanup (default: 1024)
 # ============================================================
 
 SCRIPT_DIR="$(cd "$(dirname "${BASH_SOURCE[0]}")" && pwd)"
@@ -31,31 +36,14 @@ MAX_RETRIES=${MAX_RETRIES:-3}
 SESSION_TIMEOUT=${SESSION_TIMEOUT:-0}
 HEARTBEAT_STALE_THRESHOLD=${HEARTBEAT_STALE_THRESHOLD:-600}
 HEARTBEAT_INTERVAL=${HEARTBEAT_INTERVAL:-30}
+LOG_CLEANUP_ENABLED=${LOG_CLEANUP_ENABLED:-1}
+LOG_RETENTION_DAYS=${LOG_RETENTION_DAYS:-14}
+LOG_MAX_TOTAL_MB=${LOG_MAX_TOTAL_MB:-1024}
 VERBOSE=${VERBOSE:-0}
 
-# AI CLI detection
-if [[ -n "${AI_CLI:-}" ]]; then
-    CLI_CMD="$AI_CLI"
-elif [[ -n "${CODEBUDDY_CLI:-}" ]]; then
-    CLI_CMD="$CODEBUDDY_CLI"
-elif command -v cbc &>/dev/null; then
-    CLI_CMD="cbc"
-elif command -v claude &>/dev/null; then
-    CLI_CMD="claude"
-else
-    echo "ERROR: No AI CLI found. Install CodeBuddy (cbc) or Claude Code (claude)." >&2
-    exit 1
-fi
-
-# Platform detection
-if [[ -n "${PRIZMKIT_PLATFORM:-}" ]]; then
-    PLATFORM="$PRIZMKIT_PLATFORM"
-elif [[ "$CLI_CMD" == *"claude"* ]]; then
-    PLATFORM="claude"
-else
-    PLATFORM="codebuddy"
-fi
-export PRIZMKIT_PLATFORM="$PLATFORM"
+# Source shared common helpers (CLI/platform detection + logs + deps)
+source "$SCRIPT_DIR/lib/common.sh"
+prizm_detect_cli_and_platform
 
 # Source shared heartbeat library
 source "$SCRIPT_DIR/lib/heartbeat.sh"
@@ -66,20 +54,6 @@ detect_stream_json_support "$CLI_CMD"
 # Bug list path (set in main, used by cleanup trap)
 BUG_LIST=""
 
-# Colors
-RED='\033[0;31m'
-GREEN='\033[0;32m'
-YELLOW='\033[1;33m'
-BLUE='\033[0;34m'
-MAGENTA='\033[0;35m'
-BOLD='\033[1m'
-NC='\033[0m'
-
-log_info()    { echo -e "${BLUE}[INFO]${NC}    $(date '+%Y-%m-%d %H:%M:%S') $*"; }
-log_warn()    { echo -e "${YELLOW}[WARN]${NC}    $(date '+%Y-%m-%d %H:%M:%S') $*"; }
-log_error()   { echo -e "${RED}[ERROR]${NC}   $(date '+%Y-%m-%d %H:%M:%S') $*"; }
-log_success() { echo -e "${GREEN}[SUCCESS]${NC} $(date '+%Y-%m-%d %H:%M:%S') $*"; }
-
 # ============================================================
 # Shared: Spawn AI CLI session and wait for result
 # ============================================================
@@ -234,13 +208,29 @@ trap cleanup SIGINT SIGTERM
 # ============================================================
 
 check_dependencies() {
-    if ! command -v jq &>/dev/null; then
-        log_error "jq is required but not installed. Install with: brew install jq"
-        exit 1
+    prizm_check_common_dependencies "$CLI_CMD"
+}
+
+run_log_cleanup() {
+    if [[ "$LOG_CLEANUP_ENABLED" != "1" ]]; then
+        return 0
     fi
-    if ! command -v python3 &>/dev/null; then
-        log_error "python3 is required but not installed."
-        exit 1
+
+    local cleanup_result
+    cleanup_result=$(python3 "$SCRIPTS_DIR/cleanup-logs.py" \
+        --state-dir "$STATE_DIR" \
+        --retention-days "$LOG_RETENTION_DAYS" \
+        --max-total-mb "$LOG_MAX_TOTAL_MB" 2>/dev/null) || {
+        log_warn "Log cleanup failed (continuing)"
+        return 0
+    }
+
+    local deleted reclaimed_kb
+    deleted=$(echo "$cleanup_result" | python3 -c "import sys,json; print(json.load(sys.stdin).get('deleted_files', 0))" 2>/dev/null || echo "0")
+    reclaimed_kb=$(echo "$cleanup_result" | python3 -c "import sys,json; print(int(json.load(sys.stdin).get('reclaimed_bytes', 0)/1024))" 2>/dev/null || echo "0")
+
+    if [[ "$deleted" -gt 0 ]]; then
+        log_info "Log cleanup: deleted $deleted files, reclaimed ${reclaimed_kb}KB"
     fi
 }
 
@@ -283,6 +273,7 @@ run_one() {
     fi
 
     check_dependencies
+    run_log_cleanup
 
     # Initialize state if needed
     if [[ ! -f "$STATE_DIR/pipeline.json" ]]; then
@@ -299,14 +290,14 @@ run_one() {
     local bug_title
     bug_title=$(python3 -c "
 import json, sys
-with open('$bug_list') as f:
+with open(sys.argv[1]) as f:
     data = json.load(f)
 for bug in data.get('bugs', []):
-    if bug.get('id') == '$bug_id':
+    if bug.get('id') == sys.argv[2]:
         print(bug.get('title', ''))
         sys.exit(0)
 sys.exit(1)
-" 2>/dev/null) || {
+" "$bug_list" "$bug_id" 2>/dev/null) || {
         log_error "Bug $bug_id not found in $bug_list"
         exit 1
     }
@@ -314,14 +305,14 @@ sys.exit(1)
     local bug_severity
     bug_severity=$(python3 -c "
 import json, sys
-with open('$bug_list') as f:
+with open(sys.argv[1]) as f:
     data = json.load(f)
 for bug in data.get('bugs', []):
-    if bug.get('id') == '$bug_id':
+    if bug.get('id') == sys.argv[2]:
         print(bug.get('severity', 'medium'))
         sys.exit(0)
 sys.exit(1)
-" 2>/dev/null) || bug_severity="medium"
+" "$bug_list" "$bug_id" 2>/dev/null) || bug_severity="medium"
 
     # Reset bug status
     python3 "$SCRIPTS_DIR/update-bug-status.py" \
@@ -376,14 +367,14 @@ sys.exit(1)
     echo -e "${BOLD}════════════════════════════════════════════════════${NC}"
     echo ""
 
-    cleanup() {
+    cleanup_single_bug() {
         echo ""
         log_warn "Interrupted. Killing session..."
         kill 0 2>/dev/null || true
         log_info "Session log: $session_dir/logs/session.log"
         exit 130
     }
-    trap cleanup SIGINT SIGTERM
+    trap cleanup_single_bug SIGINT SIGTERM
 
     _SPAWN_RESULT=""
     spawn_and_wait_session \
@@ -424,6 +415,7 @@ main() {
     fi
 
     check_dependencies
+    run_log_cleanup
 
     # Initialize pipeline state if needed
     if [[ ! -f "$STATE_DIR/pipeline.json" ]]; then
@@ -482,6 +474,7 @@ main() {
             log_success "  All bugs processed! Bug fix pipeline finished."
             log_success "  Total sessions: $session_count"
             log_success "════════════════════════════════════════════════════"
+            rm -f "$STATE_DIR/current-session.json"
             break
         fi
 
@@ -529,18 +522,22 @@ main() {
             --state-dir "$STATE_DIR" \
             --output "$bootstrap_prompt" >/dev/null 2>&1
 
-        # Track current session
+        # Track current session (atomic write via temp file)
         python3 -c "
-import json
+import json, sys, os
 from datetime import datetime
+bug_id, session_id, state_dir = sys.argv[1], sys.argv[2], sys.argv[3]
 data = {
-    'bug_id': '$bug_id',
-    'session_id': '$session_id',
+    'bug_id': bug_id,
+    'session_id': session_id,
     'started_at': datetime.utcnow().strftime('%Y-%m-%dT%H:%M:%SZ')
 }
-with open('$STATE_DIR/current-session.json', 'w') as f:
+target = os.path.join(state_dir, 'current-session.json')
+tmp = target + '.tmp'
+with open(tmp, 'w') as f:
     json.dump(data, f, indent=2)
-"
+os.replace(tmp, target)
+" "$bug_id" "$session_id" "$STATE_DIR"
 
         # Spawn session
         log_info "Spawning AI CLI session: $session_id"
@@ -580,6 +577,9 @@ show_help() {
     echo "  AI_CLI                AI CLI command name (auto-detected: cbc or claude)"
     echo "  VERBOSE               Set to 1 for verbose AI CLI output"
     echo "  HEARTBEAT_INTERVAL    Heartbeat log interval in seconds (default: 30)"
+    echo "  LOG_CLEANUP_ENABLED   Run log cleanup before execution (default: 1)"
+    echo "  LOG_RETENTION_DAYS    Delete logs older than N days (default: 14)"
+    echo "  LOG_MAX_TOTAL_MB      Keep total logs under N MB (default: 1024)"
     echo ""
     echo "Examples:"
     echo "  ./run-bugfix.sh run                                    # Run all bugs"

package/bundled/dev-pipeline/run.sh

@@ -17,12 +17,15 @@ set -euo pipefail
 # Environment Variables:
 #   MAX_RETRIES           Max retries per feature (default: 3)
 #   SESSION_TIMEOUT       Session timeout in seconds (default: 0 = no limit)
-#   AI_CLI                AI CLI command name (auto-detected: cbc or claude)
+#   AI_CLI                AI CLI command name (override; also readable from .prizmkit/config.json)
 #   CODEBUDDY_CLI         Legacy alias for AI_CLI (deprecated, use AI_CLI instead)
 #   PRIZMKIT_PLATFORM     Force platform: 'codebuddy' or 'claude' (auto-detected)
 #   VERBOSE               Set to 1 to enable --verbose on AI CLI (shows subagent output)
 #   HEARTBEAT_INTERVAL    Heartbeat log interval in seconds (default: 30)
 #   HEARTBEAT_STALE_THRESHOLD  Heartbeat stale threshold in seconds (default: 600)
+#   LOG_CLEANUP_ENABLED   Run periodic log cleanup (default: 1)
+#   LOG_RETENTION_DAYS    Delete logs older than N days (default: 14)
+#   LOG_MAX_TOTAL_MB      Keep total logs under N MB via oldest-first cleanup (default: 1024)
 # ============================================================
 
 SCRIPT_DIR="$(cd "$(dirname "${BASH_SOURCE[0]}")" && pwd)"
@@ -34,32 +37,14 @@ MAX_RETRIES=${MAX_RETRIES:-3}
 SESSION_TIMEOUT=${SESSION_TIMEOUT:-0}
 HEARTBEAT_STALE_THRESHOLD=${HEARTBEAT_STALE_THRESHOLD:-600}
 HEARTBEAT_INTERVAL=${HEARTBEAT_INTERVAL:-30}
+LOG_CLEANUP_ENABLED=${LOG_CLEANUP_ENABLED:-1}
+LOG_RETENTION_DAYS=${LOG_RETENTION_DAYS:-14}
+LOG_MAX_TOTAL_MB=${LOG_MAX_TOTAL_MB:-1024}
 VERBOSE=${VERBOSE:-0}
 
-# AI CLI detection: AI_CLI > CODEBUDDY_CLI > auto-detect > error
-if [[ -n "${AI_CLI:-}" ]]; then
-    CLI_CMD="$AI_CLI"
-elif [[ -n "${CODEBUDDY_CLI:-}" ]]; then
-    CLI_CMD="$CODEBUDDY_CLI"
-elif command -v cbc &>/dev/null; then
-    CLI_CMD="cbc"
-elif command -v claude &>/dev/null; then
-    CLI_CMD="claude"
-else
-    echo "ERROR: No AI CLI found. Install CodeBuddy (cbc) or Claude Code (claude)." >&2
-    exit 1
-fi
-
-# Platform detection
-if [[ -n "${PRIZMKIT_PLATFORM:-}" ]]; then
-    PLATFORM="$PRIZMKIT_PLATFORM"
-elif [[ "$CLI_CMD" == *"claude"* ]]; then
-    PLATFORM="claude"
-else
-    PLATFORM="codebuddy"
-fi
-
-export PRIZMKIT_PLATFORM="$PLATFORM"
+# Source shared common helpers (CLI/platform detection + logs + deps)
+source "$SCRIPT_DIR/lib/common.sh"
+prizm_detect_cli_and_platform
 
 # Source shared heartbeat library
 source "$SCRIPT_DIR/lib/heartbeat.sh"
@@ -70,19 +55,6 @@ detect_stream_json_support "$CLI_CMD"
 # Feature list path (set in main, used by cleanup trap)
 FEATURE_LIST=""
 
-# Colors for output
-RED='\033[0;31m'
-GREEN='\033[0;32m'
-YELLOW='\033[1;33m'
-BLUE='\033[0;34m'
-BOLD='\033[1m'
-NC='\033[0m'
-
-log_info()    { echo -e "${BLUE}[INFO]${NC}    $(date '+%Y-%m-%d %H:%M:%S') $*"; }
-log_warn()    { echo -e "${YELLOW}[WARN]${NC}    $(date '+%Y-%m-%d %H:%M:%S') $*"; }
-log_error()   { echo -e "${RED}[ERROR]${NC}   $(date '+%Y-%m-%d %H:%M:%S') $*"; }
-log_success() { echo -e "${GREEN}[SUCCESS]${NC} $(date '+%Y-%m-%d %H:%M:%S') $*"; }
-
 # ============================================================
 # Shared: Spawn an AI CLI session and wait for result
 # ============================================================
@@ -206,13 +178,42 @@ spawn_and_wait_session() {
             local dirty_files=""
             dirty_files=$(git -C "$project_root" status --porcelain 2>/dev/null || true)
             if [[ -n "$dirty_files" ]]; then
-                log_error "Session reported success but git working tree is not clean."
+                log_warn "Session reported success but git working tree is not clean."
                 echo "$dirty_files" | sed 's/^/  - /'
-                session_status="failed"
+                session_status="commit_missing"
+            else
+                # Bugfix commits (fix: / fix(scope):) are exempt from docs check.
+                local last_commit_subject=""
+                last_commit_subject=$(git -C "$project_root" log -1 --pretty=%s 2>/dev/null || true)
+                if [[ "$last_commit_subject" =~ ^fix(\(|:) ]]; then
+                    log_info "Detected bugfix commit prefix; skipping docs check."
+                else
+                    local docs_changed=""
+                    docs_changed=$(git -C "$project_root" log --name-only --format="" -1 2>/dev/null \
+                        | grep -E '(\.prizm-docs/|\.prizmkit/specs/REGISTRY\.md)' | head -1 || true)
+                    if [[ -z "$docs_changed" ]]; then
+                        log_warn "Session committed but no .prizm-docs or REGISTRY.md changes detected."
+                        session_status="docs_missing"
+                    fi
+                fi
             fi
         fi
     fi
 
+    # Persist final pipeline-resolved status back to session-status.json
+    # only for post-check statuses introduced by the pipeline itself.
+    if [[ -f "$session_status_file" && ( "$session_status" == "commit_missing" || "$session_status" == "docs_missing" ) ]]; then
+        python3 -c "
+import json, sys
+p, st = sys.argv[1], sys.argv[2]
+with open(p, 'r', encoding='utf-8') as f:
+    data = json.load(f)
+data['status'] = st
+with open(p, 'w', encoding='utf-8') as f:
+    json.dump(data, f, indent=2, ensure_ascii=False)
+" "$session_status_file" "$session_status" >/dev/null 2>&1 || true
+    fi
+
     log_info "Session result: $session_status"
 
     # Update feature status
@@ -254,23 +255,29 @@ trap cleanup SIGINT SIGTERM
 # ============================================================
 
 check_dependencies() {
-    # Check for jq
-    if ! command -v jq &>/dev/null; then
-        log_error "jq is required but not installed. Install with: brew install jq"
-        exit 1
-    fi
+    prizm_check_common_dependencies "$CLI_CMD"
+}
 
-    # Check for python3
-    if ! command -v python3 &>/dev/null; then
-        log_error "python3 is required but not installed."
-        exit 1
+run_log_cleanup() {
+    if [[ "$LOG_CLEANUP_ENABLED" != "1" ]]; then
+        return 0
     fi
 
-    # Check for AI CLI
-    if ! command -v "$CLI_CMD" &>/dev/null; then
-        log_warn "AI CLI '$CLI_CMD' not found in PATH."
-        log_warn "Set AI_CLI environment variable to the correct command."
-        log_warn "Continuing anyway (will fail when spawning sessions)..."
+    local cleanup_result
+    cleanup_result=$(python3 "$SCRIPTS_DIR/cleanup-logs.py" \
+        --state-dir "$STATE_DIR" \
+        --retention-days "$LOG_RETENTION_DAYS" \
+        --max-total-mb "$LOG_MAX_TOTAL_MB" 2>/dev/null) || {
+        log_warn "Log cleanup failed (continuing)"
+        return 0
+    }
+
+    local deleted reclaimed_kb
+    deleted=$(echo "$cleanup_result" | python3 -c "import sys,json; print(json.load(sys.stdin).get('deleted_files', 0))" 2>/dev/null || echo "0")
+    reclaimed_kb=$(echo "$cleanup_result" | python3 -c "import sys,json; print(int(json.load(sys.stdin).get('reclaimed_bytes', 0)/1024))" 2>/dev/null || echo "0")
+
+    if [[ "$deleted" -gt 0 ]]; then
+        log_info "Log cleanup: deleted $deleted files, reclaimed ${reclaimed_kb}KB"
     fi
 }
 
@@ -385,6 +392,7 @@ run_one() {
     fi
 
     check_dependencies
+    run_log_cleanup
 
     # Verify feature exists
     local feature_title
@@ -550,8 +558,8 @@ sys.exit(1)
     echo -e "${BOLD}════════════════════════════════════════════════════${NC}"
     echo ""
 
-    # Override cleanup trap for single-feature mode
-    cleanup() {
+    # Override cleanup trap for single-feature mode (use distinct name to avoid overwriting global cleanup)
+    cleanup_single_feature() {
         echo ""
         log_warn "Interrupted. Killing session..."
         # Kill all child processes
@@ -559,7 +567,7 @@ sys.exit(1)
         log_info "Session log: $session_dir/logs/session.log"
         exit 130
     }
-    trap cleanup SIGINT SIGTERM
+    trap cleanup_single_feature SIGINT SIGTERM
 
     _SPAWN_RESULT=""
     spawn_and_wait_session \
@@ -603,6 +611,7 @@ main() {
     fi
 
     check_dependencies
+    run_log_cleanup
 
     # Initialize pipeline state if needed
     if [[ ! -f "$STATE_DIR/pipeline.json" ]]; then
@@ -683,6 +692,7 @@ for f in data.get('stuck_features', []):
             log_success "  All features completed! Pipeline finished."
             log_success "  Total sessions: $session_count"
             log_success "════════════════════════════════════════════════════"
+            rm -f "$STATE_DIR/current-session.json"
             break
         fi
 
@@ -729,7 +739,7 @@ for f in data.get('stuck_features', []):
             --state-dir "$STATE_DIR" \
             --output "$bootstrap_prompt" >/dev/null 2>&1
 
-        # Update current session tracking
+        # Update current session tracking (atomic write via temp file)
         python3 -c "
 import json, sys, os
 from datetime import datetime
@@ -739,8 +749,11 @@ data = {
     'session_id': session_id,
     'started_at': datetime.utcnow().strftime('%Y-%m-%dT%H:%M:%SZ')
 }
-with open(os.path.join(state_dir, 'current-session.json'), 'w') as f:
+target = os.path.join(state_dir, 'current-session.json')
+tmp = target + '.tmp'
+with open(tmp, 'w') as f:
     json.dump(data, f, indent=2)
+os.replace(tmp, target)
 " "$feature_id" "$session_id" "$STATE_DIR"
 
         # Mark feature as in-progress before spawning session
@@ -794,6 +807,9 @@ show_help() {
     echo "  AI_CLI                AI CLI command name (auto-detected: cbc or claude)"
     echo "  HEARTBEAT_INTERVAL    Heartbeat log interval in seconds (default: 30)"
     echo "  HEARTBEAT_STALE_THRESHOLD  Heartbeat stale threshold in seconds (default: 600)"
+    echo "  LOG_CLEANUP_ENABLED   Run log cleanup before execution (default: 1)"
+    echo "  LOG_RETENTION_DAYS    Delete logs older than N days (default: 14)"
+    echo "  LOG_MAX_TOTAL_MB      Keep total logs under N MB (default: 1024)"
     echo ""
     echo "Examples:"
     echo "  ./run.sh run                                         # Run all features"

package/bundled/dev-pipeline/scripts/check-session-status.py

@@ -13,9 +13,13 @@ import argparse
 import json
 import sys
 
+from utils import setup_logging
+
 
 REQUIRED_FIELDS = ["session_id", "feature_id", "status", "timestamp"]
 
+LOGGER = setup_logging("check-session-status")
+
 
 def parse_args():
     parser = argparse.ArgumentParser(
@@ -61,7 +65,8 @@ def validate_required_fields(data):
 def determine_status(data):
     """Determine the single-line status string from the parsed data.
 
-    Returns one of: success, partial_resumable, partial_not_resumable, failed.
+    Returns one of: success, partial_resumable, partial_not_resumable,
+    failed, commit_missing, docs_missing.
     """
     status = data.get("status", "")
 
@@ -75,6 +80,10 @@ def determine_status(data):
             return "partial_not_resumable"
     elif status == "failed":
         return "failed"
+    elif status == "commit_missing":
+        return "commit_missing"
+    elif status == "docs_missing":
+        return "docs_missing"
     else:
         # Unknown status value — treat as crashed
         return "crashed"
@@ -155,4 +164,40 @@ def main():
 
 
 if __name__ == "__main__":
-    main()
+    try:
+        main()
+    except KeyboardInterrupt:
+        detail = {
+            "status": "crashed",
+            "feature_id": None,
+            "completed_phases": [],
+            "checkpoint_reached": None,
+            "tasks_completed": 0,
+            "tasks_total": 0,
+            "error_count": 1,
+            "can_resume": False,
+            "resume_from_phase": None,
+            "internal_error": "Interrupted",
+        }
+        sys.stderr.write(json.dumps(detail, indent=2, ensure_ascii=False) + "\n")
+        print("crashed")
+        sys.exit(0)
+    except SystemExit:
+        raise
+    except Exception as exc:
+        LOGGER.exception("Unhandled exception in check-session-status")
+        detail = {
+            "status": "crashed",
+            "feature_id": None,
+            "completed_phases": [],
+            "checkpoint_reached": None,
+            "tasks_completed": 0,
+            "tasks_total": 0,
+            "error_count": 1,
+            "can_resume": False,
+            "resume_from_phase": None,
+            "internal_error": str(exc),
+        }
+        sys.stderr.write(json.dumps(detail, indent=2, ensure_ascii=False) + "\n")
+        print("crashed")
+        sys.exit(0)