fix: improve error handling and code formatting in container pool initialization

Author: jleni

src/Zemu.ts

@@ -169,9 +169,12 @@ export default class Zemu {
       await Zemu.containerPool.initialize(config || defaultConfig)
       Zemu.poolInitialized = true
     } catch (error) {
-      console.warn('Container pool initialization failed, falling back to individual containers:', error)
+      console.warn(
+        `Container pool initialization failed: ${error instanceof Error ? error.message : error}. Falling back to individual containers.`
+      )
       Zemu.poolEnabled = false
       Zemu.containerPool = null
+      Zemu.poolInitialized = false
     }
   }
 
@@ -925,9 +928,9 @@ export default class Zemu {
       const nav: INavElement = {
         type: touchDevice ? ActionKind.Touch : ActionKind.RightClick,
         button: touchDevice
-          ? (imageIndex === 1 && isBlindSigning
+          ? imageIndex === 1 && isBlindSigning
             ? getTouchElement(this.startOptions.model, ButtonKind.RejectButton)
-            : getTouchElement(this.startOptions.model, ButtonKind.SwipeContinueButton))
+            : getTouchElement(this.startOptions.model, ButtonKind.SwipeContinueButton)
           : dummyButton, // For non-touch devices, use dummy button since action type determines behavior
       }
       await this.runAction(nav, filename, waitForScreenUpdate, true)
@@ -937,9 +940,7 @@ export default class Zemu {
     if (!runLastAction) return imageIndex // do not run last action if requested
 
     // Approve can be performed with Tap or PressAndHold
-    const approveButton = touchDevice 
-      ? getTouchElement(this.startOptions.model, this.startOptions.approveAction)
-      : dummyButton
+    const approveButton = touchDevice ? getTouchElement(this.startOptions.model, this.startOptions.approveAction) : dummyButton
 
     if (this.startOptions.approveAction === ButtonKind.DynamicTapButton) {
       const events = await this.getEvents()

src/actions.ts

@@ -14,7 +14,7 @@
  *  limitations under the License.
  ******************************************************************************* */
 import { getTouchElement } from './buttons'
-import { ActionKind, ButtonKind, type INavElement, type TModel, SwipeDirection, type IButton } from './types'
+import { ActionKind, ButtonKind, type IButton, type INavElement, SwipeDirection, type TModel } from './types'
 
 const dummyButton: IButton = {
   x: 0,

src/containerPool.ts

@@ -315,11 +315,11 @@ export class ContainerPool {
   private async cleanupStaleContainers(): Promise<void> {
     try {
       // List all containers that match our naming pattern
-      const containers = await this.docker.listContainers({ 
+      const containers = await this.docker.listContainers({
         all: true,
-        filters: { 
-          name: [`${BASE_NAME}-pool-`, `${BASE_NAME}-`] 
-        }
+        filters: {
+          name: [`${BASE_NAME}-pool-`, `${BASE_NAME}-`],
+        },
       })
 
       if (containers.length === 0) {

tests/globalsetup.ts

@@ -18,7 +18,7 @@ const catchExit = () => {
     process.exit(1)
   })
 
-  process.on('unhandledRejection', (reason, promise) => {
+  process.on('unhandledRejection', (reason, _promise) => {
     console.error('Unhandled rejection, cleaning up containers:', reason)
     cleanup()
     process.exit(1)