refactor(testmanagement): trim rich-text comments, add generality tests

Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>

Author: SavioBS629

src/tools/testmanagement-utils/create-testcase.ts

@@ -325,8 +325,6 @@ export async function createTestCase(
     new Set([...(testCaseParams.tags ?? []), "MCP Generated"]),
   );
 
-  // Rich-text fields must be HTML-wrapped or the TM UI shows entity-encoded
-  // text literally (PMAA-185); see rich-text.ts.
   testCaseParams.test_case_steps = wrapTestCaseSteps(
     testCaseParams.test_case_steps,
   );

src/tools/testmanagement-utils/rich-text.ts

@@ -1,12 +1,4 @@
-// TM's v2 API treats step/result, preconditions and description as rich-text
-// fields: its write-time sanitizer entity-encodes <, > and & in text nodes
-// (preserving allowed tags like <p>), and the TM UI renders the stored value
-// as rich text only when it is HTML-wrapped — bare text is displayed
-// literally, so a bare ">" surfaces to users as "&gt;" (PMAA-185).
-// Per the TM team, external API content must be wrapped in <p> tags to opt
-// into rich-text rendering. Quotes are not entity-encoded, so only values
-// containing <, > or & need the wrap; everything else is left untouched.
-
+// TM UI renders these fields as rich text only when HTML-wrapped; 
 const LOOKS_LIKE_HTML = /^\s*<[a-z][^>]*>/i;
 const HAS_ENCODABLE_CHARS = /[<>&]/;
 

src/tools/testmanagement-utils/update-testcase.ts

@@ -178,8 +178,6 @@ export async function updateTestCase(
   const testCaseBody: Record<string, any> = {};
 
   if (params.name !== undefined) testCaseBody.name = params.name;
-  // Rich-text fields must be HTML-wrapped or the TM UI shows entity-encoded
-  // text literally (PMAA-185); see rich-text.ts.
   if (params.description !== undefined)
     testCaseBody.description = wrapRichText(params.description);
   if (params.preconditions !== undefined)

tests/tools/richText.test.ts

@@ -41,6 +41,21 @@ describe('wrapRichText', () => {
       '<p>< 5 items are shown</p>',
     );
   });
+
+  it('handles any characters, not just fixed strings', () => {
+    expect(wrapRichText('émojis 🎉 and ünïcode ≥ 5 stay bare')).toBe(
+      'émojis 🎉 and ünïcode ≥ 5 stay bare',
+    );
+    expect(wrapRichText('unicode plus html char: 温度 > 30°C')).toBe(
+      '<p>unicode plus html char: 温度 > 30°C</p>',
+    );
+    expect(wrapRichText('line1 a > b\nline2 c < d')).toBe(
+      '<p>line1 a > b\nline2 c < d</p>',
+    );
+    expect(wrapRichText('a>=b && c<=d & "e"')).toBe(
+      '<p>a>=b && c<=d & "e"</p>',
+    );
+  });
 });
 
 describe('wrapTestCaseSteps', () => {