2 Commits

• Python: Add load_dotenv() to samples for .env file support (#4043)

  * Initial plan
  
  * Add load_dotenv() to 303 Python samples for environment variable loading
  
  Co-authored-by: TaoChenOSU <12570346+TaoChenOSU@users.noreply.github.com>
  
  * Update SAMPLE_GUIDELINES.md to document load_dotenv() requirement
  
  Co-authored-by: TaoChenOSU <12570346+TaoChenOSU@users.noreply.github.com>
  
  * Update samples README.md to document .env file usage
  
  Co-authored-by: TaoChenOSU <12570346+TaoChenOSU@users.noreply.github.com>
  
  * Run ruff format on all changed sample files
  
  Co-authored-by: TaoChenOSU <12570346+TaoChenOSU@users.noreply.github.com>
  
  * Clarify load_dotenv() usage in README - local dev vs production
  
  Co-authored-by: TaoChenOSU <12570346+TaoChenOSU@users.noreply.github.com>
  
  * Remove deprecated getting_started folder as requested
  
  Co-authored-by: TaoChenOSU <12570346+TaoChenOSU@users.noreply.github.com>
  
  * Document env_file_path parameter for per-client configuration
  
  Co-authored-by: TaoChenOSU <12570346+TaoChenOSU@users.noreply.github.com>
  
  * Merge main branch to resolve conflicts
  
  Co-authored-by: TaoChenOSU <12570346+TaoChenOSU@users.noreply.github.com>
  
  * Fix run_evaluation.py file that was empty in merge commit
  
  Co-authored-by: TaoChenOSU <12570346+TaoChenOSU@users.noreply.github.com>
  
  * Remove dotnet changes from merge - out of scope for this PR
  
  Co-authored-by: TaoChenOSU <12570346+TaoChenOSU@users.noreply.github.com>
  
  * Remove package and test changes from merge - only sample changes needed
  
  Co-authored-by: TaoChenOSU <12570346+TaoChenOSU@users.noreply.github.com>
  
  * Remove test_func_utils.py - only sample changes needed
  
  Co-authored-by: TaoChenOSU <12570346+TaoChenOSU@users.noreply.github.com>
  
  * Revert sample files not in original changeset - keep only load_dotenv additions
  
  Co-authored-by: TaoChenOSU <12570346+TaoChenOSU@users.noreply.github.com>
  
  * Move load_dotenv() outside snippet tag in 06_host_your_agent.py
  
  Co-authored-by: TaoChenOSU <12570346+TaoChenOSU@users.noreply.github.com>
  
  * Fix comment placement - move load_dotenv before code comments
  
  Co-authored-by: TaoChenOSU <12570346+TaoChenOSU@users.noreply.github.com>
  
  * Fix load_dotenv() placement across all samples - after docstring, before code comments
  
  Co-authored-by: TaoChenOSU <12570346+TaoChenOSU@users.noreply.github.com>
  
  * Merge latest main branch with load_dotenv changes
  
  Co-authored-by: TaoChenOSU <12570346+TaoChenOSU@users.noreply.github.com>
  
  * Remove non-sample changes from merge - keep only load_dotenv additions
  
  Co-authored-by: TaoChenOSU <12570346+TaoChenOSU@users.noreply.github.com>
  
  * Revert non-load_dotenv sample changes from merge
  
  Co-authored-by: TaoChenOSU <12570346+TaoChenOSU@users.noreply.github.com>
  
  * Fix run_evaluation.py - use main's improved version (file already had load_dotenv)
  
  Co-authored-by: TaoChenOSU <12570346+TaoChenOSU@users.noreply.github.com>
  
  * Manual update
  
  * Manual update 2
  
  * Fix Role usage and load_dotenv placement per PR review feedback
  
  Co-authored-by: eavanvalkenburg <13749212+eavanvalkenburg@users.noreply.github.com>
  
  * Fix Role usage - use string literals not enum attributes
  
  Co-authored-by: eavanvalkenburg <13749212+eavanvalkenburg@users.noreply.github.com>
  
  * Fix SAMPLE_GUIDELINES.md example - load_dotenv before docstring per guidance
  
  Co-authored-by: eavanvalkenburg <13749212+eavanvalkenburg@users.noreply.github.com>
  
  * Move load_dotenv() before docstrings in all samples per SAMPLE_GUIDELINES ordering
  
  Co-authored-by: eavanvalkenburg <13749212+eavanvalkenburg@users.noreply.github.com>
  
  * Address PR review: rename files, fix placement, add session usage, remove note
  
  Co-authored-by: eavanvalkenburg <13749212+eavanvalkenburg@users.noreply.github.com>
  
  * Update Redis README to reference renamed file redis_history_provider.py
  
  Co-authored-by: eavanvalkenburg <13749212+eavanvalkenburg@users.noreply.github.com>
  
  ---------
  
  Co-authored-by: copilot-swe-agent[bot] <198982749+Copilot@users.noreply.github.com>
  Co-authored-by: TaoChenOSU <12570346+TaoChenOSU@users.noreply.github.com>
  Co-authored-by: Tao Chen <taochen@microsoft.com>
  Co-authored-by: eavanvalkenburg <13749212+eavanvalkenburg@users.noreply.github.com>
  Co-authored-by: Eduard van Valkenburg <eavanvalkenburg@users.noreply.github.com>

  Copilot · 2026-02-19 10:55:13 +00:00
  b05fc9e849
• Python: restructure: Python samples into progressive 01-05 layout (#3862)

  * restructure: Python samples into progressive 01-05 layout
  
  - 01-get-started/: 6 numbered steps (hello agent → hosting)
  - 02-agents/: all agent concept samples (tools, middleware, providers, etc.)
  - 03-workflows/: ALL existing workflow samples preserved as-is
  - 04-hosting/: azure-functions, durabletask, a2a
  - 05-end-to-end/: demos, evaluation, hosted agents
  - Old files moved to _to_delete/ for review
  - Added AGENTS.md with structure documentation
  - autogen-migration/ and semantic-kernel-migration/ preserved at root
  
  * fix: switch to AzureOpenAI Foundry, fix CI failures
  
  - Switch all 01-get-started samples to AzureOpenAIResponsesClient with
    Azure AI Foundry project endpoint (AZURE_AI_PROJECT_ENDPOINT +
    AZURE_OPENAI_RESPONSES_DEPLOYMENT_NAME + AzureCliCredential)
  - Add _to_delete/ and 05-end-to-end/ to pyrightconfig.samples.json excludes
  - Fix test paths in packages/ that referenced old getting_started/ dirs:
    durabletask conftest + streaming test, azurefunctions conftest,
    devui conftest + capture_messages + openai_sdk_integration
  - Fix workflow_as_agent_human_in_the_loop.py import (sibling import)
  - Update hosting READMEs and tool comment paths
  - Replace root README.md with new structure overview
  - Update AGENTS.md to document Azure OpenAI Foundry as default provider
  
  * cleanup: remove _to_delete folder, copy resource files to active dirs
  
  All files in _to_delete/ were either:
  - Exact duplicates of files in the new structure (240 files)
  - Same file with only comment path updates (100 files)
  - One import-fix diff (workflow_as_agent_human_in_the_loop.py)
  - One superseded minimal_sample.py
  
  Resource files (sample.pdf, countries.json, employees.pdf, weather.json)
  copied to 02-agents/sample_assets/ and 02-agents/resources/ since active
  samples reference them.
  
  * fix: address PR review comments, centralize resources, remove root duplicates
  
  - Fix type annotation in 04_memory.py (string union -> proper types)
  - Fix old sample paths in observability files
  - Fix grammar/spelling in observability samples
  - Move sample_assets/ and resources/ to shared/ folder
  - Remove 8 duplicate observability files from 02-agents root
  - Update resource path references in multimodal_input and provider samples
  
  * fix: update broken links from old getting_started paths to new structure
  
  - Update relative paths in READMEs: getting_started/ → 01-get-started/,
    02-agents/, 03-workflows/, 04-hosting/, 05-end-to-end/
  - Fix absolute GitHub URLs in package READMEs
  - Fix broken link in ollama package README
  
  * fix: convert absolute GitHub URLs to relative paths for link checker
  
  Absolute URLs to python/samples/ on main branch 404 until PR merges.
  Converted to relative paths that linkspector can verify locally.
  
  * fix: update link for handoff sample moved to orchestrations/
  
  * fix: update chatkit-integration README path from demos/ to 05-end-to-end/
  
  * fix: update broken links in orchestrations README to match flat directory structure

  Eduard van Valkenburg · 2026-02-12 17:36:36 +00:00
  a2856d3b92