[maksg]

What/Why?

Testing

PR Checklist

• Code compiles and runs on iOS
• Code compiles and runs on Android
• Updated documentation/README if applicable
• Ran example app to verify changes
• E2E tests are passing
• Required E2E tests have been added (if applicable)

[jnowakow]

During the review I found a few issues that are worth looking at:

1. Some formulas sound identical when they mean different things. Argument.isSimple is just itemCount == 1, but one item can be a whole fraction, root or power, so nesting disappears from the speech. Actual outputs:
   • e^{\frac{x}{2}} and \frac{e^x}{2} both give "e to the power x over 2"
   • \sqrt{\frac{a}{b}} and \frac{\sqrt{a}}{b} both give "square root of a over b"

This affects ENRMLaTeXSpeech.swift, LatexSpeech.kt and the upstream enriched-markdown-ios LaTeXSpeech.swift.

2. Math inside a link is read wrong on both platforms. md4c nests the math inside the link ([see $x^2$ here](u) gives Link > [Text, LatexMathInline, Text]).

   • iOS: segmentedElementsForParagraph sets segmentStart = NSMaxRange(mathRange), which moves backwards. VoiceOver reads the link "see  here", then "Math: x squared", then "here" again as plain text.
   • Android: if (span.start < segmentPos) continue skips the math span, so the formula is silent.

3. Multi-digit numbers are read as one number, but LaTeX takes only one digit.

   • \frac12 gives "fraction 12 over end fraction" and the denominator is lost.
   • x^23 renders as x²3 but is spoken "x to the power 23".
   • x_12 gives "x sub 12".

4. Common constructs are misread:

   • \sum\limits_{i=1}^n gives "sum of limits sub i equals 1 end sub to the power n". \int\limits has the same problem.
   • \operatorname*{argmax} gives "times argmax".
   • In pmatrix/cases, the environment names are read aloud and & and \\ are silent, so rows run together.

5. The new speech tests break builds with math turned off:

   • Android: src/test/.../math/LatexSpeechTest.kt imports LatexSpeech, which only joins the source set when enableMath is on (build.gradle). With math off, the whole unit-test source set fails to compile.
   • iOS: ENRMLaTeXSpeechTests.mm would fail because {speech} stays as literal text.
   • CI never builds with math off, so this is currently hidden.

I would also consider implementing this functionality as a shared C++ code. Current solution works but any future change has to be applied in two places.