glimmerjs / glimmer-application-pipeline

Tooling for developing Glimmer standalone apps with ember-cli
MIT License
21 stars 31 forks source link

There should be a hook exposing jsTree to addons #151

Open rondale-sc opened 6 years ago

rondale-sc commented 6 years ago

In glimmer-app.ts there are a few addonProcessTree calls that expose things like:

For the src and css trees. However, since the jsTree is from the result of the src tree going through the TS compiler inside the javascriptTree function (see: https://github.com/glimmerjs/glimmer-application-pipeline/blob/master/lib/broccoli/glimmer-app.ts#L407) there is no opportunity to inject non typescript modules into the JS output.

This was made apparent to me when I tried to create a module that was in the jsTree before rollup happened in the package method. I was forced to modify the srcTree and add a TS file there which was inconvenient. I think on L407 (linked above) we should add a addonProcessTree to expose the javascriptTree to user space.

Any thoughts on this?