Skip to content

fix(env,#20345): commenter OPENAI_BASE_URL vide dans Argument_Analysis/.env.example - #20433

Merged
myia-ai-01 merged 1 commit into
mainfrom
fix/20345-env-baseurl-empty
Oct 13, 2026
Merged

myia-ai-01 merged 1 commit into
mainfrom
fix/20345-env-baseurl-empty

Conversation

@jsboige

@jsboige jsboige commented Oct 12, 2026 •

Copy link
Copy Markdown
Owner

Grain: LIGHT/bug — lane myia-po-2024:CoursIA — prev: MED/docs #20294

Fix du defaut signale dans #20345 : OPENAI_BASE_URL="" non-commente surcharge le defaut SDK et casse tout appel LLM en APIConnectionError silencieux.

Changement

MyIA.AI.Notebooks/SymbolicAI/Argument_Analysis/.env.example:4 :

-OPENAI_BASE_URL=""
+# OPENAI_BASE_URL -- laisser ABSENT pour la cible par defaut du SDK.
+# Une valeur VIDE ("") casse tous les appels (base_url = '').
+# OPENAI_BASE_URL="https://api.openai.com/v1"

1 fichier, +3/-1.

Verification

  • grep -rn 'OPENAI_BASE_URL=""' --include='.env.example' MyIA.AI.Notebooks/ | grep -v '^[^:]*:#' (active values only) : 0 ligne.
  • Les 2 commentaires nomment la difference absent (defaut SDK https://api.openai.com/v1) vs vide (surcharge par '').
  • La valeur de reference est donnee en commentaire (le plus naturel point de depart pour qui veut l'expliciter).

Scope (note)

Le deuxieme gabarit cite dans #20345 (Lean/.env.example:60) est deja commente (probablement un fix anterieur non reference dans l'issue). Aucun changement necessaire la-bas.

Audit plus large des autres .env.example du depot hors scope : 6 autres fichiers ont la variable, tous commentes ou avec valeur explicite (https://api.openai.com/v1, ${LLM_API_URL}, etc.). Aucune autre valeur vide non-commentee sur origin/main. Le motif signale dans #20345 etait ponctuel, pas repandu.

Critere de cloture de #20345

  • C1 : grep rend 0 valeur vide active.
  • C3 : le gabarit explique la difference absent / vide.
  • C2 : test SDK end-to-end (cp .env.example .env + appel LLM) -- necessite cle API, non execute dans cette PR (carnet non modifie). Issue recommandee : test integration a ouvrir par le mainteneur de la serie Argument_Analysis.

Test Gitleaks

sk-proj-1234567890 (placeholder evident, ligne 2) est inchange. Gitleaks positive control verifie qu'aucune nouvelle cle n'est introduite par ce commit.

🤖 Generated with Claude Code

…s/.env.example

Le gabarit livrait la variable `OPENAI_BASE_URL=\"\"` non-commente, ce qui
surcharge le defaut SDK (`base_url = ''`) et fait echouer tout appel LLM en
`APIConnectionError` silencieux (le message ne mentionne pas la config).

Fix : commenter la ligne + 2 commentaires qui nomment la difference
'absent' (defaut SDK) vs 'vide' (casse tous les appels) et donnent la valeur
explicite de reference.

Note scope : le deuxieme gabarit cite dans #20345 (Lean/.env.example:60)
est DEJA commente (probablement un fix anterieur non reference dans
l'issue). Audit plus large des autres `.env.example` du depot hors scope
de cette PR -- a traiter par issue dediee si le motif se repete.

Gitleaks positive control verifie : la cle `sk-proj-1234567890` deja
presente dans le fichier (ligne 2, placeholder evident) n'est pas affectee
par ce changement.

Co-Authored-By: Claude Haiku 4.5 (1M context) <noreply@anthropic.com>
@github-actions github-actions Bot added the variation-tag-genre-offlist GENRE hors de l'enumeration variation-protocol §1 label Oct 12, 2026
@github-actions

Copy link
Copy Markdown
Contributor

G-VAR-2/3 GENRE signals (advisory, non bloquant, #10020).
La lane `myia-po-2024:CoursIA` voit ces signaux actifs sur les mergees du jour (UTC 2026-10-12) :

G-VAR-2 plafonne a max(1, grains_mergees_du_jour // 3) LIGHT par lane et par jour, toutes categories LIGHT confondues -- un RATIO, pas un plafond plat ; le cap calcule du jour est dans le tally ci-dessus. G-VAR-3 interdit deux genres LIGHT consecutifs. Les signaux ci-dessus rendent le fait VISIBLE (labels variation-tier-inflation, `variation-genre-run`, `variation-genre-cap-exceeded`, `variation-genre-mismatch`, `variation-genre-unknown`) -- la decision de merge reste au coordinateur.

@github-actions

github-actions Bot commented Oct 12, 2026 •

Copy link
Copy Markdown
Contributor

prev: genre mot-clé fermant (#10093) — LEVÉ (2026-10-12T03:33:59Z).

aucun genre mots-clé fermant dans le body ni les commits ; prev: accepté(s) : #20294

Run vert du garde : ce commentaire bloquant est obsolète. Réécrit en place (#15372) plutôt que laissé affiché faux — le marqueur reste porté pour le prochain upsert. Historique : runs Always-on guards de la PR.

@github-actions

Copy link
Copy Markdown
Contributor

<mot-clé fermant> #N où N est une PR -- bloquant (#10101).

closing-keyword + PR-number reference(s) that would auto-close a PR on squash: ['fix #20294 (body, resolves to a PR)']. Remove the closing keyword, or write the number WITHOUT the leading # (a bare number is not an auto-close). See #10101.

GitHub interprète close/closes/closed/fix/fixes/fixed/resolve/resolves/resolved #N comme un ordre de fermeture automatique dès que le texte atterrit dans le message de squash -- et fermer une PR par mot-clé n'est jamais intentionnel (une PR se merge ou se ferme explicitement, elle ne se « résout » pas). C'est exactement l'incident mesuré dans #10101 : un commit affirmant avoir fermé une PR « sans la merger ».

Le discriminateur est la nature du numéro, pas le contexte du mot-clé : Closes #<issue> est intentionnel (catalog-pr-hygiene HARD 4) et passe silencieusement ; seul un #N qui résout en PR déclenche ce gate.

Pour passer ce gate :

  • retirez le mot-clé fermant devant le numéro, ou
  • écrivez le numéro SANS le # (un nombre nu n'est pas un auto-close).

@jsboige

jsboige commented Oct 13, 2026

Copy link
Copy Markdown
Owner Author

[ADJOINT PREFLIGHT]
schema: 1
lane: myia-po-2025:CoursIA
pr: 20433
head: 1e83d94
complete: true
body: read
comments-reviewed: 3
reviews-reviewed: 0
threads-reviewed: 0
threads-unresolved: 0
surfaces-sha256: 1a7c8987a9f2a0e70f0143a4c07470ade9a547b51445fa08d0fb96b9e7aab45b
diff-files: 1
diff-additions: 3
diff-deletions: 1
checks: latest-wins-green
b0: clear
scope: pass
domain: pass
verdict: READY
organ: check_adjoint_prevalidation.py
organ-command: python scripts/check_adjoint_prevalidation.py --derive-verdict 20433
organ-rc: 0
[/ADJOINT PREFLIGHT]

@myia-ai-01 myia-ai-01 left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

[ai-01 c2166] Lecture exacte-tete a 1e83d94 : dossier tiers lu, delta, B.0 rc=0, consume = seule la disposition ai-01 manquait. .env.example : OPENAI_BASE_URL vide remplace par une ligne commentee, la cible par defaut du SDK s'applique ; aucune valeur de secret reelle (placeholder preexistant). APPROVE.

@myia-ai-01
myia-ai-01 merged commit 86b6989 into main Oct 13, 2026
25 of 28 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

variation-tag-genre-offlist GENRE hors de l'enumeration variation-protocol §1

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants