dcdpr / jp / 26806865505

67%

Build Type

push

github

Committed by web-flow

Commit Message

perf(cli, md): Improve `jp c print` performance (#718)

`two_face::syntax::extra_newlines` deserializes the full `SyntaxSet`
from its embedded binary dump and returns a fresh instance on every
call. Syntect's grammar regexes are lazily compiled, so each new
instance starts cold, causing `fancy_regex::compile` to fire thousands
of times across a long conversation's code blocks.

This commit introduces a single `static SYNTAXES: LazyLock<SyntaxSet>`
and replaces every call-site in `format.rs` and `render.rs` with a
reference to it. The regexes are now compiled once per process and
reused for all subsequent highlighting.

The impact on `jp c print --style=full` for a long conversation:

| Metric                         |  Before |  After |
|--------------------------------|--------:|-------:|
| Wall clock                     | 21.25 s | 0.73 s |
| `fancy_regex::compile` samples | 73,970  |     41 |

Signed-off-by: Jean Mertz <git@jeanmertz.com>

Coverage Stats

4 of 4 new or added lines in 2 files covered. (100.0%)

8 existing lines in 2 files now uncovered.

33395 of 50221 relevant lines covered (66.5%)

251.52 hits per line