feat: standalone TorBoxClient without DI + IDisposable

[Copilot]

Adds direct instantiation of TorBoxClient without requiring a DI container, while keeping the existing DI path fully functional.

// Simple
using TorBoxClient client = new("your-api-key");

// Full options
using TorBoxClient client = new(new TorBoxClientOptions { ApiKey = key, Timeout = TimeSpan.FromSeconds(60) });

// Builder
using TorBoxClient client = new(o => { o.ApiKey = key; o.Timeout = TimeSpan.FromSeconds(60); });

Core SDK

• ITorBoxClient extends IDisposable
• TorBoxClient: 3 public standalone constructors that create and own HttpClient instances; DI constructor marked [ActivatorUtilitiesConstructor]
• Dispose() releases owned HttpClients in standalone mode, no-op in DI mode
• AuthHandler: new AuthHandler(string apiKey) constructor for standalone; DI constructor must remain public (not internal) because the DI container only resolves public constructors, even on internal types
• TorBoxClientOptions.MainApiVersionedUrl / RelayApiVersionedUrl: internal → public (read-only computed)
• RegisterCore simplified from explicit factory lambda to AddScoped<ITorBoxClient, TorBoxClient>()
• ValidateBaseUrl enforces trailing / on base URLs for correct relative URI resolution

Tests

• 10 unit tests: standalone constructors, argument validation, IDisposable contract (verifies ObjectDisposedException on disposed HttpClient, asserts actual HttpClient.Timeout via reflection)
• 2 unit tests: AuthHandler(string) bearer injection
• 2 integration tests: standalone client end-to-end (skip without TORBOX_API_KEY)

Samples & Docs

• StandaloneSetupExample.cs demonstrating all 3 patterns; Program.cs menu updated (38 examples)
• README, getting-started, configuration, architecture docs updated with standalone usage sections

CI/CD

• Both workflows (ci.yml, integration-tests.yml) changed to manual-only (workflow_dispatch)
• Removed pull_request_review trigger and the check-approval job (review approval gate) from both workflows

Verification

• 490 unit tests, 111 schema tests, 45 integration tests — all pass
• Build: 0 warnings across net6.0–net10.0
• CodeQL: 0 alerts

[Copilot]

Pull request overview

Adds a standalone (non-DI) construction path for TorBoxClient while keeping the existing DI-based registration/usage intact, including explicit ownership/disposal of HttpClient instances in standalone mode.

Changes:

• Introduces standalone TorBoxClient constructors (api key / options / configure delegate) and IDisposable behavior for owned HttpClients.
• Updates DI registration to rely on [ActivatorUtilitiesConstructor] and simplifies ITorBoxClient registration.
• Adds/updates unit + integration tests, examples, and docs to cover standalone usage.

Reviewed changes

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

Show a summary per file

File	Description
tests/TorboxSDK.UnitTests/TorBoxClientTests.cs	New unit tests for standalone constructors + dispose behavior.
tests/TorboxSDK.UnitTests/Http/AuthHandlerTests.cs	Adds tests for AuthHandler(string apiKey) authorization header behavior.
tests/TorBoxSDK.IntegrationTests/Standalone/StandaloneClientTests.cs	New live integration tests exercising standalone client end-to-end.
src/TorBoxSDK/TorBoxClientOptions.cs	Makes versioned URL helpers public and documents computed behavior.
src/TorBoxSDK/TorBoxClient.cs	Implements standalone constructors, owned HttpClient creation, and Dispose().
src/TorBoxSDK/ITorBoxClient.cs	Extends ITorBoxClient to implement IDisposable.
src/TorBoxSDK/Http/Handlers/AuthHandler.cs	Adds standalone ctor and DI ctor marked with [ActivatorUtilitiesConstructor].
src/TorBoxSDK/DependencyInjection/TorBoxServiceCollectionExtensions.cs	Simplifies core registration to AddScoped<ITorBoxClient, TorBoxClient>().
src/TorBoxSDK.Examples/Program.cs	Adds standalone setup example entry and renumbers menu options.
src/TorBoxSDK.Examples/GettingStarted/StandaloneSetupExample.cs	New example demonstrating all standalone construction patterns + disposal.
docs/getting-started.md	Documents standalone (no-DI) usage.
docs/configuration.md	Updates options table + adds standalone configuration section.
docs/architecture.md	Updates instantiation guidance to include standalone mode and disposal semantics.
README.md	Adds standalone usage section with code snippets.

[devRael1]

@copilot apply changes based on the comments in this thread

[Copilot]

@copilot apply changes based on the comments in this thread

All 6 review comments addressed in commit f15af3a:

1. XML docs for TorBoxClient(string apiKey) — now lists ArgumentNullException (null) and ArgumentException (empty) separately
2. ValidateBaseUrl — now also rejects base URLs missing a trailing / to prevent incorrect relative URI resolution
3. [Collection("Integration")] added to StandaloneClientTests + <remarks> about TORBOX_API_KEY
4. AAA convention — all tests refactored with proper separate // Arrange, // Act, // Assert phases
5. Constructor_WithCustomTimeout — now asserts HttpClient.Timeout via reflection instead of just checking sub-clients
6. Dispose_WhenStandalone — renamed to Dispose_WhenStandalone_DisposesOwnedHttpClients, now verifies ObjectDisposedException on the disposed HttpClient

[devRael1]

@copilot modifie les workflows github afin d'executer les tests unitaires + intégrations uniquement manuellement. De plus, il ne faut pas lancer automatiquement ces workflows dès lors qu'une PR est approuvée (supprime le CI / Check All Reviews Approved)

[Copilot]

@copilot modifie les workflows github afin d'executer les tests unitaires + intégrations uniquement manuellement. De plus, il ne faut pas lancer automatiquement ces workflows dès lors qu'une PR est...

Done in d1aab4e. Both workflows (ci.yml and integration-tests.yml) now trigger only on workflow_dispatch (manual). Removed the pull_request_review trigger and the entire check-approval job (review approval logic) from both workflows.