slave2 is defined with no length (i.e default length 1), but maybe that’s a copy/paste error ?
coil type is by definition bit, so valuetype=bit is redundant here. I think I may have seen that cause issues before, best just leave it out.
There is a kind of config cache, so once you have edited modbus.cfg to your satisfaction, follow -