inspector cannot inspect ember-data - ember-data/debug is missing app-reexports about data

[runspired]

should be resolved on main. Will result in the module still being present unfortunately, though empty.

[runspired]

hrm, this one is kinda tricky because I don't think we can safely add it as an app re-export and support configuring it correctly. With it being a v2-addon now its likely you should have to explicitly add it to your build.

[patricklx]

the v2 addons do have app re exports with the provided rollup addons from embroider.
Can you not use those?

[Baltazore]

Hey @patricklx if you can point me to that I can spend some1 time adding those if you ok with it @runspired

[patricklx]

Sure. It's in the default blueprint:
https://github.com/embroider-build/addon-blueprint/blob/cb4d7eee319b15cd2aae822653009a96d10baab4/files/__addonLocation__/rollup.config.mjs#L29

[patricklx]

I think it should have at least a deprecate step. So it could also be part of -ember-data package. With some macro condition?
https://github.com/emberjs/data/tree/main/packages/-ember-data/app

[runspired]

We can't use app re-export from the V2 add-on at all, we could maybe guard the import in -ember-data but regardless we'll have to break the existing config setting

[patricklx]

Why not? Some constraints by ember data internals?

[runspired]

@patricklx because we have an existing config setting used to keep it in the build in v1, the default is that its not included in the build, and vite/embroider don't have a clean production-only-import concept so I'm not sure the path forward here.

[patricklx]

Thanks 👍.
Will this be back ported to 5.3 lts?