Skip to content

fix: harden optional connectors and Snowflake references - #51

Merged
skadel merged 4 commits into
mainfrom
codex/snowflake-qualified-refs
Jul 28, 2026
Merged

fix: harden optional connectors and Snowflake references#51
skadel merged 4 commits into
mainfrom
codex/snowflake-qualified-refs

Conversation

@skadel

@skadel skadel commented Jul 27, 2026

Copy link
Copy Markdown
Owner

Summary

  • harden optional connector loading and surface actionable CLI/LLM errors
  • allow Snowflake operations without a default database when table references are fully qualified
  • validate partial Snowflake references early and avoid sending empty database/schema connection parameters
  • expand connector and Snowflake schema-import regression coverage and documentation

Validation

uff check build_query/validator.py cli/generate.py cli/main.py models/env_variables.py tests/test_cli_snowflake_schema_import.py utils/snowflake_connector.py

  • Python compilation check for the touched Snowflake files
  • targeted pytest could not run locally because the repository Python environments are unavailable/broken in this workspace

@skadel
skadel merged commit 4c58360 into main Jul 28, 2026
2 checks passed
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