Skip to content

feat: map field opt-in UI, skip LLM toggle, start over button#30

Merged
acmeguy merged 1 commit intomasterfrom
feat/map-field-opt-in-ui
Mar 31, 2026
Merged

feat: map field opt-in UI, skip LLM toggle, start over button#30
acmeguy merged 1 commit intomasterfrom
feat/map-field-opt-in-ui

Conversation

@acmeguy
Copy link
Copy Markdown

@acmeguy acmeguy commented Mar 31, 2026

Summary

  • Map, nested (ARRAY JOIN), and AI-generated fields default to unchecked (opt-in) in change preview
  • "Start Over" button on change preview step to restart from table selection
  • "Skip LLM enrichment" checkbox to bypass AI metrics and advisory passes
  • Count measure and required fields (rewrite rules + filters) always pre-selected
  • AI metrics empty selection bug fix (sends [] not undefined)

Companion PR

Test plan

  • Verify map fields show unchecked, regular columns show checked in preview
  • Verify "Start Over" resets all state and returns to table selection
  • Verify "Skip LLM" checkbox disables AI enrichment
  • Verify deselecting all AI metrics produces model with no AI metrics

🤖 Generated with Claude Code

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>
@acmeguy acmeguy merged commit bc7b81b into master Mar 31, 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.

2 participants