vuralserhat86/skills/code_review
skills/code_review/SKILL.md
--- name: code_review router_kit: FullStackKit description: PR review, code smell detection, best practice kontrolü. ⚠️ Kod incelerken kullan. Deliverable kontrolü için → quality-validator, doküman review için → peer-review. metadata: skillport: category: quality tags: [architecture, automation, best practices, clean code, code review, coding, collaboration, compliance, debugging, design patterns, development, documentation, efficiency, git, optimization, productivity, programming, pro
$ install --global
skillsauthnpx skillsauth add vuralserhat86/antigravity-agentic-skills skills/code_reviewInstall this skill globally with one command. Works with Claude Code, Cursor, and Windsurf.
Security Scan Results
3 of 9 scanners reported clean
Some scanners were skipped, did not run, or reported a non-clean status. Review each row below.
3
Scanners Passed
9
Scanners in report
Clean
TrivyContainer and dependency vulnerability scanner
95%
Clean
SemgrepStatic code analysis for vulnerabilities
95%
Clean
mcp-scan (Snyk)Model Context Protocol security validation
95%
Skipped
Snyk (dep)Open source security scanning
50%
Skipped
Socket.devSupply chain security analysis
50%
Skipped
VirusTotalMulti-engine malware detection
50%
Skipped
CrowdStrikeAdvanced threat intelligence
50%
Skipped
OSV-ScannerOpen Source Vulnerability database check
50%
Skipped
OWASP Dep-Check
50%
Last scanned: Apr 22, 2026, 1:18 AM128.3s1 file scanned
SKILL.md
- name:
- code_review
- router_kit:
- FullStackKit
- description:
- PR review, code smell detection, best practice kontrolü. ⚠️ Kod incelerken kullan. Deliverable kontrolü için → quality-validator, doküman review için → peer-review.
- category:
- quality
- tags:
- [architecture, automation, best practices, clean code, code review, coding, collaboration, compliance, debugging, design patterns, development, documentation, efficiency, git, optimization, productivity, programming, project management, quality assurance, refactoring, software engineering, standards, testing, utilities, version control, workflow] - code-smell
🔍 Code Review
Etkili kod inceleme ve kalite kontrol rehberi.
📋 İçindekiler
- Review Checklist
- Code Smell Detection
- PR Best Practices
- Review Comments
1. Review Checklist
Fonksiyonellik
- [ ] Kod istenen işi yapıyor mu?
- [ ] Edge case'ler handle ediliyor mu?
- [ ] Error handling yeterli mi?
- [ ] Input validation var mı?
Kod Kalitesi
- [ ] DRY prensibi uygulanmış mı?
- [ ] Single Responsibility takip ediliyor mu?
- [ ] Naming conventions tutarlı mı?
- [ ] Magic numbers/strings yok mu?
Güvenlik
- [ ] SQL injection riski var mı?
- [ ] XSS riski var mı?
- [ ] Sensitive data expose edilmiş mi?
- [ ] Authentication/authorization doğru mu?
Performans
- [ ] N+1 query problemi var mı?
- [ ] Gereksiz re-render var mı?
- [ ] Memory leak riski var mı?
- [ ] Büyük dosya/data handling doğru mu?
2. Code Smell Detection
Yaygın Code Smell'ler
| Smell | Açıklama | Çözüm | |-------|----------|-------| | Long Method | >20 satır fonksiyon | Extract Method | | Large Class | >300 satır class | Extract Class | | Long Parameter List | >3 parametre | Parameter Object | | Duplicate Code | Tekrarlayan bloklar | Extract Method/Class | | Dead Code | Kullanılmayan kod | Sil | | Magic Numbers | Açıklamasız değerler | Constants | | Deep Nesting | >3 seviye if/loop | Early return, Extract | | God Class | Her şeyi yapan class | Single Responsibility |
Algılama Komutları
# ESLint complexity check
npx eslint . --rule 'complexity: ["error", 10]'
# SonarQube
sonar-scanner
# Code coverage
npm run test:coverage
3. PR Best Practices
İdeal PR Boyutu
- Küçük: <200 satır (ideal)
- Orta: 200-400 satır
- Büyük: >400 satır (bölünmeli)
PR Açıklama Template
## Özet
Kısa açıklama
## Değişiklik Tipi
- [ ] Bug fix
- [ ] Yeni özellik
- [ ] Refactoring
- [ ] Breaking change
## Test
- Test X yapıldı
- Test Y sonucu: başarılı
## Screenshots (UI değişikliği varsa)
Commit Messages
feat: Add user authentication
fix: Resolve memory leak in cache
refactor: Extract validation logic
docs: Update API documentation
test: Add unit tests for user service
chore: Update dependencies
4. Review Comments
Etkili Yorum Yazma
❌ Kötü: "Bu yanlış"
✅ İyi: "Bu yaklaşım X durumunda hata verebilir. Y alternatifini düşünebilir misin?"
❌ Kötü: "Bunu değiştir"
✅ İyi: "suggestion: Bu fonksiyon extract edilse okunabilirlik artar"
Yorum Prefixleri
| Prefix | Anlam |
|--------|-------|
| blocking: | Merge edilemez, düzeltilmeli |
| suggestion: | Öneri, isteğe bağlı |
| question: | Açıklama gerekiyor |
| nitpick: | Minor, önemsiz |
| praise: | İyi iş! |
Code Review v1.1 - Enhanced
🔄 Workflow
Kaynak: Google Engineering Practices
Aşama 1: Triage (Ön Kontrol)
- [ ] CI Checks: Testler geçmiş mi? Lint hatası var mı?
- [ ] Scope: PR çok mu büyük? (>400 satır ise bölmesini iste).
- [ ] Description: "Ne" ve "Neden" açıkça anlatılmış mı?
Aşama 2: Deep Dive
- [ ] Logic: Kodun algoritması doğru ve verimli mi?
- [ ] Architecture: Mevcut mimari desenlere uyuyor mu?
- [ ] Test: Yeni özellikler için test yazılmış mı?
Aşama 3: Feedback
- [ ] Comments: Yapıcı, nazik ve net yorumlar yaz (
suggestion:,question:). - [ ] Decision: Approve, Request Changes veya Comment.
Kontrol Noktaları
| Aşama | Doğrulama | |-------|-----------| | 1 | PR açıklaması boş ise reddedildi mi? | | 2 | Breaking change varsa versiyonlamaya dikkat edildi mi? | | 3 | Yorumlar kişiye değil koda mı yönelik? |
Related Skills
vuralserhat86/zustand_state
tools
VerifiedTrustedCommunity
Production-tested setup for Zustand state management in React. Includes patterns for persistence, devtools, and TypeScript patterns. Prevents hydration mismatches and render loops.
42SKILL.mdUpdated Apr 22, 2026
vuralserhat86/xlsx
development
VerifiedTrustedCommunity
Comprehensive spreadsheet creation, editing, and analysis with support for formulas, formatting, data analysis, and visualization. When Claude needs to work with spreadsheets (.xlsx, .xlsm, .csv, .tsv, etc) for: (1) Creating new spreadsheets with formulas and formatting, (2) Reading or analyzing data, (3) Modify existing spreadsheets while preserving formulas, (4) Data analysis and visualization in spreadsheets, or (5) Recalculating formulas
42SKILL.mdUpdated Apr 22, 2026
vuralserhat86/skills/websocket_engineer
development
VerifiedTrustedCommunity
--- name: websocket_engineer router_kit: FullStackKit description: WebSocket specialist for real-time communication systems. Invoke for Socket.IO, WebSocket servers, bidirectional messaging, presence systems. Keywords: WebSocket, Socket.IO, real-time, pub/sub, Redis. triggers: - WebSocket - Socket.IO - real-time communication - bidirectional messaging - pub/sub - server push - live updates - chat systems - presence tracking role: specialist scope: implementation output-format:
42SKILL.mdUpdated Apr 22, 2026
vuralserhat86/webapp_testing
tools
VerifiedTrustedCommunity
Toolkit for interacting with and testing local web applications using Playwright. Supports verifying frontend functionality, debugging UI behavior, capturing browser screenshots, and viewing browser logs.
42SKILL.mdUpdated Apr 22, 2026