Skip to content

chore: migrate google-cloud-speech to librarian#8635

Open
jskeet wants to merge 1 commit into
googleapis:mainfrom
jskeet:scripted-migrate-google-cloud-speech
Open

chore: migrate google-cloud-speech to librarian#8635
jskeet wants to merge 1 commit into
googleapis:mainfrom
jskeet:scripted-migrate-google-cloud-speech

Conversation

@jskeet

@jskeet jskeet commented Jun 12, 2026

Copy link
Copy Markdown
Contributor

No description provided.

@jskeet jskeet requested a review from a team as a code owner June 12, 2026 17:52

@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 transitions the library to a fully auto-generated client by removing the custom "streamingRecognize" helper, its associated tests, and updating the metadata and entry point. The reviewer notes that manual edits to generated TypeScript client files should be avoided, and any custom helpers should be restored upstream in the generator.

Comment on lines +15 to -45
// ** This file is automatically generated by synthtool. **
// ** https://github.com/googleapis/synthtool **
// ** All changes to this file may be overwritten. **

import {ImprovedStreamingClient} from './helpers';
import * as v1p1beta1 from './v1p1beta1';
import * as v1 from './v1';
import * as v1p1beta1 from './v1p1beta1';
import * as v2 from './v2';

// The following code is adapted from http://www.typescriptlang.org/docs/handbook/mixins.html
// tslint:disable-next-line no-any
Object.defineProperty(
v1.SpeechClient.prototype,
'streamingRecognize',
Object.getOwnPropertyDescriptor(
ImprovedStreamingClient.prototype,
'streamingRecognize'
)!
);
Object.defineProperty(
v1p1beta1.SpeechClient.prototype,
'streamingRecognize',
Object.getOwnPropertyDescriptor(
ImprovedStreamingClient.prototype,
'streamingRecognize'
)!
);
Object.defineProperty(
v2.SpeechClient.prototype,
'streamingRecognize',
Object.getOwnPropertyDescriptor(
ImprovedStreamingClient.prototype,
'streamingRecognize'
)!
);

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.

high

Do not manually edit generated TypeScript client files. Any changes or updates must be made upstream in the generator. If the custom streamingRecognize helper (previously in helpers.ts) needs to be restored, please ensure this is handled upstream in the generator rather than manually editing the files in this repository.

References
  1. Do not manually edit generated TypeScript client files. Any changes or updates must be made upstream in the generator.

@shivanee-p shivanee-p requested review from shivanee-p and removed request for shivanee-p June 12, 2026 21:56
@shivanee-p shivanee-p marked this pull request as draft June 12, 2026 21:57
@shivanee-p shivanee-p self-requested a review June 12, 2026 21:57
@shivanee-p shivanee-p marked this pull request as ready for review June 12, 2026 21:57
@shivanee-p

Copy link
Copy Markdown
Contributor

@pearigee and I were debugging the test failures, and we found that this code is being post-processed in by librarian.js.

Additionally, the imports in the index file need to be updated to account for the deleted files.

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