Repository navigation
chore(claude): disable redundant plugins and trim skills that restate model knowledge - #354
Merged
Merged
Conversation
… model knowledge Plugins: keep explanatory-output-style (the ★ Insight hook depends on it) and the four LSP plugins. Disable the rest: - superpowers, feature-dev, commit-commands, pr-review-toolkit, claude-md-management: their usage came almost entirely from commands prescribing them, which the previous PR replaced with built-ins (Explore, code-review, simplify) or git/gh steps. superpowers also injected a long "YOU MUST" preamble into every session that pushed process skills (brainstorming, plan-writing, mandated verification) the global CLAUDE.md now steers away from. - ralph-loop: superseded by the built-in /goal and /loop. - skill-creator (also available as anthropic-skills:skill-creator), claude-code-setup, agent-sdk-dev, frontend-design: no invocations in 263 session transcripts. Skills: - Remove rust-async-patterns: 524 lines of general Tokio knowledge with nothing project-specific. - shell-scripting: the `local result="$(command)"` example is what shellcheck SC2155 flags, contradicting the skill's own "must pass shellcheck" rule; show the split form bootstrap.sh already uses. - summarize: keep the CLI usage (the part the model can't know), drop the tool-access table, the fixed template, and the ~200-word cap, and narrow the description from any "what is this" to links and media. Co-Authored-By: Claude Opus 5.5 (1M context) <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01LBftoc1JCMLow9LDFZMADs
Co-Authored-By: Claude Opus 5.5 (1M context) <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01LBftoc1JCMLow9LDFZMADs
There was a problem hiding this comment.
Code Review — Round 1
Summary
Turns off 10 plugins in settings.json (keeps explanatory-output-style and the four LSP plugins), deletes the general-purpose rust-async-patterns skill, fixes the SC2155 example in shell-scripting, and trims summarize down to the CLI usage. settings.json stays valid JSON after the trailing-comma change.
Checked:
- Leftover references: no command, agent, hook or skill in the repo still points at a disabled plugin or at
rust-async-patterns.docs/superpowers/plans/…still mentionssuperpowers:sub-skills. It is a historical plan doc that this PR does not touch.- The
skill-creatormention inCLAUDE.mdstill works throughanthropic-skills:skill-creator.
- Broken links after the skill deletion:
bootstrap.shlinks the wholeskills/directory (symlink_claude_dir "skills"), not each skill. So deleting the skill folder leaves no broken symlink in~/.claude/skills/. - Docs table:
rfc-*agents no longer exist, so theaudit-*, summarize-workchange indocs/work-review-cycle.mdis correct. shell-scripting: the new examplelocal result; result="$(command)"is the form shellcheck accepts. It removes the conflict with the skill's own "must pass shellcheck" rule.summarize: the narrower description (it no longer fires on "what is this" or "what happened in this PR") is a deliberate choice explained in the PR body. The "Without the CLI" section still covers summarizing PRs and code by hand.
Verdict
APPROVE - No blocking findings. This is a config and docs cleanup with no behavior regressions.
Automated review by Claude Code
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
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.
This is the third PR updating the setup for the Opus 5.5 generation. It covers plugins and skills. It follows #353, which removed every command and agent dependency on the plugins disabled here.
Plugins
Usage was measured by searching the 263 local session transcripts for skill and subagent invocations from each plugin.
commit-commands/pr-createcalling it; now explicitgit/ghstepspr-review-toolkit/pr-review's fallback; the built-incode-reviewandsimplifycover itfeature-dev/work's explorer and architect phases, now done directlysuperpowersralph-loop/goaland/loopclaude-md-managementaudit-docsagentskill-creatoranthropic-skills:skill-creatorclaude-code-setup,agent-sdk-dev,frontend-designKept:
explanatory-output-style(the ★ Insight hook depends on it) and the four LSP plugins (rust-analyzer,gopls,pyright,typescript).Disabling a plugin removes its skill and agent descriptions from every request and from the session-start context. To bring one back, add its
enabledPluginsentry again.Skills
rust-async-patterns: 524 lines of general Tokio and async-Rust material with nothing specific to these projects. The model already knows all of it.shell-scripting: its examplelocal result="$(command)"is exactly what shellcheck SC2155 flags, which contradicts the skill's own "all.shfiles must pass shellcheck" rule. It now shows the split form thatbootstrap.shalready uses.summarize: keeps thesummarizeCLI usage, the part the model can't know. Drops the table of how to access each input type, the fixed output template, and the "~200 words" cap. Its description no longer fires on any "what is this"; it now targets links and media.Docs: updated the shared-components table in
docs/work-review-cycle.md, deferred here from #353's review.make checkpasses, andsettings.jsonis valid JSON. The diff only touchesenabledPlugins.🤖 Generated with Claude Code
https://claude.ai/code/session_01LBftoc1JCMLow9LDFZMADs