Approve Li Yan handpainted sprite v2

docs/handpaint-batch7-li-yan-v2-report.md

@@ -0,0 +1,24 @@
+# Li Yan hand-painted v2 report
+
+- source: `handpaint-batch7-li-yan-source-v1.png`
+- base status: ok
+- action status: ok
+- base partial alpha: 0, opaque pixels: 2181374
+- action partial alpha: 0, opaque pixels: 5081530
+- base frame border hits: 0
+- action frame border hits: 0
+- base empty frames: 0
+- action empty frames: 0
+- base bbox width range: 170..262
+- base bbox height range: 286..290
+- base bottom range: 303..303
+- action bbox width range: 190..259
+- action bbox height range: 286..290
+- action bottom range: 303..303
+- contact: `handpaint-batch7-li-yan-v2-contact.png`
+- animation: `handpaint-batch7-li-yan-v2-animation.gif`
+- before/after: `handpaint-batch7-li-yan-v2-before-after.png`
+- scale compare: `handpaint-batch7-li-yan-v2-scale-compare.png`
+- battle composite: `handpaint-batch7-li-yan-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: 78
+- Approved baseline units: 79
 - Pending approval units: 0
-- Queued rework units: 13
+- Queued rework units: 12
 
 ## Checked Rules
 

docs/unit-sprite-batch-queue.json

@@ -10,6 +10,7 @@
     "unit-huang-quan",
     "unit-huang-zhong",
     "unit-jiang-wei",
+    "unit-li-yan",
     "unit-liu-bei",
     "unit-meng-huo",
     "unit-nanman-infantry",
@@ -85,8 +86,8 @@
   "pendingApproval": [],
   "nextAfterApproval": [],
   "stageCounts": {
-    "approved-baseline": 78,
-    "queued-rework": 13
+    "approved-baseline": 79,
+    "queued-rework": 12
   },
   "batches": [
     {
@@ -1172,24 +1173,10 @@
       "label": "Batch 7 unique officers",
       "total": 19,
       "stageCounts": {
-        "approved-baseline": 6,
-        "queued-rework": 13
+        "approved-baseline": 7,
+        "queued-rework": 12
       },
       "units": [
-        {
-          "stem": "unit-li-yan",
-          "reviewStage": "queued-rework",
-          "queueRole": "queued rework",
-          "baseStatus": "border-hits=32",
-          "actionStatus": "border-hits=82",
-          "battleBasePx": [
-            31.9,
-            40.6,
-            44.6,
-            44.6
-          ],
-          "spec": null
-        },
         {
           "stem": "unit-lu-bu",
           "reviewStage": "queued-rework",
@@ -1441,6 +1428,20 @@
             46.5
           ],
           "spec": null
+        },
+        {
+          "stem": "unit-li-yan",
+          "reviewStage": "approved-baseline",
+          "queueRole": "approved reference",
+          "baseStatus": "ok",
+          "actionStatus": "ok",
+          "battleBasePx": [
+            27.2,
+            41.9,
+            45.7,
+            46.3
+          ],
+          "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: 78
+- Approved baseline units: 79
 - Pending approval units: 0
-- Queued rework units: 13
+- Queued rework units: 12
 - 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 6, queued-rework 13
+- Stage counts: approved-baseline 7, queued-rework 12
 
 | unit | role | stage | status | battle base px | spec |
 | --- | --- | --- | --- | --- | --- |
-| `unit-li-yan` | queued rework | queued-rework | base border-hits=32; actions border-hits=82 | 31.9..40.6 x 44.6..44.6 |  |
 | `unit-lu-bu` | queued rework | queued-rework | base border-hits=64; actions border-hits=144 | 28.8..47.3 x 45.0..46.0 |  |
 | `unit-lu-meng` | queued rework | queued-rework | base border-hits=32; actions border-hits=82 | 30.7..44.2 x 44.6..44.6 |  |
 | `unit-ma-chao` | queued rework | queued-rework | base border-hits=64; actions border-hits=144 | 26.5..47.3 x 45.0..46.0 |  |
@@ -180,3 +179,4 @@ No next unit is currently defined.
 | `unit-huang-quan` | approved reference | approved-baseline | technical ok | 28.9..34.8 x 46.3..46.3 |  |
 | `unit-huang-zhong` | approved reference | approved-baseline | technical ok | 27.2..36.3 x 46.3..47.3 |  |
 | `unit-jiang-wei` | approved reference | approved-baseline | technical ok | 23.2..42.5 x 45.4..46.5 |  |
+| `unit-li-yan` | approved reference | approved-baseline | technical ok | 27.2..41.9 x 45.7..46.3 |  |

docs/unit-sprite-li-yan-v2-approval-packet.md

@@ -0,0 +1,36 @@
+# Li Yan hand-painted sprite v2 approval packet
+
+- Unit: `unit-li-yan`
+- Candidate version: `v2`
+- Source sheet: `docs/handpaint-batch7-li-yan-source-v1.png`
+- Contact sheet: `docs/handpaint-batch7-li-yan-v2-contact.png`
+- Animation GIF: `docs/handpaint-batch7-li-yan-v2-animation.gif`
+- Before/after: `docs/handpaint-batch7-li-yan-v2-before-after.png`
+- Battle composite: `docs/handpaint-batch7-li-yan-v2-battle-composite.png`
+- Scale compare: `docs/handpaint-batch7-li-yan-v2-scale-compare.png`
+- Local browser check: `docs/handpaint-batch7-li-yan-v2-debug-preview-battle-check.png`
+- Current approval board: `docs/unit-sprite-current-approval-board.png`
+
+## Visual Decision
+
+- Approved for continuous production run.
+- Direction: stern Yi Province gate commander and supply officer, with deep teal/bronze armor, broad sword, squared shoulders, and command-ledger details.
+- The 100% PC browser debug preview reads as a heavy defensive commander, distinct from Jiang Wei's silver-blue spear silhouette and the older generic soldier.
+- The unit body is opaque and readable at battle scale without base rings, completion badges, or other external readability helpers.
+
+## Technical Checks
+
+- Local build: `pnpm build` passed.
+- Local browser URL: `http://127.0.0.1:4173/heros_web/?debug&debugSpritePreview=unit-li-yan&v=li-yan-v2`
+- Local browser console errors: `0`
+- Local browser 404s: `0`
+- Base partial alpha: `0`
+- Action partial alpha: `0`
+- Base frame border hits: `0`
+- Action frame border hits: `0`
+- Base battle size from audit: `27.2..41.9 x 45.7..46.3`
+
+## Notes
+
+- v1 had strong art direction, but the action battle-width metric was just below the gate.
+- v2 restores the sword/action silhouette while preserving frame boundary safety and stable bottom alignment.

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: 78
+- Approved baseline units: 79
 - Pending approval units: 0
-- Queued rework units: 13
+- Queued rework units: 12
 - 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": 78,
-    "queued-rework": 13
+    "approved-baseline": 79,
+    "queued-rework": 12
   },
   "rows": [
     {
@@ -416,7 +416,7 @@
     },
     {
       "batch": "batch7-unique-officer",
-      "review_stage": "queued-rework",
+      "review_stage": "approved-baseline",
       "stem": "unit-li-yan",
       "base": {
         "key": "unit-li-yan",
@@ -426,23 +426,23 @@
           1252
         ],
         "partial_alpha": 0,
-        "opaque_pixels": 1985543,
-        "border_hits": 32,
-        "min_width": 200,
-        "max_width": 254,
-        "min_height": 279,
-        "max_height": 279,
-        "min_bottom": 281,
-        "max_bottom": 287,
+        "opaque_pixels": 2181374,
+        "border_hits": 0,
+        "min_width": 170,
+        "max_width": 262,
+        "min_height": 286,
+        "max_height": 290,
+        "min_bottom": 303,
+        "max_bottom": 303,
         "empty_frames": 0,
-        "width_swing": 54,
-        "height_swing": 0,
-        "bottom_swing": 6,
-        "battle_min_width_px": 31.9,
-        "battle_max_width_px": 40.6,
-        "battle_min_height_px": 44.6,
-        "battle_max_height_px": 44.6,
-        "status": "border-hits=32"
+        "width_swing": 92,
+        "height_swing": 4,
+        "bottom_swing": 0,
+        "battle_min_width_px": 27.2,
+        "battle_max_width_px": 41.9,
+        "battle_min_height_px": 45.7,
+        "battle_max_height_px": 46.3,
+        "status": "ok"
       },
       "actions": {
         "key": "unit-li-yan-actions",
@@ -452,23 +452,23 @@
           1252
         ],
         "partial_alpha": 0,
-        "opaque_pixels": 4453784,
-        "border_hits": 82,
-        "min_width": 200,
-        "max_width": 263,
-        "min_height": 275,
-        "max_height": 279,
-        "min_bottom": 274,
-        "max_bottom": 289,
+        "opaque_pixels": 5081530,
+        "border_hits": 0,
+        "min_width": 190,
+        "max_width": 259,
+        "min_height": 286,
+        "max_height": 290,
+        "min_bottom": 303,
+        "max_bottom": 303,
         "empty_frames": 0,
-        "width_swing": 63,
+        "width_swing": 69,
         "height_swing": 4,
-        "bottom_swing": 15,
-        "battle_min_width_px": 31.9,
-        "battle_max_width_px": 42.0,
-        "battle_min_height_px": 43.9,
-        "battle_max_height_px": 44.6,
-        "status": "border-hits=82"
+        "bottom_swing": 0,
+        "battle_min_width_px": 30.4,
+        "battle_max_width_px": 41.4,
+        "battle_min_height_px": 45.7,
+        "battle_max_height_px": 46.3,
+        "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: 78
-- queued-rework: 13
+- approved-baseline: 79
+- queued-rework: 12
 
 ## Batch Plan
 
@@ -59,7 +59,7 @@ Visual review boards are generated by `scripts/render-unit-sprite-review-boards.
 | batch7-unique-officer | approved-baseline | `unit-huang-quan` | ok | ok | 181..218 x 290..290 | 176..251 x 284..290 | 28.9..34.8 x 46.3..46.3 | w37 h0 b0 |
 | batch7-unique-officer | approved-baseline | `unit-huang-zhong` | ok | ok | 170..227 x 290..296 | 170..246 x 270..296 | 27.2..36.3 x 46.3..47.3 | w57 h6 b0 |
 | batch7-unique-officer | approved-baseline | `unit-jiang-wei` | ok | ok | 145..266 x 284..291 | 142..261 x 261..291 | 23.2..42.5 x 45.4..46.5 | w121 h7 b0 |
-| batch7-unique-officer | queued-rework | `unit-li-yan` | border-hits=32 | border-hits=82 | 200..254 x 279..279 | 200..263 x 275..279 | 31.9..40.6 x 44.6..44.6 | w54 h0 b6 |
+| batch7-unique-officer | approved-baseline | `unit-li-yan` | ok | ok | 170..262 x 286..290 | 190..259 x 286..290 | 27.2..41.9 x 45.7..46.3 | w92 h4 b0 |
 | batch1-approved-core | approved-baseline | `unit-liu-bei` | ok | ok | 138..258 x 278..289 | 148..300 x 252..284 | 22.0..41.2 x 44.4..46.2 | w120 h11 b12 |
 | batch7-unique-officer | queued-rework | `unit-lu-bu` | border-hits=64 | border-hits=144 | 180..296 x 282..288 | 180..305 x 274..290 | 28.8..47.3 x 45.0..46.0 | w116 h6 b6 |
 | batch7-unique-officer | queued-rework | `unit-lu-meng` | border-hits=32 | border-hits=82 | 192..277 x 279..279 | 192..286 x 275..279 | 30.7..44.2 x 44.6..44.6 | w85 h0 b6 |

docs/unit-sprite-rework-manifest.json

@@ -18,6 +18,7 @@
     "unit-huang-quan",
     "unit-huang-zhong",
     "unit-jiang-wei",
+    "unit-li-yan",
     "unit-liu-bei",
     "unit-meng-huo",
     "unit-nanman-infantry",