valorvie/ai-dev-skills

simplify

Use when user wants a pre-review cleanup pass on changed code by running parallel reviews for reuse, code quality, and efficiency, then applying worthwhile fixes without changing behavior.

Vedi sorgente
Documento Skill originale

Contenuto dal repository con titoli, esempi, codice, tabelle, link e immagini preservati.

Simplify

Overview

Use this as a pre-review cleanup gate. The goal is not formal approval; it is to inspect changed code for reuse opportunities, code quality issues, and efficiency issues, then fix the worthwhile findings without changing behavior.

When to Use

  • The user wants a /simplify-style cleanup pass before reviewer or code-reviewer
  • A feature is complete and the diff likely contains duplication, rough edges, or avoidable inefficiency
  • You want to reduce avoidable reviewer comments before formal review
  • The change spans enough files or logic that one simplification pass is not enough

Do not use this skill for:

  • Tiny diffs where a single local cleanup is enough
  • Behavior changes, feature work, or bug fixes that alter requirements
  • Large architectural redesigns across untouched modules
  • Formal spec validation or security review

Place in Review Flow

Recommended placement:

text
implement -> simplify -> code-reviewer and/or reviewer -> receiving-code-review

Skip the full three-agent pass when:

  • the change is tiny and obvious
  • only one or two trivial lines changed
  • the user wants immediate formal review instead of cleanup

Workflow

Phase 1: Identify Changes

  • If the user gives a base/head range, use that
  • If staged changes exist, inspect git diff HEAD so both staged and unstaged changes are included
  • Otherwise use git diff
  • If there is no git diff, review the most recently modified files the user mentioned or that you edited earlier in the session
  • Keep a compact summary of touched files plus the full diff text for the reviewers

Phase 2: Read Local Standards

  • Check local instructions first such as AGENTS.md, repo conventions, and any relevant skills
  • Follow existing naming, formatting, and structure in the touched files
  • If local conventions conflict with generic cleanup preferences, local conventions win

Phase 3: Launch Three Review Agents in Parallel

For medium or large diffs, spawn three read-only subagents in parallel in a single round. Use explorer agents when possible because this is codebase analysis, not implementation.

Each agent must receive:

  • the full diff
  • the touched file list
  • the cleanup constraint: no behavior changes, no edits, findings only

The three review domains are:

  • Code reuse review
  • Code quality review
  • Efficiency review

Use the prompt templates in:

Important:

  • Review agents are read-only; they do not edit files
  • The main agent aggregates findings and performs the fixes
  • Do not spawn multiple worker agents to edit the same touched files in parallel

Phase 4: Fix the Worthwhile Issues

  • Wait for all reviewers to finish
  • Aggregate findings and remove duplicates
  • Fix issues directly if they improve the code without changing behavior
  • If a finding is a false positive or not worth the churn, note that and skip it without arguing at length

Phase 5: Verify

  • Run the narrowest meaningful verification for the changed area
  • Prefer targeted tests first, then lint/build if appropriate
  • If no automated verification exists, state that explicitly and mention residual risk
  • Briefly summarize what was fixed, or state that the code was already clean enough

Quick Reference

GoalPreferAvoid
ReadabilityClear names, explicit branches, small coherent helpersDense one-liners, clever tricks
Scope controlTouched files, recent diff, user-specified pathsRepo-wide cleanup without request
SafetyTargeted verification after editsAssuming refactor is safe
CommentsIntent and constraintsNarrating obvious code
Agent usageThree parallel read-only reviewers, one controller applying fixesMultiple editing agents touching the same files

Common Mistakes

  • Turning simplification into redesign
  • Extracting abstractions too early
  • Renaming broadly across unrelated files
  • Optimizing for fewer lines instead of easier maintenance
  • Skipping verification because the change "looks mechanical"
  • Letting all three agents edit code instead of only reporting findings
  • Using this skill as a substitute for formal review or spec validation

Upstream Reference

This Codex skill adapts the /simplify flow used in Claude-based workflows into a Codex-native skill. See references/upstream-agent.md for the source notes and adaptation details.

dallo stesso repository

Altri Skills

Tutti gli Skills
valorvie
Community

custom-agent-router

為目標、範圍與授權已明確的非簡單 Agent 工作選擇互動模式、能力層級、風險、派工形狀、審查與有限備援。適用於需要在主 Agent 直接處理、單一工作代理與有限平行之間選擇,或需要依複雜度與風險決定能力層級及審查強度的工作;Codex 專案缺少已驗證的 Agent 設定時,也用本 Skill 引導使用者選擇建立、預覽或略過。簡單問答、唯讀確認與可直接驗證的小修改不要使用。

installazioni
1
GitHub Stars
0
Aggiornato
17 set
valorvie
Community

custom-skills-dev

Guide development of the ai-dev framework and its first-party skill collection. Use when modifying ai-dev CLI code, npx skill integration, copy/distribution logic, commands, agents, plugins, first-party skills, tests, or release documentation. Triggers: "develop custom-skills", "modify ai-dev", "add skill", "update distribution", "release version", "開發 custom-skills", "修改 ai-dev", "新增 skill", "發布版本".

installazioni
1
GitHub Stars
0
Aggiornato
17 set
valorvie
Community

custom-skills-doc-updater

Maintain documentation for the ai-dev framework and ai-dev-skills collection. Use when code or skill changes affect documented behavior, commands, agents, plugins, version releases, repository structure, README content, or changelogs. Triggers: "update docs", "sync documentation", "update README", "update changelog", "documentation audit", "doc sync", "更新文檔", "同步文件", "文檔檢查", "/doc-updater".

installazioni
1
GitHub Stars
0
Aggiornato
17 set
valorvie
Community

custom-skills-doc-writer

依讀者任務與可驗證主線撰寫、重構及整理技術文件,並依專案慣例處理文件類型、路徑、檔名、frontmatter、索引與生命週期。技術文件另保護中英混合內容中的事實、命令、路徑與識別字,英文說明文字採 ASD-STE100 大原則;使用者同時提供合法標準 PDF 與本機檢查器時可自動輔助檢查。只要使用者要新增或實質改寫計畫、調查、分析、研究、進度、指南、維運手冊、教學、參考資料、原理解釋、決策、事件、會議、變更日誌或規範,就應使用本技能;整理 docs 目錄與主題索引時也適用。單純翻譯、一小段文字潤飾、只改程式碼或只查一個事實時不要使用。

installazioni
1
GitHub Stars
0
Aggiornato
17 set