Prevent macro from running without babel-marcos

I was wondering, as I am using babel-macros in a project now, if there could be a way added so macros could “catch” it when they are being imported in a normal file.

As it seems now, there isn’t such thing. The this is unset, if packing with webpack I could check if I am running in node (browser won’t), but I also use this on node. Possible would be to check the arguments, but as the mantra of this repo is, I think explicit is better than implicit 😉

I wonder what the best way is though:

• Setting this seemed good, but won’t work with arrow functions
• Setting a property to check on the macro function itself:

    const macros = require(requirePath)
  + macros.with_babel_macros
    macros({
      references: referencePathsByImportName,
      state,
      babel,
    })
  

  Feels kinda hacky, but again we are putting macros into javascript, So who cares about hacky 😅 (This one is actually also possible (cleaner?) with Symbols)
• Passing a symbol as first argument that can be compared

  import babelMacros from 'babel-macros';
  export default ({ is_babel_macros, references, state }) => {
    if (is_babel_macros !== babelMacros.is_babel_macros) {
      ...
  

Most likely I am missing an option or some perspective, so I’d like to discuss this 😃

[reznord]

I’d be happy to take this on!

[kentcdodds]

Now it’s recommended to wrap your macro in createMacro. See the updated docs 👍

Not “recommended” … “required” 🕶