You're mixing datatypes; "TimesToRepeat" is a small integer, yet you're decrementing an integer by the same name (which is a functionally a different variable entirely).
There is no real benefit to using small integer over integer, so I'd recommend using regular integers (small integer may be deprecated at some point, too)
That said, rather than doing this manually, use the "Loop Start - Repeat a Certain Number of Times" action (I.E. a For loop), where you can put the token directly into the "Number of times" field and VoiceAttack will do the rest for you.