Issue links in overview HTML are emitted with an extra quote in href
Mantenedores costumam responder em até 1 dia
Avaliação
Esta issue ainda não foi avaliada.
Descrição
- Extension version: 0.166.1 (current
mainpackage.json) - VSCode Version: not applicable to the source-level reproduction; the extension declares
^1.141.0 - OS: Windows 11 (source-level reproduction; the generated HTML is platform-independent)
- Repository Clone Configuration (single repository/fork of an upstream repository): single repository
- GitHub Product (GitHub.com/GitHub Enterprise version x.x.x): GitHub.com
Steps to Reproduce:
- Open a pull request or issue whose description contains a link to another issue in the same repository, for example
https://github.com/microsoft/vscode-pull-request-github/issues/123. - Let the extension convert the description HTML for the issue/PR overview.
- Inspect the generated anchor for the issue link.
The issue-link branch in src/github/utils.ts currently returns:
return `href="${(await toOpenIssueWebviewUri(...)).toString()}""`;
That produces an anchor like:
<a href="vscode://github.vscode-pull-request-github/open-issue-webview?..."">#123</a>
The corresponding pull-request branch returns a normally terminated href, so this only affects links converted from /issues/<number> URLs.
The extra quote is parsed as a second, nameless attribute (rather than being part of the href value). The link may still appear to work in some Chromium versions, but the HTML is malformed and can be handled inconsistently by the webview DOM/sanitizer. It also makes the generated markup different from the PR-link path for no functional reason.
The smallest fix appears to be removing the extra " at the end of the issue branch, and adding a regression test around the HTML returned for a same-repository issue link. I have not changed the repository code yet; this report is based on the current main source and a local HTML parsing reproduction.
- Linguagem predominante
- TypeScript
- Estrelas
- 2.6k
- Forks
- 800
- Merge médio
- 20h 52min
- PRs com merge (30d)
- 35
Preparar o ambiente
Este projeto não oferece contêiner de desenvolvimento, Dockerfile nem guia de contribuição, então a configuração fica por sua conta: comece pelo README e veja nosso guia da primeira contribuição para os passos gerais.
Primeiros passos
- Leia a issue inteira e depois o guia de contribuição do projeto.
- Comente na issue dizendo que vai assumir — evita que duas pessoas façam o mesmo trabalho.
- Faça um fork do repositório e trabalhe em uma branch.
- Abra um pull request que referencie o número da issue.
Mais de microsoft/vscode-pull-request-github
-
PR overview remains stuck checking mergeability after an UNKNOWN responseTalvez já em andamento @alexr00 assumiu há 7 dias. Abertabug
Dificuldade 2/5 1-3 horas Facilidade para iniciantes 78/100
microsoft/vscode-pull-request-github#8953 · 1 responsável ·
Mantenedores costumam responder em até 1 dia
-
Issue 3 — DocumentationAberta
Dificuldade 2/5 1-3 horas Facilidade para iniciantes 68/100
microsoft/vscode-pull-request-github#8792 ·
Mantenedores costumam responder em até 1 dia
-
Copilot PR-state polling throws RangeError: Invalid array length while accumulating paginated resultsTalvez já em andamento @alexr00 assumiu há 1 dia. Aberta
microsoft/vscode-pull-request-github#9023 · 1 responsável ·
Mantenedores costumam responder em até 1 dia
-
Stacked PRs: PR state does not change in stack unless you manually change the webviewTalvez já em andamento @alexr00 assumiu há 2 dias. Abertabug
microsoft/vscode-pull-request-github#9020 · 1 responsável ·
Mantenedores costumam responder em até 1 dia
-
Colours are off for sub-buttons under Create Merge CommitTalvez já em andamento @alexr00 assumiu há 2 dias. Abertabug
microsoft/vscode-pull-request-github#9019 · 1 responsável ·
Mantenedores costumam responder em até 1 dia
Todas as issues de microsoft/vscode-pull-request-github
Issues semelhantes
-
Dificuldade 2/5 1-3 horas Facilidade para iniciantes 68/100
rajbos/ai-engineering-fluency#2340 · 1 comentário ·
Mantenedores costumam responder em até 1 dia
-
community documentation first-timers-only good first issue hacktoberfest help wanted low hanging fruit up-for-grabs
Dificuldade 1/5 Menos de uma hora Facilidade para iniciantes 70/100
lingdojo/kana-dojo#31864 · 1 comentário · 5 reações ·
Mantenedores costumam responder em até 1 dia
-
Dificuldade 2/5 1-3 horas Facilidade para iniciantes 68/100
zenstackhq/zenstack#2873 ·
Mantenedores costumam responder em até 1 dia
-
CLI: TUI shows onboarding when the provider's API key is only in the environment (e.g. OPENROUTER_API_KEY)Talvez já em andamento Um pull request vinculado a esta issue está aberto ou já foi mesclado. AbertaCLI
Dificuldade 2/5 1-3 horas Facilidade para iniciantes 67/100
cline/cline#14923 · 2 comentários ·
Mantenedores costumam responder em até 1 dia
-
Dificuldade 2/5 1-3 horas Facilidade para iniciantes 76/100
paperclipai/paperclip#15490 ·
Mantenedores costumam responder em até 1 dia