search

denali-js / cli

The official CLI for starting, building, and running Denali apps and addons
MIT License
5 stars 5 forks source link

Spinner throws an error and exits process on rebuild #19

Closed acburdine closed 7 years ago

acburdine commented 7 years ago

Experiencing this error with latest released denali cli, running denali server in an app.

denali server builds the app and starts initially, but any rebuilds trigger this error, causing the process to exit:

A promise was rejected but did not have a .catch() handler:
Error: Spinner process failed to acknowledge a command on time: start(Building api)
    at Timeout.setTimeout (/Users/acburdine/Sites/tourdefours/api/node_modules/denali-cli/src/spinner.ts:38:14)
    at ontimeout (timers.js:365:14)
    at tryOnTimeout (timers.js:237:5)
    at Timer.listOnTimeout (timers.js:207:5)

/Users/acburdine/Sites/tourdefours/api/node_modules/denali-cli/src/spinner.ts:38
      reject(new Error(`Spinner process failed to acknowledge a command on time: ${ operation }(${ args.join(', ') })`))
             ^
Error: Spinner process failed to acknowledge a command on time: start(Building api)
    at Timeout.setTimeout (/Users/acburdine/Sites/tourdefours/api/node_modules/denali-cli/src/spinner.ts:38:14)
    at ontimeout (timers.js:365:14)
    at tryOnTimeout (timers.js:237:5)
    at Timer.listOnTimeout (timers.js:207:5)

The issue occurs in denali server but because it's erroring in the spinner I think it's an issue with the CLI. Can move the issue over to denali if it's not an error here.

acburdine commented 7 years ago

Note: this is a pretty nasty issue to have because it means that I have to re-run denali server every time I make a change. Will investigate further.