Skip to content

refactor(runtime): collapse nine PooledAsyncStreamDispatcher test hook fields (#801) - #809

Merged
SunSi12138 merged 4 commits into
devfrom
cleanup/issue-801-dispatcher-test-hooks
Oct 11, 2026
Merged

SunSi12138 merged 4 commits into
devfrom
cleanup/issue-801-dispatcher-test-hooks

Conversation

@SunSi12138

Copy link
Copy Markdown
Owner

Scope

Replace nine test-only Action? fields on each production PooledAsyncStreamDispatcher with one lazy Action?[]? reference. Keep all original internal setter names and callback invocation sites; preserve Volatile/Interlocked publication, reset and pooled lifecycle. No hooks means no array allocation. Theoretical x64 per-instance reduction: 8 pointers = 64 bytes, not a measured GC or RPC throughput claim.

The previously validated source transformer was applied to dev after merged #799. The actual source was committed only after full Release build, UnitTests and Generator Tests passed. Staging verification.

Prior probe and pristine restored baseline: #38103482749. Preserve full integration, trim and NativeAOT PR gates. No API or wire changes. Related #801.

Copy link
Copy Markdown
Owner Author

Exact-head cross-platform NativeAOT evidence

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

  • PR 809 - osx-arm64 — succeeded: full Release solution build plus actual NativeAOT TCP, shared-memory, SharpPack sidecar and PreCredit streaming publish/run; job log.
  • PR 809 - 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 590e292 into dev Oct 11, 2026
6 checks passed
@SunSi12138
SunSi12138 deleted the cleanup/issue-801-dispatcher-test-hooks branch October 11, 2026 06:51
SunSi12138 added a commit that referenced this pull request Oct 11, 2026
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