Skip to content

feat(kotlin-sdk): enrich ready-timeout diagnostics#314

Merged
ninan-nn merged 1 commit intoalibaba:mainfrom
liuxiaopai-ai:codex/kotlin-ready-timeout-diagnostics
Mar 9, 2026
Merged

feat(kotlin-sdk): enrich ready-timeout diagnostics#314
ninan-nn merged 1 commit intoalibaba:mainfrom
liuxiaopai-ai:codex/kotlin-ready-timeout-diagnostics

Conversation

@liuxiaopai-ai
Copy link
Copy Markdown
Contributor

Summary

  • enrich Kotlin SDK checkReady timeout message with:
    • connection context (domain, useServerProxy)
    • actionable bridge-network hints
  • extend SandboxTest with assertions for the new diagnostics behavior

Why

This makes timeout errors more actionable in bridge/remote deployment scenarios and aligns Kotlin SDK troubleshooting with recent Python/JavaScript/C# improvements.

Validation

  • attempted: cd sdks/sandbox/kotlin && ./gradlew :sandbox:test --tests com.alibaba.opensandbox.sandbox.SandboxTest
  • could not run locally because this environment does not have a Java runtime installed

@Pangjiping Pangjiping added feature New feature or request sdk/java labels Mar 3, 2026
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, Rebase Main Plz : >

@liuxiaopai-ai liuxiaopai-ai force-pushed the codex/kotlin-ready-timeout-diagnostics branch from 003ccb7 to 3de3c25 Compare March 9, 2026 04:45
@liuxiaopai-ai
Copy link
Copy Markdown
Contributor Author

Rebased onto the latest main and force-pushed the branch. Please take another look when you have a moment. Thanks!

@liuxiaopai-ai
Copy link
Copy Markdown
Contributor Author

@ninan-nn Rebased onto the latest main and force-pushed the branch. Could you please take another look when you have a moment? Thanks!

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 9356816 into alibaba:main Mar 9, 2026
14 of 15 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.

3 participants