Plane: quadplane: transition: only set TECS throttle min in auto throttle modes

[IamPete1]

Summary

Fixes #33871

This changes to only set TECS throttle min in auto throttle modes where TECS is run. Currently thottle min is set in all cases, if there transition is into a mode which does not use TESC that throttle min does not apply until the next mode change.

Classification & Testing (check all that apply and add your own)

• Checked by a human programmer
• Non-functional change
• No-binary change
• Infrastructure change (e.g. unit tests, helper scripts)
• Automated test(s) verify changes (e.g. unit test, autotest)
• Tested manually, description below (e.g. SITL)
• Tested on hardware
• Logs attached
• Logs available on request

Description

[tridge]

Automated review note — AI-generated (Claude), cross-checked by an independent second model against the live diff. Please sanity-check before acting.

Reviewed at head bd4d6be04e.

Verdict: COMMENT

• NOTE — Tools/autotest/quadplane.py:58 — Test docstring is copy-pasted from a roll test. DetectThrottleSpike's docstring reads "Checks for steps in roll demand" — it checks throttle (servo3_raw).

• NOTE — Tools/autotest/quadplane.py:73 — Raises bare Exception where the file uses NotAchievedException. raise Exception("Throttle spike"), while hook_removed() eight lines earlier correctly raises NotAchievedException, as does the rest of quadplane.py.

Full report: https://uav.tridgell.net/DevCallReviews/2026_08_11/devcall_pr_reviews.html

[IamPete1]

I have fixed both the AI reported CI issues.

[rmackay9]

I'm not really qualified but it looks OK to me

[peterbarker]

LGTM

[peterbarker]

CI failed the QAUTOTUNE thing. Candidate fix is at #33628 Restarting it