Skip to content

feat(chronicle): bundle backstory UDM protos into @google-cloud/chronicle - #9189

Draft
quirogas wants to merge 2 commits into
googleapis:mainfrom
quirogas:chore-bundle-backstory-chronicle
Draft

feat(chronicle): bundle backstory UDM protos into @google-cloud/chronicle#9189
quirogas wants to merge 2 commits into
googleapis:mainfrom
quirogas:chore-bundle-backstory-chronicle

Conversation

@quirogas

@quirogas quirogas commented Aug 21, 2026

Copy link
Copy Markdown
Contributor

Bundle backstory Chronicle UDM protobuf definitions directly into the @google-cloud/chronicle client library package.

For b/512097657

@quirogas
quirogas requested a review from a team as a code owner August 21, 2026 19:08
@github-actions
github-actions Bot requested a review from shivanee-p August 21, 2026 19:08

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

Copy link
Copy Markdown
Contributor

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 adds several backstory protobuf definitions—including collection, data_access, entity, entity_risk, and id—and registers them in librarian.yaml and various service proto list JSON files. However, the referenced udm.proto file is missing from the pull request, which will lead to build and generation failures.

Comment thread librarian.yaml
- backstory/entity.proto
- backstory/entity_risk.proto
- backstory/id.proto
- backstory/udm.proto

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

critical

The file packages/google-cloud-chronicle/protos/backstory/udm.proto is listed here as an additional proto and is referenced in the service proto lists, but it has not been added to this pull request. Please make sure to add the udm.proto file to the repository to prevent build and generation failures.

@quirogas
quirogas marked this pull request as draft August 21, 2026 19:31
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.

1 participant