fix(multiwindow): schrap kInvalidArguments-melding bij opstart en presentatie #1094

Merged

brenno merged 1 commit from fix/multiwindow-onwindowschanged-guard into main

2026-08-01 21:55:17 +00:00

brenno commented

2026-08-01 21:47:15 +00:00

Owner

Wat

De gevendorde desktop_multi_window-fork stuurde bij vensterregistratie (AttachWindow) en vensteraanmaak (CreateWindow) een onWindowsChanged-bericht naar het venster dat op dat moment nog geen platform-message-handler heeft. Op macOS gaf dat als eerste logregel bij het opstarten:

'FlutterEngineSendPlatformMessage' returned 'kInvalidArguments'. Invalid engine handle.
Failed to send message to Flutter engine on channel 'mixin.one/desktop_multi_window'.

Fix

notifyWindowsChanged(excluding:) slaat nu het zojuist geregistreerde/aangemaakte venster over. Veilig omdat een venster zijn eigen aanmaak-event niet nodig heeft (het bevraagt getAllWindows op verzoek) en OciDeck nergens naar onWindowsChanged luistert. Andere bestaande vensters worden ongewijzigd genotificeerd.

Verificatie

Native compilatie: flutter build macos --debug slaagt.
Draaiende macOS-build: de melding is over 5 schone starts volledig verdwenen (0 matches), app start en rendert normaal.
Een Dart-test kan deze native race niet dekken; de compile-gate en de SBOM-boomhashpoort bewaken de fork.

Fork eerlijk gehouden

Notice-header (Apache-2.0 §4b), MODIFICATIONS.md en de SBOM-boomhash bijgewerkt (make sbom-verify groen).

🤖 Generated with Claude Code

## Wat De gevendorde `desktop_multi_window`-fork stuurde bij **vensterregistratie** (`AttachWindow`) en **vensteraanmaak** (`CreateWindow`) een `onWindowsChanged`-bericht naar het venster dat op dat moment nog geen platform-message-handler heeft. Op macOS gaf dat als eerste logregel bij het opstarten: ``` 'FlutterEngineSendPlatformMessage' returned 'kInvalidArguments'. Invalid engine handle. Failed to send message to Flutter engine on channel 'mixin.one/desktop_multi_window'. ``` ## Fix `notifyWindowsChanged(excluding:)` slaat nu het zojuist geregistreerde/aangemaakte venster over. Veilig omdat een venster zijn eigen aanmaak-event niet nodig heeft (het bevraagt `getAllWindows` op verzoek) en OciDeck nergens naar `onWindowsChanged` luistert. Andere bestaande vensters worden ongewijzigd genotificeerd. ## Verificatie - Native compilatie: `flutter build macos --debug` slaagt. - Draaiende macOS-build: de melding is over **5 schone starts volledig verdwenen** (0 matches), app start en rendert normaal. - Een Dart-test kan deze native race niet dekken; de compile-gate en de SBOM-boomhashpoort bewaken de fork. ## Fork eerlijk gehouden Notice-header (Apache-2.0 §4b), `MODIFICATIONS.md` en de SBOM-boomhash bijgewerkt (`make sbom-verify` groen). 🤖 Generated with [Claude Code](https://claude.com/claude-code)

brenno added 1 commit

2026-08-01 21:47:15 +00:00

fix(multiwindow): schrap kInvalidArguments-melding bij opstart en presentatie

scans / scans (pull_request) Successful in 3m45s

Details

218d02346e

De gevendorde desktop_multi_window-fork stuurde bij vensterregistratie
(AttachWindow) en vensteraanmaak (CreateWindow) een onWindowsChanged-bericht
naar het venster dat op dat moment nog geen platform-message-handler heeft
geïnstalleerd. Dat gaf op macOS als eerste logregel bij het opstarten:

  'FlutterEngineSendPlatformMessage' returned 'kInvalidArguments'.
  Invalid engine handle.
  Failed to send message to Flutter engine on channel
  'mixin.one/desktop_multi_window'.

notifyWindowsChanged(excluding:) slaat nu het zojuist geregistreerde/aangemaakte
venster over — dat heeft zijn eigen aanmaak-event niet nodig (het bevraagt
getAllWindows op verzoek), en OciDeck luistert nergens naar onWindowsChanged.
Andere reeds bestaande vensters worden ongewijzigd genotificeerd.

Geverifieerd op een draaiende macOS-build: de melding is over 5 schone starts
volledig verdwenen (0 matches), de app start en rendert normaal.

Fork-notice (Apache-2.0 §4b), MODIFICATIONS.md en de SBOM-boomhash bijgewerkt.

Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>

brenno merged commit 8a005bd26f into main

2026-08-01 21:55:17 +00:00

brenno referenced this pull request from a commit

2026-08-01 21:55:17 +00:00

Merge pull request 'fix(multiwindow): schrap kInvalidArguments-melding bij opstart en presentatie' (#1094) from fix/multiwindow-onwindowschanged-guard into main

brenno deleted branch fix/multiwindow-onwindowschanged-guard

2026-08-01 21:55:17 +00:00