chore: reverting accidental push to master

This reverts commit 6d72a72.

Author: Zhiheng Lu

lib/Context/index.ts

@@ -3,10 +3,6 @@ import { Choice, DataConfig, ResponseContext, TraceType } from '@/lib/types';
 
 import VariableManager from '../Variables';
 
-export type GetTraceOptions = {
-  sanitize?: boolean;
-};
-
 export class Context<S extends Record<string, any> = Record<string, any>> {
   private dataFilterer: DataFilterer;
 
@@ -25,14 +21,9 @@ export class Context<S extends Record<string, any> = Record<string, any>> {
     return this.dataFilterer.filterTraces(this.context.trace);
   }
 
-  // returns the entire unfiltered list of traces of the context; can configure whether trace data should be sanitized or not
-  getTrace(options: GetTraceOptions = {}) {
-    const { sanitize = true } = options;
-
-    let result = this.context.trace;
-    if (sanitize) result = this.dataFilterer.sanitizeTraces(result);
-
-    return result;
+  // returns the entire raw trace of the context
+  getTrace() {
+    return this.context.trace;
   }
 
   // returns the raw context object

lib/DataFilterer/index.ts

@@ -22,18 +22,15 @@ class DataFilterer {
     }
   }
 
-  sanitizeTraces(traces: GeneralTrace[]): GeneralTrace[] {
-    return traces.map((trace) => {
-      this.traceFilters.forEach((filter) => {
-        trace = filter(trace);
-      });
-      return trace;
-    });
-  }
-
   filterTraces(traces: GeneralTrace[]): GeneralTrace[] {
-    const filteredTraces = traces.filter(({ type }) => this.includeTypes!.has(type));
-    return this.sanitizeTraces(filteredTraces);
+    return traces
+      .filter(({ type }) => this.includeTypes!.has(type))
+      .map((trace) => {
+        this.traceFilters.forEach((filter) => {
+          trace = filter(trace);
+        });
+        return trace;
+      });
   }
 }
 

lib/Events/index.ts

@@ -4,9 +4,9 @@ import { GeneralTrace, TraceMap, TraceType } from '@/lib/types';
 
 import Context from '../Context';
 
-export type TraceEventHandler<T extends TraceType, V extends Record<string, any>> = (trace: TraceMap[T], context: Context<V>) => void;
+export type TraceEventHandler<T extends TraceType, V extends Record<string, any>> = (object: TraceMap[T], context: Context<V>) => void;
 
-export type GeneralTraceEventHandler<V extends Record<string, any>> = (trace: GeneralTrace, context: Context<V>) => void;
+export type GeneralTraceEventHandler<V extends Record<string, any>> = (object: GeneralTrace, context: Context<V>) => void;
 
 type _Map<T extends Record<string, any>, K extends TraceType = TraceType> = Map<K, Array<TraceEventHandler<K, T>>>;
 

tests/lib/Context/Context.unit.ts

@@ -12,7 +12,7 @@ import {
   START_RESPONSE_BODY_WITH_MULTIPLE_CHOICES,
   VF_APP_NEXT_STATE_1,
 } from './fixtures';
-import { SPEAK_TRACE, SPEAK_TRACE_UNSANITIZED, VFAppVariablesSchema } from '../fixtures';
+import { SPEAK_TRACE, VFAppVariablesSchema } from '../fixtures';
 
 describe('Context', () => {
   afterEach(() => {
@@ -29,15 +29,6 @@ describe('Context', () => {
     expect(context.getChips()).to.eql([...CHOICES_1, ...CHOICES_2, ...CHOICES_3]);
   });
 
-  it('getTrace(), sanitize option', () => {
-    const context = new Context({
-      ...START_RESPONSE_BODY,
-      trace: [SPEAK_TRACE_UNSANITIZED]
-    })
-
-    expect(context.getTrace({ sanitize: false })[0]).to.eql(SPEAK_TRACE_UNSANITIZED);
-  });
-
   it('end state', () => {
     const context1 = new Context(START_RESPONSE_BODY);
 

tests/lib/fixtures.ts

@@ -23,15 +23,7 @@ export const SPEAK_TRACE: SpeakTrace = {
   payload: {
     message: 'Books ought to have to have good endings.',
     src: 'data:audio/mpeg:some-large-tts-audio-file',
-  }, 
-}
-
-export const SPEAK_TRACE_UNSANITIZED: SpeakTrace = {
-  ...SPEAK_TRACE,
-  payload: {
-    ...SPEAK_TRACE.payload,
-    message: `<voice>${SPEAK_TRACE.payload.message}</voice>`
-  }
+  },
 };
 
 export const MAKE_SPEAK_TRACE = (payload: SpeakTrace['payload']): SpeakTrace => ({