#119 - ASV: words run together around added words (transChange) — "allthyprecepts", "AndI" - schaefera/utStudyBIble

claude-bot commented

2026-10-29 14:40:26 +00:00

Collaborator

Found while checking PR #115 on the phone (2026-10-29). ASV, Psalm 119, shows "Forthey" (v126), "Therefore I esteem allthyprecepts concerning allthingsto be right; AndI hate" (v128). This happens in both verse and running mode, so it isn't caused by #115.

Cause. The module has no space between a word and an added word:

<w lemma="strong:H3605">all</w><transChange type="added">thy</transChange><w lemma="strong:H6490">precepts</w>

(raw ASV ot.bzz, Ps 119:128). We print exactly what the module holds.

To decide.

Check what diatheke and JSword/AndBible print for this verse (tools/golden; ASV isn't in the golden set yet).
If they add spaces, match them: a space at a <w>/<transChange> boundary when neither side has whitespace or punctuation. Add ASV Ps 119:126–128 to the golden set as the test.
If they don't, this is a module defect. Report it to CrossWire and leave our output as it is ("Module output must match the module").

Priority is set to Should because the result is visible on every page of ASV. Change it as you see fit.

🤖 Generated with Claude Code

Found while checking PR #115 on the phone (2026-10-29). ASV, Psalm 119, shows "Forthey" (v126), "Therefore I esteem allthyprecepts concerning allthingsto be right; AndI hate" (v128). This happens in both verse and running mode, so it isn't caused by #115. **Cause.** The module has no space between a word and an added word: ``` <w lemma="strong:H3605">all</w><transChange type="added">thy</transChange><w lemma="strong:H6490">precepts</w> ``` (raw ASV `ot.bzz`, Ps 119:128). We print exactly what the module holds. **To decide.** - Check what diatheke and JSword/AndBible print for this verse (`tools/golden`; ASV isn't in the golden set yet). - If they add spaces, match them: a space at a `<w>`/`<transChange>` boundary when neither side has whitespace or punctuation. Add ASV Ps 119:126–128 to the golden set as the test. - If they don't, this is a module defect. Report it to CrossWire and leave our output as it is ("Module output must match the module"). Priority is set to Should because the result is visible on every page of ASV. Change it as you see fit. 🤖 Generated with [Claude Code](https://claude.com/claude-code)

claude-bot added the

labels

2026-10-29 14:40:26 +00:00

claude-bot referenced this issue

2026-10-29 14:48:42 +00:00

Running text: paragraphs of verses with inline numbers (#12 part 3c) #115

claude-bot referenced this issue

2026-10-29 15:40:07 +00:00

Quick-settings sheet: F3 red letters, F4 verse layout, font, night mode #12

claude-bot added this to the v0.2 — study tools milestone

2026-11-03 21:37:20 +00:00

claude-bot commented

2026-11-03 21:37:20 +00:00

Author

Collaborator

Owner decision (2026-11-03): deferred out of v0.1. The raw module has no space at these boundaries, so the fix is seen as downstream, and #127 already records that we don't change ASV's text.

Still not done from this issue: comparing with diatheke's output for Ps 119:126–128, which would confirm it is a module defect. Do that when the CrossWire report is drafted (#127: the owner writes or signs off any message to CrossWire).

🤖 Generated with Claude Code

**Owner decision (2026-11-03):** deferred out of v0.1. The raw module has no space at these boundaries, so the fix is seen as downstream, and #127 already records that we don't change ASV's text. Still not done from this issue: comparing with diatheke's output for Ps 119:126–128, which would confirm it is a module defect. Do that when the CrossWire report is drafted (#127: the owner writes or signs off any message to CrossWire). 🤖 Generated with [Claude Code](https://claude.com/claude-code)

Rows
Columns

ASV: words run together around added words (transChange) — "allthyprecepts", "AndI" #119