JunHo Kim
25256ec417
Extracted and cleaned up from a prior WSL/Windows agent-memory setup. Contains only project-agnostic content: - Claude_Memory/: feedback rules, user profile, reusable reference docs, session scratch space (claude_info/, claude_log/), mirrored skill defs - caveman/: ultra-compressed communication mode skill + subcommands - html-slide-deck/: self-contained HTML slide deck builder skill Project-specific knowledge (previously mixed in) was moved out to each project's own repository as part of this cleanup.
34 lines
1.2 KiB
Markdown
Executable File
34 lines
1.2 KiB
Markdown
Executable File
# caveman-review
|
|
|
|
One-line PR comments. Location, problem, fix. No throat-clearing.
|
|
|
|
## What it does
|
|
|
|
Generates code review comments in `L<line>: <severity> <problem>. <fix>.` format. One line per finding. Severity emoji: 🔴 bug, 🟡 risk, 🔵 nit, ❓ question. Drops "I noticed that...", hedging, and restating what the diff already shows. Keeps exact line numbers, backticked symbols, and concrete fixes.
|
|
|
|
Auto-clarity: drops terse mode for CVE-class security findings, architectural disagreements, and onboarding contexts where the author needs the *why*. Resumes terse for the rest.
|
|
|
|
Output only — does not approve, request changes, or run linters.
|
|
|
|
## How to invoke
|
|
|
|
```
|
|
/caveman-review
|
|
```
|
|
|
|
Also triggers on "review this PR", "code review", "review the diff".
|
|
|
|
## Example output
|
|
|
|
```
|
|
L42: 🔴 bug: user can be null after .find(). Add guard before .email.
|
|
L88-140: 🔵 nit: 50-line fn does 4 things. Extract validate/normalize/persist.
|
|
L23: 🟡 risk: no retry on 429. Wrap in withBackoff(3).
|
|
L107: ❓ q: why drop the cache here? Reads on next request will miss.
|
|
```
|
|
|
|
## See also
|
|
|
|
- [`SKILL.md`](./SKILL.md) — full LLM-facing instructions
|
|
- [Caveman README](../../README.md) — repo overview
|