Microsoft/vscode

Parentheses `()` and square brackets `[]` in file names can break terminal links (eg. next.js dynamic file paths)

Open

#212.109 geöffnet am 6. Mai 2024

Auf GitHub ansehen
 (6 Kommentare) (3 Reaktionen) (1 zugewiesene Person)TypeScript (10.221 Forks)batch import
bugconfirmedhelp wantedterminal-links

Repository-Metriken

Stars
 (74.848 Stars)
PR-Merge-Metriken
 (Durchschn. Merge 11h 43m) (1.000 gemergte PRs in 30 T)

Beschreibung

Does this issue occur when all extensions are disabled?: Yes

  • VS Code Version: 1.89.0
  • OS Version: macOS Sonoma 14.4.1

This issue was supposedly fixed in #169774, but it still doesn't seem to be working for me. Maybe a regression?

When the terminals logs file names that include characters ()[] (Used by NextJS routing segments for example), it splits the file name into multiple parts and is not properly clickable to open that file.

Steps to Reproduce:

In the integrated terminal, run echo "./src/(hello)/[world]/file.tsx" and hover over the output while holding cmd to see the different file segments determined by VSCode. Should look something like:

./src/(hello)/[world]/file.tsx
------ ----- - ----- ---------

Contributor Guide

Research-Richtung
Untersuche die Logik zur Linkerkennung im VS Code Terminal (wahrscheinlich in `src/vs/workbench/contrib/terminal/browser/linkDetector.ts` oder ähnlich). Sieh dir den vorherigen Fix für #169774 an, um den verwendeten regulären Ausdruck oder den Analyseansatz für Dateipfade zu verstehen. Konzentriere dich darauf, sicherzustellen, dass runde Klammern `()` und eckige Klammern `[]` korrekt als Teil des Dateinamens behandelt werden. Verfolge die Testfälle für die Linkerkennung, um zu sehen, ob sie diese Zeichen abdecken. Debugge, indem du das integrierte Terminal mit einem Pfad ausführst, der diese Zeichen enthält, und beobachte die Linkbereiche.
Tech Stack
typescript
Domain
developer experience
Issue Type
Fehler
SchwierigkeitGeschätzte Implementierungsschwierigkeit für neue Contributors, von 1 für sehr kleine Änderungen bis 5 für Expertenarbeit.
3
Geschätzte ZeitEin grober Zeitrahmen, um zu recherchieren, zu implementieren, zu testen und einen Pull Request vorzubereiten.
Halber Tag
AktivitätsstatusWie verfügbar das Issue gerade wirkt: frisch, aktiv, stale, blockiert oder wartet auf Maintainer-Input.
Aktiv
KlarheitWie klar das Issue die erwartete Änderung, Akzeptanzkriterien und den nächsten Schritt erklärt.
Klar
Voraussetzungen
GitTypeScript
EinsteigerfreundlichkeitEin Score von 1 bis 100, der schätzt, wie zugänglich dieses Issue für First-Time Contributors ist.
70