[Bug]: shell.trashItem() doesn't move file to trash when file is in OneDrive, instead it is moved to root of OneDrive

[Shurikal]

Preflight Checklist

• I have read the Contributing Guidelines for this project.
• I agree to follow the Code of Conduct that this project adheres to.
• I have searched the issue tracker for a bug report that matches the one I want to file, without success.

Electron Version

25.0.1

What operating system are you using?

macOS

Operating System Version

macOS Ventura 13.3.1 (a)

What arch are you using?

arm64 (including Apple Silicon)

Last Known Working Electron version

No response

Expected Behavior

shell.trashItem should move Item to trash

Actual Behavior

shell.trashItem moves Item to root of OneDrive folder

Testcase Gist URL

No response

Additional Information

I tried to create the Electron Fiddle app, but it creates the file in the wrong place. I have an example app which shows the problem when it is run from inside a folder in the OneDrive.

[Shurikal]

After playing a bit around with objective-c (never done that before), I think I know where the problem is:

electron/shell/common/platform_util_mac.mm

Line 145 in f247ca3

BOOL did_trash = [[NSFileManager defaultManager] trashItemAtURL:url

I observed the following behaviour with a small test script:

// File goes into root directory of onedrive
BOOL did_trash = [[NSFileManager defaultManager] trashItemAtURL:self.fileURL
                                                   resultingItemURL:nil
                                                             error:&deleteError];

// File is gone forever, doesn't go into trash
BOOL did_trash = [self.fileManager removeItemAtURL:self.fileURL error:&deleteError];

In both cases did_trash returns true.

Maybe someone has an idea how to solve this issue :)

[MarshallOfSound]

This is a bug in OneDrive implementation, the API Electron is calling is correct but OneDrive clearly isn't handling it correctly 😅 I'd suggest reporting this bug to OneDrive / msft support as it's something that their app can fix but Electron can / should not.