Repository navigation
Conversation
react-diff-viewer styles changed words as inline-flex with padding, so each Prism token inside became a flex item and words rendered as "c on st use r" in the diff preview. Claude-Session: https://claude.ai/code/session_01Yb4sbCDrvzqV8fFpnCrQK5
…g a draft release otool -L prints an unindented header per architecture for a universal binary; the check counted it as a non-system library and failed the v2.4.0 build after signing and notarization had passed. Only indented library lines are checked now. workflow_dispatch takes an optional tag to finish an existing draft release with the current workflow, since re-running a failed run replays the workflow file it started with. Claude-Session: https://claude.ai/code/session_01Yb4sbCDrvzqV8fFpnCrQK5
The desktop app is the recommended way to use GitContext; the web version is presented as a way to try it in the browser. Claude-Session: https://claude.ai/code/session_01Yb4sbCDrvzqV8fFpnCrQK5
Codex Review SummaryThis comment shows the latest Codex review activity on this pull request.
ℹ️ About Codex in GitHubYour team has set up Codex to review pull requests in this repo. Reviews are triggered when you
Codex reacts with 👀 while any review is running, comments if it has suggestions, and reacts with 👍 once all reviews finish with no findings. |
![chatgpt-codex-connector[bot]](https://avatars.githubusercontent.com/in/1144995?s=60&v=4){kind=small}
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: 9806a8024f
ℹ️ About Codex in GitHub
Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".
| } as any, | ||
| // The library's inline-flex + padding turns each Prism token inside a | ||
| // changed word into a flex item, splitting words ("c on st"). | ||
| wordDiff: { display: 'inline', padding: 0 }, |
There was a problem hiding this comment.
Add a regression test that renders or mocks ReactDiffViewer with syntax-highlighted changed words and verifies this override is supplied. This is a user-visible behavior fix for the case where Prism splits a changed word across multiple token spans, but the commit adds no test to prevent a later style or dependency change from reintroducing the broken rendering, contrary to the repository's testing requirement for every behavior change.
AGENTS.md reference: AGENTS.md:L11-L12
Useful? React with 👍 / 👎.
|
🎉 This PR is included in version 2.4.1 🎉 The release is available on GitHub release Your semantic-release bot 📦🚀 |
1 participant
otool -Larchitecture headers on a universal binary).fix(ui): word-level diff highlights no longer split words. This makes the merge cut v2.4.1 through the fixed pipeline, which also ships v2.4.0's compare-by-commit feature.workflow_dispatchcan finish an existing draft release by tag.Merge with "Create a merge commit".
https://claude.ai/code/session_01Yb4sbCDrvzqV8fFpnCrQK5