Standardize coverage script spinner handling with other Socket projects

Consolidate spinner management to be consistent across all Socket projects.
Instead of stopping spinner at each output point, prevent it from starting
when using --json or --simple flags.

Changes:
- Only start/stop spinner in default output mode
- Remove redundant spinner.stop() calls in output branches
- Maintains consistent pattern across socket-registry, socket-cli, socket-sdk-js

Author: jdalton

scripts/get-coverage-percentage.mjs

@@ -29,17 +29,24 @@ async function logCoveragePercentage(argv) {
   // Get code coverage metrics (statements, branches, functions, lines).
   let codeCoverage
   try {
-    if (!existsSync(coverageJsonPath)) {
-      spinner.start('Generating coverage data...')
-    } else {
-      spinner.start('Reading coverage data...')
+    // Only show spinner in default output mode (not JSON or simple).
+    if (!argv.json && !argv.simple) {
+      if (!existsSync(coverageJsonPath)) {
+        spinner.start('Generating coverage data...')
+      } else {
+        spinner.start('Reading coverage data...')
+      }
     }
 
     codeCoverage = await getCodeCoverage()
 
-    spinner.stop()
+    if (!argv.json && !argv.simple) {
+      spinner.stop()
+    }
   } catch (error) {
-    spinner.stop()
+    if (!argv.json && !argv.simple) {
+      spinner.stop()
+    }
     logger.error('Failed to get code coverage:', error.message)
     throw error
   }
@@ -123,15 +130,11 @@ async function logCoveragePercentage(argv) {
 
     jsonOutput.overall = overall
 
-    // Stop spinner before outputting JSON.
-    spinner.stop()
     console.log(JSON.stringify(jsonOutput, null, 2))
   } else if (argv.simple) {
     // Simple format: just the statement coverage percentage.
-    spinner.stop()
     console.log(codeCoverage.statements.percent)
   } else {
-    spinner.stop()
     // Default format: human-readable formatted output.
     logger.info(`Coverage Summary:`)
     logger.info(