Skip to content

AI update based on proto changes #606

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Closed
wants to merge 0 commits into from
Closed

AI update based on proto changes #606

wants to merge 0 commits into from

Conversation

viambot
Copy link
Member

@viambot viambot commented Aug 11, 2025

This is an AI-generated PR to update the SDK based on proto changes. The AI may make mistakes so review carefully.

Summary of Changes:
Here's a summary of the changes:

  • Optional organizationId for listMachineSummaries: The organizationId parameter is now optional in the listMachineSummaries method.
  • Cross-Organization Search: When organizationId is not provided, the method will query machine summaries across all organizations the caller has access to.
  • Implementation: The method signature, request construction (sending an empty string '' if organizationId is undefined), and JSDoc documentation have been updated accordingly.
  • Testing: Added a new unit test to verify the behavior when organizationId is omitted.

@viambot viambot closed this Aug 12, 2025
@viambot viambot force-pushed the workflow/ai-updater branch from 2854389 to 5c29f20 Compare August 12, 2025 15:17
@viambot viambot deleted the workflow/ai-updater branch August 12, 2025 15:17
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