arkaos 2.86.0 → 2.88.0

package/VERSION

@@ -1 +1 @@
-2.86.0
+2.88.0

package/dashboard/app/pages/health.vue

@@ -1,14 +1,153 @@
 <script setup lang="ts">
-import type { HealthCheck } from '~/types'
+// PR70 v2.87.0 — Health page polish.
+// - 30s auto-refresh (paused while tab is hidden) + manual refresh
+// - Last-checked timestamp in header
+// - Severity-aware rendering (fail = red, warn = yellow)
+// - Copy-fix button when a check has a fix command
+// - Healthy banner ignores warnings (only blocking failures matter)
+
+interface HealthCheck {
+  name: string
+  passed: boolean
+  fix: string
+  severity: 'fail' | 'warn'
+}
+
+interface HealthPayload {
+  checks: HealthCheck[]
+  passed: number
+  total: number
+  failed_blocking: number
+  warning_count: number
+  healthy: boolean
+  ts: string
+}
 
 const { fetchApi } = useApi()
+const toast = useToast()
+
+const {
+  data,
+  status,
+  error,
+  refresh,
+} = await fetchApi<HealthPayload>('/api/health')
+
+// ─── Auto-refresh ───────────────────────────────────────────────────────
+
+let pollTimer: ReturnType<typeof setInterval> | null = null
+
+function startPolling() {
+  stopPolling()
+  pollTimer = setInterval(() => {
+    refresh()
+  }, 30_000)
+}
+
+function stopPolling() {
+  if (pollTimer !== null) {
+    clearInterval(pollTimer)
+    pollTimer = null
+  }
+}
+
+function handleVisibility() {
+  if (typeof document === 'undefined') return
+  if (document.hidden) {
+    stopPolling()
+  } else {
+    refresh()
+    startPolling()
+  }
+}
+
+onMounted(() => {
+  if (typeof document !== 'undefined') {
+    document.addEventListener('visibilitychange', handleVisibility)
+  }
+  startPolling()
+})
+
+onBeforeUnmount(() => {
+  stopPolling()
+  if (typeof document !== 'undefined') {
+    document.removeEventListener('visibilitychange', handleVisibility)
+  }
+})
+
+// ─── Copy fix ───────────────────────────────────────────────────────────
+
+const copied = ref<string | null>(null)
+let copyTimer: ReturnType<typeof setTimeout> | null = null
+
+async function copyFix(check: HealthCheck) {
+  if (!check.fix) return
+  if (typeof navigator === 'undefined' || !navigator.clipboard) {
+    toast.add({ title: 'Clipboard unavailable', color: 'warning' })
+    return
+  }
+  try {
+    await navigator.clipboard.writeText(check.fix)
+    copied.value = check.name
+    if (copyTimer) clearTimeout(copyTimer)
+    copyTimer = setTimeout(() => { copied.value = null; copyTimer = null }, 1500)
+    toast.add({
+      title: 'Fix copied',
+      description: check.fix,
+      color: 'success',
+    })
+  } catch (err) {
+    toast.add({
+      title: 'Copy failed',
+      description: err instanceof Error ? err.message : 'unknown error',
+      color: 'error',
+    })
+  }
+}
 
-const { data, status, error, refresh } = await fetchApi<{ checks: HealthCheck[], passed: number, total: number }>('/api/health')
+onBeforeUnmount(() => {
+  if (copyTimer) clearTimeout(copyTimer)
+})
 
-const checks = computed(() => data.value?.checks ?? [])
+// ─── Format helpers ─────────────────────────────────────────────────────
+
+function formatTs(iso: string | undefined): string {
+  if (!iso) return ''
+  try {
+    return new Intl.DateTimeFormat('en-US', {
+      hour: '2-digit', minute: '2-digit', second: '2-digit',
+    }).format(new Date(iso))
+  } catch {
+    return iso
+  }
+}
+
+type CheckStatus = 'pass' | 'warn' | 'fail'
+
+function statusOf(c: HealthCheck): CheckStatus {
+  if (c.passed) return 'pass'
+  return c.severity === 'warn' ? 'warn' : 'fail'
+}
+
+const STATUS_META: Record<CheckStatus, { icon: string; color: string; label: string }> = {
+  pass: { icon: 'i-lucide-check-circle', color: 'text-green-500',  label: 'Pass' },
+  warn: { icon: 'i-lucide-alert-circle',  color: 'text-yellow-500', label: 'Warn' },
+  fail: { icon: 'i-lucide-x-circle',      color: 'text-red-500',    label: 'Fail' },
+}
+
+function statusBadgeColor(s: CheckStatus): 'success' | 'warning' | 'error' {
+  return s === 'pass' ? 'success' : s === 'warn' ? 'warning' : 'error'
+}
+
+// ─── Aggregate display ──────────────────────────────────────────────────
+
+const checks = computed<HealthCheck[]>(() => data.value?.checks ?? [])
 const passed = computed(() => data.value?.passed ?? 0)
 const total = computed(() => data.value?.total ?? 0)
-const allPassed = computed(() => passed.value === total.value && total.value > 0)
+const failedBlocking = computed(() => data.value?.failed_blocking ?? 0)
+const warningCount = computed(() => data.value?.warning_count ?? 0)
+const allPassed = computed(() => failedBlocking.value === 0 && warningCount.value === 0 && total.value > 0)
+const someWarnings = computed(() => failedBlocking.value === 0 && warningCount.value > 0)
 </script>
 
 <template>
@@ -19,11 +158,28 @@ const allPassed = computed(() => passed.value === total.value && total.value > 0
           <UDashboardSidebarCollapse />
         </template>
         <template #trailing>
+          <span
+            v-if="data?.ts"
+            class="text-xs text-muted"
+            :title="data.ts"
+          >
+            Last checked {{ formatTs(data.ts) }}
+          </span>
           <UBadge
             v-if="data"
             :label="`${passed}/${total}`"
-            :color="allPassed ? 'success' : 'warning'"
+            :color="allPassed ? 'success' : someWarnings ? 'warning' : 'error'"
             variant="subtle"
+            class="ml-3"
+          />
+        </template>
+        <template #right>
+          <UButton
+            label="Refresh"
+            variant="ghost"
+            icon="i-lucide-refresh-cw"
+            size="sm"
+            @click="refresh()"
           />
         </template>
       </UDashboardNavbar>
@@ -39,45 +195,81 @@ const allPassed = computed(() => passed.value === total.value && total.value > 0
         loading-label="Loading health checks"
         :on-retry="() => refresh()"
       >
-        <!-- Overall Status -->
+        <!-- Overall banner -->
         <div
           class="mb-6 rounded-lg border p-6 text-center"
-          :class="allPassed ? 'border-green-500/30 bg-green-500/5' : 'border-yellow-500/30 bg-yellow-500/5'"
+          :class="allPassed
+            ? 'border-green-500/30 bg-green-500/5'
+            : someWarnings
+              ? 'border-yellow-500/30 bg-yellow-500/5'
+              : 'border-red-500/30 bg-red-500/5'"
         >
           <UIcon
-            :name="allPassed ? 'i-lucide-check-circle' : 'i-lucide-alert-circle'"
-            :class="allPassed ? 'text-green-500' : 'text-yellow-500'"
+            :name="allPassed
+              ? 'i-lucide-check-circle'
+              : someWarnings
+                ? 'i-lucide-alert-circle'
+                : 'i-lucide-x-circle'"
+            :class="allPassed
+              ? 'text-green-500'
+              : someWarnings ? 'text-yellow-500' : 'text-red-500'"
             class="size-12"
           />
           <p class="mt-2 text-lg font-semibold text-highlighted">
-            {{ allPassed ? 'All Checks Passing' : `${total - passed} Check(s) Failing` }}
+            <template v-if="allPassed">All Checks Passing</template>
+            <template v-else-if="someWarnings">
+              {{ warningCount }} Warning{{ warningCount === 1 ? '' : 's' }}
+            </template>
+            <template v-else>
+              {{ failedBlocking }} Blocking Failure{{ failedBlocking === 1 ? '' : 's' }}
+            </template>
+          </p>
+          <p class="text-sm text-muted">
+            {{ passed }} of {{ total }} checks passed
+            <template v-if="warningCount && failedBlocking">
+              · {{ warningCount }} warn · {{ failedBlocking }} blocking
+            </template>
           </p>
-          <p class="text-sm text-muted">{{ passed }} of {{ total }} checks passed</p>
         </div>
 
-        <!-- Individual Checks -->
+        <!-- Check list -->
         <div class="space-y-3">
           <div
             v-for="check in checks"
             :key="check.name"
-            class="flex items-start gap-3 rounded-lg border border-default p-4"
+            class="flex items-start gap-3 rounded-lg border p-4"
+            :class="{
+              'border-default': check.passed,
+              'border-yellow-500/30 bg-yellow-500/5': !check.passed && check.severity === 'warn',
+              'border-red-500/30 bg-red-500/5': !check.passed && check.severity === 'fail',
+            }"
           >
             <UIcon
-              :name="check.passed ? 'i-lucide-check-circle' : 'i-lucide-x-circle'"
-              :class="check.passed ? 'text-green-500' : 'text-red-500'"
+              :name="STATUS_META[statusOf(check)].icon"
+              :class="STATUS_META[statusOf(check)].color"
               class="mt-0.5 size-5 shrink-0"
             />
-            <div class="flex-1">
+            <div class="flex-1 min-w-0">
               <h4 class="font-medium text-highlighted">{{ check.name }}</h4>
               <p v-if="!check.passed && check.fix" class="mt-1 text-sm text-muted">
-                Fix: {{ check.fix }}
+                Fix: <code class="font-mono text-xs">{{ check.fix }}</code>
               </p>
             </div>
+            <UButton
+              v-if="!check.passed && check.fix"
+              :icon="copied === check.name ? 'i-lucide-check' : 'i-lucide-copy'"
+              :color="copied === check.name ? 'success' : 'neutral'"
+              variant="ghost"
+              size="xs"
+              aria-label="Copy fix command"
+              @click="copyFix(check)"
+            />
             <UBadge
-              :label="check.passed ? 'Pass' : 'Fail'"
-              :color="check.passed ? 'success' : 'error'"
+              :label="STATUS_META[statusOf(check)].label"
+              :color="statusBadgeColor(statusOf(check))"
               variant="subtle"
               size="sm"
+              class="shrink-0"
             />
           </div>
         </div>

package/dashboard/app/pages/knowledge.vue

@@ -324,6 +324,90 @@ async function handleSearch() {
 function formatScore(score: number): string {
   return `${(score * 100).toFixed(0)}%`
 }
+
+// PR71 v2.88.0 — delete all chunks from a given source.
+
+const deletingSource = ref<string | null>(null)
+
+async function askDeleteSource(source: string) {
+  if (!source) return
+  if (typeof window === 'undefined') return
+  const ok = window.confirm(
+    `Delete every indexed chunk from this source?\n\n${source}\n\n`
+    + 'This removes the source from search results but does not delete the original file. '
+    + 'You can re-ingest the source later if needed.',
+  )
+  if (!ok) return
+  await deleteSource(source)
+}
+
+async function deleteSource(source: string) {
+  deletingSource.value = source
+  try {
+    const res = await $fetch<{ deleted?: number, source?: string, error?: string }>(
+      `${apiBase}/api/knowledge/sources`,
+      { method: 'DELETE', query: { source } },
+    )
+    if (res.error) {
+      toast.add({
+        title: 'Delete failed',
+        description: res.error,
+        color: 'error',
+      })
+      return
+    }
+    const deleted = res.deleted ?? 0
+    // Drop the matching rows from the in-memory list without a full re-fetch.
+    searchResults.value = searchResults.value.filter((r) => r.source !== source)
+    searchTotal.value = searchResults.value.length
+    // Refresh stats so the chunk count in the header updates.
+    if (typeof refresh === 'function') {
+      await refresh()
+    }
+    toast.add({
+      title: deleted > 0
+        ? `Deleted ${deleted} chunk${deleted === 1 ? '' : 's'}`
+        : 'Nothing to delete',
+      description: source,
+      color: 'success',
+    })
+  } catch (err) {
+    toast.add({
+      title: 'Delete failed',
+      description: err instanceof Error ? err.message : 'unknown error',
+      color: 'error',
+    })
+  } finally {
+    deletingSource.value = null
+  }
+}
+
+// PR71 — highlight the search query in the preview text.
+// Tolerates malformed regex (escapes special characters) and HTML-
+// escapes the input so v-html'd output is safe from XSS via DB rows.
+function highlightMatches(text: string, query: string): string {
+  const safe = escapeHtml(text || '')
+  const q = (query || '').trim()
+  if (!q) return safe
+  const pattern = new RegExp(`(${escapeRegex(q)})`, 'gi')
+  return safe.replace(
+    pattern,
+    '<mark class="bg-primary/20 text-primary rounded px-0.5">$1</mark>',
+  )
+}
+
+function escapeHtml(value: string): string {
+  return value
+    .replace(/&/g, '&amp;')
+    .replace(/</g, '&lt;')
+    .replace(/>/g, '&gt;')
+    .replace(/"/g, '&quot;')
+    .replace(/'/g, '&#39;')
+}
+
+function escapeRegex(value: string): string {
+  return value.replace(/[.*+?^${}()|[\]\\]/g, '\\$&')
+}
 </script>
 
 <template>
@@ -771,17 +855,30 @@ function formatScore(score: number): string {
                         {{ result.heading }}
                       </span>
                     </div>
-                    <span class="text-xs text-muted whitespace-nowrap">
-                      Score: {{ formatScore(result.score) }}
-                    </span>
+                    <div class="flex items-center gap-2 shrink-0">
+                      <span class="text-xs text-muted whitespace-nowrap">
+                        Score: {{ formatScore(result.score) }}
+                      </span>
+                      <UButton
+                        v-if="result.source"
+                        :icon="deletingSource === result.source
+                          ? 'i-lucide-loader-2'
+                          : 'i-lucide-trash-2'"
+                        :loading="deletingSource === result.source"
+                        variant="ghost"
+                        color="error"
+                        size="xs"
+                        aria-label="Delete all chunks from this source"
+                        @click.stop="askDeleteSource(result.source)"
+                      />
+                    </div>
                   </div>
                   <p v-if="result.source" class="text-xs text-muted mb-1 truncate">
                     <UIcon name="i-lucide-file-text" class="size-3 inline-block mr-1" />
                     {{ result.source }}
                   </p>
-                  <p class="text-sm text-muted line-clamp-3">
-                    {{ result.text || result.content }}
-                  </p>
+                  <!-- PR71 v2.88.0 — highlight query matches in the preview -->
+                  <p class="text-sm text-muted line-clamp-3" v-html="highlightMatches(result.text || result.content, searchQuery)" />
                 </div>
               </div>
 

package/dashboard/app/types/index.d.ts

@@ -175,6 +175,9 @@ export interface HealthCheck {
   name: string
   passed: boolean
   fix: string
+  // PR70 v2.87.0 — backend now tags every check with a severity.
+  // 'fail' is must-pass; 'warn' is recommended but non-blocking.
+  severity?: 'fail' | 'warn'
 }
 
 export interface Persona {

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "arkaos",
-  "version": "2.86.0",
+  "version": "2.88.0",
   "description": "The Operating System for AI Agent Teams",
   "type": "module",
   "bin": {

package/pyproject.toml

@@ -1,6 +1,6 @@
 [project]
 name = "arkaos-core"
-version = "2.86.0"
+version = "2.88.0"
 description = "Core engine for ArkaOS — The Operating System for AI Agent Teams"
 readme = "README.md"
 license = {text = "MIT"}

package/scripts/dashboard-api.py

@@ -612,20 +612,66 @@ def knowledge_search(q: str = Query(...), top_k: int = Query(5)):
     return {"results": results, "query": q, "total": len(results)}
 
 
+@app.delete("/api/knowledge/sources")
+def knowledge_delete_source(source: str = Query(...)):
+    """PR71 v2.88.0 — remove all chunks from a given source.
+
+    Operators sometimes ingest a noisy / wrong source and want to nuke
+    every chunk that came from it without rebuilding the whole vector
+    DB. The vector store already exposes `remove_file(source)` —
+    this endpoint just exposes it on the wire.
+
+    Returns ``{deleted: N, source: "..."}``. Refuses empty source
+    paths so a runaway client doesn't accidentally request "delete
+    everything that has no source".
+    """
+    clean = (source or "").strip()
+    if not clean:
+        return {"error": "source query param is required"}
+    store = _get_vector_store()
+    if not store:
+        return {"error": "vector store unavailable", "deleted": 0}
+    try:
+        deleted = store.remove_file(clean)
+    except Exception as exc:  # noqa: BLE001 — surface as 200+error
+        return {"error": f"delete failed: {exc}", "deleted": 0}
+    return {"deleted": int(deleted), "source": clean}
+
+
 @app.get("/api/health")
 def health():
-    checks = []
+    """PR70 v2.87.0 — per-check severity + response timestamp.
+
+    Each check now carries a `severity` field:
+      - "fail" — must-pass; missing breaks ArkaOS
+      - "warn" — recommended; missing means a degraded but workable env
+
+    Response also carries `ts` so the UI can show "last checked".
+    Frontend polls every 30s and surfaces copy-fix buttons.
+    """
+    from datetime import datetime, timezone
+
+    checks: list[dict] = []
     arkaos_home = Path.home() / ".arkaos"
 
-    def check(name, condition, fix=""):
-        checks.append({"name": name, "passed": condition, "fix": fix})
+    def check(name: str, condition: bool, fix: str = "", severity: str = "fail"):
+        checks.append({
+            "name": name,
+            "passed": condition,
+            "fix": fix,
+            "severity": severity,
+        })
 
-    check("install_dir", arkaos_home.exists(), "Run: npx arkaos install")
-    check("manifest", (arkaos_home / "install-manifest.json").exists(), "Run: npx arkaos install")
+    check("install_dir", arkaos_home.exists(), "npx arkaos install")
+    check("manifest", (arkaos_home / "install-manifest.json").exists(),
+          "npx arkaos install")
     check("constitution", (ARKAOS_ROOT / "config" / "constitution.yaml").exists())
-    check("agents_registry", (ARKAOS_ROOT / "knowledge" / "agents-registry-v2.json").exists())
-    check("commands_registry", (ARKAOS_ROOT / "knowledge" / "commands-registry-v2.json").exists())
-    check("hooks_dir", (arkaos_home / "config" / "hooks").exists(), "Run: npx arkaos install")
+    check("agents_registry",
+          (ARKAOS_ROOT / "knowledge" / "agents-registry-v2.json").exists())
+    check("commands_registry",
+          (ARKAOS_ROOT / "knowledge" / "commands-registry-v2.json").exists())
+    check("hooks_dir", (arkaos_home / "config" / "hooks").exists(),
+          "npx arkaos install")
 
     try:
         subprocess.run(["python3", "--version"], capture_output=True, timeout=2)
@@ -633,10 +679,34 @@ def health():
     except Exception:
         check("python", False, "Install Python 3.11+")
 
-    check("knowledge_db", (arkaos_home / "knowledge.db").exists(), "Run: npx arkaos index")
+    # Telemetry + knowledge — warn-only; missing them is a degraded
+    # but workable state (new installs, never-indexed-anything).
+    check("knowledge_db", (arkaos_home / "knowledge.db").exists(),
+          "Open the Knowledge tab and ingest a source",
+          severity="warn")
+    check("profile",
+          (arkaos_home / "profile.json").exists(),
+          "Open Settings → Profile to introduce yourself",
+          severity="warn")
 
     passed = sum(1 for c in checks if c["passed"])
-    return {"checks": checks, "passed": passed, "total": len(checks), "healthy": passed == len(checks)}
+    failed_blocking = sum(
+        1 for c in checks
+        if not c["passed"] and c["severity"] == "fail"
+    )
+    warning_count = sum(
+        1 for c in checks
+        if not c["passed"] and c["severity"] == "warn"
+    )
+    return {
+        "checks": checks,
+        "passed": passed,
+        "total": len(checks),
+        "failed_blocking": failed_blocking,
+        "warning_count": warning_count,
+        "healthy": failed_blocking == 0,
+        "ts": datetime.now(timezone.utc).isoformat(),
+    }
 
 
 # --- Personas ---