FazBrowse GitHub Viewer | Trending |
URL:
| Home
Tools: [Download Repo ZIP]   [Original HTTPS Page]

fix: complete schema v0.11.2 follow-ups by PsiACE · Pull Request #81 · agentclientprotocol/python-sdk · GitHub

/ python-sdk Public

fix: complete schema v0.11.2 follow-ups - #81

Merged
PsiACE merged 1 commit into
mainfrom
fix/schema-v0112-followups
Mar 26, 2026
ConversationCommits (1)ChecksFiles changed
Merged

fix: complete schema v0.11.2 follow-ups#81
PsiACE merged 1 commit into
mainfrom
fix/schema-v0112-followups

Conversation

PsiACE commented Mar 25, 2026

Copy link
Copy Markdown
Member

Summary

Complete the remaining schema v0.11.2 follow-ups:

  • support both select and boolean variants for session/set_config_option
  • pass through message_id in ClientSideConnection.prompt()
  • keep legacy params-style compatibility for the new config option request shapes

Related issues

Relates to #80

Testing

  • make check
  • make test

Docs & screenshots

No docs or example updates needed.

Checklist

  • Conventional Commit title (e.g. feat:, fix:).
  • Tests cover the change or are not required (explain above).
  • Docs/examples updated when behaviour is user-facing.
  • Schema regenerations (make gen-all) are called out if applicable.

No schema regeneration needed.

PsiACE commented Mar 25, 2026

Copy link
Copy Markdown
Member Author

cc @Nemtecl Could you help review it? before releasing version 0.9.0.

Nemtecl approved these changes Mar 25, 2026
View reviewed changes

Nemtecl left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Choose a reason Spam Abuse Off Topic Outdated Duplicate Resolved Low Quality

LGTM ! Let's try this and happy to help on iterating if needed

PsiACE merged commit a57c672 into main Mar 26, 2026
8 checks passed
PsiACE deleted the fix/schema-v0112-followups branch March 26, 2026 01:21

PsiACE commented Mar 26, 2026

Copy link
Copy Markdown
Member Author

thanks @Nemtecl , 0.9.0 is released

This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters. Learn more about bidirectional Unicode characters
Show hidden characters
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

Nemtecl Nemtecl approved these changes

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

Back | FazBrowse Home | New Git URL