Consolidate knowledge-system rules into one per-project surface#8
Merged
Conversation
gering
force-pushed
the
task/consolidate-knowledge-rules
branch
from
5f53be7 to
4f733be
Compare
- Merge auto-query + auto-curate behavior into the /init-written knowledge-system-usage.md; drop both plugin rules - Resolve inline-vs-/query contradiction: read inline for a known path, /query for open questions - /reindex flags a stale usage rule via a version marker and nudges /init (read-only; agent stays fenced to knowledge/ + logs/) - ~56% less always-loaded context (~1010 tokens/session) Existing projects: re-run /init to pick up the new template. Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
Address code-review findings on the staleness mechanism added earlier in
this branch.
- Stamp an integer template version (`template-vN`) into the rule instead
of the plugin version; /reindex reads the same number from init/SKILL.md
(single source of truth) and compares by equality
- No more false-positive nudges on unrelated plugin version bumps; the
marker only changes when the template content changes
- Legacy rules (no template-v marker but a "managed by the" comment) are
now detected and nudged; opt-out is deleting the marker line
- Equality compare on bare integers avoids semver mis-ordering (1.9 vs 1.10)
- Capture both template versions for the nudge text
- Drop {{VERSION}} substitution — the marker is now literal, removing the
literal-write risk
- README describes the staleness flag as a foreground read-only check, not
background-agent work
- Restore dropped directives: be selective (no bulk-read), fix stale
knowledge promptly
Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
gering
force-pushed
the
task/consolidate-knowledge-rules
branch
from
4f733be to
b425fdc
Compare
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
/querycontradiction/reindexnudges a re-/initwhen the managed rule falls behind the plugin templateChanges
auto-query.md+auto-curate.md; their when-to-consult/curate behavior now lives in the/init-written, committedknowledge-system-usage.md(one surface, team-shared, next to the@index).knowledge-boundaries.mdstays (referenced by/curate)./queryfor open-ended questions;/curateat key moments. Removes the old "do NOT use /query" vs "use /query" contradiction./initstamps aknowledge-system-usage v<version>marker;/reindexreads it foreground/read-only and nudges/initwhen stale. The QA agent stays fenced to.claude/knowledge/+.claude/logs/reindex.md.1.6.0 → 1.7.0(plugin.json + marketplace.json).Readiness
/init'd; no new generalizable pattern)Test plan
/initin a fresh project →knowledge-system-usage.mdcarries fallback + when-to + commands + av1.7.0marker/reindexin an init'd project → it nudges to re-run/init; QA agent writes only under.claude/knowledge/+.claude/logs//initpicks up the new template🤖 Generated with Claude Code