Feature implementation from commits 89b41b5..494583f

[codeOwlAI]

PR Summary

Schema Refinement Feature and Schema Handling Improvements

Overview

This PR adds schema refinement capabilities, improves schema name handling for duplicates, and fixes issues with falsy default values and string trimming.

Change Types

Type	Description
Enhancement	Added schema refinement feature to modify schemas before Zod conversion
Bug Fix	Fixed handling of falsy default values in schema processing
Bug Fix	Improved string processing using trim() instead of newline replacement
Enhancement	Added support for handling multiple schemas mapped to the same result string
Refactor	Formatting improvements for better code readability

---

Affected Modules

Module / File	Change Description
src/openApiToZod.ts	Added schema refinement capability and fixed falsy default value handling
src/template-context.ts	Added schemaRefiner option type and schema name normalization
src/getZodiosEndpointDefinitionList.ts	Improved string processing and schema name lookup behavior
tests/*	Added/updated tests for new functionality and fixes

---

Notes for Reviewers

• The schema refinement feature allows modifying schemas before they're converted to Zod
• Pay attention to the fix for falsy default values which now uses explicit undefined checks
• Schema name normalization now handles dot notation by splitting and using the first part

[codeowlai]

🐛 Correctness Issue

Property name mismatch with type definition.

The code initializes ctx.schemasByName but the type definition uses schemaByName (without 's'), which will cause runtime errors when accessing this property.

Current Code (Diff):

-        ctx.schemasByName = {};
+        ctx.schemaByName = {};

📝 Committable suggestion

‼️ IMPORTANT
Trust, but verify! 🕵️ Please review this suggestion with the care of a code archaeologist - check that it perfectly replaces the highlighted code, preserves all lines, maintains proper indentation, and won't break anything in production. Your future self will thank you! 🚀

Suggested change

	ctx.schemasByName = {};
	ctx.schemaByName = {};

🔄 Dependencies Affected

lib/src/template-context.ts

Function: any function using getZodiosEndpointDefinitionList result

Issue: Code expecting schemaByName property but receiving schemasByName

Suggestion: Update any code using ctx.schemasByName to use ctx.schemaByName instead

---

[codeowlai]

🐛 Correctness Issue

Non-null Assertion on Potentially Undefined Value.

The non-null assertion operator on normalizedSchemaName could cause runtime errors if the split pattern produces unexpected results.

Current Code (Diff):

-                 dependencies.add(normalizedSchemaName!);
+                 dependencies.add(normalizedSchemaName || schemaName);

📝 Committable suggestion

‼️ IMPORTANT
Trust, but verify! 🕵️ Please review this suggestion with the care of a code archaeologist - check that it perfectly replaces the highlighted code, preserves all lines, maintains proper indentation, and won't break anything in production. Your future self will thank you! 🚀

Suggested change

	dependencies.add(normalizedSchemaName!);
	dependencies.add(normalizedSchemaName || schemaName);