Asset Review

---
name: asset-review
description: Bekleyen asset-browser upload'larini incele — her biri icin onay/red karari ver, onaylananlari post-process edip runtime asset dizinine tasi, commit+push. Triggers: asset review, asset incele, bekleyenleri incele, asset review workflow, waiting review.
---

# /asset-review — Bekleyen Asset'leri Incele

`asset-browser` kurulu bir projenin `waiting-for-review` listesini inceler. Her item icin:
1. Upload edilen ham dosyayi ceker (GitHub'dan)
2. Preview eder
3. Onayla / Reddet (sebep) / Atla karari verir
4. Onaylanirsa: post-process (animasyon ise strip + WebP), runtime dizine tasi, commit+push
5. Reddedilirse: `denyReason` ile status denied olur
6. Status'u `/api/review` ile günceller

## Kullanim

```
/asset-review                    # CWD projesi
/asset-review /path/to/project   # belirli proje
```

## Akis

### 1. Proje tespiti

`<project>/asset-browser/config.json` okunur. Yoksa hata.

### 2. Bekleyen listesi

GitHub'dan taze `missing.json` ceker:
```bash
curl -s -H "Authorization: Bearer $GITHUB_TOKEN" \
  "https://api.github.com/repos/$OWNER/$REPO/contents/asset-browser/data/missing.json?ref=main" \
  | jq -r '.content' | base64 -d | jq '.items[] | select(.status=="waiting-for-review")'
```

### 3. Her bekleyen icin

1. **Dosya cek** (GitHub blobs API — >1MB destekli):
   ```bash
   node -e '... fetch blob by sha ...' > /tmp/review/<file>
   ```
2. **Preview**: `open /tmp/review/<file>` ve `magick identify`
3. **Karar**: kullaniciya sor (veya otomatik mod varsa kurallara gore karar ver):
   - Onayla → post-process + tasi
   - Reddet → sebep iste, /api/review deny
   - Atla → sonraki item

### 4a. Web-approved items (post-process eksik)

`/api/review approve` artik uploaded dosyayi **ham halde** runtime dir'e kopyaliyor. Ama bu animasyon icin DOGRU degil — post-process gerekli.

Bu durumu tespit et:
```bash
for name in $(curl -s ...missing.json | jq -r '.items[] | select(.status=="approved" and .type=="Animasyon") | .name'); do
  # runtime file dim kontrolu
  magick identify $PROJECT/game/public/assets/$name.webp
  # dim raw grid (1536x1024 / 2048x1024 / similar) ise = ham kopyalanmis, split gerekli
done
```

Ham bulunanlari listele → kullaniciya sor "bunlari post-process edip yenileyeyim mi?" → evetse magick pipeline ile:
1. Magenta transparent
2. Uygun split (6x1 horizontal / 3x2 grid / etc. isme gore `_6f` / `_8f`)
3. Trim + center + extent 256x256 per frame
4. Append horizontal strip
5. WebP q90, overwrite runtime
6. Commit + push

### 4b. Onay + post-process (tip bazli)

**Animasyon (type=="Animasyon")**:
- Raw input genelde horizontal veya grid layout + magenta separator
- `-fuzz 35% -transparent magenta` → temizle
- `-crop NxM@` ile frame'lere bol (N = frame_count / 1 veya dosya adindan _Nf)
- Her frame **aspect-preserving fit** yap: `-background none -resize 256x256 -gravity center -extent 256x256`. **ASLA** sadece `-gravity south -extent 256x256` kullanma — kaynak 256'dan uzunsa tepeyi kirpar (blacksmith head-crop bug'i buradan cikti Apr 2026). Once `-resize 256x256` ile aspect koruyarak kucult, sonra `-extent 256x256` ile padding ekle.
- `+append` ile strip birlestir (1536x256 6-frame vb.)
- `cwebp -q 90 -alpha_q 100` → WebP
- Hedef: `<project>/game/public/assets/<name>.webp` veya config'de belirlenen runtime dizini

**Resim (type=="Resim")**:
- Raw genelde buyuk PNG
- `magick IN -resize '512x512>'` → sinirla
- `cwebp -q 90 -alpha_q 100` → WebP
- Hedef runtime dizini

### 5. Status transition + commit

1. `/api/review` ile status approved yap:
   ```bash
   curl -X POST https://<alias>/api/review -d '{"name":"X","action":"approve"}'
   ```
2. Runtime dosyasini commit + push:
   ```bash
   cd <project> && git add <runtime_path> && git commit -m "feat: approve <name>" && git push
   ```

### 6. Reddet akisi

```bash
curl -X POST https://<alias>/api/review -d '{"name":"X","action":"deny","reason":"..."}'
```

### 7. Ozet

```
━━ Asset Review Complete ━━━━━━━━━━━━━━━━
  ✓ fire_loop_6f    — approved (6 frames 256x256 -> game/public/assets/fire_loop_6f.webp)
  ✗ cart_wheels_8f  — denied (Reason: proportions drift between frames)
  ⊘ smoke_rise_6f   — skipped (user undecided)
━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━
```

## Runtime dizini tespiti

`config.json` sources'ta `"category": "In Game"` olan ilk entry'nin `dir`'i runtime dizini.

Yoksa: `game/public/assets` -> fallback.

## Kurallar

- Proje ayarlarindan (config.json / AGENT.md) style kurallarini oku
- Kalite sinirlari: frame count, transparency, edge style esleme (bkz. `feedback_asset_quality_rigor.md`)
- Dosya boyutu > 1MB icin GitHub blobs API kullan (contents API 1MB sinir)
- WebP 512px max, q90, alpha_q 100
- Ham PNG'yi de `01_assets/transparent_png/` varsa oraya sakla

## Alias

Vercel stable alias `config.json`'daki title'dan turetilir:
`asset-browser-<slug>.vercel.app` (slug = title lowercase + non-alnum -> dash)

## When NOT to Use
- Hic bekleyen yoksa — sessizce bitir
- asset-browser kurulu degilse

## Red Flags
- Upload dosyasi bozuk / cok kucuk / yanlis format
- Proje kurallariyla celisen asset

## Error Handling
- API 4xx/5xx → retry (1 kere) → kullaniciya sor
- Post-process fail → kullaniciya sebep goster, item'i atla
- Git push fail → rebase + retry