fix: review updates

Author: cdcabrera

src/server.schema.ts

@@ -72,7 +72,8 @@ const isZodRawShape = (value: unknown): boolean => {
  * @param jsonSchema - Plain JSON Schema object
  * @param settings - Optional settings
  * @param settings.failFast - Fail fast on unsupported types, or be nice and attempt to convert. Defaults to true.
- * @returns Zod schema equivalent
+ * @returns A Zod schema when convertible, returns undefined when failFast and input are unsupported, or falls back
+ *     to z.any() when failFast `false`
  */
 const jsonSchemaToZod = (
   jsonSchema: unknown,

src/server.toolsHost.ts

@@ -83,7 +83,7 @@ const serializeError = (errorValue: unknown) => {
 /**
  * Result of `normalizeCreatorSchema`.
  *
- * @property tool - The normalized tool creator function.
+ * @property tool - The realized tool tuple returned by the creator function.
  * @property normalizedSchema - Normalized input schema.
  * @property manifestSchema - JSON Schema representation of the normalized input schema.
  * @property warnings - List of warnings generated during normalization.

src/server.toolsHostCreator.ts

@@ -48,26 +48,26 @@ type ResolveOptions = {
  *
  * - Probes function exports at most once with toolOptions and never re-probes without options.
  * - Supported export shapes:
- *   - A `default export` that is a function returning a tool tuple -> wraps and caches as a creator (with .toolName)
- *   - A `default export` that is a function returning an array of functions returning tool tuples -> no unwrapping, returns them directly
- *   - A `default export` that is an array of functions returning tool tuples -> no unwrapping, returns them directly
+ *   - A `default export` that is a tool creator (function returning a tool tuple) -> wraps and caches as a creator (with .toolName)
+ *   - A `default export` that is a function returning an array of tool creators (functions returning tool tuples) -> no unwrapping, returns them directly
+ *   - A `default export` that is an array of tool creators (functions returning tool tuples) -> no unwrapping, returns them directly
  *
  * @example
- * // A default export function returning a tool tuple
+ * // A default export creator (function returning a tool tuple)
  * export default () => ['toolName', { description: 'recommended', inputSchema: { ... } }, handler];
  *
  * @example
- * // A default export function returning an array of tool tuples
- * const dolorSit = () => ['toolName1', { description: 'recommended', inputSchema: { ... }  }, handler];
- * const ametConsectetur = () => ['toolName2', { description: 'recommended', inputSchema: { ... }  }, handler];
+ * // A default export function returning an array of tool creators (functions returning tool tuples)
+ * const dolorSit = () => ['toolName1', { description: 'recommended', inputSchema: { ... } }, handler];
+ * const ametConsectetur = () => ['toolName2', { description: 'recommended', inputSchema: { ... } }, handler];
  *
  * export default () => [dolorSit, ametConsectetur];
  *
  * @example
- * // A default export array of functions returning tool tuples
+ * // A default export array of tool creators (functions returning tool tuples)
  * export default [
- *   () => ['toolName1', { description: 'recommended', inputSchema: { ... }  }, handler],
- *   () => ['toolName2', { description: 'recommended', inputSchema: { ... }  }, handler]
+ *   () => ['toolName1', { description: 'recommended', inputSchema: { ... } }, handler],
+ *   () => ['toolName2', { description: 'recommended', inputSchema: { ... } }, handler]
  * ];
  *
  * @param moduleExports - The module exports object from the child process.