Update markdown-it JS package #4475

New Issue

OVERLORD · 2026-02-05T08:57:39+03:00

OVERLORD commented

2026-02-05 08:57:39 +03:00

Originally created by @ssddanbrown on GitHub (Feb 20, 2024).

Have been holding this back since it inflates bundle size by 10%, so need to see if we can properly address.
Don't think ESBuild is properly tree shaking "entities" sub package which inflates size.

I think we should be able to remove the punycode package from our own package.json, since I think this is now a specified dependency of markdown-it.

Originally created by @ssddanbrown on GitHub (Feb 20, 2024). Have been holding this back since it inflates bundle size by 10%, so need to see if we can properly address. Don't think ESBuild is properly tree shaking "entities" sub package which inflates size. I think we should be able to remove the `punycode` package from our own package.json, since I think this is now a specified dependency of markdown-it.

OVERLORD added the 🔧 Maintenance 💻 Front-End labels 2026-02-05 08:57:39 +03:00

OVERLORD closed this issue

2026-02-05 08:57:39 +03:00

OVERLORD commented

2026-02-05 08:57:43 +03:00

@ssddanbrown commented on GitHub (Apr 8, 2024):

Decided to split out MD code instead since, when coming back to this, that's the obvious route with md code using up half of the main app bundle size. Related PR: #4930
10% as separate bundle is not really significant.

@ssddanbrown commented on GitHub (Apr 8, 2024): Decided to split out MD code instead since, when coming back to this, that's the obvious route with md code using up half of the main app bundle size. Related PR: #4930 10% as separate bundle is not really significant.

Sign in to join this conversation.

Branches Tags

1 Participants

Notifications

Due Date

No due date set.

Dependencies

No dependencies set.

Reference: starred/BookStack#4475