Skip to content

feat(sdk/kotlin): allow custom execd port in Connector#806

Merged
ninan-nn merged 1 commit intoalibaba:mainfrom
Pangjiping:feat/kotlin/execd-port
Apr 27, 2026
Merged

feat(sdk/kotlin): allow custom execd port in Connector#806
ninan-nn merged 1 commit intoalibaba:mainfrom
Pangjiping:feat/kotlin/execd-port

Conversation

@Pangjiping
Copy link
Copy Markdown
Collaborator

Summary

  • Add execdPort(port: Int) setter to Sandbox.Connector, defaulting to DEFAULT_EXECD_PORT (44772) when not set. Threads through the shared initializeSandbox path without changing Builder or Resumer.

Testing

  • Not run (explain why)
  • Unit tests
  • Integration tests
  • e2e / manual verification

Breaking Changes

  • None
  • Yes (describe impact and migration path)

Checklist

  • Linked Issue or clearly described motivation
  • Added/updated docs (if needed)
  • Added/updated tests (if needed)
  • Security impact considered
  • Backward compatibility considered

Add execdPort(port: Int) setter to Sandbox.Connector, defaulting to
DEFAULT_EXECD_PORT (44772) when not set. Threads through the shared
initializeSandbox path without changing Builder or Resumer.

Co-Authored-By: Claude Opus 4.7 <noreply@anthropic.com>
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 c51baf9 into alibaba:main Apr 27, 2026
24 of 25 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

feature New feature or request sdk/java

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants