Skip to content

feat: enrich launch diagnostics and classify near-target health#145

Merged
papi-ux merged 5 commits into
masterfrom
feat/heads-up-diagnostics-detail
Jul 13, 2026
Merged

feat: enrich launch diagnostics and classify near-target health#145
papi-ux merged 5 commits into
masterfrom
feat/heads-up-diagnostics-detail

Conversation

@papi-ux

@papi-ux papi-ux commented Jul 13, 2026

Copy link
Copy Markdown
Owner

Summary

  • enrich game-detail Heads Up diagnostics with launch-profile provenance and structured health evidence
  • classify HEALTHY only when grade, strict finite profile/telemetry evidence, target association, pacing, lifecycle, issue agreement, and all risk fields agree
  • keep trial, recovery, learning, manual, upgrade, downshifted, missing, and unknown states fail-closed as warnings
  • render finite-only diagnostic history and use semantic nova_success only for healthy state
  • keep compact first paint while exposing expandable details
  • wire deterministic controller focus: Play Up → More details, More details Down → Play

Exact review identity

  • PR head: 2e8111a10ffc3f91d05e849ac47ee5eafc64edfe
  • complete five-file diff SHA-256: b9815a68a793bf5d7534a37b1db75c720e359f3e8836e8b06bd69dc2250efc14
  • exact read-only Codex review: pass, zero security or logic concerns
  • added-line security scan: clean

Local verification

  • JVM unit tests: 807/807 passed
  • CI-equivalent x86_64 JVM lane: passed
  • Android lint: passed
  • debug APK assembly: arm64-v8a, armeabi-v7a, and x86_64 passed
  • git diff --check: passed

Physical Retroid Pocket 6 verification

  • device serial: 98ea526f
  • behavioral Compose instrumentation: 1/1 passed
    • exits Android touch mode with a real DPAD event
    • observes Play focus
    • sends real DPAD Up and observes More details focus
    • sends real DPAD Center and verifies expansion
    • verifies expanded content remains within a 320dp compact root
  • final ARM64 APK package/version: com.papi.nova.debug / 1.3.1 (33)
  • final ARM64 APK SHA-256: 052653c90b64e54cfaf69a692c80d22304faf9c6a89814c2e1d03bde33632f7a
  • v1/v2 signatures verified
  • installed APK pullback matched byte-for-byte
  • runtime settled at NovaWelcomeActivity; zero fatal/ANR/native-abort markers
  • visual frame clean; device restored to Home
  • no game or stream launched

GitHub CI for exact head

  • Lint and unit test: passed
  • Analyze Java/Kotlin: passed
  • CodeQL: passed
  • public-hygiene: passed
  • Assemble release APK: passed
  • Emulator smoke test: intentionally skipped by the PR workflow

The initial lint/unit run hit the unrelated timing-sensitive NovaUpdateRecoveryTest.cancellationAfterAtomicCommitDeletesFinalArtifact. The exact test passed 10/10 fresh local repetitions, the complete x86_64 JVM lane passed, and the unchanged-SHA failed-job rerun passed.

Scope

  • Nova client only
  • no Polaris deployment, service restart, release, merge, game launch, or stream launch

@papi-ux
papi-ux marked this pull request as draft July 13, 2026 19:54
@papi-ux
papi-ux marked this pull request as ready for review July 13, 2026 21:44
@papi-ux
papi-ux merged commit f99ffa7 into master Jul 13, 2026
8 of 9 checks passed
@papi-ux
papi-ux deleted the feat/heads-up-diagnostics-detail branch July 13, 2026 22:03
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