Skip to content

refactor(3.0): delete 44 remaining verified unused methods after #806 (#801) - #810

Merged
SunSi12138 merged 1 commit into
devfrom
cleanup/issue-801-verified-dead-methods
Oct 11, 2026
Merged

SunSi12138 merged 1 commit into
devfrom
cleanup/issue-801-verified-dead-methods

Conversation

@SunSi12138

@SunSi12138 SunSi12138 commented Oct 11, 2026 •

Copy link
Copy Markdown
Owner

Current version: rebased onto latest dev after #813

Current head: 13e313ab1831
Base: 9fb2744300ee, including merged #807, #808, #809, #811, #812, and #813.

This is a clean, single commit based directly on current dev. It preserves the frozen #801 audit selection after #806 independently deleted one candidate: 44 unused non-public methods removed, with 1 bodyless source-generated logger partial intentionally retained. Scope: 31 production files, +26/-877 vs the new base. No additional candidates, public/generated ABI, or wire-contract changes are introduced by #810.

Conflict resolution and cleanup

CI for this exact head

Do not attribute historical CI to this commit. Prior rebase validation #38116153659 validated the earlier b07e85a5 head based on #806, not this replacement head.

Review: still Draft; PR Extended (full IntegrationTests, desktop compatibility and extended NativeAOT) is triggered when the PR becomes ready for review. No merge requested.

Related #801, #806, #811, #813.

Copy link
Copy Markdown
Owner Author

Exact-head cross-platform NativeAOT evidence

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

  • PR 810 - osx-arm64 — succeeded: full Release solution build plus actual NativeAOT TCP, shared-memory, SharpPack sidecar and PreCredit streaming publish/run; job log.
  • PR 810 - 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 force-pushed the cleanup/issue-801-verified-dead-methods branch from 0b735a7 to b07e85a Compare October 11, 2026 05:55
@SunSi12138 SunSi12138 changed the title refactor(3.0): remove 45 verified unused internal methods (#801) refactor(3.0): delete 44 remaining verified unused methods after #806 (#801) Oct 11, 2026
)

Preserve #807–#813 changes, especially the removed Channel stream facade
and test-only RPC error sender; remove orphan XML doc comments.
@SunSi12138
SunSi12138 force-pushed the cleanup/issue-801-verified-dead-methods branch from b07e85a to 13e313a Compare October 11, 2026 07:27
@SunSi12138
SunSi12138 marked this pull request as ready for review October 11, 2026 08:54
@SunSi12138
SunSi12138 merged commit 40c1365 into dev Oct 11, 2026
48 checks passed
@SunSi12138
SunSi12138 deleted the cleanup/issue-801-verified-dead-methods branch October 11, 2026 09:01
SunSi12138 added a commit that referenced this pull request Oct 11, 2026
SunSi12138 added a commit that referenced this pull request Oct 11, 2026
Rebase #820 onto dev after #810; keep only the explicit codec + exact-size codec entry point and migrate deadline tests to it. Do not restore #810's deleted no-codec relay.
SunSi12138 added a commit that referenced this pull request Oct 11, 2026
SunSi12138 added a commit that referenced this pull request Oct 11, 2026
…820)

Rebase #820 onto dev after #810; keep only the explicit codec + exact-size codec entry point and migrate deadline tests to it. Do not restore #810's deleted no-codec relay.
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