Skip to content

Commit 5e66b73

Browse files
committed
Remove no-op dispose() method
Since ControlPersist=60 handles all socket cleanup automatically and dispose() is not part of the Runtime interface or called anywhere, there's no reason to keep an empty method.
1 parent 5f09f73 commit 5e66b73

File tree

1 file changed

+0
-12
lines changed

1 file changed

+0
-12
lines changed

src/runtime/SSHRuntime.ts

Lines changed: 0 additions & 12 deletions
Original file line numberDiff line numberDiff line change
@@ -895,18 +895,6 @@ export class SSHRuntime implements Runtime {
895895
return { success: false, error: `Failed to delete directory: ${getErrorMessage(error)}` };
896896
}
897897
}
898-
899-
/**
900-
* Cleanup SSH control socket on disposal.
901-
*
902-
* Note: This is a no-op because:
903-
* - ControlPersist=60 automatically removes socket 60s after last use
904-
* - Multiple SSHRuntime instances may share the same connection
905-
* - Explicit cleanup could interfere with other active operations
906-
*/
907-
dispose(): void {
908-
// No-op: Let ControlPersist handle cleanup automatically
909-
}
910898
}
911899

912900
/**

0 commit comments

Comments
 (0)