@matter/types 0.13.0-alpha.0-20250318-c1aa38b08 → 0.13.0-alpha.0-20250323-770919c6a

package/src/globals/Status.ts

@@ -89,9 +89,9 @@ export enum Status {
     UnsupportedAttribute = 134,
 
     /**
-     * Out of range error or set to a reserved value. Attribute keeps its old value. Note that an attribute value may
-     * be out of range if an attribute is related to another, e.g. with minimum and maximum attributes. See the
-     * individual attribute descriptions for specific details.
+     * Out of range error or set to a reserved value. Attribute keeps its old value. Note that an attribute value may be
+     * out of range if an attribute is related to another, e.g. with minimum and maximum attributes. See the individual
+     * attribute descriptions for specific details.
      *
      * @see {@link MatterSpecification.v13.Core} § 8.10.1
      */
@@ -190,8 +190,8 @@ export enum Status {
     NoUpstreamSubscription = 197,
 
     /**
-     * A Untimed Write or Untimed Invoke interaction was used for an attribute or command that requires a Timed Write
-     * or Timed Invoke.
+     * A Untimed Write or Untimed Invoke interaction was used for an attribute or command that requires a Timed Write or
+     * Timed Invoke.
      *
      * @see {@link MatterSpecification.v13.Core} § 8.10.1
      */
@@ -241,8 +241,8 @@ export enum Status {
     NoCommandResponse = 204,
 
     /**
-     * The node requires updated TC acceptance. The user MAY be directed to visit the EnhancedSetupFlowMaintenanceUrl
-     * to complete this.
+     * The node requires updated TC acceptance. The user MAY be directed to visit the EnhancedSetupFlowMaintenanceUrl to
+     * complete this.
      *
      * @see {@link MatterSpecification.v13.Core} § 8.10.1
      */

package/src/globals/WildcardPathFlags.ts

@@ -9,8 +9,8 @@
 import { BitFlag } from "../schema/BitmapSchema.js";
 
 /**
- * The WildcardPathFlagsBitmap indicates flags that apply to the path, affecting wildcard expansion. The following
- * flags are defined:
+ * The WildcardPathFlagsBitmap indicates flags that apply to the path, affecting wildcard expansion. The following flags
+ * are defined:
  *
  * @see {@link MatterSpecification.v13.Core} § 8.9.2.3
  */
@@ -30,9 +30,6 @@ export const WildcardPathFlags = {
      */
     wildcardSkipAttributeList: BitFlag(2),
 
-    /**
-     * Reserved
-     */
     reserved: BitFlag(3),
 
     /**

package/src/tlv/TlvAny.ts

@@ -4,7 +4,7 @@
  * SPDX-License-Identifier: Apache-2.0
  */
 
-import { Logger, UnexpectedDataError } from "#general";
+import { Diagnostic, UnexpectedDataError } from "#general";
 import { ValidationDatatypeMismatchError } from "../common/ValidationError.js";
 import { TlvCodec, TlvTag, TlvType, TlvTypeLength } from "./TlvCodec.js";
 import { TlvArrayReader, TlvElement, TlvReader, TlvSchema, TlvStream, TlvWriter } from "./TlvSchema.js";
@@ -120,7 +120,9 @@ export class AnySchema extends TlvSchema<TlvStream> {
         const result = this.decodeGenericElement(reader);
         const nextElement = reader.readTagType();
         if (nextElement !== undefined) {
-            throw new UnexpectedDataError(`Unexpected data left after parsing all data: ${Logger.toJSON(nextElement)}`);
+            throw new UnexpectedDataError(
+                `Unexpected data left after parsing all data: ${Diagnostic.json(nextElement)}`,
+            );
         }
         return result;
     }
@@ -141,7 +143,7 @@ export class AnySchema extends TlvSchema<TlvStream> {
             case TlvType.Utf8String:
             case TlvType.ByteString:
                 if (tag !== undefined && !allowTag) {
-                    throw new UnexpectedDataError(`Tag detected for a native type: ${Logger.toJSON(element)}`);
+                    throw new UnexpectedDataError(`Tag detected for a native type: ${Diagnostic.json(element)}`);
                 }
                 return reader.readPrimitive(element.typeLength);
             case TlvType.Array:
@@ -164,7 +166,7 @@ export class AnySchema extends TlvSchema<TlvStream> {
             } = element;
             if (type === TlvType.EndOfContainer) break;
             if (tag !== undefined && !allowTag) {
-                throw new UnexpectedDataError(`Tag detected : ${Logger.toJSON(element)}`);
+                throw new UnexpectedDataError(`Tag detected : ${Diagnostic.json(element)}`);
             }
             result.push(this.decodeGenericElement(reader, element, allowTag));
         }
@@ -181,7 +183,7 @@ export class AnySchema extends TlvSchema<TlvStream> {
             } = element;
             if (type === TlvType.EndOfContainer) break;
             if (tag === undefined || tag.id === undefined) {
-                throw new UnexpectedDataError(`Tag missing for a structure: ${Logger.toJSON(element)}`);
+                throw new UnexpectedDataError(`Tag missing for a structure: ${Diagnostic.json(element)}`);
             }
             result[tag.id] = this.decodeGenericElement(reader, element, true);
         }

package/src/tlv/TlvNullable.ts

@@ -38,7 +38,16 @@ export class NullableSchema<T> extends TlvSchema<T | null> {
             (this.schema instanceof ArraySchema || this.schema instanceof StringSchema) &&
             (value as any).length === 0
         ) {
-            return null;
+            // But because of Spec vs SDK interpretation issues we only map empty data where the min length is >0 as null
+            // and leave the handling of null vs "" (and only for empty strings and arrays) to the implementation.
+            // see https://github.com/CHIP-Specifications/connectedhomeip-spec/issues/11387
+            // Empty UInt8Arrays are mapped to null as before for convenience until this also makes issues in the future.
+            if (
+                (this.schema instanceof StringSchema && this.schema.type === TlvType.ByteString) ||
+                (this.schema.minLength !== undefined && this.schema.minLength > 0)
+            ) {
+                return null;
+            }
         }
         return value;
     }

package/src/tlv/TlvObject.ts

@@ -62,6 +62,7 @@ export type TypeFromFields<F extends TlvFields> = Merge<
  * @see {@link MatterSpecification.v10.Core} § A.5.1 and § A.11.4
  */
 export class ObjectSchema<F extends TlvFields> extends TlvSchema<TypeFromFields<F>> {
+    readonly isFabricScoped: boolean;
     private readonly fieldById = new Array<{ name: string; field: FieldType<any> }>();
 
     constructor(
@@ -71,6 +72,7 @@ export class ObjectSchema<F extends TlvFields> extends TlvSchema<TypeFromFields<
     ) {
         super();
 
+        let isFabricScoped = false;
         // TODO Add sorting option to enforce order of fields in encoded TLV If Ty is Structure
         //  Requirements @see {@link MatterSpecification.Core.v12} § A.2.4
         for (const name in this.fieldDefinitions) {
@@ -79,7 +81,11 @@ export class ObjectSchema<F extends TlvFields> extends TlvSchema<TypeFromFields<
                 throw new Error("Repeated fields are only allowed in TLV List.");
             }
             this.fieldById[field.id] = { name, field };
+            if (field.id === FabricIndex.id) {
+                isFabricScoped = true;
+            }
         }
+        this.isFabricScoped = isFabricScoped;
     }
 
     #encodeEntryToTlv(writer: TlvWriter, name: string, value: TypeFromFields<F>, options?: TlvEncodingOptions) {

package/src/tlv/TlvString.ts

@@ -22,7 +22,7 @@ type LengthConstraints = {
  */
 export class StringSchema<T extends TlvType.ByteString | TlvType.Utf8String> extends TlvSchema<TlvToPrimitive[T]> {
     constructor(
-        private type: T,
+        readonly type: T,
         readonly minLength: number = 0,
         readonly maxLength: number = 1024,
     ) {