Repository navigation
epic: Heltec V4-R2 SAW diag builds; device identities move out of the public repo (#1373) - #1710
Merged
Merged
Conversation
The V4 was the only board family with no _diag envs. Adds [heltec_v4_diag] -- boot beacon, UART0 log mirror on GPIO43, forced caplog, the same set as the R8 (#1342) -- and _diag twins for the released OLED roles: companion BLE and USB (with power telemetry), repeater, room server and observer. monitor_rts/dtr = 0 per diag env, leaving existing envs untouched. The repeater diag twin inherits the generic identity the base env carries since #1378. Part of #1373. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
…nd\platformio.local.ini platformio.ini includes C:\Dev\.offband\platformio.local.ini when it exists, beside the device registry (#1012), so owner device names, locations and node ids are never pushed and every worktree shares one copy. Hosts without it, including CI, skip it. heltec_v4_repeater_telemetry becomes generic (node id heltec-v4-repeater) and stays in CI. heltec_v4_repeater_telemetry_stp leaves the repo; it, the patio env and the R2 SAW env are defined in the local file. Part of #1373. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
This was referenced Oct 21, 2026
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Epic #1373: Heltec V4-R2 SAW: diagnostic builds and device identities out of the public repo. Merge held for your approval.
_diagenvs: companion BLE/USB, repeater, room server, observer. Each extends its shipped env and adds only instrumentation (boot beacon, UART log mirror, forced caplog).variants/heltec_v4/platformio.ini.heltec_v4_repeater_telemetrybecomes generic, and the ST-P lab env is removed from the public file. The rootplatformio.inigains${sysenv.SystemDrive}/Dev/.offband/platformio.local.iniinextra_configs, so per-host device envs (patio, ST-P, R2 SAW) live beside the device registry (#1012), outside every checkout. A missing path is skipped, the same pattern as the existing${sysenv.PIO_SECRETS_FILE}entry, so CI and other hosts build without it.The R2 (
hv4-r2-saw-1) has run the #1410 build (82fd7a1b) since 2026-10-13.Verified after rebasing onto firmware-base on 2026-10-21:
_diagenvs, and the three per-host envs through the new include.pio test -e nativepasses 1351/1351.check_env_capability_claims(200 claims).SystemDriveis unset. It also flagged asrc/Utils.cppchange; that came from my diff command picking up a newer base commit (task: patch MACThenDecrypt with the block-alignment check, with a native test #1586), and this branch does not touch that file.Privacy note (#1410): this stops new exposure only. The removed name and coordinates remain in this repository's history and in published release assets; what to do about those is the owner's decision.
Not in this PR:
Part of #1373. Carried by #1377.
Agent: BoldCreek (session a946d569)
🤖 Generated with Claude Code