Skip to content

Conversation

@github-actions
Copy link

🔄 Mirrored from upstream

Field Value
Upstream openai / openai-agents-python PR 1604
Author @radwagusa
Created 2025-08-28
Last Updated 2025-08-28
Mirrored At 2025-08-28T20:24:00.363Z
Commit de8bd96
Status ✅ Ready for review

Original Description (sanitized)

Summary

  • Add streaming tests for PO Assistant endpoints (/po/plan/summary/stream, SSE variant).
  • Add lightweight trace logger and alias-memory helpers used by the example.
  • Ignore local logs/ and traces.
  • DevX: exclude nested dfp-bot/ from Ruff/Mypy; restrict pytest discovery to root tests/.
  • Type fixes in examples (reconcile.py, routes.py); small lint rename.

Test plan

Commands run locally:

  • make format → Ruff format + check: pass.
  • make lint → Ruff lint: pass.
  • make mypy → Strict typing: pass.
  • make tests → Collected 681 tests; 664 passed, 17 failed (see note).

Focused check for new tests:

  • uv run pytest -s -k test_streamed_summary_endpoint → passes.

Issue number

N/A

Checks

  • I've added new tests (where relevant)
  • I've added/updated the relevant documentation
  • I've run make lint and make format
  • I've made sure tests pass (CI pending; locally 664 passed, 17 failed; failures appear related to OpenAI client defaults/API key expectations and shouldn't affect the example tests)

@github-actions
Copy link
Author

This PR is stale because it has been open for 10 days with no activity.

@github-actions github-actions bot added the stale label Sep 27, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants