-
Notifications
You must be signed in to change notification settings - Fork 12
fix(cortex): align KNOWLEDGE_CATEGORIES with current schema.md #128
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Changes from all commits
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
| Original file line number | Diff line number | Diff line change |
|---|---|---|
|
|
@@ -55,10 +55,11 @@ SENSITIVE_EXTENSIONS = {".pem", ".key", ".p12", ".pfx", ".jks", ".keystore", ".k | |
| SENSITIVE_PREFIXES = ("secret", "credentials", ".env", "id_rsa", "id_ed25519", "id_ecdsa") | ||
|
|
||
| KNOWLEDGE_CATEGORIES = [ | ||
| "entities", "concepts", "summaries", "synthesis", "decisions", "how-to", | ||
| "people", "ventures", "topics", "synthesis", "decisions", | ||
| "learning", "research", | ||
| ] | ||
|
Comment on lines
57
to
60
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more.
Updating Useful? React with 👍 / 👎.
Owner
Author
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Fixed — schema-template.md and the deployed schema now use the same category list. All references to entities/concepts/summaries/how-to have been replaced with the new names. |
||
|
|
||
| EXTRA_DIRECTORIES = ["learning", "learning/archive", "daily"] | ||
| EXTRA_DIRECTORIES = ["learning/archive", "daily"] | ||
|
|
||
| CONFIG_DIR = Path.home() / ".config" / "cortex" | ||
| CONFIG_FILE = CONFIG_DIR / "config" | ||
|
|
@@ -464,7 +465,7 @@ def _setup_symlink(store_path: Path) -> bool: | |
| "# Memory Index\n\n" | ||
| "## Quick Links\n" | ||
| "- [Knowledge Base Index](Knowledge Base/index.md)\n" | ||
| "- [All Entities](Knowledge Base/entities/index.md)\n" | ||
| "- [All People](Knowledge Base/people/index.md)\n" | ||
|
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Learning ops files countedLow Severity Adding Additional Locations (1)Reviewed by Cursor Bugbot for commit 5887505. Configure here.
Owner
Author
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. By design — learning/ is now a first-class category in KNOWLEDGE_CATEGORIES, so it's created by setup and counted by status. This is intentional (it was previously an 'extra' directory). The EXTRA_DIRECTORIES list was reduced accordingly. |
||
| "- [Recent Decisions](Knowledge Base/decisions/index.md)\n" | ||
| ) | ||
| print("Created MEMORY.md routing table") | ||
|
|
||


Uh oh!
There was an error while loading. Please reload this page.