Skip to content

Add Network migration readiness audit - #50

Merged
rchatham merged 3 commits into
feature/network-migration-local-network-notesfrom
feature/network-migration-readiness-audit
Aug 10, 2026
Merged

Add Network migration readiness audit#50
rchatham merged 3 commits into
feature/network-migration-local-network-notesfrom
feature/network-migration-readiness-audit

Conversation

@rchatham

@rchatham rchatham commented Aug 9, 2026

Copy link
Copy Markdown
Owner

Summary

  • Add the authoritative readiness audit for graduating .networkFramework from experimental opt-in to stable opt-in, then default, and eventually removing MultipeerConnectivity.
  • Inventory completed capabilities and accepted non-parity against the current implementation and tests.
  • Classify optional PR Add Network migration readiness audit #50+ work by when it becomes required: before stable, before default, only before removal, product-dependent, or optional.
  • Record release/versioning constraints and a risk register.
  • Reconcile and cross-link the existing migration plan, PR plan, backend guide, and README.

Decision

The current Network backend is suitable for bounded experimental opt-in evaluation. It is not ready for stable/default status or MultipeerConnectivity removal. No runtime merge blocker was found for the current stack, so this PR is documentation-only and does not change the default backend.

Key gates

  • Before stable: individual peer authentication, public Network failure/permission observability, discovery/idle resource bounds, soak/churn testing, and recorded physical-device/topology validation.
  • Before default: at least one stable opt-in release cycle, a secure replacement for the current unauthenticated default initialization path, complete migration guidance, and rollback criteria.
  • Before MC removal: deprecate/replace MC-specific public surfaces and either implement or intentionally remove stream/resource behavior in a breaking release.

Verification

  • swift test — 113 tests passed.
  • xcodebuild test -project PeerConnectivity.xcodeproj -scheme PeerConnectivity -destination 'platform=iOS Simulator,name=iPhone 16,OS=18.3.1' -configuration Debug — 113 tests passed.
  • git diff --check and local Markdown link validation — passed.
  • Correctness review: no merge-blocking findings.
  • Security review: no merge-blocking security findings.

Risks

The current group-PSK model does not authenticate individual peers; Network startup/permission failures are not consistently surfaced; discovery is unbounded and connections have no idle timeout; and simulator/loopback tests do not replace physical-device validation. These are explicitly recorded as pre-stable gates. No security behavior changes in this documentation-only PR.

@rchatham
rchatham changed the base branch from feature/network-migration-local-network-notes to master August 10, 2026 04:04
@rchatham
rchatham changed the base branch from master to feature/network-migration-local-network-notes August 10, 2026 04:07
@rchatham
rchatham merged commit 51732c0 into feature/network-migration-local-network-notes Aug 10, 2026
2 of 4 checks passed
@rchatham
rchatham deleted the feature/network-migration-readiness-audit branch August 10, 2026 04:10
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