fix(hooks): add JsonSchema derive to HooksConfig and BuiltinHooksConfig
Upstream main now derives schemars::JsonSchema on all config structs. Our HooksConfig and BuiltinHooksConfig were missing it, causing CI Build (Smoke) failure when the merge commit was compiled.
This commit is contained in:
parent
eb60d0fb81
commit
69f4b95f8e
@ -1300,6 +1300,7 @@ pub(crate) async fn agent_turn(
|
||||
max_tool_iterations,
|
||||
None,
|
||||
None,
|
||||
None,
|
||||
)
|
||||
.await
|
||||
}
|
||||
@ -2126,6 +2127,7 @@ pub async fn run(
|
||||
config.agent.max_tool_iterations,
|
||||
None,
|
||||
None,
|
||||
None,
|
||||
)
|
||||
.await?;
|
||||
final_output = response.clone();
|
||||
@ -2245,6 +2247,7 @@ pub async fn run(
|
||||
config.agent.max_tool_iterations,
|
||||
None,
|
||||
None,
|
||||
None,
|
||||
)
|
||||
.await
|
||||
{
|
||||
|
||||
@ -1761,7 +1761,7 @@ impl Default for ObservabilityConfig {
|
||||
|
||||
// ── Hooks ────────────────────────────────────────────────────────
|
||||
|
||||
#[derive(Debug, Clone, Serialize, Deserialize)]
|
||||
#[derive(Debug, Clone, Serialize, Deserialize, JsonSchema)]
|
||||
pub struct HooksConfig {
|
||||
/// Enable lifecycle hook execution.
|
||||
///
|
||||
@ -1781,7 +1781,7 @@ impl Default for HooksConfig {
|
||||
}
|
||||
}
|
||||
|
||||
#[derive(Debug, Clone, Serialize, Deserialize)]
|
||||
#[derive(Debug, Clone, Serialize, Deserialize, JsonSchema)]
|
||||
pub struct BuiltinHooksConfig {
|
||||
/// Enable the command-logger hook (logs tool calls for auditing).
|
||||
pub command_logger: bool,
|
||||
|
||||
@ -409,6 +409,7 @@ impl DelegateTool {
|
||||
agent_config.max_iterations,
|
||||
None,
|
||||
None,
|
||||
None,
|
||||
),
|
||||
)
|
||||
.await;
|
||||
|
||||
Loading…
Reference in New Issue
Block a user