Port 'Fix verse range marker placement bug' #558

Activity

1. Enkidu93

   added

   portingThis issue consists of porting code from machine.py to machine.This issue consists of porting code from machine.py to machine.

   on Oct 26, 2026
2. github-project-automation

   added this to Servalon Oct 26, 2026
3. github-project-automation

   moved this to 🆕 New in Servalon Oct 26, 2026
4. claude commented on Oct 26, 2026

   

   claudeBot

   on Oct 26, 2026 – with Claude

   Last edited by claude

   Contributor

   More actions

   Claude finished @Enkidu93's task in 5m 49s —— View job • claude/issue-558-20261026-1309 • [Create PR ➔](https://github.com/sillsdev/machine/compare/master...claude/issue-558-20261026-1309?quick_pull=1&title=Issue%20%23558%3A%20Changes%20from%20Claude&body=This%20PR%20addresses%20issue%20%23558%0A%0AGenerated%20with%20%5BClaude%20Code%5D(https%3A%2F%2Fclaude.ai%2Fcode)

   ---

   Port of machine.py PR #371 is done

   • Read the porting issue and machine.py PR Check for Paratext project files case-insensitively #371
   • Port UsfmUpdateBlock.Rows (replaces Metadata) and the internal UsfmUpdateBlockBuilder
   • Port the marker placement fixes in PlaceMarkersUsfmUpdateBlockHandler: empty rows are ignored, per-row alignments are merged, out-of-range pairs are rejected, and closing markers can't land before their opening marker
   • Add and update tests
   • ./local_check.sh --agent-strict → local_check: PASSED
   • Open the PR: Port 'Fix verse range marker placement bug' from machine.py #559

   Not ported: the machine.py warning logs, because SIL.Machine has no logging dependency. I also skipped the machine.py tests that only cover that logging and the unaligned-end placement.

   Public API change: UsfmUpdateBlock.Metadata and the block's mutator methods are gone. Rows replaces Metadata, and the mutators now live on an internal builder. This matches the machine.py change.
   · branch claude/issue-558-20261026-1309