Skip to content

Remove npx dependency from OpenAI Agents tests - #8

Merged
brianstrauch merged 4 commits into
mainfrom
remove/openai-agents-npx
Sep 10, 2026
Merged

brianstrauch merged 4 commits into
mainfrom
remove/openai-agents-npx

Conversation

@brianstrauch

Copy link
Copy Markdown
Member

Summary:

  • replace the external MCP subprocess in the no-worker test with the existing in-process tracking server
  • use a streamable HTTP MCP server in public examples
  • remove stale Node prerequisites from contributor documentation

Verification:

  • make sync
  • make lint
  • make test (214 passed, 16 skipped)
  • tooling tests (70 passed)
  • repository conventions
  • wheel and sdist checks

@brianstrauch
brianstrauch requested a review from a team as a code owner September 3, 2026 22:26
@DABH

DABH commented Sep 4, 2026

Copy link
Copy Markdown
Collaborator

Merging this first means #6's merge, and every later re-sync of these regions, conflicts.

Two options I see: (1) open the same three-file change against sdk-python (or fold it into #1793) and pick it up here by re-sync; (2) or, if we want this repo to own openai_agents from now on, make that explicit by dropping upstream from plugin.toml and retiring the rule in AGENTS.md rather than bypassing it. Maybe prefer the first while #1793 is still open? Wdyt?

Comment thread python/openai_agents/tests/contrib/openai_agents/test_openai.py Outdated
@DABH DABH mentioned this pull request Sep 4, 2026
@brianstrauch
brianstrauch requested a review from DABH September 4, 2026 20:45
@brianstrauch
brianstrauch merged commit 7ce5d7c into main Sep 10, 2026
18 checks passed
@brianstrauch
brianstrauch deleted the remove/openai-agents-npx branch September 10, 2026 18:49
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants