[chrikrah]

Fixes #919.

WebVTTFile.parse dropped any cue holding a reference outside the six names of the syntax table, such as ' or é. Cue text and annotations now go through html.unescape, as @ceberam laid out in the issue. An unknown reference or a bare & stays literal text. _escape is untouched.

One addition: 
 is still rejected. It resolves to a line break, and the cue would then be written back across two lines.

$ python -m pytest tests/test_webvtt.py -q      # base b501ec8: 11 passed
12 passed in 0.11s
$ python -m pytest tests -q
1108 passed, 6 skipped, 1 xfailed, 3 warnings in 83.16s      # base b501ec8: 1107 passed

Reverting webvtt.py and keeping the tests fails test_webvtt_cue_html_character_references, and test_vtt_cue_commponents for its two invalid-entity cases, which now assert literal text. ruff 0.16.10 check and format are clean.

$ uv run --no-sync ty check      # ty 0.0.33, base b501ec8: exit 0, 469 diagnostics
Found 469 diagnostics      # exit 0, no errors

Signed off for DCO. A cue holding 
 is still dropped with a warning, as before, and keeping it needs _escape to write line breaks back. I'd leave that until a real file has one. Fine with you?

[github-actions]

✅ DCO Check Passed

Thanks @chrikrah, all your commits are properly signed off. 🎉

[mergify]

Merge Protections

🔴 1 of 2 protections blocking · waiting on 👀 reviews

	Protection	Waiting on
🔴	Require two reviewer for test updates	👀 reviews
🟢	Enforce conventional commit	—

🔴 Require two reviewer for test updates

Waiting for

• #approved-reviews-by >= 2

This rule is failing.

When test data is updated, we require two reviewers

• #approved-reviews-by >= 2

---

Show 1 satisfied protection

🟢 Enforce conventional commit

Make sure that we follow https://www.conventionalcommits.org/en/v1.0.0/

• title ~= ^(fix|feat|docs|style|refactor|perf|test|build|ci|chore|revert)(?:\(.+\))?(!)?: