Config validation rejects breadsearch provider URLs so no breadsearch provider can be configured #47

Closed
opened 2026-10-15 12:38:37 +08:00 by Breadway · 0 comments
Owner

Found while building the RAM-30 daemon-driven breadsearch test. Confidence: high, reproduced live. validate_endpoint in crates/breadram-core/src/config/mod.rs accepts unix:, http://, https:// and hook: URLs but has no breadsearch: branch, so a provider with the documented url breadsearch:/run/user/1000/breadmill.sock fails startup with unsafe endpoint bench-mill: endpoint needs a scheme such as http://. The adapter registry (adapter/mod.rs), the socket helpers, and tests/breadsearch_adapter.rs all use the breadsearch: shape, so validation and the adapter disagree and validation wins at startup. No breadsearch provider can be managed by any daemon with config validation enabled. Expected: accept breadsearch: with an absolute socket path, automation permitted because it names a local socket like unix:. Reproduce: add providers name=bench-mill kind=breadsearch url=breadsearch:/tmp/x.sock allow_automation=true to a daemon config and start breadramd.

Found while building the RAM-30 daemon-driven breadsearch test. Confidence: high, reproduced live. validate_endpoint in crates/breadram-core/src/config/mod.rs accepts unix:, http://, https:// and hook: URLs but has no breadsearch: branch, so a provider with the documented url breadsearch:/run/user/1000/breadmill.sock fails startup with unsafe endpoint bench-mill: endpoint needs a scheme such as http://. The adapter registry (adapter/mod.rs), the socket helpers, and tests/breadsearch_adapter.rs all use the breadsearch:<socket path> shape, so validation and the adapter disagree and validation wins at startup. No breadsearch provider can be managed by any daemon with config validation enabled. Expected: accept breadsearch: with an absolute socket path, automation permitted because it names a local socket like unix:. Reproduce: add [[providers]] name=bench-mill kind=breadsearch url=breadsearch:/tmp/x.sock allow_automation=true to a daemon config and start breadramd.
Sign in to join this conversation.
No labels
No milestone
No project
No assignees
1 participant
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference
Breadway/breadram#47
No description provided.