Skip to content

refactor(client): compact ten SessionRefresh test-only hooks into lazy storage (#801) - #808

Merged
SunSi12138 merged 2 commits into
devfrom
cleanup/issue-801-client-session-test-hooks
Oct 11, 2026
Merged

SunSi12138 merged 2 commits into
devfrom
cleanup/issue-801-client-session-test-hooks

Conversation

@SunSi12138

Copy link
Copy Markdown
Owner

Scope

Reduce 10 test-only delegate instance fields on every SharpLinkClient to one lazily allocated Delegate?[]? pointer. Preserve their original internals-visible names via typed properties, Volatile/CAS synchronization, and every test callback/race trigger. No production client with no hooks allocates the array. Estimated static x64 object reference reduction 9 × 8 = 72 bytes per client; no new claimed measured heap/throughput improvement.

The audited deterministic transform was applied to dev after merged #799, modifying three client SessionRefresh production files only. The real branch commit was made after complete Release build, UnitTests, Generator and Integration suites succeeded. Staging verification.

Earlier real Linux NativeAOT + Integration verification: #38107812236. PR-level checks must be verified for exact source head. No public API change. Related #801.

Copy link
Copy Markdown
Owner Author

Exact-head cross-platform NativeAOT evidence

Actions #38115419654 checked out this PR's exact committed source cbcf3aa970794bf29165bf6db29874d451250661 with an explicit commit-hash assertion.

  • PR 808 - osx-arm64 — succeeded: full Release solution build plus actual NativeAOT TCP, shared-memory, SharpPack sidecar and PreCredit streaming publish/run; job log.
  • PR 808 - win-x64 — succeeded: full Release solution build plus actual NativeAOT TCP, shared-memory, SharpPack sidecar and PreCredit streaming publish/run; job log.

No test substitution or modified checkout: workflow lives on the separate audit branch, and checks the PR's source at the pinned SHA. This is real desktop NativeAOT evidence; does not claim mobile hardware coverage.

@SunSi12138
SunSi12138 merged commit aa281aa into dev Oct 11, 2026
32 checks passed
@SunSi12138
SunSi12138 deleted the cleanup/issue-801-client-session-test-hooks branch October 11, 2026 06:37
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant