refactor: remove debug logs and simplify dispose function

- Remove console.log debugging statement from plugin
- Make dispose function synchronous and remove artificial delay
- Add test delays to ensure proper server cleanup
- Improve plugin performance and cleanup behavior

Author: yaonyan

packages/plugin-code-execution/src/plugin.ts

@@ -90,7 +90,6 @@ export function createCodeExecutionPlugin(
           const needsDefinitions = definitionsOf.filter(
             (def) => !hasDefinitions.includes(def),
           );
-          console.log("Needs definitions for:", needsDefinitions);
 
           if (needsDefinitions.length > 0) {
             const definitionTexts: string[] = [];
@@ -125,12 +124,10 @@ export function createCodeExecutionPlugin(
       );
     },
 
-    dispose: async () => {
+    dispose: () => {
       if (executor) {
         executor.stop();
         executor = null;
-        // Wait for process to terminate
-        await new Promise((resolve) => setTimeout(resolve, 100));
       }
     },
   };

packages/plugin-code-execution/tests/code_execution.test.ts

@@ -39,6 +39,7 @@ Deno.test(
       assertEquals(result.content.length > 0, true);
     } finally {
       await server.close?.();
+      await new Promise((r) => setTimeout(r, 1000));
     }
   },
 );
@@ -76,6 +77,7 @@ Deno.test(
       assertEquals(result.content.length > 0, true);
     } finally {
       await server.close?.();
+      await new Promise((r) => setTimeout(r, 1000));
     }
   },
 );
@@ -120,6 +122,7 @@ Deno.test(
       }
     } finally {
       await server.close?.();
+      await new Promise((r) => setTimeout(r, 1000));
     }
   },
 );