fix(app): fix terminal banner render state (#17240)

Closes RQA-3840

Author: mjhuff

app/src/organisms/Desktop/Devices/ProtocolRun/ProtocolRunHeader/RunHeaderBannerContainer/TerminalRunBannerContainer.tsx

@@ -40,12 +40,15 @@ export function useTerminalRunBannerContainer({
   const completedWithErrors =
     (commandErrorList != null && commandErrorList.length > 0) ||
     highestPriorityError != null
+  // TODO(jh, 01-10-25): Adding /commandErrors to notifications accomplishes the below with reduced latency.
+  const completedWithNoErrors =
+    commandErrorList != null && commandErrorList.length === 0
 
   const showSuccessBanner =
     runStatus === RUN_STATUS_SUCCEEDED &&
     isRunCurrent &&
     !isResetRunLoading &&
-    !completedWithErrors
+    completedWithNoErrors
 
   // TODO(jh, 08-14-24): Ideally, the backend never returns the "user cancelled a run" error and
   //  cancelledWithoutRecovery becomes unnecessary.