Automatically create docs from Zod schema

[tudddorrr]

No description provided.

[claude]

Code Review - PR #802

This PR successfully automates documentation generation from Zod schemas, removing manual params definitions. The core implementation is solid with good test coverage. Here are specific improvements needed:

---

🔴 Critical

Missing type: nativeEnum
Location: src/lib/docs/schema-introspector.ts:63

The knownTypes array doesn't include nativeEnum, which is used for TypeScript enums. The CLAUDE.md example (line 505) uses z.enum() but this appears to be incorrect - it should be z.nativeEnum() for TypeScript enums.

// Current (line 63):
const knownTypes = ['string', 'number', 'boolean', 'array', 'object', 'enum'] as const

// Should be:
const knownTypes = ['string', 'number', 'boolean', 'array', 'object', 'enum', 'nativeEnum'] as const

Fix: Add 'nativeEnum' to knownTypes array and update ExtractedParam['type'] to include it (line 6).

---

🟡 Major

Unsafe assert without error message
Location: src/lib/docs/schema-introspector.ts:53

The assert(inner) call will throw a generic error if innerType is missing. This makes debugging difficult if Zod's internal structure changes.

// Current (line 52-53):
const inner = internals.def.innerType
assert(inner)

// Better:
const inner = internals.def.innerType
assert(inner, `Expected innerType for ${type} but got undefined`)

Impact: Silent failures or unclear error messages during schema introspection.

---

🔵 Minor

CLAUDE.md example uses wrong enum method
Location: CLAUDE.md:505

The example shows z.enum(IntegrationType, ...) but TypeScript enums require z.nativeEnum(). The z.enum() method expects string literal arrays, not TypeScript enum objects.

// Current (incorrect):
type: z.enum(IntegrationType, { ... })

// Should be:
type: z.nativeEnum(IntegrationType, { ... })

Fix: Update the CLAUDE.md example to use z.nativeEnum().

---

✅ Test Coverage

Good test coverage overall. Consider adding:

• Test for z.nativeEnum() (TypeScript enums)
• Test for z.coerce.string() and other coerced types besides numbers
• Test verifying that assert fails gracefully (though this may be excessive)

---

Overall: Strong implementation with automated parameter extraction working correctly. The three issues above should be addressed before merging to ensure robustness with all Zod schema types.

[codecov]

Codecov Report

❌ Patch coverage is 96.15385% with 5 lines in your changes missing coverage. Please review.
✅ Project coverage is 98.51%. Comparing base (159a36a) to head (6dc5f92).
⚠️ Report is 2 commits behind head on develop.

Files with missing lines	Patch %	Lines
src/lib/docs/schema-introspector.ts	93.05%	5 Missing ⚠️

Additional details and impacted files

@@             Coverage Diff             @@
##           develop     #802      +/-   ##
===========================================
- Coverage    98.53%   98.51%   -0.02%     
===========================================
  Files          327      328       +1     
  Lines        18923    19005      +82     
  Branches      1776     1791      +15     
===========================================
+ Hits         18645    18723      +78     
- Misses         270      274       +4     
  Partials         8        8              

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:

• ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
• 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.