Skip to content

chore: initial Librarian migration#8453

Open
jskeet wants to merge 1 commit into
googleapis:mainfrom
jskeet:librarian-migrate
Open

chore: initial Librarian migration#8453
jskeet wants to merge 1 commit into
googleapis:mainfrom
jskeet:librarian-migrate

Conversation

@jskeet
Copy link
Copy Markdown
Contributor

@jskeet jskeet commented Jun 8, 2026

Adds librarian.yaml to the root of the repository, with all libraries skipped initially. Also updates the release-please configuration to use the new node-librarian strategy, which updates the version numbers in librarian.yaml.

Adds librarian.yaml to the root of the repository, with all libraries
skipped initially. Also updates the release-please configuration to
use the new node-librarian strategy, which updates the version numbers
in librarian.yaml.
@jskeet jskeet requested review from pearigee and quirogas June 8, 2026 08:26
@jskeet jskeet requested a review from a team as a code owner June 8, 2026 08:26
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 introduces a new librarian.yaml configuration file defining various Google Cloud Node.js libraries and updates the release type in release-please-config.json to node-librarian. The review feedback points out several libraries in librarian.yaml that are missing their explicit npm package_name under the nodejs block, providing code suggestions to add the correct package names.

Comment thread librarian.yaml
Comment on lines +172 to +175
nodejs:
nodejs_apis:
- omit_common_resources: true
path: google/cloud/advisorynotifications/v1
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 npm package name for google-cloud-advisorynotifications is @google-cloud/advisory-notifications. Please specify the correct package_name under the nodejs block.

    nodejs:
      package_name: '@google-cloud/advisory-notifications'
      nodejs_apis:
        - omit_common_resources: true
          path: google/cloud/advisorynotifications/v1

Comment thread librarian.yaml
apis:
- path: google/cloud/rapidmigrationassessment/v1
copyright_year: "2026"
skip_generate: true
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 npm package name for google-cloud-rapidmigrationassessment is @google-cloud/rapid-migration-assessment. Please specify the correct package_name under the nodejs block.

    skip_generate: true
    nodejs:
      package_name: '@google-cloud/rapid-migration-assessment'

Comment thread librarian.yaml
apis:
- path: google/cloud/securesourcemanager/v1
copyright_year: "2026"
skip_generate: true
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 npm package name for google-cloud-securesourcemanager is @google-cloud/secure-source-manager. Please specify the correct package_name under the nodejs block.

    skip_generate: true
    nodejs:
      package_name: '@google-cloud/secure-source-manager'

Comment thread librarian.yaml
apis:
- path: google/cloud/securitycentermanagement/v1
copyright_year: "2026"
skip_generate: true
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 npm package name for google-cloud-securitycentermanagement is @google-cloud/security-center-management. Please specify the correct package_name under the nodejs block.

    skip_generate: true
    nodejs:
      package_name: '@google-cloud/security-center-management'

Comment thread librarian.yaml
apis:
- path: google/cloud/servicehealth/v1
copyright_year: "2026"
skip_generate: true
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 npm package name for google-cloud-servicehealth is @google-cloud/service-health. Please specify the correct package_name under the nodejs block.

    skip_generate: true
    nodejs:
      package_name: '@google-cloud/service-health'

Comment thread librarian.yaml
- path: google/cloud/telcoautomation/v1
- path: google/cloud/telcoautomation/v1alpha1
copyright_year: "2026"
skip_generate: true
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 npm package name for google-cloud-telcoautomation is @google-cloud/telco-automation. Please specify the correct package_name under the nodejs block.

    skip_generate: true
    nodejs:
      package_name: '@google-cloud/telco-automation'

Comment thread librarian.yaml
- path: google/cloud/visionai/v1
- path: google/cloud/visionai/v1alpha1
copyright_year: "2026"
skip_generate: true
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 npm package name for google-cloud-visionai is @google-cloud/vision-ai. Please specify the correct package_name under the nodejs block.

    skip_generate: true
    nodejs:
      package_name: '@google-cloud/vision-ai'

Comment thread librarian.yaml
apis:
- path: google/cloud/vmmigration/v1
copyright_year: "2026"
skip_generate: true
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 npm package name for google-cloud-vmmigration is @google-cloud/vm-migration. Please specify the correct package_name under the nodejs block.

    skip_generate: true
    nodejs:
      package_name: '@google-cloud/vm-migration'

Comment thread librarian.yaml
apis:
- path: google/cloud/vmwareengine/v1
copyright_year: "2026"
skip_generate: true
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 npm package name for google-cloud-vmwareengine is @google-cloud/vmware-engine. Please specify the correct package_name under the nodejs block.

    skip_generate: true
    nodejs:
      package_name: '@google-cloud/vmware-engine'

Comment thread librarian.yaml
apis:
- path: google/cloud/workloadmanager/v1
copyright_year: "2026"
skip_generate: true
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 npm package name for google-cloud-workloadmanager is @google-cloud/workload-manager. Please specify the correct package_name under the nodejs block.

    skip_generate: true
    nodejs:
      package_name: '@google-cloud/workload-manager'

@jskeet
Copy link
Copy Markdown
Contributor Author

jskeet commented Jun 8, 2026

@quirogas This was just the default migrate tool (tweaked a tiny bit to skip generation) - do you have any thoughts around the Gemini comments? (We can tweak these manually rather than changing the migrate tool.)

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