Repository navigation
[codex] Clarify Python SDK skill invocation - #3
Conversation
|
I took a look at this PR:
The only thing to resolve before merging is that this PR is still marked as Draft. GitHub will not allow a Draft PR to be merged, so please mark it as |
d8465c7 to
a7ac7ee
Compare
|
Rebased the contribution onto the current modular Python SDK while preserving the original author. The moved Validation:
|
ZhiXiao-Lin
left a comment
There was a problem hiding this comment.
Reviewed the Python SDK examples and bindings. The examples now use the synchronous session.send() / session.tool() APIs correctly, SessionOptions is imported before use, and both supported skill-directory configuration paths are documented consistently. CI is green.
ZhiXiao-Lin
left a comment
There was a problem hiding this comment.
Revalidated after merging current main (v6.0.0) into the contribution. The only conflict was the Python SDK wheel-install example; the current 6.0.0 placeholder guidance was retained while preserving the PR's skill-discovery and synchronous API corrections.
Local validation passed: formatting, event-protocol artifacts, SDK API alignment, Python example syntax, and cargo check --manifest-path sdk/python/Cargo.toml. GitHub CI is now green for Check, SDK build check, and Windows check.
Summary
Clarifies the Python SDK path for folder-style skills and direct skill invocation.
skill_dirs,search_skills, andSkillusage in the Python SDK READMEawait session.send(...)usage to the current synchronous APIawait session.send(...)Why
The SDK already supports loading folder-style skills through
skill_dirsand invoking them withsession.tool("Skill", {...}), but the Python-facing docs did not show that path clearly. The old example also mixed in async-style usage that does not match the current PyO3 API.Validation
git diff --check origin/main..HEADpython3 -m py_compile examples/test_skill_tool.pyRust checks were not run locally because
cargois not installed in this environment.