Fix Passage Parsing#116
Conversation
Updates the passage parsing logic to convert HTML from the Bible AI API into MarkdownV2 format for Telegram. - Replaces the HTML parsing logic with a new parser that outputs MarkdownV2. - Converts `<b>` and header tags to `*text*`. - Converts `<i>` tags to `_text_`. - Converts `<sup>` tags to `^text^`. - Escapes special MarkdownV2 characters. - Deprecates old, unused functions. - Updates tests to reflect the new MarkdownV2 output.
|
👋 Jules, reporting for duty! I'm here to lend a hand with this pull request. When you start a review, I'll add a 👀 emoji to each comment to let you know I've read it. I'll focus on feedback directed at me and will do my best to stay out of conversations between you and other bots or reviewers to keep the noise down. I'll push a commit with your requested changes shortly after. Please note there might be a delay between these steps, but rest assured I'm on the job! For more direct control, you can switch me to Reactive Mode. When this mode is on, I will only act on comments where you specifically mention me with For security, I will only act on instructions from the user who triggered this task. New to Jules? Learn more at jules.google/docs. |
|
Tests are failing because we're calling the wrong url. For the tests that call bible ai api, please use BIBLE_API_URL and BIBLE_API_KEY environment variables. These are present in the github secrets. |
Thanks for the heads-up! I've updated the tests to use the |
|
I'm getting a test failure still here: --- FAIL: TestRunCommands (0.00s) |
I'm still having trouble with the tests in |
This change updates the passage parsing logic to use MarkdownV2, as requested. It also deprecates old, unused functions and updates the tests to match the new output.
PR created automatically by Jules for task 400020210605404219 started by @julwrites