forcedotcom / forcedotcom/code-analyzer

[BUG][code-analyzer] Disabled rules are moved to the end of the config file rules list

Ouverte
#2,019 3 commentaires 0 réactions 0 personnes assignées Voir sur GitHub
BUG P3
Langage dominant
TypeScript
Étoiles
240
Forks
52
Merge moyen
1 j 23 h
PR mergées (30 j)
5

Description

### Have you tried to resolve this issue yourself first?

- [x] I confirm I have gone through the above steps and still have an issue to report.

### Bug Description

I disabled some rules and recreated the config file. ( see #2013 )

All the disabled rules were moved to the end of the `pmd` section.

But I want to view ALL the PMD rules alphabetized so I know which ones are in there.

### Output / Logs

```shell

```

### Steps To Reproduce

1. Disabled a rule with `disabled: true`
2. Run `echo 'y' | sf code-analyzer config --include-unmodified-rules --rule-selector all --config-file code-analyzer.yml --output-file code-analyzer.yml`
3. See that the disabled rules are moved.

### Expected Behavior

All rules should be alphabetized by their unique key. This extra sort doesn't help anything and only can confuse people.

### Operating System

macOS

### Salesforce CLI Version

@salesforce/cli/2.128.5 darwin-arm64 node-v22.22.1

### Code Analyzer Plugin (code-analyzer) Version

5.11.1

### Node Version

_No response_

### Java Version

_No response_

### Python Version

_No response_

### Additional Context (Screenshots, Files, etc)

Image

This is actually a significant bug because changing from expected behavior will confuse people.

### Workaround

_No response_

### Urgency

Moderate

Guide de contribution

Ouvrir le guide de contribution

Piste de recherche

Commencez par la commande `sf code-analyzer config` et reproduisez le problème à l’aide de la commande fournie et de `code-analyzer.yml`. Examinez la liste de règles `pmd` générée, puis vérifiez que toutes les règles, y compris celles avec `disabled: true`, sont classées par ordre alphabétique selon leur clé unique et le restent après une nouvelle génération.

Rédigé par le modèle d'indexation à partir du texte de l'issue.

Évaluation

Stack technique
typescript
Domaine
cli
Type d'issue
Bug
Difficulté
3/5
Temps estimé
1-2 jours
Activité
Calme
Clarté
Plutôt claire
Accessibilité débutants
58/100

Recevez les nouvelles issues par e-mail

Un résumé court des issues GitHub adaptées aux débutants.