NullVoxPopuli

> I think you could use dynamic import inside closeBundle to not be blocked on esm conversion: oh! that's a good idea! I always forget that's a thing. been so...

Nevermind -- our build setup is removing all imports........... 

Been trying a few different compilation techniques, and it seems that there is a lot of: ``` [!] SyntaxError: Named export 'readJsonSync' not found. The requested module 'fs-extra' is a...

woah, it's green

I think this can be customized via: https://github.com/embroider-build/embroider/blob/main/packages/addon-dev/src/rollup-app-reexports.ts#L11 ```js addon.appReexports(['**/*.js'], { mapFilename: (filename) => { if (filename.includes('/components/') && filename.endsWith('index.js')) { return path.dirname(filename) + '.js'; } } }), ``` does that...

> since index-modules is a supported thing Not in ESM node :sweat_smile: (which is a bit of an oversight for co-location -- I _also_ learned this by running in to...

looks like this is a problem with `importSync` I can't reproduce the behavior in the tests correctly, but this simple file causes an error: ```js import { importSync } from...

Looks like there are a few things happening here -- gonna have separate issue for them all. - The original issue, which this PR is concerned with: https://github.com/embroider-build/embroider/issues/1674 - A...

Info from @mansona, we might need https://github.com/embroider-build/embroider/pull/1650/commits/9e7d875195aef33ea8f30401862b441a06a88a14 or https://github.com/browserify/resolve/pull/320 to support it

I found that things _work ok_ in the test environment _without_ interacting with the `@embroider/macros`, but in the latest commit, #e7a924e, I added some macros usage to re-break the test....