|
| 1 | +# APPLY_PR_DEBUG_SURFACES_NETWORK_SUPPORT |
| 2 | + |
| 3 | +## Purpose |
| 4 | +Apply the approved shared network support layer in a small, boundary-safe implementation step. |
| 5 | + |
| 6 | +## Workflow |
| 7 | +PLAN_PR -> BUILD_PR -> APPLY_PR |
| 8 | + |
| 9 | +## PR Purpose |
| 10 | +One PR purpose only: implement summary-level shared network providers, panels, and optional presets. |
| 11 | + |
| 12 | +## Apply Scope |
| 13 | +### Create shared providers |
| 14 | +- `provider.net.connection.snapshot` |
| 15 | +- `provider.net.latency.snapshot` |
| 16 | +- `provider.net.replication.snapshot` |
| 17 | +- `provider.net.divergence.snapshot` |
| 18 | +- `provider.net.events.snapshot` |
| 19 | + |
| 20 | +### Create shared panels |
| 21 | +- `panel.net.connection` |
| 22 | +- `panel.net.latency` |
| 23 | +- `panel.net.replication` |
| 24 | +- `panel.net.divergence` |
| 25 | +- `panel.net.events` |
| 26 | + |
| 27 | +### Create optional shared presets |
| 28 | +- `preset.net.status` |
| 29 | +- `preset.net.replication` |
| 30 | +- `preset.net.diagnostics` |
| 31 | + |
| 32 | +### Keep local |
| 33 | +- transport/protocol adapters |
| 34 | +- authority extraction |
| 35 | +- stack-specific scene/debug mappings |
| 36 | + |
| 37 | +## Execution Order |
| 38 | +1. providers |
| 39 | +2. panels (provider-driven only) |
| 40 | +3. optional presets |
| 41 | +4. shared registration entry points |
| 42 | +5. local adapter harness validation |
| 43 | + |
| 44 | +## Rules |
| 45 | +- preserve existing provider/panel/preset conventions |
| 46 | +- keep adoption opt-in |
| 47 | +- keep shared layer transport-agnostic |
| 48 | +- no deep inspectors |
| 49 | +- no protocol-specific implementation |
| 50 | +- no packet inspectors |
| 51 | +- no auth/matchmaking tooling |
| 52 | + |
| 53 | +## Validation |
| 54 | +- run syntax checks for touched modules |
| 55 | +- validate with local adapter harness or sample integration |
| 56 | +- verify shared layer consumes provider snapshots only |
| 57 | + |
| 58 | +## Apply Outcome Contract |
| 59 | +- additive, non-destructive changes only |
| 60 | +- no engine-core rewrites |
| 61 | +- no project-specific adapter logic moved into shared layer |
0 commit comments