Microsoft/vscode

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

Open

#212.109 aberto em 6 de mai. de 2024

Ver no GitHub
 (6 comments) (3 reactions) (1 assignee)TypeScript (10.221 forks)batch import
bugconfirmedhelp wantedterminal-links

Métricas do repositório

Stars
 (74.848 stars)
Métricas de merge de PR
 (Mesclagem média 11h 43m) (1.000 fundiu PRs em 30d)

Description

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
------ ----- - ----- ---------

Guia do colaborador

Direção de pesquisa
Examine a lógica de detecção de links no terminal do VS Code (provavelmente em `src/vs/workbench/contrib/terminal/browser/linkDetector.ts` ou similar). Revise a correção anterior para #169774 para entender a regex ou abordagem de análise usada para caminhos de arquivo. Foco em garantir que parênteses `()` e colchetes `[]` sejam tratados corretamente como parte do nome do arquivo. Rastreie os casos de teste para detecção de links para ver se cobrem esses caracteres. Depure executando o terminal integrado com um caminho contendo esses caracteres e observe as regiões dos links.
Pilha de tecnologia
typescript
Domain
developer experience
Tipo Issue
Bug
DifficultyDificuldade de implementação estimada para um novo contribuidor, de 1 para alterações muito pequenas a 5 para trabalho de nível especializado.
3
Tempo estimadoUm intervalo de tempo aproximado para um colaborador experiente investigar, implementar, testar e preparar um pull request.
Meio dia
Status da atividadeQuão disponível o issue aparece agora: novo, ativo, obsoleto, bloqueado ou aguardando entrada do mantenedor.
Ativo
ClarityCom que clareza o issue explica a mudança esperada, os critérios de aceitação e a próxima etapa.
Claro
Prerequisites
GitTypeScript
Simpatia para novatosUma pontuação de 1 a 100 que estima o quão acessível este issue é para colaboradores iniciantes.
70