humanspeak / svelte-markdown / 29035531576 / 1

94%

main: 95%

push

github

refactor(cleanup): drop dead legacy HTML functions and bench module (#351)

* refactor(cleanup): drop dead legacy HTML functions and bench module

Remove production-dead code from the published surface:

- Delete parseHtmlBlock, processHtmlTokens, containsMultipleTags from
  token-cleanup.ts. All three had zero non-test importers and were
  superseded by the single-pass expandHtmlBlockNested / pairFlatHtmlTokens
  path, leaving a duplicate stack-based pairing implementation free to
  drift from the live one.
- Delete their dedicated test files and trim the cases in
  token-cleanup.test.ts that exercised them.
- Delete stream-benchmark.ts and its test; the benchmark helper had no
  production importer but still shipped in the package. Its lone consumer,
  the perf-bench dev route, now inlines the ~20-line append-parse helper.

Refresh stale doc comments that referenced the removed functions.

* docs(guard): checkpoint 1 for plan 008 — ON TRACK

Snapshot c974b80 reviewed against 008-remove-dead-html-code. All five
Done criteria reproduced in-tree. Drift check was non-empty (plan 007
touched token-cleanup.ts after planning), so every "Current state" fact
was re-verified at f5f1e4c before accepting no STOP was skipped.

Records one deviation from Step 3's letter: the executor inlined a port
of benchmarkAppendStream into the dev route rather than deleting the UI.
Accepted on spirit, surfaced to the operator.

* docs(guard): close-out report for plan 008 — PASS

Final gate at c974b80. All eight Done criteria re-run from scratch and
reproduced green, including a clean-build dist audit confirming neither
the bench module nor any deleted symbol ships.

Key evidence: the diff is a pure deletion — every added line in src/lib
is a comment or a trimmed import, so no live-path logic changed.

Records the operator-ratified Step 3 deviation, the non-empty drift check
(facts re-verified at f5f1e4c, no STOP skipped), and the check that
#350 on main reintroduces no depe... (continued)

843 of 940 branches covered (89.68%)

Branch coverage included in aggregate %.

1391 of 1430 relevant lines covered (97.27%)

4721.11 hits per line