withastro/astro

🐛 BUG: version 2.8.1 removes trailing comma on save

Fechada

#14.665 aberto em 29 de mar. de 2024

Ver no GitHubGuia do colaborador
 (7 comentários) (0 reação) (0 responsável)TypeScript (3.732 forks)batch import
good first issuepkg: language-tools

Métricas do repositório

Stars
 (61.918 estrelas)
Métricas de merge de PR
 (Mesclagem média 4d 23h) (197 fundiu PRs em 30d)

Description

Describe the Bug

When I save an astro file, it removes trailing commas from multi-line function calls.

If I downgrade to version 2.8.0 and save, it adds them back again when I save.

I wonder if it is also related to withastro/language-tools#823 - I found another issue that sounded like it was similarly affected, withastro/language-tools#829 - though the specific fix for that one did not address this one.

Steps to Reproduce

  1. Open any .astro file with multi-line function calls. E.g. 
    foo(
  var1,
  var2,
);
  2. save the file
  3. trailing commas are removed. For the above example it would change to: 
    foo(
  var1,
  var2
);

EDIT: After comments, here is a better example, since the above ends up on a single line which does not have trailing comma:

  someReallyLongNAmeThingThatIsReallyLong(
    someReallyLongVarThatISLongAnd,
    SomeOtherReallyLongThing,
  );

Guia do colaborador

Direção de pesquisa
Investigue a lógica de formatação do servidor de linguagem do Astro, especificamente no plugin Prettier ou no formatador interno. Examine as issues referenciadas withastro/language tools#823 e #829 para obter contexto relacionado. Foque em como as vírgulas finais são tratadas em chamadas de função multilinha dentro de arquivos .astro. O bug parece ser uma regressão na versão 2.8.1, portanto compare o código de formatação entre as versões 2.8.0 e 2.8.1.
Pilha de tecnologia
typescriptnodejs
Domain
frontend
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.
1-3 horas
Status da atividadeQuão disponível o issue aparece agora: novo, ativo, obsoleto, bloqueado ou aguardando entrada do mantenedor.
Recente
ClarityCom que clareza o issue explica a mudança esperada, os critérios de aceitação e a próxima etapa.
Claro
Prerequisites
TypeScriptAstro
Simpatia para novatosUma pontuação de 1 a 100 que estima o quão acessível este issue é para colaboradores iniciantes.
50