Skip to content

Clean up test output noise#57

Merged
jkebinger merged 3 commits intomainfrom
cleanup-test-warnings
Oct 9, 2025
Merged

Clean up test output noise#57
jkebinger merged 3 commits intomainfrom
cleanup-test-warnings

Conversation

@jkebinger
Copy link
Copy Markdown
Contributor

Summary

  • Handle ZodDefault types in code generation mappers
  • Remove debug console.log statements
  • Simplify test error handling

Changes

  • Add ZodDefaultDef to supported Zod types
  • Unwrap ZodDefault to inner type in ZodBaseMapper
  • Remove converting... debug logs from coerce.ts
  • Simplify test error handling to avoid duplicate output
  • Update test expectations for correct ZodDefault unwrapping

Result

  • Eliminates "Unknown zod type: ZodDefault" warnings
  • Eliminates "converting..." debug output
  • Cleaner test output overall

🤖 Generated with Claude Code

jkebinger and others added 3 commits October 9, 2025 14:28
- Add ZodDefaultDef to SupportedZodTypes
- Unwrap ZodDefault to inner type in ZodBaseMapper
- Eliminates "Unknown zod type: ZodDefault" warnings

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-Authored-By: Claude <noreply@anthropic.com>
- Remove debug console.log statements from coerce.ts
- Simplify test error handling to avoid duplicate JSON output
- All "Unknown zod type" and "converting..." messages eliminated

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-Authored-By: Claude <noreply@anthropic.com>
Copy link
Copy Markdown
Contributor

@jdwyah jdwyah left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

thanks

@jkebinger jkebinger enabled auto-merge (squash) October 9, 2025 19:36
@jkebinger jkebinger merged commit 1d2ddf5 into main Oct 9, 2025
6 checks passed
@jkebinger jkebinger deleted the cleanup-test-warnings branch October 9, 2025 19:37
@jkebinger jkebinger restored the cleanup-test-warnings branch October 9, 2025 19:44
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

2 participants