chore(obsidian): remove the browser Obsidian GUI service (memory system intact)#85
Merged
Merged
Conversation
…em intact) The vault is now browsed with native Obsidian opened at data/memory-vault; the streamed-desktop GUI service is removed. The memory substrate is UNTOUCHED — the memory-vault MCP (mcpvault), the vault dir, and rag-ingestion -> qdrant semantic indexing all stay. Removed (GUI only): - v2/plugins/obsidian/plugin.yaml (the whole service plugin). - Caddy /obsidian/* reverse-proxy route (auth/caddy/Caddyfile). - dashboard SERVICES entry + SSO_ROUTES['obsidian'] + ops-api ALLOWED_SERVICES entry + the obsidian-shape regression test. - obsidian-specific render tests; renamed test_obsidian_memory.py -> test_memory_vault.py, KEEPING the generic shm_size renderer test and all memory-vault MCP tests. Parity/plugin counts adjusted (13->12 service plugins). Kept: the generic `shm_size` schema field + its generic renderer test (reusable). Stale "Obsidian GUI browses the same dir" comments updated to "native Obsidian". Gates: v2 ruff clean + 206 passed / 6 skipped; root dashboard tests green; re-render diff = ONLY the obsidian service removed, llamacpp block byte-identical (GPU-97fe65ee pin preserved); docker compose config exit 0. Co-Authored-By: Claude Opus 4.8 <[email protected]>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
What
Removes the streamed-desktop Obsidian GUI service (the Selkies/KasmVNC
linuxserver/obsidiancontainer). The operator now browses the vault with native Windows Obsidian opened atC:\dev\ordo-ai-stack\data\memory-vault.The memory system is 100% intact — the vault dir, the
memory-vaultMCP (gateway-spawned mcpvault), and the rag-ingestion → qdrant semantic indexing are all UNTOUCHED. Only the GUI viewer service is removed.Changes
v2/plugins/obsidian/plugin.yaml(the whole service plugin)./obsidian/*reverse-proxy route (auth/caddy/Caddyfile).dashboard/services_catalog.pySERVICES,SSO_ROUTES['obsidian']indashboard/static/index.html(CRLF preserved), theobsidianentry inv2/docker/ops-api/main.pyALLOWED_SERVICES, and the obsidian-shape regression test intests/test_services_and_throughput.py.test_obsidian_memory.py→test_memory_vault.py, keeping the genericshm_sizerenderer test and all memory-vault MCP tests. Parity/plugin counts adjusted (13→12 service plugins) intest_parity_render.py+test_plugins.py.shm_sizeschema field inplugins.py/compose.py(reusable). Stale "Obsidian GUI browses the same dir" comments updated to "native Obsidian".v2/README.mdplugin list + vaultREADME.mdupdated to say the vault is browsed via native Obsidian.Validation evidence
--source out/ordo.yaml): diff = ONLY the obsidian service removed; llamacpp block byte-identical (md5 match,GPU-97fe65eepin preserved, 9=9 pin refs).docker compose config→ exit 0.--no-deps, both env files); caddy reloaded gracefully.read_note CONVENTIONS.mdthrough the gateway → 200 with real note content.documentscollection intact, 18h/15h uptime./obsidian/through the edge: now 302→SSO→404 fall-through (no 502, no error spam in caddy logs).data/obsidian-configdeleted,linuxserver/obsidianimage removed.Attestation
Only the obsidian GUI removed; dashboard/ops-api/caddy updated to match. GPU services (llamacpp/agent/comfyui) and the memory system (vault + memory-vault MCP + rag/qdrant) untouched.
🤖 Generated with Claude Code