Skip to content

Phase 29A: instrument base-station startup diagnostics - #10

Merged
Zaknin merged 1 commit into
mainfrom
phase/29a-base-station-startup-instrumentation
Jun 16, 2026
Merged

Zaknin merged 1 commit into
mainfrom
phase/29a-base-station-startup-instrumentation

Conversation

@Zaknin

@Zaknin Zaknin commented Jun 16, 2026

Copy link
Copy Markdown
Owner

Summary

  • Adds instrumentation-only base-station startup diagnostics for Supervisor Bluetooth wake stages and Configurator Scan observations.
  • Writes bounded JSONL diagnostics under %LOCALAPPDATA%\PimaxVrcSupervisor\Diagnostics\BaseStations with deterministic station hashes and no full Bluetooth addresses.
  • Adds scripts/collect-base-station-startup-diagnostics.ps1 to package logs, diagnostics, process state, and read-only Bluetooth adapter metadata for Phase 29B analysis.

Behavior preservation

  • No base-station timing, retry, burst, payload, scan, discovery, protocol, TUI, SteamVR, shutdown, or startup behavior changes are intended.
  • PowerOnCommandTimeout, retry delay, OpenVR cycles, pass counts, and command bytes remain unchanged.
  • Diagnostics write failures are non-fatal.

Validation

  • dotnet build .\PimaxVrcSupervisor\PimaxVrcSupervisor.csproj -c Release passed.
  • dotnet test .\PimaxVrcSupervisor.Tests\PimaxVrcSupervisor.Tests.csproj -c Release passed: 126 tests.
  • dotnet build .\PimaxVrcSupervisor.ConfigEditor\PimaxVrcSupervisor.ConfigEditor.csproj -c Release passed.
  • dotnet build .\PimaxVrcSupervisor.SteamVrHost\PimaxVrcSupervisor.SteamVrHost.csproj -c Release passed.
  • cargo test --manifest-path .\PimaxVrcSupervisor.Tui\Cargo.toml passed: 12 tests.
  • python -m mkdocs build --strict passed with the existing MkDocs Material warning.
  • Static scan found no device/service/process mutation APIs in the new diagnostics path; the collector uses read-only Get-PnpDevice metadata.

Interactive smoke

  • Baseline startup with instrumented Supervisor: burst 1 succeeded, burst 2 succeeded, all four stations powered on.
  • Supervisor diagnostics captured 1 burst started, 1 burst completed, 8 station attempts, 8 successes, 0 timeouts, and session completion.
  • Instrumented Configurator Scan completed: 4 base stations found, 4 shown.
  • Diagnostic package created at %TEMP%\PimaxVrcSupervisorDiagnostics\BaseStationStartup-20260616-235605.

Next phase

Phase 29B should analyze repeated base-station startup sessions and determine the discovery/readiness failure stage.

@Zaknin
Zaknin marked this pull request as ready for review June 16, 2026 20:09
@Zaknin
Zaknin merged commit d347151 into main Jun 16, 2026
1 check passed
@Zaknin
Zaknin deleted the phase/29a-base-station-startup-instrumentation branch July 27, 2026 17:07
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