[ggivo]

Summary

Removes direct Jedis reference from Transaction class to reduce coupling.

Changes

• Added protected hook methods onAfterExec() and onAfterDiscard() in Transaction
• Jedis.multi() now creates an anonymous subclass that overrides these hooks to call resetState()
• Removed Jedis field from Transaction

Testing

Verified by TransactionCommandsTest ✅

---

Note

Medium Risk
Changes transaction lifecycle cleanup by moving Jedis.resetState() to hook callbacks, which could affect connection/state handling after EXEC/DISCARD if any edge cases rely on the old direct reference.

Overview
Decouples Transaction from Jedis while preserving state cleanup behavior. Transaction no longer stores a Jedis reference; instead it exposes protected hooks onAfterExec() and onAfterDiscard() that run in the finally blocks of exec()/discard().

Jedis.multi() now creates a Transaction backed by getConnection() and overrides those hooks to call resetState(), keeping legacy Jedis state reset semantics without tight coupling.

^{Written by Cursor Bugbot for commit cc1fe6f. This will update automatically on new commits. Configure here.}

[github-actions]

Test Results

   308 files  ±0     308 suites  ±0   11m 39s ⏱️ -6s
11 136 tests ±0  10 921 ✅  - 159  215 💤 +159  0 ❌ ±0 
 5 851 runs  ±0   5 784 ✅  -  58   67 💤 + 58  0 ❌ ±0 

Results for commit cc1fe6f. ± Comparison against base commit ae4ed55.

This pull request skips 159 tests.

redis.clients.jedis.commands.jedis.ClusterHotkeysCommandsTest ‑ hotkeysGetNotSupportedInCluster
redis.clients.jedis.commands.jedis.ClusterHotkeysCommandsTest ‑ hotkeysResetNotSupportedInCluster
redis.clients.jedis.commands.jedis.ClusterHotkeysCommandsTest ‑ hotkeysStartNotSupportedInCluster
redis.clients.jedis.commands.jedis.ClusterHotkeysCommandsTest ‑ hotkeysStopNotSupportedInCluster
redis.clients.jedis.commands.jedis.ClusterHotkeysCommandsTest ‑ hotkeysWithSlotsOnSingleClusterNode
redis.clients.jedis.commands.jedis.HotkeysCommandsTest[1] ‑ hotkeysBothMetrics
redis.clients.jedis.commands.jedis.HotkeysCommandsTest[1] ‑ hotkeysDurationOption
redis.clients.jedis.commands.jedis.HotkeysCommandsTest[1] ‑ hotkeysGetBeforeStart
redis.clients.jedis.commands.jedis.HotkeysCommandsTest[1] ‑ hotkeysLifecycle
redis.clients.jedis.commands.jedis.HotkeysCommandsTest[1] ‑ hotkeysResponseFields
…

♻️ This comment has been updated with latest results.

[atakavci]

LGTM.
lets label it as breaking.

[jit-ci]

🛡️ Jit Security Scan Results

✅ No security findings were detected in this PR

---

^{Security scan by Jit}