stacklok / toolhive / 23534304405

push

github

Move TTL refresh into LocalStorage.Load, remove Session.Touch() (#4337)

Session.Touch() was a leaky abstraction — for Redis-backed sessions it was
a no-op (GETEX already refreshes the TTL on read), while for LocalStorage
the in-memory timestamp update was driven by the caller (Manager.Get) rather
than the storage backend that actually needs it.

TTL refresh is now a storage concern: LocalStorage.Load updates the session's
last-access timestamp via a package-local `touchable` interface, and
Manager.Get no longer calls Touch(). Touch() is removed from the Session
interface and from the MockMultiSession mock, eliminating the
`.EXPECT().Touch().AnyTimes()` boilerplate across all session-related tests.

Co-authored-by: taskbot <taskbot@users.noreply.github.com>

33 of 52 new or added lines in 1 file covered. (63.46%)

51199 of 78551 relevant lines covered (65.18%)

131.14 hits per line