@dxos/tracing 0.8.2-staging.7ac8446 → 0.8.2-staging.8c95c61

package/src/metrics/base.ts

@@ -15,7 +15,7 @@ export abstract class BaseCounter {
   /**
    * @internal
    */
-  _assign(instance: any, name: string) {
+  _assign(instance: any, name: string): void {
     this._instance = instance;
     this.name = name;
   }

package/src/metrics/map-counter.ts

@@ -15,7 +15,7 @@ export class MapCounter extends BaseCounter {
     this.units = units;
   }
 
-  inc(key: string, by = 1) {
+  inc(key: string, by = 1): void {
     const prev = this.values.get(key) ?? 0;
     this.values.set(key, prev + by);
   }

package/src/metrics/time-series-counter.ts

@@ -19,7 +19,7 @@ export class TimeSeriesCounter extends BaseCounter {
     this.units = units;
   }
 
-  inc(by = 1) {
+  inc(by = 1): void {
     this._currentValue += by;
     this._totalValue += by;
   }

package/src/metrics/time-usage-counter.ts

@@ -15,7 +15,7 @@ export class TimeUsageCounter extends BaseCounter {
 
   private _lastTickTime = performance.now();
 
-  record(time: number) {
+  record(time: number): void {
     this._currentValue += time;
     this._totalValue += time;
   }

package/src/metrics/unary-counter.ts

@@ -15,7 +15,7 @@ export class UnaryCounter extends BaseCounter {
     this.units = units;
   }
 
-  inc(by = 1) {
+  inc(by = 1): void {
     this.value += by;
   }
 

package/src/remote/metrics.ts

@@ -35,23 +35,23 @@ interface MetricsMethods {
 export class RemoteMetrics implements MetricsMethods {
   private _metrics = new Set<MetricsMethods>();
 
-  registerProcessor(processor: MetricsMethods) {
+  registerProcessor(processor: MetricsMethods): void {
     this._metrics.add(processor);
   }
 
-  increment(name: string, value?: number, data?: MetricData) {
+  increment(name: string, value?: number, data?: MetricData): void[] {
     return Array.from(this._metrics.values()).map((processor) => processor.increment(name, value, data));
   }
 
-  distribution(name: string, value: number, data?: MetricData) {
+  distribution(name: string, value: number, data?: MetricData): void[] {
     return Array.from(this._metrics.values()).map((processor) => processor.distribution(name, value, data));
   }
 
-  set(name: string, value: number | string, data?: MetricData) {
+  set(name: string, value: number | string, data?: MetricData): void[] {
     return Array.from(this._metrics.values()).map((processor) => processor.set(name, value, data));
   }
 
-  gauge(name: string, value: number, data?: MetricData) {
+  gauge(name: string, value: number, data?: MetricData): void[] {
     return Array.from(this._metrics.values()).map((processor) => processor.gauge(name, value, data));
   }
 }

package/src/remote/tracing.ts

@@ -26,11 +26,11 @@ export class RemoteTracing {
   private _tracing: TracingMethods | undefined;
   private _spanMap = new Map<TracingSpan, RemoteSpan>();
 
-  registerProcessor(processor: TracingMethods) {
+  registerProcessor(processor: TracingMethods): void {
     this._tracing = processor;
   }
 
-  flushSpan(span: TracingSpan) {
+  flushSpan(span: TracingSpan): void {
     if (!this._tracing) {
       return;
     }

package/src/trace-processor.ts

@@ -113,7 +113,7 @@ export class TraceProcessor {
     this.diagnosticsChannel.unref();
   }
 
-  setInstanceTag(tag: string) {
+  setInstanceTag(tag: string): void {
     this._instanceTag = tag;
     this.diagnostics.setInstanceTag(tag);
   }
@@ -122,7 +122,7 @@ export class TraceProcessor {
    * @internal
    */
   // TODO(burdon): Comment.
-  createTraceResource(params: TraceResourceConstructorParams) {
+  createTraceResource(params: TraceResourceConstructorParams): void {
     const id = this.resources.size;
 
     // Init metrics counters.
@@ -154,7 +154,7 @@ export class TraceProcessor {
     this._markResourceDirty(id);
   }
 
-  createTraceSender() {
+  createTraceSender(): TraceSender {
     return new TraceSender(this);
   }
 
@@ -165,7 +165,7 @@ export class TraceProcessor {
   }
 
   // TODO(burdon): Not implemented.
-  addLink(parent: any, child: any, opts: AddLinkOptions) {}
+  addLink(parent: any, child: any, opts: AddLinkOptions): void {}
 
   //
   // Getters
@@ -236,7 +236,7 @@ export class TraceProcessor {
     return [...this.resources.values()].filter((res) => res.annotation === annotation);
   }
 
-  refresh() {
+  refresh(): void {
     for (const resource of this.resources.values()) {
       const instance = resource.instance.deref();
       if (!instance) {
@@ -278,7 +278,7 @@ export class TraceProcessor {
   /**
    * @internal
    */
-  _flushSpan(runtimeSpan: TracingSpan) {
+  _flushSpan(runtimeSpan: TracingSpan): void {
     const span = runtimeSpan.serialize();
     this.spans.set(span.id, span);
     this.spanIdList.push(span.id);
@@ -289,19 +289,19 @@ export class TraceProcessor {
     this.remoteTracing.flushSpan(runtimeSpan);
   }
 
-  private _markResourceDirty(id: number) {
+  private _markResourceDirty(id: number): void {
     for (const subscription of this.subscriptions) {
       subscription.dirtyResources.add(id);
     }
   }
 
-  private _markSpanDirty(id: number) {
+  private _markSpanDirty(id: number): void {
     for (const subscription of this.subscriptions) {
       subscription.dirtySpans.add(id);
     }
   }
 
-  private _clearResources() {
+  private _clearResources(): void {
     // TODO(dmaretskyi): Use FinalizationRegistry to delete finalized resources first.
     while (this.resourceIdList.length > MAX_RESOURCE_RECORDS) {
       const id = this.resourceIdList.shift()!;
@@ -309,14 +309,14 @@ export class TraceProcessor {
     }
   }
 
-  private _clearSpans() {
+  private _clearSpans(): void {
     while (this.spanIdList.length > MAX_SPAN_RECORDS) {
       const id = this.spanIdList.shift()!;
       this.spans.delete(id);
     }
   }
 
-  private _pushLog(log: LogEntry) {
+  private _pushLog(log: LogEntry): void {
     this.logs.push(log);
     if (this.logs.length > MAX_LOG_RECORDS) {
       this.logs.shift();
@@ -413,7 +413,7 @@ export class TracingSpan {
     return this._ctx;
   }
 
-  markSuccess() {
+  markSuccess(): void {
     this.endTs = performance.now();
     this._traceProcessor._flushSpan(this);
 
@@ -422,7 +422,7 @@ export class TracingSpan {
     }
   }
 
-  markError(err: unknown) {
+  markError(err: unknown): void {
     this.endTs = performance.now();
     this.error = serializeError(err);
     this._traceProcessor._flushSpan(this);
@@ -444,7 +444,7 @@ export class TracingSpan {
     };
   }
 
-  private _markInBrowserTimeline() {
+  private _markInBrowserTimeline(): void {
     if (typeof globalThis?.performance?.measure === 'function') {
       performance.measure(this.name, { start: this.startTs, end: this.endTs! });
     }

package/src/tracing.test.ts

@@ -24,7 +24,7 @@ class FeedStore {
   }
 
   @trace.span()
-  async openFeed(ctx: Context) {
+  async openFeed(ctx: Context): Promise<Feed> {
     const feed = new Feed();
     feed.key = Math.random().toString(36).substring(2, 15);
     this._feeds.set(feed.key, feed);
@@ -41,10 +41,10 @@ class Feed {
   key!: string;
 
   @trace.span()
-  async open(ctx: Context) {}
+  async open(ctx: Context): Promise<void> {}
 
   @trace.span()
-  async close() {
+  async close(): Promise<void> {
     throw new Error('Not implemented');
   }
 }