dangernoodle-io / taipan-cli / 24962625066

67%

push

github

jae/factory ota fix (#16)

* chore(deps): bump espflasher fork to pogopin's pin

Pin tinygo.org/x/espflasher replace directive to commit
0b39023b136a (jae/auto-reset branch) to match pogopin's
working pin. Newer than the previous pin and includes
serial reset behavior changes used by the flash subcommand.

Co-Authored-By: Claude Opus 4.7 <noreply@anthropic.com>

* fix(flash): use factory image by default; distinguish app-only OTA

GitHub releases publish two assets per board: a full factory image
(taipanminer-<board>-factory.bin: bootloader + partition table +
otadata + app at 0x20000) and an OTA app-only image
(taipanminer-<board>.bin: app starting at offset 0). The previous
asset selector used `strings.Contains` and picked whichever
sorted first, then unconditionally parsed the app descriptor at
offset 0x20 — which is only correct for app-only images. Live
flashes against fresh tdongle-s3 boards failed with
"invalid app descriptor magic: got 0x00000050".

- release.go: exact-match asset selection by name. Adds a
  --factory/--ota suggestion when the requested type is missing
  but the other exists.
- appdesc.go: parses at 0x20 (OTA) or 0x20020 (factory) based on
  caller intent; on magic mismatch, probes the alternate offset
  and suggests the matching flag in the error.
- precheck.go: passes factory through to ParseFirmwareInfo; size
  check uses the app slice (file - 0x20000) for factory.
- flash.go: FlashOptions.Factory branches the image plan. Factory
  image flashes at offset 0x0 (NVS at 0x9000 overlays the empty
  region); OTA keeps the existing otadata-erase + app-at-0x20000
  layout.
- cli/flash.go: --ota opts into the smaller app-only image for
  already-provisioned devices. Factory is the default and works
  on fresh boards.

Closes T1-43.

Co-Authored-By: Claude Opus 4.7 <noreply@anthropic.com>

---------

Co-authored-by: Claude Opus 4.7 <noreply@anthropic.com>

65 of 98 new or added lines in 5 files covered. (66.33%)

783 of 2004 relevant lines covered (39.07%)

0.45 hits per line