Repository navigation
docs(onboarding): refresh the graph from hooks instead of a watcher - #541
Conversation
…175) Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01FuX9kDSsfALRNYaDsKN4K6
![gemini-code-assist[bot]](https://avatars.githubusercontent.com/in/956858?s=60&v=4){kind=small}
There was a problem hiding this comment.
Code Review
This pull request adds documentation on how to configure hooks (for Claude Code and git) to refresh the graph database outside of the MCP server, avoiding the need for a file watcher. The review feedback points out that the ingestion commands in both the Claude Code and git hook examples should target ./src instead of . to remain consistent with Step 1 and prevent unnecessary full rebuilds. Additionally, the Claude Code JSON configuration contains redundant nesting that should be simplified.
Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01FuX9kDSsfALRNYaDsKN4K6
|
/gemini review |
There was a problem hiding this comment.
Code Review
This pull request adds documentation on how to configure external hooks (such as Claude Code and git hooks) to trigger incremental ingestion outside of the MCP server. Feedback was provided regarding the Claude Code hook configuration in .claude/settings.json, which contains an extra level of nesting that does not match the expected schema, along with a suggested correction.
|
2 participants
Requested by Andrei · project thread
Before: #175 still asked for a watch mode, and the docs didn't say how to keep the graph current outside the MCP server.
After:
docs/how-to/AGENT_ONBOARDING.mdexplains why cgis has no watcher and gives two hook recipes that runcgis ingest -ionce a burst of edits is over. One is a Claude CodeStophook (once per finished turn, not once per edit). The other is a set of gitpost-checkout/post-merge/post-rewritehooks.This is the last step of #175: incremental ingest, the freshness signal (#443), the ingest race fix (#535) and opt-in lazy refresh (#538) have shipped. Watch mode is replaced by these recipes. The remaining ideas have their own issues: #539 (cache raw edges so a cross-file rebuild skips re-extraction) and #540 (decide on turning auto-refresh on by default).
How. Docs only. I checked that the hook JSON parses and that the command is extracted correctly.
Closes #175
🤖 Generated with Claude Code
https://claude.ai/code/session_01FuX9kDSsfALRNYaDsKN4K6
Generated by Claude Code