loki-mode 5.32.2 → 5.34.0

package/autonomy/run.sh

@@ -1067,14 +1067,16 @@ import_github_issues() {
     local pending_file=".loki/queue/pending.json"
     local task_count=0
 
-    # Ensure pending.json exists with correct format for GitHub import
+    # BUG #14 fix: Normalize to bare [] format (consistent with init_loki_dir
+    # and all other queue consumers). Previously used {"tasks":[]} wrapper here
+    # but bare [] everywhere else, causing format mismatch.
     if [ ! -f "$pending_file" ]; then
-        echo '{"tasks":[]}' > "$pending_file"
-    elif jq -e 'type == "array"' "$pending_file" &>/dev/null; then
-        # Normalize bare array format to {"tasks":[...]} for GitHub import compatibility
+        echo '[]' > "$pending_file"
+    elif jq -e 'type == "object"' "$pending_file" &>/dev/null; then
+        # Normalize {"tasks":[...]} wrapper to bare array
         local _tmp_normalize
         _tmp_normalize=$(mktemp)
-        jq '{tasks: .}' "$pending_file" > "$_tmp_normalize" && mv "$_tmp_normalize" "$pending_file"
+        jq 'if type == "object" then .tasks // [] else . end' "$pending_file" > "$_tmp_normalize" && mv "$_tmp_normalize" "$pending_file"
         rm -f "$_tmp_normalize"
     fi
 
@@ -1094,8 +1096,8 @@ import_github_issues() {
         url=$(echo "$issue" | jq -r '.url')
         labels=$(echo "$issue" | jq -c '[.labels[].name]')
 
-        # Check if task already exists
-        if jq -e ".tasks[] | select(.github_issue == $number)" "$pending_file" &>/dev/null; then
+        # Check if task already exists (bare array format)
+        if jq -e ".[] | select(.github_issue == $number)" "$pending_file" &>/dev/null; then
             log_info "Issue #$number already imported, skipping"
             continue
         fi
@@ -1137,10 +1139,10 @@ import_github_issues() {
                 created_at: $created
             }')
 
-        # Append to pending.json with temp file cleanup on error
+        # Append to pending.json (bare array format) with temp file cleanup on error
         local temp_file
         temp_file=$(mktemp)
-        if jq ".tasks += [$task_json]" "$pending_file" > "$temp_file" && mv "$temp_file" "$pending_file"; then
+        if jq ". += [$task_json]" "$pending_file" > "$temp_file" && mv "$temp_file" "$pending_file"; then
             log_info "Imported issue #$number: $title"
             task_count=$((task_count + 1))
         else
@@ -1293,8 +1295,8 @@ export_tasks_to_github() {
         return 0
     fi
 
-    # Export non-GitHub tasks as issues
-    jq -c '.tasks[] | select(.source != "github")' "$pending_file" 2>/dev/null | while read -r task; do
+    # Export non-GitHub tasks as issues (handles both bare array and wrapper formats)
+    jq -c 'if type == "object" then .tasks // [] else . end | .[] | select(.source != "github")' "$pending_file" 2>/dev/null | while read -r task; do
         local title desc
         title=$(echo "$task" | jq -r '.title')
         desc=$(echo "$task" | jq -r '.description // ""')
@@ -2415,13 +2417,20 @@ write_dashboard_state() {
     local project_path=$(pwd)
     local _tmp_state="${output_file}.tmp"
 
+    # BUG #49 fix: Escape project path/name for JSON to handle special chars
+    # (spaces, quotes, backslashes in directory names)
+    local project_name_escaped
+    local project_path_escaped
+    project_name_escaped=$(printf '%s' "$project_name" | sed 's/\\/\\\\/g; s/"/\\"/g')
+    project_path_escaped=$(printf '%s' "$project_path" | sed 's/\\/\\\\/g; s/"/\\"/g')
+
     cat > "$_tmp_state" << EOF
 {
   "timestamp": "$(date -u +%Y-%m-%dT%H:%M:%SZ)",
   "version": "$version",
   "project": {
-    "name": "$project_name",
-    "path": "$project_path"
+    "name": "$project_name_escaped",
+    "path": "$project_path_escaped"
   },
   "mode": "$mode",
   "provider": "${PROVIDER_NAME:-claude}",
@@ -3870,6 +3879,167 @@ if top:
 SOLUTIONS_SCRIPT
 }
 
+# ============================================================================
+# Checkpoint/Snapshot System (v5.34.0)
+# Git-based checkpoints after task completion with state snapshots
+# Inspired by Cursor Self-Driving Codebases + Entire.io provenance tracking
+# ============================================================================
+
+create_checkpoint() {
+    # Create a git checkpoint after task completion
+    # Args: $1 = task description, $2 = task_id (optional)
+    local task_desc="${1:-task completed}"
+    local task_id="${2:-unknown}"
+    local checkpoint_dir=".loki/state/checkpoints"
+    local iteration="${ITERATION_COUNT:-0}"
+
+    mkdir -p "$checkpoint_dir"
+
+    # Only checkpoint if there are uncommitted changes
+    if ! git diff --quiet 2>/dev/null && ! git diff --cached --quiet 2>/dev/null; then
+        log_info "No uncommitted changes to checkpoint"
+        return 0
+    fi
+
+    # Capture git state
+    local git_sha
+    git_sha=$(git rev-parse HEAD 2>/dev/null || echo "no-git")
+    local git_branch
+    git_branch=$(git branch --show-current 2>/dev/null || echo "unknown")
+
+    # Snapshot .loki state files
+    local timestamp
+    timestamp=$(date -u +"%Y-%m-%dT%H:%M:%SZ")
+    local checkpoint_id="cp-${iteration}-$(date +%s)"
+    local cp_dir="${checkpoint_dir}/${checkpoint_id}"
+
+    mkdir -p "$cp_dir"
+
+    # Copy critical state files (lightweight -- not full .loki/)
+    for f in state/orchestrator.json queue/pending.json queue/completed.json queue/in-progress.json queue/current-task.json; do
+        if [ -f ".loki/$f" ]; then
+            local target_dir="$cp_dir/$(dirname "$f")"
+            mkdir -p "$target_dir"
+            cp ".loki/$f" "$cp_dir/$f" 2>/dev/null || true
+        fi
+    done
+
+    # Write checkpoint metadata
+    local safe_desc
+    safe_desc=$(printf '%s' "$task_desc" | sed 's/\\/\\\\/g; s/"/\\"/g' | head -c 200)
+    cat > "$cp_dir/metadata.json" << CPEOF
+{
+  "id": "${checkpoint_id}",
+  "timestamp": "${timestamp}",
+  "iteration": ${iteration},
+  "task_id": "${task_id}",
+  "task_description": "${safe_desc}",
+  "git_sha": "${git_sha}",
+  "git_branch": "${git_branch}",
+  "provider": "${PROVIDER_NAME:-claude}",
+  "phase": "$(cat .loki/state/orchestrator.json 2>/dev/null | python3 -c 'import sys,json; print(json.load(sys.stdin).get("currentPhase","unknown"))' 2>/dev/null || echo 'unknown')"
+}
+CPEOF
+
+    # Maintain checkpoint index for fast listing
+    local index_file="${checkpoint_dir}/index.jsonl"
+    printf '{"id":"%s","ts":"%s","iter":%d,"task":"%s","sha":"%s"}\n' \
+        "$checkpoint_id" "$timestamp" "$iteration" "$safe_desc" "$git_sha" \
+        >> "$index_file"
+
+    # Retention: keep last 50 checkpoints, prune older
+    local cp_count
+    cp_count=$(find "$checkpoint_dir" -maxdepth 1 -type d -name "cp-*" 2>/dev/null | wc -l | tr -d ' ')
+    if [ "$cp_count" -gt 50 ]; then
+        local to_remove=$((cp_count - 50))
+        find "$checkpoint_dir" -maxdepth 1 -type d -name "cp-*" 2>/dev/null | sort | head -n "$to_remove" | while read -r old_cp; do
+            rm -r "$old_cp" 2>/dev/null || true
+        done
+        # Rebuild index from remaining checkpoints
+        : > "$index_file"
+        for remaining in "$checkpoint_dir"/cp-*/metadata.json; do
+            [ -f "$remaining" ] || continue
+            python3 -c "
+import json,sys
+m=json.load(open('$remaining'))
+print(json.dumps({'id':m['id'],'ts':m['timestamp'],'iter':m['iteration'],'task':m.get('task_description',''),'sha':m['git_sha']}))
+" >> "$index_file" 2>/dev/null || true
+        done
+    fi
+
+    log_info "Checkpoint created: ${checkpoint_id} (git: ${git_sha:0:8})"
+}
+
+rollback_to_checkpoint() {
+    # Rollback state files to a specific checkpoint
+    # Args: $1 = checkpoint_id
+    local checkpoint_id="$1"
+    local checkpoint_dir=".loki/state/checkpoints"
+    local cp_dir="${checkpoint_dir}/${checkpoint_id}"
+
+    if [ ! -d "$cp_dir" ]; then
+        log_error "Checkpoint not found: ${checkpoint_id}"
+        return 1
+    fi
+
+    # Read checkpoint metadata
+    local git_sha
+    git_sha=$(python3 -c "import json; print(json.load(open('${cp_dir}/metadata.json'))['git_sha'])" 2>/dev/null || echo "")
+
+    log_warn "Rolling back to checkpoint: ${checkpoint_id}"
+
+    # Create a pre-rollback checkpoint first
+    create_checkpoint "pre-rollback snapshot" "rollback"
+
+    # Restore state files
+    for f in state/orchestrator.json queue/pending.json queue/completed.json queue/in-progress.json queue/current-task.json; do
+        if [ -f "${cp_dir}/${f}" ]; then
+            local target_dir=".loki/$(dirname "$f")"
+            mkdir -p "$target_dir"
+            cp "${cp_dir}/${f}" ".loki/${f}" 2>/dev/null || true
+        fi
+    done
+
+    # Log the rollback
+    local timestamp
+    timestamp=$(date -u +"%Y-%m-%dT%H:%M:%SZ")
+    printf '{"event":"rollback","checkpoint":"%s","git_sha":"%s","timestamp":"%s"}\n' \
+        "$checkpoint_id" "$git_sha" "$timestamp" \
+        >> ".loki/events.jsonl" 2>/dev/null || true
+
+    log_info "State files restored from checkpoint: ${checkpoint_id}"
+
+    if [ -n "$git_sha" ] && [ "$git_sha" != "no-git" ]; then
+        log_info "Git SHA at checkpoint: ${git_sha}"
+        log_info "To rollback code: git reset --hard ${git_sha}"
+    fi
+}
+
+list_checkpoints() {
+    # List recent checkpoints
+    local checkpoint_dir=".loki/state/checkpoints"
+    local index_file="${checkpoint_dir}/index.jsonl"
+    local limit="${1:-10}"
+
+    if [ ! -f "$index_file" ]; then
+        echo "No checkpoints found."
+        return
+    fi
+
+    tail -n "$limit" "$index_file" | python3 -c "
+import sys, json
+lines = sys.stdin.readlines()
+for line in reversed(lines):
+    try:
+        cp = json.loads(line)
+        sha = cp.get('sha','')[:8]
+        task = cp.get('task','')[:60]
+        print(f\"  {cp['id']}  {cp['ts']}  [{sha}]  {task}\")
+    except:
+        continue
+"
+}
+
 start_dashboard() {
     log_header "Starting Loki Dashboard"
 
@@ -4665,12 +4835,12 @@ build_prompt() {
     fi
 
     # Human directive injection (from HUMAN_INPUT.md)
+    # NOTE: Do NOT unset LOKI_HUMAN_INPUT here - build_prompt runs in a subshell
+    # (command substitution) so unset would not affect the parent shell.
+    # The caller (run_autonomous) clears it after consuming the prompt.
     local human_directive=""
     if [ -n "${LOKI_HUMAN_INPUT:-}" ]; then
         human_directive="HUMAN_DIRECTIVE (PRIORITY): $LOKI_HUMAN_INPUT Execute this directive BEFORE continuing normal tasks."
-        # Clear after consumption so it doesn't repeat every iteration
-        unset LOKI_HUMAN_INPUT
-        rm -f "${TARGET_DIR:-.}/.loki/HUMAN_INPUT.md"
     fi
 
     # Queue task injection (from dashboard or API)
@@ -4795,6 +4965,15 @@ run_autonomous() {
 
         local prompt=$(build_prompt $retry "$prd_path" $ITERATION_COUNT)
 
+        # BUG #5 fix: Clear LOKI_HUMAN_INPUT in the parent shell after build_prompt
+        # consumed it. build_prompt runs in a subshell (command substitution), so
+        # any unset inside it does not affect the parent. Clear here to prevent
+        # the same directive from repeating every iteration.
+        if [ -n "${LOKI_HUMAN_INPUT:-}" ]; then
+            unset LOKI_HUMAN_INPUT
+            rm -f "${TARGET_DIR:-.}/.loki/HUMAN_INPUT.md"
+        fi
+
         echo ""
         log_header "Attempt $((retry + 1)) of $MAX_RETRIES"
         log_info "Prompt: $prompt"
@@ -5234,11 +5413,19 @@ check_human_intervention() {
     local loki_dir="${TARGET_DIR:-.}/.loki"
 
     # Check for PAUSE file
+    # BUG #4 fix: Check handle_pause return value before deleting PAUSE file.
+    # handle_pause returns 1 if STOP was requested during the pause, so we must
+    # propagate that as return 2 (stop) instead of always returning 1 (continue).
     if [ -f "$loki_dir/PAUSE" ]; then
         log_warn "PAUSE file detected - pausing execution"
         notify_intervention_needed "Execution paused via PAUSE file"
         handle_pause
+        local pause_result=$?
         rm -f "$loki_dir/PAUSE"
+        if [ "$pause_result" -eq 1 ]; then
+            # STOP was requested during pause
+            return 2
+        fi
         return 1
     fi
 
@@ -5287,8 +5474,13 @@ check_human_intervention() {
         rm -f "$loki_dir/signals/COUNCIL_REVIEW_REQUESTED"
         if type council_vote &>/dev/null && council_vote; then
             log_header "COMPLETION COUNCIL: FORCE REVIEW - PROJECT COMPLETE"
-            # Complete the missing steps: COMPLETED marker, memory consolidation, report
-            touch "$loki_dir/COMPLETED"
+            # BUG #17 fix: Write COMPLETED marker, generate council report, and
+            # run memory consolidation (matching the normal council approval path
+            # in council_should_stop).
+            echo "Council force-review approved at iteration $ITERATION_COUNT on $(date -u +%Y-%m-%dT%H:%M:%SZ)" > "$loki_dir/COMPLETED"
+            if type council_write_report &>/dev/null; then
+                council_write_report
+            fi
             log_info "Running memory consolidation..."
             run_memory_consolidation
             notify_all_complete
@@ -5764,6 +5956,9 @@ main() {
     # Compound learnings into structured solution files (v5.30.0)
     compound_session_to_solutions
 
+    # Create session-end checkpoint (v5.34.0)
+    create_checkpoint "session end (iterations=$ITERATION_COUNT)" "session-end"
+
     # Log session end for audit
     audit_log "SESSION_END" "result=$result,prd=$PRD_PATH"
 

package/dashboard/__init__.py

@@ -7,7 +7,7 @@ Modules:
     control: Session control API (start/stop/pause/resume)
 """
 
-__version__ = "5.32.2"
+__version__ = "5.34.0"
 
 # Expose the control app for easy import
 try:

package/dashboard/control.py

@@ -56,10 +56,13 @@ app = FastAPI(
     version="1.0.0"
 )
 
-# CORS middleware for dashboard frontend
+# CORS middleware for dashboard frontend - restricted to localhost by default.
+# Set LOKI_DASHBOARD_CORS to override (comma-separated origins).
+_cors_default = "http://localhost:57374,http://127.0.0.1:57374"
+_cors_origins = os.environ.get("LOKI_DASHBOARD_CORS", _cors_default).split(",")
 app.add_middleware(
     CORSMiddleware,
-    allow_origins=["*"],
+    allow_origins=[o.strip() for o in _cors_origins if o.strip()],
     allow_credentials=True,
     allow_methods=["*"],
     allow_headers=["*"],
@@ -511,4 +514,5 @@ async def asyncio_sleep(seconds: float):
 if __name__ == "__main__":
     import uvicorn
     port = int(os.environ.get("LOKI_DASHBOARD_PORT", "57374"))
-    uvicorn.run(app, host="0.0.0.0", port=port)
+    host = os.environ.get("LOKI_DASHBOARD_HOST", "127.0.0.1")
+    uvicorn.run(app, host=host, port=port)

package/dashboard/server.py

@@ -209,9 +209,10 @@ app = FastAPI(
     lifespan=lifespan,
 )
 
-# Add CORS middleware - allow_origins=* is safe because server binds to
-# 127.0.0.1 by default. Set LOKI_DASHBOARD_HOST to override if LAN access needed.
-_cors_origins = os.environ.get("CORS_ALLOWED_ORIGINS", "*").split(",")
+# Add CORS middleware - restricted to localhost by default.
+# Set LOKI_DASHBOARD_CORS to override (comma-separated origins).
+_cors_default = "http://localhost:57374,http://127.0.0.1:57374"
+_cors_origins = os.environ.get("LOKI_DASHBOARD_CORS", _cors_default).split(",")
 app.add_middleware(
     CORSMiddleware,
     allow_origins=[o.strip() for o in _cors_origins if o.strip()],
@@ -1983,6 +1984,149 @@ async def force_council_review():
     return {"success": True, "message": "Council review requested"}
 
 
+# =============================================================================
+# Checkpoint API (v5.34.0)
+# =============================================================================
+
+class CheckpointCreate(BaseModel):
+    """Schema for creating a checkpoint."""
+    message: Optional[str] = Field(None, description="Optional description for the checkpoint")
+
+
+def _sanitize_checkpoint_id(checkpoint_id: str) -> str:
+    """Validate checkpoint_id contains only safe characters for file paths."""
+    if not checkpoint_id or ".." in checkpoint_id or not _SAFE_ID_RE.match(checkpoint_id):
+        raise HTTPException(
+            status_code=400,
+            detail="Invalid checkpoint_id: must contain only alphanumeric characters, hyphens, and underscores",
+        )
+    return checkpoint_id
+
+
+@app.get("/api/checkpoints")
+async def list_checkpoints(limit: int = Query(default=20, ge=1, le=200)):
+    """List recent checkpoints from index.jsonl."""
+    loki_dir = _get_loki_dir()
+    index_file = loki_dir / "state" / "checkpoints" / "index.jsonl"
+    checkpoints = []
+
+    if index_file.exists():
+        try:
+            for line in index_file.read_text().strip().split("\n"):
+                if line.strip():
+                    try:
+                        checkpoints.append(json.loads(line))
+                    except json.JSONDecodeError:
+                        pass
+        except Exception:
+            pass
+
+    # Return most recent first, limited
+    checkpoints.reverse()
+    return checkpoints[:limit]
+
+
+@app.get("/api/checkpoints/{checkpoint_id}")
+async def get_checkpoint(checkpoint_id: str):
+    """Get checkpoint details by ID."""
+    checkpoint_id = _sanitize_checkpoint_id(checkpoint_id)
+    loki_dir = _get_loki_dir()
+    metadata_file = loki_dir / "state" / "checkpoints" / checkpoint_id / "metadata.json"
+
+    if not metadata_file.exists():
+        raise HTTPException(status_code=404, detail="Checkpoint not found")
+
+    try:
+        return json.loads(metadata_file.read_text())
+    except (json.JSONDecodeError, IOError) as e:
+        raise HTTPException(status_code=500, detail=f"Failed to read checkpoint: {e}")
+
+
+@app.post("/api/checkpoints", status_code=201)
+async def create_checkpoint(body: CheckpointCreate = None):
+    """Create a new checkpoint capturing current state."""
+    import subprocess
+    import shutil
+
+    loki_dir = _get_loki_dir()
+    checkpoints_dir = loki_dir / "state" / "checkpoints"
+    checkpoints_dir.mkdir(parents=True, exist_ok=True)
+
+    # Generate checkpoint ID from timestamp
+    now = datetime.now(timezone.utc)
+    checkpoint_id = now.strftime("chk-%Y%m%d-%H%M%S")
+
+    # Create checkpoint directory
+    checkpoint_dir = checkpoints_dir / checkpoint_id
+    checkpoint_dir.mkdir(parents=True, exist_ok=True)
+
+    # Capture git SHA
+    git_sha = ""
+    try:
+        result = subprocess.run(
+            ["git", "rev-parse", "HEAD"],
+            capture_output=True, text=True, timeout=5,
+        )
+        if result.returncode == 0:
+            git_sha = result.stdout.strip()
+    except Exception:
+        pass
+
+    # Copy key state files into checkpoint
+    state_files = [
+        "dashboard-state.json",
+        "session.json",
+    ]
+    for fname in state_files:
+        src = loki_dir / fname
+        if src.exists():
+            try:
+                shutil.copy2(str(src), str(checkpoint_dir / fname))
+            except Exception:
+                pass
+
+    # Copy queue directory if present
+    queue_src = loki_dir / "queue"
+    if queue_src.exists():
+        try:
+            shutil.copytree(str(queue_src), str(checkpoint_dir / "queue"), dirs_exist_ok=True)
+        except Exception:
+            pass
+
+    # Build metadata
+    message = ""
+    if body and body.message:
+        message = body.message
+
+    metadata = {
+        "id": checkpoint_id,
+        "created_at": now.isoformat(),
+        "git_sha": git_sha,
+        "message": message,
+        "files": [f.name for f in checkpoint_dir.iterdir() if f.is_file()],
+    }
+
+    # Write metadata.json
+    (checkpoint_dir / "metadata.json").write_text(json.dumps(metadata, indent=2))
+
+    # Append to index.jsonl
+    index_file = checkpoints_dir / "index.jsonl"
+    with open(str(index_file), "a") as f:
+        f.write(json.dumps(metadata) + "\n")
+
+    # Retention policy: keep last 50 checkpoints
+    MAX_CHECKPOINTS = 50
+    all_dirs = sorted(
+        [d for d in checkpoints_dir.iterdir() if d.is_dir()],
+        key=lambda d: d.name,
+    )
+    while len(all_dirs) > MAX_CHECKPOINTS:
+        oldest = all_dirs.pop(0)
+        shutil.rmtree(str(oldest), ignore_errors=True)
+
+    return metadata
+
+
 # =============================================================================
 # Agent Management API (v5.25.0)
 # =============================================================================
@@ -2046,6 +2190,7 @@ async def get_agents():
 @app.post("/api/agents/{agent_id}/kill")
 async def kill_agent(agent_id: str):
     """Kill a specific agent by ID."""
+    agent_id = _sanitize_agent_id(agent_id)
     agents_file = _get_loki_dir() / "state" / "agents.json"
     if not agents_file.exists():
         raise HTTPException(404, "No agents file found")
@@ -2296,7 +2441,7 @@ def run_server(host: str = None, port: int = None) -> None:
     """Run the dashboard server."""
     import uvicorn
     if host is None:
-        # Default to localhost-only; CORS * is safe since not exposed to LAN
+        # Default to localhost-only for security
         host = os.environ.get("LOKI_DASHBOARD_HOST", "127.0.0.1")
     if port is None:
         port = int(os.environ.get("LOKI_DASHBOARD_PORT", "57374"))

package/dashboard/static/index.html

@@ -3750,7 +3750,7 @@ var LokiDashboard=(()=>{var N=Object.defineProperty;var se=Object.getOwnProperty
           ${t}
         </div>
       </div>
-    `,this._attachEventListeners()}_attachEventListeners(){let e=this.shadowRoot.getElementById("time-range-select");e&&e.addEventListener("change",r=>this._setFilter("timeRange",r.target.value));let t=this.shadowRoot.getElementById("signal-type-select");t&&t.addEventListener("change",r=>this._setFilter("signalType",r.target.value));let a=this.shadowRoot.getElementById("source-select");a&&a.addEventListener("change",r=>this._setFilter("source",r.target.value));let i=this.shadowRoot.getElementById("refresh-btn");i&&i.addEventListener("click",()=>this._loadData());let s=this.shadowRoot.getElementById("close-detail");s&&s.addEventListener("click",()=>this._closeDetail()),this.shadowRoot.querySelectorAll(".list-item").forEach(r=>{r.addEventListener("click",()=>{let n=r.dataset.type,d=r.dataset.id,p=this._findItemData(n,d);p&&this._selectMetric(n,p)}),r.addEventListener("keydown",n=>{(n.key==="Enter"||n.key===" ")&&(n.preventDefault(),r.click())})})}_findItemData(e,t){if(!this._metrics?.aggregation)return null;switch(e){case"preference":return this._metrics.aggregation.preferences?.find(a=>a.preference_key===t);case"error_pattern":return this._metrics.aggregation.error_patterns?.find(a=>a.error_type===t);case"success_pattern":return this._metrics.aggregation.success_patterns?.find(a=>a.pattern_name===t);case"tool_efficiency":return this._metrics.aggregation.tool_efficiencies?.find(a=>a.tool_name===t);default:return null}}};customElements.get("loki-learning-dashboard")||customElements.define("loki-learning-dashboard",H);var me=[{id:"overview",label:"Overview"},{id:"decisions",label:"Decision Log"},{id:"convergence",label:"Convergence"},{id:"agents",label:"Agents"}],O=class extends c{static get observedAttributes(){return["api-url","theme"]}constructor(){super(),this._loading=!1,this._error=null,this._api=null,this._activeTab="overview",this._pollInterval=null,this._councilState=null,this._verdicts=[],this._convergence=[],this._agents=[],this._selectedAgent=null,this._lastDataHash=null}connectedCallback(){super.connectedCallback(),this._setupApi(),this._loadData(),this._startPolling()}disconnectedCallback(){super.disconnectedCallback(),this._stopPolling()}attributeChangedCallback(e,t,a){t!==a&&(e==="api-url"&&this._api&&(this._api.baseUrl=a,this._loadData()),e==="theme"&&this._applyTheme())}_setupApi(){let e=this.getAttribute("api-url")||window.location.origin;this._api=g({baseUrl:e})}_startPolling(){this._pollInterval=setInterval(()=>this._loadData(),3e3),this._visibilityHandler=()=>{document.hidden?this._pollInterval&&(clearInterval(this._pollInterval),this._pollInterval=null):this._pollInterval||(this._loadData(),this._pollInterval=setInterval(()=>this._loadData(),3e3))},document.addEventListener("visibilitychange",this._visibilityHandler)}_stopPolling(){this._pollInterval&&(clearInterval(this._pollInterval),this._pollInterval=null),this._visibilityHandler&&(document.removeEventListener("visibilitychange",this._visibilityHandler),this._visibilityHandler=null)}async _loadData(){try{let[t,a,i,s]=await Promise.allSettled([this._api._get("/api/council/state"),this._api._get("/api/council/verdicts"),this._api._get("/api/council/convergence"),this._api._get("/api/agents")]);t.status==="fulfilled"&&(this._councilState=t.value),a.status==="fulfilled"&&(this._verdicts=a.value.verdicts||[]),i.status==="fulfilled"&&(this._convergence=i.value.dataPoints||[]),s.status==="fulfilled"&&(this._agents=Array.isArray(s.value)?s.value:[]),this._error=null}catch(t){this._error=t.message}let e=JSON.stringify({s:this._councilState,v:this._verdicts,c:this._convergence,a:this._agents,e:this._error});e!==this._lastDataHash&&(this._lastDataHash=e,this.render())}async _forceReview(){try{await this._api._post("/api/council/force-review"),this.dispatchEvent(new CustomEvent("council-action",{detail:{action:"force-review"},bubbles:!0}))}catch(e){this._error=`Failed to force review: ${e.message}`,this.render()}}async _killAgent(e){if(confirm(`Kill agent ${e}?`))try{await this._api._post(`/api/agents/${e}/kill`),this.dispatchEvent(new CustomEvent("council-action",{detail:{action:"kill-agent",agentId:e},bubbles:!0})),await this._loadData()}catch(t){this._error=`Failed to kill agent: ${t.message}`,this.render()}}async _pauseAgent(e){try{await this._api._post(`/api/agents/${e}/pause`),await this._loadData()}catch(t){this._error=`Failed to pause agent: ${t.message}`,this.render()}}async _resumeAgent(e){try{await this._api._post(`/api/agents/${e}/resume`),await this._loadData()}catch(t){this._error=`Failed to resume agent: ${t.message}`,this.render()}}_setTab(e){this._activeTab=e,this.render()}_selectAgent(e){this._selectedAgent=this._selectedAgent?.id===e.id?null:e,this.render()}render(){let e=this.shadowRoot;e&&(e.innerHTML=`
+    `,this._attachEventListeners()}_attachEventListeners(){let e=this.shadowRoot.getElementById("time-range-select");e&&e.addEventListener("change",r=>this._setFilter("timeRange",r.target.value));let t=this.shadowRoot.getElementById("signal-type-select");t&&t.addEventListener("change",r=>this._setFilter("signalType",r.target.value));let a=this.shadowRoot.getElementById("source-select");a&&a.addEventListener("change",r=>this._setFilter("source",r.target.value));let i=this.shadowRoot.getElementById("refresh-btn");i&&i.addEventListener("click",()=>this._loadData());let s=this.shadowRoot.getElementById("close-detail");s&&s.addEventListener("click",()=>this._closeDetail()),this.shadowRoot.querySelectorAll(".list-item").forEach(r=>{r.addEventListener("click",()=>{let n=r.dataset.type,d=r.dataset.id,p=this._findItemData(n,d);p&&this._selectMetric(n,p)}),r.addEventListener("keydown",n=>{(n.key==="Enter"||n.key===" ")&&(n.preventDefault(),r.click())})})}_findItemData(e,t){if(!this._metrics?.aggregation)return null;switch(e){case"preference":return this._metrics.aggregation.preferences?.find(a=>a.preference_key===t);case"error_pattern":return this._metrics.aggregation.error_patterns?.find(a=>a.error_type===t);case"success_pattern":return this._metrics.aggregation.success_patterns?.find(a=>a.pattern_name===t);case"tool_efficiency":return this._metrics.aggregation.tool_efficiencies?.find(a=>a.tool_name===t);default:return null}}};customElements.get("loki-learning-dashboard")||customElements.define("loki-learning-dashboard",H);var me=[{id:"overview",label:"Overview"},{id:"decisions",label:"Decision Log"},{id:"convergence",label:"Convergence"},{id:"agents",label:"Agents"}],O=class extends c{static get observedAttributes(){return["api-url","theme"]}constructor(){super(),this._loading=!1,this._error=null,this._api=null,this._activeTab="overview",this._pollInterval=null,this._councilState=null,this._verdicts=[],this._convergence=[],this._agents=[],this._selectedAgent=null,this._lastDataHash=null}connectedCallback(){super.connectedCallback(),this._setupApi(),this._loadData(),this._startPolling()}disconnectedCallback(){super.disconnectedCallback(),this._stopPolling()}attributeChangedCallback(e,t,a){t!==a&&(e==="api-url"&&this._api&&(this._api.baseUrl=a,this._loadData()),e==="theme"&&this._applyTheme())}_setupApi(){let e=this.getAttribute("api-url")||window.location.origin;this._api=g({baseUrl:e})}_startPolling(){this._pollInterval=setInterval(()=>this._loadData(),3e3),this._visibilityHandler=()=>{document.hidden?this._pollInterval&&(clearInterval(this._pollInterval),this._pollInterval=null):this._pollInterval||(this._loadData(),this._pollInterval=setInterval(()=>this._loadData(),3e3))},document.addEventListener("visibilitychange",this._visibilityHandler)}_stopPolling(){this._pollInterval&&(clearInterval(this._pollInterval),this._pollInterval=null),this._visibilityHandler&&(document.removeEventListener("visibilitychange",this._visibilityHandler),this._visibilityHandler=null),this._pendingRaf&&(cancelAnimationFrame(this._pendingRaf),this._pendingRaf=null)}async _loadData(){try{let[t,a,i,s]=await Promise.allSettled([this._api._get("/api/council/state"),this._api._get("/api/council/verdicts"),this._api._get("/api/council/convergence"),this._api._get("/api/agents")]);t.status==="fulfilled"&&(this._councilState=t.value),a.status==="fulfilled"&&(this._verdicts=a.value.verdicts||[]),i.status==="fulfilled"&&(this._convergence=i.value.dataPoints||[]),s.status==="fulfilled"&&(this._agents=Array.isArray(s.value)?s.value:[]),this._error=null}catch(t){this._error=t.message}let e=JSON.stringify({s:this._councilState,v:this._verdicts,c:this._convergence,a:this._agents,e:this._error});e!==this._lastDataHash&&(this._lastDataHash=e,this.render())}async _forceReview(){try{await this._api._post("/api/council/force-review"),this.dispatchEvent(new CustomEvent("council-action",{detail:{action:"force-review"},bubbles:!0}))}catch(e){this._error=`Failed to force review: ${e.message}`,this.render()}}async _killAgent(e){if(confirm(`Kill agent ${e}?`))try{await this._api._post(`/api/agents/${e}/kill`),this.dispatchEvent(new CustomEvent("council-action",{detail:{action:"kill-agent",agentId:e},bubbles:!0})),await this._loadData()}catch(t){this._error=`Failed to kill agent: ${t.message}`,this.render()}}async _pauseAgent(e){try{await this._api._post(`/api/agents/${e}/pause`),await this._loadData()}catch(t){this._error=`Failed to pause agent: ${t.message}`,this.render()}}async _resumeAgent(e){try{await this._api._post(`/api/agents/${e}/resume`),await this._loadData()}catch(t){this._error=`Failed to resume agent: ${t.message}`,this.render()}}_setTab(e){this._activeTab=e,this.render()}_selectAgent(e){this._selectedAgent=this._selectedAgent?.id===e.id?null:e,this.render()}render(){let e=this.shadowRoot;e&&(this._pendingRaf&&(cancelAnimationFrame(this._pendingRaf),this._pendingRaf=null),e.innerHTML=`
       <style>${this.getBaseStyles()}${this._getStyles()}</style>
       <div class="council-dashboard">
         <div class="council-header">
@@ -3758,7 +3758,7 @@ var LokiDashboard=(()=>{var N=Object.defineProperty;var se=Object.getOwnProperty
             <h2 class="title">Completion Council</h2>
             ${this._councilState?.enabled!==!1?'<span class="badge badge-active">Active</span>':'<span class="badge badge-inactive">Disabled</span>'}
           </div>
-          <button class="btn btn-primary" onclick="this.getRootNode().host._forceReview()">
+          <button class="btn btn-primary" id="force-review-btn">
             Force Review
           </button>
         </div>
@@ -3767,7 +3767,7 @@ var LokiDashboard=(()=>{var N=Object.defineProperty;var se=Object.getOwnProperty
           ${me.map(t=>`
             <button
               class="tab ${this._activeTab===t.id?"active":""}"
-              onclick="this.getRootNode().host._setTab('${t.id}')"
+              data-tab="${t.id}"
             >${t.label}</button>
           `).join("")}
         </div>
@@ -3778,7 +3778,7 @@ var LokiDashboard=(()=>{var N=Object.defineProperty;var se=Object.getOwnProperty
 
         ${this._error?`<div class="error-banner">${this._error}</div>`:""}
       </div>
-    `)}_renderTabContent(){switch(this._activeTab){case"overview":return this._renderOverview();case"decisions":return this._renderDecisions();case"convergence":return this._renderConvergence();case"agents":return this._renderAgents();default:return""}}_renderOverview(){let e=this._councilState||{},t=e.consecutive_no_change||0,a=e.done_signals||0,i=e.total_votes||0,s=e.approve_votes||0,r=this._verdicts.length>0?this._verdicts[this._verdicts.length-1]:null,n=this._agents.filter(d=>d.alive).length;return`
+    `,this._attachEventListeners())}_attachEventListeners(){let e=this.shadowRoot;if(!e)return;let t=e.getElementById("force-review-btn");t&&t.addEventListener("click",()=>this._forceReview()),e.querySelectorAll(".tab[data-tab]").forEach(a=>{a.addEventListener("click",()=>this._setTab(a.dataset.tab))})}_renderTabContent(){switch(this._activeTab){case"overview":return this._renderOverview();case"decisions":return this._renderDecisions();case"convergence":return this._renderConvergence();case"agents":return this._renderAgents();default:return""}}_renderOverview(){let e=this._councilState||{},t=e.consecutive_no_change||0,a=e.done_signals||0,i=e.total_votes||0,s=e.approve_votes||0,r=this._verdicts.length>0?this._verdicts[this._verdicts.length-1]:null,n=this._agents.filter(d=>d.alive).length;return`
       <div class="overview-grid">
         <div class="stat-card">
           <div class="stat-label">Council Status</div>
@@ -3918,7 +3918,7 @@ var LokiDashboard=(()=>{var N=Object.defineProperty;var se=Object.getOwnProperty
           </div>
         `).join("")}
       </div>
-    `;return requestAnimationFrame(()=>{let t=this.shadowRoot;t&&t.querySelectorAll(".agent-card[data-agent-index]").forEach(a=>{let i=parseInt(a.dataset.agentIndex,10),s=this._agents[i];s&&(a.addEventListener("click",()=>this._selectAgent(s)),a.querySelectorAll("[data-action]").forEach(r=>{r.addEventListener("click",n=>{n.stopPropagation();let d=r.dataset.action,p=r.dataset.agentId;d==="pause"?this._pauseAgent(p):d==="kill"?this._killAgent(p):d==="resume"&&this._resumeAgent(p)})}))})}),e}_formatTime(e){if(!e)return"";try{return new Date(e).toLocaleTimeString([],{hour:"2-digit",minute:"2-digit"})}catch{return e}}_getStyles(){return`
+    `;return this._pendingRaf=requestAnimationFrame(()=>{this._pendingRaf=null;let t=this.shadowRoot;t&&t.querySelectorAll(".agent-card[data-agent-index]").forEach(a=>{let i=parseInt(a.dataset.agentIndex,10),s=this._agents[i];s&&(a.addEventListener("click",()=>this._selectAgent(s)),a.querySelectorAll("[data-action]").forEach(r=>{r.addEventListener("click",n=>{n.stopPropagation();let d=r.dataset.action,p=r.dataset.agentId;d==="pause"?this._pauseAgent(p):d==="kill"?this._killAgent(p):d==="resume"&&this._resumeAgent(p)})}))})}),e}_formatTime(e){if(!e)return"";try{return new Date(e).toLocaleTimeString([],{hour:"2-digit",minute:"2-digit"})}catch{return e}}_getStyles(){return`
       :host {
         display: block;
         font-family: 'Inter', -apple-system, BlinkMacSystemFont, sans-serif;