Hacktoberfest 2026 : les issues que les mainteneurs ont marquées pour octobre, ouvertes et accessibles aux débutants. Parcourir les issues Hacktoberfest

A pending document-link confirmation holds every Tool launch in the renderer

Ouverte
  • Quelque chose ne va pas ?
#1,234 1 commentaire 0 réactions 0 personnes assignées Voir sur GitHub

Les mainteneurs répondent en général sous 1 jour

Personne n'a encore pris cette issue.

Évaluation

Difficulté
4/5
Temps estimé
1-2 jours
Accessibilité débutants
48/100
Type d'issue
Bug
Clarté
Clairement spécifiée
Activité
Active
Stack technique
typescript
Domaine
desktop, developer-experience

Piste de recherche

Start with lib/src/components/wall/use-dor-control.ts, especially queueToolSpawn, launch(), and the confirmDocumentLinkOpen call. Trace how the queue and sourceChanged() interact when a confirmation stays open, then add a regression test through use-dor-control showing that another pane's Tool request can proceed. The issue is resolved when the test passes and a pending confirmation no longer blocks other launches.

Rédigé par le modèle d'indexation à partir du texte de l'issue.

Description

While the document-link confirmation added in #1201 is open, every other Tool request in that renderer waits for it. That includes dor tool and dor open from agents in other panes, OSC 367 previews, and approval completions.

Why it happens. queueToolSpawn is a module-level serial queue shared by every Wall in the renderer, and the whole launch() runs inside it. launch() now awaits confirmDocumentLinkOpen, which resolves only when the user answers, a later link replaces the request, or the request aborts. No timeout applies. The other awaits inside the queue (interruptToPrompt, typeToolCommand) are bounded by grace periods or timeouts. This one is bounded only by the user.

To trigger it. In a builtin:file Markdown Tool, click a relative link to an .html or .svg file and leave the dialog open. A dor tool … or dor open … from another pane in the same window then waits until the dialog is answered.

A possible fix. Run the handler lookup and the confirmation before taking the queue. Then, inside it, re-check sourceChanged(), which already runs after the confirmation today. The source's toolNavigating marker is set before the queue is taken, so that re-check still notices a source that moved on while the dialog was open. I haven't built this. The sandbox this nightly sweep ran in couldn't install dependencies, so I had no way to write the regression test through use-dor-control. That is why this is an issue rather than a PR.

@nedtwigg: do you want the confirmation moved outside the queue, or is holding other launches while it is open acceptable?

Langage dominant
TypeScript
Étoiles
5
Forks
1
Merge moyen
10 h 51 min
PR mergées (30 j)
507

Préparer son environnement

Ce projet ne fournit ni conteneur de développement, ni Dockerfile, ni guide de contribution : l'installation est à votre charge. Commencez par son README, et consultez notre guide de la première contribution pour les étapes générales.

Par où commencer

  1. Lisez l'issue en entier, puis le guide de contribution du projet.
  2. Signalez en commentaire que vous la prenez — cela évite que deux personnes fassent le même travail.
  3. Forkez le dépôt et travaillez sur une branche.
  4. Ouvrez une pull request qui référence le numéro de l'issue.

Autres issues de diffplug/dormouse

Toutes les issues de diffplug/dormouse

Issues similaires

Plus d'issues TypeScript

Recevez les nouvelles issues par e-mail

Un résumé court des issues GitHub adaptées aux débutants.