@bobfrankston/lxlan 0.1.18 → 0.1.19

package/README.md

@@ -25,68 +25,87 @@ It does **NOT** know:
 │  • LxClient - device discovery & management     │
 │  • LxDevice - individual device control         │
 │  • LxProtocol - message encode/decode           │
-│  • LxTransport - INTERFACE ONLY                 │
+│  • LxTransport - alias for UdpTransport         │
 ├─────────────────────────────────────────────────┤
-│  Transport Implementation (pick one):           │
-│  • @bobfrankston/lxlan-node     → rmfudp        │
-│  • @bobfrankston/lxlan-browser  → httpudp-client│
+│  Shared Transport (pick one):                   │
+│  • @bobfrankston/node-transport    → Node dgram │
+│  • @bobfrankston/browser-transport → httpudp WS │
+├─────────────────────────────────────────────────┤
+│  @bobfrankston/udp-transport (interface only)   │
 └─────────────────────────────────────────────────┘
 ```
 
+**Legacy wrappers** (`lxlan-node`, `lxlan-browser`) pre-wire transport + event emitter and re-export everything. These are **deprecated** — use `@bobfrankston/lxlan` directly with a shared transport package instead (see Direct Usage below).
+
 ## Transport Abstraction
 
-The package exports `LxTransport` interface that must be implemented by transport layers:
+`LxTransport` is an alias for `UdpTransport` from `@bobfrankston/udp-transport`:
 
 ```typescript
-export interface LxTransport {
-  bind(): Promise<void>;
-  close(): void;
-  send(ip: string, port: number, data: Buffer): void;
-  broadcast(data: Buffer, port: number, addresses: string[]): void;
-  onMessage(handler: (data: Buffer, rinfo: RemoteInfo) => void): void;
-  onError(handler: (err: Error) => void): void;
+export interface UdpTransport {
+    bind(): Promise<void>;
+    close(): void;
+    send(ip: string, port: number, data: Uint8Array): void;
+    broadcast(data: Uint8Array, port: number): void;
+    onMessage(handler: (data: Uint8Array, rinfo: RemoteInfo) => void): void;
+    onError(handler: (err: Error) => void): void;
+    onClose(handler: () => void): void;
 }
 ```
 
-**Implementations:**
-- **Node.js**: `@bobfrankston/lxlan-node` provides `rmfudp`-based transport
-- **Browser**: `@bobfrankston/lxlan-browser` provides `httpudp-client`-based transport
+**Shared transport implementations:**
+- **Node.js**: `NodeUdpTransport` from `@bobfrankston/node-transport` (dgram)
+- **Browser**: `BrowserUdpTransport` from `@bobfrankston/browser-transport` (httpudp WebSocket)
 
-## Installation
+See [transport-architecture.md](../../../../doc/transport-architecture.md) for the full transport stack reference.
 
-This package is typically installed as a dependency of the platform-specific wrappers:
+## Installation
 
 ```bash
-# For Node.js projects
-npm install @bobfrankston/lxlan-node
-
-# For browser projects
-npm install @bobfrankston/lxlan-browser
+npm install @bobfrankston/lxlan
 ```
 
-Direct usage (if providing your own transport):
+Then pick a transport for your platform:
 
 ```bash
-npm install @bobfrankston/lxlan
+# Node.js
+npm install @bobfrankston/node-transport
+
+# Browser
+npm install @bobfrankston/browser-transport
 ```
 
-## Direct Usage (Advanced)
+> **Note:** The legacy convenience wrappers `@bobfrankston/lxlan-node` and `@bobfrankston/lxlan-browser` are **deprecated**. Use `@bobfrankston/lxlan` directly with a shared transport package instead.
 
-If you're implementing your own transport:
+## Direct Usage (No Wrapper)
+
+Pass a transport factory — LxClient creates the transport and emitter internally:
 
 ```typescript
 import { LxClient } from '@bobfrankston/lxlan';
-import { MyCustomTransport } from './my-transport';
+import { NodeUdpTransport } from '@bobfrankston/node-transport';
 
-const transport = new MyCustomTransport();
-const client = new LxClient({ transport });
+const client = new LxClient({
+    Transport: NodeUdpTransport,   // has static .create(port)
+});
 
 await client.start();
 
 client.on('device', (device) => {
-  console.log('Found:', device.label);
-  device.setPower(true);
-  device.setColor({ h: 120, s: 100, b: 50 });
+    console.log('Found:', device.label);
+    device.setPower(true);
+    device.setColor({ h: 120, s: 100, b: 50 });
+});
+```
+
+For custom transport config (e.g., browser with specific httpudp URL):
+
+```typescript
+import { LxClient } from '@bobfrankston/lxlan';
+import { BrowserUdpTransport } from '@bobfrankston/browser-transport';
+
+const client = new LxClient({
+    Transport: { create: (port) => new BrowserUdpTransport({ httpudpUrl: 'ws://myserver:8080' }) },
 });
 ```
 
@@ -101,6 +120,10 @@ client.on('device', (device) => {
 - 🔄 Automatic retry with configurable timeouts
 - 📝 Comprehensive device info (firmware, WiFi, uptime)
 
+## LxDevice
+
+Extends [`DeviceBase`](../../../../Others/deviceapi/devdefs) from `@bobfrankston/devdefs` — shared state (`mac`, `ip`, `port`, `power`, `label`, `online`, `lastSeen`, `markSeen()`) and transport plumbing. See [devdefs README](../../../../Others/deviceapi/devdefs/README.md) for base class details and MAC utilities (`mac12`, `macmac`).
+
 ## Device Control
 
 ```typescript
@@ -144,11 +167,12 @@ client.on('deviceInfo', (device) => {
 
 ## Dependencies
 
+- `@bobfrankston/devdefs` - Shared device base class (`DeviceBase`, `mac12`, `macmac`)
 - `@bobfrankston/colorlib` - Color space conversions (HSB ↔ RGB ↔ Kelvin)
 
-## Keeping Transport Packages in Sync
+## Legacy Wrappers (Deprecated)
 
-**IMPORTANT:** `lxlan-browser` and `lxlan-node` are parallel transport adapters. When modifying types or exports in this core package, ensure both adapters are updated if needed. They should expose equivalent functionality for their respective platforms.
+`lxlan-node` and `lxlan-browser` are legacy convenience wrappers that pre-wire transport + event emitter. They still work but are **deprecated** — prefer direct usage with shared transport packages. No new features will be added to the wrappers.
 
 ## Importing Types
 
@@ -160,12 +184,16 @@ import type { StateEventInfo, DeviceState } from '@bobfrankston/lxlan';
 
 ## Related Packages
 
-- **[@bobfrankston/lxlan-node](../lxlan-node)** - Node.js transport wrapper (keep in sync!)
-- **[@bobfrankston/lxlan-browser](../lxlan-browser)** - Browser transport wrapper (keep in sync!)
-- **[@bobfrankston/rmfudp](../../../../utils/udp/rmfudp)** - Node.js UDP transport
-- **[@bobfrankston/httpudp-client](../../../../utils/udp/httpudp-client)** - Browser WebSocket UDP client
+- **[@bobfrankston/devdefs](../../../../Others/deviceapi/devdefs)** - Shared device base class and MAC utilities
+- **[@bobfrankston/lxlan-node](../lxlan-node)** - Node.js convenience wrapper (deprecated)
+- **[@bobfrankston/lxlan-browser](../lxlan-browser)** - Browser convenience wrapper (deprecated)
+- **[@bobfrankston/udp-transport](../../../../utils/udp/udp-transport)** - UdpTransport interface definition
+- **[@bobfrankston/node-transport](../../../../utils/udp/node-transport)** - Shared Node.js UDP transport (NodeUdpTransport, NodeEventEmitter)
+- **[@bobfrankston/browser-transport](../../../../utils/udp/browser-transport)** - Shared browser UDP transport (BrowserUdpTransport, BrowserEventEmitter)
 - **[@bobfrankston/httpudp](../../../../utils/udp/httpudp)** - WebSocket-to-UDP proxy server
+- **[@bobfrankston/httpudp-client](../../../../utils/udp/httpudp-client)** - Browser WebSocket UDP client (used by browser-transport)
 - **[@bobfrankston/colorlib](../../../../utils/colorlib)** - Color utilities
+- **[Transport Architecture](../../../../doc/transport-architecture.md)** - Full transport stack reference
 
 ## TypeScript Configuration
 

package/client.d.ts

@@ -1,21 +1,18 @@
-import { LxTransport } from './transport.js';
-import { LxEventEmitter } from './events.js';
+import type { UdpTransportFactory } from '@bobfrankston/udp-transport';
 import { LxDevice } from './device.js';
 /**
  * Options for LxClient constructor
  */
 export interface LxClientOptions {
-    /** Transport layer implementation (required) */
-    transport: LxTransport;
-    /** Event emitter implementation (required - use Node EventEmitter or browser equivalent) */
-    eventEmitter: LxEventEmitter;
-    /** LIFX port (default 56700) */
+    /** Transport factory — must have .create(port) (e.g. NodeUdpTransport, BrowserUdpTransport) */
+    Transport: UdpTransportFactory;
+    /** LIFX port override (default 56700) */
     port?: number;
     /** Auto-discovery interval in ms (0 = manual only, default 0) */
     discoveryInterval?: number;
     /** Enable client-to-client state broadcast for multi-client sync (default false) */
     clientBroadcastEnabled?: boolean;
-    /** Custom logger function (default: console.log) */
+    /** Custom logger function (default: no-op) */
     logger?: (msg: string, colors?: string) => void;
 }
 /**

package/client.js

@@ -1,4 +1,6 @@
 import { hsbk16ToHsbk } from '@bobfrankston/colorlib';
+import { SimpleEventEmitter } from '@bobfrankston/udp-transport';
+import { mac12 } from '@bobfrankston/devdefs';
 import { LxDevice } from './device.js';
 import { MessageType, LIFX_PORT, MessageTypeName } from './types.js';
 import { encodeMessage, decodeMessage, decodeState, decodeStatePower, decodeStateLabel, decodeStateVersion, decodeStateGroup, decodeStateService, decodeStateHostInfo, decodeStateHostFirmware, decodeStateWifiInfo, decodeStateInfo } from './protocol.js';
@@ -33,11 +35,11 @@ export class LxClient {
     /** Cached devices by MAC address (lowercase) */
     devices = new Map();
     constructor(options) {
-        this.emitter = options.eventEmitter;
-        this.transport = options.transport;
+        this.emitter = new SimpleEventEmitter();
         this.port = options.port ?? LIFX_PORT;
+        this.transport = options.Transport.create(this.port);
         this.clientBroadcastEnabled = options.clientBroadcastEnabled ?? false;
-        this.log = options.logger ?? ((msg, colors) => console.log(msg));
+        this.log = options.logger ?? (() => { });
         this.transport.onMessage((data, rinfo) => {
             // Ensure data is Uint8Array for protocol decoder
             const buf = data instanceof Uint8Array ? data : new Uint8Array(data);
@@ -109,7 +111,7 @@ export class LxClient {
      * @returns LxDevice or undefined if not found
      */
     getDevice(mac) {
-        return this.devices.get(mac.toLowerCase());
+        return this.devices.get(mac12(mac));
     }
     /**
      * Add device manually (bypassing discovery).
@@ -121,7 +123,7 @@ export class LxClient {
      * @returns LxDevice instance (new or existing)
      */
     addDevice(mac, ip, port = LIFX_PORT) {
-        const normalizedMac = mac.toLowerCase();
+        const normalizedMac = mac12(mac);
         let device = this.devices.get(normalizedMac);
         if (!device) {
             device = new LxDevice(normalizedMac, ip);

package/device.d.ts

@@ -1,5 +1,5 @@
+import { DeviceBase } from '@bobfrankston/devdefs';
 import { HSBK, ColorInput } from '@bobfrankston/colorlib';
-import { LxTransport } from './transport.js';
 import { GroupInfo } from './types.js';
 /**
  * Represents a LIFX device on the LAN.
@@ -9,23 +9,9 @@ import { GroupInfo } from './types.js';
  * Always use `client.addDevice(mac, ip)` or obtain devices from `client.devices`.
  * Directly instantiated devices will not have a transport and will fail at runtime.
  */
-export declare class LxDevice {
-    /** MAC address (lowercase, colon-separated) */
-    readonly mac: string;
-    /** Current IP address */
-    ip: string;
-    /** UDP port (default LIFX_PORT = 56700) */
-    port: number;
-    /** Power state (true = on) */
-    power: boolean;
+export declare class LxDevice extends DeviceBase {
     /** Current color in HSBK format */
     color: HSBK;
-    /** Device label (user-assigned name) */
-    label: string;
-    /** Whether device is responding */
-    online: boolean;
-    /** Last message timestamp (Date.now()) */
-    lastSeen: number;
     /** Vendor ID (1 = LIFX) */
     vendor: number;
     /** Product ID (see Products table) */
@@ -46,7 +32,6 @@ export declare class LxDevice {
     location: GroupInfo;
     /** Recent messages for deduplication (sequence:type -> timestamp) */
     private recentMessages;
-    private transport?;
     /** Pending requests awaiting responses (message type -> timeout) */
     private pendingRequests;
     /** Default timeout for requests in milliseconds */
@@ -58,19 +43,13 @@ export declare class LxDevice {
      * Do not call directly - use `client.addDevice(mac, ip)` instead.
      * Devices created directly will not have a transport and will throw errors when used.
      *
-     * @param mac - MAC address (will be normalized to lowercase)
+     * @param mac - MAC address (will be normalized to lowercase, no separators)
      * @param ip - IP address (optional, can be discovered)
      * @internal
      */
     constructor(mac: string, ip?: string);
-    /** Throw error if transport is not set */
-    private requireTransport;
-    /** Throw error if IP address is not set */
-    private requireIP;
     /** Get product name from product ID */
     get productName(): string;
-    /** Set the transport (used by LxClient when managing devices) */
-    setTransport(transport: LxTransport): void;
     /** Send raw message to device */
     send(type: number, payload?: Buffer): void;
     /** Start tracking a request with timeout */
@@ -143,8 +122,6 @@ export declare class LxDevice {
      * Listen for 'deviceInfo' event on client to get results.
      */
     getDeviceInfo(): void;
-    /** Update last seen timestamp and online status */
-    markSeen(): void;
     /** Notify that a response was received for a message type */
     markResponseReceived(messageType: number): void;
     /**

package/device.js

@@ -1,3 +1,4 @@
+import { DeviceBase } from '@bobfrankston/devdefs';
 import { parseColor, hsbkTo16 } from '@bobfrankston/colorlib';
 import { MessageType, Products, LIFX_PORT } from './types.js';
 import { encodeMessage, encodeSetPower, encodeSetColor, encodeSetWaveformOptional, encodeSetLabel, encodeSetGroup, encodeSetLocation } from './protocol.js';
@@ -9,23 +10,9 @@ import { encodeMessage, encodeSetPower, encodeSetColor, encodeSetWaveformOptiona
  * Always use `client.addDevice(mac, ip)` or obtain devices from `client.devices`.
  * Directly instantiated devices will not have a transport and will fail at runtime.
  */
-export class LxDevice {
-    /** MAC address (lowercase, colon-separated) */
-    mac;
-    /** Current IP address */
-    ip;
-    /** UDP port (default LIFX_PORT = 56700) */
-    port = LIFX_PORT;
-    /** Power state (true = on) */
-    power = false;
+export class LxDevice extends DeviceBase {
     /** Current color in HSBK format */
     color = { h: 0, s: 0, b: 0, k: 3500 };
-    /** Device label (user-assigned name) */
-    label = '';
-    /** Whether device is responding */
-    online = false;
-    /** Last message timestamp (Date.now()) */
-    lastSeen = 0;
     /** Vendor ID (1 = LIFX) */
     vendor = 0;
     /** Product ID (see Products table) */
@@ -46,7 +33,6 @@ export class LxDevice {
     location = { id: '', label: '', updatedAt: 0 };
     /** Recent messages for deduplication (sequence:type -> timestamp) */
     recentMessages = new Map();
-    transport;
     /** Pending requests awaiting responses (message type -> timeout) */
     pendingRequests = new Map();
     /** Default timeout for requests in milliseconds */
@@ -58,32 +44,17 @@ export class LxDevice {
      * Do not call directly - use `client.addDevice(mac, ip)` instead.
      * Devices created directly will not have a transport and will throw errors when used.
      *
-     * @param mac - MAC address (will be normalized to lowercase)
+     * @param mac - MAC address (will be normalized to lowercase, no separators)
      * @param ip - IP address (optional, can be discovered)
      * @internal
      */
     constructor(mac, ip) {
-        this.mac = mac.toLowerCase();
-        this.ip = ip || '';
-    }
-    /** Throw error if transport is not set */
-    requireTransport() {
-        if (!this.transport)
-            throw new Error(`Device ${this.mac} has no UDP transport - cannot send. Use lxlan-node or lxlan-http.`);
-    }
-    /** Throw error if IP address is not set */
-    requireIP() {
-        if (!this.ip)
-            throw new Error(`Device ${this.mac} has no IP address - cannot send. Perform discovery or add device with IP using client.addDevice(mac, ip).`);
+        super(mac, ip, LIFX_PORT);
     }
     /** Get product name from product ID */
     get productName() {
         return Products[this.product] ?? `Unknown (${this.product})`;
     }
-    /** Set the transport (used by LxClient when managing devices) */
-    setTransport(transport) {
-        this.transport = transport;
-    }
     /** Send raw message to device */
     send(type, payload) {
         this.requireTransport();
@@ -197,7 +168,7 @@ export class LxDevice {
      * @param brightness - Brightness 0-100 (default 100)
      * @param duration - Transition time in milliseconds (default 0)
      */
-    setWhite(kelvin, brightness = 100, duration = 0) {
+    setWhite(kelvin, brightness = LxDevice.MaxBrightness, duration = 0) {
         this.setColor({ h: 0, s: 0, b: brightness, k: kelvin }, duration);
     }
     /** Query full state */
@@ -285,11 +256,6 @@ export class LxDevice {
         this.getInfo();
         this.getState();
     }
-    /** Update last seen timestamp and online status */
-    markSeen() {
-        this.lastSeen = Date.now();
-        this.online = true;
-    }
     /** Notify that a response was received for a message type */
     markResponseReceived(messageType) {
         this.clearRequestTimeout(messageType);

package/events.d.ts

@@ -1,24 +1,7 @@
 /**
- * Minimal EventEmitter interface for browser compatibility
- * Node.js implementations can use node:events directly
- * Browser implementations can provide their own or use a library
+ * Re-export shared event emitter types from @bobfrankston/udp-transport.
+ * LxEventEmitter is kept as an alias for backwards compatibility.
  */
-export interface LxEventEmitter {
-    on(event: string, listener: (...args: any[]) => void): this;
-    once(event: string, listener: (...args: any[]) => void): this;
-    off(event: string, listener: (...args: any[]) => void): this;
-    emit(event: string, ...args: any[]): boolean;
-    removeAllListeners(event?: string): this;
-}
-/**
- * Base class for objects that emit events
- * This is platform-agnostic and can be implemented for Node.js or browser
- */
-export declare abstract class LxEventEmitterBase implements LxEventEmitter {
-    abstract on(event: string, listener: (...args: any[]) => void): this;
-    abstract once(event: string, listener: (...args: any[]) => void): this;
-    abstract off(event: string, listener: (...args: any[]) => void): this;
-    abstract emit(event: string, ...args: any[]): boolean;
-    abstract removeAllListeners(event?: string): this;
-}
+export { LanEventEmitter, SimpleEventEmitter } from '@bobfrankston/udp-transport';
+export { LanEventEmitter as LxEventEmitter } from '@bobfrankston/udp-transport';
 //# sourceMappingURL=events.d.ts.map

package/events.js

@@ -1,7 +1,6 @@
 /**
- * Base class for objects that emit events
- * This is platform-agnostic and can be implemented for Node.js or browser
+ * Re-export shared event emitter types from @bobfrankston/udp-transport.
+ * LxEventEmitter is kept as an alias for backwards compatibility.
  */
-export class LxEventEmitterBase {
-}
+export { SimpleEventEmitter } from '@bobfrankston/udp-transport';
 //# sourceMappingURL=events.js.map

package/index.d.ts

@@ -1,15 +1,13 @@
 /**
  * @bobfrankston/lxlan — Platform-agnostic LIFX LAN protocol core.
  *
- * DO NOT import this package directly in application code.
- * Use the platform-specific wrapper instead:
- *   - Node.js:  @bobfrankston/lxlan-node
- *   - Browser:  @bobfrankston/lxlan-browser
+ * Usage: pass a UdpTransportFactory (e.g. NodeUdpTransport, BrowserUdpTransport).
+ * The client creates transport and event emitter internally.
  */
 export * from './types.js';
 export { LxDevice } from './device.js';
 export { LxClient, LxClientOptions } from './client.js';
-export { UdpTransport, LxTransport, RemoteInfo } from './transport.js';
-export { LxEventEmitter, LxEventEmitterBase } from './events.js';
+export { UdpTransport, UdpTransportFactory, LxTransport, RemoteInfo, StateEventInfo } from './transport.js';
+export { LxEventEmitter, LanEventEmitter, SimpleEventEmitter } from './events.js';
 export { encodeMessage, decodeMessage, encodeSetWifiConfiguration, encodeGetWifiConfiguration, decodeStateWifiConfiguration, encodeSetLabel, encodeSetColor, encodeSetPower, encodeSetGroup, encodeSetLocation, encodeSetWaveformOptional, encodeGetService, decodeState, decodeStatePower, decodeStateLabel, decodeStateVersion, decodeStateGroup, decodeStateService, decodeStateHostInfo, decodeStateHostFirmware, decodeStateWifiInfo, decodeStateInfo, FrameFlags, ProtocolBits, nextSequence, getSource, } from './protocol.js';
 //# sourceMappingURL=index.d.ts.map

package/index.js

@@ -1,14 +1,12 @@
 /**
  * @bobfrankston/lxlan — Platform-agnostic LIFX LAN protocol core.
  *
- * DO NOT import this package directly in application code.
- * Use the platform-specific wrapper instead:
- *   - Node.js:  @bobfrankston/lxlan-node
- *   - Browser:  @bobfrankston/lxlan-browser
+ * Usage: pass a UdpTransportFactory (e.g. NodeUdpTransport, BrowserUdpTransport).
+ * The client creates transport and event emitter internally.
  */
 export * from './types.js';
 export { LxDevice } from './device.js';
 export { LxClient } from './client.js';
-export { LxEventEmitterBase } from './events.js';
+export { SimpleEventEmitter } from './events.js';
 export { encodeMessage, decodeMessage, encodeSetWifiConfiguration, encodeGetWifiConfiguration, decodeStateWifiConfiguration, encodeSetLabel, encodeSetColor, encodeSetPower, encodeSetGroup, encodeSetLocation, encodeSetWaveformOptional, encodeGetService, decodeState, decodeStatePower, decodeStateLabel, decodeStateVersion, decodeStateGroup, decodeStateService, decodeStateHostInfo, decodeStateHostFirmware, decodeStateWifiInfo, decodeStateInfo, FrameFlags, ProtocolBits, nextSequence, getSource, } from './protocol.js';
 //# sourceMappingURL=index.js.map

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@bobfrankston/lxlan",
-  "version": "0.1.18",
+  "version": "0.1.19",
   "description": "LIFX LAN protocol library for device control via UDP",
   "type": "module",
   "main": "index.js",
@@ -32,16 +32,18 @@
   },
   "dependencies": {
     "@bobfrankston/colorlib": "^0.1.8",
+    "@bobfrankston/devdefs": "^0.1.0",
     "@bobfrankston/udp-transport": "^1.0.2"
   },
   "devDependencies": {
-    "@types/node": "^25.2.1"
+    "@types/node": "^25.3.0"
   },
   "publishConfig": {
     "access": "public"
   },
   ".dependencies": {
     "@bobfrankston/colorlib": "file:../../../../utils/colorlib",
+    "@bobfrankston/devdefs": "file:../../../../Others/deviceapi/devdefs",
     "@bobfrankston/udp-transport": "file:../../../../utils/udp/udp-transport"
   }
 }

package/protocol.js

@@ -21,17 +21,17 @@ export function nextSequence() {
 export function getSource() {
     return globalSource;
 }
-/** MAC string to 8-byte Uint8Array (6 bytes MAC + 2 padding) */
+/** MAC string to 8-byte Uint8Array (6 bytes MAC + 2 padding). Accepts bare or colon-separated. */
 function macToBuffer(mac) {
     const buf = new Uint8Array(8);
-    if (mac && mac !== '00:00:00:00:00:00') {
-        const bytes = mac.split(':').map(b => parseInt(b, 16));
+    const clean = mac.replace(/[:\-]/g, '');
+    if (clean && clean !== '000000000000') {
         for (let i = 0; i < 6; i++)
-            buf[i] = bytes[i] || 0;
+            buf[i] = parseInt(clean.substring(i * 2, i * 2 + 2), 16) || 0;
     }
     return buf;
 }
-/** Uint8Array to MAC string */
+/** Uint8Array to colon-separated lowercase MAC string */
 function bufferToMac(buf, offset = 0) {
     const parts = [];
     for (let i = 0; i < 6; i++) {

package/transport.d.ts

@@ -2,6 +2,6 @@
  * Re-export UDP transport interfaces from @bobfrankston/udp-transport
  * LxTransport is kept as an alias for backwards compatibility
  */
-export { UdpTransport, RemoteInfo } from '@bobfrankston/udp-transport';
+export { UdpTransport, UdpTransportFactory, RemoteInfo, StateEventInfo } from '@bobfrankston/udp-transport';
 export { UdpTransport as LxTransport } from '@bobfrankston/udp-transport';
 //# sourceMappingURL=transport.d.ts.map

package/types.d.ts

@@ -108,11 +108,6 @@ export interface DeviceState {
     uptime?: number;
     downtime?: number;
 }
-/** Info passed with state events */
-export interface StateEventInfo {
-    /** true if from device response, false if from peer client broadcast */
-    primary: boolean;
-}
 /** LIFX port */
 export declare const LIFX_PORT = 56700;
 /** Header size */