Skip to content

fix(core): flank strikethrough tildes like asterisk emphasis, matching GitHub - #11

Merged
juliusmarminge merged 1 commit into
pingdotgg:fork/t3-mobilefrom
ahalekelly:fix/strikethrough-flanking
Nov 9, 2026
Merged

juliusmarminge merged 1 commit into
pingdotgg:fork/t3-mobilefrom
ahalekelly:fix/strikethrough-flanking

Conversation

@ahalekelly

@ahalekelly ahalekelly commented Oct 29, 2026 •

Copy link
Copy Markdown

🤖 Generated with Claude Opus 5.5

Carries software-mansion-labs/md4c#6 in packages/core/cpp/enrmrkd/enrmrkd.c, so T3 Code mobile strikes the same tildes web and GitHub do. ~$0.48 vs ~$0.65 per run and files in ~/foo and ~/bar stop striking, and a~~b~~c starts. The md4c PR has the full comparison and tests; packages/core tests pass here.

Not included: the web wasm bundle (src/web/wasm/md4c.js) still needs an Emscripten rebuild. T3 Code doesn't use it, so I left it for your next bundle rebuild rather than commit a binary built on my machine.

Follows up pingdotgg/t3code#18156, which will become the tarball bump.

Note

Fix strikethrough flanking to match asterisk emphasis, like GitHub

Changes the inline-mark collector in enrmrkd.c so strikethrough runs of one or two tildes get opener and closer flags like * emphasis. Each side of the run gets a level of 0, 1, or 2 based on whitespace, punctuation, or other characters. A run is a potential closer when the left level is positive and at least the right level, and a potential opener when the right level is positive and at least the left level.

Macroscope summarized d46f457.

…g GitHub

Carries software-mansion-labs/md4c fix/strikethrough-flanking. md4c's
tilde opener and closer checks each looked at only one side of the run,
so `~$0.48 vs ~$0.65` and `~/foo and ~/bar` struck through on mobile
while web (remark-gfm) and GitHub leave them plain, and intraword
`a~~b~~c` did not strike.

Co-Authored-By: Claude Opus 5.5 <[email protected]>
@coderabbitai

coderabbitai Bot commented Oct 29, 2026

Copy link
Copy Markdown

Important

Review skipped

Auto reviews are disabled on base/target branches other than the default branch.

Please check the settings in the CodeRabbit UI or the .coderabbit.yaml file in this repository. To trigger a single review, invoke the @coderabbitai review command.

Run configuration
  • Source mode: Auto
  • Repository: Unavailable
  • Branch: Unavailable
  • File: No configuration file from this repository applied
  • Commit: Unavailable
  • Layers: Organization UI
  • Review profile: CHILL
  • Plan: Advanced
  • Run ID: 7f01e62f-8346-41d0-a258-e2002b56f69f

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

Use the checkbox below for a quick retry:

  • 🔍 Trigger review
  • Autofix · Keep fixing CodeRabbit findings and required CI, and resolving merge conflicts

Comment @coderabbitai help to get the list of available commands.

@juliusmarminge
juliusmarminge merged commit d23a608 into pingdotgg:fork/t3-mobile Nov 9, 2026
2 checks passed
@juliusmarminge

Copy link
Copy Markdown
Member

Thanks! Landed on fork/t3-mobile with the golden AST re-recorded (H~2~O now strikes its middle when subscript is off, as on GitHub) and the WASM bundle rebuilt. It ships in T3 Code's 1.2.0-t3.18 archive (pingdotgg/t3code#32180). Upstream stays software-mansion-labs/md4c#6.

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.

2 participants