Merge plan for the open pull requests #119

The order for handling the open pull requests. Within a phase, merge one PR, update the next one against develop (fixing the conflicts it now has), then merge that. Most of these PRs touch src/app.cpp and include/app.hpp, so merging several at once only creates more conflicts.

Phase 1: Szello field fixes

All four were running in the Szello debug build.

• Parse each AOG UDP datagram on its own so short packets do not drop the next one #107 UDP parser: review and merge. It conflicts with none of Switch section control on automatically when an implement connects #102, Reconnect implements that missed a TC restart #105, Keep implements in automatic mode on AOG's current section demand #106 or Add Tractor Facilities (PGN 65033) for the TECU #82.
• Reconnect implements that missed a TC restart #105 TC restart recovery: review and merge. Confirmed in the field run on 14 Oct.
• Keep implements in automatic mode on AOG's current section demand #106 Section setpoint refresh: mark ready, update after Reconnect implements that missed a TC restart #105, review and merge.
  • Behaviour change: with AOG's section Auto off, the implement is now told to turn off.
• Switch section control on automatically when an implement connects #102 Section control on connect / field open: update after Keep implements in automatic mode on AOG's current section demand #106, review and merge.
• Cut log noise: duplicate UDP warnings, global-destination RTS, non-section clients #98 Log noise: rebuild the UDP part on Parse each AOG UDP datagram on its own so short packets do not drop the next one #107's parse_datagram(), keeping the duplicate filter for the second socket.
  • Reply to the index = 0 review comment: Parse each AOG UDP datagram on its own so short packets do not drop the next one #107 fixes it.
• Bump AgIsoStack++ to upstream main 02f3fe9 #120 AgIsoStack bump to upstream 02f3fe9 (brings Take a CF offline only after two missed requests for address claim Open-Agriculture/AgIsoStack-plus-plus#834 and Ignore short TC process data frames and truncated DET objects Open-Agriculture/AgIsoStack-plus-plus#857, among others): bench run, then merge just before the release.
  • Add the candleLight/gs_usb adapter option on Windows #121 candleLight/gs_usb adapter option, stacked on Bump AgIsoStack++ to upstream main 02f3fe9 #120: test with a real adapter, then merge after Bump AgIsoStack++ to upstream main 02f3fe9 #120.
• Upstream AgIsoStack Ignore TC server messages that have no source control function Open-Agriculture/AgIsoStack-plus-plus#784 (crash guard for messages with no source): mark ready and get it merged, then bump the pin again. Bump AgIsoStack++ to upstream main 02f3fe9 #120 does not include it.
• Make the TC number configurable, with automatic selection #123 TC number setting: test on a bus if a tractor with its own TC is available, review and merge.
• Release with Phase 1, then delete test/szello-debug.

Phase 2: TECU stack (#82 → #118 → #122)

• Add Tractor Facilities (PGN 65033) for the TECU #82 Tractor Facilities: update after Phase 1, since it conflicts with Cut log noise: duplicate UDP warnings, global-destination RTS, non-section clients #98 and Switch section control on automatically when an implement connects #102.
  • Fix the PGN 65032 comment, which says "not gated by log level".
  • Review and merge.
• Run the TECU as a secondary Tractor ECU when the tractor has one #118 Secondary TECU (gap filling): field test at Szello, which has a John Deere TECU and tecuEnabled off.
  • Check the log for "Requested tractor facilities", then the "Sending / Not sending" lines.
  • Then mark it ready, review and merge.
• Answer requests for Time/Date (PGN 65254) from the TECU #122 Time/Date on request (replaces Broadcast Time/Date (PGN 65254, FEE6) from the TECU #87, which is closed): test on a bus, review and merge after Run the TECU as a secondary Tractor ECU when the tractor has one #118.
  • It also carries the fix for Run the TECU as a secondary Tractor ECU when the tractor has one #118's secondary TECU duplicating the tractor's messages for up to 2.5 s after start. That commit can move into Run the TECU as a secondary Tractor ECU when the tractor has one #118.

Phase 3: Track stack (#84 → #85)

• Announce guidance track data (DDI 507-511, 513) to implements #84 Guidance track data: merge develop in, review and merge.
  • Switch section control on automatically when an implement connects #102 and Announce guidance track data (DDI 507-511, 513) to implements #84 both handle PGN 0xF3. Git reports this as a conflict on Announce guidance track data (DDI 507-511, 513) to implements #84: resolve it to one branch that does both jobs (register the field for the track code, and call on_aog_field_opened()). Don't keep two branches, because the second never runs. The branch test/upcoming-integration has the combined version.
  • It conflicts in 6 files since Send GNSS quality (DDI 514) to implements #83 was squash-merged, and clashes with almost every other PR, so do it once, after Phases 1 and 2.
• Add track control level negotiation (DDI 505/506) and track control state (DDI 515) #85 Track control levels: update on Announce guidance track data (DDI 507-511, 513) to implements #84, test with TRACK hardware, review and merge.

Phase 4: Drafts that need work first

• Resolve section control targets from linked process data only #111 Section control from linked DDIs: fix the failing section_control_method test.
  • It expects DDI 141 for the Kuhn Espro fixtures and finds none.
  • Also resolve the CMakeLists.txt conflict.
• Support dedicated-parent section feedback and pace measurement subscriptions #76 Kverneland section feedback: check what is still needed after Cache element parents for the section state lookup #103, then ask the author to update it, or update it for them.
• Add on-demand hydrated DDOP snapshots for debugging #75 DDOP hydration snapshot: decide whether it's still wanted, then update it or close it.

Phase 5: Close or decide

• Implement TECU capabilities announcement and tramline support for specific seeders #73 tramline: closed, since Add Tractor Facilities (PGN 65033) for the TECU #82–Add track control level negotiation (DDI 505/506) and track control state (DDI 515) #85 and Answer requests for Time/Date (PGN 65254) from the TECU #122 replace it. The tag backup/tramline-pre-split-20260919 keeps its full state.
• Vader tempo having multiple booms #29 vader_tempo: closed.

🤖 Generated with Claude Code