Skip to content

phase1d: add telemetry module with tier-selection events#292

Closed
MGudgin wants to merge 0 commit into
mainfrom
user/gudge/downlevel_phase1d
Closed

phase1d: add telemetry module with tier-selection events#292
MGudgin wants to merge 0 commit into
mainfrom
user/gudge/downlevel_phase1d

Conversation

@MGudgin

@MGudgin MGudgin commented May 12, 2026

Copy link
Copy Markdown
Member

📖 Description

Introduces a platform-agnostic telemetry module defining IsolationTier and TelemetryEvent types, plus an emit() helper that writes MXC_TELEMETRY-prefixed lines through the existing Logger.

No callers yet — emit sites land in phase1_integrate (AppContainer runner) and phase4 (dispatcher).

🔗 References

Stacked on top of #291 (phase1c → phase1b).

🔍 Validation

  • cargo test clean
  • End-to-end coverage lives in Win25H2Safe-Tests.ps1 in phase5 (held)

@MGudgin MGudgin force-pushed the user/gudge/downlevel_phase1c branch from 1a3f242 to 15b40af Compare May 12, 2026 21:06
@MGudgin MGudgin force-pushed the user/gudge/downlevel_phase1d branch from b591eb9 to fb3a708 Compare May 12, 2026 21:06
@MGudgin MGudgin force-pushed the user/gudge/downlevel_phase1c branch from 15b40af to 69c3d62 Compare May 12, 2026 21:16
@MGudgin MGudgin force-pushed the user/gudge/downlevel_phase1d branch from fb3a708 to ff81092 Compare May 12, 2026 21:16
@MGudgin MGudgin changed the base branch from user/gudge/downlevel_phase1c to main May 12, 2026 21:19
@MGudgin MGudgin force-pushed the user/gudge/downlevel_phase1d branch 2 times, most recently from af72afa to eabb1c2 Compare May 13, 2026 20:08
@MGudgin MGudgin closed this May 14, 2026
Copilot AI review requested due to automatic review settings May 14, 2026 13:51
@MGudgin MGudgin force-pushed the user/gudge/downlevel_phase1d branch from eabb1c2 to 16269fd Compare May 14, 2026 13:51

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Copilot wasn't able to review any files in this pull request.

@MGudgin MGudgin deleted the user/gudge/downlevel_phase1d branch May 14, 2026 14:39
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.

2 participants