refactor: address more feedback

Author: rs-amp

src/commands/content-item/move.ts

@@ -203,7 +203,7 @@ export const handler = async (argv: Arguments<CopyItemBuilderOptions & Configura
 
       dir: '', // unused
 
-      logFile: revertLog,
+      logFile: argv.logFile,
       revertLog: argv.revertLog
     });
   } else {

src/commands/event/archive.ts

@@ -275,7 +275,7 @@ export const handler = async (argv: Arguments<ArchiveOptions & ConfigurationPara
   }
 
   const events = await getEvents({
-    id: id as string,
+    id,
     client,
     hubId,
     name

src/commands/hub/clone.spec.ts

@@ -3,8 +3,6 @@ import { createLog, getDefaultLogPath, openRevertLog } from '../../common/log-he
 import { ensureDirectoryExists } from '../../common/import/directory-utils';
 import Yargs from 'yargs/yargs';
 
-import * as copier from '../content-item/copy';
-
 import * as content from './steps/content-clone-step';
 import * as settings from './steps/settings-clone-step';
 import * as schema from './steps/schema-clone-step';
@@ -45,9 +43,6 @@ jest.mock('./steps/schema-clone-step', () => ({
 jest.mock('./steps/type-clone-step', () => ({ TypeCloneStep: mockStep('Clone Content Types', () => success[2]) }));
 jest.mock('./steps/content-clone-step', () => ({ ContentCloneStep: mockStep('Clone Content', () => success[3]) }));
 
-// eslint-disable-next-line @typescript-eslint/no-explicit-any
-const copierAny = copier as any;
-
 jest.mock('../../common/log-helpers', () => ({
   ...jest.requireActual('../../common/log-helpers'),
   getDefaultLogPath: jest.fn()

src/commands/hub/clone.ts

@@ -4,7 +4,6 @@ import { join } from 'path';
 import { ConfigurationParameters } from '../configure';
 
 import { ensureDirectoryExists } from '../../common/import/directory-utils';
-import { FileLog } from '../../common/file-log';
 import { loadCopyConfig } from '../../common/content-item/copy-config';
 import { CloneHubBuilderOptions } from '../../interfaces/clone-hub-builder-options';