[Claude]

The Client property setter used a verbose manual null-check pattern instead of the idiomatic .NET 6+ helper.

Change

Replaced in CreateDistributedNonceServiceInstance.Client setter:

// Before
if (value is null)
{
    throw new ArgumentNullException(nameof(value));
}

// After
ArgumentNullException.ThrowIfNull(value);

ThrowIfNull auto-captures the parameter name via [CallerArgumentExpression], eliminating the nameof boilerplate while preserving identical runtime behavior.

---

⌨️ Start Copilot coding agent tasks without leaving your editor — available in VS Code, Visual Studio, JetBrains IDEs and Eclipse.

[Copilot]

Pull request overview

This PR updates DistributedNonceService to incorporate the blockchain chain ID into the distributed lock key, preventing cross-chain lock collisions when the same address is used on multiple networks (e.g., Ethereum + Polygon).

Changes:

• Updated distributed lock key format from DistributedNonce_{address} to DistributedNonce_{chainId}_{address}.
• Added per-instance chain ID retrieval/caching via an eth_chainId RPC call.
• Extended integration tests to mock eth_chainId and added a test intended to validate multi-chain isolation.

Reviewed changes

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

File	Description
src/DistributedNonce/Services/DistributedNonceService.cs	Adds chain ID fetching/caching and incorporates chain ID into lock keys for GetNextNonceAsync and ResetNonceAsync.
tests/DistributedNonce.Tests/DistributedNonceIntegrationTests.cs	Updates RPC mocks to handle eth_chainId and adds an integration test intended to cover chain-based isolation.

[sonarqubecloud]

Quality Gate failed

Failed conditions
0.0% Coverage on New Code (required ≥ 80%)

See analysis details on SonarQube Cloud