You don't necessarily need the contact sensor in the trigger; you just need a way to evaluate when you should stop the repeat, with the trick being that that evaluation actually needs to be run as part of the repeat. One of your rules actually looks like it should work and I'm not sure why it wouldn't, but I should note that "Stop Repeating Actions" would be sufficient, and you don't need "Cancel Timed Actions" which does that (I think...if not it would certainly explain this) plus a lot more that may have undesired side effects in more complicated rules.
Every time I see posts like this, I like to refer to this post from Bruce (Rule Machine author) about repeated notifications in Rule Machine, showing a few different ways it can be done. Perhaps one can be of inspiration to you: