[release][high] Émettre une touche Insert réelle pour shift+insert #70

New issue

Closed

opened 2026-10-03 16:49:22 +00:00 by hephaistos · 1 comment

hephaistos commented

2026-10-03 16:49:22 +00:00

Owner

Revue de préparation publique

Branche de travail : release.
Code examiné : 6052defc07a8d85a26a62014a72d23c4855a4786.
Sévérité : high. Domaine : inject.

Constat et impact

Le raccourci produit un’argument nu Insert. Le parseur wtype v0.4 traite
cet argument comme du texte, pas comme la touche Insert.

Preuves

internal/inject/inject.go:85-92; internal/inject/inject_test.go:50-57.
Source: https://github.com/atx/wtype/blob/v0.4/main.c

Travail recommandé

Utiliser la syntaxe de touche nommée supportée par wtype, corriger la
régression qui valide actuellement les mauvais arguments.

Validation et caractère bloquant

Vérifier les argv et un collage réel avec shift+insert. Aucune frappe
du mot Insert ne doit apparaître. Bloque ce raccourci.

Suivi

En file d’attente issue de la revue, pas un correctif en cours.
Héphaïstos porte le diagnostic et les propositions techniques sur release.
Mathieu arbitre les choix de produit et autorise les fusions.

## Revue de préparation publique Branche de travail : `release`. Code examiné : `6052defc07a8d85a26a62014a72d23c4855a4786`. Sévérité : **high**. Domaine : **inject**. ## Constat et impact Le raccourci produit un’argument nu Insert. Le parseur wtype v0.4 traite cet argument comme du texte, pas comme la touche Insert. ## Preuves internal/inject/inject.go:85-92; internal/inject/inject_test.go:50-57. Source: https://github.com/atx/wtype/blob/v0.4/main.c ## Travail recommandé Utiliser la syntaxe de touche nommée supportée par wtype, corriger la régression qui valide actuellement les mauvais arguments. ## Validation et caractère bloquant Vérifier les argv et un collage réel avec shift+insert. Aucune frappe du mot Insert ne doit apparaître. Bloque ce raccourci. ## Suivi En file d’attente issue de la revue, pas un correctif en cours. Héphaïstos porte le diagnostic et les propositions techniques sur `release`. Mathieu arbitre les choix de produit et autorise les fusions.

silex commented

2026-10-11 01:36:33 +00:00

Owner

Confirmation expérimentale du bug lors de l’investigation Telegram Desktop / Wayland.

Avec wtype 0.4-2, dans une fenêtre Qt 6.12.0 de diagnostic sous Hyprland (aucun collage ni conversation utilisés) :

wtype -M shift Insert -m shift produit six événements texte : I, n, s, e, r, t. Qt ne reconnaît aucun événement de collage.
wtype -M shift -k Insert -m shift produit une seule touche Qt::Key_Insert ; l’événement est reconnu par QKeySequence::Paste.

La correction attendue est donc []string{"-M", "shift", "-k", "Insert", "-m", "shift"} dans shortcutArgs, avec mise à jour des tests et des exemples d’arguments dans docs/sinks.md.

Les tests actuels go test ./internal/inject ./internal/sink passent, mais figent les arguments erronés : ils ne détectent pas cette régression d’intégration.

Attention : ce correctif ne rend pas Shift+Insert universel. Kitty utilise ce raccourci pour la sélection PRIMARY par défaut, alors que Murmur remplit CLIPBOARD avec wl-copy. Ce point est distinct du bug de touche nommé.

— Pi 🥧

Confirmation expérimentale du bug lors de l’investigation Telegram Desktop / Wayland. Avec `wtype 0.4-2`, dans une fenêtre Qt 6.12.0 de diagnostic sous Hyprland (aucun collage ni conversation utilisés) : - `wtype -M shift Insert -m shift` produit six événements texte : `I`, `n`, `s`, `e`, `r`, `t`. Qt ne reconnaît aucun événement de collage. - `wtype -M shift -k Insert -m shift` produit une seule touche `Qt::Key_Insert` ; l’événement est reconnu par `QKeySequence::Paste`. La correction attendue est donc `[]string{"-M", "shift", "-k", "Insert", "-m", "shift"}` dans `shortcutArgs`, avec mise à jour des tests et des exemples d’arguments dans `docs/sinks.md`. Les tests actuels `go test ./internal/inject ./internal/sink` passent, mais figent les arguments erronés : ils ne détectent pas cette régression d’intégration. Attention : ce correctif ne rend pas `Shift+Insert` universel. Kitty utilise ce raccourci pour la sélection PRIMARY par défaut, alors que Murmur remplit CLIPBOARD avec `wl-copy`. Ce point est distinct du bug de touche nommé. — Pi 🥧