diff --git a/.debugging/test-migrate-verify.sh b/.debugging/test-migrate-verify.sh old mode 100644 new mode 100755 diff --git a/src/commands/migrate/helpers/handle-migrate-action.js b/src/commands/migrate/helpers/handle-migrate-action.js index 6f5504e6..131ef45d 100644 --- a/src/commands/migrate/helpers/handle-migrate-action.js +++ b/src/commands/migrate/helpers/handle-migrate-action.js @@ -3,7 +3,7 @@ import { loadMigrateConfig } from '../../../shared/config/loader.js'; import { detectAndRoute } from '../../../version-router.js'; import { logSystemInfo, ensureHeapSize } from '../../../shared/utils/concurrency.js'; -import logger from '../../../shared/utils/logger.js'; +import logger, { enableFileLogging } from '../../../shared/utils/logger.js'; import { applyMigrateOptions } from './apply-migrate-options.js'; import { buildMigratePerfConfig } from './build-migrate-perf-config.js'; @@ -16,6 +16,7 @@ export async function handleMigrateAction(options, shutdownCoordinator) { const perfConfig = buildMigratePerfConfig(config, options); ensureHeapSize(perfConfig.maxMemoryMB); + enableFileLogging('migrate'); logSystemInfo(perfConfig); const { migrateAll, pipelineId } = await detectAndRoute(config.sonarqube); diff --git a/src/commands/migrate/index.js b/src/commands/migrate/index.js index bf736899..d9ab35dc 100644 --- a/src/commands/migrate/index.js +++ b/src/commands/migrate/index.js @@ -1,6 +1,6 @@ // -------- Migrate Command -------- -import logger, { enableFileLogging } from '../../shared/utils/logger.js'; +import logger from '../../shared/utils/logger.js'; import { ShutdownCoordinator } from '../../shared/utils/shutdown.js'; import { handleMigrateAction } from './helpers/handle-migrate-action.js'; import { handleCommandError } from '../transfer/helpers/handle-command-error.js'; @@ -36,7 +36,6 @@ export function registerMigrateCommand(program) { try { if (options.verbose) logger.level = 'debug'; - enableFileLogging('migrate'); logger.info('=== CloudVoyager - Full Organization Migration ==='); await handleMigrateAction(options, shutdownCoordinator); logger.info('=== Migration completed successfully ==='); diff --git a/src/commands/sync-metadata/helpers/handle-sync-metadata-action.js b/src/commands/sync-metadata/helpers/handle-sync-metadata-action.js index cef4b72a..450be987 100644 --- a/src/commands/sync-metadata/helpers/handle-sync-metadata-action.js +++ b/src/commands/sync-metadata/helpers/handle-sync-metadata-action.js @@ -3,7 +3,7 @@ import { loadMigrateConfig } from '../../../shared/config/loader.js'; import { detectAndRoute } from '../../../version-router.js'; import { resolvePerformanceConfig, logSystemInfo, ensureHeapSize } from '../../../shared/utils/concurrency.js'; -import logger from '../../../shared/utils/logger.js'; +import logger, { enableFileLogging } from '../../../shared/utils/logger.js'; export async function handleSyncMetadataAction(options, shutdownCoordinator) { const config = await loadMigrateConfig(options.config); @@ -22,6 +22,7 @@ export async function handleSyncMetadataAction(options, shutdownCoordinator) { ...(options.maxMemory && { maxMemoryMB: options.maxMemory }) }); ensureHeapSize(perfConfig.maxMemoryMB); + enableFileLogging('sync-metadata'); logSystemInfo(perfConfig); migrateConfig.dryRun = false; diff --git a/src/commands/sync-metadata/index.js b/src/commands/sync-metadata/index.js index 64852936..a2f9cabe 100644 --- a/src/commands/sync-metadata/index.js +++ b/src/commands/sync-metadata/index.js @@ -1,6 +1,6 @@ // -------- Sync Metadata Command -------- -import logger, { enableFileLogging } from '../../shared/utils/logger.js'; +import logger from '../../shared/utils/logger.js'; import { ShutdownCoordinator } from '../../shared/utils/shutdown.js'; import { handleSyncMetadataAction } from './helpers/handle-sync-metadata-action.js'; import { handleCommandError } from '../transfer/helpers/handle-command-error.js'; @@ -28,7 +28,6 @@ export function registerSyncMetadataCommand(program) { try { if (options.verbose) logger.level = 'debug'; - enableFileLogging('sync-metadata'); logger.info('=== CloudVoyager - Issue & Hotspot Metadata Sync ==='); await handleSyncMetadataAction(options, shutdownCoordinator); logger.info('=== Metadata sync completed successfully ==='); diff --git a/src/commands/transfer/helpers/handle-transfer-action.js b/src/commands/transfer/helpers/handle-transfer-action.js index 0afe0f76..d4d77a7f 100644 --- a/src/commands/transfer/helpers/handle-transfer-action.js +++ b/src/commands/transfer/helpers/handle-transfer-action.js @@ -4,7 +4,7 @@ import { loadConfig, requireProjectKeys } from '../../../shared/config/loader.js import { detectAndRoute } from '../../../version-router.js'; import { resolvePerformanceConfig, logSystemInfo, ensureHeapSize } from '../../../shared/utils/concurrency.js'; import { ProgressTracker } from '../../../shared/utils/progress.js'; -import logger from '../../../shared/utils/logger.js'; +import logger, { enableFileLogging } from '../../../shared/utils/logger.js'; export async function handleTransferAction(options, shutdownCoordinator) { const config = await loadConfig(options.config); @@ -30,6 +30,7 @@ export async function handleTransferAction(options, shutdownCoordinator) { ...(options.maxMemory && { maxMemoryMB: options.maxMemory }), }); ensureHeapSize(perfConfig.maxMemoryMB); + enableFileLogging('transfer'); logSystemInfo(perfConfig); const { transferProject, pipelineId } = await detectAndRoute(config.sonarqube); diff --git a/src/commands/transfer/index.js b/src/commands/transfer/index.js index 5768c076..5401ab53 100644 --- a/src/commands/transfer/index.js +++ b/src/commands/transfer/index.js @@ -1,6 +1,6 @@ // -------- Transfer Command -------- -import logger, { enableFileLogging } from '../../shared/utils/logger.js'; +import logger from '../../shared/utils/logger.js'; import { ShutdownCoordinator } from '../../shared/utils/shutdown.js'; import { handleTransferAction } from './helpers/handle-transfer-action.js'; import { handleCommandError } from './helpers/handle-command-error.js'; @@ -30,7 +30,6 @@ export function registerTransferCommand(program) { try { if (options.verbose) logger.level = 'debug'; - enableFileLogging('transfer'); await handleTransferAction(options, shutdownCoordinator); logger.info('=== Transfer completed successfully ==='); process.exit(0); diff --git a/src/commands/verify/helpers/handle-verify-action.js b/src/commands/verify/helpers/handle-verify-action.js index 061746b5..fad36b20 100644 --- a/src/commands/verify/helpers/handle-verify-action.js +++ b/src/commands/verify/helpers/handle-verify-action.js @@ -3,7 +3,7 @@ import { loadMigrateConfig } from '../../../shared/config/loader.js'; import { verifyAll } from '../../../shared/verification/verify-pipeline.js'; import { resolvePerformanceConfig, logSystemInfo, ensureHeapSize } from '../../../shared/utils/concurrency.js'; -import logger from '../../../shared/utils/logger.js'; +import logger, { enableFileLogging } from '../../../shared/utils/logger.js'; import { parseOnlyComponents } from './parse-only-components.js'; export async function handleVerifyAction(options) { @@ -21,6 +21,7 @@ export async function handleVerifyAction(options) { ...(options.maxMemory && { maxMemoryMB: options.maxMemory }) }); ensureHeapSize(perfConfig.maxMemoryMB); + enableFileLogging('verify'); logSystemInfo(perfConfig); const results = await verifyAll({ diff --git a/src/commands/verify/index.js b/src/commands/verify/index.js index 08c9f026..713c98a8 100644 --- a/src/commands/verify/index.js +++ b/src/commands/verify/index.js @@ -1,6 +1,6 @@ // -------- Verify Command -------- -import logger, { enableFileLogging } from '../../shared/utils/logger.js'; +import logger from '../../shared/utils/logger.js'; import { CloudVoyagerError } from '../../shared/utils/errors.js'; import { handleVerifyAction } from './helpers/handle-verify-action.js'; @@ -22,7 +22,6 @@ export function registerVerifyCommand(program) { .action(async (options) => { try { if (options.verbose) logger.level = 'debug'; - enableFileLogging('verify'); logger.info('=== CloudVoyager - Migration Verification ==='); await handleVerifyAction(options); logger.info('=== Verification completed successfully — all checks passed ===');