loki-mode 7.5.17 → 7.5.28

package/README.md

@@ -27,7 +27,7 @@
 - **Truly autonomous** -- Describe what you want, walk away, come back to working code with tests
 - **Production quality built in** -- 11 quality gates (`skills/quality-gates.md`), blind 3-reviewer code review (`run.sh:run_code_review()`), anti-sycophancy checks
 - **Self-hosted and private** -- Your keys, your infrastructure, no data leaves your network
-- **5 AI providers** -- Claude, Codex, Gemini, Cline, Aider with automatic failover (`loki-ts/src/runner/providers.ts`)
+- **4 active AI providers** -- Claude, Codex, Cline, Aider with automatic failover (`loki-ts/src/runner/providers.ts`). Gemini CLI deprecated v7.5.18; Antigravity CLI coming soon.
 - **Legacy system healing** -- `loki heal` archaeology/stabilize/isolate/modernize/validate phases (v6.67.0, see `skills/healing.md`)
 - **Memory system** -- Episodic/semantic/procedural with vector search (v5.15.0, see `memory/engine.py`)
 - **MCP server** -- 15 tools including ChromaDB code search (`mcp/server.py`)
@@ -304,15 +304,16 @@ Loki Mode is the only platform that is fully self-hosted, open source, and inclu
 
 ## Multi-Provider Support
 
-| Provider | Autonomous Flag | Parallel Agents | Install |
-|----------|:-:|:-:|---------|
-| **Claude Code** | `--dangerously-skip-permissions` | Yes (10+) | `npm i -g @anthropic-ai/claude-code` |
-| **Codex CLI** | `--full-auto` | Sequential | `npm i -g @openai/codex` |
-| **Gemini CLI** | `--approval-mode=yolo` | Sequential | `npm i -g @google/gemini-cli` |
-| **Cline CLI** | `--auto-approve` | Sequential | `npm i -g @anthropic-ai/cline` |
-| **Aider** | `--yes-always` | Sequential | `pip install aider-chat` |
+| Provider | Status | Autonomous Flag | Parallel Agents | Install |
+|----------|--------|:-:|:-:|---------|
+| **Claude Code** | Active (Tier 1) | `--dangerously-skip-permissions` | Yes (10+) | `npm i -g @anthropic-ai/claude-code` |
+| **Codex CLI** | Active (Tier 3) | `--full-auto` | Sequential | `npm i -g @openai/codex` |
+| **Cline CLI** | Active (Tier 2) | `--auto-approve` | Sequential | `npm i -g @anthropic-ai/cline` |
+| **Aider** | Active (Tier 3) | `--yes-always` | Sequential | `pip install aider-chat` |
+| **Google Gemini CLI** | DEPRECATED v7.5.18 | -- | -- | Upstream deprecated; runtime removed. `LOKI_PROVIDER=gemini` exits with migration message. |
+| **Anthropic Antigravity CLI** | Coming soon | -- | -- | Integration planned. |
 
-Claude gets full features (subagents, parallelization, MCP, Task tool). Other providers run sequentially. Auto-failover switches providers when rate-limited. See [Provider Guide](skills/providers.md).
+Claude gets full features (subagents, parallelization, MCP, Task tool). Other active providers run sequentially. Auto-failover switches providers when rate-limited. See [Provider Guide](skills/providers.md).
 
 ---
 

package/SKILL.md

@@ -3,13 +3,13 @@ name: loki-mode
 description: Multi-agent autonomous startup system. Triggers on "Loki Mode". Takes a spec (PRD, GitHub issue, OpenAPI doc, etc.) to deployed product with minimal human intervention. Requires --dangerously-skip-permissions flag.
 ---
 
-# Loki Mode v7.5.17
+# Loki Mode v7.5.28
 
 **You are an autonomous agent. You make decisions. You do not ask questions. You do not stop.**
 
 **Spec in, product out.** A "spec" is whatever describes the work: a Markdown PRD, a GitHub issue, an OpenAPI doc, a Jira ticket -- a PRD is one form of spec.
 
-**Multi-provider (stable since v5.0.0):** Claude/Codex/Gemini/Cline/Aider with abstract model tiers and degraded mode for non-Claude providers. See `skills/providers.md`. **Current track (v7.5.x):** Phase 1 RARV-C closure -- real provider judges, gate-failure flock, synthetic PRD e2e, status `--json`, dead code cleanup.
+**Multi-provider (stable since v5.0.0):** Claude/Codex/Cline/Aider with abstract model tiers and degraded mode for non-Claude providers. See `skills/providers.md`. **Current track (v7.5.x):** Phase 1 RARV-C closure -- real provider judges, gate-failure flock, synthetic PRD e2e, status `--json`, dead code cleanup.
 
 **Runtime migration in progress:** A bash-to-Bun migration is underway on the `feat/bun-migration` branch. The first phase (shipped in v7.3.0) routes a small set of read-only commands -- `version`, `status`, `stats`, `doctor`, `provider show/list`, `memory list/index` -- through a Bun runtime via `bin/loki`. Every other command remains on the Bash runtime (`autonomy/loki`). Rollback is available with `LOKI_LEGACY_BASH=1`. See `UPGRADING.md` and `docs/architecture/ADR-001-runtime-migration.md` for the full plan.
 
@@ -93,17 +93,17 @@ These rules guide autonomous operation. Test results and code quality always tak
 
 **Default since v5.3.0 (reaffirmed in v7.5.13):** Haiku disabled for quality. Use `--allow-haiku` or `LOKI_ALLOW_HAIKU=true` to enable.
 
-| Task Type | Tier | Claude (default) | Claude (--allow-haiku) | Codex (GPT-5.3) | Gemini |
-|-----------|------|------------------|------------------------|------------------|--------|
-| Spec analysis, architecture, system design | **planning** | opus | opus | effort=xhigh | thinking=high |
-| Feature implementation, complex bugs | **development** | opus | sonnet | effort=high | thinking=medium |
-| Code review (planned: 3 parallel reviewers) | **development** | opus | sonnet | effort=high | thinking=medium |
-| Integration tests, E2E, deployment | **development** | opus | sonnet | effort=high | thinking=medium |
-| Unit tests, linting, docs, simple fixes | **fast** | sonnet | haiku | effort=low | thinking=low |
+| Task Type | Tier | Claude (default) | Claude (--allow-haiku) | Codex (GPT-5.3) |
+|-----------|------|------------------|------------------------|------------------|
+| Spec analysis, architecture, system design | **planning** | opus | opus | effort=xhigh |
+| Feature implementation, complex bugs | **development** | opus | sonnet | effort=high |
+| Code review (planned: 3 parallel reviewers) | **development** | opus | sonnet | effort=high |
+| Integration tests, E2E, deployment | **development** | opus | sonnet | effort=high |
+| Unit tests, linting, docs, simple fixes | **fast** | sonnet | haiku | effort=low |
 
 **Parallelization rule (Claude only):** Launch up to 10 agents simultaneously for independent tasks.
 
-**Degraded mode (Codex/Gemini/Cline/Aider):** No parallel agents or Task tool. Codex has MCP support. Runs RARV cycle sequentially. See `skills/model-selection.md`.
+**Degraded mode (Codex/Cline/Aider):** No parallel agents or Task tool. Codex has MCP support. Runs RARV cycle sequentially. See `skills/model-selection.md`.
 
 **Git worktree parallelism:** For true parallel feature development, use `--parallel` flag with run.sh. See `skills/parallel-workflows.md`.
 
@@ -210,7 +210,6 @@ loki start --issue 123                       # Explicit issue mode (overrides de
 # With provider selection (supports .md and .json PRDs)
 loki start --provider claude ./prd.md        # Default, full features
 loki start --provider codex ./prd.json       # GPT-5.3 Codex, degraded mode
-loki start --provider gemini ./prd.md        # Gemini 3 Pro, degraded mode
 loki start --provider cline ./prd.md         # Cline CLI, degraded mode
 loki start --provider aider ./prd.md         # Aider (18+ providers), degraded mode
 
@@ -225,9 +224,10 @@ loki start 123 --ship                        # Issue -> PR -> auto-merge
 **Provider capabilities:**
 - **Claude**: Opus 4.6, 1M context (beta), 128K output, adaptive thinking, agent teams, full features (Task tool, parallel agents, MCP)
 - **Codex**: GPT-5.3, 400K context, 128K output, MCP support, --full-auto mode, degraded (sequential only, no Task tool)
-- **Gemini**: Degraded mode (sequential only, no Task tool, 1M context)
 - **Cline**: Multi-provider CLI, degraded mode (sequential only, no Task tool)
 - **Aider**: 18+ provider backends, degraded mode (sequential only, no Task tool)
+- **Google Gemini CLI**: DEPRECATED starting v7.5.18 (upstream deprecated; runtime removed)
+- **Anthropic Antigravity CLI**: Coming soon
 
 ---
 
@@ -350,7 +350,7 @@ See `CHANGELOG.md` entries [7.5.7], [7.5.8], [7.5.13] for the per-fix list and r
 
 | Feature | Added | Notes |
 |---------|-------|-------|
-| Multi-provider support (5 providers) | v5.0.0 | claude, codex, gemini, cline, aider -- see `providers/` |
+| Multi-provider support (4 providers) | v5.0.0 | claude, codex, cline, aider -- see `providers/` |
 | CONTINUITY.md working memory | v5.35.0 | Auto-managed by run.sh, updated each iteration |
 | Quality gates 3-reviewer system | v5.35.0 | 5 specialist reviewers in `skills/quality-gates.md`; execution in run.sh |
 | Memory System (episodic/semantic/procedural) | v5.15.0 | Full implementation in `memory/` |
@@ -381,4 +381,4 @@ See `CHANGELOG.md` entries [7.5.7], [7.5.8], [7.5.13] for the per-fix list and r
 
 ---
 
-**v7.5.17 | [Autonomi](https://www.autonomi.dev/) flagship product | ~260 lines core**
+**v7.5.28 | [Autonomi](https://www.autonomi.dev/) flagship product | ~260 lines core**

package/VERSION

@@ -1 +1 @@
-7.5.17
+7.5.28

package/autonomy/completion-council.sh

@@ -1518,9 +1518,32 @@ council_evaluate() {
     # Compute threshold using the same ceiling(2/3) formula as council_vote and council_aggregate_votes
     local _eval_threshold=$(( (COUNCIL_SIZE * 2 + 2) / 3 ))
 
-    # Step 1: Aggregate votes from all members
-    local aggregate_result
-    aggregate_result=$(council_aggregate_votes)
+    # Step 1: Aggregate votes from all members.
+    # Phase C (v7.5.20): try the Claude `--agents <json>` + `--json-schema`
+    # dispatch first. When the locally installed Claude CLI supports both flags
+    # AND the call returns parseable findings, the helper writes both per-voter
+    # verdict files AND the round-N.json shape consumed by the existing
+    # transcript writer / aggregator readers downstream. On any failure
+    # (unsupported flags, missing binary, parse error, etc.) it returns 1 and
+    # we fall through to the existing heuristic council_aggregate_votes path.
+    local aggregate_result=""
+    local _va_helper
+    _va_helper="$(dirname "${BASH_SOURCE[0]}")/lib/voter-agents.sh"
+    if [ -f "$_va_helper" ]; then
+        # shellcheck disable=SC1090
+        . "$_va_helper" 2>/dev/null || true
+        if declare -f loki_council_dispatch_agents >/dev/null 2>&1; then
+            if loki_council_dispatch_agents "$ITERATION_COUNT" "${COUNCIL_PRD_PATH:-}"; then
+                local _va_round_file="$COUNCIL_STATE_DIR/votes/round-${ITERATION_COUNT}.json"
+                if [ -f "$_va_round_file" ]; then
+                    aggregate_result=$(_RF="$_va_round_file" python3 -c "import json, os; print(json.load(open(os.environ['_RF'])).get('verdict', 'CONTINUE'))" 2>/dev/null || echo "")
+                fi
+            fi
+        fi
+    fi
+    if [ -z "$aggregate_result" ]; then
+        aggregate_result=$(council_aggregate_votes)
+    fi
 
     if [ "$aggregate_result" = "COMPLETE" ]; then
         # Step 2: Check if unanimous -- compare complete_count to COUNCIL_SIZE

package/autonomy/lib/claude-flags.sh

@@ -0,0 +1,132 @@
+#!/usr/bin/env bash
+# autonomy/lib/claude-flags.sh -- Phase B (v7.5.19) helpers.
+#
+# Compute Claude Code CLI flag values automatically from existing Loki state.
+# No new env vars introduced (per binding constraint: nothing user-facing changes).
+# Every value derives from RARV tier, complexity score, and budget config.
+#
+# Public API (all functions are pure: they read env + filesystem, write stdout).
+#   loki_effort_for_tier <tier> [complexity]    -- emit one of: low|medium|high|xhigh|max
+#   loki_remaining_budget                        -- emit dollar amount remaining as plain int/float, OR empty when unlimited
+#   loki_fallback_for_primary <model>           -- emit fallback model alias (opus->sonnet, sonnet->haiku if allowed), OR empty when no fallback applies
+#   loki_claude_flag_supported <flag>           -- 0 if `claude --help` lists the flag, 1 otherwise
+#
+# All functions are side-effect free. The caller (providers/claude.sh, loki-ts) decides whether to pass the result.
+
+# Guard against double-source.
+if [ "${__LOKI_CLAUDE_FLAGS_SH_LOADED:-0}" = "1" ]; then
+    return 0 2>/dev/null || true
+fi
+__LOKI_CLAUDE_FLAGS_SH_LOADED=1
+
+# ---------- Effort tier mapping ----------
+# RARV tier names: planning (xhigh), development (high), fast (medium).
+# Complexity shift: if the project is "complex", bump each tier one notch up.
+# Honors LOKI_ALLOW_HAIKU=true semantics for the "fast" tier (medium stays medium; haiku is a separate model concept).
+loki_effort_for_tier() {
+    local tier="${1:-development}"
+    local complexity="${2:-${LOKI_COMPLEXITY:-standard}}"
+    local effort=""
+    case "$tier" in
+        planning)    effort="xhigh" ;;
+        development) effort="high" ;;
+        fast)        effort="medium" ;;
+        # Capability aliases used elsewhere in the codebase.
+        best)        effort="xhigh" ;;
+        balanced)    effort="high" ;;
+        cheap)       effort="low" ;;
+        *)           effort="high" ;;
+    esac
+    # Complex projects get one bump up. "max" is the ceiling and only for explicit user override (not auto-bumped to from xhigh).
+    if [ "$complexity" = "complex" ]; then
+        case "$effort" in
+            low)    effort="medium" ;;
+            medium) effort="high" ;;
+            high)   effort="xhigh" ;;
+            # xhigh stays xhigh; max not auto-reached.
+        esac
+    fi
+    printf '%s' "$effort"
+}
+
+# ---------- Remaining budget ----------
+# Compute remaining budget = LOKI_BUDGET_LIMIT - cumulative spend so far.
+# Spend lives in .loki/metrics/budget.json under "current_spend" (per autonomy/run.sh:8167+).
+# Emit empty string when budget is unlimited (LOKI_BUDGET_LIMIT unset or 0).
+# Emit empty when remaining <= 0 (caller decides what to do; we never emit 0).
+loki_remaining_budget() {
+    local limit="${LOKI_BUDGET_LIMIT:-}"
+    # Empty or zero limit means no cap; emit nothing.
+    if [ -z "$limit" ] || [ "$limit" = "0" ] || [ "$limit" = "0.0" ] || [ "$limit" = "0.00" ]; then
+        return 0
+    fi
+    local budget_file="${TARGET_DIR:-.}/.loki/metrics/budget.json"
+    local spend="0"
+    if [ -f "$budget_file" ]; then
+        spend=$(python3 -c "
+import json, sys
+try:
+    with open('$budget_file') as f:
+        d = json.load(f)
+    v = d.get('current_spend', 0)
+    print(float(v))
+except Exception:
+    print(0)
+" 2>/dev/null)
+    fi
+    # Compute remaining via python3 (bash floats are unreliable across awk/bc variations).
+    python3 -c "
+import sys
+try:
+    limit = float('$limit')
+    spend = float('$spend')
+    rem = limit - spend
+    # Strictly positive; otherwise emit nothing (caller decides whether to bail or warn).
+    if rem > 0:
+        # Round to 2 decimal places for the CLI.
+        print(f'{rem:.2f}')
+except Exception:
+    pass
+" 2>/dev/null
+}
+
+# ---------- Fallback model ----------
+# Map primary model alias to a sensible fallback for `--fallback-model`.
+# opus -> sonnet (always safe; sonnet is cheaper + similar capability)
+# sonnet -> haiku ONLY when LOKI_ALLOW_HAIKU=true; else emit nothing
+# haiku -> nothing (already at the bottom)
+# Anything else (specific dated IDs, alt-provider names) -> nothing (no auto-fallback we can reason about)
+loki_fallback_for_primary() {
+    local primary="${1:-}"
+    case "$primary" in
+        opus)   printf '%s' "sonnet" ;;
+        sonnet)
+            if [ "${LOKI_ALLOW_HAIKU:-false}" = "true" ]; then
+                printf '%s' "haiku"
+            fi
+            ;;
+        # No fallback for haiku or other model names.
+        *) ;;
+    esac
+}
+
+# ---------- Flag-support detection ----------
+# Returns 0 if `claude --help` lists the named flag, 1 otherwise.
+# Cached per-process so we do not shell out N times per iteration.
+loki_claude_flag_supported() {
+    local flag="${1:-}"
+    [ -z "$flag" ] && return 1
+    # Per-process cache: __LOKI_CLAUDE_HELP_CACHE populated on first call.
+    if [ -z "${__LOKI_CLAUDE_HELP_CACHE:-}" ]; then
+        if command -v claude >/dev/null 2>&1; then
+            __LOKI_CLAUDE_HELP_CACHE=$(claude --help 2>&1 || true)
+        else
+            __LOKI_CLAUDE_HELP_CACHE="__no_claude__"
+        fi
+        export __LOKI_CLAUDE_HELP_CACHE
+    fi
+    case "$__LOKI_CLAUDE_HELP_CACHE" in
+        *"$flag"*) return 0 ;;
+        *) return 1 ;;
+    esac
+}

package/autonomy/lib/mcp-config.sh

@@ -0,0 +1,160 @@
+#!/usr/bin/env bash
+# autonomy/lib/mcp-config.sh -- Phase D (v7.5.22) helpers.
+#
+# Compute MCP-config paths for the Claude Code CLI `--mcp-config` flag.
+# The flag is variadic per Commander (`--mcp-config <configs...>`), which
+# means Claude expects each path as a SEPARATE argv element following the
+# flag. The helper here emits a space-separated string for ergonomics; the
+# caller in `providers/claude.sh::_loki_build_claude_auto_flags` splits on
+# whitespace and pushes each path as its own array entry, so the variadic
+# shape is honored on the wire. Paths must not contain whitespace (both the
+# Loki bundle path and the `$HOME`-rooted overlay path are whitespace-safe
+# under normal use; the helper does not quote inside the joined string).
+#
+# Public API (all functions read env + filesystem, write stdout):
+#   loki_mcp_config_path        -- emit absolute path to .loki/mcp-config.json.
+#                                  Re-emits (overwrites) the bundle each call.
+#                                  Returns 0 on success, 1 only if the dir
+#                                  cannot be created or the file cannot be
+#                                  written.
+#   loki_user_mcp_config_path   -- emit absolute path to ~/.claude/mcp.json
+#                                  if present + readable, else empty. Always
+#                                  returns 0.
+#   loki_mcp_config_argv        -- emit one space-separated string of paths
+#                                  (Loki bundle first, optional user overlay
+#                                  second). Caller MUST split on whitespace
+#                                  and pass each path as a separate argv
+#                                  element to satisfy Claude's variadic flag.
+#                                  Returns 0 on success, 1 if the bundle
+#                                  emission fails.
+#
+# No side effects beyond writing .loki/mcp-config.json (idempotent;
+# regenerated each call).
+
+# Guard against double-source.
+if [ "${__LOKI_MCP_CONFIG_SH_LOADED:-0}" = "1" ]; then
+    return 0 2>/dev/null || true
+fi
+__LOKI_MCP_CONFIG_SH_LOADED=1
+
+# ---------- Loki MCP bundle path ----------
+# Emits the absolute path to .loki/mcp-config.json (TARGET_DIR-relative).
+# Bundle includes the always-on `loki-mode` server. Phase G (v7.5.24):
+# when any supported LSP binary (typescript-language-server, pylsp, gopls,
+# rust-analyzer) is on PATH, the bundle also gets a single `lsp-proxy`
+# entry that fans out to per-language servers at runtime. Detection uses
+# `command -v` only; absence is a normal state, not an error.
+#
+# Idempotent-write: the new bundle bytes are compared against the existing
+# file (`cmp -s`); the file is only rewritten when content differs. This
+# preserves mtime across calls when nothing changed, which matters for the
+# Phase G LSP-detection regression test (no LSP -> no bundle churn).
+#
+# The bundle mirrors the repo's .mcp.json `loki-mode` entry: a single
+# stdio MCP server backed by `python3 -m mcp.server`. Caller may extend
+# this in the future without API breakage; consumers should treat the
+# bundle as opaque JSON.
+loki_mcp_config_path() {
+    local base="${TARGET_DIR:-.}"
+    local mcp_dir="${base}/.loki"
+    local mcp_path="${mcp_dir}/mcp-config.json"
+
+    # Resolve to absolute path early so callers always get a stable value
+    # even if cwd changes later in the iteration.
+    if ! mkdir -p "$mcp_dir" 2>/dev/null; then
+        return 1
+    fi
+
+    # Phase G: detect supported LSP binaries on PATH. The lsp-proxy server
+    # routes by language at runtime, so we register it once when ANY of the
+    # supported binaries is present (multiple binaries -> still one entry).
+    local lsp_detected=0
+    local lsp_bin
+    for lsp_bin in typescript-language-server pylsp gopls rust-analyzer; do
+        if command -v "$lsp_bin" >/dev/null 2>&1; then
+            lsp_detected=1
+            break
+        fi
+    done
+
+    # Build the bundle to a temp file, then compare bytes before rewriting.
+    # This makes the helper idempotent: identical bundle bytes -> no write,
+    # mtime stable. python3 is used so JSON encoding is canonical.
+    local tmp_bundle
+    tmp_bundle="${mcp_path}.tmp.$$"
+    if ! _MCP_OUT="$tmp_bundle" _MCP_LSP="$lsp_detected" python3 -c "
+import json, os
+out = os.environ['_MCP_OUT']
+lsp = os.environ.get('_MCP_LSP', '0') == '1'
+servers = {
+    'loki-mode': {
+        'command': 'python3',
+        'args': ['-m', 'mcp.server'],
+    },
+}
+if lsp:
+    servers['lsp-proxy'] = {
+        'command': 'python3',
+        'args': ['-m', 'mcp.lsp_proxy'],
+    }
+bundle = {'mcpServers': servers}
+with open(out, 'w') as f:
+    json.dump(bundle, f, indent=2)
+" 2>/dev/null; then
+        rm -f "$tmp_bundle" 2>/dev/null
+        return 1
+    fi
+
+    # Idempotent write: only replace the file when bytes differ. cmp -s
+    # exits 0 when identical, so we keep the existing file in that case.
+    if [ -f "$mcp_path" ] && cmp -s "$tmp_bundle" "$mcp_path" 2>/dev/null; then
+        rm -f "$tmp_bundle" 2>/dev/null
+    else
+        mv -f "$tmp_bundle" "$mcp_path" 2>/dev/null || {
+            rm -f "$tmp_bundle" 2>/dev/null
+            return 1
+        }
+    fi
+
+    # Emit absolute path -- python3 handles realpath portably.
+    _MCP_OUT="$mcp_path" python3 -c "
+import os, sys
+print(os.path.abspath(os.environ['_MCP_OUT']))
+" 2>/dev/null
+    return 0
+}
+
+# ---------- User overlay path ----------
+# Echoes ~/.claude/mcp.json if it exists and is readable, else empty.
+# Always returns 0 -- a missing overlay is a normal state, not an error.
+loki_user_mcp_config_path() {
+    local user_path="${HOME}/.claude/mcp.json"
+    if [ -f "$user_path" ] && [ -r "$user_path" ]; then
+        printf '%s' "$user_path"
+    fi
+    return 0
+}
+
+# ---------- Combined --mcp-config argv value ----------
+# Emits a single space-separated string of paths. The caller in
+# `providers/claude.sh` splits on whitespace and pushes each path as its own
+# argv element so Claude's variadic `--mcp-config <configs...>` receives
+# separate argv entries (not one joined value). Loki bundle first, then
+# user overlay if present. Paths must not contain whitespace.
+#
+# Returns 1 if the Loki bundle cannot be emitted (the caller should then
+# skip the flag entirely rather than pass a malformed value).
+loki_mcp_config_argv() {
+    local loki_path user_path
+    loki_path=$(loki_mcp_config_path) || return 1
+    if [ -z "$loki_path" ]; then
+        return 1
+    fi
+    user_path=$(loki_user_mcp_config_path)
+    if [ -n "$user_path" ]; then
+        printf '%s %s' "$loki_path" "$user_path"
+    else
+        printf '%s' "$loki_path"
+    fi
+    return 0
+}