Uh oh!
There was an error while loading. Please reload this page.
docs(prompts): tell Prime Claude never to edit KBFuncs.nlp - #209
Merged
ddehilster merged 1 commit intoAug 24, 2026
Merged
Conversation
KBFuncs.nlp is VisualText's shared knowledge base function library — it ships in the engine's spec/ library and in seven analyzer templates, and SaveKB, JsonKB and AddUniqueCon all live there. An analyzer is not the place to change it: the edit is invisible to anyone reading the passes, drifts from the copy everyone else has, and is lost when the analyzer is recreated from the template. The primer already said to keep the scaffolding; it did not say the library file is off limits or where new functions should go. It now says both — add your own @decl pass and a `nlp MyFuncs` line to analyzer.seq after KBFuncs — and extends the rule to the other shared library passes in the VisualText spec/ directory. Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>
Uh oh!
There was an error while loading. Please reload this page.
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 freeto 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.
spec/KBFuncs.nlpis VisualText's shared knowledge base function library — it ships in the engine'sspec/library and in seven of the analyzer templates (Address Parser, Email Addresses, Knowledge Base, NLPPlus Interface, parse-en-us, Telephone Numbers, URLs), andSaveKB,JsonKB,AddUniqueConand the rest all live there.An analyzer is not the place to change it. An edit there is invisible to anyone reading your passes, drifts from the copy everyone else has, and is lost the moment the analyzer is recreated from the template.
The primer already told Claude to keep the Knowledge Base template's scaffolding, but never said the library file itself is off limits, and never said where new functions should go instead. It now says both: put your own functions in your own
@DECLpass and add anlp MyFuncsline toanalyzer.seqafter KBFuncs, calling the library's functions from there. The rule extends to the other shared library passes in the VisualTextspec/directory.Placed as its own bullet directly after the "keep its scaffolding" bullet in the Knowledge Base template section.
Prompt text only — no engine, analyzer, or library files touched.
🤖 Generated with Claude Code