Docs/claude code integration#1386
Open
AnuGuin wants to merge 2 commits into
Open
Conversation
- tests/fixtures/cursor_session.jsonl: 9 anonymised lines covering all event types (user prompt w/ XML tags, assistant text+tokens, thinking block, tool use, tool result, closing response) - tests/test_cursor_parser.py: 16 tests covering XML stripping, token attrs, tool result merging, orphan skipping, edge cases, and registry sanity checks
Created docs/integrations/claude-code.md covering supported features, installation, configuration paths, hooks specification, session push behavior, and caveats.
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.
Closes #1358
Purpose / Description
Adds the missing integration documentation page for Claude Code
(
docs/integrations/claude-code.md). Claude Code is a first-class IDEin Observal with full hook support, session parsing, and e2e tests, but
had no setup or reference documentation page for users to find.
Fixes
Fixes #1358
Approach
Created
docs/integrations/claude-code.mdsourced directly from theimplementation files rather than inferred — every path, event name, and
behavior is verified against:
observal_cli/ide/claude_code.py— adapter scan logic, hook detection thresholdobserval_cli/ide_specs/claude_code_hooks_spec.py— hook spec version, exact hook JSONobserval_cli/sessions/claude_code.py— JSONL discovery, cursor logic, subagent pushobserval-server/services/ide/claude_code.py— agent config generation, frontmatter formatobserval-server/schemas/ide_registry.py— config paths, scope labels, event mapThe page covers all sections required by the issue:
observal doctor --patch,observal pull)settings.json, full event map)How Has This Been Tested?
Documentation-only change. Verified no broken links and that all file
paths, event names, and version constants match the current codebase.
Ran the registry and adapter test suites to confirm no regressions:
pytest tests/test_cli_ide_adapters.py -q → 59 passed in 0.73s
pytest tests/test_ide_registry.py -q → 56 passed in 0.23s
pytest tests/test_constants_sync.py -q → 16 passed in 0.21s
Checklist
AI Assistance
Was generative AI tooling used to co-author this PR?