a2aproject / a2aproject/A2A

[Bug]: ListTaskPushNotificationConfig paging is not coherent with ListTasks paging

Offen Anfängerfreundlich
#1,420 0 Kommentare 0 Reaktionen 0 zugewiesene Personen Auf GitHub ansehen
Vorherrschende Sprache
Shell
Sterne
25.7k
Forks
2.6k
Ø Merge
3 T. 6 Std.
Gemergte PRs (30 T.)
16

Beschreibung

### What happened?

In ListTasksRequest, the page size is defined as:
```
// Maximum number of tasks to return. Must be between 1 and 100.
// Defaults to 50 if not specified.
optional int32 page_size = 3;
```
while in ListTaskPushNotificationConfigRequest:

```
// The maximum number of configurations to return.
int32 page_size = 2;
```

I think both should be handled in the same way

### Relevant log output

```shell

```

### Code of Conduct

- [x] I agree to follow this project's Code of Conduct

Beitragsleitfaden

Beitragsleitfaden öffnen

Rechercherichtung

Find the protobuf definitions for ListTasksRequest and ListTaskPushNotificationConfigRequest. Compare the page_size field handling (optional vs. required, default values). Ensure both are consistent, likely making the second optional with a default. Look for existing tests around pagination to verify the change.

Vom Indexierungsmodell aus dem Issue-Text verfasst.

Bewertung

Bereich
api, backend
Issue-Typ
Bug
Schwierigkeit
2/5
Geschätzter Aufwand
1-3 Stunden
Aktivitätsstatus
Veraltet
Klarheit
Klar beschrieben
Anfängerfreundlichkeit
65/100

Neue Issues direkt in Ihr Postfach

Eine kurze Übersicht über anfängerfreundliche GitHub-Issues.