fix: RootResponse class

[cuongph87]

Types of changes

• Fixes Fix RootResponse class #93
• Adds required modifier to always-present properties

• Bug fix (non-breaking change which fixes an issue)
• New feature (non-breaking change which adds functionality)
• Breaking change (fix or feature that would cause existing functionality to change)
• I have read the CONTRIBUTING document.
• My code follows the code style of this project.
• My change requires a change to the documentation.
• I have updated the documentation accordingly.
• I have added tests to cover my changes.
• All new and existing tests passed.

---

Note

Introduce RootResponseLinks and make RootResponse include required properties and new fields aligned with Horizon 24.x; update tests and JSON fixture accordingly.

• Responses:
  • Update StellarDotnetSdk/Responses/RootResponse.cs:
    • Make class sealed; add required Links property and mark always-present fields as required.
    • Add history_latest_ledger_closed_at and supported_protocol_version properties.
  • Add StellarDotnetSdk/Responses/RootResponseLinks.cs defining typed _links for account, accounts, account_transactions, assets, claimable_balances, effects, fee_stats, friendbot, ledger, ledgers, liquidity_pools, offer, offers, operation, operations, order_book, payments, self, strict_receive_paths, strict_send_paths, trade_aggregations, trades, transaction, transactions.
• Tests:
  • Update StellarDotnetSdk.Tests/TestData/Responses/root.json to Horizon/Core 24.x and testnet URLs; include new fields.
  • Expand StellarDotnetSdk.Tests/Responses/RootDeserializerTest.cs to assert new properties and all link hrefs.

^{Written by Cursor Bugbot for commit 473d54c. Configure here.}

[Copilot]

Pull request overview

This PR fixes the RootResponse class by adding proper null-safety, introducing a dedicated RootResponseLinks class, and updating test data to match current Horizon API responses. The changes enforce that always-present properties are marked as required, improving type safety and preventing null reference issues.

Key Changes:

• Introduced required modifier to all always-present properties in RootResponse
• Created a new RootResponseLinks class to properly model the _links section
• Updated test data to reflect current Horizon API structure

Reviewed changes

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

File	Description
StellarDotnetSdk/Responses/RootResponseLinks.cs	New class modeling the _links section with strongly-typed link properties
StellarDotnetSdk/Responses/RootResponse.cs	Added required modifiers, XML documentation, new Links property, and removed nullable disable directive
StellarDotnetSdk.Tests/TestData/Responses/root.json	Updated test data with current Horizon API structure and protocol versions
StellarDotnetSdk.Tests/Responses/RootDeserializerTest.cs	Expanded test assertions to validate all properties including the new Links property

[jopmiddelkamp]

bugbot run