Skip to content

chore: librarian generate pull request: 20260408T070849Z#16576

Merged
ohmayr merged 1 commit intomainfrom
librarian-20260408T070849Z
Apr 8, 2026
Merged

chore: librarian generate pull request: 20260408T070849Z#16576
ohmayr merged 1 commit intomainfrom
librarian-20260408T070849Z

Conversation

@cloud-sdk-librarian-robot
Copy link
Copy Markdown
Collaborator

PR created by the Librarian CLI to generate Cloud Client Libraries code from protos.

BEGIN_COMMIT

BEGIN_NESTED_COMMIT
docs: Fix documentation URL AIInference MessageTransform service_account_email field

PiperOrigin-RevId: 895485801
Library-IDs: google-cloud-pubsub
Source-link: googleapis/googleapis@afb6d4b3
END_NESTED_COMMIT

BEGIN_NESTED_COMMIT
feat: A new field mime_type is added to message .google.cloud.translation.v3.AdaptiveMtTranslateRequest

PiperOrigin-RevId: 895422613
Library-IDs: google-cloud-translate
Source-link: googleapis/googleapis@d6837a5d
END_NESTED_COMMIT

BEGIN_NESTED_COMMIT
feat: populate the persisted_data_checksums field with object checksums on write object / query write status responses

PiperOrigin-RevId: 895357062
Library-IDs: google-cloud-storage
Source-link: googleapis/googleapis@d73dfb3c
END_NESTED_COMMIT

BEGIN_NESTED_COMMIT
docs: improve wording around object_checksums in bidi write object requests

PiperOrigin-RevId: 895357062
Library-IDs: google-cloud-storage
Source-link: googleapis/googleapis@d73dfb3c
END_NESTED_COMMIT

BEGIN_NESTED_COMMIT
feat: added support for detecting key-value pairs in client provided metadata

PiperOrigin-RevId: 895098649
Library-IDs: google-cloud-dlp
Source-link: googleapis/googleapis@544bab0c
END_NESTED_COMMIT

BEGIN_NESTED_COMMIT
feat: Support app authentication with admin-consent scopes for Chat API ListMessages, GetMessage, GetSpaceEvent and ListSpaceEvents

PiperOrigin-RevId: 893619899
Library-IDs: google-apps-chat
Source-link: googleapis/googleapis@8d52a0bd
END_NESTED_COMMIT

BEGIN_NESTED_COMMIT
docs: Update reference documentation for Chat API ListMessages, GetMessage, GetSpaceEvent and ListSpaceEvents.

PiperOrigin-RevId: 893619899
Library-IDs: google-apps-chat
Source-link: googleapis/googleapis@8d52a0bd
END_NESTED_COMMIT

BEGIN_NESTED_COMMIT
feat: Addition of Section and SectionItem APIs

PiperOrigin-RevId: 893488928
Library-IDs: google-apps-chat
Source-link: googleapis/googleapis@76fdbf75
END_NESTED_COMMIT

END_COMMIT

This pull request is generated with proto changes between
googleapis/googleapis@582172de
(exclusive) and
googleapis/googleapis@afb6d4b3
(inclusive).

Librarian Version: v0.0.0-20260407125916-07fdb844a681
Language Image: us-central1-docker.pkg.dev/cloud-sdk-librarian-prod/images-prod/python-librarian-generator@sha256:234b9d1f2ddb057ed7ac6a38db0bf8163d839c65c6cf88ade52530cddebce59e

Generation failed for

  • google-cloud-spanner

@cloud-sdk-librarian-robot
Copy link
Copy Markdown
Collaborator Author

One or more libraries have failed to generate, please review PR description for a list of failed libraries.
For each failed library, open a ticket in that library’s repository and then you may resolve this comment and merge.

@snippet-bot
Copy link
Copy Markdown

snippet-bot Bot commented Apr 8, 2026

Here is the summary of changes.

You are about to add 15 region tags.

This comment is generated by snippet-bot.
If you find problems with this result, please file an issue at:
https://github.com/googleapis/repo-automation-bots/issues.
To update this comment, add snippet-bot:force-run label or use the checkbox below:

  • Refresh this comment

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 updates several Google Cloud client libraries, most notably introducing "Sections" management functionality to the Google Apps Chat library and adding shared regional certificate authority support to Memorystore. Other updates include new checksum fields in Storage and mime_type support in Translation. Feedback highlights a potential code generation error in the Translation library where the mime_type field was incorrectly added to GlossaryConfig instead of AdaptiveMtTranslateRequest. Additionally, a docstring inconsistency was identified in the Chat library regarding the retention of a "Developer Preview" notice that should be removed to match other updates in the PR.

Comment on lines +387 to +390
mime_type: str = proto.Field(
proto.STRING,
number=4,
)
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 mime_type field is being added to GlossaryConfig. However, the commit message for this change ("feat: A new field mime_type is added to message .google.cloud.translation.v3.AdaptiveMtTranslateRequest") and the corresponding proto change in googleapis/googleapis@d6837a5d indicate that this field should be part of AdaptiveMtTranslateRequest directly, not GlossaryConfig. This appears to be a code generation issue.

Comment on lines +190 to +191
r"""Request message for deleting a section. `Developer
Preview <https://developers.google.com/workspace/preview>`__.
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.

medium

The docstring for DeleteSectionRequest includes a "Developer Preview" notice. This seems inconsistent with other changes in this PR that remove similar notices from other docstrings. For consistency, this should be removed if the feature is generally available.

r"""Request message for deleting a section."""

@ohmayr ohmayr merged commit 56ccbd8 into main Apr 8, 2026
30 of 31 checks passed
@ohmayr ohmayr deleted the librarian-20260408T070849Z branch April 8, 2026 17:57
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