refactor(proto): consolidate SummarizeArticleResponse status fields (#813)

Browse Source

* refactor(proto): consolidate SummarizeArticleResponse status fields

Replace redundant boolean/string status fields (cached, skipped, error,
error_type, reason) with a SummarizeStatus enum and a single
status_detail string. This addresses L-8 lint issue by reducing field
count and making the response status unambiguous.

- Add SummarizeStatus enum (UNSPECIFIED, SUCCESS, CACHED, SKIPPED, ERROR)
- Replace cached/skipped booleans with status enum field
- Merge error/error_type/reason into statusDetail string
- Reserve old field numbers (4, 7, 8, 9, 10) for wire compatibility
- Update server handlers and client code to use new fields
- Regenerate TypeScript types and OpenAPI docs

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>

* fix(proto): restore error and errorType fields for programmatic error handling

Keep the new SummarizeStatus enum and statusDetail for consolidated
status tracking, but restore the separate error and errorType fields
(proto field numbers 9, 10) to preserve structured error information
for downstream consumers that need to programmatically handle errors.

---------

Co-authored-by: Claude Opus 4.6 <noreply@anthropic.com>
Co-authored-by: Elie Habib <elie.habib@gmail.com>

...

This commit is contained in:

Nicolas Gomes Ferreira Dos Santos

2026-03-02 13:15:44 -08:00

committed by GitHub

parent a88a0c8e59

commit ee27b91c8f

8 changed files with 63 additions and 50 deletions

Show all changes Ignore whitespace when comparing lines Ignore changes in amount of whitespace Ignore changes in whitespace at EOL

Download Patch File Download Diff File Expand all files Collapse all files

2

docs/api/NewsService.openapi.json

File diff suppressed because one or more lines are too long