feat(ir): add Tag Tinker ESL app by sosnek · Pull Request #2856 · BruceDevices/firmware

sosnek · 2026-09-03T23:02:26Z

Proposed Changes

Add a 1:1 user-facing port of TagTinker (I12BP8) so Bruce can talk to infrared electronic shelf labels the same way the Flipper app does.

The app lives under Infrared → Tag Tinker (src/modules/ir/esl/). Wire protocol is vendored TagTinker C (esl_proto.h only). UI, storage, NFC decode, Set Image / Set Text, LED Test, broadcast Change Page / Diagnostic Page, and WiFi Plugins (Bruce STA → https://tagtinker.jhackerr.workers.dev) match Flipper labels and behavior. BLE phone-sync and the companion-UART path are out of scope.

Small core touch: ScrollableTextArea::show() also dismisses on Esc so About Back works. Tag Tinker raises loopTask to 16 KB via the Arduino stack hook so nested IR menus + WiFi scan do not Cache/MMU-fault on ESP32-S3.

Types of Changes

New feature (Infrared → Tag Tinker)
Minor core UX fix (scrollable text dismisses on Esc)

Verification

Host tests (no hardware):

cd tools/esl_host_tests && make test

Device: LilyGo T-Embed CC1101, SmartTAG Color 2.6.

Main / Settings / About (credits + description)
Targeted: Type Barcode, Scan NFC (PN532 prefix pages 0–11), Rename, Tag Info
Set Image (296x152 BMP), Set Text, LED Test
Broadcast Change Page / Diagnostic Page
WiFi Plugins: connect, Refresh catalog, render Crypto Price → Page / Transmit
Esc aborts TX and dismisses About

pio run -e lilygo-t-embed-cc1101 (full, non-LITE) and flash to /dev/cu.usbmodem2101.

Testing

Covered by tools/esl_host_tests (BMP, PP4, TX sequences, store, NFC barcode decode, text, WiFi manifest/header parse, menu labels). Network and IR TX are device-verified on T-Embed as above; host tests do not hit the radio or the worker.

Linked Issues

Closes #2686.

User-Facing Change

Infrared: add Tag Tinker (Flipper-faithful ESL app). Send images, text, LED test, broadcast pages, NFC scan, and WiFi Plugins to supported infrared shelf labels. About now dismisses on Back/Esc.

Further Comments

Credits in-app: Developed by I12BP8, Ported by sosnek, Research by furrtek, NFC by 7h30th3r0n3.

Use only on tags you own or are authorized to test.

Spec for a faithful port of TagTinker's IR ESL transmit flow onto Bruce ESP32 firmware: vendored wire protocol from TagTinker PR BruceDevices#53, a new ESP-IDF v5 RMT PP4 IR driver (~1.25 MHz hardware carrier), and an image TX UX. Verification target is the SmartTAG Color 2.6 (type 1626) on a LilyGo T-Embed CC1101. M0 (IR first-light) and M1 (image TX) specced in detail; M2-M5 recorded as an approved roadmap. Pins the load-bearing details that a first implementation attempt would otherwise get wrong: exact per-stage frame repeat counts, the extern "C" linkage rule for the vendored C core, RMT idle level (IR LED must idle off), the copy encoder and carrier polarity, and the encode-fully-to-RAM ordering invariant that keeps SD reads from straddling IR pin setup.

Seven TDD tasks. The protocol core, PP4 symbol timing, frame sequencing and BMP pixel mapping are all pure C with no hardware dependency, so they are unit-tested on the host with cc; only the RMT driver and the Bruce UI need the device. Golden test vectors were generated by compiling the real PR BruceDevices#53 protocol on the host, so frame bytes and CRCs are measured rather than derived by hand. Also corrects two things in the spec that this planning work uncovered: - The vendored files must live in esl/protocol/ with the compat shim at esl/tagtinker_app.h, because tagtinker_proto.c includes "../tagtinker_app.h" and that path has to resolve to the shim. - The 80 MHz tick values were slightly wrong ({4840, 19352, 9676, 14512}). Derived properly they are burst 3226 and gaps {4838, 19353, 9676, 14515}, and the plan computes them from the 64 MHz cycle counts at compile time so no converted constant is ever hardcoded. Co-authored-by: Cursor <cursoragent@cursor.com>