fix(store): escape forward slashes in FTS5 sanitize and normalize topic_key queries#210
Open
jorgehara wants to merge 4 commits intoGentleman-Programming:mainfrom
Open
Conversation
- Add internal/claudecode package with export/import/sync functionality - Export Engram observations to Claude Code's native memory folder as .md files - Import memories from Claude Code's memory folder into Engram - Add CLI commands: claude-code-export, claude-code-import, claude-code-sync - Support bidirectional sync between Engram and Claude Code memory - Allow users who switch between Claude Code native and other agents to have unified memory
- Add lazy bidirectional sync: when mem_search is called, automatically import new memories from Claude Code's native memory folder - Consistent with Engram philosophy: agent-driven, no auto-capture, import only on explicit user request (search) - Add comprehensive tests for Claude Code memory import - Add documentation explaining the feature and design decisions for Alan This enables users who switch between Claude Code native and other agents to have unified memory without manual sync steps.
Add 'engram agents' command that detects and shows usage statistics for all AI coding agents on the system (Claude Code, Gemini CLI, OpenCode, Cursor, Codex). Features: - Reads native history files from each agent - Shows sessions, projects, and activity range per agent - ASCII bar chart visualization for easy comparison - JSON output for scripting (--json flag) - Read-only, no data written to Engram store Consistent with Engram philosophy: simple, agent-driven, no auto-capture.
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
1 participant
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
/to\/insanitizeFTSto prevent FTS5 "malformed query" errors when searching topic keys containing slashes (e.g.,sdd/summa-kit/spec)normalizeTopicKey()before the topic_key direct lookup path so human-readable slash-separated queries match stored hyphen-separated topic keysChanges
internal/store/store.go/in sanitizeFTS + normalize topic_key queriesTest Plan
go test ./internal/store/...)go test ./internal/store/... -run Search)Checklist
Co-Authored-Bytrailers