fix: correct API parameter formats for createRedactions

- Move includeAnnotations/includeText to strategyOptions (not root level)
- Use camelCase for API parameters (caseSensitive, wholeWordsOnly)
- Put appearance options in 'content' object with correct names (fillColor, outlineColor)
- Simplify createRedactions handler to pass through strategyOptions directly
- Remove unsupported stroke_width parameter

These changes align with the Nutrient API OpenAPI specification.

Author: jdrhyne

src/nutrient_dws/api/direct.py

@@ -321,18 +321,23 @@ def create_redactions_preset(
         """
         options = {
             "strategy": "preset",
-            "strategy_options": {"preset": preset},
-            "include_annotations": include_annotations,
-            "include_text": include_text,
+            "strategy_options": {
+                "preset": preset,
+                "includeAnnotations": include_annotations,
+                "includeText": include_text,
+            },
         }
 
         # Add appearance options if provided
+        content = {}
         if appearance_fill_color:
-            options["appearance_fill_color"] = appearance_fill_color
+            content["fillColor"] = appearance_fill_color
         if appearance_stroke_color:
-            options["appearance_stroke_color"] = appearance_stroke_color
-        if appearance_stroke_width is not None:
-            options["appearance_stroke_width"] = appearance_stroke_width
+            content["outlineColor"] = appearance_stroke_color
+        # Note: stroke width is not supported by the API
+
+        if content:
+            options["content"] = content
 
         return self._process_file("create-redactions", input_file, output_path, **options)
 
@@ -378,19 +383,22 @@ def create_redactions_regex(
             "strategy": "regex",
             "strategy_options": {
                 "pattern": pattern,
-                "case_sensitive": case_sensitive,
+                "caseSensitive": case_sensitive,
+                "includeAnnotations": include_annotations,
+                "includeText": include_text,
             },
-            "include_annotations": include_annotations,
-            "include_text": include_text,
         }
 
         # Add appearance options if provided
+        content = {}
         if appearance_fill_color:
-            options["appearance_fill_color"] = appearance_fill_color
+            content["fillColor"] = appearance_fill_color
         if appearance_stroke_color:
-            options["appearance_stroke_color"] = appearance_stroke_color
-        if appearance_stroke_width is not None:
-            options["appearance_stroke_width"] = appearance_stroke_width
+            content["outlineColor"] = appearance_stroke_color
+        # Note: stroke width is not supported by the API
+
+        if content:
+            options["content"] = content
 
         return self._process_file("create-redactions", input_file, output_path, **options)
 
@@ -438,20 +446,23 @@ def create_redactions_text(
             "strategy": "text",
             "strategy_options": {
                 "text": text,
-                "case_sensitive": case_sensitive,
-                "whole_words_only": whole_words_only,
+                "caseSensitive": case_sensitive,
+                "wholeWordsOnly": whole_words_only,
+                "includeAnnotations": include_annotations,
+                "includeText": include_text,
             },
-            "include_annotations": include_annotations,
-            "include_text": include_text,
         }
 
         # Add appearance options if provided
+        content = {}
         if appearance_fill_color:
-            options["appearance_fill_color"] = appearance_fill_color
+            content["fillColor"] = appearance_fill_color
         if appearance_stroke_color:
-            options["appearance_stroke_color"] = appearance_stroke_color
-        if appearance_stroke_width is not None:
-            options["appearance_stroke_width"] = appearance_stroke_width
+            content["outlineColor"] = appearance_stroke_color
+        # Note: stroke width is not supported by the API
+
+        if content:
+            options["content"] = content
 
         return self._process_file("create-redactions", input_file, output_path, **options)
 

src/nutrient_dws/builder.py

@@ -230,36 +230,14 @@ def _map_tool_to_action(self, tool: str, options: dict[str, Any]) -> dict[str, A
                     action["position"] = options["position"]
 
             case "createRedactions":
-                # Handle create redactions with strategy options
+                # Handle create redactions - pass through directly
+                # The direct.py already formats everything correctly
                 if "strategy" in options:
                     action["strategy"] = options["strategy"]
                 if "strategy_options" in options:
-                    # Map strategy_options based on strategy type
-                    strategy = options.get("strategy", "")
-                    if strategy == "preset":
-                        action["preset"] = options["strategy_options"].get("preset")
-                    elif strategy == "regex":
-                        action["pattern"] = options["strategy_options"].get("pattern")
-                        if "case_sensitive" in options["strategy_options"]:
-                            action["caseSensitive"] = options["strategy_options"]["case_sensitive"]
-                    elif strategy == "text":
-                        action["text"] = options["strategy_options"].get("text")
-                        if "case_sensitive" in options["strategy_options"]:
-                            action["caseSensitive"] = options["strategy_options"]["case_sensitive"]
-                        if "whole_words_only" in options["strategy_options"]:
-                            action["wholeWordsOnly"] = options["strategy_options"][
-                                "whole_words_only"
-                            ]
-
-                # Copy over other options
-                for key, value in options.items():
-                    if key not in ["strategy", "strategy_options"]:
-                        # Convert snake_case to camelCase for API
-                        camel_key = "".join(
-                            word.capitalize() if i else word
-                            for i, word in enumerate(key.split("_"))
-                        )
-                        action[camel_key] = value
+                    action["strategyOptions"] = options["strategy_options"]
+                if "content" in options:
+                    action["content"] = options["content"]
 
             case "optimize":
                 # Handle optimize action with camelCase conversion