You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
Move the perf, logs, events, network, and audio session routes behind the named observability facade while retaining SessionStore-owned live state, existing admission/resource owners, and exact response behavior.
Reviewed exact head 1b92b012574e2f420be12802cca7458bfd489098: no code or architecture findings. The facade is a real ownership boundary, not forwarding ceremony: routing imports one narrow capability input, observability owns family dispatch/composition, specialized operations remain private, store-owned state and admission/resource owners are preserved, and anti-regrowth/deep-import gates are meaningful. All exact-head CI is green and move-adjusted production LOC is -1. BLOCKED only on the readiness record: stamp the PR body’s Validation with this exact head so the local/fallow/affected evidence and documented host-load noise are tied to a named commit.
Re-reviewed exact head 1b92b012574e2f420be12802cca7458bfd489098: the sole readiness-record blocker is resolved. Validation now stamps this exact head and ties the focused, layering, Fallow, affected, and host-load comparison evidence to it. Code/architecture remain clean, all exact-head CI and iOS/Android/macOS/Linux Smoke lanes pass, package size is -138 B, and move-adjusted production is -1 LOC. READY for human review and merge.
Preview removed because the pull request was closed.
2026-09-01 14:39 UTC
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
ready-for-humanValid work that needs human implementation, judgment, or maintainer merge
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
Move the perf, logs, events, network, and audio session routes behind the named observability facade while retaining SessionStore-owned live state, existing admission/resource owners, and exact response behavior.
Closes #2176
Validation
1b92b012574e2f420be12802cca7458bfd489098.pnpm check:layering: 183 tests passed; full guard clean.pnpm check:fallow --base origin/main: no issues in 37 changed files.pnpm check:affected --run: format, lint, typecheck, layering, DI, Fallow, metadata, build, package, Node integration, and macOS coverage lanes passed. The related-test phase had 6 host-load failures with 2,248/2,254 tests passing. The mutation-model self-test independently reproduces 3 current-main registry failures; the clean-main comparison reproduces the 4 iOS/interaction timeout failures.