anti-ai-slop

Intent

This skill reviews the diff against main and removes changes that are likely “AI slop”: unnecessary, non-idiomatic, or inconsistent additions that do not materially contribute to the intended feature/fix.

Guiding principle: keep the smallest change that accomplishes the goal, and match the local conventions of each touched file.

Scope and diff source

Operate only on changes introduced on this branch relative to main.

Determine the diff source in this order:

1. git diff --cached (if non-empty)
2. git diff (if non-empty)
3. git diff main..HEAD (fallback)
   If main does not exist locally, use git diff master..HEAD.

Do not refactor unrelated code outside the displayed diff hunks.

Non-goals

• Do not redesign architecture, rename APIs, or move code unless required by the feature/fix in the diff.
• Do not remove meaningful domain comments that encode invariants, reasoning, constraints, or known pitfalls.
• Do not remove validation at genuine trust boundaries (I/O, parsing, network, user input, third-party callbacks).
• Do not change formatting rules; follow repository tooling and the file’s established style.

What counts as AI slop

A) Comment and documentation bloat

Remove comments/docstrings that merely narrate the code, restate names, or are inconsistent with the file’s comment density and tone.

Python (before):

def clamp(x: float, lo: float, hi: float) -> float:
    # Ensure x is not less than lo and not greater than hi
    if x < lo:
        return lo
    if x > hi:
        return hi
    return x

Python (after):

def clamp(x: float, lo: float, hi: float) -> float:
    if x < lo:
        return lo
    if x > hi:
        return hi
    return x

B) Unnecessary defensive checks inside trusted codepaths

Remove checks that are redundant because upstream validation/types/invariants already guarantee inputs.

Rust (before):

pub fn normalize_tag(tag: &str) -> String {
    if tag.is_empty() {
        return String::new();
    }
    tag.trim().to_ascii_lowercase()
}