-
Notifications
You must be signed in to change notification settings - Fork 3.3k
Consider extensible pattern for protocol flow-control exceptions #1788
Copy link
Copy link
Open
Labels
P2Moderate issues affecting some users, edge cases, potentially valuable featureModerate issues affecting some users, edge cases, potentially valuable featureenhancementRequest for a new feature that's not currently supportedRequest for a new feature that's not currently supportedneeds decisionIssue is actionable, needs maintainer decision on whether to implementIssue is actionable, needs maintainer decision on whether to implementv2Ideas, requests and plans for v2 of the SDK which will incorporate major changes and fixesIdeas, requests and plans for v2 of the SDK which will incorporate major changes and fixes
Metadata
Metadata
Assignees
Labels
P2Moderate issues affecting some users, edge cases, potentially valuable featureModerate issues affecting some users, edge cases, potentially valuable featureenhancementRequest for a new feature that's not currently supportedRequest for a new feature that's not currently supportedneeds decisionIssue is actionable, needs maintainer decision on whether to implementIssue is actionable, needs maintainer decision on whether to implementv2Ideas, requests and plans for v2 of the SDK which will incorporate major changes and fixesIdeas, requests and plans for v2 of the SDK which will incorporate major changes and fixes
PR #1780 adds specific
except UrlElicitationRequiredError: raiseblocks in tool handlers to prevent this exception from being wrapped as a tool error.Per SEP-1036,
UrlElicitationRequiredErroris not a regular error—it's a protocol-level flow control mechanism equivalent toelicitation/create. It needs to propagate as a JSON-RPC error response.If more continuation-style exceptions are added in the future, consider an attribute-based pattern:
For now, YAGNI applies—we only have one such case.