Skip to content

Ensure OpenAI dependencies are updated in CI#341

Open
zhongkaifu wants to merge 1 commit intomainfrom
codex/analyze-importerror-for-asyncopenai
Open

Ensure OpenAI dependencies are updated in CI#341
zhongkaifu wants to merge 1 commit intomainfrom
codex/analyze-importerror-for-asyncopenai

Conversation

@zhongkaifu
Copy link
Copy Markdown
Owner

Motivation

  • Tests were failing with ImportError: cannot import name 'AsyncOpenAI' from 'openai' due to an openai / openai-agents version mismatch.
  • The CI installs dependencies via pip install -r requirements.txt, which can still result in an older openai being present (cache, resolver behavior, or preinstalled package).
  • Ensure CI installs a compatible openai (1.x) before running tests so openai-agents can import AsyncOpenAI.

Description

  • Update .github/workflows/ci.yml to explicitly upgrade/install openai>=1.30.0 and openai-agents during the Install dependencies step: pip install -U "openai>=1.30.0" "openai-agents".
  • Keep the existing pip install -r requirements.txt step to install the rest of the dependencies.
  • Add a small Python snippet in CI to print installed versions: prints openai.__version__ and agents.__version__ (or unknown) to help verify the installed packages.

Testing

  • No automated tests were executed as part of this change (CI workflow change only).
  • CI will still run the repository test suite with pytest --junitxml=reports/pytest.xml as before; the added version prints will help diagnose any remaining dependency issues.
  • The change was applied and committed; validation of the fix will occur on the next CI run where the versions will be printed in the job logs.

Codex Task

@chatgpt-codex-connector
Copy link
Copy Markdown

You have reached your Codex usage limits for code reviews. You can see your limits in the Codex usage dashboard.
To continue using code reviews, you can upgrade your account or add credits to your account and enable them for code reviews in your settings.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant