Update safari extension

[ChangeHow]

Description

Improvements:

• Speed up extension by loading plugins asynchronously.
• Support parseBookmarksWithGo option to unlimit the size of bookmarks
  • [Safari] ...Search bookmarks not working #17188
  • [Safari] Search Bookmarks command doesn't work #14384
  • [Safari] fails to run #13833

Screencast

Checklist

• I read the extension guidelines
• I read the documentation about publishing
• I ran npm run build and tested this distribution build in Raycast
• I checked that files in the assets folder are used by the extension itself
• I checked that assets used by the README are placed outside of the metadata folder

[raycastbot]

Thank you for your contribution! 🎉

🔔 @loris @thomaspaulmann @SKaplanOfficial @iwex @xilopaint @ridemountainpig @1weiho @xmok you might want to have a look.

You can use this guide to learn how to check out the Pull Request locally in order to test it.

Due to our current reduced availability, the initial review may take up to 10-15 business days

[greptile-apps]

PR Summary

This PR adds a Go-based bookmarks parser to the Safari extension to improve performance with large bookmark files.

• Added Go implementation in /extensions/safari/src/go/ with a more efficient parser that solves "object size is too large" errors
• Added new preference option parseBookmarksWithGo in package.json to toggle between Node-based and Go-based parsing methods
• Implemented benchmark tests in src/tests/bookmark.bench.ts to compare performance between the two parsing methods
• Replaced third-party LRUCache with Raycast's built-in Cache API for better integration
• Improved Pinyin support performance by optimizing handler installation and adding performance monitoring

_{💡 (1/5) You can manually trigger the bot by mentioning @greptileai in a comment!}

_{24 file(s) reviewed, 18 comment(s)
Edit PR Review Bot Settings | Greptile}

[ChangeHow]

@pernielsentikaer Hi, for asking help if you have time.

I just met a issue that I can't run my go exe file in extension even if I run build:go_parser script in build script. The extension keep warning that can't find tools/bookmarks-parser.

I have no idea how to solve this issue.

[ChangeHow]

@pernielsentikaer Hi, for asking help if you have time.

I just met a issue that I can't run my go exe file in extension even if I run build:go_parser script in build script. The extension keep warning that can't find tools/bookmarks-parser.

I have no idea how to solve this issue.

I just checked the development doc. It seems that I need to move my exe file to the asset folder. I'll try it later.

[ksandvik]

Is there a need to use Go, why not Swift (or Objective-C) with a bridge as this is a Safari MacOSX only limit?

[ChangeHow]

Is there a need to use Go, why not Swift (or Objective-C) with a bridge as this is a Safari MacOSX only limit?

The main reason is that I don't know other high performance language except Go😄.

If you are familiar with Swift or OC, feel free to edit this PR.

The other reason is that I try to handle data with WASM in the earlier version, and Go has good support for that.

[ChangeHow]

Is there a need to use Go, why not Swift (or Objective-C) with a bridge as this is a Safari MacOSX only limit?

I have met a performance issue that the Go parser has a wrose performance than node. Especially in the first time running


It seems that go paresr need a time to load or warm up 🥲.

If you have time, please confirm it works well in your Mac. I can only assume that Go parser can provide the bigger size plist file support than node. @ksandvik

[pernielsentikaer]

Let's not make it slower 😊