fix: stop a slow reply from pinning a word to hiragana

Making a conversion timeout an ordinary exception put it in the same arm as a
hard API failure, where caching the hiragana fallback is deliberate - so one slow
reply recorded the unconverted form and that word rendered as hiragana for the
life of the cache. A timeout says the request was slow, not that the word has no
conversion, so it now returns the fallback without caching it and the next
message asks again.

The retry test builds its own remembering cache: the shared fixture's get()
always returns null, so against it the API is called every time and the test
would have passed even with the timeout cached.

Co-Authored-By: Claude <noreply@anthropic.com>

1 files changed, 7 insertions, 0 deletions

diff --git a/platform-paper/src/main/kotlin/dev/m1sk9/lunaticChat/paper/converter/RomanjiConverter.kt b/platform-paper/src/main/kotlin/dev/m1sk9/lunaticChat/paper/converter/RomanjiConverter.kt
index b502d63..e1a11f8 100644
--- a/platform-paper/src/main/kotlin/dev/m1sk9/lunaticChat/paper/converter/RomanjiConverter.kt
+++ b/platform-paper/src/main/kotlin/dev/m1sk9/lunaticChat/paper/converter/RomanjiConverter.kt
@@ -93,6 +93,13 @@ class RomanjiConverter(
                 // pin every word of the message to its unconverted form for good, because the words
                 // are converted concurrently and the timeout cancels all of them at once.
                 throw e
+            } catch (e: ConversionTimeoutException) {
+                // Returned without caching: a timeout says the request was slow, not that the word
+                // has no conversion, so recording the hiragana would pin it for the life of the
+                // cache over one slow reply. A hard API failure is different - the fallback is
+                // cached there deliberately.
+                logger.warning("Timed out converting $hiragana, leaving it uncached: ${e.message}")
+                return hiragana
             } catch (e: Exception) {
                 logger.warning("Failed to convert $hiragana: ${e.message}")
                 hiragana // Use hiragana if API fails