fix(runner): make RunnerContext.get_env() read live from os.environ

RunnerContext.__post_init__ snapshots os.environ once at construction.
After that, get_env() reads from the frozen dict. Multiple endpoints
mutate os.environ at runtime (POST /workflow, POST /repos/add,
auth refresh), but the context is never recreated — mark_dirty()
resets the bridge while reusing the same stale context.

Store explicit constructor overrides separately in _overrides and
make get_env() check overrides first, then fall through to live
os.environ. self.environment remains populated for backward compat.

Made-with: Cursor

Author: bobbravo2

components/runners/ambient-runner/ambient_runner/platform/context.py

@@ -28,12 +28,15 @@ class RunnerContext:
     metadata: Dict[str, Any] = field(default_factory=dict)
 
     def __post_init__(self) -> None:
-        """Merge environment variables (explicit overrides win)."""
+        """Store explicit overrides for precedence in get_env(); keep environment populated for backward compatibility."""
+        self._overrides = dict(self.environment)
         self.environment = {**os.environ, **self.environment}
 
     def get_env(self, key: str, default: Optional[str] = None) -> Optional[str]:
-        """Get an environment variable value."""
-        return self.environment.get(key, default)
+        """Get an environment variable, with explicit overrides winning. Reads live from os.environ for non-overridden keys."""
+        if key in self._overrides:
+            return self._overrides[key]
+        return os.environ.get(key, default)
 
     def set_metadata(self, key: str, value: Any) -> None:
         """Set a metadata value."""

components/runners/ambient-runner/tests/test_context.py

@@ -0,0 +1,40 @@
+"""Tests for RunnerContext."""
+
+import os
+
+import pytest
+
+from ambient_runner.platform.context import RunnerContext
+
+
+class TestRunnerContextGetEnv:
+    """Verify get_env() reads live from os.environ and respects explicit overrides."""
+
+    def test_get_env_sees_os_environ_mutations_after_creation(self):
+        """get_env() must return current os.environ value when key is mutated at runtime."""
+        key = "_RUNNER_CONTEXT_TEST_MUTATION_"
+        try:
+            if key in os.environ:
+                del os.environ[key]
+            ctx = RunnerContext(session_id="s1", workspace_path="/tmp")
+            assert ctx.get_env(key) is None
+            os.environ[key] = "runtime-mutated"
+            assert ctx.get_env(key) == "runtime-mutated"
+        finally:
+            os.environ.pop(key, None)
+
+    def test_explicit_overrides_win_over_os_environ(self):
+        """Explicit overrides passed at construction must take precedence over os.environ."""
+        key = "_RUNNER_CONTEXT_TEST_OVERRIDE_"
+        try:
+            os.environ[key] = "from-os-environ"
+            ctx = RunnerContext(
+                session_id="s1",
+                workspace_path="/tmp",
+                environment={key: "from-constructor"},
+            )
+            assert ctx.get_env(key) == "from-constructor"
+            os.environ[key] = "mutated-after-creation"
+            assert ctx.get_env(key) == "from-constructor"
+        finally:
+            os.environ.pop(key, None)