@matter-server/ws-controller 0.5.7 → 0.5.9

package/src/controller/LegacyDataInjector.ts

@@ -95,20 +95,20 @@ export interface LegacyVendorInfo {
 
 /** Node data from Python Matter Server nodes map */
 export interface LegacyNodeData {
-    node_id: number;
+    node_id: number | bigint;
     date_commissioned: string;
     last_interview: string;
     interview_version: number;
     available: boolean;
     is_bridge: boolean;
     attributes: Record<string, unknown>;
-    attribute_subscriptions: unknown[];
+    attribute_subscriptions: readonly [];
 }
 
 /** Structure of the <compressedFabricId>.json file */
 export interface LegacyServerFile {
     vendor_info: Record<string, LegacyVendorInfo>;
-    last_node_id: number;
+    last_node_id: number | bigint;
     nodes: Record<string, LegacyNodeData>;
 }
 
@@ -279,9 +279,13 @@ export namespace LegacyDataInjector {
             }
             commissionedNodes.push([NodeId(BigInt(nodeId)), {}]);
             let newNode = true;
+            const attributes = Object.entries(nodeDetails.attributes);
+            if (attributes.length === 0) {
+                continue;
+            }
             logger.info(`Injecting node ${nodeId} into storage`);
             const nodeWrites = new Array<MaybePromise<void>>();
-            for (const [attributeKey, value] of Object.entries(nodeDetails.attributes)) {
+            for (const [attributeKey, value] of attributes) {
                 let currentEndpointId: string | undefined;
                 let currentClusterId: string | undefined;
                 let endpointStorage: StorageContext | undefined;

package/src/controller/MatterController.ts

@@ -122,9 +122,10 @@ export class MatterController {
 
             // Check if the nextNodeId needs to be updated based on legacy data
             const lastNodeId = legacyData.nodeData?.last_node_id;
-            if (typeof lastNodeId === "number") {
-                if (config.nextNodeId <= lastNodeId) {
-                    const newNextNodeId = lastNodeId + 10;
+            if (typeof lastNodeId === "number" || typeof lastNodeId === "bigint") {
+                // Compare as BigInt to safely handle both number and bigint types
+                if (BigInt(config.nextNodeId) <= BigInt(lastNodeId)) {
+                    const newNextNodeId = BigInt(lastNodeId) + 10n;
                     logger.info(
                         `Updating nextNodeId from ${config.nextNodeId} to ${newNextNodeId} (legacy last_node_id: ${lastNodeId})`,
                     );

package/src/model/ModelMapper.ts

@@ -15,7 +15,7 @@ import {
     EventModel,
     FeatureMap,
     GeneratedCommandList,
-    MatterModel,
+    Matter,
 } from "@matter/main/model";
 
 type AttributeDetails = { readonly [key: string]: AttributeModel | undefined };
@@ -53,7 +53,7 @@ export type ClusterMapType = {
 // Build the cluster map at module load time
 const clusterMapBuilder: { [key: string]: ClusterMapEntry | undefined } = {};
 
-MatterModel.standard.clusters.forEach(cluster => {
+Matter.clusters.forEach(cluster => {
     if (cluster.id === undefined) {
         return;
     } // Skip clusters without an ID

package/src/server/ConfigStorage.ts

@@ -20,7 +20,7 @@ function sanitizeForLog(key: string, value: unknown): string {
 
 interface ConfigData {
     fabricLabel: string;
-    nextNodeId: number; // formally wrong, should be bigint
+    nextNodeId: number | bigint;
     wifiSsid?: string;
     wifiCredentials?: string;
     threadDataset?: string;
@@ -68,7 +68,7 @@ export class ConfigStorage {
         const fabricLabel = (await this.#configStore.has("fabricLabel"))
             ? await this.#configStore.get<string>("fabricLabel")
             : (this.#env.vars.string("fabricLabel") ?? this.#data.fabricLabel);
-        const nextNodeId = await this.#configStore.get<number>("nextNodeId", this.#data.nextNodeId);
+        const nextNodeId = await this.#configStore.get<number | bigint>("nextNodeId", this.#data.nextNodeId);
 
         const wifiSsid = (await this.#configStore.has("wifiSsid"))
             ? await this.#configStore.get<string>("wifiSsid", "")

package/src/server/Converters.ts

@@ -90,7 +90,7 @@ export function convertWebSocketTagBasedToMatter(
 
     // Handle lists
     if (Array.isArray(value) && model.type === "list") {
-        return value.map(v => convertWebSocketTagBasedToMatter(v, model.members[0], clusterModel));
+        return value.map(v => convertWebSocketTagBasedToMatter(v, model.members.at(0), clusterModel));
     }
 
     // Handle structs - convert numeric keys to camelCased member names
@@ -136,7 +136,7 @@ export function convertCommandDataToMatter(
 
     // Handle lists
     if (Array.isArray(value) && model.type === "list") {
-        return value.map(v => convertCommandDataToMatter(v, model.members[0], clusterModel));
+        return value.map(v => convertCommandDataToMatter(v, model.members.at(0), clusterModel));
     }
 
     // Handle structs - convert numeric keys to camelCased member names
@@ -303,7 +303,7 @@ function convertMatterToWebSocket(
 
         case ConvKind.List:
             return Array.isArray(value)
-                ? value.map(v => convertMatterToWebSocket(v, model.members[0], clusterModel, tagBased))
+                ? value.map(v => convertMatterToWebSocket(v, model.members.at(0), clusterModel, tagBased))
                 : value;
 
         case ConvKind.Struct: {
@@ -527,7 +527,7 @@ export function convertWebsocketDataToMatter(value: any, model: ValueModel): any
             value = parseChipJSON(value);
         }
         if (Array.isArray(value)) {
-            return value.map(v => convertWebsocketDataToMatter(v, model.members[0]));
+            return value.map(v => convertWebsocketDataToMatter(v, model.members.at(0)!));
         }
     }
 

package/src/server/WebSocketControllerHandler.ts

@@ -361,7 +361,6 @@ export class WebSocketControllerHandler implements WebServerHandler {
                 }
             }
         });
-        console.log("send close to clients");
 
         const wss = this.#wss;
         // Wait for the WebSocket server to close properly
@@ -379,7 +378,7 @@ export class WebSocketControllerHandler implements WebServerHandler {
     async #handleWebSocketRequest(
         connId: string,
         data: string,
-    ): Promise<{ response: ErrorResultMessage | SuccessResultMessage<any>; enableListeners?: boolean }> {
+    ): Promise<{ response: ErrorResultMessage | SuccessResultMessage; enableListeners?: boolean }> {
         let messageId: string | undefined;
         try {
             logger.debug(`[${connId}] WebSocket request`, () => data);
@@ -603,7 +602,9 @@ export class WebSocketControllerHandler implements WebServerHandler {
             }
         }
 
-        await this.#config.set({ nextNodeId: nextNodeId + 1 });
+        await this.#config.set({
+            nextNodeId: typeof nextNodeId === "bigint" ? nextNodeId + 1n : nextNodeId + 1,
+        });
 
         const { nodeId } = await this.#commandHandler.commissionNode({
             nodeId: NodeId(nextNodeId),
@@ -657,7 +658,9 @@ export class WebSocketControllerHandler implements WebServerHandler {
                 break;
         }
 
-        await this.#config.set({ nextNodeId: nextNodeId + 1 });
+        await this.#config.set({
+            nextNodeId: typeof nextNodeId === "bigint" ? nextNodeId + 1n : nextNodeId + 1,
+        });
 
         const { nodeId } = await this.#commandHandler.commissionNode(commissionRequest);
 
@@ -796,7 +799,6 @@ export class WebSocketControllerHandler implements WebServerHandler {
             cluster_id: clusterId,
             command_name: commandName,
             payload,
-            response_type,
             timed_request_timeout_ms: timedInteractionTimeoutMs,
         } = args;
 
@@ -810,8 +812,8 @@ export class WebSocketControllerHandler implements WebServerHandler {
                 typeof timedInteractionTimeoutMs === "number" ? Millis(timedInteractionTimeoutMs) : undefined,
         });
 
-        // Test nodes and null response_type return null
-        if (TestNodeCommandHandler.isTestNodeId(nodeId) || response_type === null) {
+        // Test nodes return null
+        if (TestNodeCommandHandler.isTestNodeId(nodeId)) {
             return null;
         }
         const cmdResult = this.#convertCommandDataToWebSocket(ClusterId(clusterId), commandName, result);

package/src/tsconfig.json

@@ -1,7 +1,13 @@
 {
     "extends": "../../tools/tsc/tsconfig.lib.json",
     "compilerOptions": {
-        "types": ["globals"]
+        "types": [
+            "globals"
+        ]
     },
-    "references": []
-}
+    "references": [
+        {
+            "path": "../../ws-client/src"
+        }
+    ]
+}

package/src/types/CommandHandler.ts

@@ -211,11 +211,9 @@ export type OpenCommissioningWindowResponse = {
     qrCode: string;
 };
 
-import { AttributesData, MatterNode } from "./WebSocketMessageTypes.js";
+import type { AttributesData, MatterNodeData } from "@matter-server/ws-client";
 
-/** MatterNode details for WebSocket API - re-export from WebSocketMessageTypes */
-export type { AttributesData };
-export type MatterNodeData = MatterNode;
+export type { AttributesData, MatterNodeData };
 
 /**
  * Interface for node command handlers.