Approve Yi Ji handpainted sprite v2

docs/handpaint-batch7-yi-ji-v2-report.md

@@ -0,0 +1,24 @@
+# Yi Ji hand-painted v2 report
+
+- source: `handpaint-batch7-yi-ji-source-v1.png`
+- base status: ok
+- action status: ok
+- base partial alpha: 0, opaque pixels: 2272400
+- action partial alpha: 0, opaque pixels: 5642111
+- base frame border hits: 0
+- action frame border hits: 0
+- base empty frames: 0
+- action empty frames: 0
+- base bbox width range: 174..212
+- base bbox height range: 286..288
+- base bottom range: 303..303
+- action bbox width range: 167..273
+- action bbox height range: 286..295
+- action bottom range: 303..303
+- contact: `handpaint-batch7-yi-ji-v2-contact.png`
+- animation: `handpaint-batch7-yi-ji-v2-animation.gif`
+- before/after: `handpaint-batch7-yi-ji-v2-before-after.png`
+- scale compare: `handpaint-batch7-yi-ji-v2-scale-compare.png`
+- battle composite: `handpaint-batch7-yi-ji-v2-battle-composite.png`
+- manual checks: run build and local browser verification after generation
+- note: battle composite is a scale review image; gameplay roster is unchanged during sample approval

docs/unit-sprite-art-quality-gate.md

@@ -6,9 +6,9 @@ This gate catches obvious sprite-art failures before user approval. It does not
 
 - Manifest: `docs/unit-sprite-rework-manifest.json`
 - Audit: `docs/unit-sprite-rework-audit.json`
-- Approved baseline units: 88
+- Approved baseline units: 89
 - Pending approval units: 0
-- Queued rework units: 3
+- Queued rework units: 2
 
 ## Checked Rules
 

docs/unit-sprite-batch-queue.json

@@ -90,13 +90,14 @@
     "unit-wu-strategist-river",
     "unit-wu-yi",
     "unit-yan-yan",
+    "unit-yi-ji",
     "unit-zhang-fei"
   ],
   "pendingApproval": [],
   "nextAfterApproval": [],
   "stageCounts": {
-    "approved-baseline": 88,
-    "queued-rework": 3
+    "approved-baseline": 89,
+    "queued-rework": 2
   },
   "batches": [
     {
@@ -1182,24 +1183,10 @@
       "label": "Batch 7 unique officers",
       "total": 19,
       "stageCounts": {
-        "approved-baseline": 16,
-        "queued-rework": 3
+        "approved-baseline": 17,
+        "queued-rework": 2
       },
       "units": [
-        {
-          "stem": "unit-yi-ji",
-          "reviewStage": "queued-rework",
-          "queueRole": "queued rework",
-          "baseStatus": "border-hits=32",
-          "actionStatus": "border-hits=98",
-          "battleBasePx": [
-            38.8,
-            39.3,
-            44.7,
-            44.9
-          ],
-          "spec": null
-        },
         {
           "stem": "unit-zhao-yun",
           "reviewStage": "queued-rework",
@@ -1451,6 +1438,20 @@
             46.0
           ],
           "spec": null
+        },
+        {
+          "stem": "unit-yi-ji",
+          "reviewStage": "approved-baseline",
+          "queueRole": "approved reference",
+          "baseStatus": "ok",
+          "actionStatus": "ok",
+          "battleBasePx": [
+            27.8,
+            33.9,
+            45.7,
+            46.0
+          ],
+          "spec": null
         }
       ]
     }

docs/unit-sprite-batch-queue.md

@@ -6,9 +6,9 @@ Use this as the long-running queue for the full hand-painted sprite rework. It i
 
 ## Current State
 
-- Approved baseline units: 88
+- Approved baseline units: 89
 - Pending approval units: 0
-- Queued rework units: 3
+- Queued rework units: 2
 - Manifest: `docs/unit-sprite-rework-manifest.json`
 - Audit: `docs/unit-sprite-rework-audit.json`
 
@@ -157,11 +157,10 @@ No next unit is currently defined.
 
 - Batch key: `batch7-unique-officer`
 - Total units: 19
-- Stage counts: approved-baseline 16, queued-rework 3
+- Stage counts: approved-baseline 17, queued-rework 2
 
 | unit | role | stage | status | battle base px | spec |
 | --- | --- | --- | --- | --- | --- |
-| `unit-yi-ji` | queued rework | queued-rework | base border-hits=32; actions border-hits=98 | 38.8..39.3 x 44.7..44.9 |  |
 | `unit-zhao-yun` | queued rework | queued-rework | base border-hits=64; actions border-hits=144 | 26.5..47.3 x 45.0..46.0 |  |
 | `unit-zhuge-liang` | queued rework | queued-rework | base border-hits=32; actions border-hits=98 | 38.8..39.3 x 44.7..44.9 |  |
 | `unit-cao-cao` | approved reference | approved-baseline | technical ok | 32.9..36.4 x 46.3..47.3 |  |
@@ -180,3 +179,4 @@ No next unit is currently defined.
 | `unit-sima-yi` | approved reference | approved-baseline | technical ok | 31.3..33.5 x 46.0..46.0 |  |
 | `unit-wang-ping` | approved reference | approved-baseline | technical ok | 28.0..33.7 x 45.7..46.0 |  |
 | `unit-yan-yan` | approved reference | approved-baseline | technical ok | 30.4..37.5 x 45.7..46.0 |  |
+| `unit-yi-ji` | approved reference | approved-baseline | technical ok | 27.8..33.9 x 45.7..46.0 |  |

docs/unit-sprite-next-steps.md

@@ -4,9 +4,9 @@ This file is generated by `scripts/write-unit-sprite-next-steps.py`.
 
 ## Current Gate
 
-- Approved baseline units: 88
+- Approved baseline units: 89
 - Pending approval units: 0
-- Queued rework units: 3
+- Queued rework units: 2
 - Manifest: `docs/unit-sprite-rework-manifest.json`
 - Audit: `docs/unit-sprite-rework-audit.json`
 - Current approval board: not generated until the next pending sample.

docs/unit-sprite-rework-audit.json

@@ -11,8 +11,8 @@
     "batch7-unique-officer": 19
   },
   "stage_counts": {
-    "approved-baseline": 88,
-    "queued-rework": 3
+    "approved-baseline": 89,
+    "queued-rework": 2
   },
   "rows": [
     {
@@ -4976,7 +4976,7 @@
     },
     {
       "batch": "batch7-unique-officer",
-      "review_stage": "queued-rework",
+      "review_stage": "approved-baseline",
       "stem": "unit-yi-ji",
       "base": {
         "key": "unit-yi-ji",
@@ -4986,23 +4986,23 @@
           1252
         ],
         "partial_alpha": 0,
-        "opaque_pixels": 1999925,
-        "border_hits": 32,
-        "min_width": 243,
-        "max_width": 246,
-        "min_height": 280,
-        "max_height": 281,
-        "min_bottom": 281,
-        "max_bottom": 287,
+        "opaque_pixels": 2272400,
+        "border_hits": 0,
+        "min_width": 174,
+        "max_width": 212,
+        "min_height": 286,
+        "max_height": 288,
+        "min_bottom": 303,
+        "max_bottom": 303,
         "empty_frames": 0,
-        "width_swing": 3,
-        "height_swing": 1,
-        "bottom_swing": 6,
-        "battle_min_width_px": 38.8,
-        "battle_max_width_px": 39.3,
-        "battle_min_height_px": 44.7,
-        "battle_max_height_px": 44.9,
-        "status": "border-hits=32"
+        "width_swing": 38,
+        "height_swing": 2,
+        "bottom_swing": 0,
+        "battle_min_width_px": 27.8,
+        "battle_max_width_px": 33.9,
+        "battle_min_height_px": 45.7,
+        "battle_max_height_px": 46.0,
+        "status": "ok"
       },
       "actions": {
         "key": "unit-yi-ji-actions",
@@ -5012,23 +5012,23 @@
           1252
         ],
         "partial_alpha": 0,
-        "opaque_pixels": 4453376,
-        "border_hits": 98,
-        "min_width": 220,
-        "max_width": 255,
-        "min_height": 275,
-        "max_height": 281,
-        "min_bottom": 274,
-        "max_bottom": 289,
+        "opaque_pixels": 5642111,
+        "border_hits": 0,
+        "min_width": 167,
+        "max_width": 273,
+        "min_height": 286,
+        "max_height": 295,
+        "min_bottom": 303,
+        "max_bottom": 303,
         "empty_frames": 0,
-        "width_swing": 35,
-        "height_swing": 6,
-        "bottom_swing": 15,
-        "battle_min_width_px": 35.1,
-        "battle_max_width_px": 40.7,
-        "battle_min_height_px": 43.9,
-        "battle_max_height_px": 44.9,
-        "status": "border-hits=98"
+        "width_swing": 106,
+        "height_swing": 9,
+        "bottom_swing": 0,
+        "battle_min_width_px": 26.7,
+        "battle_max_width_px": 43.6,
+        "battle_min_height_px": 45.7,
+        "battle_max_height_px": 47.1,
+        "status": "ok"
       }
     },
     {

docs/unit-sprite-rework-audit.md

@@ -29,8 +29,8 @@ Visual review boards are generated by `scripts/render-unit-sprite-review-boards.
 
 ## Review Stage Counts
 
-- approved-baseline: 88
-- queued-rework: 3
+- approved-baseline: 89
+- queued-rework: 2
 
 ## Batch Plan
 
@@ -139,7 +139,7 @@ Visual review boards are generated by `scripts/render-unit-sprite-review-boards.
 | batch4-wu | approved-baseline | `unit-wu-strategist-river` | ok | ok | 164..249 x 286..294 | 159..244 x 286..295 | 26.2..39.8 x 45.7..47.0 | w85 h8 b0 |
 | batch4-wu | approved-baseline | `unit-wu-yi` | ok | ok | 153..209 x 288..290 | 147..238 x 282..288 | 24.4..33.4 x 46.0..46.3 | w56 h2 b0 |
 | batch7-unique-officer | approved-baseline | `unit-yan-yan` | ok | ok | 190..235 x 286..288 | 170..242 x 271..295 | 30.4..37.5 x 45.7..46.0 | w45 h2 b0 |
-| batch7-unique-officer | queued-rework | `unit-yi-ji` | border-hits=32 | border-hits=98 | 243..246 x 280..281 | 220..255 x 275..281 | 38.8..39.3 x 44.7..44.9 | w3 h1 b6 |
+| batch7-unique-officer | approved-baseline | `unit-yi-ji` | ok | ok | 174..212 x 286..288 | 167..273 x 286..295 | 27.8..33.9 x 45.7..46.0 | w38 h2 b0 |
 | batch1-approved-core | approved-baseline | `unit-zhang-fei` | ok | ok | 143..306 x 274..289 | 188..286 x 181..290 | 22.8..48.9 x 43.8..46.2 | w163 h15 b14 |
 | batch7-unique-officer | queued-rework | `unit-zhao-yun` | border-hits=64 | border-hits=144 | 166..296 x 282..288 | 166..305 x 274..290 | 26.5..47.3 x 45.0..46.0 | w130 h6 b6 |
 | batch7-unique-officer | queued-rework | `unit-zhuge-liang` | border-hits=32 | border-hits=98 | 243..246 x 280..281 | 220..255 x 275..281 | 38.8..39.3 x 44.7..44.9 | w3 h1 b6 |

docs/unit-sprite-rework-manifest.json

@@ -98,6 +98,7 @@
     "unit-wu-strategist-river",
     "unit-wu-yi",
     "unit-yan-yan",
+    "unit-yi-ji",
     "unit-zhang-fei"
   ],
   "pendingApproval": [],

docs/unit-sprite-yi-ji-v2-approval-packet.md

@@ -0,0 +1,35 @@
+# Yi Ji hand-painted v2 approval packet
+
+- unit: `unit-yi-ji`
+- candidate: `handpaint-batch7-yi-ji-v2`
+- source: `docs/handpaint-batch7-yi-ji-source-v1.png`
+- contact sheet: `docs/handpaint-batch7-yi-ji-v2-contact.png`
+- animation: `docs/handpaint-batch7-yi-ji-v2-animation.gif`
+- before/after: `docs/handpaint-batch7-yi-ji-v2-before-after.png`
+- battle composite: `docs/handpaint-batch7-yi-ji-v2-battle-composite.png`
+- current approval board: `docs/unit-sprite-current-approval-board.png`
+- browser check: `docs/handpaint-batch7-yi-ji-v2-debug-preview-battle-check.png`
+
+## Result
+
+- Local build: `pnpm build` passed.
+- Local browser URL: `http://127.0.0.1:4173/heros_web/?debug&debugSpritePreview=unit-yi-ji&v=yi-ji-v2`
+- Local browser console errors: `0`
+- Local browser 404s: `0`
+- Sample gate: passed.
+
+## Sprite safety
+
+- base partial alpha: 0
+- action partial alpha: 0
+- base frame border hits: 0
+- action frame border hits: 0
+- base battle size: 27.8..33.9 x 45.7..46.0
+- frame bbox width: 174..212 base, 167..273 action
+- frame bbox height: 286..288 base, 286..295 action
+
+## Visual notes
+
+- v1 was attractive but too narrow for the approved visual baseline, so v2 widens the court robe sleeves.
+- Pale jade-green robes, official cap, bamboo slips, and sealed letter make Yi Ji read as a diplomatic civil officer rather than a warrior.
+- The brighter scholar silhouette stays distinct from Ma Liang's white robe and Pang Tong/Sima Yi's darker strategist silhouettes.