Skip to content

feat(agentidentity): onboard v1 and v1beta clients#17740

Merged
noahdietz merged 1 commit into
googleapis:mainfrom
noahdietz:onboard-agentidentity
Jul 16, 2026
Merged

feat(agentidentity): onboard v1 and v1beta clients#17740
noahdietz merged 1 commit into
googleapis:mainfrom
noahdietz:onboard-agentidentity

Conversation

@noahdietz

@noahdietz noahdietz commented Jul 16, 2026

Copy link
Copy Markdown
Contributor

Onboard the v1 and v1beta API versions of the Agent Identity API.

Internal intake bugs http://b/534344011 http://b/533456597

@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 introduces the google-cloud-agentidentity client library, including support for both v1 and v1beta versions. The review identified several issues: the copyright year in librarian.yaml is set to a future date (2026), the API enablement links in the README files should point to the direct console URL, the documentation index has a duplicated heading, and the example workload identity in the type definitions refers to an incorrect service (aiplatform).

Comment thread librarian.yaml
apis:
- path: google/cloud/agentidentity/v1
- path: google/cloud/agentidentity/v1beta
copyright_year: "2026"

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 copyright year is set to 2026, which is in the future. This appears to be a typo and should be updated to the current year.


.. _Select or create a Cloud Platform project.: https://console.cloud.google.com/project
.. _Enable billing for your project.: https://cloud.google.com/billing/docs/how-to/modify-project#enable_billing_for_a_project
.. _Enable the Agent Identity.: https://cloud.google.com/agentidentity/

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 link to enable the API points to the general product documentation. For a better user experience, it should point directly to the API enablement page in the Cloud Console.

Suggested change
.. _Enable the Agent Identity.: https://cloud.google.com/agentidentity/
.. _Enable the Agent Identity.: https://console.cloud.google.com/flows/enableapi?apiid=agentidentity.googleapis.com


.. _Select or create a Cloud Platform project.: https://console.cloud.google.com/project
.. _Enable billing for your project.: https://cloud.google.com/billing/docs/how-to/modify-project#enable_billing_for_a_project
.. _Enable the Agent Identity.: https://cloud.google.com/agentidentity/

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 link to enable the API points to the general product documentation. For a better user experience, it should point directly to the API enablement page in the Cloud Console.

Suggested change
.. _Enable the Agent Identity.: https://cloud.google.com/agentidentity/
.. _Enable the Agent Identity.: https://console.cloud.google.com/flows/enableapi?apiid=agentidentity.googleapis.com

Comment on lines +17 to +18
API Reference
-------------

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 "API Reference" heading is duplicated. It would be clearer to have a single main heading and then use subheadings for each API version (e.g., v1 and v1beta).

Comment on lines +127 to +130
Optional. Input only. Represents the workload identity in
IAM ``principal://`` format of the agent(s) that will use
this AuthProvider. Example:
``principal://agents.global.org-${ORG_ID}.system.id.goog/resources/aiplatform/projects/{PROJECT_ID}/locations/{LOCATIONS}/reasoningEngines/{ID}``

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 example workload identity appears to be for a different service (aiplatform/reasoningEngines). To avoid confusion, please consider updating it to a more generic example or one specific to Agent Identity.

Comment on lines +127 to +130
Optional. Input only. Represents the workload identity in
IAM ``principal://`` format of the agent(s) that will use
this AuthProvider. Example:
``principal://agents.global.org-${ORG_ID}.system.id.goog/resources/aiplatform/projects/{PROJECT_ID}/locations/{LOCATIONS}/reasoningEngines/{ID}``

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 example workload identity appears to be for a different service (aiplatform/reasoningEngines). To avoid confusion, please consider updating it to a more generic example or one specific to Agent Identity.

@noahdietz
noahdietz marked this pull request as ready for review July 16, 2026 17:38
@noahdietz
noahdietz requested review from a team as code owners July 16, 2026 17:38
@snippet-bot

snippet-bot Bot commented Jul 16, 2026

Copy link
Copy Markdown

Here is the summary of changes.

You are about to add 64 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

@noahdietz noahdietz closed this Jul 16, 2026
@noahdietz noahdietz reopened this Jul 16, 2026
@noahdietz noahdietz added the do not merge Indicates a pull request not ready for merge, due to either quality or timing. label Jul 16, 2026
@noahdietz

Copy link
Copy Markdown
Contributor Author

Holding for the bulk release to go out first

@noahdietz noahdietz removed the do not merge Indicates a pull request not ready for merge, due to either quality or timing. label Jul 16, 2026
@noahdietz
noahdietz merged commit 9eead95 into googleapis:main Jul 16, 2026
70 checks passed
@noahdietz
noahdietz deleted the onboard-agentidentity branch July 16, 2026 20:40
@release-please release-please Bot mentioned this pull request Jul 16, 2026
ohmayr pushed a commit that referenced this pull request Jul 17, 2026
🤖 I have created a release *beep* *boop*
---


<details><summary>google-cloud-agentidentity: 0.1.0</summary>

## 0.1.0 (2026-07-16)


### Features

* **agentidentity:** onboard v1 and v1beta clients
([#17740](#17740))
([9eead95](9eead95))
</details>

---
This PR was generated with [Release
Please](https://github.com/googleapis/release-please). See
[documentation](https://github.com/googleapis/release-please#release-please).

Co-authored-by: release-please[bot] <55107282+release-please[bot]@users.noreply.github.com>
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