feat: enhance UUID handling in Primitive class and add corresponding tests

Signed-off-by: Vladislav Polyakov <[email protected]>

Author: polRk

packages/value/src/index.ts

@@ -28,7 +28,12 @@ export type JSValue =
 export function fromYdb(value: Ydb.Value, type: Ydb.Type): Value {
 	switch (type.type.case) {
 		case 'typeId':
-			return new Primitive({ value: value.value, high128: value.high128 }, new PrimitiveType(type.type.value))
+			let pValue = new Primitive({ value: value.value }, new PrimitiveType(type.type.value))
+			if (value.high128) {
+				pValue.high128 = value.high128
+			}
+
+			return pValue
 		case 'listType':
 			return new List(...value.items.map((v) => fromYdb(v, (type.type.value as unknown as Ydb.ListType).item!)))
 		case 'tupleType':

packages/value/src/primitive.ts

@@ -5,7 +5,7 @@ import * as Ydb from '@ydbjs/api/value'
 import { type GenericDateConstructor, formatISO9075 } from 'date-fns'
 
 import { type Type, TypeKind } from './type.js'
-import { bigIntsFromUuid } from './uuid.js'
+import { bigIntsFromUuid, uuidFromBigInts } from './uuid.js'
 import { type Value } from './value.js'
 
 export class PrimitiveType implements Type {
@@ -51,7 +51,6 @@ export class Primitive implements Value<PrimitiveType> {
 		this.type = type
 		this.value = value?.value?.value
 		this.#value = value
-		this.high128 = value.high128
 	}
 
 	encode(): Ydb.Value {
@@ -282,15 +281,21 @@ export class UuidType extends PrimitiveType {
 }
 
 export class Uuid extends Primitive implements Value<UuidType> {
+	low128: bigint = 0n
 	override high128: bigint = 0n
 
 	constructor(value: string) {
 		let { low128, high128 } = bigIntsFromUuid(value)
 
 		super({ value: { case: 'low128', value: low128 }, high128 }, new UuidType())
 
+		this.low128 = low128
 		this.high128 = high128
 	}
+
+	override toString(): string {
+		return uuidFromBigInts(this.low128, this.high128)
+	}
 }
 
 export class DateType extends PrimitiveType {

packages/value/tests/primitive.test.ts

@@ -11,6 +11,7 @@ import {
 	Timestamp,
 	Uint32,
 	Uint64,
+	Uuid,
 } from '../dist/esm/primitive.js'
 
 test('Bool primitive', async (t) => {
@@ -144,3 +145,18 @@ test('Timestamp primitive', async (t) => {
 		}
 	`)
 })
+
+test('UUID primitive', async (t) => {
+	const uuid = new Uuid('00112233-4455-6677-8899-aabbccddeeff')
+
+	t.expect(uuid).toMatchInlineSnapshot(`
+		Uuid {
+		  "high128": 18441921395520346504n,
+		  "low128": 7383445245961249331n,
+		  "type": UuidType {},
+		  "value": 7383445245961249331n,
+		}
+	`)
+
+	t.expect(uuid.toString()).toBe('00112233-4455-6677-8899-aabbccddeeff')
+})