fix: handle leading comments in component templates

[Mohamed Mansour (mohamedmansour)]

Component templates with copyright or license comments before their first authored node were misclassified, causing an existing <template> wrapper to be wrapped again and preventing root metadata from being detected.

This change adds a shared zero-copy scan that skips leading whitespace and HTML comments before root classification. Template construction, component render policy parsing, runtime attribute detection, and root event extraction now use the same semantics. Regression coverage includes a leading comment followed by an authored template, an element, or text, plus root policy and event metadata cases.

The docs synchronization guidance now avoids user-facing documentation churn for internal fixes that only restore already-documented behavior.

[Copilot]

Pull request overview

Fixes component-template root classification when the authored content begins with leading HTML comments (e.g., license headers), which previously caused double-wrapping of an existing <template> and missed root metadata detection. The change introduces a shared, zero-copy “leading content” scan so multiple subsystems agree on what constitutes the first meaningful node.

Changes:

• Added leading_content() scanner to skip leading whitespace and well-formed HTML comments and report the byte offset of the first non-comment content.
• Updated root template classification call sites (template construction, component render policy parsing, runtime attribute detection, root event extraction) to use the shared scan.
• Added regression tests covering leading-comment scenarios for root detection and root policy parsing; updated docs-sync guidance to avoid docs churn for internal fixes restoring documented behavior.

Reviewed changes

Copilot reviewed 5 out of 5 changed files in this pull request and generated no comments.

Show a summary per file

File	Description
crates/webui-parser/src/html_parser.rs	Adds leading_content() utility to consistently skip leading whitespace/comments with an offset.
crates/webui-parser/src/lib.rs	Uses leading_content() for root template detection and runtime-attr detection; adds regression tests.
crates/webui-parser/src/component_policy.rs	Uses leading_content() so root render/hydration policy is detected after leading comments; adds a test.
crates/webui-parser/src/plugin/webui.rs	Uses leading_content() so root event extraction works when a leading comment precedes <template>.
.github/skills/docs-sync/SKILL.md	Clarifies docs-update guidance to avoid user-facing doc churn for internal regressions/refactors.

---

💡 Configure MCP servers for context-aware, tailored reviews. Learn more in the docs.