Remove @embroider/macros from dependencies

[NullVoxPopuli]

macros is an external as it is a build time behavior controlled by the consumer

also we don't want macros to show up in optimized deps

[BobrImperator]

In that case it needs to be a peerDependency at the very least, since @embroider/macros must be present at build time for consumers. While it's also not generally installed in apps themselves unless the apps have their own need for it.
I'm not opposed here but AFAIK most addons I've seen declare it a dependency, which could've been a multi-year mistake 😅.
Overall this likely is a breaking change?

liquid-fire#package.json
ember-qunit@9.04#package.json -> ember-qunit@9.1.0 already changed it.

[NullVoxPopuli]

In that case it needs to be a peerDependency at the very least,

nay, it's not a runtime dependency

since @embroider/macros must be present at build time for consumers.

there is nothing to be present -- macros is actually just signalling to the consumers' babel

I'm not opposed here but AFAIK most addons I've seen declare it a dependency,

they are all wrong 🙈 lol

which could've been a multi-year mistake

yup!

Overall this likely is a breaking change?

nope -- bugfix -- in the worst case, users won't notice -- in the best case, it unblocks people from using vite (blockers include: duplicate macros state (isTesting, etc))