Vamsi-klu opened a new pull request, #67054:
URL: https://github.com/apache/airflow/pull/67054
Fixes the connection form so provider-declared extra widgets hydrate from
existing raw Extras JSON instead of serializing widget defaults back into
`extra` during initial render.
The previous store initialization path called `setParamsDict`, which rewrote
every schema-declared value into `conf`. For Snowflake, saving raw
`{"account":"1234"}` could be replaced by defaults such as `{"insecure_mode":
false}` before the user touched any extra widget.
This change:
- adds a hydration path that reads existing raw JSON into widget values
without mutating `conf`
- tracks widget keys that users actually edit and serializes only those keys
back into `conf`
- keeps raw-only JSON keys while allowing touched widgets to override
matching raw keys
- adds regression coverage for schema-declared values, raw-only values, and
boolean defaults
closes: #57984
Tests:
- `pnpm test useParamStore TriggerDAGForm`
- `pnpm tsc --noEmit`
- `pnpm lint`
---
##### Was generative AI tooling used to co-author this PR?
- [X] Yes — Codex (GPT-5)
Generated-by: Codex (GPT-5) following [the
guidelines](https://github.com/apache/airflow/blob/main/contributing-docs/05_pull_requests.rst#gen-ai-assisted-contributions)
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
