Skip to content

Add config options to notion schema generator header#38

Merged
schickling merged 1 commit intomainfrom
schickling/notion-schema-config
Jan 5, 2026
Merged

Add config options to notion schema generator header#38
schickling merged 1 commit intomainfrom
schickling/notion-schema-config

Conversation

@schickling
Copy link
Copy Markdown
Collaborator

Summary

  • Include generation options in generated file header as a @config comment
  • Uses JSON-style formatting with unquoted keys for readability
  • Includes: includeWrite, typedOptions, includeApi, transforms, and schema name override
  • Only shows non-default values to keep comments concise
  • Refactored tests to use snapshots for better clarity

Test plan

  • All 504 unit tests passing
  • Type checking passes with no errors
  • Linting passes with no errors
  • 22 codegen tests with inline snapshots covering:
    • Basic schema generation
    • Config comment formatting with various options
    • Special character handling in keys and values
    • Write schema generation
    • Typed options generation

🤖 Generated with Claude Code

Include generation options (includeWrite, typedOptions, includeApi, transforms, schemaName override) in the generated file header as a @config comment with JSON-style formatting using unquoted keys for readability. This helps developers understand how each generated file was created and regenerate it with the same settings.

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

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

vercel Bot commented Jan 5, 2026

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Review Updated (UTC)
overeng-effect-utils-react-inspector Ready Ready Preview, Comment Jan 5, 2026 9:15pm
overeng-effect-utils-schema-form-aria Ready Ready Preview, Comment Jan 5, 2026 9:15pm

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant