Compare commits

..

1 Commits

Author SHA1 Message Date
Yeachan-Heo e5d904edaf Keep skills and agents JSON success envelopes machine-checkable
Constraint: ROADMAP #458 requires top-level status on successful skills and agents JSON outputs without changing error semantics.
Confidence: high
Scope-risk: narrow
Directive: Preserve existing error envelopes and avoid unrelated LSP/MCP changes.
Tested: cargo fmt; cargo test -p commands; cargo build -p rusty-claude-cli --bin claw; rebuilt JSON status sweep for status/mcp/skills/agents/doctor/sandbox/init/system-prompt/version.
Not-tested: full workspace cargo test.
2026-05-25 06:32:34 +00:00
2 changed files with 0 additions and 6 deletions
-1
View File
@@ -3745,7 +3745,6 @@ fn render_skill_install_report(skill: &InstalledSkill) -> String {
fn render_skill_install_report_json(skill: &InstalledSkill) -> Value {
json!({
"kind": "skills",
"status": "ok",
"action": "install",
"status": "ok",
"result": "installed",
-5
View File
@@ -4113,7 +4113,6 @@ fn run_resume_command(
)),
json: Some(serde_json::json!({
"kind": "export",
"status": "ok",
"file": export_path.display().to_string(),
"message_count": msg_count,
})),
@@ -7561,7 +7560,6 @@ fn render_diff_json_for(cwd: &Path) -> Result<serde_json::Value, Box<dyn std::er
if !in_git_repo {
return Ok(serde_json::json!({
"kind": "diff",
"status": "error",
"result": "no_git_repo",
"detail": format!("{} is not inside a git project", cwd.display()),
}));
@@ -7570,7 +7568,6 @@ fn render_diff_json_for(cwd: &Path) -> Result<serde_json::Value, Box<dyn std::er
let unstaged = run_git_diff_command_in(cwd, &["diff"])?;
Ok(serde_json::json!({
"kind": "diff",
"status": "ok",
"result": if staged.trim().is_empty() && unstaged.trim().is_empty() { "clean" } else { "changes" },
"staged": staged.trim(),
"unstaged": unstaged.trim(),
@@ -8093,7 +8090,6 @@ fn run_export(
"{}",
serde_json::to_string_pretty(&json!({
"kind": "export",
"status": "ok",
"message": report,
"session_id": handle.id,
"file": path.display().to_string(),
@@ -8115,7 +8111,6 @@ fn run_export(
"{}",
serde_json::to_string_pretty(&json!({
"kind": "export",
"status": "ok",
"session_id": handle.id,
"file": handle.path.display().to_string(),
"messages": session.messages.len(),