[JeffreyCA]

Fixes #1038

This PR fixes the Python and C# notetaking Responses samples so notes saved on a fresh hosted session land in that session instead of a shared "default" file.

Issue

azd ai agent invoke --new-session doesn't send agent_session_id in the request body for hosted agents. The platform creates the session and returns its ID in the x-agent-session-id header, and azd saves that ID for the next invoke. Both handlers only read the body field, though, so that first call wrote to notes_default.jsonl.

The next invoke sends the real session ID, so get_notes reads notes_<id>.jsonl and comes back empty. Invokes that already had a session ID worked, which is why the bug only shows up after --new-session.

This isn't an azd regression. azd has left the body field empty for new remote sessions since /invocations support was added. The Python agentserver SDK resolves the session ID as the body field, then FOUNDRY_AGENT_SESSION_ID, then a derived value. The samples skipped the middle step. The .NET SDK reads the same variable.

Fix

Both handlers now resolve the session ID in this order:

• agent_session_id from the request body, used by local runs and explicit callers
• FOUNDRY_AGENT_SESSION_ID, set by the platform in hosted containers
• "default", kept for local curl calls that pass no session

Local behaviour doesn't change, because azd ai agent invoke --local always sends a session ID in the body.

Notes still aren't shared across sessions. Each hosted session has its own sandbox and $HOME, so a note saved in one session won't show up after a second --new-session. That part of the report is by design.

Testing

• Checked the Python fallback order with a small script covering body-only, env-only, both, and neither.
• Ran py_compile on the Python sample and dotnet build on the C# sample.
• Not yet verified against a live hosted deployment. The repro steps in Note-Taking Agent does not invoke tools when starting a new session in remote #1038 should now show the saved note in step 4.

[azure-pipelines]

Azure Pipelines:
There may be pipelines that require an authorized user to comment /azp run to run.

[Copilot]

🟢 Approval recommended

Both implementations correctly address the missing request-body session ID while preserving local behavior.

0 open findings

What changed in this PR

Fixes note persistence for newly created hosted sessions by using the platform-provided session ID fallback.

Changes:

• Resolves session IDs from request body, environment, then "default".
• Applies equivalent behavior to Python and C# samples.

File	Description
main.py	Adds hosted-session environment fallback.
Program.cs	Adds equivalent C# fallback and helper.

🧠 Review effort: Balanced

---

💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.