Skip to content

fix(status-pages): import escapeHtml from @maple/strings in markdown.ts - #2468

Merged
JeremyFunk merged 1 commit into
mainfrom
fix/status-page-markdown-escape-import
Nov 10, 2026
Merged

JeremyFunk merged 1 commit into
mainfrom
fix/status-page-markdown-escape-import

fix(status-pages): import escapeHtml from @maple/strings in markdown.ts

4893b3f
Select commit
Loading
Failed to load commit list.
Maple Review Bot / Maple / review succeeded Nov 10, 2026 in 35s

Confidence 10/10 · No issues found

🟢 Confidence 10/10 · safe to merge. Import-only fix; escapeHtml resolves from @maple/strings, a declared dependency that escape.ts already imports from.
no findings · tests covered · risk low

Restores main's status-page build by importing escapeHtml from @maple/strings, where #1438 moved it, instead of the now-deleted re-export in ./escape. Safe to merge.

  • markdown.ts imports escapeHtml from @maple/strings; ./escape keeps safeUrl
What was checked
  • escape.ts at head no longer exports escapeHtml; the only remaining ./escape imports across status-page are escapeXml, safeUrl and safeColor
  • lib/strings' escapeHtml escapes & < > \" ', matching what markdown.ts needs in both element and quoted-attribute positions
  • @maple/strings is already a workspace:* dependency of @maple/status-page, so no manifest change is needed

4893b3f · Reply "won't fix" to dismiss a finding, or mention @maple-review-bot to ask about one.