Skip to content

Conversation

@michdr
Copy link
Contributor

@michdr michdr commented Nov 26, 2025

Note

Handle missing foreign keys during extra seed extraction and add an E2E probe test for seeds with extra columns.

  • Fix (local generation):
    • Safely handle missing foreign_keys by iterating over (tgt_g_table.foreign_keys or []) when extracting context FK for extra seed columns in mostlyai/sdk/_local/execution/step_generate_data.py.
  • Tests:
    • Extend end-to-end test to probe with seed containing extra columns (e.g., {"a": ["a1"], "x": ["x"]}) in tests/_local/end_to_end/test_simple_flat.py.

Written by Cursor Bugbot for commit 655c63b. This will update automatically on new commits. Configure here.

@michdr michdr requested a review from a team as a code owner November 26, 2025 08:43
@michdr michdr merged commit f388c63 into main Nov 26, 2025
18 checks passed
@michdr michdr deleted the fix-ex-seed-fk-none branch November 26, 2025 09: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.

3 participants