At some point, the MAS migration broke due to what I assume is a change in behaviour of fs.cpSync. To copy directories now, you need to include recursive: true as an option. This is likely something that changed in macOS, as Node doesn't seem to make any mention of this.
While investigating and fixing the issue, I ran into a case where the app can crash due to certain singletons that might try to register a duplicate handler.
Summary
At some point, the MAS migration broke due to what I assume is a change in behaviour of
fs.cpSync. To copy directories now, you need to includerecursive: trueas an option. This is likely something that changed in macOS, as Node doesn't seem to make any mention of this.While investigating and fixing the issue, I ran into a case where the app can crash due to certain singletons that might try to register a duplicate handler.
This PR fixes both of the above issues.
Ticket Link
https://mattermost.atlassian.net/browse/MM-60080