Skip to content

feat: List tasks method implementation#383

Merged
bartek-gralewicz merged 23 commits into
epic/1.0_breaking_changesfrom
bgralewicz/list_tasks_implementation
Apr 7, 2026
Merged

feat: List tasks method implementation#383
bartek-gralewicz merged 23 commits into
epic/1.0_breaking_changesfrom
bgralewicz/list_tasks_implementation

Conversation

@bartek-gralewicz
Copy link
Copy Markdown
Contributor

Description

Implementation of the listTasks method. Changes made accordingly to the current spec definition as well as the following PRs:

Fixes #172 🦕

gemini-code-assist[bot]

This comment was marked as resolved.

@bartek-gralewicz

This comment was marked as outdated.

gemini-code-assist[bot]

This comment was marked as resolved.

@bartek-gralewicz

This comment was marked as outdated.

Copy link
Copy Markdown
Contributor

@gemini-code-assist gemini-code-assist Bot left a comment

Choose a reason for hiding this comment

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

Code Review

This pull request implements the listTasks functionality across gRPC, JSON-RPC, and REST transports, adding a list method to the TaskStore interface and an implementation in InMemoryTaskStore that supports filtering and pagination. The review feedback identifies several logic and performance issues, including the incorrect handling of zero-valued status enums in filters, inefficient Date object creation during sorting and searching, and the risk of state corruption due to shallow copying of task objects.

Comment thread src/server/store.ts Outdated
Comment thread src/server/store.ts
Comment thread src/server/store.ts Outdated
Comment thread src/server/store.ts Outdated
Comment thread src/server/transports/rest/rest_transport_handler.ts Outdated
Comment thread src/server/transports/rest/rest_transport_handler.ts
Base automatically changed from bgralewicz/remove_obsolete_jsonrpc_types to epic/1.0_breaking_changes April 2, 2026 12:54
@github-actions
Copy link
Copy Markdown

github-actions Bot commented Apr 2, 2026

🧪 Code Coverage

⬇️ Download Full Report

Base PR Delta
src/server/express/rest_handler.ts 89.12% 89.47% 🟢 +0.35%
src/server/grpc/grpc_service.ts 64.62% 69.81% 🟢 +5.19%
src/server/request_handler/default_request_handler.ts 77.99% 78.17% 🟢 +0.18%
src/server/store.ts 100% 64.83% 🔴 -35.17%
src/server/transports/jsonrpc/jsonrpc_transport_handler.ts 77.13% 75.54% 🔴 -1.59%
src/server/transports/rest/rest_transport_handler.ts 98.67% 98.81% 🟢 +0.14%
Total 83.89% 83.7% 🔴 -0.19%

Generated by coverage-comment.yml

@bartek-gralewicz bartek-gralewicz marked this pull request as ready for review April 2, 2026 14:23
@bartek-gralewicz bartek-gralewicz requested a review from a team as a code owner April 2, 2026 14:23
@bartek-gralewicz bartek-gralewicz merged commit 7d4c472 into epic/1.0_breaking_changes Apr 7, 2026
9 checks passed
@bartek-gralewicz bartek-gralewicz deleted the bgralewicz/list_tasks_implementation branch April 7, 2026 09:16
ishymko added a commit that referenced this pull request May 11, 2026
🤖 I have created a release *beep* *boop*
---

## 1.0.0-alpha.0 (2026-05-11)

See the [v0.3 -> v1.0-alpha.0 migration
guide](https://github.com/a2aproject/a2a-js/blob/v1.0.0-alpha.0/docs/migration-guide.md).

**Note**: Enabling backward compatibility with v0.3 is tracked in
[#452](#452).


### ⚠ BREAKING CHANGES

* Drop support for node 18
([#368](#368))
* Make ServerCallContext parameter mandatory across all places
([#405](#405))
* Remove JSON-RPC client
([#353](#353))
* Remove transport-specific exports
([#404](#404))
* Update codebase to use A2A 1.0.0 data model
([#375](#375))
* Remove A2AExpressApp
([#363](#363))

### Features

* Add A2A Version Header
([#422](#422))
([b5f3db7](b5f3db7))
* Add cache-headers logic to the agent card handler
([#435](#435))
([955b52b](955b52b))
* Add resource scoping
([#450](#450))
([c527086](c527086))
* Add support for custom authentication scheme and credentials in
auth-headers ([#430](#430))
([5a4389b](5a4389b))
* AgentCardSignature support
([#448](#448))
([4a41a8c](4a41a8c))
* Enforce events ordering
([#437](#437))
([157cf48](157cf48))
* Enriched Error Model
([#427](#427))
([c130778](c130778))
* Implement listTasks method
([#383](#383))
([7d4c472](7d4c472))
* Send current task as the first event after subscribing to it
([#418](#418))
([4bfcf5f](4bfcf5f))
* Support multi-tenancy
([#419](#419))
([1877877](1877877))

### Code Refactoring

* remove A2AExpressApp
([#363](#363))
([0b84728](0b84728))

---
This PR was generated with [Release
Please](https://github.com/googleapis/release-please). See
[documentation](https://github.com/googleapis/release-please#release-please).

---------

Co-authored-by: bartek-gralewicz <bgralewicz@google.com>
Co-authored-by: Ivan Shymko <ishymko@google.com>
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.

2 participants