lib/yaml-subset.awk keeps the first duplicate key where the CI parser rejects the file
Les mainteneurs répondent en général sous 1 jour
Personne n'a encore pris cette issue.
Évaluation
- Difficulté
- 4/5
- Temps estimé
- 3-5 jours
- Accessibilité débutants
- 54/100
- Type d'issue
- Bug
- Clarté
- Plutôt claire
- Activité
- Active
- Domaine
- build-system, testing
Piste de recherche
Read lib/yaml-subset.awk and trace how --strict is passed by parse-concern-value.sh; inspect scripts/shared-copies.txt and the consumers of non-strict mode. Add duplicate-key rejection for keys in the same mapping in strict mode, propagate the canonical change with bash scripts/sync-shared-copies.sh, and check that non-strict consumers retain their current behavior. Done means strict mode rejects the duplicate case while existing non-strict behavior is preserved.
Rédigé par le modèle d'indexation à partir du texte de l'issue.
Description
Problem
When a mapping repeats a key, the shared awk YAML reader lib/yaml-subset.awk keeps the first value. The CI config parser rejects the same file. So a local reader and CI can disagree about one config.
Concrete case: babysit_merge_policy.py reads docs/conventions/pr-pipeline.yaml through this reader (via parse-concern-value.sh --strict) to decide the unattended merge rung. If the merge: block contained a key twice, CI would reject the file but the local reader would act on the first value. The CI rejection keeps the bad config off main, so this is latent today.
Expected
--strict mode treats a duplicate key in the same mapping as a parse error, the same as the CI parser.
Scope
- Canonical source:
lib/yaml-subset.awk. - Copies registered in
scripts/shared-copies.txt(16 files on main); propagate withbash scripts/sync-shared-copies.sh. - Check each consumer of the non-strict mode keeps its current behavior, or opt it in.
Related
- #6699: the Python
yaml_subset.pyreports the wrong line for a duplicate key in a sequence item. It is a separate reader. - #6202: auto-merge-safe-prs. Found by the Phase 8 security review on #7324.
🤖 Generated with Claude Code
- Langage dominant
- Shell
- Étoiles
- 22
- Forks
- 2
- Merge moyen
- 8 h 27 min
- PR mergées (30 j)
- 957
Préparer son environnement
- Aucun Dockerfile ni fichier Docker Compose
- Aucun modèle de pull request
- Lire le guide de contribution
Par où commencer
- Lisez l'issue en entier, puis le guide de contribution du projet.
- Signalez en commentaire que vous la prenez — cela évite que deux personnes fassent le même travail.
- Forkez le dépôt et travaillez sur une branche.
- Ouvrez une pull request qui référence le numéro de l'issue.
Autres issues de melodic-software/claude-code-plugins
-
priority: medium
Difficulté 2/5 1-3 heures Accessibilité débutants 68/100
melodic-software/claude-code-plugins#7351 · 2 commentaires ·
Les mainteneurs répondent en général sous 1 jour
-
agent-ready priority: high work-class: scoped
Difficulté 2/5 1-3 heures Accessibilité débutants 74/100
melodic-software/claude-code-plugins#7312 · 3 commentaires ·
Les mainteneurs répondent en général sous 1 jour
-
priority: medium
Difficulté 2/5 1-3 heures Accessibilité débutants 76/100
melodic-software/claude-code-plugins#7288 · 2 commentaires ·
Les mainteneurs répondent en général sous 1 jour
-
knowledge: docpage-digest dual verification defines no verdict vocabulary or pass/fail thresholdOuverteneeds-human
Difficulté 2/5 1-3 heures Accessibilité débutants 78/100
melodic-software/claude-code-plugins#7286 · 2 commentaires ·
Les mainteneurs répondent en général sous 1 jour
-
priority: low
Difficulté 2/5 1-3 heures Accessibilité débutants 76/100
melodic-software/claude-code-plugins#7282 · 2 commentaires ·
Les mainteneurs répondent en général sous 1 jour
Toutes les issues de melodic-software/claude-code-plugins
Issues similaires
-
area:contributor-experience difficulty:easy good first issue Stellar Wave
Difficulté 2/5 1-3 heures Accessibilité débutants 62/100
Les mainteneurs répondent en général sous 2 jours
-
Update OpenShell to 0.1.5-pre.2Ouvertepackage-update
Difficulté 2/5 1-3 heures Accessibilité débutants 62/100
oSoWoSo/vOid_Community_repOsitory#436 · 1 commentaire ·
Les mainteneurs répondent en général sous 1 jour
-
documentation good first issue
Difficulté 2/5 1-3 heures Accessibilité débutants 78/100
Maki-Zeninn/stellar-router#1485 ·
Les mainteneurs répondent en général sous 1 jour
-
Pin pnpm 9 with a packageManager fieldPeut-être pris @Rohit-Solanki1890 l’a pris aujourd’hui. Ouvertedevops dx good first issue
Difficulté 1/5 Moins d'une heure Accessibilité débutants 82/100
StellarLock/StellarLock#980 · 2 commentaires ·
Les mainteneurs répondent en général sous 1 jour
-
bug
Difficulté 2/5 1-3 heures Accessibilité débutants 72/100
hestiacp/hestiacp#5798 · 2 commentaires ·
Les mainteneurs répondent en général sous 1 jour