@tanstack/offline-transactions 1.1.0 → 1.1.1

package/src/outbox/OutboxManager.ts

@@ -1,4 +1,3 @@
-import { withSpan } from '../telemetry/tracer'
 import {
   MissingTemporalConstructorError,
   TransactionSerializer,
@@ -49,45 +48,26 @@ export class OutboxManager {
   }
 
   async add(transaction: OfflineTransaction): Promise<void> {
-    return withSpan(
-      `outbox.add`,
-      {
-        'transaction.id': transaction.id,
-        'transaction.mutationFnName': transaction.mutationFnName,
-        'transaction.keyCount': transaction.keys.length,
-      },
-      async () => {
-        const key = this.getStorageKey(transaction.id)
-        const serialized = this.serializer.serialize(transaction)
-        await this.storage.set(key, serialized)
-      },
-    )
+    const key = this.getStorageKey(transaction.id)
+    const serialized = this.serializer.serialize(transaction)
+    await this.storage.set(key, serialized)
   }
 
   async get(id: string): Promise<OfflineTransaction | null> {
-    return withSpan(`outbox.get`, { 'transaction.id': id }, async (span) => {
-      const key = this.getStorageKey(id)
-      const data = await this.storage.get(key)
-
-      if (!data) {
-        span.setAttribute(`result`, `not_found`)
-        return null
-      }
+    const key = this.getStorageKey(id)
+    const data = await this.storage.get(key)
+    if (!data) return null
 
-      try {
-        const transaction = this.serializer.deserialize(data)
-        span.setAttribute(`result`, `found`)
-        return transaction
-      } catch (error) {
-        if (error instanceof MissingTemporalConstructorError) {
-          error.message = `transaction ${id}: ${error.message}`
-          throw error
-        }
-        console.warn(`Failed to deserialize transaction ${id}:`, error)
-        span.setAttribute(`result`, `deserialize_error`)
-        return null
+    try {
+      return this.serializer.deserialize(data)
+    } catch (error) {
+      if (error instanceof MissingTemporalConstructorError) {
+        error.message = `transaction ${id}: ${error.message}`
+        throw error
       }
-    })
+      console.warn(`Failed to deserialize transaction ${id}:`, error)
+      return null
+    }
   }
 
   async getAll(): Promise<Array<OfflineTransaction>> {
@@ -100,44 +80,40 @@ export class OutboxManager {
     const read = { revision: this.removalRevision }
     this.activeReads.add(read)
     try {
-      return await withSpan(`outbox.getAll`, {}, async (span) => {
-        const keys = await this.storage.keys()
-        const transactionKeys = keys.filter((key) =>
-          key.startsWith(this.keyPrefix),
-        )
-
-        span.setAttribute(`transactionCount`, transactionKeys.length)
-
-        const transactions: Array<OfflineTransaction> = []
-
-        for (const key of transactionKeys) {
-          const data = await this.storage.get(key)
-          if (data) {
-            try {
-              const transaction = this.serializer.deserialize(data)
-              transactions.push(transaction)
-            } catch (error) {
-              if (error instanceof MissingTemporalConstructorError) {
-                error.message = `transaction ${key.slice(this.keyPrefix.length)}: ${error.message}`
-                throw error
-              }
-              console.warn(
-                `Failed to deserialize transaction from key ${key}:`,
-                error,
-              )
+      const keys = await this.storage.keys()
+      const transactionKeys = keys.filter((key) =>
+        key.startsWith(this.keyPrefix),
+      )
+
+      const transactions: Array<OfflineTransaction> = []
+
+      for (const key of transactionKeys) {
+        const data = await this.storage.get(key)
+        if (data) {
+          try {
+            const transaction = this.serializer.deserialize(data)
+            transactions.push(transaction)
+          } catch (error) {
+            if (error instanceof MissingTemporalConstructorError) {
+              error.message = `transaction ${key.slice(this.keyPrefix.length)}: ${error.message}`
+              throw error
             }
+            console.warn(
+              `Failed to deserialize transaction from key ${key}:`,
+              error,
+            )
           }
         }
+      }
 
-        const currentTransactions = transactions
-          .filter(
-            ({ id }) => (this.removedAtRevision.get(id) ?? 0) <= read.revision,
-          )
-          .sort((a, b) => a.createdAt.getTime() - b.createdAt.getTime())
-        // Keep the read registered until replay admission completes. A durable
-        // removal cannot land between filtering and this synchronous consumer.
-        return consume(currentTransactions)
-      })
+      const currentTransactions = transactions
+        .filter(
+          ({ id }) => (this.removedAtRevision.get(id) ?? 0) <= read.revision,
+        )
+        .sort((a, b) => a.createdAt.getTime() - b.createdAt.getTime())
+      // Keep the read registered until replay admission completes. A durable
+      // removal cannot land between filtering and this synchronous consumer.
+      return await consume(currentTransactions)
     } finally {
       this.activeReads.delete(read)
       this.pruneRemovalHistory()
@@ -157,29 +133,21 @@ export class OutboxManager {
     id: string,
     updates: Partial<OfflineTransaction>,
   ): Promise<void> {
-    return withSpan(`outbox.update`, { 'transaction.id': id }, async () => {
-      const existing = await this.get(id)
-      if (!existing) {
-        throw new OutboxTransactionNotFoundError(`Transaction ${id} not found`)
-      }
-
-      const updated = { ...existing, ...updates }
-      await this.add(updated)
-    })
+    const existing = await this.get(id)
+    if (!existing) {
+      throw new OutboxTransactionNotFoundError(`Transaction ${id} not found`)
+    }
+    await this.add({ ...existing, ...updates })
   }
 
   async remove(id: string): Promise<void> {
-    return withSpan(`outbox.remove`, { 'transaction.id': id }, async () => {
-      const key = this.getStorageKey(id)
-      await this.storage.delete(key)
-      this.recordRemovals([id])
-    })
+    const key = this.getStorageKey(id)
+    await this.storage.delete(key)
+    this.recordRemovals([id])
   }
 
   async removeMany(ids: Array<string>): Promise<void> {
-    return withSpan(`outbox.removeMany`, { count: ids.length }, async () => {
-      await Promise.all(ids.map((id) => this.remove(id)))
-    })
+    await Promise.all(ids.map((id) => this.remove(id)))
   }
 
   async clear(): Promise<void> {

package/dist/cjs/retry/NonRetriableError.d.cts

@@ -1 +0,0 @@
-export { NonRetriableError } from '../types.cjs';

package/dist/cjs/telemetry/tracer.cjs

@@ -1,27 +0,0 @@
-"use strict";
-Object.defineProperty(exports, Symbol.toStringTag, { value: "Module" });
-const noopSpan = {
-  setAttribute: () => {
-  },
-  setAttributes: () => {
-  },
-  setStatus: () => {
-  },
-  recordException: () => {
-  },
-  end: () => {
-  }
-};
-async function withSpan(name, attrs, fn, _options) {
-  return await fn(noopSpan);
-}
-async function withNestedSpan(name, attrs, fn, _options) {
-  return await fn(noopSpan);
-}
-function withSyncSpan(name, attrs, fn, _options) {
-  return fn(noopSpan);
-}
-exports.withNestedSpan = withNestedSpan;
-exports.withSpan = withSpan;
-exports.withSyncSpan = withSyncSpan;
-//# sourceMappingURL=tracer.cjs.map

package/dist/cjs/telemetry/tracer.cjs.map

@@ -1 +0,0 @@
-{"version":3,"file":"tracer.cjs","sources":["../../../src/telemetry/tracer.ts"],"sourcesContent":["export interface SpanAttrs {\n  [key: string]: string | number | boolean | undefined\n}\n\ninterface WithSpanOptions {\n  parentContext?: any\n}\n\n// No-op span implementation\nconst noopSpan = {\n  setAttribute: () => {},\n  setAttributes: () => {},\n  setStatus: () => {},\n  recordException: () => {},\n  end: () => {},\n}\n\n/**\n * Lightweight span wrapper with error handling.\n * No-op implementation - telemetry has been removed.\n *\n * By default, creates spans at the current context level (siblings).\n * Use withNestedSpan if you want parent-child relationships.\n */\nexport async function withSpan<T>(\n  name: string,\n  attrs: SpanAttrs,\n  fn: (span: any) => Promise<T>,\n  _options?: WithSpanOptions,\n): Promise<T> {\n  return await fn(noopSpan)\n}\n\n/**\n * Like withSpan but propagates context so child spans nest properly.\n * No-op implementation - telemetry has been removed.\n */\nexport async function withNestedSpan<T>(\n  name: string,\n  attrs: SpanAttrs,\n  fn: (span: any) => Promise<T>,\n  _options?: WithSpanOptions,\n): Promise<T> {\n  return await fn(noopSpan)\n}\n\n/**\n * Creates a synchronous span for non-async operations\n * No-op implementation - telemetry has been removed.\n */\nexport function withSyncSpan<T>(\n  name: string,\n  attrs: SpanAttrs,\n  fn: (span: any) => T,\n  _options?: WithSpanOptions,\n): T {\n  return fn(noopSpan)\n}\n\n/**\n * Get the current tracer instance\n * No-op implementation - telemetry has been removed.\n */\nexport function getTracer() {\n  return null\n}\n"],"names":[],"mappings":";;AASA,MAAM,WAAW;AAAA,EACf,cAAc,MAAM;AAAA,EAAC;AAAA,EACrB,eAAe,MAAM;AAAA,EAAC;AAAA,EACtB,WAAW,MAAM;AAAA,EAAC;AAAA,EAClB,iBAAiB,MAAM;AAAA,EAAC;AAAA,EACxB,KAAK,MAAM;AAAA,EAAC;AACd;AASA,eAAsB,SACpB,MACA,OACA,IACA,UACY;AACZ,SAAO,MAAM,GAAG,QAAQ;AAC1B;AAMA,eAAsB,eACpB,MACA,OACA,IACA,UACY;AACZ,SAAO,MAAM,GAAG,QAAQ;AAC1B;AAMO,SAAS,aACd,MACA,OACA,IACA,UACG;AACH,SAAO,GAAG,QAAQ;AACpB;;;;"}

package/dist/cjs/telemetry/tracer.d.cts

@@ -1,30 +0,0 @@
-export interface SpanAttrs {
-    [key: string]: string | number | boolean | undefined;
-}
-interface WithSpanOptions {
-    parentContext?: any;
-}
-/**
- * Lightweight span wrapper with error handling.
- * No-op implementation - telemetry has been removed.
- *
- * By default, creates spans at the current context level (siblings).
- * Use withNestedSpan if you want parent-child relationships.
- */
-export declare function withSpan<T>(name: string, attrs: SpanAttrs, fn: (span: any) => Promise<T>, _options?: WithSpanOptions): Promise<T>;
-/**
- * Like withSpan but propagates context so child spans nest properly.
- * No-op implementation - telemetry has been removed.
- */
-export declare function withNestedSpan<T>(name: string, attrs: SpanAttrs, fn: (span: any) => Promise<T>, _options?: WithSpanOptions): Promise<T>;
-/**
- * Creates a synchronous span for non-async operations
- * No-op implementation - telemetry has been removed.
- */
-export declare function withSyncSpan<T>(name: string, attrs: SpanAttrs, fn: (span: any) => T, _options?: WithSpanOptions): T;
-/**
- * Get the current tracer instance
- * No-op implementation - telemetry has been removed.
- */
-export declare function getTracer(): null;
-export {};

package/dist/esm/retry/NonRetriableError.d.ts

@@ -1 +0,0 @@
-export { NonRetriableError } from '../types.js';

package/dist/esm/telemetry/tracer.d.ts

@@ -1,30 +0,0 @@
-export interface SpanAttrs {
-    [key: string]: string | number | boolean | undefined;
-}
-interface WithSpanOptions {
-    parentContext?: any;
-}
-/**
- * Lightweight span wrapper with error handling.
- * No-op implementation - telemetry has been removed.
- *
- * By default, creates spans at the current context level (siblings).
- * Use withNestedSpan if you want parent-child relationships.
- */
-export declare function withSpan<T>(name: string, attrs: SpanAttrs, fn: (span: any) => Promise<T>, _options?: WithSpanOptions): Promise<T>;
-/**
- * Like withSpan but propagates context so child spans nest properly.
- * No-op implementation - telemetry has been removed.
- */
-export declare function withNestedSpan<T>(name: string, attrs: SpanAttrs, fn: (span: any) => Promise<T>, _options?: WithSpanOptions): Promise<T>;
-/**
- * Creates a synchronous span for non-async operations
- * No-op implementation - telemetry has been removed.
- */
-export declare function withSyncSpan<T>(name: string, attrs: SpanAttrs, fn: (span: any) => T, _options?: WithSpanOptions): T;
-/**
- * Get the current tracer instance
- * No-op implementation - telemetry has been removed.
- */
-export declare function getTracer(): null;
-export {};

package/dist/esm/telemetry/tracer.js

@@ -1,27 +0,0 @@
-const noopSpan = {
-  setAttribute: () => {
-  },
-  setAttributes: () => {
-  },
-  setStatus: () => {
-  },
-  recordException: () => {
-  },
-  end: () => {
-  }
-};
-async function withSpan(name, attrs, fn, _options) {
-  return await fn(noopSpan);
-}
-async function withNestedSpan(name, attrs, fn, _options) {
-  return await fn(noopSpan);
-}
-function withSyncSpan(name, attrs, fn, _options) {
-  return fn(noopSpan);
-}
-export {
-  withNestedSpan,
-  withSpan,
-  withSyncSpan
-};
-//# sourceMappingURL=tracer.js.map

package/dist/esm/telemetry/tracer.js.map

@@ -1 +0,0 @@
-{"version":3,"file":"tracer.js","sources":["../../../src/telemetry/tracer.ts"],"sourcesContent":["export interface SpanAttrs {\n  [key: string]: string | number | boolean | undefined\n}\n\ninterface WithSpanOptions {\n  parentContext?: any\n}\n\n// No-op span implementation\nconst noopSpan = {\n  setAttribute: () => {},\n  setAttributes: () => {},\n  setStatus: () => {},\n  recordException: () => {},\n  end: () => {},\n}\n\n/**\n * Lightweight span wrapper with error handling.\n * No-op implementation - telemetry has been removed.\n *\n * By default, creates spans at the current context level (siblings).\n * Use withNestedSpan if you want parent-child relationships.\n */\nexport async function withSpan<T>(\n  name: string,\n  attrs: SpanAttrs,\n  fn: (span: any) => Promise<T>,\n  _options?: WithSpanOptions,\n): Promise<T> {\n  return await fn(noopSpan)\n}\n\n/**\n * Like withSpan but propagates context so child spans nest properly.\n * No-op implementation - telemetry has been removed.\n */\nexport async function withNestedSpan<T>(\n  name: string,\n  attrs: SpanAttrs,\n  fn: (span: any) => Promise<T>,\n  _options?: WithSpanOptions,\n): Promise<T> {\n  return await fn(noopSpan)\n}\n\n/**\n * Creates a synchronous span for non-async operations\n * No-op implementation - telemetry has been removed.\n */\nexport function withSyncSpan<T>(\n  name: string,\n  attrs: SpanAttrs,\n  fn: (span: any) => T,\n  _options?: WithSpanOptions,\n): T {\n  return fn(noopSpan)\n}\n\n/**\n * Get the current tracer instance\n * No-op implementation - telemetry has been removed.\n */\nexport function getTracer() {\n  return null\n}\n"],"names":[],"mappings":"AASA,MAAM,WAAW;AAAA,EACf,cAAc,MAAM;AAAA,EAAC;AAAA,EACrB,eAAe,MAAM;AAAA,EAAC;AAAA,EACtB,WAAW,MAAM;AAAA,EAAC;AAAA,EAClB,iBAAiB,MAAM;AAAA,EAAC;AAAA,EACxB,KAAK,MAAM;AAAA,EAAC;AACd;AASA,eAAsB,SACpB,MACA,OACA,IACA,UACY;AACZ,SAAO,MAAM,GAAG,QAAQ;AAC1B;AAMA,eAAsB,eACpB,MACA,OACA,IACA,UACY;AACZ,SAAO,MAAM,GAAG,QAAQ;AAC1B;AAMO,SAAS,aACd,MACA,OACA,IACA,UACG;AACH,SAAO,GAAG,QAAQ;AACpB;"}

package/src/retry/NonRetriableError.ts

@@ -1 +0,0 @@
-export { NonRetriableError } from '../types'

package/src/telemetry/tracer.ts

@@ -1,66 +0,0 @@
-export interface SpanAttrs {
-  [key: string]: string | number | boolean | undefined
-}
-
-interface WithSpanOptions {
-  parentContext?: any
-}
-
-// No-op span implementation
-const noopSpan = {
-  setAttribute: () => {},
-  setAttributes: () => {},
-  setStatus: () => {},
-  recordException: () => {},
-  end: () => {},
-}
-
-/**
- * Lightweight span wrapper with error handling.
- * No-op implementation - telemetry has been removed.
- *
- * By default, creates spans at the current context level (siblings).
- * Use withNestedSpan if you want parent-child relationships.
- */
-export async function withSpan<T>(
-  name: string,
-  attrs: SpanAttrs,
-  fn: (span: any) => Promise<T>,
-  _options?: WithSpanOptions,
-): Promise<T> {
-  return await fn(noopSpan)
-}
-
-/**
- * Like withSpan but propagates context so child spans nest properly.
- * No-op implementation - telemetry has been removed.
- */
-export async function withNestedSpan<T>(
-  name: string,
-  attrs: SpanAttrs,
-  fn: (span: any) => Promise<T>,
-  _options?: WithSpanOptions,
-): Promise<T> {
-  return await fn(noopSpan)
-}
-
-/**
- * Creates a synchronous span for non-async operations
- * No-op implementation - telemetry has been removed.
- */
-export function withSyncSpan<T>(
-  name: string,
-  attrs: SpanAttrs,
-  fn: (span: any) => T,
-  _options?: WithSpanOptions,
-): T {
-  return fn(noopSpan)
-}
-
-/**
- * Get the current tracer instance
- * No-op implementation - telemetry has been removed.
- */
-export function getTracer() {
-  return null
-}