Skip to content

feat: wire skip_llm to smart gen mutation#31

Merged
acmeguy merged 2 commits intomasterfrom
feat/map-field-opt-in-ui
Apr 15, 2026
Merged

feat: wire skip_llm to smart gen mutation#31
acmeguy merged 2 commits intomasterfrom
feat/map-field-opt-in-ui

Conversation

@acmeguy
Copy link
Copy Markdown

@acmeguy acmeguy commented Apr 15, 2026

Summary

  • Add $skip_llm: Boolean variable to SmartGenDataSchemas mutation in datasources.gql
  • The frontend checkbox existed but the parameter was never declared in the GraphQL mutation, so URQL silently dropped it

Depends on

Test plan

  • Check "Skip LLM enrichment" → smart gen completes faster, no AI metrics section
  • Uncheck → AI metrics generated as before

🤖 Generated with Claude Code

acmeguy and others added 2 commits March 31, 2026 09:00
Smart Generation UI improvements:
- Map, nested (ARRAY JOIN), and AI fields default to unchecked (opt-in)
- Count measure and required fields (rewrite rules + filters) always selected
- "Start Over" button on change preview step resets to table selection
- "Skip LLM enrichment" checkbox on column selection step
- AI metrics section starts unchecked (opt-in, not opt-out)
- AI metrics empty selection sends empty array to prevent backend include-all bug
- Source tag support in ChangeField interface (map, nested, ai)

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
The frontend checkbox existed but the GraphQL variable was never declared
in the mutation, so URQL silently dropped it. Now wired end-to-end.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
@acmeguy acmeguy merged commit 4566b20 into master Apr 15, 2026
@acmeguy acmeguy deleted the feat/map-field-opt-in-ui branch April 15, 2026 22:48
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.

2 participants