You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
Hey folks, went down a rabbit hole yesterday trying to add Firefox support. I have a major portion of the extension ported to Plasmo, which (in theory) gives full cross-browsers support for all browsers.
I told Windsurf to test it and here's what it spit out:
I don't have the devtools panel working yet, but I'm starting to see the outline of how this could be nice:
After diving deep into the code for the day, I can see the response to this being two-fold:
Negative, this is too big of a refactor
Positive, because it will make things easier to expand in the future
To be frank, I'm not sure where I fall on this. I think, with attention, this would be really beneficial to refactor things out to simplify the core logic. Add typescript types to messages being passed back/forth.
However, if this is 95% complete with everything needed now (other than non-Chrome extension support), there's a decent argument this isn't worth the effort.
reacted with thumbs up emoji reacted with thumbs down emoji reacted with laugh emoji reacted with hooray emoji reacted with confused emoji reacted with heart emoji reacted with rocket emoji reacted with eyes emoji
Uh oh!
There was an error while loading. Please reload this page.
-
Hey folks, went down a rabbit hole yesterday trying to add Firefox support. I have a major portion of the extension ported to Plasmo, which (in theory) gives full cross-browsers support for all browsers.
I told Windsurf to test it and here's what it spit out:
I don't have the devtools panel working yet, but I'm starting to see the outline of how this could be nice:
After diving deep into the code for the day, I can see the response to this being two-fold:
To be frank, I'm not sure where I fall on this. I think, with attention, this would be really beneficial to refactor things out to simplify the core logic. Add typescript types to messages being passed back/forth.
However, if this is 95% complete with everything needed now (other than non-Chrome extension support), there's a decent argument this isn't worth the effort.
You can see my WIP here: https://github.com/drmikecrowe/browser-tools-mcp/tree/universal-extension/universal-extension
Couple of notes:
build/
folder so end users don't have to compile.Beta Was this translation helpful? Give feedback.
All reactions