Refactor SecureRandom

[terrier989]

No description provided.

[Copilot]

Pull request overview

This PR refactors the SecureRandom class to improve code organization and maintainability. The changes include extracting reseeding logic into a separate method, adding a new factory constructor, improving documentation with inline comments, and enhancing test coverage with more comprehensive distribution tests.

Key Changes:

• Introduced a new public factory constructor SecureRandom.chaCha20() for creating ChaCha20-based random number generators
• Refactored the reseeding logic by extracting it into a dedicated _mixSystemRandom() method
• Made maxDurationBeforeReseed nullable to allow disabling time-based reseeding
• Enhanced test suite with additional distribution validation tests and better test descriptions

Reviewed changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated 4 comments.

File	Description
cryptography/lib/src/cryptography/secure_random.dart	Refactored SecureRandom implementation: added factory constructor, extracted reseeding logic into _mixSystemRandom(), made maxDurationBeforeReseed nullable, renamed _seed to _seedForDeterministicSequence, improved inline documentation, and enhanced modulo bias handling with clearer comments
cryptography/test/secure_random_test.dart	Updated tests to use new naming conventions, added comprehensive distribution tests for nextBool(), nextInt(), nextDouble(), and nextUint52(), increased sample sizes for better statistical validation, and added bounds checking for distribution tests

---

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.