Skip to content

Add crit, dodge and counter upgrade stats - #35

Merged
Finesssee merged 1 commit into
mainfrom
feat/crit-dodge-counter
Oct 13, 2026
Merged

Finesssee merged 1 commit into
mainfrom
feat/crit-dodge-counter

Conversation

@Finesssee

Copy link
Copy Markdown
Collaborator

Upgrade points v2, part 2 of 3. Viewers can now spend points on three more stats.

Stat Each level (1-8) Ceiling Quick duel HP fight
Crit 4% a hit becomes a crit; HP fight: strike or heavy +50% 50% yes yes
Dodge 4% a hit or crit on you misses 50% yes no
Counter +2% counter damage, and 2% a foe's miss becomes your counter 60% / 50% yes no
  • A quick-duel swing checks luck, crit, dodge, then counter, each on its own roll slot (rolls[k*24+i]). The room sends 120 rolls.
  • New crit, dodge, counter profile columns (additive migration). A rank reset trims levels from counter first.
  • Pets keep boosting power, guard and luck only (PET_STATS); legendary text now says "+1 power, guard and luck".
  • The overlay floats DODGE (and DODGE then COUNTER), LUCKY CRIT, and HP-fight crits.
  • The fighter page shows 10 presets; Tank and Bruiser wait for vitality (PR 3).
  • Docs: CONTRACTS §2, §4, §6, DUELS.md, STREAMER_SETUP.md.

Tests: bun run check (384 unit tests) and bun run test:ui pass.

Three new stats bought with upgrade points. Crit turns a hit into a crit (HP fights: +50% on a strike
or heavy). Dodge turns a hit or crit on you into a miss. Counter adds counter damage and turns a foe's
miss into your counter. Each check reads its own roll slot, so the room sends 120 rolls. Pets keep
boosting power, guard and luck only. The overlay shows DODGE and LUCKY CRIT.
@coderabbitai

coderabbitai Bot commented Oct 13, 2026

Copy link
Copy Markdown

Important

  • 🔍 Trigger review

This repository does not receive automatic reviews because it has fewer than 10 stars.

⚙️ Run configuration
  • Configuration used: Repository: nesszer/pixfray/.coderabbit.yaml
  • Review profile: CHILL
  • Plan: Advanced
  • Run ID: a19eb047-4ce2-4143-abbc-1d339e79a6db
  • Autofix · Keep fixing CodeRabbit findings and required CI, and resolving merge conflicts

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands.

@Finesssee
Finesssee merged commit 9914e63 into main Oct 13, 2026
3 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