fix: YAML-escape peer_display_name in the test-send flow (pr3 iter 8)

torlando-agent[bot] · claude · torlando-agent[bot] · commit fbc098a78219 · 2026-05-31T04:17:42.000-04:00
peer_display_name was interpolated raw into the Maestro flow's `tapOn` while content/has_file_name go through _yaml_escape; a display name with a quote or control char would break the generated YAML. Route it through _yaml_escape too. (greptile #81) Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
diff --git a/Tests/interop/conftest.py b/Tests/interop/conftest.py
@@ -349,7 +349,7 @@ def assert_bubble_visible(
             "    text: \"Chats\"",
             "    optional: true",
             "- waitForAnimationToEnd: { timeout: 2000 }",
-            f"- tapOn: \"{peer_display_name}\"",
+            f"- tapOn: \"{_yaml_escape(peer_display_name)}\"",
             "- waitForAnimationToEnd: { timeout: 2500 }",
         ]
         if content is not None:

Original file line number	Diff line number	Diff line change
`@@ -349,7 +349,7 @@ def assert_bubble_visible(`
`349`	`349`	`" text: \"Chats\"",`
`350`	`350`	`" optional: true",`
`351`	`351`	`"- waitForAnimationToEnd: { timeout: 2000 }",`
`352`		`- f"- tapOn: \"{peer_display_name}\"",`
	`352`	`+ f"- tapOn: \"{_yaml_escape(peer_display_name)}\"",`
`353`	`353`	`"- waitForAnimationToEnd: { timeout: 2500 }",`
`354`	`354`	`]`
`355`	`355`	`if content is not None:`