fix(cli): drop duplicate dc.gate handlers that shadowed the on/off fix

[Elektr0Vodka]

Follow-up to #10. On dmc-observer-dev, set dc.gate on still disables duty-cycle region gating, even though #9/#10 fixed the parser.

Problem

src/helpers/CommonCLI.cpp on the observer contains the dc.gate / dc.gate.thresh / dc.gate.hyst set handlers twice in the same else if chain, and the three get readbacks twice as well. This is a leftover from the 2026-09-11 dmc-dev → observer merge (the same merge that duplicated the docs chapter #10 removed).

• The first copy, right after handleObserverSetCmd, is the observer's original and still does dc_gate_enabled = atoi(...) != 0.
• The second copy, after allow.read.only, came from dmc-dev and has the on/off fix.

The first copy matches first, so the fix is dead code. set dc.gate on replies OK and turns gating off. 1/0 work, which is why the Toolbox is unaffected. dmc-dev has each handler only once and is not affected.

Found while verifying the observer-mqtt-dev 9805406 release notes; that release lists this as a known issue.

Change

Remove the observer-original copy (set + get, 34 lines) and keep the copy that is identical to dmc-dev. The set and get chains now match dmc-dev line for line, so future dmc-dev merges into the observer won't conflict on these lines again. handleObserverSetCmd / handleObserverGetCmd don't handle any dc.gate key, so nothing earlier in the chain intercepts them.

Testing

• pio run -e heltec_v4_repeater_observer_mqtt, heltec_v4_room_server_observer_mqtt and RAK_4631_repeater build.
• Diffed the allow.read.only → dc.gate set/get chains against origin/dmc-dev: identical.
• Not tested on a device. CommonCLI.cpp isn't reachable from the native test envs.

🤖 Generated with Claude Code