stenciljs/eslint-plugin

bug: `props-must-be-readonly` reports error although mutable is set when attribute is set via constant value

Open

#196 aberto em 26 de mar. de 2026

Ver no GitHub
 (0 comments) (0 reactions) (0 assignees)TypeScript (37 forks)auto 404
Bug: ValidatedHelp Wanted

Métricas do repositório

Stars
 (71 stars)
Métricas de merge de PR
 (Métricas PR pendentes)

Description

Prerequisites

Stencil ESLint Version

1.2.0

Current Behavior

When declaring a @Prop via enum or constant, the props-must-be-readonly is reporting an error although mutable was set to true.

@Prop({ reflect: true, attribute: Attribute.withPadding, mutable: true }) withPadding: boolean = false;

=>

  26:77  error  Class properties decorated with @Prop() should be readonly  stencil/props-must-be-readonly

In comparison, when setting the attribute directly without an enum or const, it works as expected (no error is reported)

 @Prop({ reflect: true, attribute: 'with-padding', mutable: true }) withPadding: boolean = false;

Expected Behavior

The @Prop declaration

@Prop({ reflect: true, attribute: Attribute.withPadding, mutable: true }) withPadding: boolean = false;

should not throw an error, as mutable is set to true (cf. docs).

Steps to Reproduce

  • Declare a property as above
  • Run the linting rules with props-must-be-readonly enabled

Code Reproduction URL

https://github.com/pinussilvestrus/test-stencil/commit/0d943e601dd76a51bd819527802f6d6d4f77985c

Additional Information

No response

Guia do colaborador

Direção de pesquisa
Investigue a lógica da regra ESLint 'props must be readonly' para determinar por que ela falha em reconhecer `mutable: true` quando o valor de `attribute` é uma constante (por exemplo, enum). Verifique se a regra apenas avalia valores literais de string para `attribute` e ignora expressões constantes. Considere atualizar a regra para avaliar expressões constantes usando informações de tipo ou rastreando valores de variáveis.
Pilha de tecnologia
typescript
Domain
frontendweb dev
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.
2
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.
Ativo
ClarityCom que clareza o issue explica a mudança esperada, os critérios de aceitação e a próxima etapa.
Claro
Prerequisites
JavaScriptTypeScriptESLint
Simpatia para novatosUma pontuação de 1 a 100 que estima o quão acessível este issue é para colaboradores iniciantes.
60