本文へ移動
cccskills
無料GitHub で公開

review-code-smells

Review touched code for maintainability smells such as duplication, awkward indirection, suspicious clones, dead code, poor naming, and Boy Scout cleanup opportunities. Use when a broad review needs a dedicated maintainability pass or when the user asks for code-smell review specifically.

インストール方法を見る

含まれるファイル(1)

  • SKILL.md2.1 KB

SKILL.md(原文)

インストールする前に、エージェントに与えられる指示の中身を確認できます。

Review Code Smells — Incan Compiler

Purpose

/review-code-smells is a file-focused report-only reviewer for local maintainability issues.

Own:

  • duplication
  • dead code
  • awkward or unnecessary indirection
  • suspicious clones
  • poor naming in touched code
  • touched-code cleanup opportunities that satisfy the Boy Scout rule

Do not own:

  • architecture-level placement decisions
  • docs truthfulness
  • test style details
  • final branch-clean judgment

Output artifact

Write a slice report at:

  • .agents/state/review-report.code-smells.md

Do not write to the canonical .agents/state/review-report.md.

Workflow

  1. Review the touched code files assigned by the orchestrator.
  2. Flag:
    • duplicated branches or helpers
    • no-op/unused/dead surfaces
    • awkward APIs or naming in touched code
    • needless complexity relative to the actual requirement
  3. Keep this pass practical. Do not invent refactors unrelated to the touched surface.
  4. Classify each finding as either:
    • maintainability
    • design-tension
    • or bug This classification is for downstream handling only. It must not be used to hide the finding.

Slice report shape

Keep slice reports findings-first. Only list clean surfaces when that clean call is non-obvious or useful for the orchestrator.

# Review Slice Report

- role: code-smells
- worker: <agent-id or stable label>
- status: in_progress | clean | blocked

## Scope
- assigned files:
  - loaves/toolchain/incan-cli/src/lib.rs

## Findings

- [ ] warning | maintainability | dead cli surface | loaves/toolchain/incan-cli/src/lib.rs:313
  `--bin` is exposed publicly but has no effect.

## Reviewed Clean Surfaces
- loaves/toolchain/incan-cli/src/commands/init.rs — maintainability reviewed; no findings

If there are no findings, say so explicitly.

レビュー

まだレビューはありません。使ってみた感想をお寄せください。

同じリポジトリのスキル

概要と使いどころ

Add a new learning to the agent learnings file. Use when the user says /add-learning, asks to record a lesson, or when an implementation produced a reusable insight worth preserving for future agents.

日本語の概要は準備中です。原文の説明を表示しています。

encero-systems/incan612026年10月12日 更新

bump-rfc

無料

Transition an Incan RFC from one status to the next. Use when the user asks to promote, advance, or finalize an RFC, or says /bump-rfc. Handles Draft → Planned, Planned → In Progress, and In Progress → Implemented transitions.

日本語の概要は準備中です。原文の説明を表示しています。

encero-systems/incan612026年10月12日 更新

closeout

無料

Safely close out completed Incan work after a PR is merged by verifying merge status, syncing the base branch, removing task-owned local worktrees/assets, deleting merged local/remote branches, pruning refs, and reporting dirty or ambiguous leftovers. Use when the user says /closeout, asks to clean up after a merged PR, or wants local RFC/issue branch/worktree cleanup.

日本語の概要は準備中です。原文の説明を表示しています。

encero-systems/incan612026年10月12日 更新

Drafts a GitHub issue title and body using the target repository's issue templates under .github/ISSUE_TEMPLATE. Use when the user asks to create, draft, or file a GitHub issue, bug report, feature request, chore, documentation issue, or RFC proposal, or wants issue text that matches the repo's template.

日本語の概要は準備中です。原文の説明を表示しています。

encero-systems/incan612026年10月12日 更新

Drafts implementation plans with TDD, documentation updates, and repository verification commands before coding. Use when the user asks for an implementation plan, /create-plan, or structured pre-implementation design for work in encero workspaces (e.g. Incan, IncQL).

日本語の概要は準備中です。原文の説明を表示しています。

encero-systems/incan612026年10月12日 更新

Generate a PR description following the repository's pull request template. Use when the user asks to create, draft, or generate a PR description for a pull request. The skill automatically locates the PR template in the target repository and fills it in based on the git diff.

日本語の概要は準備中です。原文の説明を表示しています。

encero-systems/incan612026年10月12日 更新

encero-systems のスキルをすべて見る

このスキルの問題を報告する