Deduplicate removeUndefined helper

Author: cameroncooke

src/mcp/tools/session-management/session_set_defaults.ts

@@ -1,5 +1,6 @@
 import * as z from 'zod';
 import { persistSessionDefaultsPatch } from '../../../utils/config-store.ts';
+import { removeUndefined } from '../../../utils/remove-undefined.ts';
 import { sessionStore, type SessionDefaults } from '../../../utils/session-store.ts';
 import { sessionDefaultsSchema } from '../../../utils/session-defaults-schema.ts';
 import { createTypedToolWithContext } from '../../../utils/typed-tool-factory.ts';
@@ -16,16 +17,6 @@ type Params = z.infer<typeof schemaObj>;
 
 type SessionSetDefaultsContext = Record<string, never>;
 
-function removeUndefined<T extends Record<string, unknown>>(input: T): Partial<T> {
-  const result: Partial<T> = {};
-  for (const [key, value] of Object.entries(input)) {
-    if (value !== undefined) {
-      result[key as keyof T] = value as T[keyof T];
-    }
-  }
-  return result;
-}
-
 export async function sessionSetDefaultsLogic(
   params: Params,
   neverContext: SessionSetDefaultsContext,

src/utils/project-config.ts

@@ -3,6 +3,7 @@ import { parse as parseYaml, stringify as stringifyYaml } from 'yaml';
 import type { FileSystemExecutor } from './FileSystemExecutor.ts';
 import type { SessionDefaults } from './session-store.ts';
 import { log } from './logger.ts';
+import { removeUndefined } from './remove-undefined.ts';
 import { runtimeConfigFileSchema, type RuntimeConfigFile } from './runtime-config-schema.ts';
 
 const CONFIG_DIR = '.xcodebuildmcp';
@@ -49,16 +50,6 @@ function toError(value: unknown): Error {
   return value instanceof Error ? value : new Error(String(value));
 }
 
-function removeUndefined<T extends Record<string, unknown>>(input: T): Partial<T> {
-  const result: Partial<T> = {};
-  for (const [key, value] of Object.entries(input)) {
-    if (value !== undefined) {
-      result[key as keyof T] = value as T[keyof T];
-    }
-  }
-  return result;
-}
-
 function hasValue<T extends Record<string, unknown>>(defaults: T, key: keyof T): boolean {
   return Object.prototype.hasOwnProperty.call(defaults, key) && defaults[key] !== undefined;
 }