/clean-code-review

← Vissza az áttekintéshez

Összefoglaló

Kész a kód — mielőtt kiszállítod, kérj mély átvizsgálást. 132 ellenőrzést futtat 7 csoportban (clarity, smells, SOLID, arch, tests, safety, DDD), csoportonként egy párhuzamos ágenssel, és súlyosság szerint szintetizálja az eredményeket. Működik helyi változásokra, staged fájlokra, git-tartományokra, pull/merge-request linkre vagy explicit fájlútvonalakra — git nélkül is, ha útvonalakat adsz meg.

A szkriptelt detekciók futnak le először a collect.sh-n keresztül, és előszámolt találatokként kerülnek a csoport-ágensekhez — az ágensek megerősítik vagy elvetik ezeket, nem futtatják újra a detekciót. A vendor és generált fájlok már eleve ki vannak zárva. Nagyon nagy diffeknél az egyes ellenőrzésekre eső találatszám korlátozható; a riport jelzi, ha ez történt.

Mikor használd

Megírtad a kódot, és strukturált átvizsgálást szeretnél PR nyitása előtt — nem hangulatellenőrzést, hanem szisztematikus végigfutást 132 nevesített kritériumon. Akkor nyúlj a /clean-code-review-hoz, ha csoportosított, súlyossággal jelölt és összegzett eredménylistát szeretnél.

A skill nem deklarál trigger kifejezéseket — hívd meg közvetlenül: /clean-code-review.

Meghívás

/clean-code-review [TARGET ...] [GROUP ...]

Célok (alapértelmezett: összes helyi változás):

Csoportok (kis- és nagybetű-érzéketlen, kihagyva mind a 7 fut):

Egy csoportnévvel egyező token mindig csoportot jelöl, sosem fájlt vagy refet — pl. a /clean-code-review tests a tests csoportot választja ki, nem egy tests nevű könyvtárat.

Nyelvi kapu: ha a célban nincs támogatott nyelvű fájl, a review megáll, mielőtt bármelyik ágenst elindítaná: "No supported language files detected. Supported: TypeScript, JavaScript, C#, Python, Swift, Kotlin, Java, C++. Unanalysed: {contents of unanalysed.txt}."

Súlyossági szintek: Kritikus (helyességet, biztonságot vagy védettséget akadályoz) · Súlyos (jelentős tervezési hiba, hiányzó követelmény vagy valószínű bug) · Közepes (nem optimális, de működőképes) · Apró (stílus, elnevezés vagy szőrszálhasogatás)

1. példa — Helyi változások teljes átvizsgálása (szemléltető példa)

Bemenet:

/clean-code-review

Kimenet / Eredmény:

## Clean Code Review

**5 findings · 0 Critical · 4 Major · 0 Moderate · 1 Minor** (post-deduplication)
**Target:** staged unstaged untracked
**Groups run:** clarity, smells, solid, arch, tests, safety, ddd · 132 checks declared
**Languages detected:** typescript

---

### `src/services/UserService.ts`

[solid-01] · Major · Single Responsibility Principle | src/services/UserService.ts:12 | Split UserService — separate auth, email, and billing into distinct classes

### `src/services/PaymentProcessor.ts`

[solid-05] · Major · Dependency Inversion Principle | src/services/PaymentProcessor.ts:34 | Replace direct dependency on PostgresClient with an interface — inject the implementation
[smells-04] · Major · Code Duplication | src/services/PaymentProcessor.ts:58 | Remove duplicate validation logic — reuse existing validateInput in src/utils/validation.ts

### `src/services/__tests__/PaymentProcessor.test.ts`

[tests-07] · Major · Test Edge Case Coverage | src/services/__tests__/PaymentProcessor.test.ts:20 | Add edge case test for chargeCard — cover null input

### `src/utils/format.ts`

[clarity-02] · Minor · Function/Method Naming | src/utils/format.ts:8 | Rename doStuff to formatCurrency — name does not express intent

2. példa — Egyetlen csoport egy git-tartományra (szemléltető példa)

Bemenet:

/clean-code-review main..HEAD safety

Kimenet / Eredmény:

## Clean Code Review

**2 findings · 1 Critical · 1 Major · 0 Moderate · 0 Minor** (post-deduplication)
**Target:** ref: main..HEAD
**Groups run:** safety · 32 checks declared
**Languages detected:** python

---

### `src/services/order_service.py`

[safety-01] · Critical · Resource Leak | src/services/order_service.py:23 | Wrap db_connection in a `with` block to guarantee cleanup

### `src/services/notification_service.py`

[safety-06] · Major · Blocking I/O in Async Context | src/services/notification_service.py:41 | Replace blocking requests.get inside async send_notification with an async HTTP client

Kapcsolódó eszközök