Hacktoberfest 2026 : les issues que les mainteneurs ont marquées pour octobre, ouvertes et accessibles aux débutants. Parcourir les issues Hacktoberfest

fix(source-control): review evidence rejects completion records ending in account=fallback

Ouverte Adaptée aux débutants
  • Quelque chose ne va pas ?
#8,366 1 commentaire 0 réactions 0 personnes assignées Voir sur GitHub

Les mainteneurs répondent en général sous 1 jour

Personne n'a encore pris cette issue.

Évaluation

Difficulté
2/5
Temps estimé
1-3 heures
Accessibilité débutants
85/100
Type d'issue
Bug
Clarté
Clairement spécifiée
Activité
Active
Stack technique
python, shell
Domaine
security, testing

Piste de recherche

Lis plugins/source-control/skills/babysit-prs/scripts/babysit_review_evidence.py autour de RECORD_RE et _record, puis examine les cas associés dans scripts/tests/test_babysit_review_evidence.py. Ajoute des tests pour les enregistrements complets et incrémentiels se terminant par account=fallback, ainsi que pour un cas avec du texte final mal formé. Le travail est terminé lorsque le suffixe de fallback est associé aux mêmes champs de preuve, tandis que tout autre texte final reste rejeté.

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

Description

priority: low status: confirmed

Problem

RECORD_RE in plugins/source-control/skills/babysit-prs/scripts/babysit_review_evidence.py (lines 55-61) anchors the completion record at $ right after scope=whole or scope=incremental since=<sha>. The lane producer adds account=fallback to the end of that record whenever the review ran on the fallback account. See melodic-software/ci-workflows .github/workflows/pr-review.yml lines 1179-1213 (read at d61aeac):

if [ "$ACCOUNT" = fallback ]; then
  account=" account=fallback"
...
echo "::notice title=claude-lane-review-complete::$record findings=$FINDINGS inline=$INLINE $scope$account"

_record rejects that record as record-malformed. So a lane whose covering review, or any link in its incremental chain, was reviewed on the fallback account never binds.

Risk

The gate fails closed, so nothing merges that should not. The cost is that it stays stuck: every pull request whose review fell back to the second account after a usage limit cannot reach pr-merge until a new review runs on the primary account. Priority is low because the gate stays safe and only throughput is lost.

Fix

Accept an optional trailing account=fallback (and only that token) after the findings=… scope=… form in RECORD_RE. Treat the record exactly like a primary-account record. The skip forms carry no account suffix in the producer, so leave them unchanged.

Done when

  • A record ending in account=fallback binds with the same findings, inline and scope as the same record without the suffix.
  • Any other trailing text is still record-malformed, and a test covers it.
  • Tests in scripts/tests/test_babysit_review_evidence.py cover the whole and incremental forms with the suffix, and fail before the fix.

needs-human: this changes what the unattended merge gate accepts as review evidence, so a human reviews and merges it.

Langage dominant
Shell
Étoiles
22
Forks
2
Merge moyen
6 h 58 min
PR mergées (30 j)
951

Préparer son environnement

Par où commencer

  1. Lisez l'issue en entier, puis le guide de contribution du projet.
  2. Signalez en commentaire que vous la prenez — cela évite que deux personnes fassent le même travail.
  3. Forkez le dépôt et travaillez sur une branche.
  4. Ouvrez une pull request qui référence le numéro de l'issue.

Autres issues de melodic-software/claude-code-plugins

Toutes les issues de melodic-software/claude-code-plugins

Issues similaires

Plus d'issues Shell/Bash

Recevez les nouvelles issues par e-mail

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