diff --git a/docs/handpaint-batch5-rebel-archer-fire-source-v1.png b/docs/handpaint-batch5-rebel-archer-fire-source-v1.png new file mode 100644 index 0000000..0e46c5a Binary files /dev/null and b/docs/handpaint-batch5-rebel-archer-fire-source-v1.png differ diff --git a/docs/handpaint-batch5-rebel-archer-fire-v1-animation.gif b/docs/handpaint-batch5-rebel-archer-fire-v1-animation.gif new file mode 100644 index 0000000..62dd453 Binary files /dev/null and b/docs/handpaint-batch5-rebel-archer-fire-v1-animation.gif differ diff --git a/docs/handpaint-batch5-rebel-archer-fire-v1-battle-composite.png b/docs/handpaint-batch5-rebel-archer-fire-v1-battle-composite.png new file mode 100644 index 0000000..c945e22 Binary files /dev/null and b/docs/handpaint-batch5-rebel-archer-fire-v1-battle-composite.png differ diff --git a/docs/handpaint-batch5-rebel-archer-fire-v1-before-after.png b/docs/handpaint-batch5-rebel-archer-fire-v1-before-after.png new file mode 100644 index 0000000..6af35ed Binary files /dev/null and b/docs/handpaint-batch5-rebel-archer-fire-v1-before-after.png differ diff --git a/docs/handpaint-batch5-rebel-archer-fire-v1-contact.png b/docs/handpaint-batch5-rebel-archer-fire-v1-contact.png new file mode 100644 index 0000000..853c501 Binary files /dev/null and b/docs/handpaint-batch5-rebel-archer-fire-v1-contact.png differ diff --git a/docs/handpaint-batch5-rebel-archer-fire-v1-debug-preview-battle-check.png b/docs/handpaint-batch5-rebel-archer-fire-v1-debug-preview-battle-check.png new file mode 100644 index 0000000..e970289 Binary files /dev/null and b/docs/handpaint-batch5-rebel-archer-fire-v1-debug-preview-battle-check.png differ diff --git a/docs/handpaint-batch5-rebel-archer-fire-v1-report.md b/docs/handpaint-batch5-rebel-archer-fire-v1-report.md new file mode 100644 index 0000000..f0bdde9 --- /dev/null +++ b/docs/handpaint-batch5-rebel-archer-fire-v1-report.md @@ -0,0 +1,24 @@ +# Rebel Fire Archer Hand-Painted v1 + +- source: `handpaint-batch5-rebel-archer-fire-source-v1.png` +- base status: ok +- action status: ok +- base partial alpha: 0, opaque pixels: 1860546 +- action partial alpha: 0, opaque pixels: 4643712 +- base frame border hits: 0 +- action frame border hits: 0 +- base empty frames: 0 +- action empty frames: 0 +- base bbox width range: 161..249 +- base bbox height range: 288..295 +- base bottom range: 303..303 +- action bbox width range: 161..252 +- action bbox height range: 285..295 +- action bottom range: 303..303 +- contact: `handpaint-batch5-rebel-archer-fire-v1-contact.png` +- animation: `handpaint-batch5-rebel-archer-fire-v1-animation.gif` +- before/after: `handpaint-batch5-rebel-archer-fire-v1-before-after.png` +- scale compare: `handpaint-batch5-rebel-archer-fire-v1-scale-compare.png` +- battle composite: `handpaint-batch5-rebel-archer-fire-v1-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-batch5-rebel-archer-fire-v1-scale-compare.png b/docs/handpaint-batch5-rebel-archer-fire-v1-scale-compare.png new file mode 100644 index 0000000..cf99c46 Binary files /dev/null and b/docs/handpaint-batch5-rebel-archer-fire-v1-scale-compare.png differ diff --git a/docs/unit-sprite-art-quality-gate.md b/docs/unit-sprite-art-quality-gate.md index 3c7b39e..a4f05d5 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: 22 +- Approved baseline units: 23 - Pending approval units: 0 -- Queued rework units: 69 +- Queued rework units: 68 ## Checked Rules diff --git a/docs/unit-sprite-current-approval-board.png b/docs/unit-sprite-current-approval-board.png index 6c61583..3aa65ee 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-next-steps.md b/docs/unit-sprite-next-steps.md index e91fbb1..e873f7d 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: 22 +- Approved baseline units: 23 - Pending approval units: 0 -- Queued rework units: 69 +- Queued rework units: 68 - 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,7 +14,11 @@ This file is generated by `scripts/write-unit-sprite-next-steps.py`. ## Next Unit Queue -No queued next unit is defined in the manifest. +1. `unit-rebel-archer-veteran` - queued-rework, base `border-hits=64`, actions `border-hits=140` +2. `unit-rebel-cavalry-veteran` - queued-rework, base `border-hits=64`, actions `border-hits=144` +3. `unit-rebel-leader` - queued-rework, base `border-hits=32`, actions `border-hits=72` +4. `unit-rebel-leader-warlord` - queued-rework, base `border-hits=32`, actions `border-hits=78` +5. `unit-rebel-ragged` - queued-rework, base `border-hits=32`, actions `border-hits=72` ## Commands Before Approval @@ -41,4 +45,5 @@ pnpm build ## First Task After Current Approval -- No first task is defined. +- Start with `unit-rebel-archer-veteran`. +- Generate source art, assemble sheets, produce contact/GIF/before-after/scale/battle screenshots, then rerun the gates. diff --git a/docs/unit-sprite-rebel-archer-fire-v1-approval-packet.md b/docs/unit-sprite-rebel-archer-fire-v1-approval-packet.md new file mode 100644 index 0000000..f4dd521 --- /dev/null +++ b/docs/unit-sprite-rebel-archer-fire-v1-approval-packet.md @@ -0,0 +1,44 @@ +# Rebel Fire Archer v1 Approval Packet + +- unit: `unit-rebel-archer-fire` +- version: `v1` +- source: `docs/handpaint-batch5-rebel-archer-fire-source-v1.png` +- contact sheet: `docs/handpaint-batch5-rebel-archer-fire-v1-contact.png` +- animation: `docs/handpaint-batch5-rebel-archer-fire-v1-animation.gif` +- battle preview: `docs/handpaint-batch5-rebel-archer-fire-v1-debug-preview-battle-check.png` +- before/after: `docs/handpaint-batch5-rebel-archer-fire-v1-before-after.png` +- scale compare: `docs/handpaint-batch5-rebel-archer-fire-v1-scale-compare.png` +- current approval board: `docs/unit-sprite-current-approval-board.png` + +## Visual Decision + +Approved for the continuous production run. + +- 50px battle readability: pass +- Yellow Turban variant identity: pass +- archer bow/quiver/flaming-arrow silhouette: pass +- avoids cavalry, spear, or unique-hero read: pass +- opaque unit body: pass +- frame boundary safety: pass +- idle/action scale stability: pass + +## Technical Checks + +- local browser URL: `http://127.0.0.1:4173/heros_web/?debug&debugSpritePreview=unit-rebel-archer-fire&v=rebel-archer-fire-v1-final` +- Local browser console errors: `0` +- 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: `161..249` +- base bbox height range: `288..295` +- base bottom range: `303..303` +- action bbox width range: `161..252` +- action bbox height range: `285..295` +- action bottom range: `303..303` + +## Notes + +The v1 sheet keeps the approved Yellow Turban ragged archer read while making the fire-arrow role visible through small flame tips and torch/item poses without adding oversized magic effects. diff --git a/docs/unit-sprite-review-approved-and-pending.png b/docs/unit-sprite-review-approved-and-pending.png index 5aff3d1..bc63a1c 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-batch5-rebel-other.png b/docs/unit-sprite-review-batch5-rebel-other.png index de611bc..0e7c525 100644 Binary files a/docs/unit-sprite-review-batch5-rebel-other.png and b/docs/unit-sprite-review-batch5-rebel-other.png differ diff --git a/docs/unit-sprite-review-index.png b/docs/unit-sprite-review-index.png index 8f5774c..ddab419 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 01a6ce7..dab4db5 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": 22, - "queued-rework": 69 + "approved-baseline": 23, + "queued-rework": 68 }, "rows": [ { @@ -1556,7 +1556,7 @@ }, { "batch": "batch5-rebel-other", - "review_stage": "queued-rework", + "review_stage": "approved-baseline", "stem": "unit-rebel-archer-fire", "base": { "key": "unit-rebel-archer-fire", @@ -1566,23 +1566,23 @@ 1252 ], "partial_alpha": 0, - "opaque_pixels": 1934290, - "border_hits": 64, - "min_width": 245, - "max_width": 246, - "min_height": 266, - "max_height": 266, - "min_bottom": 281, - "max_bottom": 287, + "opaque_pixels": 1860546, + "border_hits": 0, + "min_width": 161, + "max_width": 249, + "min_height": 288, + "max_height": 295, + "min_bottom": 303, + "max_bottom": 303, "empty_frames": 0, - "width_swing": 1, - "height_swing": 0, - "bottom_swing": 6, - "battle_min_width_px": 39.1, - "battle_max_width_px": 39.3, - "battle_min_height_px": 42.5, - "battle_max_height_px": 42.5, - "status": "border-hits=64" + "width_swing": 88, + "height_swing": 7, + "bottom_swing": 0, + "battle_min_width_px": 25.7, + "battle_max_width_px": 39.8, + "battle_min_height_px": 46.0, + "battle_max_height_px": 47.1, + "status": "ok" }, "actions": { "key": "unit-rebel-archer-fire-actions", @@ -1592,23 +1592,23 @@ 1252 ], "partial_alpha": 0, - "opaque_pixels": 4315467, - "border_hits": 140, - "min_width": 220, - "max_width": 255, - "min_height": 265, - "max_height": 272, - "min_bottom": 274, - "max_bottom": 289, + "opaque_pixels": 4643712, + "border_hits": 0, + "min_width": 161, + "max_width": 252, + "min_height": 285, + "max_height": 295, + "min_bottom": 303, + "max_bottom": 303, "empty_frames": 0, - "width_swing": 35, - "height_swing": 7, - "bottom_swing": 15, - "battle_min_width_px": 35.1, - "battle_max_width_px": 40.7, - "battle_min_height_px": 42.3, - "battle_max_height_px": 43.5, - "status": "border-hits=140" + "width_swing": 91, + "height_swing": 10, + "bottom_swing": 0, + "battle_min_width_px": 25.7, + "battle_max_width_px": 40.3, + "battle_min_height_px": 45.5, + "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 eeaf0bc..efa5746 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: 22 -- queued-rework: 69 +- approved-baseline: 23 +- queued-rework: 68 ## Batch Plan @@ -79,7 +79,7 @@ Visual review boards are generated by `scripts/render-unit-sprite-review-boards. | batch7-unique-officer | queued-rework | `unit-pang-tong` | 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 | | batch1-approved-rebel | approved-baseline | `unit-rebel` | ok | ok | 195..284 x 271..281 | 180..286 x 178..289 | 31.2..45.4 x 43.3..44.9 | w89 h10 b8 | | batch1-approved-rebel | approved-baseline | `unit-rebel-archer` | ok | ok | 185..263 x 272..278 | 116..284 x 228..288 | 29.6..42.0 x 43.5..44.4 | w78 h6 b1 | -| batch5-rebel-other | queued-rework | `unit-rebel-archer-fire` | border-hits=64 | border-hits=140 | 245..246 x 266..266 | 220..255 x 265..272 | 39.1..39.3 x 42.5..42.5 | w1 h0 b6 | +| batch5-rebel-other | approved-baseline | `unit-rebel-archer-fire` | ok | ok | 161..249 x 288..295 | 161..252 x 285..295 | 25.7..39.8 x 46.0..47.1 | w88 h7 b0 | | batch5-rebel-other | queued-rework | `unit-rebel-archer-veteran` | border-hits=64 | border-hits=140 | 249..250 x 276..277 | 223..259 x 275..278 | 39.8..39.9 x 44.1..44.2 | w1 h1 b6 | | batch1-approved-rebel | approved-baseline | `unit-rebel-cavalry` | ok | ok | 139..289 x 266..281 | 167..286 x 186..282 | 22.2..46.2 x 42.5..44.9 | w150 h15 b3 | | batch5-rebel-other | queued-rework | `unit-rebel-cavalry-veteran` | 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 | diff --git a/docs/unit-sprite-rework-manifest.json b/docs/unit-sprite-rework-manifest.json index 6abcbac..380cbc7 100644 --- a/docs/unit-sprite-rework-manifest.json +++ b/docs/unit-sprite-rework-manifest.json @@ -15,6 +15,7 @@ "unit-liu-bei", "unit-rebel", "unit-rebel-archer", + "unit-rebel-archer-fire", "unit-rebel-cavalry", "unit-shu-archer", "unit-shu-cavalry", @@ -36,7 +37,13 @@ ], "pendingApproval": [], "currentApprovalCandidate": {}, - "nextAfterApproval": [], + "nextAfterApproval": [ + "unit-rebel-archer-veteran", + "unit-rebel-cavalry-veteran", + "unit-rebel-leader", + "unit-rebel-leader-warlord", + "unit-rebel-ragged" + ], "reviewStages": { "approved-baseline": "Approved hand-painted reference or approved completed unit.", "pending-approval": "Prepared sample waiting for user approval; do not expand, commit, push, or deploy.", diff --git a/src/assets/images/units/unit-rebel-archer-fire-actions.png b/src/assets/images/units/unit-rebel-archer-fire-actions.png index cc80c7b..9f2f1e8 100644 Binary files a/src/assets/images/units/unit-rebel-archer-fire-actions.png and b/src/assets/images/units/unit-rebel-archer-fire-actions.png differ diff --git a/src/assets/images/units/unit-rebel-archer-fire.png b/src/assets/images/units/unit-rebel-archer-fire.png index 12db59d..9869d69 100644 Binary files a/src/assets/images/units/unit-rebel-archer-fire.png and b/src/assets/images/units/unit-rebel-archer-fire.png differ