stacklok / toolhive / 23134320457

push

github

Remove session v1 and SessionManagementV2 feature flag (#4128)

* Remove session v1 and SessionManagementV2 feature flag

Session v2 is now the sole implementation. The SessionManagementV2
config option has been deleted along with all v1 code paths:
VMCPSession, sessionIDAdapter, the global-router-based discovery
fallback, and related tests. SessionFactory is now a required field
on server.Config. All "V2" naming has been dropped — factories,
managers, tests, and e2e fixtures now use plain "session" terminology.
Dead v1 capability-injection helpers (injectCapabilities,
injectOptimizerCapabilities, setSessionResourcesDirect) have been
removed, as has the operator's conditional HMAC-secret guard.

Affected: pkg/vmcp/server, pkg/vmcp/session, pkg/vmcp/config,
          pkg/vmcp/discovery, cmd/vmcp, cmd/thv-operator, test/e2e

Related-to: #3872

* use mocks

---------

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

240 of 334 new or added lines in 8 files covered. (71.86%)

48783 of 75626 relevant lines covered (64.51%)

71.69 hits per line