docs: add example tests a2a#240
Conversation
Summary of ChangesHello @cchinchilla-dev, I'm Gemini Code Assist1! I'm currently reviewing this pull request and will post my feedback shortly. In the meantime, here's a summary to help you and other reviewers quickly get up to speed! This pull request enhances the Highlights
Changelog
Activity
Using Gemini Code AssistThe full guide for Gemini Code Assist can be found on our documentation page, here are some quick tips. Invoking Gemini You can request assistance from Gemini at any point by creating a comment using either
Customization To customize Gemini Code Assist for GitHub experience, repository maintainers can create a configuration file and/or provide a custom code review style guide (such as PEP-8 for Python) by creating and adding files to a Limitations & Feedback Gemini Code Assist may make mistakes. Please leave feedback on any instances where its feedback is incorrect or counter productive. You can react with 👍 and 👎 on @gemini-code-assist comments. If you're interested in giving your feedback about your experience with Gemini Code Assist for Github and other Google products, sign up here. You can also get AI-powered code generation, chat, as well as code reviews directly in the IDE at no cost with the Gemini Code Assist IDE Extension. Footnotes
|
There was a problem hiding this comment.
Code Review
The pull request introduces valuable example tests for the a2a core package, significantly enhancing documentation and providing executable usage snippets. The examples cover various core types and functions, which is a great addition to the project. There are a couple of minor improvements related to error handling in JSON unmarshaling and adherence to the repository's guideline for using constructor functions when creating core types.
|
Applied all suggestions — switched to constructor usage and added explicit error handling for |
|
Hi, sorry for not responding on the issue. I think that'd be a great contribution, but we're about to promote release/spec-v1 to main with tons of changes. Can you please merge and check if things work after that. |
eff6a21 to
3616064
Compare
|
Hi @yarolegovich! No worries at all — completely understand the pace with the spec v1 transition. Rebased on latest main and all 12 examples pass against the v1 module. |
a2a/example_test.go
Outdated
| "errors" | ||
| "fmt" | ||
|
|
||
| "github.com/a2aproject/a2a-go/a2a" |
There was a problem hiding this comment.
this is a legacy package now, should be github.com/a2aproject/a2a-go/v1/a2a.
the old name is still available because I imported it for compatibility layer (a2acompat)
There was a problem hiding this comment.
You're right, thanks! Updated the import path and made a few adjustments to align with the v1 API. All 12 examples pass.
🤖 I have created a release *beep* *boop* --- ## [1.0.0](v1.0.0-alpha.3...v1.0.0) (2026-03-17) ### Features * implement the new rest error handling ([#282](#282)) ([a3bda30](a3bda30)) * use v2 suffix for module ID and provide compat support ([#270](#270)) ([dd1b6ba](dd1b6ba)), closes [#250](#250) ### Bug Fixes * a2asrv jsonrpc Content-Type ([#265](#265)) ([2568a46](2568a46)) * bugs before going from alpha ([#279](#279)) ([b1f055c](b1f055c)) * GetTaskRequest nil pointer assignment check ([#258](#258)) ([440bb79](440bb79)) * inject headers into service params ([#277](#277)) ([d33f3bd](d33f3bd)), closes [#275](#275) * propagate cancelation signal using task store ([#272](#272)) ([5e1d462](5e1d462)), closes [#245](#245) * regenerate spec and fix returnImmediately ([#284](#284)) ([2eee0b9](2eee0b9)) * task modified after save ([#266](#266)) ([c15febe](c15febe)) * taskupdater result mutable ([#274](#274)) ([6038d92](6038d92)) * update pushsender ([#256](#256)) ([5f7a594](5f7a594)) * use enum values as in the spec ([#261](#261)) ([eb98981](eb98981)), closes [#251](#251) ### Documentation * **a2asrv:** add Example_* test functions for pkg.go.dev documentation ([#262](#262)) ([7888e37](7888e37)) * add example tests a2a ([#240](#240)) ([4fe08a9](4fe08a9)) --- This PR was generated with [Release Please](https://github.com/googleapis/release-please). See [documentation](https://github.com/googleapis/release-please#release-please).
Description
Ref #209 🦕
Motivation
The
a2a/core types package does not include testable examples (Example_*functions). Usage snippets exist in the README anddoc.go, but they are not executable, not validated bygo test, and do not render as function-level examples on pkg.go.dev.Changes
Add
a2a/example_test.gousingpackage a2a_test(external test package), following Go'sExampleXxx/ExampleType_Methodnaming convention.Examples added for the following public API surface:
NewMessageNewMessageForTaskNewSubmittedTaskTaskState.Terminal()UnmarshalEventJSONUnmarshalEventJSON(Message)NewError/Error.WithDetailsNewStatusUpdateEventNewArtifactEventMessage.MarshalJSON/Task.MarshalJSONkinddiscriminator in JSON outputAll examples include
// Output:comments and are validated bygo test.Tests
The examples themselves are tests — validated through output matching. All 12 pass. Existing tests are unaffected.
go test ./a2a/ -v -run ExampleAdditional context
Scoped to the
a2a/core package to keep the review focused. If this aligns with the project's direction, I'm planning follow-up PRs fora2aclient/anda2asrv/.Let me know if you'd like any changes — happy to iterate.