Repository navigation
chore(dedupe): remove knowledge-manager/search reimplementations, thin command entrypoints — 1.4.0 - #24
Merged
Conversation
Knowledge management and vault search move to the km plugin (treylom/tofukyung-plugins). Removed 7 skill directories, 10 .agents specs and commands/search.md, plus the wrapper meta .agents/cmd-thiscode-setup.yaml (A2 — /thiscode:setup stays declared by bootstrap.yaml). References across README (en/ko), docs, hermes-plugin, scripts and tests updated to point at the km plugin; agents.yaml regenerated (17 -> 6). Fixed the router yaml test to point at .agents/model-router.yaml (A4 — model-router itself is kept). Co-Authored-By: Claude Fable 5.1 <noreply@anthropic.com>
Feature removal (km/search skills move to the km plugin) is a minor bump. Both .claude-plugin/plugin.json and .claude-plugin/marketplace.json go 1.3.2 -> 1.4.0. Co-Authored-By: Claude Fable 5.1 <noreply@anthropic.com>
The 8-Phase table and the install dispatch list still sent Phase 5 to a km-at skill that A1 removed from this repo. Both lines now point at the km plugin's Mode R preflight, with a note to fall back to the README install pointer when the plugin is absent. Co-Authored-By: Claude Fable 5.1 <noreply@anthropic.com>
… knowledge/search (B1/B2/B5) Co-Authored-By: Claude Fable 5.1 <noreply@anthropic.com>
…runtime with manifest (+ tests) (B3/B4) Co-Authored-By: Claude Fable 5.1 <noreply@anthropic.com>
Co-Authored-By: Claude Fable 5.1 <noreply@anthropic.com>
Co-Authored-By: Claude Fable 5.1 <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
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.
What
.agents/*.yamlspecs andcommands/search.md(25 tracked files). Knowledge management and vault search are provided by the knowledge-manager plugin (treylom/tofukyung-plugins)..agents/cmd-thiscode-setup.yamlas well, so all four slash-command wrapper specs are gone./thiscode:setupstays declared by.agents/bootstrap.yaml..agents/model-router.yaml; fixedtests/agents/test-router-yaml.sh, which pointed at a file that has not existed since the 2026-05-16 rename (.agents/thiscode-model-router.yaml). That test was red onmainand is green here.commands/init.md/commands/slack-configure.mdturns two existing tests red (tests/init/test-commands-init.sh,tests/init/bot-launcher.test.mjs), so the change was reverted per the order's conditional. See the report for details.1.3.2 → 1.4.0in.claude-plugin/plugin.jsonand.claude-plugin/marketplace.json(feature removal = minor).Reference cleanup
Every place that named a removed object was updated: README (en/ko) incl. a new install pointer under the role-boundary paragraph,
AGENTS.md,GEMINI.md,docs/*,hermes-plugin/*,package.json,scripts/lib/codexsync.mjs,scripts/sync-skills-to-codex.sh,skills/help,skills/slack-configure,skills/thiscode_agy, and the test suite.agents.yamlwas regenerated (17 → 6 agents), which also clears the deadcmd-searchentry. Eight test files that only ever exercised the removed skills were deleted; the.agentscount assertion was updated to the measured value.The one remaining mention lives in
docs/RECENT-CHANGES.md, a dated change log — kept on purpose as a historical record.Verification
npm testbash scripts/install-browser-gate.sh --self-testbash scripts/validate-agents.shnode scripts/check-agents-index.mjstests/agents/*(6 scripts)Draft — not ready for review yet.