fix(types): pass contextId as options

remove second param `contextId` for the types' methods and pass `contextId` inside optional
`options` param

Author: niloysikdar

src/types/types.test.ts

@@ -27,10 +27,9 @@ describe('Hypercode Types API methods', () => {
         status: 200,
       });
 
-      const result = await hyper.types.string(
-        'Who is the CEO of SpaceX?',
+      const result = await hyper.types.string('Who is the CEO of SpaceX?', {
         contextId,
-      );
+      });
 
       expect(typeof result.data).toBe('string');
       expect(result.data).toBe('Elon Musk');
@@ -72,7 +71,7 @@ describe('Hypercode Types API methods', () => {
 
       const result = await hyper.types.string(
         'How many planets are in the Solar System?',
-        contextId,
+        { contextId },
       );
 
       expect(typeof result.data).toBe('number');
@@ -118,7 +117,7 @@ describe('Hypercode Types API methods', () => {
 
       const result = await hyper.types.float(
         'How many billion years old is the universe?',
-        contextId,
+        { contextId },
       );
 
       expect(typeof result.data).toBe('number');
@@ -157,10 +156,9 @@ describe('Hypercode Types API methods', () => {
         status: 200,
       });
 
-      const result = await hyper.types.boolean(
-        'Can cats see in the dark?',
+      const result = await hyper.types.boolean('Can cats see in the dark?', {
         contextId,
-      );
+      });
 
       expect(typeof result.data).toBe('boolean');
       expect(result.data).toBe(true);
@@ -204,7 +202,7 @@ describe('Hypercode Types API methods', () => {
 
       const { data } = await hyper.types.datetime(
         'What is the date of the Apollo 11 moon landing?',
-        contextId,
+        { contextId },
       );
 
       expect(typeof data).toBe('string');

src/types/types.ts

@@ -12,6 +12,10 @@ type allowedTypes =
   | 'boolean_array'
   | 'datetime_array';
 
+type MethodParamsOptions = {
+  contextId: string;
+};
+
 export class Types {
   constructor(private readonly hyper: Hyper) {}
 
@@ -40,51 +44,51 @@ export class Types {
     return res;
   }
 
-  async string(query: string, contextId?: string) {
+  async string(query: string, options?: MethodParamsOptions) {
     const data = await this.makeRequest<string>({
       endpointType: 'string',
       query,
-      contextId,
+      contextId: options?.contextId,
     });
 
     return data;
   }
 
-  async integer(query: string, contextId?: string) {
+  async integer(query: string, options?: MethodParamsOptions) {
     const data = await this.makeRequest<number>({
       endpointType: 'integer',
       query,
-      contextId,
+      contextId: options?.contextId,
     });
 
     return data;
   }
 
-  async float(query: string, contextId?: string) {
+  async float(query: string, options?: MethodParamsOptions) {
     const data = await this.makeRequest<number>({
       endpointType: 'float',
       query,
-      contextId,
+      contextId: options?.contextId,
     });
 
     return data;
   }
 
-  async boolean(query: string, contextId?: string) {
+  async boolean(query: string, options?: MethodParamsOptions) {
     const data = await this.makeRequest<boolean>({
       endpointType: 'boolean',
       query,
-      contextId,
+      contextId: options?.contextId,
     });
 
     return data;
   }
 
-  async datetime(query: string, contextId?: string) {
+  async datetime(query: string, options?: MethodParamsOptions) {
     const data = await this.makeRequest<string>({
       endpointType: 'datetime',
       query,
-      contextId,
+      contextId: options?.contextId,
     });
 
     return data;