Minor Clarity Issues

[setzeus]

Description

This small PR addresses two bugs for the Clarity contracts recently opened under a single issue.

Closes: #971

Changes

Both changes mentioned in linked issue are implemented. Removed the unnecessary (ok ...) wrapper & reused the 'version-int' variable.

Testing Information

No new tests were added but all unit tests still intact.

Checklist:

• I have performed a self-review of my code
• My changes generate no new warnings
• New and existing unit tests pass locally with my changes
• Any dependent changes have been merged and published in downstream modules

[djordon]

Looks good!

But isn't there something to test here? Before the failure mode would occur if the contract caller wasn't the "protocol caller". And I think sBTC would still be being minted in that case, since that happens before the protocol caller check. I might be misreading or misunderstanding, but if not we should add an explicit test that contract calls from non-protocol callers would not lead to sBTC being minted.

[djordon]

I misread the issue, we do that check before hand so there is nothing to really test.

[djordon]

✅ Yeah looks good!