@nxtedition/deepstream.io-client-js 31.2.19 → 32.0.2

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@nxtedition/deepstream.io-client-js",
-  "version": "31.2.19",
+  "version": "32.0.2",
   "description": "the javascript client for deepstream.io",
   "homepage": "http://deepstream.io",
   "type": "module",
@@ -36,36 +36,36 @@
     "singleQuote": true
   },
   "dependencies": {
-    "@nxtedition/json-path": "^1.0.8",
-    "bufferutil": "^4.0.8",
+    "@nxtedition/json-path": "^2.0.2",
+    "bufferutil": "^4.1.0",
     "component-emitter2": "^1.3.5",
     "invariant": "^2.2.4",
     "lodash.clonedeep": "^4.5.0",
-    "utf-8-validate": "^6.0.5",
+    "utf-8-validate": "^6.0.6",
     "varint": "^6.0.0",
-    "ws": "^8.18.0",
+    "ws": "^8.19.0",
     "xuid": "^4.1.3",
     "xxhash-wasm": "^1.0.2"
   },
   "devDependencies": {
-    "@types/node": "^22.10.3",
-    "eslint": "^9.14.0",
-    "eslint-config-prettier": "^9.1.0",
+    "@types/node": "^25.0.10",
+    "eslint": "^9.39.2",
+    "eslint-config-prettier": "^10.1.8",
     "eslint-config-standard": "^17.1.0",
     "eslint-plugin-import": "^2.31.0",
-    "eslint-plugin-n": "^17.12.0",
+    "eslint-plugin-n": "^17.23.2",
     "eslint-plugin-node": "^11.1.0",
     "eslint-plugin-promise": "^7.1.0",
     "husky": "^9.1.6",
-    "lint-staged": "^15.2.10",
+    "lint-staged": "^16.2.7",
     "mitata": "^1.0.10",
     "pinst": "^3.0.0",
-    "prettier": "^3.3.3",
+    "prettier": "^3.8.1",
     "rxjs": "^7.8.1",
     "tsd": "^0.33.0",
-    "type-fest": "^4.33.0",
+    "type-fest": "^5.4.1",
     "typescript": "^5.6.3",
-    "typescript-eslint": "^8.12.2"
+    "typescript-eslint": "^8.53.1"
   },
   "peerDependencies": {
     "rxjs": ">=6.x"

package/src/client.test-d.ts

@@ -96,6 +96,9 @@ expectError(ds.record.set('n', 'n1.x2', {}))
 expectError(ds.record.set('n', 'n1.n2.n3', { n4: 22 }))
 
 expectAssignable<string>(await ds.record.get('p', 'p1'))
+expectAssignable<{ name: string; version: string; state: number; data: string }>(
+  await ds.record.get2('p', 'p1'),
+)
 expectAssignable<string>(await ds.record.get('p', 'p1', { signal: new AbortController().signal }))
 expectAssignable<string>(await ds.record.get('p', { path: 'p1' }))
 expectAssignable<string | undefined>(await ds.record.get('p', 'p2'))

package/src/record/record-handler.d.ts

@@ -113,6 +113,73 @@ export default class RecordHandler<Records = Record<string, unknown>> {
     ): Observable<Get<Lookup<Records, Name>, Path>>
   }
 
+  observe2: {
+    <Name extends string>(
+      name: Name,
+      options: ObserveOptions,
+    ): Observable<{
+      name: string
+      version: string
+      state: number
+      data: Lookup<Records, Name>
+    }>
+
+    <Name extends string, Path extends string | string[]>(
+      name: Name,
+      options: ObserveOptionsWithPath<Path>,
+    ): Observable<{
+      name: string
+      version: string
+      state: number
+      data: Get<Lookup<Records, Name>, Path>
+    }>
+
+    <Name extends string>(
+      name: Name,
+      state?: number,
+      options?: ObserveOptions,
+    ): Observable<{
+      name: string
+      version: string
+      state: number
+      data: Lookup<Records, Name>
+    }>
+
+    <Name extends string, Path extends string | string[]>(
+      name: Name,
+      state?: number,
+      options?: ObserveOptionsWithPath<Path>,
+    ): Observable<{
+      name: string
+      version: string
+      state: number
+      data: Get<Lookup<Records, Name>, Path>
+    }>
+
+    <Name extends string, Path extends string | string[]>(
+      name: Name,
+      path: Path,
+      options?: ObserveOptionsWithPath<Path>,
+    ): Observable<{
+      name: string
+      version: string
+      state: number
+      data: Get<Lookup<Records, Name>, Path>
+    }>
+
+    <Name extends string, Path extends string | string[]>(
+      name: Name,
+      path: Path,
+      state?: number,
+      options?: ObserveOptionsWithPath<Path>,
+    ): Observable<{
+      name: string
+      version: string
+      state: number
+      data: Get<Lookup<Records, Name>, Path>
+    }>
+  }
+
   get: {
     <Name extends string>(name: Name, options: ObserveOptions): Promise<Lookup<Records, Name>>
 
@@ -147,11 +214,11 @@ export default class RecordHandler<Records = Record<string, unknown>> {
     ): Promise<Get<Lookup<Records, Name>, Path>>
   }
 
-  observe2: {
+  get2: {
     <Name extends string>(
       name: Name,
       options: ObserveOptions,
-    ): Observable<{
+    ): Promise<{
       name: string
       version: string
       state: number
@@ -161,7 +228,7 @@ export default class RecordHandler<Records = Record<string, unknown>> {
     <Name extends string, Path extends string | string[]>(
       name: Name,
       options: ObserveOptionsWithPath<Path>,
-    ): Observable<{
+    ): Promise<{
       name: string
       version: string
       state: number
@@ -172,7 +239,7 @@ export default class RecordHandler<Records = Record<string, unknown>> {
       name: Name,
       state?: number,
       options?: ObserveOptions,
-    ): Observable<{
+    ): Promise<{
       name: string
       version: string
       state: number
@@ -183,7 +250,7 @@ export default class RecordHandler<Records = Record<string, unknown>> {
       name: Name,
       state?: number,
       options?: ObserveOptionsWithPath<Path>,
-    ): Observable<{
+    ): Promise<{
       name: string
       version: string
       state: number
@@ -194,7 +261,7 @@ export default class RecordHandler<Records = Record<string, unknown>> {
       name: Name,
       path: Path,
       options?: ObserveOptionsWithPath<Path>,
-    ): Observable<{
+    ): Promise<{
       name: string
       version: string
       state: number
@@ -206,7 +273,7 @@ export default class RecordHandler<Records = Record<string, unknown>> {
       path: Path,
       state?: number,
       options?: ObserveOptionsWithPath<Path>,
-    ): Observable<{
+    ): Promise<{
       name: string
       version: string
       state: number

package/src/record/record-handler.js

@@ -44,17 +44,10 @@ function onUpdate(record, subscription) {
   }
 
   if (!subscription.synced || subscription.record.state < subscription.state) {
-    if (subscription.timeoutValue > 0) {
-      if (!subscription.timeout) {
-        subscription.timeout = timers.setTimeout(onTimeout, subscription.timeoutValue, subscription)
-      } else {
-        subscription.timeout.refresh()
-      }
-    }
     return
   }
 
-  if (subscription.timeout) {
+  if (subscription.timeout != null) {
     timers.clearTimeout(subscription.timeout)
     subscription.timeout = null
   }
@@ -101,6 +94,34 @@ function onTimeout(subscription) {
   )
 }
 
+class Subscription {
+  /** @type {unknown} */
+  subscriber = null
+  /** @type {unknown} */
+  path = null
+  /** @type {number} */
+  state = 0
+  /** @type {AbortSignal|null} */
+  signal = null
+  /** @type {boolean} */
+  dataOnly = false
+
+  /** @type {Record|null} */
+  record = null
+  /** @type {Timeout|null} */
+  timeout = null
+  /** @type {Function?} */
+  abort = null
+  /** @type {object|Array} */
+  data = kEmpty
+  /** @type {boolean} */
+  synced = false
+
+  index = -1
+
+  onUpdate = onUpdate
+}
+
 class RecordHandler {
   constructor(options, connection, client) {
     this.JSON = jsonPath
@@ -163,13 +184,9 @@ class RecordHandler {
   _onPruning(rec, value) {
     if (value) {
       this._stats.pruning += 1
-    } else {
-      this._stats.pruning -= 1
-    }
-
-    if (value) {
       this._pruning.add(rec)
     } else {
+      this._stats.pruning -= 1
       this._pruning.delete(rec)
     }
   }
@@ -242,15 +259,16 @@ class RecordHandler {
     }
 
     let record = this._records.get(name)
-
     if (!record) {
       record = new Record(name, this)
       this._stats.records += 1
       this._stats.created += 1
       this._records.set(name, record)
+    } else {
+      record.ref()
     }
 
-    return record.ref()
+    return record
   }
 
   provide(pattern, callback, options) {
@@ -638,36 +656,19 @@ class RecordHandler {
     }
 
     return new rxjs.Observable((subscriber) => {
-      // TODO (perf): Make a class
-      const subscription = {
-        /** @readonly @type {unknown} */
-        subscriber,
-        /** @readonly @type {unknown} */
-        path,
-        /** @readonly @type {number} */
-        state,
-        /** @type {AbortSignal|null} */
-        signal,
-        /** @readonly @type {boolean} */
-        dataOnly,
-        /** @readonly @type {number} */
-        timeoutValue: timeout,
-
-        /** @type {Record|null} */
-        record: null,
-        /** @type {Timeout|null} */
-        timeout: null,
-        /** @type {Function?} */
-        abort: null,
-        /** @type {object|Array} */
-        data: kEmpty,
-        /** @type {boolean} */
-        synced: false,
-
-        index: -1,
-        onUpdate,
+      if (signal?.aborted) {
+        subscriber.error(new utils.AbortError())
+        return
       }
 
+      const subscription = new Subscription()
+
+      subscription.subscriber = subscriber
+      subscription.path = path
+      subscription.state = state
+      subscription.signal = signal
+      subscription.dataOnly = dataOnly
+
       subscriber.add(() => {
         if (subscription.timeout) {
           timers.clearTimeout(subscription.timeout)
@@ -700,6 +701,10 @@ class RecordHandler {
       } else {
         onSync(subscription)
       }
+
+      if (timeout > 0 && (!subscription.synced || subscription.record.state < subscription.state)) {
+        subscription.timeout = timers.setTimeout(onTimeout, timeout, subscription)
+      }
     })
   }
 

package/src/record/record.js

@@ -16,9 +16,9 @@ class Record {
     this._handler = handler
     this._name = name
     this._version = ''
-    this._data = jsonPath.EMPTY
+    this._data = jsonPath.EMPTY_OBJ
     this._state = C.RECORD_STATE.VOID
-    this._refs = 0
+    this._refs = 1
     this._subscriptions = []
 
     /** @type {Array|null} */
@@ -141,6 +141,11 @@ class Record {
       throw new Error('already observing')
     }
 
+    if (this._emittingArr === this._observers) {
+      // TODO (perf): Shift from start if emitting?
+      this._observers = this._observers.slice()
+    }
+
     subscription.index = this._observers.push(subscription) - 1
   }