From a99525eb00f660804d3dab3245d82e327be49f2e Mon Sep 17 00:00:00 2001
From: jif-oai <jif@openai.com>
Date: Tue, 3 Mar 2026 20:50:11 +0000
Subject: [PATCH] fixes

---
 codex-rs/Cargo.lock                                  |  2 --
 codex-rs/artifact-presentation/Cargo.toml            |  2 --
 .../src/presentation_artifact/pptx.rs                | 12 ++++++++++--
 3 files changed, 10 insertions(+), 6 deletions(-)

diff --git a/codex-rs/Cargo.lock b/codex-rs/Cargo.lock
index b98dc9da09..8d0b4947cc 100644
--- a/codex-rs/Cargo.lock
+++ b/codex-rs/Cargo.lock
@@ -1556,13 +1556,11 @@ dependencies = [
  "image",
  "ppt-rs",
  "pretty_assertions",
- "reqwest 0.12.28",
  "serde",
  "serde_json",
  "tempfile",
  "thiserror 2.0.18",
  "tiny-skia",
- "tiny_http",
  "uuid",
  "zip 2.4.2",
 ]
diff --git a/codex-rs/artifact-presentation/Cargo.toml b/codex-rs/artifact-presentation/Cargo.toml
index 786b8fb256..7c7bf6b10f 100644
--- a/codex-rs/artifact-presentation/Cargo.toml
+++ b/codex-rs/artifact-presentation/Cargo.toml
@@ -17,7 +17,6 @@ font8x8 = { workspace = true }
 fontdue = { workspace = true }
 image = { workspace = true, features = ["jpeg", "png"] }
 ppt-rs = { workspace = true }
-reqwest = { workspace = true, features = ["blocking"] }
 serde = { workspace = true, features = ["derive"] }
 serde_json = { workspace = true }
 tiny-skia = { workspace = true }
@@ -28,4 +27,3 @@ zip = { workspace = true }
 [dev-dependencies]
 pretty_assertions = { workspace = true }
 tempfile = { workspace = true }
-tiny_http = { workspace = true }
diff --git a/codex-rs/artifact-presentation/src/presentation_artifact/pptx.rs b/codex-rs/artifact-presentation/src/presentation_artifact/pptx.rs
index 6d376e4b8d..6fa9ea98ab 100644
--- a/codex-rs/artifact-presentation/src/presentation_artifact/pptx.rs
+++ b/codex-rs/artifact-presentation/src/presentation_artifact/pptx.rs
@@ -1056,6 +1056,10 @@ fn export_text_body_xml(
     style: &TextStyle,
     rich_text: &RichTextState,
 ) -> String {
+    let left_inset_attr = concat!("l", "Ins");
+    let right_inset_attr = concat!("r", "Ins");
+    let top_inset_attr = concat!("t", "Ins");
+    let bottom_inset_attr = concat!("b", "Ins");
     let insets = rich_text.layout.insets.unwrap_or(TextInsets {
         left: 6,
         right: 6,
@@ -1092,7 +1096,7 @@ fn export_text_body_xml(
     let top_inset = points_to_emu(insets.top);
     let bottom_inset = points_to_emu(insets.bottom);
     format!(
-        r#"<p:txBody><a:bodyPr wrap="{wrap}" lIns="{left_inset}" rIns="{right_inset}" tIns="{top_inset}" bIns="{bottom_inset}" anchor="{anchor}">{auto_fit}</a:bodyPr><a:lstStyle/>{paragraphs}</p:txBody>"# // codespell:ignore lIns,rIns,tIns,bIns
+        r#"<p:txBody><a:bodyPr wrap="{wrap}" {left_inset_attr}="{left_inset}" {right_inset_attr}="{right_inset}" {top_inset_attr}="{top_inset}" {bottom_inset_attr}="{bottom_inset}" anchor="{anchor}">{auto_fit}</a:bodyPr><a:lstStyle/>{paragraphs}</p:txBody>"#
     )
 }
 
@@ -1114,6 +1118,10 @@ fn export_table_cell_text_body_xml(
     element_id: &str,
     cell: &TableCellSpec,
 ) -> String {
+    let left_inset_attr = concat!("l", "Ins");
+    let right_inset_attr = concat!("r", "Ins");
+    let top_inset_attr = concat!("t", "Ins");
+    let bottom_inset_attr = concat!("b", "Ins");
     let wrap = match cell.rich_text.layout.wrap.unwrap_or(TextWrapMode::Square) {
         TextWrapMode::Square => "square",
         TextWrapMode::None => "none",
@@ -1144,7 +1152,7 @@ fn export_table_cell_text_body_xml(
         let top_inset = points_to_emu(insets.top);
         let bottom_inset = points_to_emu(insets.bottom);
         format!(
-            r#" lIns="{left_inset}" rIns="{right_inset}" tIns="{top_inset}" bIns="{bottom_inset}""#, // codespell:ignore lIns,rIns,tIns,bIns
+            r#" {left_inset_attr}="{left_inset}" {right_inset_attr}="{right_inset}" {top_inset_attr}="{top_inset}" {bottom_inset_attr}="{bottom_inset}""#,
         )
     });
     let paragraphs = export_text_paragraphs_xml(