Conversation
…s/commands, normalize naming - Delete 73 files: dead docs (archive/, enterprise/, reports/), deprecated ontologies, duplicate images, one-shot migration scripts, over-engineered agentic workflows - Consolidate skills 14→10: merge search-enhanced/progressive-disclosure into search, agent-coordination into blackboard, delete organization (all duplicated content) - Consolidate commands 15→11: delete init (subset of setup), discover (ontology handles), search-enhanced (redundant), team-audit (custodian covers) - Rewrite skills: search 494→167 lines, format 745→197 lines, blackboard absorbs coordination - Strip lib/__init__.py from 139-line kitchen-sink re-exports to 1-line docstring - Normalize skill naming: drop redundant mnemonic- prefix (core, search, format, setup, blackboard) - Delete 8 GitHub workflow files (agentic lock.yml + .md pairs) - Fix all cross-references in docs, ADRs, tools, README - 397/397 tests passing throughout
There was a problem hiding this comment.
Pull request overview
This PR aggressively reduces repository size by removing deprecated docs/scripts and consolidating skills/commands, including renaming skills to a consistent scheme (dropping the mnemonic- prefix) and updating documentation references accordingly.
Changes:
- Removed legacy migration/cleanup scripts and large archived documentation/research content.
- Consolidated/renamed skills and commands (e.g.,
search-enhancedmerged intosearch) and updated docs/ADRs to match new paths/names. - Simplified
lib/__init__.pyfrom a large re-export surface to a minimal module docstring.
Reviewed changes
Copilot reviewed 75 out of 104 changed files in this pull request and generated 3 comments.
Show a summary per file
| File | Description |
|---|---|
| scripts/migrate_scope_paths.py | Removed legacy one-off migration script as part of repo de-bloat. |
| scripts/migrate_namespaces.py | Removed legacy namespace migration script. |
| scripts/fix_malformed_memories.py | Removed legacy memory repair script. |
| scripts/cleanup_memory_paths.py | Removed legacy path cleanup script. |
| reports/ai-memory-filesystem-research/state.json | Removed generated research artifact content. |
| reports/ai-memory-filesystem-research/README.md | Removed research index doc for the deleted report. |
| reports/ai-memory-filesystem-research/2026-01-24-report-metadata.json | Removed report metadata artifact. |
| reports/ai-memory-filesystem-research/2026-01-24-executive-summary.md | Removed executive summary artifact. |
| reports/README.md | Removed reports index to match removal of reports/ content. |
| lib/init.py | Simplified package init to avoid kitchen-sink re-exports. |
| lib/README.md | Removed large internal library README (doc consolidation). |
| docs/validation.md | Updated skill path reference to skills/format/SKILL.md. |
| docs/tutorials/README.md | Removed tutorials index README (docs consolidation). |
| docs/semantic-search.md | Updated references from search-enhanced to consolidated search skill. |
| docs/ontologies.md | Removed docs mention of deleted /mnemonic:discover command. |
| docs/library-reference.md | Removed link to deleted lib/README.md. |
| docs/enterprise/research-validation.md | Removed enterprise docs subtree content (de-bloat). |
| docs/enterprise/productivity-roi.md | Removed enterprise docs subtree content (de-bloat). |
| docs/enterprise/developer-experience.md | Removed enterprise docs subtree content (de-bloat). |
| docs/enterprise/compliance-governance.md | Removed enterprise docs subtree content (de-bloat). |
| docs/enterprise/README.md | Removed enterprise docs landing page. |
| docs/archive/templates/plugin-hooks/mnemonic-suggest.py | Removed archived template content. |
| docs/archive/templates/plugin-hooks/hooks.json.template | Removed archived template content. |
| docs/archive/templates/plugin-hooks/README.md | Removed archived template content. |
| docs/archive/templates/mnemonic-protocol.md | Removed archived template content. |
| docs/archive/templates/cursor-rule.mdc | Removed archived template content. |
| docs/archive/templates/copilot-instructions.md | Removed archived template content. |
| docs/archive/templates/codex-skill/SKILL.md | Removed archived template content. |
| docs/archive/templates/CONVENTIONS.md | Removed archived template content. |
| docs/archive/templates/AGENTS.md | Removed archived template content. |
| docs/archive/integrations/windsurf.md | Removed archived integration doc. |
| docs/archive/integrations/opencode.md | Removed archived integration doc. |
| docs/archive/integrations/github-copilot.md | Removed archived integration doc. |
| docs/archive/integrations/gemini-cli.md | Removed archived integration doc. |
| docs/archive/integrations/cursor.md | Removed archived integration doc. |
| docs/archive/integrations/continue.md | Removed archived integration doc. |
| docs/archive/integrations/codex-cli.md | Removed archived integration doc. |
| docs/archive/integrations/aider.md | Removed archived integration doc. |
| docs/archive/integrations/README.md | Removed archived integrations index. |
| docs/archive/community/quickstart-memory-bank.md | Removed archived community doc. |
| docs/archive/community/mnemonic-vs-memory-bank.md | Removed archived community doc. |
| docs/archive/community/migration-from-memory-bank.md | Removed archived community doc. |
| docs/archive/community/adoption-stories.md | Removed archived community doc. |
| docs/archive/community/README.md | Removed archived community index. |
| docs/archive/community/CONTRIBUTING-COMMUNITY.md | Removed archived community contributing guide. |
| docs/architecture/path-resolution-integration.md | Removed outdated integration guide as part of docs consolidation. |
| docs/architecture.md | Updated docs to match consolidated skills/commands and removed references to deleted docs. |
| docs/api-reference.md | Removed redundant Python API reference doc. |
| docs/agent-coordination.md | Removed coordination doc in favor of blackboard skill reference. |
| docs/adrs/adr-009-unified-path-resolution.md | Removed link to deleted path-resolution integration guide. |
| docs/adrs/adr-008-custom-ontologies.md | Updated skill path references to renamed format skill. |
| docs/adrs/adr-004-mif-schema-validation.md | Updated naming references for format skill (but see suggestion). |
| docs/adrs/adr-002-mif-level-3-format.md | Updated references to renamed core/format skills. |
| docs/adrs/adr-001-filesystem-based-storage.md | Updated references to renamed core/format skills. |
| commands/team-audit.md | Removed command as part of command consolidation. |
| commands/search-enhanced.md | Removed command as search is consolidated into /mnemonic:search. |
| commands/init.md | Removed redundant init command (subset of setup). |
| commands/discover.md | Removed discover command (Ontology now covers). |
| commands/capture.md | Updated schema reference to format skill. |
| README.md | Updated top-level docs/command/skill references to consolidated, renamed set. |
| .github/workflows/update-docs.md | Removed workflow as part of CI/workflow reduction. |
| .github/workflows/plan.md | Removed workflow as part of CI/workflow reduction. |
| .github/workflows/issue-triage.md | Removed workflow as part of CI/workflow reduction. |
| .github/workflows/ci-doctor.md | Removed workflow as part of CI/workflow reduction. |
| .claude-plugin/plugin.json | Simplified plugin manifest (but see suggestion). |
| "./agents/mnemonic-search-subcall.md", | ||
| "./agents/compression-worker.md", | ||
| "./agents/ontology-discovery.md" | ||
| ] |
There was a problem hiding this comment.
The plugin manifest no longer declares any commands, skills, or agents. If the runtime relies on plugin.json for registration (as the removed content suggests), this will prevent the plugin from loading any functionality. Consider restoring explicit commands/skills/agents entries (or, if you intentionally switched to auto-discovery, keep the change but ensure the loader is updated accordingly and documented).
Suggested change
| ] | |
| ], | |
| "commands": [], | |
| "skills": [], | |
| "agents": [] |
| --- | ||
| title: "MIF Schema as Single Source of Truth for Validation" | ||
| description: "Validation tool parses MIF schema from mnemonic-format SKILL.md directly" | ||
| description: "Validation tool parses MIF schema from format SKILL.md directly" |
There was a problem hiding this comment.
These references appear to use an incorrect path (format/SKILL.md) and are inconsistent with other updated docs that point to skills/format/SKILL.md. Update these to the full skills/format/SKILL.md path to avoid broken references and keep ADR wording consistent.
Suggested change
| description: "Validation tool parses MIF schema from format SKILL.md directly" | |
| description: "Validation tool parses MIF schema from skills/format/SKILL.md directly" |
| ### Option 3: Parse Schema from SKILL.md (Selected) | ||
|
|
||
| **Description**: Validation tool parses rules directly from `mnemonic-format/SKILL.md`. | ||
| **Description**: Validation tool parses rules directly from `format/SKILL.md`. |
There was a problem hiding this comment.
These references appear to use an incorrect path (format/SKILL.md) and are inconsistent with other updated docs that point to skills/format/SKILL.md. Update these to the full skills/format/SKILL.md path to avoid broken references and keep ADR wording consistent.
Suggested change
| **Description**: Validation tool parses rules directly from `format/SKILL.md`. | |
| **Description**: Validation tool parses rules directly from `skills/format/SKILL.md`. |
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
mnemonic-prefix)What was removed
docs/archive/(24 files),docs/enterprise/(6 files),reports/(7 files).lock.yml+.mdpairs)api-reference.md,architecture/subdir,tutorials/README.md,lib/README.mdlib/__init__.pystripped from 139-line kitchen-sink re-exports to 1-line docstringWhat was consolidated
search-enhanced+progressive-disclosure→search;agent-coordination→blackboard;organizationdeleted (all duplicated)search494→167 lines,format745→197 linesinit(subset ofsetup),discover(ontology handles),search-enhanced(redundant),team-audit(custodian covers)mnemonic-core→core,mnemonic-search→search, etc.Test plan
tools/mnemonic-validatehardcoded path updated toskills/format/SKILL.md