Conversation
There was a problem hiding this comment.
Code Review
This pull request refactors the document parsing logic by introducing helper utilities in document_utils.ts and error.ts, and adding dedicated parser classes for Kitchener Utilities bills and Morgan Stanley release confirmations. It also removes the old standalone CLI scripts. The reviewer feedback focuses on improving the robustness of the regular expressions used for parsing (such as handling flexible decimal places, trailing carriage returns, and whitespaces), avoiding mutating function parameters in prefixMessage, and optimizing stringFromLines to prevent double regex execution.
Important
The consumer version of Gemini Code Assist on GitHub is being sunset. Starting June 18, 2026, new organization installations will be blocked, and all code review activity will officially cease on July 17, 2026.
For more details on the timeline and next steps, please review the Help Documentation.
|
/gemini review |
There was a problem hiding this comment.
Code Review
This pull request refactors the PDF parsing logic by introducing a shared utility module (document_utils.ts) and implementing structured document parsers for Kitchener Utilities bills and Morgan Stanley release confirmations, while removing the legacy standalone scripts. The review feedback focuses on improving the robustness of the regular expressions used for parsing numbers, dates, and amounts (such as making decimal places and parentheses optional) and correcting a truthiness check in the utility function to safely handle empty string matches.
Important
The consumer version of Gemini Code Assist on GitHub is being sunset. Starting June 18, 2026, new organization installations will be blocked, and all code review activity will officially cease on July 17, 2026.
For more details on the timeline and next steps, please review the Help Documentation.
No description provided.