add provider bindings MVP to goose-sdk, and add python wheel publishing - #10208
Conversation
![chatgpt-codex-connector[bot]](https://avatars.githubusercontent.com/in/1144995?s=60&v=4){kind=small}
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: af9e6e1cfe
ℹ️ About Codex in GitHub
Your team has set up Codex to 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 👍.
Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".
jamadeo
changed the title
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: b4056a05e6
ℹ️ About Codex in GitHub
Your team has set up Codex to 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 👍.
Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: a74fa02698
ℹ️ About Codex in GitHub
Your team has set up Codex to 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 👍.
Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".
| goose-provider-types \ | ||
| goose-sdk-types \ | ||
| goose-download-manager \ | ||
| goose-local-inference \ |
There was a problem hiding this comment.
Fresh evidence beyond the earlier path-dependency report: on this tree cargo package -p goose-local-inference --allow-dirty --no-verify now fails because Cargo strips git sources and the optional mlx-lm/mlx-lm-utils dependencies have no registry versions. Since this recipe publishes goose-local-inference before goose-providers/goose-sdk, crates-publish stops here and the SDK release cannot proceed until that crate is made publishable or excluded from this publish chain.
Useful? React with 👍 / 👎.
There was a problem hiding this comment.
we'll need to update the git dependencies before this will work
![github-advanced-security[bot]](https://avatars.githubusercontent.com/in/57789?s=60&v=4){kind=small}
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: 9e6bfb095e
ℹ️ About Codex in GitHub
Your team has set up Codex to 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 👍.
Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".
| os: macos-14 | ||
| - name: macos-x86_64 | ||
| os: macos-13 | ||
| - name: linux-x86_64 |
There was a problem hiding this comment.
When this new Linux matrix entry runs, python-wheel builds the binary wheel directly on ubuntu-latest, and setup.py preserves bdist_wheel's host platform tag, producing a *-linux_x86_64.whl. PyPI rejects Linux binary wheels with that unsupported platform tag unless they are built/repaired as manylinux or musllinux, so the publish job will fail or partially publish whenever publish=true includes this artifact.
Useful? React with 👍 / 👎.
DOsinga
left a comment
DOsinga
left a comment
There was a problem hiding this comment.
Approving based on core-team ownership; Jack will follow through on the remaining packaging/publishing details.
…mestamp-timezone * origin/main: (34 commits) feat: askai bot hints for followups (aaif-goose#10291) feat(desktop): per-message usage stats UI (tokens, cost, TTFT, tok/s) (aaif-goose#10210) fix(server): return effective context limit from /model-info (aaif-goose#10165) fix(deps): gate rcgen aws_lc_rs feature behind rustls-tls (aaif-goose#10116) Disable thinking preservation for Groq (aaif-goose#10284) Bound code-mode execution with timeout and cancellation (aaif-goose#10214) chore(deps): bump docker/setup-buildx-action from 4.1.0 to 4.2.0 (aaif-goose#10232) chore(deps): bump docker/build-push-action from 7.2.0 to 7.3.0 (aaif-goose#10233) chore(deps): bump docker/metadata-action from 6.1.0 to 6.2.0 (aaif-goose#10235) chore(deps): bump mockall from 0.14.0 to 0.15.0 (aaif-goose#10240) chore(deps): bump uniffi from 0.31.1 to 0.32.0 (aaif-goose#10247) chore(deps): bump sigstore-verify from 0.9.0 to 0.10.0 (aaif-goose#10250) chore(deps): bump cmov from 0.5.3 to 0.5.4 (aaif-goose#10282) chore(deps): bump candle-transformers from 0.10.2 to 0.11.0 (aaif-goose#10242) chore(deps): bump candle-core from 0.10.2 to 0.11.0 (aaif-goose#10241) fix: recover malformed tool calls from GLM/Minimax models instead of rejecting them (aaif-goose#10230) chore(deps): bump jsonschema from 0.30.0 to 0.46.9 (aaif-goose#10244) move google provider into goose-providers (aaif-goose#10216) chore(deps): bump umya-spreadsheet from 2.3.3 to 3.0.0 (aaif-goose#10248) add provider bindings MVP to goose-sdk, and add python wheel publishing (aaif-goose#10208) ...
3 participants
for #9803