feat(cache): cache game/localization versions from /metadata with configurable TTL

[MarTrepodi]

Summary

Adds an instance-level version cache so version-less get_game_data(), get_localization(), and get_latest_game_data_version() calls no longer make a fresh /metadata round-trip every time, plus two bug fixes picked up along the way.

Fixes #116

Version cache (8cdd771)

• Both SwgohComlink and SwgohComlinkAsync now cache latestGamedataVersion and latestLocalizationBundleVersion on the instance for version_cache_ttl seconds (default 3600).
• Cold-cache lookups single-flight behind a threading.Lock (sync) / asyncio.Lock (async) so concurrent callers don't stampede /metadata.
• Successful get_game_metadata() calls refresh the cache opportunistically; explicit version arguments always bypass it.
• Escape hatches: version_cache_ttl=0 restores the previous uncached behavior, math.inf caches for the client lifetime, and invalidate_version_cache() / refresh=True force a re-fetch.

Fix: /metadata payload key (1909912) — fixes #116

get_game_metadata() sent the snake_case client_specs key, but the comlink /metadata request schema requires camelCase clientSpecs and declares additionalProperties: false, so requests with client specs were rejected by the server. Verified against the comlink OpenAPI spec (v0.40.1). The mocked unit tests asserted the wrong key and were updated alongside the fix.

Fix: exception constructor logging (f841fa8)

Constructing a SwgohComlinkException logged an error-level traceback via logger.exception() even for expected, caught validation errors, and emitted NoneType: None tracebacks outside an except block. The constructor override is removed; callers own logging context.

Testing

• New tests/unit/test_version_cache.py (214 lines) covering TTL expiry, single-flight behavior, opportunistic refresh, explicit-version bypass, invalidation, and refresh=True for both clients.
• New tests/unit/test_exceptions.py cases asserting constructors no longer log.
• Existing metadata payload tests updated to assert the correct clientSpecs key.

🤖 Generated with Claude Code