Skip to content

Fix/2109#316

Open
conradmugabe wants to merge 3 commits into
mainfrom
fix/2109
Open

Fix/2109#316
conradmugabe wants to merge 3 commits into
mainfrom
fix/2109

Conversation

@conradmugabe

Copy link
Copy Markdown
Contributor

Checklist

  • Tests were added/updated according to the feature/bugfix/change made
  • Version was rolled according to semver requirements
  • API endpoints openapi schema was updated if applicable

Changes

Screenshots

Screencast.from.2026-07-07.19-12-28.mp4

conradmugabe and others added 3 commits July 7, 2026 19:00
Prettier rewrites the auto-generated release-notes bullets from * to -.
Isolated here so the fix commits stay free of unrelated formatting drift.

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
The currency-escape step escaped any $ followed by a digit, which
corrupted math delimiters whose content opened with a number
($$0.075...$$, $250 \text{ mL}...$), so KaTeX rendered them broken.
Mask $$...$$ blocks and inline $...$ spans containing a LaTeX command
before escaping, then restore them verbatim, so real currency is still
escaped while math delimiters survive.

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
Unit tests assert preprocessLaTeX keeps $$...$$ and inline $...$ math
delimiters intact (digit-leading, leading-space, mixed with currency)
while still escaping real currency. Component tests render the failing
equations through Markdown and assert a .katex node with no raw $$ leak.

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
@conradmugabe conradmugabe self-assigned this Jul 7, 2026
@conradmugabe conradmugabe requested a review from mamigot as a code owner July 7, 2026 16:15
@conradmugabe conradmugabe added the run-tests Trigger PR validation pipeline label Jul 7, 2026
@mamigot

mamigot commented Jul 7, 2026

Copy link
Copy Markdown
Member

Claude finished @conradmugabe's task —— View job


E2E Coverage Review

  • Dismiss stale REQUEST_CHANGES reviews (none found)
  • Identify changed files
  • Run mechanical coverage check (503 checkpoints, no regression)
  • Apply decision logic and post review

Result: No new app/**/page.tsx routes were introduced. All existing coverage is intact. PR approved.

--- | Branch: fix/2109

@mamigot mamigot left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

✅ E2E Coverage — No Issues

No new app routes were introduced. All existing coverage is intact.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

run-tests Trigger PR validation pipeline

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants