Skip to content

feat(proto): add enums for mode fields in SummarizeArticle and GetPopulationExposure#816

Open
NewCoder3294 wants to merge 1 commit intokoala73:mainfrom
NewCoder3294:feat/proto-enum-mode-fields
Open

feat(proto): add enums for mode fields in SummarizeArticle and GetPopulationExposure#816
NewCoder3294 wants to merge 1 commit intokoala73:mainfrom
NewCoder3294:feat/proto-enum-mode-fields

Conversation

@NewCoder3294
Copy link
Contributor

Summary

  • Add SummarizeMode enum (SUMMARIZE_MODE_UNSPECIFIED, SUMMARIZE_MODE_BRIEF, SUMMARIZE_MODE_ANALYSIS, SUMMARIZE_MODE_TRANSLATE) to summarize_article.proto and change SummarizeArticleRequest.mode from string to SummarizeMode
  • Add PopulationExposureMode enum (POPULATION_EXPOSURE_MODE_UNSPECIFIED, POPULATION_EXPOSURE_MODE_COUNTRIES, POPULATION_EXPOSURE_MODE_EXPOSURE) to get_population_exposure.proto and change GetPopulationExposureRequest.mode from string to PopulationExposureMode
  • Regenerate TypeScript client/server code and OpenAPI docs to reflect the new enum types
  • Update all server handlers and client call sites to use enum values instead of raw string comparisons

Closes #192

Test plan

  • Verify npx tsc --noEmit passes with zero errors
  • Verify summarization flow works end-to-end (brief, analysis, translate modes)
  • Verify population exposure endpoint works for both countries and exposure modes
  • Verify cache key generation still produces consistent keys with the new enum values

🤖 Generated with Claude Code

…tionExposureRequest.mode

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

vercel bot commented Mar 2, 2026

@NewCoder3294 is attempting to deploy a commit to the Elie Team on Vercel.

A member of the Team first needs to authorize it.

@koala73 koala73 added Not Ready to Merge PR has conflicts, failing checks, or needs work Low Value Trivial, unnecessary, or not aligned with project needs labels Mar 3, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Low Value Trivial, unnecessary, or not aligned with project needs Not Ready to Merge PR has conflicts, failing checks, or needs work

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Convert string-typed enum fields to proper proto enums

2 participants