chore: use record instead of object

Author: kmruiz

src/common/config.ts

@@ -319,11 +319,11 @@ function parseCliConfig(args: string[]): CliOptions {
     return parsed;
 }
 
-export function warnAboutDeprecatedOrUnknownCliArgs(args: object, warn: (msg: string) => void): void {
+export function warnAboutDeprecatedOrUnknownCliArgs(args: Record<string, unknown>, warn: (msg: string) => void): void {
     let usedDeprecatedArgument = false;
     let usedInvalidArgument = false;
 
-    const knownArgs = args as UserConfig & CliOptions;
+    const knownArgs = args as unknown as UserConfig & CliOptions;
     // the first position argument should be used
     // instead of --connectionString, as it's how the mongosh works.
     if (knownArgs.connectionString) {

tests/unit/common/config.test.ts

@@ -660,7 +660,7 @@ describe("CLI arguments", () => {
                 cliArgs = { [cliArg]: "RandomString" } as unknown as CliOptions & UserConfig & { _?: string[] };
                 warn = vi.fn();
 
-                warnAboutDeprecatedOrUnknownCliArgs(cliArgs, warn);
+                warnAboutDeprecatedOrUnknownCliArgs(cliArgs as unknown as Record<string, unknown>, warn);
             });
 
             it(`warns the usage of ${cliArg} as it is deprecated`, () => {