fixup! build: update all non-major dependencies

Author: ok7sai

pkg-externals.bzl

@@ -53,6 +53,7 @@ PKG_EXTERNALS = [
     "selenium-webdriver",
     "safevalues",
     "safevalues/dom",
+    "youtube",
 ]
 
 # Creates externals for a given package and its entry-points.

src/youtube-player/BUILD.bazel

@@ -79,6 +79,7 @@ ng_project(
         ":youtube-player",
         "//:node_modules/@angular/core",
         "//:node_modules/@angular/platform-browser",
+        "//:node_modules/@types/youtube",
         "//:node_modules/rxjs",
     ],
 )

src/youtube-player/fake-youtube-player.ts

@@ -6,6 +6,8 @@
  * found in the LICENSE file at https://angular.dev/license
  */
 
+import type * as YT from 'youtube';
+
 // A re-creation of YT.PlayerState since enum values cannot be bound to the window object.
 const playerState = {
   UNSTARTED: -1,

src/youtube-player/youtube-player.spec.ts

@@ -15,6 +15,7 @@ import {
   YouTubePlayer,
 } from './youtube-player';
 import {PlaceholderImageQuality} from './youtube-player-placeholder';
+import type * as YT from 'youtube';
 
 declare var window: Window;
 
@@ -200,7 +201,9 @@ describe('YoutubePlayer', () => {
       fixture.detectChanges();
       events.onReady({target: playerSpy});
 
-      const playerVars: YT.PlayerVars = {modestbranding: YT.ModestBranding.Modest};
+      const playerVars: YT.PlayerVars = {
+        modestbranding: (window as Window & {YT?: typeof YT}).YT!.ModestBranding.Modest,
+      };
       fixture.componentInstance.playerVars = playerVars;
       fixture.changeDetectorRef.markForCheck();
       fixture.detectChanges();
@@ -389,7 +392,9 @@ describe('YoutubePlayer', () => {
       fixture.detectChanges();
 
       testComponent.youtubePlayer.playVideo();
-      expect(testComponent.youtubePlayer.getPlayerState()).toBe(YT.PlayerState.PLAYING);
+      expect(testComponent.youtubePlayer.getPlayerState()).toBe(
+        (window as Window & {YT?: typeof YT}).YT!.PlayerState.PLAYING,
+      );
 
       events.onReady({target: playerSpy});
 
@@ -401,7 +406,9 @@ describe('YoutubePlayer', () => {
       fixture.detectChanges();
 
       testComponent.youtubePlayer.pauseVideo();
-      expect(testComponent.youtubePlayer.getPlayerState()).toBe(YT.PlayerState.PAUSED);
+      expect(testComponent.youtubePlayer.getPlayerState()).toBe(
+        (window as Window & {YT?: typeof YT}).YT!.PlayerState.PAUSED,
+      );
 
       events.onReady({target: playerSpy});
 
@@ -413,7 +420,9 @@ describe('YoutubePlayer', () => {
       fixture.detectChanges();
 
       testComponent.youtubePlayer.stopVideo();
-      expect(testComponent.youtubePlayer.getPlayerState()).toBe(YT.PlayerState.CUED);
+      expect(testComponent.youtubePlayer.getPlayerState()).toBe(
+        (window as Window & {YT?: typeof YT}).YT!.PlayerState.CUED,
+      );
 
       events.onReady({target: playerSpy});
 

src/youtube-player/youtube-player.ts

@@ -9,6 +9,8 @@
 // Workaround for: https://github.com/bazelbuild/rules_nodejs/issues/1265
 /// <reference types="youtube" preserve="true" />
 
+import type * as YT from 'youtube';
+
 import {
   ChangeDetectionStrategy,
   Component,
@@ -576,7 +578,10 @@ export class YouTubePlayer implements AfterViewInit, OnChanges, OnDestroy {
 
     // A player can't be created if the API isn't loaded,
     // or there isn't a video or playlist to be played.
-    if (typeof YT === 'undefined' || (!this.videoId && !this.playerVars?.list)) {
+    if (
+      typeof (window as Window & {YT?: typeof YT}).YT === 'undefined' ||
+      (!this.videoId && !this.playerVars?.list)
+    ) {
       return;
     }
 
@@ -597,7 +602,11 @@ export class YouTubePlayer implements AfterViewInit, OnChanges, OnDestroy {
       params.videoId = this.videoId;
     }
     const player = this._ngZone.runOutsideAngular(
-      () => new YT.Player(this.youtubeContainer.nativeElement, params),
+      () =>
+        new (window as Window & {YT?: typeof YT}).YT!.Player(
+          this.youtubeContainer.nativeElement,
+          params,
+        ),
     );
 
     const whenReady = (event: YT.PlayerEvent) => {