Skip to content

feat(skills): add omit-needless-words review skill #5496

Closed

Description

@purp

User Story

I review OpenShell PRs as a maintainer. I need an agent review that flags prose and code a change doesn't need, so contributors trim it before merge and docs don't drift.

Description

Add a contributor skill, omit-needless-words, to .agents/skills/. It reviews a PR, branch, diff, doc, skill, or comment block against five rules:

  1. Crisp prose in docs, skills, and comments.
  2. Present tense: no history or roadmap.
  3. Words only where code or existing text doesn't already say it.
  4. DRY: each fact lives in one place; everything else links to it.
  5. As much as needed, as little as possible: code and tests answer to the spec.

The output is a one-sentence verdict followed by numbered path:line findings, each with a concrete fix.

Context

A review of #5407 with this skill cut its README roughly in half and removed duplicated defaults and validation code.

Definition of Done

  • .agents/skills/omit-needless-words/SKILL.md exists with metadata.internal: true.
  • The contributor skill table in CONTRIBUTING.md lists it.

Activity

  1. added a commit that references this issue on Nov 12, 2026
    7a9b19a
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Metadata

Assignees

No one assigned

    Labels

    No labels
    No labels

    Type

    No type

    Projects

    No projects

      Milestone

      No milestone

      Relationships

      None yet

      Development

      No branches or pull requests

      Issue actions