fix(solidity): move basic-oracle-manipulation ruleid annotation to the matching line

The `pattern: $X.div($Y)` match for `underlyingUnit.mul(balanceWithInvested()).div(totalSupply())`
is textually on the line with the `:` (else) branch of the ternary, not the
line where the enclosing assignment starts. Confirmed by parsing this
snippet directly: the `.div(...)` call_expression's span is entirely on
that line. The `// ruleid:` annotation was one line above the wrong branch,
which happened to line up with a less precise (pre-tree-sitter-solidity
v1.2.13) parse of this multi-line ternary. Move the annotation immediately
above the branch actually being matched.

Surfaced by semgrep/ocaml-tree-sitter-semgrep#626 (tree-sitter-solidity
bump to v1.2.13), which produces the correct AST here.

Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>

Author: malaverdiere

solidity/security/basic-oracle-manipulation.sol

@@ -97,9 +97,9 @@ contract OneRingVault is ERC20Upgradeable, OwnableUpgradeable {
         balance = IStrategy(activeStrategy).investedBalanceInUSD();
     }
     function getSharePrice() public view returns (uint256 _sharePrice) {
-        // ruleid: basic-oracle-manipulation
         _sharePrice = totalSupply() == 0
             ? underlyingUnit
+            // ruleid: basic-oracle-manipulation
             : underlyingUnit.mul(balanceWithInvested()).div(totalSupply());
 
         if (_sharePrice < underlyingUnit) {