zigbee-herdsman 4.1.2 → 4.2.1

package/dist/adapter/deconz/driver/frameParser.js

@@ -33,114 +33,113 @@ var __importStar = (this && this.__importStar) || (function () {
         return result;
     };
 })();
-var __importDefault = (this && this.__importDefault) || function (mod) {
-    return (mod && mod.__esModule) ? mod : { "default": mod };
-};
 Object.defineProperty(exports, "__esModule", { value: true });
 exports.frameParserEvents = void 0;
 const node_events_1 = require("node:events");
+const buffalo_1 = require("../../../buffalo");
 const logger_1 = require("../../../utils/logger");
 const Zdo = __importStar(require("../../../zspec/zdo"));
-const constants_1 = __importDefault(require("./constants"));
+const constants_1 = require("./constants");
 const driver_1 = require("./driver");
 const NS = "zh:deconz:frameparser";
-const MIN_BUFFER_SIZE = 3;
 const littleEndian = true;
 const lastReceivedGpInd = { srcId: 0, commandId: 0, frameCounter: 0 };
 exports.frameParserEvents = new node_events_1.EventEmitter();
 function parseReadParameterResponse(view) {
+    const seqNumber = view.getUint8(1);
+    const status = view.getUint8(2);
     const parameterId = view.getUint8(7);
+    let pos = 8;
+    let result = null;
     switch (parameterId) {
-        case constants_1.default.PARAM.Network.MAC: {
-            const mac = view.getBigUint64(8, littleEndian).toString(16);
-            let result = mac;
-            while (result.length < 16) {
-                result = `0${result}`;
-            }
-            result = `0x${result}`;
-            logger_1.logger.debug(`MAC: ${result}`, NS);
-            return result;
-        }
-        case constants_1.default.PARAM.Network.PAN_ID: {
-            const panId = view.getUint16(8, littleEndian);
-            logger_1.logger.debug(`PANID: ${panId.toString(16)}`, NS);
-            return panId;
-        }
-        case constants_1.default.PARAM.Network.NWK_ADDRESS: {
-            const nwkAddr = view.getUint16(8, littleEndian);
-            logger_1.logger.debug(`NWKADDR: ${nwkAddr.toString(16)}`, NS);
-            return nwkAddr;
-        }
-        case constants_1.default.PARAM.Network.EXT_PAN_ID: {
-            const extPanId = view.getBigUint64(8, littleEndian).toString(16);
-            let res = extPanId;
-            while (res.length < 16) {
-                res = `0${res}`;
-            }
-            res = `0x${res}`;
-            logger_1.logger.debug(`EXT_PANID: ${res}`, NS);
-            return res;
-        }
-        case constants_1.default.PARAM.Network.APS_EXT_PAN_ID: {
-            const apsExtPanId = view.getBigUint64(8, littleEndian).toString(16);
-            let resAEPID = apsExtPanId;
-            while (resAEPID.length < 16) {
-                resAEPID = `0${resAEPID}`;
-            }
-            resAEPID = `0x${resAEPID}`;
-            logger_1.logger.debug(`APS_EXT_PANID: ${resAEPID}`, NS);
-            return resAEPID;
-        }
-        case constants_1.default.PARAM.Network.NETWORK_KEY: {
-            const networkKey1 = view.getBigUint64(9).toString(16);
-            let res1 = networkKey1;
-            while (res1.length < 16) {
-                res1 = `0${res1}`;
-            }
-            const networkKey2 = view.getBigUint64(17).toString(16);
-            let res2 = networkKey2;
-            while (res2.length < 16) {
-                res2 = `0${res2}`;
+        case constants_1.ParamId.MAC_ADDRESS: {
+            result = view.getBigUint64(pos, littleEndian);
+            break;
+        }
+        case constants_1.ParamId.APS_TRUST_CENTER_ADDRESS: {
+            result = view.getBigUint64(pos, littleEndian);
+            break;
+        }
+        case constants_1.ParamId.NWK_PANID: {
+            result = view.getUint16(pos, littleEndian);
+            break;
+        }
+        case constants_1.ParamId.STK_PROTOCOL_VERSION: {
+            result = view.getUint16(pos, littleEndian);
+            break;
+        }
+        case constants_1.ParamId.NWK_NETWORK_ADDRESS: {
+            result = view.getUint16(pos, littleEndian);
+            break;
+        }
+        case constants_1.ParamId.NWK_EXTENDED_PANID: {
+            result = view.getBigUint64(pos, littleEndian);
+            break;
+        }
+        case constants_1.ParamId.APS_USE_EXTENDED_PANID: {
+            result = view.getBigUint64(pos, littleEndian);
+            break;
+        }
+        case constants_1.ParamId.STK_NETWORK_KEY: {
+            result = Buffer.alloc(16);
+            pos += 1; // key index
+            for (let i = 0; i < 16; i++) {
+                result[i] = view.getUint8(pos);
+                pos += 1;
             }
-            logger_1.logger.debug("NETWORK_KEY: hidden", NS);
-            return `0x${res1}${res2}`;
+            break;
+        }
+        case constants_1.ParamId.STK_CURRENT_CHANNEL: {
+            result = view.getUint8(pos);
+            break;
         }
-        case constants_1.default.PARAM.Network.CHANNEL: {
-            const channel = view.getUint8(8);
-            logger_1.logger.debug(`CHANNEL: ${channel}`, NS);
-            return channel;
+        case constants_1.ParamId.APS_CHANNEL_MASK: {
+            result = view.getUint32(pos, littleEndian);
+            break;
         }
-        case constants_1.default.PARAM.Network.CHANNEL_MASK: {
-            const chMask = view.getUint32(8, littleEndian);
-            logger_1.logger.debug(`CHANNELMASK: ${chMask.toString(16)}`, NS);
-            return chMask;
+        case constants_1.ParamId.STK_FRAME_COUNTER: {
+            result = view.getUint32(pos, littleEndian);
+            break;
         }
-        case constants_1.default.PARAM.Network.PERMIT_JOIN: {
-            const permitJoin = view.getUint8(8);
-            logger_1.logger.debug(`PERMIT_JOIN: ${permitJoin}`, NS);
-            return permitJoin;
+        case constants_1.ParamId.STK_PERMIT_JOIN: {
+            result = view.getUint8(pos);
+            break;
         }
-        case constants_1.default.PARAM.Network.WATCHDOG_TTL: {
-            const ttl = view.getUint32(8);
-            logger_1.logger.debug(`WATCHDOG_TTL: ${ttl}`, NS);
-            return ttl;
+        case constants_1.ParamId.DEV_WATCHDOG_TTL: {
+            result = view.getUint32(pos, littleEndian);
+            break;
+        }
+        case constants_1.ParamId.STK_NWK_UPDATE_ID: {
+            result = view.getUint8(pos);
+            break;
         }
         default:
             //throw new Error(`unknown parameter id ${parameterId}`);
             logger_1.logger.debug(`unknown parameter id ${parameterId}`, NS);
-            return null;
+            break;
     }
+    if (parameterId in constants_1.ParamId) {
+        let p = result;
+        if (parameterId === constants_1.ParamId.STK_NETWORK_KEY) {
+            // don't show in logs
+            p = "<hidden>";
+        }
+        else if (typeof result === "bigint") {
+            p = `0x${result.toString(16).padStart(16, "0")}`;
+        }
+        logger_1.logger.debug(`Received read parameter response for ${constants_1.ParamId[parameterId]}, seq: ${seqNumber}, status: ${status}, parameter: ${p}`, NS);
+    }
+    return result;
 }
 function parseReadFirmwareResponse(view) {
-    const fw = [view.getUint8(5), view.getUint8(6), view.getUint8(7), view.getUint8(8)];
-    logger_1.logger.debug(`read firmware version response - version: ${fw}`, NS);
+    const fw = view.getUint32(5, littleEndian);
+    logger_1.logger.debug(`read firmware version response - version: 0x${fw.toString(16)}`, NS);
     return fw;
 }
 function parseDeviceStateResponse(view) {
-    const flag = view.getUint8(5);
-    logger_1.logger.debug(`device state: ${flag.toString(2)}`, NS);
-    exports.frameParserEvents.emit("receivedDataNotification", flag);
-    return flag;
+    const deviceState = view.getUint8(5);
+    exports.frameParserEvents.emit("deviceStateUpdated", deviceState);
+    return deviceState;
 }
 function parseChangeNetworkStateResponse(view) {
     const status = view.getUint8(2);
@@ -148,163 +147,198 @@ function parseChangeNetworkStateResponse(view) {
     logger_1.logger.debug(`change network state - status: ${status} new state: ${state}`, NS);
     return state;
 }
-function parseQuerySendDataStateResponse(view) {
-    try {
-        const commandId = view.getUint8(0);
-        const seqNr = view.getUint8(1);
-        const status = view.getUint8(2);
-        if (status !== 0) {
-            if (status !== 5) {
-                logger_1.logger.debug(`DATA_CONFIRM RESPONSE - seqNr.: ${seqNr} status: ${status}`, NS);
-            }
-            return null;
-        }
-        const frameLength = 7;
-        const payloadLength = view.getUint16(5, littleEndian);
-        const deviceState = view.getUint8(7);
-        const requestId = view.getUint8(8);
-        const destAddrMode = view.getUint8(9);
-        let destAddr64;
-        let destAddr16;
-        let destEndpoint;
-        let destAddr = "";
-        if (destAddrMode === constants_1.default.PARAM.addressMode.IEEE_ADDR) {
-            let res = view.getBigUint64(10, littleEndian).toString(16);
-            while (res.length < 16) {
-                res = `0${res}`;
-            }
-            destAddr64 = res;
-            // const buf2 = view.buffer.slice(18, view.buffer.byteLength);
-            destAddr = destAddr64;
-        }
-        else {
-            destAddr16 = view.getUint16(10, littleEndian);
-            // const buf2 = view.buffer.slice(12, view.buffer.byteLength);
-            destAddr = destAddr16.toString(16);
-        }
-        if (destAddrMode === constants_1.default.PARAM.addressMode.NWK_ADDR || destAddrMode === constants_1.default.PARAM.addressMode.IEEE_ADDR) {
-            destEndpoint = view.getUint8(view.byteLength - 7);
+function parseApsConfirmResponse(view) {
+    const buf = new buffalo_1.Buffalo(Buffer.from(view.buffer));
+    const commandId = buf.readUInt8();
+    const seqNr = buf.readUInt8();
+    const status = buf.readUInt8();
+    if (status !== constants_1.CommandStatus.Success) {
+        logger_1.logger.debug(`Response APS-DATA.confirm seq: ${seqNr} status: ${constants_1.CommandStatus[status]} (error)`, NS);
+        return null;
+    }
+    const frameLength = buf.readUInt16();
+    const payloadLength = buf.readUInt16();
+    // payload
+    const deviceState = buf.readUInt8();
+    const requestId = buf.readUInt8();
+    const destAddrMode = buf.readUInt8();
+    let destAddr64;
+    let destAddr16;
+    let destEndpoint;
+    let destAddr = "";
+    if (destAddrMode === constants_1.ApsAddressMode.Nwk) {
+        destAddr16 = buf.readUInt16();
+        destAddr = destAddr16.toString(16).padStart(4, "0");
+        destEndpoint = buf.readUInt8();
+    }
+    else if (destAddrMode === constants_1.ApsAddressMode.Group) {
+        destAddr16 = buf.readUInt16();
+        destAddr = destAddr16.toString(16).padStart(4, "0");
+    }
+    else if (destAddrMode === constants_1.ApsAddressMode.Ieee) {
+        destAddr64 = buf.readUInt64().toString(16).padStart(16, "0");
+        destAddr = destAddr64;
+        destEndpoint = buf.readUInt8();
+    }
+    else {
+        logger_1.logger.debug(`Response APS-DATA.confirm seq: ${seqNr} unsupported address mode: ${destAddrMode}`, NS);
+    }
+    const srcEndpoint = buf.readUInt8();
+    const confirmStatus = buf.readUInt8();
+    // resolve APS-DATA.request promise
+    const i = driver_1.apsBusyQueue.findIndex((r) => r.request && r.request.requestId === requestId);
+    if (i < 0) {
+        return null;
+    }
+    const req = driver_1.apsBusyQueue[i];
+    let strstatus = "unknown";
+    const hexstatus = `0x${confirmStatus.toString(16).padStart(2, "0")}`;
+    if (confirmStatus in constants_1.ApsStatusCode) {
+        strstatus = constants_1.ApsStatusCode[confirmStatus];
+    }
+    if (confirmStatus === constants_1.ApsStatusCode.Success) {
+        req.resolve(confirmStatus);
+    }
+    else {
+        req.reject(new Error(`Failed APS-DATA.request with confirm status: ${strstatus} (${hexstatus})`));
+    }
+    //remove from busyqueue
+    driver_1.apsBusyQueue.splice(i, 1);
+    logger_1.logger.debug(`APS-DATA.confirm  destAddr: 0x${destAddr} APS request id: ${requestId} confirm status: ${strstatus} ${hexstatus}`, NS);
+    exports.frameParserEvents.emit("deviceStateUpdated", deviceState);
+    return {
+        commandId,
+        seqNr,
+        status,
+        frameLength,
+        payloadLength,
+        deviceState,
+        requestId,
+        destAddrMode,
+        destAddr16,
+        destAddr64,
+        destEndpoint,
+        srcEndpoint,
+        confirmStatus,
+    };
+}
+// TODO(mpi): The ../buffalo/buffalo.ts already provides this, so we should reuse it instead of a own implementation?!
+class UDataView {
+    littleEndian = true;
+    pos = 0;
+    view;
+    constructor(view, littleEndian) {
+        this.view = view;
+        this.littleEndian = littleEndian;
+    }
+    getI8() {
+        if (this.pos + 1 <= this.view.byteLength) {
+            this.pos += 1;
+            return this.view.getInt8(this.pos - 1);
         }
-        const srcEndpoint = view.getUint8(view.byteLength - 6);
-        const confirmStatus = view.getInt8(view.byteLength - 5);
-        let newStatus = deviceState.toString(2);
-        for (let l = 0; l <= 8 - newStatus.length; l++) {
-            newStatus = `0${newStatus}`;
+        throw new RangeError();
+    }
+    getU8() {
+        if (this.pos + 1 <= this.view.byteLength) {
+            this.pos += 1;
+            return this.view.getUint8(this.pos - 1);
         }
-        // resolve send data request promise
-        const i = driver_1.apsBusyQueue.findIndex((r) => r.request && r.request.requestId === requestId);
-        if (i < 0) {
-            return null;
+        throw new RangeError();
+    }
+    getU16() {
+        if (this.pos + 2 <= this.view.byteLength) {
+            this.pos += 2;
+            return this.view.getUint16(this.pos - 2, this.littleEndian);
         }
-        clearTimeout(driver_1.enableRtsTimeout);
-        (0, driver_1.enableRTS)(); // enable ReadyToSend because confirm received
-        const req = driver_1.apsBusyQueue[i];
-        // TODO timeout (at driver.ts)
-        if (confirmStatus !== 0) {
-            // reject if status is not SUCCESS
-            //logger.debug("REJECT APS_REQUEST - request id: " + requestId + " confirm status: " + confirmStatus, NS);
-            req.reject(new Error(`confirmStatus=${confirmStatus}`));
+        throw new RangeError();
+    }
+    getU32() {
+        if (this.pos + 4 <= this.view.byteLength) {
+            this.pos += 4;
+            return this.view.getUint16(this.pos - 4, this.littleEndian);
         }
-        else {
-            //logger.debug("RESOLVE APS_REQUEST - request id: " + requestId + " confirm status: " + confirmStatus, NS);
-            req.resolve(confirmStatus);
-        }
-        //remove from busyqueue
-        driver_1.apsBusyQueue.splice(i, 1);
-        logger_1.logger.debug(`DATA_CONFIRM RESPONSE - destAddr: 0x${destAddr} request id: ${requestId} confirm status: ${confirmStatus}`, NS);
-        exports.frameParserEvents.emit("receivedDataNotification", deviceState);
-        return {
-            commandId,
-            seqNr,
-            status,
-            frameLength,
-            payloadLength,
-            deviceState,
-            requestId,
-            destAddrMode,
-            destAddr16,
-            destAddr64,
-            destEndpoint,
-            srcEndpoint,
-            confirmStatus,
-        };
+        throw new RangeError();
     }
-    catch (error) {
-        logger_1.logger.debug(`DATA_CONFIRM RESPONSE - ${error}`, NS);
-        return null;
+    getU64() {
+        if (this.pos + 8 <= this.view.byteLength) {
+            this.pos += 8;
+            return this.view.getBigUint64(this.pos - 8, this.littleEndian);
+        }
+        throw new RangeError();
     }
 }
-function parseReadReceivedDataResponse(view) {
+function parseApsDataIndicationResponse(inview) {
     // min 28 bytelength
     try {
-        let buf2;
-        let buf3;
-        const commandId = view.getUint8(0);
-        const seqNr = view.getUint8(1);
-        const status = view.getUint8(2);
-        if (status !== 0) {
-            if (status !== 5) {
-                logger_1.logger.debug(`DATA_INDICATION RESPONSE - seqNr.: ${seqNr} status: ${status}`, NS);
-            }
+        const uview = new UDataView(inview, true);
+        const commandId = uview.getU8();
+        const seqNr = uview.getU8();
+        const status = uview.getU8();
+        if (status !== constants_1.CommandStatus.Success) {
+            logger_1.logger.debug(`Response APS-DATA.indication seq: ${seqNr} status: ${constants_1.CommandStatus[status]}`, NS);
             return null;
         }
-        const frameLength = view.getUint16(3, littleEndian);
-        const payloadLength = view.getUint16(5, littleEndian);
-        const deviceState = view.getUint8(7);
-        const destAddrMode = view.getUint8(8);
+        const frameLength = uview.getU16();
+        const payloadLength = uview.getU16();
+        //------ start of payload ----------------------------------
+        const deviceState = uview.getU8();
+        const destAddrMode = uview.getU8();
         let destAddr64;
         let destAddr16;
-        let destAddr = "";
-        if (destAddrMode === constants_1.default.PARAM.addressMode.IEEE_ADDR) {
-            let res = view.getBigUint64(9, littleEndian).toString(16);
-            while (res.length < 16) {
-                res = `0${res}`;
-            }
-            destAddr64 = res;
-            buf2 = view.buffer.slice(17, view.buffer.byteLength);
+        let destAddr;
+        if (destAddrMode === constants_1.ApsAddressMode.Ieee) {
+            destAddr64 = uview.getU64().toString(16).padStart(16, "0");
+            destAddr16 = 0xfffe;
             destAddr = destAddr64;
         }
-        else {
-            destAddr16 = view.getUint16(9, littleEndian);
-            buf2 = view.buffer.slice(11, view.buffer.byteLength);
+        else if (destAddrMode === constants_1.ApsAddressMode.Nwk || destAddrMode === constants_1.ApsAddressMode.Group) {
+            destAddr16 = uview.getU16();
             destAddr = destAddr16.toString(16);
         }
-        view = new DataView(buf2);
-        const destEndpoint = view.getUint8(0);
-        const srcAddrMode = view.getUint8(1);
+        else {
+            throw new Error(`unsupported destination address mode: ${destAddrMode}`);
+        }
+        const destEndpoint = uview.getU8();
+        const srcAddrMode = uview.getU8();
         let srcAddr64;
-        let srcAddr16;
-        let srcAddr = "";
-        if (srcAddrMode === constants_1.default.PARAM.addressMode.NWK_ADDR || srcAddrMode === 0x04) {
-            srcAddr16 = view.getUint16(2, littleEndian);
-            buf3 = view.buffer.slice(4, view.buffer.byteLength);
+        let srcAddr16 = 0xfffe;
+        let srcAddr;
+        if (srcAddrMode === constants_1.ApsAddressMode.Nwk || srcAddrMode === constants_1.ApsAddressMode.NwkAndIeee) {
+            srcAddr16 = uview.getU16();
             srcAddr = srcAddr16.toString(16);
-        }
-        if (srcAddrMode === constants_1.default.PARAM.addressMode.IEEE_ADDR || srcAddrMode === 0x04) {
-            let res = view.getBigUint64(2, littleEndian).toString(16);
-            while (res.length < 16) {
-                res = `0${res}`;
+            if (srcAddrMode === constants_1.ApsAddressMode.NwkAndIeee) {
+                srcAddr64 = uview.getU64().toString(16).padStart(16, "0");
             }
-            srcAddr64 = res;
-            buf3 = view.buffer.slice(10, view.buffer.byteLength);
-            srcAddr = srcAddr64;
-        }
-        // biome-ignore lint/style/noNonNullAssertion: ignored using `--suppress`
-        view = new DataView(buf3); // XXX: not validated?
-        const srcEndpoint = view.getUint8(0);
-        const profileId = view.getUint16(1, littleEndian);
-        const clusterId = view.getUint16(3, littleEndian);
-        const asduLength = view.getUint16(5, littleEndian);
-        const asduPayload = Buffer.from(view.buffer.slice(7, asduLength + 7));
-        const lqi = view.getUint8(view.byteLength - 8);
-        const rssi = view.getInt8(view.byteLength - 3);
-        let newStatus = deviceState.toString(2);
-        for (let l = 0; l <= 8 - newStatus.length; l++) {
-            newStatus = `0${newStatus}`;
-        }
-        logger_1.logger.debug(`DATA_INDICATION RESPONSE - seqNr. ${seqNr} srcAddr: 0x${srcAddr} destAddr: 0x${destAddr} profile id: 0x${profileId.toString(16)} cluster id: 0x${clusterId.toString(16)} lqi: ${lqi}`, NS);
-        logger_1.logger.debug(`response payload: ${asduPayload.toString("hex")}`, NS);
-        exports.frameParserEvents.emit("receivedDataNotification", deviceState);
+        }
+        else {
+            throw new Error(`unsupported source address mode: ${srcAddrMode}`);
+        }
+        //  else if (srcAddrMode === PARAM.PARAM.addressMode.IEEE_ADDR) {
+        //     srcAddr64 = uview.getU64().toString(16).padStart(16, '0');
+        //     srcAddr = srcAddr64;
+        // }
+        const srcEndpoint = uview.getU8();
+        const profileId = uview.getU16();
+        const clusterId = uview.getU16();
+        const asduLength = uview.getU16();
+        const asdu = new Uint8Array(asduLength);
+        for (let i = 0; i < asduLength; i++) {
+            asdu[i] = uview.getU8();
+        }
+        // The following two bytes depends on protocol version 2 or 3
+        // for now just discard
+        uview.getU16();
+        const lqi = uview.getU8();
+        // version >= 2
+        let rssi = 0;
+        try {
+            rssi = uview.getI8();
+        }
+        catch (_) { }
+        logger_1.logger.debug(`Response APS-DATA.indication seq: ${seqNr} srcAddr: 0x${srcAddr} destAddr: 0x${destAddr} profile id: 0x${profileId.toString(16).padStart(4, "0")} cluster id: 0x${clusterId.toString(16).padStart(4, "0")} lqi: ${lqi}`, NS);
+        //logger.debug(`Response payload: [${Array.from(asdu).map(x =>x.toString(16).padStart(2, '0')).join(' ')}]`, NS);
+        exports.frameParserEvents.emit("deviceStateUpdated", deviceState);
+        const asduPayload = Buffer.from(asdu);
         const response = {
             commandId,
             seqNr,
@@ -332,17 +366,17 @@ function parseReadReceivedDataResponse(view) {
         return response;
     }
     catch (error) {
-        logger_1.logger.debug(`DATA_INDICATION RESPONSE - ${error}`, NS);
+        logger_1.logger.debug(`Response APS-DATA.indication error: ${error}`, NS);
         return null;
     }
 }
-function parseEnqueueSendDataResponse(view) {
+function parseApsDataRequestResponse(view) {
     try {
         const status = view.getUint8(2);
         const requestId = view.getUint8(8);
         const deviceState = view.getUint8(7);
-        logger_1.logger.debug(`DATA_REQUEST RESPONSE - request id: ${requestId} status: ${status}`, NS);
-        exports.frameParserEvents.emit("receivedDataNotification", deviceState);
+        logger_1.logger.debug(`Response APS-DATA.request APS request id: ${requestId} status: ${constants_1.CommandStatus[status]}`, NS);
+        exports.frameParserEvents.emit("deviceStateUpdated", deviceState);
         return deviceState;
     }
     catch (error) {
@@ -352,8 +386,12 @@ function parseEnqueueSendDataResponse(view) {
 }
 function parseWriteParameterResponse(view) {
     try {
+        const status = view.getUint8(2);
         const parameterId = view.getUint8(7);
-        logger_1.logger.debug(`write parameter response - parameter id: ${parameterId} - status: ${view.getUint8(2)}`, NS);
+        if (parameterId in constants_1.ParamId) {
+            // should always be true
+            logger_1.logger.debug(`Write parameter response parameter: ${constants_1.ParamId[parameterId]}, status: ${constants_1.CommandStatus[status]}`, NS);
+        }
         return parameterId;
     }
     catch (error) {
@@ -361,15 +399,14 @@ function parseWriteParameterResponse(view) {
         return null;
     }
 }
-function parseReceivedDataNotification(view) {
+function parseDeviceStateChangedNotification(view) {
     try {
         const deviceState = view.getUint8(5);
-        logger_1.logger.debug(`DEVICE_STATE changed: ${deviceState.toString(2)}`, NS);
-        exports.frameParserEvents.emit("receivedDataNotification", deviceState);
+        exports.frameParserEvents.emit("deviceStateUpdated", deviceState);
         return deviceState;
     }
     catch (error) {
-        logger_1.logger.debug(`parseReceivedDataNotification - ${error}`, NS);
+        logger_1.logger.debug(`parsedeviceStateUpdated - ${error}`, NS);
         return null;
     }
 }
@@ -435,43 +472,73 @@ function parseGreenPowerDataIndication(view) {
 }
 function parseMacPollCommand(_view) {
     //logger.debug("Received command MAC_POLL", NS);
-    return 28;
+    return constants_1.FirmwareCommand.MacPollIndication;
 }
 function parseBeaconRequest(_view) {
     logger_1.logger.debug("Received Beacon Request", NS);
-    return 31;
+    return constants_1.FirmwareCommand.Beacon;
+}
+function parseDebugLog(view) {
+    let dbg = "";
+    const buf = new buffalo_1.Buffalo(Buffer.from(view.buffer));
+    /* const commandId = */ buf.readUInt8();
+    /* const seqNr = */ buf.readUInt8();
+    const status = buf.readUInt8();
+    if (status !== constants_1.CommandStatus.Success) {
+        // unlikely
+        return null;
+    }
+    /* const frameLength = */ buf.readUInt16();
+    const payloadLength = buf.readUInt16();
+    for (let i = 0; i < payloadLength && buf.isMore(); i++) {
+        const ch = buf.readUInt8();
+        if (ch >= 32 && ch <= 127) {
+            dbg += String.fromCharCode(ch);
+        }
+    }
+    if (dbg.length !== 0) {
+        logger_1.logger.debug(`firmware log: ${dbg}`, NS);
+    }
+    return null;
 }
 function parseUnknownCommand(view) {
     const id = view.getUint8(0);
-    logger_1.logger.debug(`received unknown command - id ${id}`, NS);
+    if (id in constants_1.FirmwareCommand) {
+        logger_1.logger.debug(`received unsupported command: ${constants_1.FirmwareCommand[id]} id: 0x${id.toString(16).padStart(2, "0")}`, NS);
+    }
+    else {
+        logger_1.logger.debug(`received unknown command: id: 0x${id.toString(16).padStart(2, "0")}`, NS);
+    }
     return id;
 }
 function getParserForCommandId(id) {
     switch (id) {
-        case constants_1.default.PARAM.FrameType.ReadParameter:
+        case constants_1.FirmwareCommand.ReadParameter:
             return parseReadParameterResponse;
-        case constants_1.default.PARAM.FrameType.WriteParameter:
+        case constants_1.FirmwareCommand.WriteParameter:
             return parseWriteParameterResponse;
-        case constants_1.default.PARAM.FrameType.ReadFirmwareVersion:
+        case constants_1.FirmwareCommand.FirmwareVersion:
             return parseReadFirmwareResponse;
-        case constants_1.default.PARAM.FrameType.ReadDeviceState:
+        case constants_1.FirmwareCommand.Status:
             return parseDeviceStateResponse;
-        case constants_1.default.PARAM.APS.DATA_INDICATION:
-            return parseReadReceivedDataResponse;
-        case constants_1.default.PARAM.APS.DATA_REQUEST:
-            return parseEnqueueSendDataResponse;
-        case constants_1.default.PARAM.APS.DATA_CONFIRM:
-            return parseQuerySendDataStateResponse;
-        case constants_1.default.PARAM.FrameType.DeviceStateChanged:
-            return parseReceivedDataNotification;
-        case constants_1.default.PARAM.NetworkState.CHANGE_NETWORK_STATE:
+        case constants_1.FirmwareCommand.ApsDataIndication:
+            return parseApsDataIndicationResponse;
+        case constants_1.FirmwareCommand.ApsDataRequest:
+            return parseApsDataRequestResponse;
+        case constants_1.FirmwareCommand.ApsDataConfirm:
+            return parseApsConfirmResponse;
+        case constants_1.FirmwareCommand.StatusChangeIndication:
+            return parseDeviceStateChangedNotification;
+        case constants_1.FirmwareCommand.ChangeNetworkState:
             return parseChangeNetworkStateResponse;
-        case constants_1.default.PARAM.FrameType.GreenPowerDataInd:
+        case constants_1.FirmwareCommand.ZgpDataIndication:
             return parseGreenPowerDataIndication;
-        case 28:
+        case constants_1.FirmwareCommand.MacPollIndication:
             return parseMacPollCommand;
-        case 31:
+        case constants_1.FirmwareCommand.Beacon:
             return parseBeaconRequest;
+        case constants_1.FirmwareCommand.DebugLog:
+            return parseDebugLog;
         default:
             return parseUnknownCommand;
         //throw new Error(`unknown command id ${id}`);
@@ -479,46 +546,46 @@ function getParserForCommandId(id) {
 }
 function processFrame(frame) {
     const [seqNumber, status, command, commandId] = parseFrame(frame);
-    //logger.debug(`process frame with seq: ${seqNumber} status: ${status}`, NS);
+    // logger.debug(`Process frame with cmd: 0x${commandId.toString(16).padStart(2,'0')}, seq: ${seqNumber} status: ${status}`, NS);
     let queue = driver_1.busyQueue;
-    if (commandId === constants_1.default.PARAM.APS.DATA_INDICATION || commandId === constants_1.default.PARAM.APS.DATA_REQUEST || commandId === constants_1.default.PARAM.APS.DATA_CONFIRM) {
+    if (commandId === constants_1.FirmwareCommand.ApsDataRequest) {
         queue = driver_1.apsBusyQueue;
     }
-    const i = queue.findIndex((r) => r.seqNumber === seqNumber);
+    const i = queue.findIndex((r) => r.seqNumber === seqNumber && r.commandId === commandId);
     if (i < 0) {
         return;
     }
     const req = queue[i];
-    if (commandId === constants_1.default.PARAM.APS.DATA_REQUEST) {
-        // if confirm is true resolve request only when data confirm arrives
-        // TODO only return if a confirm was requested. if no confirm needed: go ahead
-        //if (req.confirm === true) {
-        return;
-        //}
+    if (commandId === constants_1.FirmwareCommand.ApsDataRequest) {
+        if (status === constants_1.CommandStatus.Success) {
+            // wait for APS-DATA.confirm to arrive
+            return;
+        }
+        // TODO(mpi): Within the timeout we should reschedule the APS-DATA.request (given that network state = connected)
+        // continue to reject as there will be no APS-DATA.confirm
     }
     //remove from busyqueue
     queue.splice(i, 1);
-    if (status !== 0) {
-        // reject if status is not SUCCESS
-        //logger.debug("REJECT REQUEST", NS);
-        req.reject(new Error(`status=${status}`));
+    if (status === constants_1.CommandStatus.Success) {
+        req.resolve(command);
     }
     else {
-        //logger.debug("RESOLVE REQUEST", NS);
-        req.resolve(command);
+        let cmdName;
+        if (commandId in constants_1.FirmwareCommand) {
+            cmdName = constants_1.FirmwareCommand[commandId];
+        }
+        else {
+            cmdName = `0x${commandId.toString(16).padStart(2, "0")}`;
+        }
+        req.reject(new Error(`Command ${cmdName} failed with status: ${constants_1.CommandStatus[status]}`));
     }
 }
 function parseFrame(frame) {
-    if (frame.length < MIN_BUFFER_SIZE) {
-        logger_1.logger.debug("received frame size to small - discard frame", NS);
-        return [null, null, null, null];
-    }
+    // at this point frame.buffer.length is at least 5 bytes long
     const view = new DataView(frame.buffer);
     const commandId = view.getUint8(0);
     const seqNumber = view.getUint8(1);
     const status = view.getUint8(2);
-    //const frameLength = view.getUint16(3, littleEndian);
-    //const payloadLength = view.getUint16(5, littleEndian);
     const parser = getParserForCommandId(commandId);
     return [seqNumber, status, parser(view), commandId];
 }