style: fix formatting and clean up sync note

Collapse the multi-line `assert (... is True)` in the custom-delimiter
test per ruff format, which had broken CI. Move the "keep signatures
in sync with extract_umis_from_read_name" note out of the user-facing
docstring and into an inline comment.

Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>

Author: msto

fgpyo/platform/illumina.py

@@ -100,19 +100,15 @@ def copy_umi_from_read_name(
 
     `strict`, `read_name_delimiter`, and `umi_delimiter` are forwarded to
     [`extract_umis_from_read_name`][fgpyo.platform.illumina.extract_umis_from_read_name] — see
-    that function for their semantics. If you change the signature of either function, update
-    the other to keep them in sync.
+    that function for their semantics.
 
     Args:
         rec: The alignment record to update.
         strict: If `True` and UMI invalid, will throw an exception.
-            Forwarded to `extract_umis_from_read_name`.
         remove_umi: If `True`, the UMI will be removed from the read name after copying.
         read_name_delimiter: The delimiter separating the components of the read name.
-            Forwarded to `extract_umis_from_read_name`, and also used to strip the UMI
-            segment when `remove_umi` is `True`.
+            Also used to strip the UMI segment when `remove_umi` is `True`.
         umi_delimiter: The delimiter separating multiple UMIs.
-            Forwarded to `extract_umis_from_read_name`.
 
     Returns:
         `True` if the UMI was successfully extracted, False if otherwise.
@@ -121,6 +117,7 @@ def copy_umi_from_read_name(
         ValueError: If the read name does not end with a valid UMI.
         ValueError: If the record already has a populated `RX` SAM tag.
     """
+    # NB: Keep the signature of this function in sync with `extract_umis_from_read_name`.
     assert rec.query_name is not None, "Alignment record must have a query name"
 
     umi = extract_umis_from_read_name(

tests/fgpyo/platform/test_illumina.py

@@ -153,9 +153,7 @@ def test_copy_umi_from_read_name_custom_read_name_delimiter() -> None:
     """Test that a custom read_name_delimiter is used for both extraction and UMI removal."""
     builder = SamBuilder()
     read = builder.add_single(name="abc_def_ghi_AAAA+TTTT")
-    assert (
-        copy_umi_from_read_name(read, remove_umi=True, read_name_delimiter="_") is True
-    )
+    assert copy_umi_from_read_name(read, remove_umi=True, read_name_delimiter="_") is True
     assert read.get_tag("RX") == "AAAA-TTTT"
     assert read.query_name == "abc_def_ghi"