Pin the suffix-comma peel and name the maiden reach (#308)

derek73 · Jul 31, 2026 · Jul 31, 2026 · Jul 31, 2026 · Jul 31, 2026 · Jul 31, 2026
commit 90be6ba5f685871bea8eadc76e205668e462c520
diff --git a/nameparser/_pipeline/_script_segment.py b/nameparser/_pipeline/_script_segment.py
@@ -250,6 +250,15 @@ def _peel_honorific_tail(state: ParseState) -> ParseState:
     that returns None where the outright index would raise removes an
     unpinned reliance on the FAMILY_COMMA gate landing first.
 
+    That last token is the last of the NAME PART, which under NO_COMMA
+    reaches into a maiden clause: maiden tokens are still main-stream
+    here (extract_delimited has masked only bracketed content), so
+    "김민준 née 박씨" peels 씨 off the MAIDEN name 박씨 and hands it to
+    the person's suffix list -- "née Ms. Park". Intended rather than
+    incidental: the honorific is the reader's regardless of which of
+    her names it was glued to, and a name-part-final honorific is
+    exactly what this peels.
+
     Longest-first. ONE peel, never recursive: 김민준박사님 gives up
     its 님 and keeps its glued 박사, though 박사 is itself a listed
     tail, which is accepted rather than chased. No script precondition

diff --git a/tests/v2/cases.py b/tests/v2/cases.py
@@ -878,6 +878,16 @@ def __post_init__(self) -> None:
                "hide it -- this now agrees with the comma-written "
                "'Dr 김민준씨, Jr.', where the suffix comma had "
                "already put 씨 within reach"),
+    Case("ko_honorific_glued_given_suffix_comma", "Dr 김민준씨, Jr.",
+         {"title": "Dr", "family": "김", "given": "민준",
+          "suffix": "씨, Jr."},
+         classification="fix(#308)",
+         notes="the peel indexes the NAME part, not the token stream: "
+               "under a suffix comma segments[0] is a strict subset, "
+               "and the peel site is found within it. Pairs with "
+               "ko_honorific_glued_given_trailing_suffix, whose "
+               "comma-less spelling of the same name reaches the same "
+               "answer by the scan-back instead"),
     Case("zh_honorific_glued_surname", "王先生",
          {"family": "王", "suffix": "先生"},
          locale="zh",

diff --git a/tests/v2/pipeline/test_script_segment.py b/tests/v2/pipeline/test_script_segment.py
@@ -516,7 +516,7 @@ def test_a_trailing_post_nominal_does_not_hide_the_peel_site() -> None:
                        lexicon=_LEX_TAILS)) == ["김", "민준", "씨", "jr"]
 
 
-def test_the_peel_only_looks_at_the_last_name_token() -> None:
+def test_the_peel_only_looks_at_the_last_non_post_nominal_token() -> None:
     # a tail anywhere but the end is somebody's name, not an
     # honorific: only the trailing position is an honorific site, the
     # same reasoning the trailing-only suffix gate follows

diff --git a/tools/differential/corpus_cjk.jsonl b/tools/differential/corpus_cjk.jsonl
@@ -1,6 +1,7 @@
 "Andersonさん"
 "Anderson선생님"
 "Dr 김민준, Jr."
+"Dr 김민준씨, Jr."
 "John 王"
 "〆木 ひろ"
 "〆木 太郎"