Send special message if we live sync javascript file and the liveEdit option is on

[KristinaKoeva]

Commin PR telerik/mobile-cli-lib#773

[enchev]

👍

[rosen-vladimirov]

You can use this.$fs.readText instead of readFile and skip .toString().

[rosen-vladimirov]

I would advise you to change the sendMessage of IOSLiveSyncService to return future, which is resolved when all the data is written:

private sendMessage(message: string): IFuture<void> {
    return (() => {
        let socketWriteFuture = new Future<void>();
        try {
            let length = Buffer.byteLength(message, "utf16le");
            let payload = new Buffer(length + 4);
            payload.writeInt32BE(length, 0);
            payload.write(message, 4, length, "utf16le");

            // Have to check how to add the handler only for current sending. In case error is not raised, the socket will remain alive and next message will add a new error handler. This will be a problem in a long living process
            this.socket.once("error", err => {
                this.$logger.trace("Error while sending message: ", err);

                this.destroySocket();

                socketWriteFuture.throw(err);
            });

            this.socket.write(payload, "utf16le", () => {
                socketWriteFuture.return();
            });

        } catch(err) {
            this.$logger.trace("Error while sending page reload:", err);
            this.destroySocket();
        }

        socketWriteFuture.wait();
    }).future<void>()();
}

My idea is to make sure the message is sent. According to Node.js documentation The optional callback parameter will be executed when the data is finally written out - this may not be immediately.

[tzraikov]

👍

[rosen-vladimirov]

👍 after green build