fix(ports): never hop to default when stored port busy; prefer stored port and let bind micro-retry handle release to avoid port swapping on recompiles

dsarno · dsarno · commit f4bc7cd4fded · 2025-08-08T08:37:02.000-07:00
diff --git a/UnityMcpBridge/Editor/Helpers/PortManager.cs b/UnityMcpBridge/Editor/Helpers/PortManager.cs
@@ -53,6 +53,8 @@ public static int GetPortWithFallback()
                     Debug.Log($"Stored port {storedConfig.unity_port} became available after short wait");
                     return storedConfig.unity_port;
                 }
+                // Prefer sticking to the same port; let the caller handle bind retries/fallbacks
+                return storedConfig.unity_port;
             }
 
             // If no valid stored port, find a new one and save it

Original file line number	Diff line number	Diff line change
`@@ -53,6 +53,8 @@ public static int GetPortWithFallback()`
`53`	`53`	`Debug.Log($"Stored port {storedConfig.unity_port} became available after short wait");`
`54`	`54`	`return storedConfig.unity_port;`
`55`	`55`	`}`
	`56`	`+ // Prefer sticking to the same port; let the caller handle bind retries/fallbacks`
	`57`	`+ return storedConfig.unity_port;`
`56`	`58`	`}`
`57`	`59`
`58`	`60`	`// If no valid stored port, find a new one and save it`