feat(abyss-integration): consume abyss skill-manifest for capability discovery (v4.8.0)#52
Merged
Merged
Conversation
…discovery (v4.8.0) Code-abyss 4.8.0 now reads abyss's skill-manifest JSON when the installed abyss binary is >= 0.5.22. The manifest is the contract: - providers.cli.commands — what abyss exposes - providers.mcp.tools — MCP tools list - providers.daemon.verbs — daemon socket capabilities - schema_version: 1 — shape pin Falls back to hard-coded defaults when abyss is missing, < 0.5.22, or the manifest fails to parse. The pre-existing injectClaudeHooks, hook scripts, and adapter shapes are unchanged — this patch is ADDITIVE. MIN_ABYSS_VERSION bumped 0.3.0 → 0.5.20 (the most recent stable; v0.5.x has been thoroughly dogfooded against hono/helix/vite/FastAPI/Django/ SQLAlchemy). 0.5.22+ unlocks dynamic capability discovery.
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.
Summary
code-abyss 4.8.0 reads abyss's
skill-manifestJSON at install time instead of carrying a hand-maintained mirror of abyss's surface. The pre-existing inject/adapter/MCP paths are unchanged — this patch is additive, with fallback to hand-coded defaults whenever the manifest is unavailable.What discovery actually picks up
Real smoke against
abyss 0.5.23:```
abyss v0.5.23: 19 CLI commands, 8 MCP tools; daemon verbs: ping, stats, reindex, logs, mcp, subscribe
MCP tools (from manifest): search_context, get_symbols, find_callers, impact_analysis,
code_map, evolution, index_project, arch_map ← discovery 自动有
MCP tools (null fallback): search_context, get_symbols, find_callers, impact_analysis,
code_map, evolution, index_project ← hand-coded 漏 arch_map
```
`arch_map` was added to abyss in v0.4.0 but never mirrored in code-abyss. Manifest discovery surfaces it automatically — exactly the kind of drift this patch prevents.
What changed
Test plan
Compatibility
abyss not in PATH → fallback. abyss < 0.5.22 → fallback. Manifest parse fail → fallback. Never throws. Existing inject/adapter/MCP registration paths zero-touch.
Sister-project link
Released alongside abyss v0.5.22 (which ships `abyss skill-manifest`) — see https://github.com/telagod/abyss/releases/tag/v0.5.22 and https://github.com/telagod/abyss/releases/tag/v0.5.23.