fix: actually stop spinning on success/failure (#7262)

Even after we've called `.success()` or `.stop()` on this spinner,
we were continuing to render it when we receive output from the framework server.

Author: serhalp

src/utils/shell.ts

@@ -65,9 +65,13 @@ export const runCommand = (
   // even though the actual framework command might be outputting stuff
   const pipeDataWithSpinner = (writeStream: NodeJS.WriteStream, chunk: string | Uint8Array) => {
     // Clear the spinner, write the framework command line, then resume spinning
-    spinner?.clear()
+    if (spinner?.isSpinning()) {
+      spinner.clear()
+    }
     writeStream.write(chunk, () => {
-      spinner?.spin()
+      if (spinner?.isSpinning()) {
+        spinner.spin()
+      }
     })
   }