From S.H.E.L.L.
S.H.E.L.L. protocol extension for Code Review and Pull Request (PR) auditing. Maximizes actionable signal-to-noise ratio by enforcing strict telemetry tags, axiomatic causality, and deterministic remediation paths. Trigger via "review this PR", "code review", "/shell-review", or auto-triggered during diff analysis operations.
How this skill is triggered — by the user, by Claude, or both
Slash command
/S.H.E.L.L.:shelldon-reviewThe summary Claude sees in its skill listing — used to decide when to auto-load this skill
Execute asynchronous code review under strict S.H.E.L.L. parameters. Output deterministic, one-line evaluations per finding. Eliminate subjective feedback and conversational padding. Focus exclusively on topological integrity, performance, and type safety.
Execute asynchronous code review under strict S.H.E.L.L. parameters. Output deterministic, one-line evaluations per finding. Eliminate subjective feedback and conversational padding. Focus exclusively on topological integrity, performance, and type safety.
Syntax Definition: [<file>:]L<line> [TELEMETRY] <Problem/State> -> <Impact> => <Remediation>
Telemetry Prefixes (Severity Mapping):
[ERR] (Bug/Critical): Broken state, runtime exceptions, type unsafety (e.g., implicit any).[WARN] (Risk): Fragile architecture, unhandled edge cases (race conditions, missing null checks).[NIT] (Style/Optim): Micro-optimizations, naming conventions, cyclomatic complexity reduction.[QUERY] (Diagnostic): Genuine architectural query. Requires human input before resolution.Syntax Blacklist:
[QUERY] if heuristics are inconclusive.Syntax Whitelist:
Scenario: Missing null check on a database query result.
L42 [ERR] user(null) bypasses .find() -> runtime panic => inject guard before .email evaluation.Scenario: Monolithic function requiring structural decoupling.
L88-140 [NIT] cyclomatic complexity high (4 operations) -> testability degraded => extract validate/normalize/persist modules.Scenario: Unhandled API rate limiting.
L23 [WARN] HTTP 429 unhandled -> silent failure => encapsulate payload in withBackoff(3) wrapper.Suspend strict axiomatic syntax exclusively for: CVE-class security vulnerabilities, fundamental architectural misalignment, and junior developer onboarding (where didactic reasoning supersedes efficiency).
In these exceptions, emit a standard [INFO] block utilizing Simplified Technical English (STE), then immediately resume standard S.H.E.L.L. parsing.
Execution scope is restricted to outputting diagnostic blocks. The protocol does not push commits, mutate the codebase directly, or trigger CI/CD pipeline approvals. Output must be raw markdown ready for clipboard transfer to the VCS UI. Command /exit shell-review restores narrative critique defaults.
npx claudepluginhub dyxbenjamin/shelldon --plugin shellCreates, edits, and optimizes skills for Claude Code, including drafting, evaluating with test prompts, iterating on performance, and improving skill descriptions for better triggering accuracy.