fix: fall back to session_id for multi-turn when conversation_id is missing

The ResponseHandler context only populates conversation_id when the
request includes an explicit "conversation" field. Most callers (invoke
scripts, Playground) only send session_id. Without this fallback,
conversation_id is always None and the adapter creates a fresh Copilot
SDK session on every request, breaking multi-turn.

Also fixes Windows --no-logs in integration test deploy script.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>

Author: jodeklotzms

sdk/agentserver/azure-ai-agentserver-githubcopilot/azure/ai/agentserver/githubcopilot/_copilot_adapter.py

@@ -362,7 +362,24 @@ async def handle_create(request, context, cancellation_signal):
     async def _handle_create(self, request, context, cancellation_signal):
         """Handle POST /responses — bridge Copilot SDK events to RAPI stream."""
         input_text = _extract_input_with_attachments(request)
+
+        # Resolve conversation identity for multi-turn session reuse.
+        # Prefer conversation_id from the context (set when the request includes
+        # a "conversation" field).  Fall back to session_id from the parsed
+        # request so that callers who only pass session_id still get multi-turn.
         conversation_id = getattr(context, "conversation_id", None)
+        if not conversation_id:
+            # Try session_id from the parsed request object
+            parsed = getattr(context, "request", None) or getattr(context, "parsed", None)
+            if parsed is not None:
+                conversation_id = getattr(parsed, "session_id", None)
+            # Last resort: check the raw request
+            if not conversation_id:
+                if isinstance(request, dict):
+                    conversation_id = request.get("session_id")
+                else:
+                    conversation_id = getattr(request, "session_id", None)
+
         response_id = getattr(context, "response_id", None) or "unknown"
 
         logger.info(f"Request: input={input_text[:100]!r} conversation_id={conversation_id}")

sdk/agentserver/azure-ai-agentserver-githubcopilot/tests/integration/deploy.py

@@ -69,23 +69,38 @@ def build_image(staging_dir: Path, acr: str, name: str, tag: str) -> str:
     full_image = f"{acr}.azurecr.io/{name}:{tag}"
     print(f"Building {full_image} via ACR Tasks...")
 
+    is_win = sys.platform == "win32"
+
     cmd = ["az", "acr", "build",
            "--registry", acr,
            "--image", f"{name}:{tag}",
            "--platform", "linux/amd64",
            "--file", str(staging_dir / "Dockerfile"),
            str(staging_dir)]
 
-    is_win = sys.platform == "win32"
-    env = {**os.environ, "PYTHONIOENCODING": "utf-8", "PYTHONUTF8": "1"} if is_win else None
-    proc = subprocess.Popen(
-        cmd, stdout=subprocess.PIPE, stderr=subprocess.STDOUT,
-        encoding="utf-8", errors="replace", shell=is_win, env=env,
-    )
-    for line in proc.stdout:
-        sys.stdout.write(line)
-        sys.stdout.flush()
-    returncode = proc.wait()
+    if is_win:
+        # Skip log streaming on Windows to avoid colorama + cp1252 encoding crash.
+        cmd.insert(3, "--no-logs")
+        print("  (Windows: using --no-logs to avoid encoding issues)")
+        env = {**os.environ, "PYTHONIOENCODING": "utf-8", "PYTHONUTF8": "1"}
+        result = subprocess.run(
+            cmd, capture_output=True, encoding="utf-8", errors="replace",
+            shell=True, env=env,
+        )
+        if result.stdout:
+            sys.stdout.write(result.stdout)
+        if result.stderr:
+            sys.stderr.write(result.stderr)
+        returncode = result.returncode
+    else:
+        proc = subprocess.Popen(
+            cmd, stdout=subprocess.PIPE, stderr=subprocess.STDOUT,
+            encoding="utf-8", errors="replace",
+        )
+        for line in proc.stdout:
+            sys.stdout.write(line)
+            sys.stdout.flush()
+        returncode = proc.wait()
 
     if returncode != 0:
         print("\nWarning: az acr build returned non-zero exit code.", file=sys.stderr)