provider_registry.rs has a static EMBEDDED array populated manually via include_str!. This is easy to forget when adding a new provider, creating a class of bug where the Docker runtime knows a provider but the host binary does not. Fix: add a build.rs that scans providers/.toml at compile time and generates the embedded list automatically, or at minimum add a unit test asserting every providers/.toml basename appears in EMBEDDED. The filesystem-wins-over-embedded fallback logic and user-provider overlay are correct and should be preserved. Raised in code review with Codex.
provider_registry.rs has a static EMBEDDED array populated manually via include_str!. This is easy to forget when adding a new provider, creating a class of bug where the Docker runtime knows a provider but the host binary does not. Fix: add a build.rs that scans providers/.toml at compile time and generates the embedded list automatically, or at minimum add a unit test asserting every providers/.toml basename appears in EMBEDDED. The filesystem-wins-over-embedded fallback logic and user-provider overlay are correct and should be preserved. Raised in code review with Codex.