Skip to content

Polish tetris_matrix with reactive combo visuals#351

Open
smcga wants to merge 1 commit intomainfrom
codex/enhance-tetris-effect-visuals
Open

Polish tetris_matrix with reactive combo visuals#351
smcga wants to merge 1 commit intomainfrom
codex/enhance-tetris-effect-visuals

Conversation

@smcga
Copy link
Copy Markdown
Owner

@smcga smcga commented Apr 8, 2026

Motivation

  • Make the tetris_matrix effect feel more impressive and responsive by adding combo-driven visual energy and subtle motion without changing gameplay/timeline behavior.
  • Keep parameter surface and defaults compatible (speed, level, glow, contrast, ghost, seed) while improving visual feedback tied to audio/line clears.

Description

  • Add a compact energy model and helper getRecentLineBursts and compute comboPulse, burstEnergy, and intensity to drive visuals.
  • Introduce subtle screen shake via ctx.translate(shakeX, shakeY) and strengthen reactive screen glow by folding comboPulse and intensity into screenGlow.
  • Add row-pressure highlights, active-piece trail accents, and a HUD status lamp (HOT/SYNC/CALM) driven by comboPulse and audio.beat.
  • Add a vignette pass and scanline/tint tweaks to increase perceived impact; update src/renderer/effects/tetrisMatrixEffect.ts, update effect manifest and README descriptions, and regenerate docs/effects.md.
  • Update unit tests (src/renderer/effects/tetrisMatrixEffect.test.ts) to mock ctx.translate and assert the shake translation and HUD label rendering.

Testing

  • Ran unit tests: npm run test -- src/renderer/effects/tetrisMatrixEffect.test.ts src/renderer/debug/effectDebug.test.ts, and all tests passed (48 tests total).
  • Regenerated effect docs with npm run docs:effects and ran docs checks with npm run docs:check, and both checks passed.

Codex Task

@vercel
Copy link
Copy Markdown

vercel Bot commented Apr 8, 2026

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
codexperimental Ready Ready Preview, Comment Apr 8, 2026 1:19am

Request Review

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant