docs(skills): split graphstore-ingestion into -dsl + -builder#148
Merged
KailasMahavarkar merged 1 commit intomainfrom Apr 19, 2026
Merged
docs(skills): split graphstore-ingestion into -dsl + -builder#148KailasMahavarkar merged 1 commit intomainfrom
KailasMahavarkar merged 1 commit intomainfrom
Conversation
…lder (python) One-skill-fits-all was 777 lines and mixed two audiences. Split by the actual consumer: - tools/skills/graphstore-dsl/SKILL.md (425 lines, 19 KB, ~5k tokens) For runtime LLM emission of DSL text. Grammar reference, every verb with clause order, WHERE / MATCH / BATCH syntax, string escape rules, value types, gotchas. No Python API. Use when an LLM is ingesting, distilling, or querying at runtime and the output is text that gets parsed + executed line by line. - tools/skills/graphstore-builder/SKILL.md (674 lines, 27 KB, ~7k tokens) For Python adapter authors importing graphstore. Full q/F/P/agg/Time/ EvolveWhen/EvolveThen API with signatures + emission, composition primitives (.pipe, |, .with_), plugin registration, 6 patterns (A-F) rewritten end-to-end in typed builder calls. No DSL grammar dump. Both reference each other for completeness. Descriptions spell out which to load per use case. Content verified against codebase: - Every q.* signature matches src/graphstore/query/verbs/*.py - Config defaults match src/graphstore/config.py (remember_weights [0.52, 0.25, 0.15, 0.08], recency_half_life_days 7300.0, graph_signal_enabled true, fusion_method weighted) - P / F / agg / Time / EvolveWhen / EvolveThen APIs compile-checked via .venv/bin/python dry runs benchmarks/framework/adapters/graphstore_.py docstring now points at graphstore-builder (adapter uses the Python API, not runtime DSL). Em dash sweep: zero in either new file (Rule 9). Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
KailasMahavarkar
added a commit
that referenced
this pull request
Apr 20, 2026
v0.4 ships retrieval observability triangle: - REMEMBER signal telemetry + rich meta["signals"] (#150) - SYS EXPLAIN REMEMBER dry-run (#151) - ANSWER verb with pluggable reader LLM (#152) Plus: - Skills split: graphstore-dsl (runtime) + graphstore-builder (Python) (#148) - Skill-guided LLM ingest adapter + LoCoMo wiring fix (#149) - Docusaurus docs site @ graphstore-docs.orkait.com (#142-147) Breaking changes: none. All additions are additive. Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
Merged
KailasMahavarkar
added a commit
that referenced
this pull request
Apr 20, 2026
* chore(release): bump v0.3.0 -> v0.4.0 v0.4 ships retrieval observability triangle: - REMEMBER signal telemetry + rich meta["signals"] (#150) - SYS EXPLAIN REMEMBER dry-run (#151) - ANSWER verb with pluggable reader LLM (#152) Plus: - Skills split: graphstore-dsl (runtime) + graphstore-builder (Python) (#148) - Skill-guided LLM ingest adapter + LoCoMo wiring fix (#149) - Docusaurus docs site @ graphstore-docs.orkait.com (#142-147) Breaking changes: none. All additions are additive. Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com> * chore(release): bump pyproject.toml version to 0.4.0 Missed in 07c9986. Pairs with src/graphstore/__init__.py bump. --------- Co-authored-by: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
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.
Previous `graphstore-ingestion` skill was 777 lines and mixed two very different audiences: LLMs emitting DSL text at runtime, and Python authors writing adapters with the typed builder. Each LLM session pays the full token cost of both.
Split
Each drops content the other audience doesn't need. Combined they still cost less per-session than the old mega-skill, because only one loads per use case.
graphstore-dsl covers
graphstore-builder covers
Verified against codebase
Other changes
Em dash sweep
Zero em dashes in either new file (Rule 9).
Removed
`tools/skills/graphstore-ingestion/` - old skill removed.