Skip to content

feat: add v1.0 error types (ExtensionSupportRequired, VersionNotSupported)#15

Open
zeroasterisk wants to merge 1 commit intoactioncard:mainfrom
zeroasterisk:feat/v1-error-types
Open

feat: add v1.0 error types (ExtensionSupportRequired, VersionNotSupported)#15
zeroasterisk wants to merge 1 commit intoactioncard:mainfrom
zeroasterisk:feat/v1-error-types

Conversation

@zeroasterisk
Copy link
Copy Markdown

Part of #13 (A2A v1.0 Protocol Support)

Changes

Adds the two missing A2A v1.0 error type constructors to A2A.JSONRPC.Error:

  • extension_support_required/1 — code -32008
  • version_not_supported/1 — code -32009

Both follow the existing pattern (default data nil, optional data param).

Includes tests for both new constructors.

Verified Error Codes

All 14 error codes verified against a2a.proto HEAD (Section 5.4 Error Code Mappings):

Constructor Code Status
parse_error -32700
invalid_request -32600
method_not_found -32601
invalid_params -32602
internal_error -32603
task_not_found -32001
task_not_cancelable -32002
push_notification_not_supported -32003
unsupported_operation -32004
content_type_not_supported -32005
invalid_agent_response -32006
authenticated_extended_card_not_configured -32007
extension_support_required -32008 NEW
version_not_supported -32009 NEW

Fixes part of #13

…rted)

Add the two missing A2A v1.0 error constructors:
- extension_support_required/1 (-32008)
- version_not_supported/1 (-32009)

All 14 error codes verified against a2a.proto HEAD:
  -32700 parse_error
  -32600 invalid_request
  -32601 method_not_found
  -32602 invalid_params
  -32603 internal_error
  -32001 task_not_found
  -32002 task_not_cancelable
  -32003 push_notification_not_supported
  -32004 unsupported_operation
  -32005 content_type_not_supported
  -32006 invalid_agent_response
  -32007 authenticated_extended_card_not_configured
  -32008 extension_support_required (NEW)
  -32009 version_not_supported (NEW)

Part of actioncard#13
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.

1 participant