Summary

Defer initialization of the Handlebars template engine and syntax highlighter to their first use, reducing application startup time by avoiding expensive upfront work.

Context

ForgeTemplateService and MarkdownFormat previously initialized their heaviest dependencies — the Handlebars engine and the SyntaxHighlighter — eagerly at construction time. These initializations are non-trivial in cost and are not always needed immediately (or at all) during a given session, so paying for them unconditionally at startup was wasteful.

Changes

• ForgeTemplateService: Replaced the eagerly-initialized Arc<RwLock<Handlebars>> with Arc<OnceCell<RwLock<Handlebars>>>. Added a private get_hb() async helper that creates the Handlebars instance on the first call and returns a reference to it on all subsequent calls. The new() constructor now does no heavy work.
• MarkdownFormat: Changed the highlighter field from a plain SyntaxHighlighter to a OnceLock<SyntaxHighlighter>. The highlighter is created via get_or_init the first time format() is called, leaving construction of MarkdownFormat itself lightweight.

Key Implementation Details

Both changes use the standard lazy-initialization primitives idiomatic to their execution context:

• tokio::sync::OnceCell for the async ForgeTemplateService (supports .await in the initializer closure).
• std::sync::OnceLock for the synchronous MarkdownFormat (no async required).

The public API of both types is unchanged — callers see no difference in behavior.

Testing

# Run template service tests
cargo insta test --accept -p forge_services

# Run display/markdown tests
cargo insta test --accept -p forge_display

# Quick type-check across the workspace
cargo check

Fixes #2574