Merge pull request #11 from DMontgomery40/cleanup

Cleanup

Author: DMontgomery40

.env.example

@@ -3,6 +3,7 @@
 
 # ===== API Keys =====
 OPENAI_API_KEY=sk-your-openai-key
+OPENROUTER_API_KEY=your-openrouter-key  # Optional (only if chat.openrouter.enabled=true)
 VOYAGE_API_KEY=your-voyage-key
 COHERE_API_KEY=your-cohere-key
 JINA_API_KEY=your-jina-key

README.md

@@ -1,5 +1,5 @@
 <p align="center">
-  <img src="mkdocs/docs/assets/images/tribrid-logo.png" alt="TriBridRAG" width="200" />
+  <img src="web/public/favicon.svg" alt="TriBridRAG" width="200" />
 </p>
 
 <h1 align="center">TriBridRAG</h1>
@@ -18,6 +18,36 @@
 
 ---
 
+## Screenshots
+
+<details>
+  <summary>
+    <strong>RAG Chat + Eval Dataset Manager</strong> — grounded answers with sources, plus regression evaluation (click to expand)
+    <br />
+    <img src="assets/chat-and-dataset.png" alt="TriBridRAG UI showing RAG chat next to the evaluation dataset manager" width="900" />
+  </summary>
+  <br />
+  <p align="center">
+    <a href="assets/chat-and-dataset.png">
+      <img src="assets/chat-and-dataset.png" alt="TriBridRAG UI showing RAG chat next to the evaluation dataset manager" width="1000" />
+    </a>
+  </p>
+</details>
+
+<details>
+  <summary>
+    <strong>Indexing Settings + Grafana Dashboards</strong> — tune chunking/embedding and watch indexing metrics live (click to expand)
+    <br />
+    <img src="assets/indexing-settings-with-grafana-on-side.png" alt="TriBridRAG indexing settings UI alongside Grafana dashboards" width="900" />
+  </summary>
+  <br />
+  <p align="center">
+    <a href="assets/indexing-settings-with-grafana-on-side.png">
+      <img src="assets/indexing-settings-with-grafana-on-side.png" alt="TriBridRAG indexing settings UI alongside Grafana dashboards" width="1000" />
+    </a>
+  </p>
+</details>
+
 ## The Problem
 
 Single-method RAG systems fail in predictable ways:

assets/chat-and-dataset.png

@@ -106,6 +106,18 @@
     "OPENAI_API_KEY",
     "OPENROUTER_API_KEY",
     "COHERE_API_KEY",
+    "ANTHROPIC_API_KEY",
+    "GOOGLE_API_KEY",
+    "VOYAGE_API_KEY",
+    "JINA_API_KEY",
+    # Integrations (UI-only presence checks; values are never returned)
+    "LANGTRACE_API_KEY",
+    "LANGCHAIN_API_KEY",
+    "LANGSMITH_API_KEY",
+    "NETLIFY_API_KEY",
+    "GRAFANA_API_KEY",
+    "SLACK_WEBHOOK_URL",
+    "DISCORD_WEBHOOK_URL",
     # Postgres infra
     "POSTGRES_DSN",
     "POSTGRES_HOST",

assets/indexing-settings-with-grafana-on-side.png

@@ -318,6 +318,7 @@ def main() -> None:
             EvalRunMeta,
             EvalRunsResponse,
             EvalAnalyzeComparisonResponse,
+            EvalAnalyzeComparisonRequest,
             EvalComparisonResult,
             # Domain models - Reranker training eval
             CorpusEvalProfile,
@@ -330,6 +331,11 @@ def main() -> None:
             RerankerTrainMetricsResponse,
             RerankerTrainDiffRequest,
             RerankerTrainDiffResponse,
+            # Domain models - Prompts (Eval → System Prompts)
+            PromptsResponse,
+            PromptMetadata,
+            PromptUpdateRequest,
+            PromptUpdateResponse,
         )
     except ImportError as e:
         print(f"ERROR: Could not import models: {e}")
@@ -407,6 +413,7 @@ def main() -> None:
         EvalRunMeta,
         EvalRunsResponse,
         EvalAnalyzeComparisonResponse,
+        EvalAnalyzeComparisonRequest,
         EvalComparisonResult,
         CorpusEvalProfile,
         RerankerTrainStartRequest,
@@ -418,6 +425,10 @@ def main() -> None:
         RerankerTrainMetricsResponse,
         RerankerTrainDiffRequest,
         RerankerTrainDiffResponse,
+        PromptsResponse,
+        PromptMetadata,
+        PromptUpdateRequest,
+        PromptUpdateResponse,
     ]
 
     print(f"Processing {len(all_models)} models...")

scripts/check_banned.py

@@ -0,0 +1,39 @@
+#!/usr/bin/env python3
+"""
+Print the configured Codex CLI model from ~/.codex/config.toml.
+
+This is intentionally simple and machine-readable (prints just the model string).
+"""
+
+from __future__ import annotations
+
+from pathlib import Path
+
+
+def main() -> None:
+    config_path = Path("~/.codex/config.toml").expanduser()
+    if not config_path.exists():
+        print("")
+        return
+
+    try:
+        import tomllib  # py>=3.11
+    except Exception:
+        print("")
+        return
+
+    try:
+        data = tomllib.loads(config_path.read_text(encoding="utf-8"))
+    except Exception:
+        print("")
+        return
+
+    model = data.get("model", "")
+    if model is None:
+        model = ""
+    print(model)
+
+
+if __name__ == "__main__":
+    main()
+

scripts/generate_types.py

@@ -24,6 +24,26 @@
 
 router = APIRouter(tags=["config"])
 
+_SECRETS_CHECK_ALLOWLIST = {
+    # Provider secrets
+    "OPENAI_API_KEY",
+    "OPENROUTER_API_KEY",
+    "ANTHROPIC_API_KEY",
+    "GOOGLE_API_KEY",
+    "COHERE_API_KEY",
+    "VOYAGE_API_KEY",
+    "JINA_API_KEY",
+    # Integrations (UI surfaces these; backend does not expose values)
+    "LANGTRACE_API_KEY",
+    "LANGCHAIN_API_KEY",
+    "LANGSMITH_API_KEY",
+    "NETLIFY_API_KEY",
+    "GRAFANA_API_KEY",
+    "SLACK_WEBHOOK_URL",
+    "DISCORD_WEBHOOK_URL",
+}
+
+
 @router.get("/config", response_model=TriBridConfig)
 async def get_config(scope: CorpusScope = Depends()) -> TriBridConfig:
     repo_id = scope.resolved_repo_id
@@ -106,7 +126,35 @@ async def reset_config(scope: CorpusScope = Depends()) -> TriBridConfig:
 async def check_secrets(keys: str = Query(..., description="Comma-separated env var names")) -> dict[str, bool]:
     """Return which secret env vars are configured (never returns values)."""
     names = [k.strip() for k in (keys or "").split(",") if k.strip()]
-    return {name: bool(os.getenv(name)) for name in names}
+    if not names:
+        return {}
+
+    invalid = [name for name in names if name not in _SECRETS_CHECK_ALLOWLIST]
+    if invalid:
+        raise HTTPException(
+            status_code=400,
+            detail=f"Unsupported secret key(s): {', '.join(sorted(set(invalid)))}",
+        )
+
+    # Use explicit getenv calls (no dynamic env lookups) to keep env usage auditable/"LAW"-compliant.
+    status = {
+        "OPENAI_API_KEY": bool(os.getenv("OPENAI_API_KEY")),
+        "OPENROUTER_API_KEY": bool(os.getenv("OPENROUTER_API_KEY")),
+        "ANTHROPIC_API_KEY": bool(os.getenv("ANTHROPIC_API_KEY")),
+        "GOOGLE_API_KEY": bool(os.getenv("GOOGLE_API_KEY")),
+        "COHERE_API_KEY": bool(os.getenv("COHERE_API_KEY")),
+        "VOYAGE_API_KEY": bool(os.getenv("VOYAGE_API_KEY")),
+        "JINA_API_KEY": bool(os.getenv("JINA_API_KEY")),
+        "LANGTRACE_API_KEY": bool(os.getenv("LANGTRACE_API_KEY")),
+        "LANGCHAIN_API_KEY": bool(os.getenv("LANGCHAIN_API_KEY")),
+        "LANGSMITH_API_KEY": bool(os.getenv("LANGSMITH_API_KEY")),
+        "NETLIFY_API_KEY": bool(os.getenv("NETLIFY_API_KEY")),
+        "GRAFANA_API_KEY": bool(os.getenv("GRAFANA_API_KEY")),
+        "SLACK_WEBHOOK_URL": bool(os.getenv("SLACK_WEBHOOK_URL")),
+        "DISCORD_WEBHOOK_URL": bool(os.getenv("DISCORD_WEBHOOK_URL")),
+    }
+
+    return {name: status[name] for name in names}
 
 
 @router.get("/mcp/status", response_model=MCPStatusResponse)