Skip to content

fix: avoid quadratic slugify cache lookup - #2783

Merged
sy-records merged 1 commit into
docsifyjs:developfrom
sy-records:fix/2782
Aug 11, 2026
Merged

fix: avoid quadratic slugify cache lookup#2783
sy-records merged 1 commit into
docsifyjs:developfrom
sy-records:fix/2782

Conversation

@sy-records

Copy link
Copy Markdown
Member

Summary

Related issue, if any:

Fixed #2782

What kind of change does this PR introduce?

  • Bugfix
  • Feature
  • Code style update (formatting, renaming)
  • Refactoring (no functional changes, no api changes)
  • Build related changes
  • Documentation content changes
  • Other (please describe): perf

For any code change,

  • Related documentation has been updated, if needed
  • Related tests have been added or updated, if needed

Does this PR introduce a breaking change?

  • Yes
  • No

Tested in the following browsers:

  • Chrome
  • Firefox
  • Safari
  • Edge

@vercel

vercel Bot commented Aug 10, 2026

Copy link
Copy Markdown

@sy-records is attempting to deploy a commit to the Docsify Team on Vercel.

A member of the Team first needs to authorize it.

@vercel

vercel Bot commented Aug 10, 2026

Copy link
Copy Markdown

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
docsify-preview Ready Ready Preview Aug 10, 2026 9:43am

@sy-records
sy-records merged commit a8f151e into docsifyjs:develop Aug 11, 2026
8 checks passed
@sy-records
sy-records deleted the fix/2782 branch August 11, 2026 03:24
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.

performance issue: slugify() is O(n^2) in the number of headings on a page, search growth is quadratic

1 participant