chore(skills): flag dead clipboard-pipeline emitter bindings in ckeditor5-reviewing

Add a defect-pattern: a handler bound to editor.plugins.get(Clipboard) for
inputTransformation/contentInsertion is silently dead — those events fire on
ClipboardPipeline, which Clipboard only requires.

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
This commit is contained in:
Elian Doran 2026-06-13 20:49:25 +03:00
parent f92eb3c65c
commit f8c10bd88f
No known key found for this signature in database

View File

@ -47,6 +47,13 @@ group covers the monorepo wiring/convention defects. For the idiomatic "how it s
- Fix: declare the attribute to trigger reconversion, or use a separate `attributeToAttribute`/
`attributeToElement` converter, or `editor.editing.reconvertItem(item)`.
**Clipboard-pipeline handler bound to the wrong emitter (silently dead).**
- Spot: `listenTo(editor.plugins.get(Clipboard), 'inputTransformation'|'contentInsertion', …)` — the
handler is attached to the `Clipboard` *umbrella* glue plugin (a real one in `ckeditor5-math/src/automath.ts:28`).
- Why: those events fire on **`ClipboardPipeline`**, not `Clipboard` (which only `requires` it), so the
handler never runs — no error, no test failure (a Trilium handler sat dead ~1 year this way).
- Fix: `listenTo(editor.plugins.get(ClipboardPipeline), 'inputTransformation', …)`.
## Schema
**Object/limit flags wrong.**