openai / openai/codex-plugin-cc

`review --base <nonexistent-sha>` exits 0 and reviews a far wider diff

Ouverte
#653 0 commentaires 0 réactions 0 personnes assignées Voir sur GitHub

Personne n'a encore pris cette issue.

Langage dominant
JavaScript
Étoiles
33.3k
Forks
2.3k
Métriques de merge des PR
Aucune PR mergée en 30 j

Description

Passing a base that does not exist in the repository does not fail the run. The companion proceeds, silently falls back to a much wider diff than the caller asked for, and exits 0 — so a CI step or a wrapper script reads it as a clean, scoped review.

Why it is expensive rather than merely wrong: the failure mode is a review that looks successful. In our use the base is a wave-start SHA passed by a script; a typo produced a run that swept months of history, returned findings about code nobody had touched in the change under review, and reported success. Nothing in the output said the base had not resolved.

Expected: exit non-zero with "base not found in this repository", before any model call.

Workaround we adopted: validate before every launch — git cat-file -e "<sha>^{commit}" || exit 1.

Environment: codex-plugin-cc companion via the app server; observed 2026-08-09.

Related but distinct: #611 §1 covers a run exiting 0 with no verdict on a valid base.

Guide de contribution

Aucun guide de contribution indexé pour ce dépôt

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.

Piste de recherche

Commencez par le traitement de review --base par le serveur de l’application compagnon, puis suivez la manière dont le SHA fourni est résolu avant l’exécution des chemins du diff et de l’appel au modèle. Reproduisez le problème avec une base inexistante et vérifiez qu’il se termine avec un code de sortie non nul et avec "base not found in this repository", sans effectuer d’appel au modèle.

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

Évaluation

Stack technique
javascript
Domaine
cli
Type d'issue
Bug
Difficulté
3/5
Temps estimé
1-2 jours
Activité
Calme
Clarté
Clairement spécifiée
Accessibilité débutants
68/100

Recevez les nouvelles issues par e-mail

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