Skip to content

chore(sdks/go): fix go client issues - #683

Merged
hittyt merged 12 commits into
opensandbox-group:mainfrom
Pangjiping:chore/go-sdks-base
Apr 11, 2026
Merged

chore(sdks/go): fix go client issues#683
hittyt merged 12 commits into
opensandbox-group:mainfrom
Pangjiping:chore/go-sdks-base

Conversation

@Pangjiping

@PangjipingPangjiping commented Apr 10, 2026

Copy link
Copy Markdown
Collaborator

Summary

  • fix sdk bugs
  • add more e2e tests
  • simply init package struct

Testing

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

Breaking Changes

  • None - not released
  • 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

@Pangjiping
Pangjipingforce-pushed the chore/go-sdks-base branch 3 times, most recently from 155f839 to a58e7b8CompareApril 10, 2026 12:18
@Pangjiping
Pangjiping marked this pull request as ready for review April 10, 2026 13:55

@chatgpt-codex-connectorchatgpt-codex-connectorBot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit:1d87371d33

ℹ️ About Codex in GitHub

Codex has been enabled to automatically review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

When you sign up for Codex through ChatGPT, Codex can also answer questions or update the PR, like "@codex address that feedback".

Comment threadsdks/sandbox/go/Makefile
Comment threadsdks/sandbox/go/examples/agent_loop/main.go

@hittythittyt left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

LGTM

@hittyt
hittyt merged commit 8f7d090 into opensandbox-group:mainApr 11, 2026
31 of 33 checks passed
@Pangjiping
Pangjiping deleted the chore/go-sdks-base branch April 11, 2026 06:24
@PangjipingPangjiping added bug Something isn't working sdk/go labels Apr 13, 2026
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bugSomething isn't workingsdk/go

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants

@Pangjiping@hittyt@ninan-nn