scriptoriumadmin/fabro
1
0
Fork 0
mirror of https://github.com/fabro-sh/fabro.git synced 2026-10-07 03:00:29 +00:00
Code Issues Projects Releases Packages Wiki Activity Actions 4

refactor(mcp): reuse manifest args emptiness check

Browse source
This commit is contained in:
Bryan Helmkamp 2026-05-11 12:09:48 -04:00
parent 3a7219c865
commit 19f7479490
No known key found for this signature in database
1 changed files with 1 additions and 14 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

15
lib/crates/fabro-mcp-server/src/run_tools.rs
View file

@ -918,7 +918,7 @@ fn mcp_manifest_args(spec: &CreateRunSpec) -> Option<types::ManifestArgs> {
sandbox: spec.sandbox.clone(),
verbose: None,
};
(!mcp_manifest_args_is_empty(&payload)).then_some(payload)
(!fabro_manifest::manifest_args_is_empty(&payload)).then_some(payload)
}
fn mcp_run_overrides(spec: &CreateRunSpec) -> Option<RunLayer> {
@ -934,19 +934,6 @@ fn mcp_run_overrides(spec: &CreateRunSpec) -> Option<RunLayer> {
})
}
fn mcp_manifest_args_is_empty(args: &types::ManifestArgs) -> bool {
args.auto_approve.is_none()
&& args.docker_image.is_none()
&& args.dry_run.is_none()
&& args.input.is_empty()
&& args.label.is_empty()
&& args.model.is_none()
&& args.preserve_sandbox.is_none()
&& args.provider.is_none()
&& args.sandbox.is_none()
&& args.verbose.is_none()
}
fn json_to_toml_value(key: &str, value: &Value) -> ToolResult<toml::Value> {
match value {
Value::Null => Err(ToolError::message(format!(