We can delete line 46 and 47 and update line 45 to } TUBE_SET_AUTO_EJECT_STATE_T;

add a space between message and )

Change // to /// for doxygen to catch it.

Can we remove the parameter and switch directly over currentPostTreatmentState?

The public function name signalGoToInstallState is not sufficiently descriptive. Would renaming it to requestTubeSetInstall() be appropriate?

I prefer you create a local BOOL result so you can have a simpler return statement.

Should go to your new complete state here.

The number looks incorrect. This was an issue in the staging branch, which is fixed now. Can you please pull latest staging and update the values of your enums?

check alarm status instead of switch state.

Uncommented

Remove extra blank line.

Why is this service prefixed with "Tx"?
Also, Praneeth and I agreed to put these state machine services into a sub-folder of Services. See how he did it and do the same.

DrySelfTests unit.

DrySelfTests unit initialized.

Remove extra blank line.

.. handles dry self-tests execution for pre-treatment and treatment modes.

Should be in "", not <>. And should be placed alphabetically down before TxParams.h.

Calling signal function here seems circular (that's how we got here in the first place).
Shouldn't we set state to the install state here instead?

Put before TxParams.

I think Praneeth created a function for this. Make sure they're identical.

Is the folder name necessary?

Remove unnecessary spaces before =

Align =

Align =.