Zio 1 update

[kanderson250]

Explicitly kill the tokens when the tracer around a ZIO transaction finishes. Do not increment a refcount.

Adds a unit test with back-to-back transactions.

[codecov-commenter]

Codecov Report

All modified and coverable lines are covered by tests ✅

Comparison is base (75bceb7) 70.81% compared to head (6281c1d) 70.76%.
Report is 54 commits behind head on main.

Additional details and impacted files

@@             Coverage Diff              @@
##               main    #1739      +/-   ##
============================================
- Coverage     70.81%   70.76%   -0.05%     
+ Complexity    10097     9932     -165     
============================================
  Files           831      827       -4     
  Lines         40346    39850     -496     
  Branches       6131     6036      -95     
============================================
- Hits          28570    28199     -371     
+ Misses         9025     8925     -100     
+ Partials       2751     2726      -25     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

[obenkenobi]

Maybe change

public static void clearThreadTokenAndRefCountAndTxn(AgentBridge.TokenAndRefCount tokenAndRefCount) {
    AgentBridge.activeToken.remove();
    if (tokenAndRefCount != null && tokenAndRefCount.refCount.decrementAndGet() == 0) {
      tokenAndRefCount.token.expire();
      tokenAndRefCount.token = null;
    }
  }

to be

public static void clearThreadTokenAndRefCountAndTxn(AgentBridge.TokenAndRefCount tokenAndRefCount) {
    AgentBridge.activeToken.remove();
    if (tokenAndRefCount != null) {
      tokenAndRefCount.token.expire();
      tokenAndRefCount.token = null;
    }
  }

[obenkenobi]

Please refer to this issue for more details on the fix