ServerRuntime: config copy in init

Author: tobitege

opendevin/runtime/server/runtime.py

@@ -1,3 +1,4 @@
+import copy
 from typing import Any, Optional
 
 from opendevin.core.config import AppConfig
@@ -43,6 +44,7 @@ def __init__(
         plugins: list[PluginRequirement] | None = None,
         sandbox: Sandbox | None = None,
     ):
+        self.config = copy.deepcopy(config)
         super().__init__(config, event_stream, sid, plugins)
         self.file_store = LocalFileStore(config.workspace_base)
         if sandbox is None:

tests/unit/test_runtime.py

@@ -141,6 +141,9 @@ async def _load_runtime(
         runtime = ServerRuntime(
             config=config, event_stream=event_stream, sid=sid, plugins=plugins
         )
+        assert (
+            runtime.sandbox.run_as_devin == run_as_devin
+        ), f'run_as_devin in sandbox should be {run_as_devin}'
         await runtime.ainit()
         from opendevin.runtime.tools import (
             RuntimeTool,  # deprecate this after ServerRuntime is deprecated