Skip to content

P0 UI: suppress historical SV fault alarms on startup; add true-color COM board indicator - #328

Merged
masarray merged 14 commits into
mainfrom
fix/p0-stale-stop-ui-board-status-20261025
Oct 25, 2026
Merged

masarray merged 14 commits into
mainfrom
fix/p0-stale-stop-ui-board-status-20261025

Conversation

@masarray

Copy link
Copy Markdown
Owner

Problem reproduced

At Studio launch, ESP32-P4 can be healthy, idle and verified while SVLIFE state=STOPPED stop_cause=RUNTIME_FAULT refers to an older hardware stop in board RAM. Existing DeviceController::processLine promoted this latched past cause into svStopReason, causing a frightening red center toast and red footer even though no SV ran/stopped in the current Studio session.

Fix

  • Separate board historical last stop cause (svLastStopCause) from active unexpected STOP observed as RUNNING → STOPPED in the present Studio connection (svUnexpectedStopObserved).
  • Preserve original fail-closed real hardware stop behavior; support STOP cause arriving before/after the running transition; de-duplicate one toast per observed incident and never warn on startup, replayed SHOW, old SVTXFAULT, operator STOP or reconnect.
  • Add small BoardConnectionIndicator.qml at the far left of the footer: discreet outlined LED + COM port + status caption. Gray DISCONNECTED, blue CONNECTING, amber PREPARING, green READY/RUNNING, red ERROR only for active stop/setup error. Drawn from current serial connection, board identity and supervisor—not stale history. Tooltip includes useful grounded next-step context.
  • Strict Qt native deterministic harness regressions for stale STOP followed by genuine fault and manual STOP; static GUI contract tests updated. No firmware/timing/SV/PTP or stable release changes.

Acceptance gates

Qt QML build, deterministic session harness, Python UI source guard, Windows native preview package and lifecycle tests. Do not merge until verified green. Field acceptance remains open in issue #324.

@chatgpt-codex-connector

Copy link
Copy Markdown

Codex Review Summary

This comment shows the latest Codex review activity on this pull request.

Review Status Commit Review trigger
📝 Code Review 🔄 Running since 2026-10-25T00:05:38.355023Z aa0fb02 PR opened
ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review" or "@codex security review".

Codex reacts with 👀 while any review is running, comments if it has suggestions, and reacts with 👍 once all reviews finish with no findings.

@masarray
masarray merged commit c8d351f into main Oct 25, 2026
11 checks passed
masarray added a commit that referenced this pull request Oct 25, 2026
…ble (#329)

Publish exact tested Windows Studio/ESP32-P4 preview from PR #328 only as GitHub prerelease, checking artifact hashes and firmware provenance; retain immutable M3.H.1 public stable. Document physical receiver acceptance blocker and branch ruleset governance gate.
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