Skip to content

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

Merged
purp merged 1 commit into
NVIDIA:mainfrom
purp:feat/5496-omit-needless-words/purp
Nov 12, 2026
Merged

purp merged 1 commit into
NVIDIA:mainfrom
purp:feat/5496-omit-needless-words/purp

Conversation

@purp

@purp purp commented Nov 11, 2026

Copy link
Copy Markdown
Collaborator

Summary

Add a contributor skill that reviews a change for wordy, historical, duplicated, or out-of-spec prose and code, and returns a verdict with concrete fixes.

Related Issue

Closes #5496

Changes

  • Add .agents/skills/omit-needless-words/SKILL.md (internal).
  • List it in the contributor skill table in CONTRIBUTING.md.

Testing

  • Checks appropriate to the affected code and behavior pass (pre-commit hook; sync-agent-infra check 1: internal metadata, unique name, table row)
  • Unit tests added/updated (if applicable)
  • E2E tests added/updated (if applicable)

Checklist

  • Follows Conventional Commits
  • Commits are signed off (DCO)

Signed-off-by: Jim Meyer <jimeyer@nvidia.com>
@purp
purp enabled auto-merge November 11, 2026 21:45
@purp purp self-assigned this Nov 11, 2026
@purp purp added the area:build CI/CD workflows, releases, build targets, packaging, and artifact publication. label Nov 11, 2026
@purp
purp added this pull request to the merge queue Nov 12, 2026
Merged via the queue into NVIDIA:main with commit 7a9b19a Nov 12, 2026
94 of 95 checks passed
@purp
purp deleted the feat/5496-omit-needless-words/purp branch November 12, 2026 07:28
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area:build CI/CD workflows, releases, build targets, packaging, and artifact publication.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

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

2 participants