Skip to content

feat: add audit-project SKILL.md#17

Merged
avifenesh merged 2 commits intomainfrom
feat/add-audit-project-skill
Apr 15, 2026
Merged

feat: add audit-project SKILL.md#17
avifenesh merged 2 commits intomainfrom
feat/add-audit-project-skill

Conversation

@avifenesh
Copy link
Copy Markdown
Contributor

@avifenesh avifenesh commented Apr 15, 2026

Summary

Add `skills/audit-project/SKILL.md` so the file-based skill referenced by agentsys's Code Review category actually exists in this repo.

Why

Workspace audit surfaced that agentsys's Skills table lists `audit-project` under Code Review but the plugin had no SKILL.md - only commands. Rather than drop the listing (which reflects a real invocable workflow), adding the file so the 40-skills ecosystem claim is accurate.

Contents

  • Standard SKILL.md frontmatter with trigger phrases ("review my code", "audit the codebase", "security review", etc.)
  • Documents the 8-phase workflow by delegation to `commands/audit-project.md`
  • Lists all 10 role-based reviewers and severity classification
  • Pure documentation of existing behavior - no code changes

Test plan

  • agnix validation passes (frontmatter schema correct)
  • Trigger phrases cover the user-invocation patterns
  • No new dependencies or workflow

Note

Low Risk
Low risk documentation-only change that adds a new skill descriptor file without modifying runtime code or workflows.

Overview
Adds a new skills/audit-project/SKILL.md file so the audit-project skill exists as a file-based entry.

The document defines trigger phrases, arguments, and a high-level description of the existing 8-phase /audit-project workflow, including the reviewer roles and severity classification, and links to the corresponding commands/ docs.

Reviewed by Cursor Bugbot for commit 4cf32d3. Configure here.

The agentsys ecosystem catalog lists 'audit-project' under its Code
Review skill category, but the plugin had no SKILL.md - only commands.
This creates the missing skill so the 40-skills claim is accurate.

The skill delegates to the existing /audit-project command and
documents the 10 role-based reviewers, severity classification, and
typical invocations. Pure documentation of existing behavior - no
code changes and no new workflow.
Copy link
Copy Markdown

@gemini-code-assist gemini-code-assist bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Code Review

This pull request introduces the audit-project skill, which provides a multi-agent iterative code review workflow. The documentation outlines the tool's behavior, including agent selection, severity classification, and the iteration loop. Feedback focuses on maintaining consistency in the naming of specialized reviewers throughout the document to align with the defined role names.

Comment thread skills/audit-project/SKILL.md Outdated
Comment thread skills/audit-project/SKILL.md Outdated
avifenesh added a commit to agent-sh/agent-sh.dev that referenced this pull request Apr 15, 2026
plugins.json was missing 6 plugins entirely - prepare-delivery,
gate-and-ship, skillers, onboard, can-i-help, glidemq. Adding them
with correct agent/skill/command counts derived from each plugin repo.

skills.json was missing the skills for those new plugins plus
audit-project (just added in agent-sh/audit-project#17). Adding 9
skill entries: recommend, skillers-compact, onboard, can-i-help,
audit-project, glide-mq, glide-mq-migrate-bullmq, glide-mq-migrate-bee.

audit-project's skills count in plugins.json bumped from 0 -> 1 to
reflect the new SKILL.md upstream.

Totals after this change:
- 20 plugin entries (19 marketplace + glidemq)
- 49 agents (sum across plugins matches agentsys ecosystem total)
- 41 skills (sum across plugins matches skills.json entries)
avifenesh added a commit to agent-sh/agentsys that referenced this pull request Apr 15, 2026
The agnix plugin has a real SKILL.md at skills/agnix/SKILL.md but was
missing from the agentsys Skills category table and from STATIC_SKILLS
in generate-docs.js. With audit-project now also having a real SKILL.md
(agent-sh/audit-project#17), the canonical ecosystem total is 41:

- 4 prepare-delivery + 1 next-task + 1 ship + 1 deslop + 1 audit-project
- 1 drift-detect + 9 enhance + 1 sync-docs + 1 repo-intel + 8 perf
- 1 learn + 1 agnix + 1 consult + 1 debate + 2 web-ctl + 2 skillers
- 1 onboard + 1 can-i-help + 3 glidemq

Updated:
- README.md: 40 -> 41 in 4 locations, added 'Linting | agnix' row
- site/content.json: regenerated, 40 -> 41
- site/index.html: 40 -> 41 in meta/og/twitter/hero/section title/data-target
- site/ux-spec.md: 40 -> 41 in badge, subtitle, meta tags
- scripts/generate-docs.js: added agnix to STATIC_SKILLS

Tests pass (3507).
The 'security', 'performance', 'API' etc. shortened forms in the
description and Phase 2 prose conflict with the canonical names in the
Reviewers table (security-expert, performance-engineer, api-designer,
etc.). Aligning everything on the full names that match the role-based
agent IDs in commands/audit-project-agents.md.
@avifenesh avifenesh merged commit a7c7b95 into main Apr 15, 2026
3 checks passed
@avifenesh avifenesh deleted the feat/add-audit-project-skill branch April 15, 2026 08:42
avifenesh added a commit to agent-sh/agent-sh.dev that referenced this pull request Apr 15, 2026
* docs: add 6 missing plugins and align counts

plugins.json was missing 6 plugins entirely - prepare-delivery,
gate-and-ship, skillers, onboard, can-i-help, glidemq. Adding them
with correct agent/skill/command counts derived from each plugin repo.

skills.json was missing the skills for those new plugins plus
audit-project (just added in agent-sh/audit-project#17). Adding 9
skill entries: recommend, skillers-compact, onboard, can-i-help,
audit-project, glide-mq, glide-mq-migrate-bullmq, glide-mq-migrate-bee.

audit-project's skills count in plugins.json bumped from 0 -> 1 to
reflect the new SKILL.md upstream.

Totals after this change:
- 20 plugin entries (19 marketplace + glidemq)
- 49 agents (sum across plugins matches agentsys ecosystem total)
- 41 skills (sum across plugins matches skills.json entries)

* review: agnix in deps, drop simplify, derive 18-repos stat from plugin count

Gemini and Cursor flagged:
- prepare-delivery description listed 'simplify' (not a real plugin) and
  was missing 'agnix' from its dependencies array
- skills.astro had hardcoded '33 skills across 14 plugins' (now 41/20)
- index.astro hero badge had hardcoded '18 repos' even though pluginCount
  is already computed; using {pluginCount} here too so it stays in sync
avifenesh added a commit to agent-sh/agentsys that referenced this pull request Apr 15, 2026
* docs: add agnix to skills table, align skill count 40 -> 41

The agnix plugin has a real SKILL.md at skills/agnix/SKILL.md but was
missing from the agentsys Skills category table and from STATIC_SKILLS
in generate-docs.js. With audit-project now also having a real SKILL.md
(agent-sh/audit-project#17), the canonical ecosystem total is 41:

- 4 prepare-delivery + 1 next-task + 1 ship + 1 deslop + 1 audit-project
- 1 drift-detect + 9 enhance + 1 sync-docs + 1 repo-intel + 8 perf
- 1 learn + 1 agnix + 1 consult + 1 debate + 2 web-ctl + 2 skillers
- 1 onboard + 1 can-i-help + 3 glidemq

Updated:
- README.md: 40 -> 41 in 4 locations, added 'Linting | agnix' row
- site/content.json: regenerated, 40 -> 41
- site/index.html: 40 -> 41 in meta/og/twitter/hero/section title/data-target
- site/ux-spec.md: 40 -> 41 in badge, subtitle, meta tags
- scripts/generate-docs.js: added agnix to STATIC_SKILLS

Tests pass (3507).

* review: address gemini and copilot comments

- skills-grid: add agnix card so the displayed list matches the 41 count
- COMMAND_SUMMARIES: 385 -> 399 rules in /agnix tagline (was missed in
  the original agnix rule-count migration)
- content.json meta.description: 40 -> 41 (overlooked in main change)
- README skills table: move Linting row from after Code Review to after
  Analysis, matching scripts/generate-docs.js categoryOrder

* fix: meta.description 40 -> 41 skills (gen-docs only updates stats not description)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant