tool-registry-publish: type fetchImpl test stub without an unsafe cast (CL-7096)

Author: TheGreatAxios

packages/tool-registry-publish/src/publish.test.ts

@@ -9,6 +9,7 @@ import {
   shouldPublishTarball,
   sha512Integrity,
   type ApiCall,
+  type FetchTarballPut,
 } from "./publish";
 import { tarballsCoverRequiredSeedPackages } from "./registry";
 
@@ -143,7 +144,7 @@ describe("publishCorbitsToolsRegistry", () => {
               status: 200,
               headers: { "Content-Type": "application/json" },
             },
-          )) as typeof fetch,
+          )) satisfies FetchTarballPut,
       });
       throw new Error("expected EmptyRegistryPublishError");
     } catch (cause) {

packages/tool-registry-publish/src/publish.ts

@@ -275,13 +275,18 @@ export async function isCorbitsToolsRegistrySeeded(
   );
 }
 
+export type FetchTarballPut = (
+  input: string,
+  init: RequestInit,
+) => Promise<Response>;
+
 async function putTarball(
   hubUrl: string,
   cookies: string[],
   tenantId: string,
   assetId: string,
   tarball: PackedTarball,
-  fetchImpl: typeof fetch,
+  fetchImpl: FetchTarballPut,
 ): Promise<PublishSummary> {
   const headers: Record<string, string> = {
     "Content-Type": "application/octet-stream",
@@ -315,7 +320,7 @@ export type PublishCorbitsToolsRegistryArgs = {
   cookies: string[];
   hubUrl: string;
   tenantId: string;
-  fetchImpl?: typeof fetch;
+  fetchImpl?: FetchTarballPut;
   log?: (line: string) => void;
   /** Test seam. Production omits this and runs `checkToolPackageFreshness`. */
   checkFreshness?: () => Promise<void>;