Greptile Summary

This PR fixes three locations where numeric values were concatenated directly with "em" instead of using the existing makeEm() utility, which rounds to 4 decimal places and removes trailing zeros. The result is cleaner, consistent CSS output (e.g. 0.1176em instead of 0.11764999999999999em).\n\nKey changes:\n- src/domTree.ts — SymbolNode.toMarkup(): this.italic + "em" → makeEm(this.italic) for margin-right\n- src/delimiter.ts — makeStackedDelim(): .toFixed(3) + "em" → makeEm() for SVG width/height attributes (minor precision bump from 3→4 dp, behaviorally identical for integer/1000 inputs)\n- src/functions/enclose.ts — mathmlBuilder: thk + "em" → makeEm(thk) for the \\fcolorbox border style\n\nAll three files already import makeEm from ../units, so no new imports are required. The fix is minimal, correct, and consistent with how the rest of the codebase produces em values.

Confidence Score: 5/5

Safe to merge — all three changes are correct, imports are already present, and no logic is altered beyond decimal truncation.

The changes are tiny and mechanical: swap raw string concatenation for an already-imported helper that's used identically everywhere else in the codebase. makeEm already exists in all three files' import lists. The only behavioural delta in delimiter.ts (3→4 dp) is inconsequential because the inputs are always integers divided by 1000, giving at most 3 significant decimal digits. No new risk is introduced.

No files require special attention.

Important Files Changed

Flowchart

%%{init: {'theme': 'neutral'}}%%
flowchart TD
    A["Raw number\n(e.g. 0.11764999999...)"] -->|"old: + \"em\""| B["Long CSS string\n0.11764999999999999em"]
    A -->|"new: makeEm(n)"| C["makeEm(n)\n+n.toFixed(4) + 'em'"]
    C --> D["Clean CSS string\n0.1176em"]

    subgraph "Call sites fixed"
        E["domTree.ts\nSymbolNode.toMarkup()\nmargin-right: italic"]
        F["delimiter.ts\nmakeStackedDelim()\nSVG width / height"]
        G["enclose.ts\nmathmlBuilder()\nborder thickness thk"]
    end

    E --> A
    F --> A
    G --> A

_{Reviews (1): Last reviewed commit: "fix: use makeEm() consistently to trunca..." | Re-trigger Greptile}

Could you please also update snapshots?

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 93.30%. Comparing base (314f856) to head (08c8746).
⚠️ Report is 109 commits behind head on main.

@@            Coverage Diff             @@
##             main    #4181      +/-   ##
==========================================
+ Coverage   93.15%   93.30%   +0.14%     
==========================================
  Files          91       91              
  Lines        6795     6810      +15     
  Branches     1582     1582              
==========================================
+ Hits         6330     6354      +24     
- Misses        429      454      +25     
+ Partials       36        2      -34     

Thanks for the review @grigoriy-reshetniak! I've updated the PR:

• Switched to template literals in both domTree.ts and enclose.ts
• Updated the snapshot (0.10903em → 0.109em)

Let me know if anything else needs changing.

Looks good, thank you!

🎉 This PR is included in version 0.16.43 🎉

The release is available on:

• npm package (@latest dist-tag)
• GitHub release

Your semantic-release bot 📦🚀