ember-learn / ember-cli-addon-docs

Easy, beautiful docs for your OSS Ember addons
https://ember-learn.github.io/ember-cli-addon-docs
MIT License
176 stars 143 forks source link

Ensure AddonDocs doesn't break fastboot #187

Open samselikoff opened 6 years ago

samselikoff commented 6 years ago

Tried upgrading Storefront's AddonDocs to latest version, still ran into errors.

Did some digging, found these:

Only solution was to ember install ember-fetch to Storefront (as a devDependency, so just for the dummy app). Seems weird/broken to me, but if this is the only solution we should add to the docs?

dfreeman commented 6 years ago

The issue seems to be a mismatch in expectations between ember-fetch and ember-cli-fastboot. The former uses the updateFastBootManifest hook to add a Node-compatible version of fetch to the build, but that hook is only ever invoked for top-level addons, not nested ones.

The only thing I immediately saw in the issue tracker was a semi-related issue about fastbootDependencies not being recursively discovered, where @kratiahuja suggested that that hook might provide an alternative. If it's not being recursively invoked itself, though, then I'm not sure how these pieces are actually supposed to fit together ¯\_(ツ)_/¯

dfreeman commented 6 years ago

I think https://github.com/tchak/ember-fetch-adapter/issues/1 and https://github.com/ember-cli/ember-fetch/issues/98 are two more recent examples of this popping up specifically due to updateFastBootManifest rather than the older fastbootDependencies issue