Skip to content

fix: update python and js pilot clients to 1.9-*#90

Merged
TeoSlayer merged 1 commit intomainfrom
fix_sdks
May 6, 2026
Merged

fix: update python and js pilot clients to 1.9-*#90
TeoSlayer merged 1 commit intomainfrom
fix_sdks

Conversation

@Alexgodoroja
Copy link
Copy Markdown
Collaborator

Main codebase impact

None. release.yml does not touch sdk/, the Go driver and registry keep MyPoloScore for internal use, and the SDK publish workflows
(publish-node-sdk.yml, publish-python-sdk.yml) trigger only on release: published — they cannot block or modify the GitHub release.

How to release

Tag push triggers everything; no gh release create needed.

Stable (this PR's target):

git checkout main && git pull                                                                                                                    
git tag -a v1.9.2 -m "v1.9.2"                                                                                                                  
git push origin v1.9.2                                                                                                                           

Prerelease (SDK publish auto-skips on tags containing -rc / -beta):

git tag -a v1.9.2-rc1 -m "v1.9.2-rc1"                                                                                                            
git push origin v1.9.2-rc1                                                                                                                     

Watch and verify:

gh run list --workflow=release.yml --limit 3
gh run list --workflow=publish-node-sdk.yml --limit 3                                                                                            
gh run list --workflow=publish-python-sdk.yml --limit 3
                                                                                                                                                 
gh release view v1.9.2                                                                                                                         
npm view pilotprotocol@1.9.2 version                                                                                                             
curl -s https://pypi.org/pypi/pilotprotocol/1.9.2/json | jq -r .info.version                                                                     

Test plan

  • npm test in sdk/node — 111/111 pass
  • pytest in sdk/python — 124 pass (1 pre-existing /private/var symlink failure in
    TestLibraryDiscoveryFallbacks::test_repo_bin_directory, unrelated to this PR)
  • go build clean on sdk/cgo
  • grep -ri polo sdk/ returns nothing
  • First post-merge tag push: confirm release.yml, publish-node-sdk.yml, publish-python-sdk.yml all green and versions land on npm +
    PyPI

@Alexgodoroja Alexgodoroja requested a review from TeoSlayer as a code owner May 6, 2026 06:57
Comment thread cmd/pilotctl/main.go
"dgram": map[string]interface{}{"args": []string{"<address|hostname>", "<port>", "--data <msg>"}, "description": "Send a UDP-style datagram"},
"listen": map[string]interface{}{"args": []string{"<port>", "[--count <n>]"}, "description": "Listen for incoming datagrams"},
"broadcast": map[string]interface{}{"args": []string{"<network_id>", "<message>"}, "description": "Broadcast a datagram to all network members"},
"connect": map[string]interface{}{"args": []string{"<address|hostname>", "[port]", "[--message <msg>]"}, "description": "Open a raw stream connection"},
Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

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

from pre-commits

"endpoint": "203.0.113.5:4000", // must go
"real_addr": "203.0.113.5:4000", // must go
"public_addr": "203.0.113.5:4000", // must go
"endpoint": "203.0.113.5:4000", // must go
Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

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

recommits

@TeoSlayer TeoSlayer merged commit a4da65b into main May 6, 2026
5 of 6 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants