Skip to content

fix(openai): preserve empty hosted MCP outputs - #1747

Merged
seratch merged 1 commit into
openai:mainfrom
Excelius-Wang:fix/preserve-empty-mcp-output
Aug 23, 2026
Merged

seratch merged 1 commit into
openai:mainfrom
Excelius-Wang:fix/preserve-empty-mcp-output

Conversation

@Excelius-Wang

Copy link
Copy Markdown
Contributor

Summary

This pull request fixes hosted MCP response conversion so an explicit empty-string output remains "" instead of being collapsed to undefined.

The nullish fallback preserves the existing handling of null and omitted outputs. It also adds focused regression coverage and a patch changeset for @openai/agents-openai.

Test plan

  • pnpm exec vitest run packages/agents-openai/test/openaiResponsesModel.helpers.test.ts --mode full
  • .agents/skills/code-change-verification/scripts/run.sh
  • pnpm changeset:validate-result

Issue number

N/A

Checks

  • I've added new tests
  • Documentation is not required for this focused conversion fix
  • I've run pnpm test and the examples build checks
  • I've run .agents/skills/code-change-verification/scripts/run.sh
  • I've confirmed all verification steps pass
  • I've completed the Codex implementation review
  • I've added a changeset

@changeset-bot

changeset-bot Bot commented Aug 23, 2026

Copy link
Copy Markdown

🦋 Changeset detected

Latest commit: 92f3fd7

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 6 packages
Name Type
@openai/agents-openai Patch
@openai/agents Patch
@openai/agents-core Patch
@openai/agents-realtime Patch
@openai/agents-extensions Patch
realtime-react-native Patch

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

@seratch seratch added this to the 0.17.x milestone Aug 23, 2026
@seratch
seratch enabled auto-merge (squash) August 23, 2026 22:24
@seratch
seratch merged commit 02720cd into openai:main Aug 23, 2026
6 checks passed
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