fix(markdown): add dim attribute to code block style for better visibility (tailcallhq#752)

tusharmath · web-flow · commit f03db932d17e · 2025-05-04T11:54:52.000+05:30
diff --git a/crates/forge_display/src/markdown.rs b/crates/forge_display/src/markdown.rs
@@ -19,7 +19,9 @@ impl MarkdownFormat {
         let compound_style = CompoundStyle::new(Some(Color::Cyan), None, Attribute::Bold.into());
         skin.inline_code = compound_style.clone();
 
-        let codeblock_style = CompoundStyle::new(Some(Color::Cyan), None, Default::default());
+        let mut codeblock_style = CompoundStyle::new(None, None, Default::default());
+        codeblock_style.add_attr(Attribute::Dim);
+
         skin.code_block = LineStyle::new(codeblock_style, Default::default());
 
         Self { skin, max_consecutive_newlines: 2 }
diff --git a/crates/forge_domain/src/snapshots/forge_domain__tool_call_record__tests__display_successful_call.snap b/crates/forge_domain/src/snapshots/forge_domain__tool_call_record__tests__display_successful_call.snap
@@ -5,6 +5,5 @@ snapshot_kind: text
 ---
 ---
 tool_name: fs_read
-status: Success
 ---
 Contents of the file
diff --git a/crates/forge_domain/src/snapshots/forge_domain__tool_call_record__tests__display_with_special_chars.snap b/crates/forge_domain/src/snapshots/forge_domain__tool_call_record__tests__display_with_special_chars.snap
@@ -5,6 +5,5 @@ snapshot_kind: text
 ---
 ---
 tool_name: test_tool
-status: Success
 ---
 Result with <tags> & special "characters"
diff --git a/crates/forge_domain/src/tool_call_record.rs b/crates/forge_domain/src/tool_call_record.rs
@@ -23,8 +23,6 @@ impl Display for ToolCallRecord {
         writeln!(f, "tool_name: {tool_name}")?;
         if self.tool_result.is_error {
             writeln!(f, "status: Failure")?;
-        } else {
-            writeln!(f, "status: Success")?;
         }
         writeln!(f, "---")?;
 

Original file line number	Diff line number	Diff line change
`@@ -23,8 +23,6 @@ impl Display for ToolCallRecord {`
`23`	`23`	`writeln!(f, "tool_name: {tool_name}")?;`
`24`	`24`	`if self.tool_result.is_error {`
`25`	`25`	`writeln!(f, "status: Failure")?;`
`26`		`- } else {`
`27`		`- writeln!(f, "status: Success")?;`
`28`	`26`	`}`
`29`	`27`	`writeln!(f, "---")?;`
`30`	`28`