chore(cli/update): adjust for feedback and format

Author: CorieW

genkit-tools/cli/src/commands/update.ts

@@ -22,8 +22,13 @@ import * as fs from 'fs';
 import inquirer from 'inquirer';
 import * as os from 'os';
 import * as path from 'path';
-import { PACKAGE_MANAGERS, getPackageManager, PackageManager, UpdateError } from '../utils/global';
+import {
+  PACKAGE_MANAGERS,
+  PackageManager,
+  getPackageManager,
+} from '../utils/package-manager';
 import { detectCLIRuntime } from '../utils/runtime-detector';
+import { UpdateError } from '../utils/utils';
 import { version as currentVersion, name } from '../utils/version';
 import { UPDATE_NOTIFICATIONS_OPT_OUT_CONFIG_TAG } from './config';
 
@@ -112,7 +117,9 @@ async function getGCSLatestData(): Promise<GCSLatestResponse> {
   );
 
   if (response.status !== 200) {
-    throw new UpdateError(`Failed to fetch GCS latest.json: ${response.statusText}`);
+    throw new UpdateError(
+      `Failed to fetch GCS latest.json: ${response.statusText}`
+    );
   }
 
   return response.data as GCSLatestResponse;
@@ -128,7 +135,9 @@ export async function getAvailableVersionsFromNpm(
     const response = await axios.get(`https://registry.npmjs.org/${name}`);
 
     if (response.status !== 200) {
-      throw new UpdateError(`Failed to fetch npm versions: ${response.statusText}`);
+      throw new UpdateError(
+        `Failed to fetch npm versions: ${response.statusText}`
+      );
     }
 
     const data: NpmRegistryResponse = response.data;

genkit-tools/cli/src/utils/global.ts renamed to genkit-tools/cli/src/utils/package-manager.ts

@@ -59,10 +59,3 @@ export const PACKAGE_MANAGERS: Record<string, PackageManager> = {
 export const getPackageManager = (name: string) => {
   return PACKAGE_MANAGERS[name] || PACKAGE_MANAGERS.npm;
 };
-
-export class UpdateError extends Error {
-  constructor(message: string) {
-    super(message);
-    this.name = 'UpdateError';
-  }
-}

genkit-tools/cli/src/utils/utils.ts

@@ -0,0 +1,22 @@
+/**
+ * Copyright 2025 Google LLC
+ *
+ * Licensed under the Apache License, Version 2.0 (the "License");
+ * you may not use this file except in compliance with the License.
+ * You may obtain a copy of the License at
+ *
+ *     http://www.apache.org/licenses/LICENSE-2.0
+ *
+ * Unless required by applicable law or agreed to in writing, software
+ * distributed under the License is distributed on an "AS IS" BASIS,
+ * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
+ * See the License for the specific language governing permissions and
+ * limitations under the License.
+ */
+
+export class UpdateError extends Error {
+  constructor(message: string) {
+    super(message);
+    this.name = 'UpdateError';
+  }
+}

genkit-tools/cli/tests/commands/update_test.ts

@@ -87,16 +87,16 @@ jest.mock('@genkit-ai/tools-common/utils', () => ({
 }));
 
 // Import after mocking
+import { getUserSettings } from '@genkit-ai/tools-common/utils';
 import axios from 'axios';
+import { UPDATE_NOTIFICATIONS_OPT_OUT_CONFIG_TAG } from '../../src/commands/config';
 import {
   checkForUpdates,
   getAvailableVersionsFromNpm,
   getLatestVersionFromGCS,
   showUpdateNotification,
 } from '../../src/commands/update';
 import { detectCLIRuntime } from '../../src/utils/runtime-detector';
-import { UPDATE_NOTIFICATIONS_OPT_OUT_CONFIG_TAG } from '../../src/commands/config';
-import { getUserSettings } from '@genkit-ai/tools-common/utils';
 
 const mockedAxios = axios as jest.Mocked<typeof axios>;
 const mockedDetectCLIRuntime = detectCLIRuntime as jest.MockedFunction<