minor fixes

Author: myang1220

packages/compass-user-data/src/index.ts

@@ -1,3 +1,3 @@
 export type { Stats, ReadAllResult, ReadAllWithStatsResult } from './user-data';
-export { IUserData, FileUserData } from './user-data';
+export { type IUserData, FileUserData } from './user-data';
 export { z } from 'zod';

packages/compass-user-data/src/user-data.ts

@@ -73,7 +73,6 @@ export abstract class IUserData<T extends z.Schema> {
   protected readonly validator: T;
   protected readonly serialize: SerializeContent<z.input<T>>;
   protected readonly deserialize: DeserializeContent;
-
   constructor(
     validator: T,
     {
@@ -96,6 +95,7 @@ export abstract class IUserData<T extends z.Schema> {
     id: string,
     data: Partial<z.input<T>>
   ): Promise<z.output<T>>;
+  abstract setOrgAndGroupId(orgId: string, groupId: string): void;
 }
 
 export class FileUserData<T extends z.Schema> extends IUserData<T> {
@@ -333,15 +333,28 @@ export class FileUserData<T extends z.Schema> extends IUserData<T> {
     });
     return await this.readOne(id);
   }
+
+  setOrgAndGroupId(orgId: string, groupId: string): void {
+    // sample log error
+    log.error(
+      mongoLogId(1_001_000_237), // not sure what this log id should be
+      'UserData',
+      'setOrgAndGroupId should not be called for FileUserData',
+      {
+        orgId,
+        groupId,
+      }
+    );
+  }
 }
 
 // TODO: update endpoints to reflect the merged api endpoints
 export class AtlasUserData<T extends z.Schema> extends IUserData<T> {
   private readonly authenticatedFetch;
   // should this BASE_URL be a parameter passed to the constructor?
   // this might make future usage of this code easier, if we want to call a different endpoint
-  private readonly orgId;
-  private readonly groupId;
+  private orgId: string = '';
+  private groupId: string = '';
   private readonly BASE_URL = 'cluster-connection.cloud-local.mongodb.com';
   constructor(
     validator: T,

packages/my-queries-storage/src/compass-query-storage.ts

@@ -54,9 +54,7 @@ export abstract class CompassQueryStorage<TSchema extends z.Schema> {
   }
 
   setOrgAndGroupId(orgId: string, groupId: string): void {
-    if (this.userData instanceof AtlasUserData) {
-      this.userData.setOrgAndGroupId(orgId, groupId);
-    }
+    this.userData.setOrgAndGroupId?.(orgId, groupId);
   }
 
   abstract saveQuery(data: Partial<z.input<TSchema>>): Promise<void>;