Repository navigation
Follow-up from #2085 review: stale Observability module card #2133
Description
Minor (P2) items from the Codex review of #2085. Not blocking the merge; collected here so they don't get lost.
- Update the Observability module card in
Sources/AGENTS.md(around line 71). It still listsAnalyticsEventPolicy-> Speech as a grandfathered crossing, which Module edges: move DictationInputDeviceSelectionReason to Support #2085 removes. Note main has also dropped theActivationTelemetrybaseline entry since, so check whether that half of the sentence is stale too. Module edges: move DictationInputDeviceSelectionReason to Support #2085 (comment)
Activity
r3dbars commentedon Oct 17, 2026
Already fixed on main. Commit b83446e ("Document resolved lower-module dependency seams", merged in #2173 on 2026-10-08) rewrote the Observability card in
Sources/AGENTS.md. It dropped the whole "Grandfathered:ActivationTelemetryandAnalyticsEventPolicyname Dictation and Speech types" sentence, so both halves the review flagged are gone. The card now says Observability may depend on Support and Corecore-vocab, and telemetry consumes plain values.Checked on current origin/main:
Sources/AGENTS.md,Sources/Observability/AGENTS.mdanddocs/repo-layout.mdhave no grandfathered-crossing text. The only hit for "grandfather" is "Nothing is grandfathered." indocs/repo-layout.md.check-module-boundaries.py --explain Sources/Observability/AnalyticsEventPolicy.swiftshows it may depend on Support and Core:core-vocab only. That matches the card.Sources/AGENTS.md.No PR needed.