Extended RandomResolver to support inclusive max range.

[lextatic]

Added

• Added inclusive methods to IRandom interface.
• New test cases.

Changed

• Modified RandomResolver to receive optional configuration to configure max as inclusive or exclusive. Inclusive by default.

[Copilot]

Pull request overview

Updates the statescript RandomResolver and the core IRandom abstraction to support inclusive max bounds (with configurability), and refreshes docs/tests to reflect the new behavior.

Changes:

• Added inclusive-range methods to IRandom (NextIntInclusive, NextSingleInclusive, NextDoubleInclusive, NextInt64Inclusive).
• Extended RandomResolver with a maxInclusive configuration (defaulting to inclusive) and updated float/double behavior accordingly.
• Updated documentation and expanded unit tests/helpers for the new inclusive/exclusive paths.

Reviewed changes

Copilot reviewed 7 out of 7 changed files in this pull request and generated 5 comments.

Show a summary per file

File	Description
docs/statescript/resolvers/random-resolver.md	Documents the new maxInclusive parameter and inclusive-by-default behavior.
docs/effects/components.md	Updates the documented IRandom interface to include inclusive methods.
Forge/Statescript/Properties/RandomResolver.cs	Implements inclusive/exclusive max selection and new float/double handling.
Forge/Core/IRandom.cs	Adds new inclusive methods to the public random provider interface.
Forge.Tests/Statescript/Resolvers/RandomResolverTests.cs	Adds coverage for inclusive-by-default and exclusive configuration paths.
Forge.Tests/Helpers/StatescriptTestHelpers.cs	Extends test random providers to implement the new IRandom members.
Forge.Tests/Effects/ChanceToApplyComponentTests.cs	Updates the test random provider to satisfy the expanded IRandom interface.

---

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