fix: add diagnostic logging for window position drift on Windows multi-monitor

Temporary diagnostic logging to understand the exact coordinate behavior
during show/hide/resize cycles on Windows with mixed DPI monitors.

Logs tagged [PI-DIAG] and [NL-DIAG] show:
- Saved position vs getBounds() at each step
- Position changes after setBounds, show, setAlwaysOnTop, focus
- Display info including scaleFactor

Author: edospadoni

src/main/classes/controllers/PhoneIslandController.ts

@@ -24,7 +24,10 @@ export class PhoneIslandController {
       if (window) {
         const bounds = window.getBounds()
         if (bounds.height !== h || bounds.width !== w) {
+          Log.info('[PI-DIAG] resize: before setBounds getBounds=', JSON.stringify(bounds), 'requestedSize=', JSON.stringify({ w, h }))
           window.setBounds({ width: w, height: h })
+          const afterBounds = window.getBounds()
+          Log.info('[PI-DIAG] resize: after setBounds({w,h}) getBounds=', JSON.stringify(afterBounds))
           PhoneIslandWindow.currentSize = { width: w, height: h }
         }
         //make sure the size is equal to [0,0] when you want to close the phone island, otherwise the size will not close and will generate slowness problems.
@@ -33,8 +36,12 @@ export class PhoneIslandController {
         } else {
           // Don't show window during warm-up
           if (!window.isVisible() && !this.isWarmingUp) {
+            const beforeShow = window.getBounds()
             window.show()
+            const afterShow = window.getBounds()
             window.setAlwaysOnTop(true, 'screen-saver')
+            const afterAlwaysOnTop = window.getBounds()
+            Log.info('[PI-DIAG] resize show: beforeShow=', JSON.stringify(beforeShow), 'afterShow=', JSON.stringify(afterShow), 'afterAlwaysOnTop=', JSON.stringify(afterAlwaysOnTop))
           }
         }
       }
@@ -48,8 +55,16 @@ export class PhoneIslandController {
     try {
       const window = this.window.getWindow()
       if (window) {
+        const savedPosition = AccountController.instance.getAccountPhoneIslandPosition()
+        Log.info('[PI-DIAG] showPhoneIsland: savedPosition=', JSON.stringify(savedPosition), 'size=', JSON.stringify(size), 'getBounds_before=', JSON.stringify(window.getBounds()))
+
+        const displays = screen.getAllDisplays()
+        Log.info('[PI-DIAG] showPhoneIsland: displays=', JSON.stringify(displays.map(d => ({ id: d.id, bounds: d.bounds, workArea: d.workArea, scaleFactor: d.scaleFactor }))))
 
         this.resize(size)
+        const afterResize = window.getBounds()
+        Log.info('[PI-DIAG] showPhoneIsland: afterResize getBounds=', JSON.stringify(afterResize))
+
         if (process.platform !== 'linux') {
           const phoneIslandPosition = AccountController.instance.getAccountPhoneIslandPosition()
           if (phoneIslandPosition) {
@@ -63,8 +78,11 @@ export class PhoneIslandController {
                   (phoneIslandPosition.y + size.h) < (area.y + area.height))
               )
             }, false)
+            Log.info('[PI-DIAG] showPhoneIsland: isOnDisplay=', isPhoneIslandOnDisplay)
             if (isPhoneIslandOnDisplay) {
               window?.setBounds({ x: phoneIslandPosition.x, y: phoneIslandPosition.y }, false)
+              const afterSetPos = window.getBounds()
+              Log.info('[PI-DIAG] showPhoneIsland: after setBounds({x,y}) getBounds=', JSON.stringify(afterSetPos))
             } else {
               window?.center()
             }
@@ -85,6 +103,8 @@ export class PhoneIslandController {
     try {
       const window = this.window.getWindow()
       const phoneIslandBounds = window?.getBounds()
+      const previousSaved = AccountController.instance.getAccountPhoneIslandPosition()
+      Log.info('[PI-DIAG] hidePhoneIsland: getBounds=', JSON.stringify(phoneIslandBounds), 'previousSaved=', JSON.stringify(previousSaved))
       if (phoneIslandBounds) {
         AccountController.instance.setAccountPhoneIslandPosition({
           x: phoneIslandBounds.x,

src/main/classes/windows/NethLinkWindow.ts

@@ -71,6 +71,7 @@ export class NethLinkWindow extends BaseWindow {
   show(): void {
     try {
       const accountBounds = AccountController.instance.getAccountNethLinkBounds()
+      Log.info('[NL-DIAG] show: savedBounds=', JSON.stringify(accountBounds), 'getBounds_before=', JSON.stringify(this._window?.getBounds()))
       if (accountBounds) {
         const isAccountBoundsOnDisplay = screen.getAllDisplays().reduce((result, display) => {
           const area = display.workArea
@@ -84,16 +85,20 @@ export class NethLinkWindow extends BaseWindow {
         }, false)
         if (isAccountBoundsOnDisplay) {
           this._window?.setBounds(accountBounds, false)
+          Log.info('[NL-DIAG] show: after setBounds getBounds=', JSON.stringify(this._window?.getBounds()))
         } else {
           this._setBounds()
         }
       } else {
         this._setBounds()
       }
       super.show()
+      const afterShow = this._window?.getBounds()
       this._window?.setVisibleOnAllWorkspaces(true)
       this._window?.focus()
       this._window?.setVisibleOnAllWorkspaces(false)
+      const afterFocus = this._window?.getBounds()
+      Log.info('[NL-DIAG] show: afterShow=', JSON.stringify(afterShow), 'afterFocus=', JSON.stringify(afterFocus))
     }
     catch (e: any) {
       if (e.message === 'Object has been destroyed') {
@@ -107,6 +112,9 @@ export class NethLinkWindow extends BaseWindow {
 
   hide(..._args: any): void {
     try {
+      const boundsAtHide = this._window?.getBounds()
+      const previousSaved = AccountController.instance.getAccountNethLinkBounds()
+      Log.info('[NL-DIAG] hide: getBounds=', JSON.stringify(boundsAtHide), 'previousSaved=', JSON.stringify(previousSaved))
       this.saveBounds()
       this._window?.hide()
     } catch (e) {