diff --git a/docs/handpaint-batch6-nanman-officer-warlord-source-v3.png b/docs/handpaint-batch6-nanman-officer-warlord-source-v3.png new file mode 100644 index 0000000..465177f Binary files /dev/null and b/docs/handpaint-batch6-nanman-officer-warlord-source-v3.png differ diff --git a/docs/handpaint-batch6-nanman-officer-warlord-v3-animation.gif b/docs/handpaint-batch6-nanman-officer-warlord-v3-animation.gif new file mode 100644 index 0000000..9a72474 Binary files /dev/null and b/docs/handpaint-batch6-nanman-officer-warlord-v3-animation.gif differ diff --git a/docs/handpaint-batch6-nanman-officer-warlord-v3-battle-composite.png b/docs/handpaint-batch6-nanman-officer-warlord-v3-battle-composite.png new file mode 100644 index 0000000..ea4d477 Binary files /dev/null and b/docs/handpaint-batch6-nanman-officer-warlord-v3-battle-composite.png differ diff --git a/docs/handpaint-batch6-nanman-officer-warlord-v3-before-after.png b/docs/handpaint-batch6-nanman-officer-warlord-v3-before-after.png new file mode 100644 index 0000000..bc33054 Binary files /dev/null and b/docs/handpaint-batch6-nanman-officer-warlord-v3-before-after.png differ diff --git a/docs/handpaint-batch6-nanman-officer-warlord-v3-contact.png b/docs/handpaint-batch6-nanman-officer-warlord-v3-contact.png new file mode 100644 index 0000000..88e76af Binary files /dev/null and b/docs/handpaint-batch6-nanman-officer-warlord-v3-contact.png differ diff --git a/docs/handpaint-batch6-nanman-officer-warlord-v3-debug-preview-battle-check.png b/docs/handpaint-batch6-nanman-officer-warlord-v3-debug-preview-battle-check.png new file mode 100644 index 0000000..916eff2 Binary files /dev/null and b/docs/handpaint-batch6-nanman-officer-warlord-v3-debug-preview-battle-check.png differ diff --git a/docs/handpaint-batch6-nanman-officer-warlord-v3-report.md b/docs/handpaint-batch6-nanman-officer-warlord-v3-report.md new file mode 100644 index 0000000..e05b8c9 --- /dev/null +++ b/docs/handpaint-batch6-nanman-officer-warlord-v3-report.md @@ -0,0 +1,24 @@ +# Nanman Warlord Officer Hand-Painted v3 + +- source: `handpaint-batch6-nanman-officer-warlord-source-v3.png` +- base status: ok +- action status: ok +- base partial alpha: 0, opaque pixels: 2173656 +- action partial alpha: 0, opaque pixels: 5048896 +- base frame border hits: 0 +- action frame border hits: 0 +- base empty frames: 0 +- action empty frames: 0 +- base bbox width range: 204..234 +- base bbox height range: 284..288 +- base bottom range: 303..303 +- action bbox width range: 173..259 +- action bbox height range: 256..295 +- action bottom range: 303..303 +- contact: `handpaint-batch6-nanman-officer-warlord-v3-contact.png` +- animation: `handpaint-batch6-nanman-officer-warlord-v3-animation.gif` +- before/after: `handpaint-batch6-nanman-officer-warlord-v3-before-after.png` +- scale compare: `handpaint-batch6-nanman-officer-warlord-v3-scale-compare.png` +- battle composite: `handpaint-batch6-nanman-officer-warlord-v3-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 diff --git a/docs/handpaint-batch6-nanman-officer-warlord-v3-scale-compare.png b/docs/handpaint-batch6-nanman-officer-warlord-v3-scale-compare.png new file mode 100644 index 0000000..50205d6 Binary files /dev/null and b/docs/handpaint-batch6-nanman-officer-warlord-v3-scale-compare.png differ diff --git a/docs/unit-sprite-art-quality-gate.md b/docs/unit-sprite-art-quality-gate.md index 24173b8..8997c05 100644 --- a/docs/unit-sprite-art-quality-gate.md +++ b/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: 34 +- Approved baseline units: 35 - Pending approval units: 0 -- Queued rework units: 57 +- Queued rework units: 56 ## Checked Rules diff --git a/docs/unit-sprite-current-approval-board.png b/docs/unit-sprite-current-approval-board.png index 7b27749..eaf2e04 100644 Binary files a/docs/unit-sprite-current-approval-board.png and b/docs/unit-sprite-current-approval-board.png differ diff --git a/docs/unit-sprite-nanman-officer-warlord-v3-approval-packet.md b/docs/unit-sprite-nanman-officer-warlord-v3-approval-packet.md new file mode 100644 index 0000000..07f35ae --- /dev/null +++ b/docs/unit-sprite-nanman-officer-warlord-v3-approval-packet.md @@ -0,0 +1,36 @@ +# Unit Sprite Approval Packet: unit-nanman-officer-warlord v3 + +- Status: approved during continuous production run +- Source art: `docs/handpaint-batch6-nanman-officer-warlord-source-v3.png` +- Contact sheet: `docs/handpaint-batch6-nanman-officer-warlord-v3-contact.png` +- Animation GIF: `docs/handpaint-batch6-nanman-officer-warlord-v3-animation.gif` +- Before/after: `docs/handpaint-batch6-nanman-officer-warlord-v3-before-after.png` +- Scale compare: `docs/handpaint-batch6-nanman-officer-warlord-v3-scale-compare.png` +- Browser check: `docs/handpaint-batch6-nanman-officer-warlord-v3-debug-preview-battle-check.png` +- Current approval board: `docs/unit-sprite-current-approval-board.png` +- Debug preview URL: `http://127.0.0.1:4173/heros_web/?debug&debugSpritePreview=unit-nanman-officer-warlord&v=nanman-officer-warlord-v3-console-clean` + +## Visual Decision + +Nanman warlord officer v3 is accepted as a high-ranking southern commander. v1 had an edge-heavy slash frame that looked too close to cropping, and v2 failed the height-stability quality gate. v3 keeps the stronger horned helm, red cloak, and heavy cleaver silhouette while stabilizing base-frame height for battle-scale readability. + +## Technical Checks + +- Base partial alpha: `0` +- Action partial alpha: `0` +- Base frame border hits: `0` +- Action frame border hits: `0` +- Base empty frames: `0` +- Action empty frames: `0` +- Base bbox width range: `204..234` +- Base bbox height range: `284..288` +- Base bottom range: `303..303` +- Action bbox width range: `173..259` +- Action bbox height range: `256..295` +- Action bottom range: `303..303` +- Local build: `passed` +- Local browser console errors: `0` + +## Notes + +The warlord reads clearly at desktop battle scale through a horned helm, red cloak mass, and heavy cleaver silhouette. It stays opaque, avoids frame-edge contact, and does not rely on external bases, rings, or badges for readability. diff --git a/docs/unit-sprite-next-steps.md b/docs/unit-sprite-next-steps.md index 38ed352..bad224b 100644 --- a/docs/unit-sprite-next-steps.md +++ b/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: 34 +- Approved baseline units: 35 - Pending approval units: 0 -- Queued rework units: 57 +- Queued rework units: 56 - Manifest: `docs/unit-sprite-rework-manifest.json` - Audit: `docs/unit-sprite-rework-audit.json` - Current approval board: not generated until the next pending sample. @@ -14,10 +14,9 @@ This file is generated by `scripts/write-unit-sprite-next-steps.py`. ## Next Unit Queue -1. `unit-nanman-officer-warlord` - queued-rework, base `border-hits=64`, actions `border-hits=144` -2. `unit-nanman-shaman` - queued-rework, base `border-hits=32`, actions `border-hits=98` -3. `unit-nanman-shaman-ember` - queued-rework, base `border-hits=32`, actions `border-hits=98` -4. `unit-nanman-shaman-storm` - queued-rework, base `border-hits=32`, actions `border-hits=98` +1. `unit-nanman-shaman` - queued-rework, base `border-hits=32`, actions `border-hits=98` +2. `unit-nanman-shaman-ember` - queued-rework, base `border-hits=32`, actions `border-hits=98` +3. `unit-nanman-shaman-storm` - queued-rework, base `border-hits=32`, actions `border-hits=98` ## Commands Before Approval @@ -44,5 +43,5 @@ pnpm build ## First Task After Current Approval -- Start with `unit-nanman-officer-warlord`. +- Start with `unit-nanman-shaman`. - Generate source art, assemble sheets, produce contact/GIF/before-after/scale/battle screenshots, then rerun the gates. diff --git a/docs/unit-sprite-review-approved-and-pending.png b/docs/unit-sprite-review-approved-and-pending.png index 22db54c..4f1fb7f 100644 Binary files a/docs/unit-sprite-review-approved-and-pending.png and b/docs/unit-sprite-review-approved-and-pending.png differ diff --git a/docs/unit-sprite-review-batch6-nanman-special.png b/docs/unit-sprite-review-batch6-nanman-special.png index 43a6b47..b80f2c8 100644 Binary files a/docs/unit-sprite-review-batch6-nanman-special.png and b/docs/unit-sprite-review-batch6-nanman-special.png differ diff --git a/docs/unit-sprite-review-index.png b/docs/unit-sprite-review-index.png index d31232a..482ba85 100644 Binary files a/docs/unit-sprite-review-index.png and b/docs/unit-sprite-review-index.png differ diff --git a/docs/unit-sprite-rework-audit.json b/docs/unit-sprite-rework-audit.json index 21a95c9..e7379d5 100644 --- a/docs/unit-sprite-rework-audit.json +++ b/docs/unit-sprite-rework-audit.json @@ -11,8 +11,8 @@ "batch7-unique-officer": 19 }, "stage_counts": { - "approved-baseline": 34, - "queued-rework": 57 + "approved-baseline": 35, + "queued-rework": 56 }, "rows": [ { @@ -1157,7 +1157,7 @@ }, { "batch": "batch6-nanman-special", - "review_stage": "queued-rework", + "review_stage": "approved-baseline", "stem": "unit-nanman-officer-warlord", "base": { "key": "unit-nanman-officer-warlord", @@ -1167,23 +1167,23 @@ 1252 ], "partial_alpha": 0, - "opaque_pixels": 2280384, - "border_hits": 64, - "min_width": 224, - "max_width": 266, - "min_height": 282, + "opaque_pixels": 2173656, + "border_hits": 0, + "min_width": 204, + "max_width": 234, + "min_height": 284, "max_height": 288, - "min_bottom": 281, - "max_bottom": 287, + "min_bottom": 303, + "max_bottom": 303, "empty_frames": 0, - "width_swing": 42, - "height_swing": 6, - "bottom_swing": 6, - "battle_min_width_px": 35.8, - "battle_max_width_px": 42.5, - "battle_min_height_px": 45.0, + "width_swing": 30, + "height_swing": 4, + "bottom_swing": 0, + "battle_min_width_px": 32.6, + "battle_max_width_px": 37.4, + "battle_min_height_px": 45.4, "battle_max_height_px": 46.0, - "status": "border-hits=64" + "status": "ok" }, "actions": { "key": "unit-nanman-officer-warlord-actions", @@ -1193,23 +1193,23 @@ 1252 ], "partial_alpha": 0, - "opaque_pixels": 5101401, - "border_hits": 144, - "min_width": 224, - "max_width": 275, - "min_height": 275, - "max_height": 290, - "min_bottom": 274, - "max_bottom": 289, + "opaque_pixels": 5048896, + "border_hits": 0, + "min_width": 173, + "max_width": 259, + "min_height": 256, + "max_height": 295, + "min_bottom": 303, + "max_bottom": 303, "empty_frames": 0, - "width_swing": 51, - "height_swing": 15, - "bottom_swing": 15, - "battle_min_width_px": 35.8, - "battle_max_width_px": 43.9, - "battle_min_height_px": 43.9, - "battle_max_height_px": 46.3, - "status": "border-hits=144" + "width_swing": 86, + "height_swing": 39, + "bottom_swing": 0, + "battle_min_width_px": 27.6, + "battle_max_width_px": 41.4, + "battle_min_height_px": 40.9, + "battle_max_height_px": 47.1, + "status": "ok" } }, { diff --git a/docs/unit-sprite-rework-audit.md b/docs/unit-sprite-rework-audit.md index 3e6480c..a8da4b6 100644 --- a/docs/unit-sprite-rework-audit.md +++ b/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: 34 -- queued-rework: 57 +- approved-baseline: 35 +- queued-rework: 56 ## Batch Plan @@ -72,7 +72,7 @@ Visual review boards are generated by `scripts/render-unit-sprite-review-boards. | batch6-nanman-special | approved-baseline | `unit-nanman-infantry-spear` | ok | ok | 138..219 x 286..288 | 137..263 x 277..289 | 22.0..35.0 x 45.7..46.0 | w81 h2 b0 | | batch6-nanman-special | approved-baseline | `unit-nanman-officer` | ok | ok | 163..215 x 288..288 | 159..238 x 280..288 | 26.0..34.3 x 46.0..46.0 | w52 h0 b0 | | batch6-nanman-special | approved-baseline | `unit-nanman-officer-bronze` | ok | ok | 177..214 x 284..288 | 155..264 x 238..295 | 28.3..34.2 x 45.4..46.0 | w37 h4 b0 | -| batch6-nanman-special | queued-rework | `unit-nanman-officer-warlord` | border-hits=64 | border-hits=144 | 224..266 x 282..288 | 224..275 x 275..290 | 35.8..42.5 x 45.0..46.0 | w42 h6 b6 | +| batch6-nanman-special | approved-baseline | `unit-nanman-officer-warlord` | ok | ok | 204..234 x 284..288 | 173..259 x 256..295 | 32.6..37.4 x 45.4..46.0 | w30 h4 b0 | | batch6-nanman-special | queued-rework | `unit-nanman-shaman` | 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 | | batch6-nanman-special | queued-rework | `unit-nanman-shaman-ember` | 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 | | batch6-nanman-special | queued-rework | `unit-nanman-shaman-storm` | 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 | diff --git a/docs/unit-sprite-rework-manifest.json b/docs/unit-sprite-rework-manifest.json index 02baa11..b823b79 100644 --- a/docs/unit-sprite-rework-manifest.json +++ b/docs/unit-sprite-rework-manifest.json @@ -19,6 +19,7 @@ "unit-nanman-infantry-spear", "unit-nanman-officer", "unit-nanman-officer-bronze", + "unit-nanman-officer-warlord", "unit-rebel", "unit-rebel-archer", "unit-rebel-archer-fire", @@ -49,7 +50,6 @@ "pendingApproval": [], "currentApprovalCandidate": {}, "nextAfterApproval": [ - "unit-nanman-officer-warlord", "unit-nanman-shaman", "unit-nanman-shaman-ember", "unit-nanman-shaman-storm" diff --git a/src/assets/images/units/unit-nanman-officer-warlord-actions.png b/src/assets/images/units/unit-nanman-officer-warlord-actions.png index 4b1a639..1f14398 100644 Binary files a/src/assets/images/units/unit-nanman-officer-warlord-actions.png and b/src/assets/images/units/unit-nanman-officer-warlord-actions.png differ diff --git a/src/assets/images/units/unit-nanman-officer-warlord.png b/src/assets/images/units/unit-nanman-officer-warlord.png index a1c1fe5..2721dcb 100644 Binary files a/src/assets/images/units/unit-nanman-officer-warlord.png and b/src/assets/images/units/unit-nanman-officer-warlord.png differ