Skip to content

Render grounded explanations as rich Markdown and LaTeX - #22

Merged
jjjhenriksen merged 1 commit into
mainfrom
codex/explanation-rendering
Jul 20, 2026
Merged

jjjhenriksen merged 1 commit into
mainfrom
codex/explanation-rendering

Conversation

@jjjhenriksen

Copy link
Copy Markdown
Owner

Summary

  • render model explanations as structured Markdown with inline and display LaTeX
  • normalize the delimiter forms commonly emitted by models
  • remove revision-hash/debug metadata from the learner-facing verification note
  • replace generic prerequisite copy with concise, proof-specific explanations
  • refresh the recorded example and Devpost narrative to match the educational product vision

Why

The explanation panel was showing literal math delimiters and implementation-oriented verification details. That made the experience feel like a debug console instead of a source-aware learning companion. This change makes the explanation itself the teaching surface while preserving evidence and trust boundaries.

Validation

  • npm run typecheck
  • npm run lint
  • npm test (32 files, 121 tests)
  • npm run proof:validate
  • npm run build
  • npm run test:e2e -- --workers=1 (35 passed, 5 skipped)
  • targeted recorded-example e2e (2 passed)
  • local Chrome visual QA confirmed inline and display KaTeX rendering

Production has not been redeployed from this branch; it remains on the previously reviewed deployment until merge.

@jjjhenriksen
jjjhenriksen marked this pull request as ready for review July 20, 2026 03:49
@chatgpt-codex-connector

Copy link
Copy Markdown

You have reached your Codex usage limits for code reviews. You can see your limits in the Codex usage dashboard.
To continue using code reviews, you can upgrade your account or add credits to your account and enable them for code reviews in your settings.

@jjjhenriksen
jjjhenriksen merged commit 4bd2963 into main Jul 20, 2026
1 check passed
@jjjhenriksen
jjjhenriksen deleted the codex/explanation-rendering branch July 20, 2026 03:49
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant