Skip to content

Stabilize randomized bot tests - #72

Merged
8tp merged 1 commit into
mainfrom
agent/stabilize-randomized-tests
Aug 8, 2026
Merged

Stabilize randomized bot tests#72
8tp merged 1 commit into
mainfrom
agent/stabilize-randomized-tests

Conversation

@8tp

@8tp 8tp commented Aug 8, 2026

Copy link
Copy Markdown
Owner

Summary

  • measure bot-emote cooldown from the time an emote is actually emitted, rather than when its randomized delivery is scheduled
  • seed the aggressive-versus-conservative challenge-rate simulation so it tests personality behavior deterministically

Root cause

TypeScript 7's dependency PR alternated between green and red without code changes. One failure was a statistical assertion using the process-wide random stream. The other exposed a real timing bug: different randomized emote delays could make delivered reactions land less than the configured cooldown apart even though their scheduling times passed the check.

Impact

CI no longer depends on lucky random samples for this personality assertion, and players receive the full configured cooldown between visible bot emotes.

Validation

  • npm run typecheck
  • affected 54 tests passed in 10 consecutive runs
  • git diff --check
  • full unrestricted test/build gate delegated to GitHub Actions because the local managed sandbox blocks the socket E2E suite

@8tp
8tp merged commit 391d97a into main Aug 8, 2026
1 check 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