René's URL Explorer Experiment

Title: SK-2963: warn at startup when a beta/dev build targets a Production vault by Devesh-Skyflow · Pull Request #274 · skyflowapi/skyflow-python · GitHub

Open Graph Title: SK-2963: warn at startup when a beta/dev build targets a Production vault by Devesh-Skyflow · Pull Request #274 · skyflowapi/skyflow-python

X Title: SK-2963: warn at startup when a beta/dev build targets a Production vault by Devesh-Skyflow · Pull Request #274 · skyflowapi/skyflow-python

Description: Summary Corrective action from the Fifth Third Bank RCA (CUST-4287): a beta/pre-release SDK build ended up running in a customer's Production environment undetected. This adds a startup warning so that doesn't happen silently again. Adds a WARN-level log, emitted once in Skyflow.Builder.build(), when: the SDK's own version (SDK_VERSION) is non-GA — any PEP 440 bN or .devN+ suffix, and at least one configured vault's env is explicitly Env.PROD. Note on this SDK specifically: unlike Java/Node/Go, Python has no SDK-side default-to-PROD for a missing env — validate_vault_config only validates env if the key is present at all, and samples always set it explicitly. So any_vault_is_prod() here only fires on an explicit Env.PROD, matching this SDK's real behavior rather than assuming a default that doesn't exist in this codebase. Changes skyflow/utils/_utils.py: is_non_ga_version() and any_vault_is_prod() — small pure functions, exported through skyflow/utils/__init__.py for direct unit testing. skyflow/utils/_skyflow_messages.py: new SkyflowMessages.Warning.BETA_BUILD_WARNING. skyflow/client/skyflow.py: wires the check into Builder.build(), right before constructing Skyflow(self). Goes through the existing log_warn(..., self.__logger), so it respects the configured log level exactly like the existing UPDATE_LOG_LEVEL_DEPRECATED warning does. Testing tests/utils/test__utils.py: unit tests for is_non_ga_version (GA/beta/dev/None/garbage) and any_vault_is_prod (empty, none-prod, one-of-many-prod, missing-env). tests/client/test_skyflow.py (TestSkyflowBetaBuildWarning): patches skyflow.client.skyflow.SDK_VERSION to a fake beta value and exercises the real build() wiring end-to-end — confirms it warns for an explicit PROD vault and for one-of-several, stays silent when no vault is PROD, stays silent when env is omitted entirely, and (unpatched) stays silent against this checkout's real GA version. Verification caveat: I wasn't able to execute this repo's test suite in my sandbox — no pip/venv available and no way to install this project's dependencies (pydantic, httpx, cryptography, etc.) without sudo. I syntax-checked every changed file with python -m py_compile (all clean) and hand-verified the is_non_ga_version regex against the test cases with a dependency-free script, but please run pytest / your CI on this PR before merging — it hasn't had the equivalent of the full-suite verification the companion Java and Node PRs got. Related Cross-SDK design + task tracking: SK-2963 (this is the reference-pattern implementation for skyflow-java; skyflow-go/js/... follow the same shape in their own repos). SK-2977 owns the final public-facing message wording — the copy here is a placeholder pending that sign-off. 🤖 Generated with Claude Code

Open Graph Description: Summary Corrective action from the Fifth Third Bank RCA (CUST-4287): a beta/pre-release SDK build ended up running in a customer's Production environment undetected. This adds a startup warning...

X Description: Summary Corrective action from the Fifth Third Bank RCA (CUST-4287): a beta/pre-release SDK build ended up running in a customer's Production environment undetected. This adds a startup war...

Opengraph URL: https://github.com/skyflowapi/skyflow-python/pull/274

X: @github

direct link

Domain: github.com

Links:

Viewport: width=device-width