Skip to content

refactor: update schema export docs from flat props to nested schema object#59

Merged
pyramation merged 2 commits intomainfrom
devin/1773817552-schema-config-refactor
Mar 18, 2026
Merged

refactor: update schema export docs from flat props to nested schema object#59
pyramation merged 2 commits intomainfrom
devin/1773817552-schema-config-refactor

Conversation

@pyramation
Copy link
Contributor

Summary

Updates all skill documentation to reflect the schema export config refactor from flat properties to a nested object, matching constructive-io/constructive#848.

Before: schemaOnly: true, schemaOnlyOutput: './schemas', schemaOnlyFilename: 'public.graphql'
After: schema: { enabled: true, output: './schemas', filename: 'public.graphql' }

Changes across 3 docs files + rebuilt zip:

  • SKILL.md — Updated code examples in Schema Export section and GenerateOptions interface definition
  • references/generate-schemas.md — Updated all code examples, options table, generateMulti() usage, and behavioral description
  • references/cli-reference.md — Added new Schema Export Options table (--schema-enabled, --schema-output, --schema-filename)

Review & Testing Checklist for Human

  • Verify behavioral description accuracy: generate-schemas.md line 61 now states schema export works alongside generators when both are enabled. Confirm this matches the actual implementation in constructive#848 — the old behavior was mutually exclusive (schema export OR code generation, never both).
  • Verify CLI flags match implementation: The new CLI flags --schema-enabled, --schema-output, --schema-filename in cli-reference.md should match what was implemented in constructive's cli/index.ts and cli/handler.ts.
  • Ensure constructive#848 merges first: These docs will be incorrect until the companion implementation PR lands. Consider merge ordering.

Notes

  • The .zip file shows as newly created because it wasn't previously tracked. It's the standard skill packaging format per AGENTS.md.
  • constructive-private-skills and constructive-db were audited and have no references to the old schema config syntax (no changes needed).

Link to Devin session: https://app.devin.ai/sessions/a0ac766d75d340d490fd265bfafaac83
Requested by: @pyramation

…object

Migrate all references from schemaOnly/schemaOnlyOutput/schemaOnlyFilename
to the new schema: { enabled, output, filename } config structure.

Updates SKILL.md, generate-schemas.md, and cli-reference.md.
@devin-ai-integration
Copy link

🤖 Devin AI Engineer

I'll be helping with this pull request! Here's what you should know:

✅ I will automatically:

  • Address comments on this PR. Add '(aside)' to your comment to have me ignore it.
  • Look at CI failures and help fix them

Note: I can only respond to comments from users who have write access to this repository.

⚙️ Control Options:

  • Disable automatic comment and CI monitoring

@pyramation pyramation merged commit dd32eed into main Mar 18, 2026
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