Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[Apple Notes] Extension fails with "JS heap out of memory" error #17137

Open
kesslink opened this issue Feb 17, 2025 · 7 comments · May be fixed by #17192
Open

[Apple Notes] Extension fails with "JS heap out of memory" error #17137

kesslink opened this issue Feb 17, 2025 · 7 comments · May be fixed by #17192
Labels
bug Something isn't working extension: apple-notes Issues related to the apple-notes extension extension Issues related to one of the extensions in the Store

Comments

@kesslink
Copy link

Extension

https://www.raycast.com/raycast/apple-notes

Raycast Version

1.91.2

macOS Version

15.1.1

Description

Error:

Error: Worker terminated due to reaching memory limit: JS heap out of memory

[kOnExit]:worker:313:26
Worker.<computed>.onexit:worker:229:20

Steps To Reproduce

When I launch the command "Search Notes" the error message appears.
My Notes app is open and running when this happens.

Current Behaviour

The error just says "Something went wrong." That's all I got.

Expected Behaviour

No response

@kesslink kesslink added bug Something isn't working extension Issues related to one of the extensions in the Store labels Feb 17, 2025
@raycastbot raycastbot added the extension: apple-notes Issues related to the apple-notes extension label Feb 17, 2025
@raycastbot
Copy link
Collaborator

raycastbot commented Feb 17, 2025

Thank you for opening this issue!

🔔 @tumispro @ron-myers @mathieudutour @StevenRCE0 @SKaplanOfficial @tylerwince @xilopaint @ridemountainpig @teemusuvinen you might want to have a look.

💡 Author and Contributors commands

The author and contributors of raycast/apple-notes can trigger bot actions by commenting:

  • @raycastbot close this issue Closes the issue.
  • @raycastbot close as not planned Closes the issue as not planned.
  • @raycastbot rename this issue to "Awesome new title" Renames the issue.
  • @raycastbot reopen this issue Reopens the issue.
  • @raycastbot assign me Assigns yourself to the issue.
  • @raycastbot good first issue Adds the "Good first issue" label to the issue.
  • @raycastbot keep this issue open Make sure the issue won't go stale and will be kept open by the bot.

@thomaslombart thomaslombart changed the title [Apple Notes] ...Extension doesn't work [Apple Notes] Extension fails with "JS heap out of memory" error Feb 18, 2025
@thomaslombart
Copy link
Collaborator

Thanks for reporting the issue. How many notes do you have?

@kesslink
Copy link
Author

kesslink commented Feb 18, 2025 via email

@thomaslombart
Copy link
Collaborator

All right, so that's why it happens. Currently, we don't limit the amount of notes when retrieving them, but we should both limit it to a thousand and add typeahead search to improve the experience.

@kesslink
Copy link
Author

kesslink commented Feb 19, 2025 via email

@thomaslombart
Copy link
Collaborator

thomaslombart commented Feb 19, 2025

Is that something that can be fixed in the extension?

Yes.

I can't be the only person with this number of notes.

The heap out of memory error is pretty common to extensions, but you're the first creating an issue for that, 6709 notes is a pretty big number!

And is there something I can do to workaround the issue in the meantime?

Not really; this requires fixing the extension, and I currently don't have much bandwidth for that. I'll try and see if I can do something about it on Friday.

@kesslink
Copy link
Author

kesslink commented Feb 19, 2025 via email

@saldanaj97 saldanaj97 linked a pull request Feb 20, 2025 that will close this issue
5 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working extension: apple-notes Issues related to the apple-notes extension extension Issues related to one of the extensions in the Store
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants