refactor(cli-with-sentry): use logger helper methods in verify-package script

- Replace manual color formatting with logger.info(), logger.success(), logger.fail()
- Remove duplicate helper functions (success, error, info) that reimplemented logger methods
- Cleaner, more maintainable code with consistent output formatting

This is the first of several scripts that should be updated to use logger methods
instead of manual color formatting. See related files that need similar updates:
- packages/cli/scripts/verify-package.mjs
- packages/socket/scripts/verify-package.mjs
- scripts/build.mjs (partial - complex formatting)
- And ~20 other build/test scripts

Author: jdalton

packages/cli-with-sentry/scripts/verify-package.mjs

@@ -10,27 +10,6 @@ const __filename = fileURLToPath(import.meta.url)
 const __dirname = path.dirname(__filename)
 const packageRoot = path.resolve(__dirname, '..')
 
-/**
- * Format a success message.
- */
-function success(msg) {
-  return `${colors.green('✓')} ${msg}`
-}
-
-/**
- * Format an error message.
- */
-function error(msg) {
-  return `${colors.red('✗')} ${msg}`
-}
-
-/**
- * Format an info message.
- */
-function info(msg) {
-  return `${colors.blue('ℹ')} ${msg}`
-}
-
 /**
  * Check if a file exists and is readable.
  */
@@ -56,12 +35,12 @@ async function validate() {
   const errors = []
 
   // Check package.json exists and validate Sentry configuration.
-  logger.log(info('Checking package.json...'))
+  logger.info('Checking package.json...')
   const pkgPath = path.join(packageRoot, 'package.json')
   if (!(await fileExists(pkgPath))) {
     errors.push('package.json does not exist')
   } else {
-    logger.log(success('package.json exists'))
+    logger.success('package.json exists')
 
     // Validate package.json configuration.
     const pkg = JSON.parse(await fs.readFile(pkgPath, 'utf-8'))
@@ -70,7 +49,7 @@ async function validate() {
     if (!pkg.dependencies?.['@sentry/node']) {
       errors.push('package.json missing @sentry/node in dependencies')
     } else {
-      logger.log(success('@sentry/node is in dependencies'))
+      logger.success('@sentry/node is in dependencies')
     }
 
     // Validate files array.
@@ -88,68 +67,68 @@ async function validate() {
       }
     }
     if (errors.length === 0) {
-      logger.log(success('package.json files array is correct'))
+      logger.success('package.json files array is correct')
     }
   }
 
   // Check root files exist (LICENSE, CHANGELOG.md).
   const rootFiles = ['LICENSE', 'CHANGELOG.md']
   for (const file of rootFiles) {
-    logger.log(info(`Checking ${file}...`))
+    logger.info(`Checking ${file}...`)
     const filePath = path.join(packageRoot, file)
     if (!(await fileExists(filePath))) {
       errors.push(`${file} does not exist`)
     } else {
-      logger.log(success(`${file} exists`))
+      logger.success(`${file} exists`)
     }
   }
 
   // Check dist files exist and validate Sentry integration.
   const distFiles = ['index.js', 'cli.js.bz', 'shadow-npm-inject.js']
   for (const file of distFiles) {
-    logger.log(info(`Checking dist/${file}...`))
+    logger.info(`Checking dist/${file}...`)
     const filePath = path.join(packageRoot, 'dist', file)
     if (!(await fileExists(filePath))) {
       errors.push(`dist/${file} does not exist`)
     } else {
-      logger.log(success(`dist/${file} exists`))
+      logger.success(`dist/${file} exists`)
     }
   }
 
   // Verify Sentry is referenced in the build (check for @sentry/node require).
-  logger.log(info('Checking for Sentry integration in build...'))
+  logger.info('Checking for Sentry integration in build...')
   const buildPath = path.join(packageRoot, 'build', 'cli.js')
   if (await fileExists(buildPath)) {
     const buildContent = await fs.readFile(buildPath, 'utf-8')
     if (!buildContent.includes('@sentry/node')) {
       errors.push('Sentry integration not found in build/cli.js')
     } else {
-      logger.log(success('Sentry integration found in build'))
+      logger.success('Sentry integration found in build')
     }
   } else {
     errors.push('build/cli.js does not exist (required for Sentry validation)')
   }
 
   // Check data directory exists.
-  logger.log(info('Checking data directory...'))
+  logger.info('Checking data directory...')
   const dataPath = path.join(packageRoot, 'data')
   if (!(await fileExists(dataPath))) {
     errors.push('data directory does not exist')
   } else {
-    logger.log(success('data directory exists'))
+    logger.success('data directory exists')
 
     // Check data files.
     const dataFiles = [
       'alert-translations.json',
       'command-api-requirements.json',
     ]
     for (const file of dataFiles) {
-      logger.log(info(`Checking data/${file}...`))
+      logger.info(`Checking data/${file}...`)
       const filePath = path.join(dataPath, file)
       if (!(await fileExists(filePath))) {
         errors.push(`data/${file} does not exist`)
       } else {
-        logger.log(success(`data/${file} exists`))
+        logger.success(`data/${file} exists`)
       }
     }
   }
@@ -164,23 +143,23 @@ async function validate() {
   if (errors.length > 0) {
     logger.log(`${colors.red('Errors:')}`)
     for (const err of errors) {
-      logger.log(`  ${error(err)}`)
+      logger.fail(`  ${err}`)
     }
     logger.log('')
-    logger.log(error('Package validation FAILED'))
+    logger.fail('Package validation FAILED')
     logger.log('')
     process.exit(1)
   }
 
-  logger.log(success('Package validation PASSED'))
+  logger.success('Package validation PASSED')
   logger.log('')
   process.exit(0)
 }
 
 // Run validation.
 validate().catch(e => {
   logger.error('')
-  logger.error(error(`Unexpected error: ${e.message}`))
+  logger.fail(`Unexpected error: ${e.message}`)
   logger.error('')
   process.exit(1)
 })