The reopen event is not triggered when upgrading to 0.84.0

[imawizrd]

Issue Type

Before opening an issue, please search and see if it has already been raised.

• Bug Report

• Feature Request

• Successfully reproduced against the latest version of NW.js?

Please use our mailing list or Gitter chatroom to ask questions. The issue tracker is only for bugs and feature requests, in English only. Please note that issues without a repro or code snippet are less likely to be resolved.

Current/Missing Behavior

The reopen event is never triggered.

Expected/Proposed Behavior

Each time the user clicks the Mac OS Dock icon of the application, the reopen event should be triggered, allowing the application to decide how to handle this event. This is because sometimes the app is minimized and sometimes it's hidden, so it may require different logic.

Additional Info

• Operating System: macOS 14.13.1
• NW.js Version: 0.84.0
• Repro link: minrepro.zip
• Code snippet:
• Crash report:

Run npm i and start it with npm start, then follow these steps:

1. click the close button of the window.
2. Then click the dock icon to reopen. It should reopen.
3. Close the window with close button.
4. Click the dock icon again.
5. These errors show in console, and the window is not reopened:

[69103:259:0216/194738.214409:ERROR:api_binding_hooks.cc(236)] Check failed: false. No callback found for the specified request ID.
[69103:259:0216/194738.843692:ERROR:api_binding_hooks.cc(236)] Check failed: false. No callback found for the specified request ID.
[69103:259:0216/194739.056158:ERROR:api_binding_hooks.cc(236)] Check failed: false. No callback found for the specified request ID.
[69103:259:0216/194739.236249:ERROR:api_binding_hooks.cc(236)] Check failed: false. No callback found for the specified request ID.

6. Quit the app completely
7. Edit package.json and downgrade nw to "^0.83.0"
8. Update modules with rm -rf node_modules && npm install nw --nwjs_build_type=sdk && npm i
9. Now version 83 should be installed. Repeat steps 1-4, the console errors in step 5 don't show anymore.
10. Edit package json to upgrade to 84, the console errors in 5 show again.

[imawizrd]

This problem is still happening in latest version 0.85.0

friendly ping @rogerwang

[imawizrd]

I found a seemingly working solution:

do not use:

nw.App.on("onreopen", () => {})

instead use:

nw.App.onReopen(() => {})

I found inspiration in this comment: #7860 (comment).

The website documentation does not mention this method for adding listeners.

The Typescript compiler may generate an error message because the type definitions are outdated:

declare namespace nw {
  export interface App {
    onReopen: {
      addListener: (arg: any) => void
    }
  }
}