Skip to content

Commit acc3d82

Browse files
committed
refactor(project): WatchHandler emit error event
1 parent 3941bd1 commit acc3d82

2 files changed

Lines changed: 8 additions & 3 deletions

File tree

packages/project/lib/build/ProjectBuilder.js

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -318,7 +318,7 @@ class ProjectBuilder {
318318
this.#log.info(`Update succeeded in ${this._getElapsedTime(startTime)}`);
319319
} catch (err) {
320320
this.#log.error(`Update failed`);
321-
this.#log.error(err);
321+
throw err;
322322
} finally {
323323
this._deregisterCleanupSigHooks(cleanupSigHooks);
324324
await this._executeCleanupTasks();

packages/project/lib/build/helpers/WatchHandler.js

Lines changed: 7 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -94,8 +94,13 @@ class WatchHandler extends EventEmitter {
9494
this.#sourceChanges = new Map();
9595

9696
this.#updateInProgress = true;
97-
await this.#handleResourceChanges(sourceChanges);
98-
this.#updateInProgress = false;
97+
try {
98+
await this.#handleResourceChanges(sourceChanges);
99+
} catch (err) {
100+
this.emit("error", err);
101+
} finally {
102+
this.#updateInProgress = false;
103+
}
99104

100105
if (this.#sourceChanges.size > 0) {
101106
// New changes have occurred during processing, trigger queue again

0 commit comments

Comments
 (0)