Menü
Shared review criteria and output format for code review agents
Mit Codex oder Claude installieren Kopieren Sie diesen Prompt, fügen Sie ihn in Codex, Claude oder einen anderen Assistant ein und lassen Sie die Skill-Seite prüfen und installieren.
Basierend auf der SOC-Berufsklassifikation
Investigate per-module Compose compiler metrics in this repo to find low-skippability hotspots and the underlying unstable classes.
Generate a Roborazzi visual-regression test (VRT) class for a Jetpack Compose composable in the sunsetscrob project, with all three required annotations — @RunWith(AndroidJUnit4::class), @GraphicsMode(GraphicsMode.Mode.NATIVE), and @Category(VRT::class) — plus the captureScreenshot wiring from :test_helper:integration. Use this skill whenever the user asks to add or create a VRT, a Roborazzi test, a screenshot test, a visual regression test, スクリーンショットテスト, スクショテスト, or VRT 追加 / VRT 作って — or points at a composable and asks for a visual regression test. The skill exists specifically to prevent forgetting @Category(VRT::class), which silently drops the test into the wrong CI bucket without failing compilation.
| name | code-review |
| description | Shared review criteria and output format for code review agents |
| tools | ["Bash(gh api:*)","Bash(gh pr list:*)","Bash(gh pr view:*)","Bash(gh pr diff:*)","Bash(gh pr checks:*)","Bash(gh pr checkout:*)","Bash(gh run list:*)","Bash(gh run view:*)","Read","Glob","Grep"] |
Shared review criteria and output format for code review agents.
@Immutable annotation, ImmutableList usageSunsetNavKey, callback hoisting (Screens never own a navigator), viewModelFor(key) usage in destinationsdata/repository, not directly on data/api or
data/dbAlways categorize findings into the following 3 categories:
Issues that must be fixed. Bugs, security risks, build errors, etc.
Issues that should be fixed. Convention violations, potential bugs, performance issues, etc.
Points to consider improving. Refactoring suggestions, readability improvements, etc.
## Critical issues (must fix)
- `feature/auth/LoginViewModel.kt:42` - API key is hardcoded. Please retrieve it from `local.properties`.
## Warnings (should fix)
- `feature/album/ui/screen/AlbumScreen.kt:15` - Screen and Content are not separated. Please separate them according to conventions.
- `core/entity/Track.kt:8` - Missing `@Immutable` annotation.
## Suggestions (consider improving)
- `feature/home/ui/viewmodel/HomeViewModel.kt:30` - Extracting this logic into a separate function would improve readability.
---
If a category has no findings, write "None".