Skip to content

feat(hands): overhaul fighter animation, gore, and injury audio - #288

Merged
user1303836 merged 1 commit into
mainfrom
fix/hands-animation-gore
Aug 7, 2026
Merged

feat(hands): overhaul fighter animation, gore, and injury audio#288
user1303836 merged 1 commit into
mainfrom
fix/hands-animation-gore

Conversation

@user1303836

Copy link
Copy Markdown
Owner

Summary

  • rebuild the embedded boxer asset with 34 anatomy-checked clips, direction-aware gait, normalized blending, authored state transitions, and contact-synchronized reactions
  • add deterministic full/reduced/off blood presentation with fixed-step particles, anatomical hit routing, dislocations, severed heads/hands, pooled stumps/gibs, and restoration
  • generate six recording-free NumPy FM injury voices and route them through gesture-safe, lifecycle-safe Web Audio only when the matching arcade injury applies
  • gzip the embedded GLB at generation time so the production activity remains within its exact three-file bundle budget

Validation

  • uv run ruff check .
  • uv run ruff format --check .
  • uv run mypy src/
  • uv run pytest tests/ — 1,891 passed
  • cd web/hands && npm run typecheck
  • cd web/hands && npm test -- --run — 19 files / 231 tests passed
  • cd web/hands && npm run build — exact three-file static scan passed; hands.js is 4,703,153 bytes / 3,071,090 bytes gzip
  • fighter and injury-audio generators reproduced byte-identical assets
  • Chromium Model Lab visual pass covered all clips; production GLB loading and all six WAV decodes were verified

Review

Independent final review approved the latest tree with no remaining actionable findings.

Only HANDS paths changed. No files were deleted, and no live vector or SQLite databases were accessed.

@greptile-apps greptile-apps Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Your trial has ended. Reactivate Greptile to resume code reviews.

@user1303836
user1303836 merged commit 992a4f5 into main Aug 7, 2026
6 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant