Repository navigation
Conversation
|
The following comment was made by an LLM, it may be inaccurate: Found a related PR: #47724: fix(desktop): surface default path opening errors This appears to be addressing the same issue (#47722) as the current PR (#47891). Both are fixing error reporting for default file opener functionality in the desktop app. You may want to check if #47724 is still open and whether the current PR duplicates or builds on that work. |
|
Automated PR Cleanup Thank you for contributing to opencode. Due to the high volume of PRs from users and AI agents, we periodically close older PRs using automated criteria so maintainers can focus review time on the most active and community-supported contributions. This PR was closed because it matched the following cleanup criteria:
PRs created within the last month are not affected by this cleanup. If you believe this PR was closed incorrectly, or if you are still actively working on it, please leave a comment explaining why it should be reopened. A maintainer can review and reopen it if appropriate. Thanks again for taking the time to contribute. |
Issue for this PR
Closes #47722
Type of change
What does this PR do?
Reject non-empty errors returned by Electron's
shell.openPathso the existing UI error handler reports failed opens. Successful opens and explicitly selected applications keep their existing behavior.How did you verify your code works?
draft-storeis blocked because Bun 1.3.14 lacksnode:sqlite.Screenshots / recordings
Not captured. Full desktop UI interaction was not manually exercised.
Checklist