Skip to content

test(code-interpreter/python): cover async endpoint headers#509

Merged
ninan-nn merged 1 commit intoalibaba:mainfrom
Gujiassh:test/python-code-interpreter-async-endpoint-headers
Mar 25, 2026
Merged

test(code-interpreter/python): cover async endpoint headers#509
ninan-nn merged 1 commit intoalibaba:mainfrom
Gujiassh:test/python-code-interpreter-async-endpoint-headers

Conversation

@Gujiassh
Copy link
Copy Markdown
Contributor

Summary

  • add async Python code-interpreter regression coverage proving SandboxEndpoint.headers reaches both the generated OpenAPI client and the SSE execution path
  • lock endpoint-header precedence over connection-level headers so future parity changes do not regress the current async behavior
  • keep this follow-up test-only and scoped to the Python async package while the sync parity fix is handled separately in fix(code-interpreter/python): apply endpoint headers in sync adapter #504

Validation

  • cd sdks/code-interpreter/python && uv run pytest tests/test_adapter_eager_init.py tests/test_code_service_adapter_streaming.py -q
  • cd sdks/code-interpreter/python && uv run ruff check tests/test_adapter_eager_init.py tests/test_code_service_adapter_streaming.py

Ultraworked with [Sisyphus](https://github.com/code-yeongyu/oh-my-opencode)

Co-authored-by: Sisyphus <clio-agent@sisyphuslabs.ai>
Copy link
Copy Markdown
Collaborator

@ninan-nn ninan-nn left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@ninan-nn ninan-nn merged commit 17b1689 into alibaba:main Mar 25, 2026
11 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.

3 participants