Address code review nitpicks

- Remove unnecessary empty line in checkSshReachability loop
- Add REUSED_SSH_CHECK_TIMEOUT_RATIO constant for timeout allocation

Co-authored-by: tdgroot <1165302+tdgroot@users.noreply.github.com>

Author: Copilot

src/Brancher/BrancherHypernodeManager.php

@@ -14,6 +14,12 @@
 
 class BrancherHypernodeManager
 {
+    /**
+     * Ratio of timeout to allocate for initial SSH check when reusing a Brancher.
+     * The remaining time is reserved for logbook flow checks if SSH fails.
+     */
+    private const REUSED_SSH_CHECK_TIMEOUT_RATIO = 0.5;
+
     private LoggerInterface $log;
     private HypernodeClient $hypernodeClient;
 
@@ -121,7 +127,6 @@ private function checkSshReachability(
         $consecutiveSuccesses = 0;
 
         while ((microtime(true) - $startTime) < $timeout) {
-
             $connection = @fsockopen(sprintf("%s.hypernode.io", $brancherHypernode), 22);
             if ($connection) {
                 fclose($connection);
@@ -234,8 +239,8 @@ private function waitForAvailabilityInternal(
                 )
             );
 
-            // Allocate 50% of timeout for initial SSH check to leave time for fallback
-            $sshCheckTimeout = (int) ($timeout * 0.5);
+            // Allocate a portion of timeout for initial SSH check to leave time for fallback
+            $sshCheckTimeout = (int) ($timeout * self::REUSED_SSH_CHECK_TIMEOUT_RATIO);
             $sshCheckStartTime = microtime(true);
 
             if ($this->checkSshReachability($brancherHypernode, $sshCheckTimeout, $reachabilityCheckCount, $reachabilityCheckInterval)) {