fix(service): trim dangling connectors from truncated help clauses

A clause cut can strand a connector — e.g. a colon introducing a list
the newline cut drops, as in im.reactions.list's message_id — and the
help-line joiner then renders "…获取方式：.". Extend inlineClause's
trailing trim from sentence enders to stranded connectors (：:，,、);
mid-clause punctuation is untouched.

Change-Id: Ibc0372c2faba033aca26d3089e1754a5e58c3448

Author: liangshuo-1

cmd/service/paramhelp.go

@@ -74,9 +74,10 @@ var markdownLinkRe = regexp.MustCompile(`\[([^\]]*)\]\([^)]*\)`)
 
 // inlineClause compresses metadata prose into one help clause: markdown links
 // keep their text, the clause cuts at the first rune in stops, whitespace
-// collapses, trailing sentence punctuation goes (the clause join adds its
-// own), and the result caps at max runes. The two policies below differ only
-// in where they cut and how much they keep.
+// collapses, trailing punctuation goes — sentence enders (the clause join adds
+// its own) and connectors a cut can strand, like a colon introducing a list the
+// newline cut dropped — and the result caps at max runes. The two policies
+// below differ only in where they cut and how much they keep.
 func inlineClause(s, stops string, max int) string {
 	if s == "" {
 		return ""
@@ -86,7 +87,7 @@ func inlineClause(s, stops string, max int) string {
 		s = s[:i]
 	}
 	s = strings.Join(strings.Fields(s), " ")
-	s = strings.TrimRight(s, "。.")
+	s = strings.TrimRight(s, "。.：:，,、")
 	return util.TruncateStrWithEllipsis(s, max)
 }
 

cmd/service/sanitize_test.go

@@ -32,3 +32,19 @@ func TestSanitizeOptionDesc(t *testing.T) {
 		t.Errorf("truncation = %q (%d runes), want 40 runes ending in ...", got, len(r))
 	}
 }
+
+func TestSanitizeFieldDesc_TrimsDanglingPunctuation(t *testing.T) {
+	// A clause cut can strand a connector (e.g. a colon introducing a list the
+	// newline cut drops, as in im.reactions.list's message_id); the help line
+	// joiner then renders "…获取方式：." — so dangling punctuation must go too.
+	cases := map[string]string{
+		"待查询的消息ID。ID 获取方式：\n- 调用接口获取": "待查询的消息ID。ID 获取方式",
+		"see the list below:\nitem":  "see the list below",
+		"逗号结尾，\n下一行":                 "逗号结尾",
+	}
+	for in, want := range cases {
+		if got := sanitizeFieldDesc(in); got != want {
+			t.Errorf("sanitizeFieldDesc(%q) = %q, want %q", in, got, want)
+		}
+	}
+}