Multiple commands in one file break Sketch about skpm HOT 14 CLOSED

yes, I fixed that when we were using rollup but switching to webpack broke it.

Can you tell me what is your use-case? I had one in mind specifically (onClick handler for react-sketchapp) but I'm curious if there is more

from skpm.

I have a handful of commands that I'm placing in the Sketch menu which are contextually similar, but call different methods (ex "Clear all guides", "Clear guides in artboard", "Clear horizontal guides", etc.) They're all effectively one liners into other methods so it seems gratuitous to put them all in their own separate files. I suppose one could argue that I make the whole thing a class and export that.

The last time I ran it, I was looking for a solution for syncing the current context with your sketch-web-view. I had two commands, one that launched the web view and one that was a listener for selection change events which would sync data into the html content of the webview. I ultimately opted for splitting them apart because they were different enough.

from skpm.

got it. I'll try to find a fix

from skpm.

No rush on my end. The hack works fine for me, but I figured I'd report it in case it's something you care about, and because it does make for some slightly unpredictable results.

from skpm.

I actually don't know how to do it. Is there any webpack expert around who have an idea?

from skpm.

For reference, here's the actual error in Console.app

ReferenceError: Can't find variable: exports.
Plugin “All artboard guides”, line 43:
» Object.defineProperty(exports, '__esModule', { value: true });

For giggles, I tried doing this:

var exports
export var foo = function (){}
export var bar = function (){}

export default function(){
  exports
}

But that gets compiled in a way that gives exports a new name so it doesn't clobber the actual exports variable, which I suppose is good behavior.

I guess the only solution I can picture is something that just prepends var exports to the file, but I'm not sure that's any better than my hack mentioned in the OP

from skpm.

thinking about this a bit more: would adding var exports at the top of the file with a webpack plugin (https://webpack.js.org/plugins/banner-plugin/) make it work?

from skpm.

Hmm… interesting idea. I scrapped this project in favor of a cleaner solution, but I'll try this out once I get back to this stage.

from skpm.

I manage to do it here: 804046e

from skpm.

I can confirm it works 👍

from skpm.

That was quick! Thanks

from skpm.

@mathieudutour could you please bump the version and push to npm? 🤗

from skpm.

It's a breaking change, I'd like to get #44 in as well, and maybe fixing #45 before releasing

from skpm.

Ok, I can live with the github version, thanks!

from skpm.