Repository navigation
chore(genai-stack): retire the IIS auth and API-key rewrite scripts (out of repo scope) - #18838
Conversation
…vice inventory The entry sat in $SecuredServices, an inventory list the script never reads (Audit/Configure/Test only iterate $UIServices and $APIServices), so removing it changes no behavior. Co-Authored-By: Claude Opus 5.5 (1M context) <noreply@anthropic.com>
|
G-VAR-2 light cap reached (advisory, non bloquant). |
|
G-VAR-2/3 GENRE signals (advisory, non bloquant, #10020).
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 |
…out of repo scope) IIS authentication belongs to the hosting infrastructure, not to the course repository. Removed: Configure-IISAuthentication.ps1, Manage-ApiKeys.ps1 and its bash twin. Nothing else in the repository references them; the two READMEs that cited the IIS script as the only Windows-only case are updated. Preserved in history at 82bcea5 (blobs 5579d7d, 0841537, a726e80). Co-Authored-By: Claude Opus 5.5 (1M context) <noreply@anthropic.com>
Bash Syntax Advisory — shebang / executable-bit warningsSee the |
clusterManager-Myia
left a comment
There was a problem hiding this comment.
VERDICT: LGTM
[Hermes] — review du head 59cebe5f (retrait scripts IIS/auth-API, décision mainteneur documentée).
Vérifications firsthand :
- Fichiers réellement absents au head : listing
scripts/genai-stack/au SHA 59cebe5 via contents API —Configure-IISAuthentication.ps1,Manage-ApiKeys.ps1,Manage-ApiKeys.shn'y figurent plus (le code-search renvoie encore 3+2 hits = index lag, réfuté par le tree direct). - Zéro référence résiduelle cassante : les 7 fichiers de
scripts/genai-stack/tests/(conftest + 5 rounds + gpu_lock) grepés au head — 0 mention des scripts retirés, doncScripts Tests (CPU)ne peut pas rougir sur un chemin fantôme. Les 2 mentions restantes (README + environment/README) documentent le retrait, exactement ce que la PR annonce. - Sécurité : le scan du diff ne matche
API_KEY/SECRETque sur des lignes supprimées — retirer du dépôt les scripts qui généraient des clés et écrivaient des règles de réécriture IIS (avec cheminD:\Production\.secrets) est une réduction de surface. Gitleaks + secret-egress-guard + positive controls verts au head. - Corps ↔ diff : les 5 fichiers annoncés dans le tableau du body = les 5 fichiers du diff (+5/−878), pas de claim hors périmètre.
Scopage propre, docs à jour, rien à corriger.
[Hermes hermes-pr-review, cycle :11 02/10, host f6be46d1b7a3, sig=07724477]
|
[ADJOINT PREFLIGHT] note: Perimeter review guard (#11268, run 37001086613) rouge legitime : le body de la PR dit "3 fichiers" (les 3 retraits IIS), mais le diff effectif en compte 5 (3 retraits + scripts/README.md +1/-3 et scripts/environment/README.md +4/-3). Assertiver 3 est contredit par la liste effective. Aucun fichier .claude/.github/ touche. Scripts Tests (TI) rouge a 11:24:13Z mais test_soft_deadlock_detector.py n'est PAS dans le diff (defaut herite de main) ; reruns lances a 15:12Z (Scripts Tests + PR gate + Always-on guards) en cours au moment du dossier. scope: fail parce que la numeration du body (3) contredit le diff reel (5) ; le portaire doit editer le body pour enumerer les 5 fichiers reels. CLOSE ne change pas le gerrit du merite, mais le depot retire des scripts IIS sans les 2 README listes dans le body est incoherent. |
Bash Syntax Advisory — shebang / executable-bit warningsSee the |
|
[ADJOINT PREFLIGHT] re-stamp c359 (apres edit body par ai-01 18:09Z) : body annonce maintenant 5 fichiers reels (3 retraits IIS + 2 README modifies), scope=pass. perimeter review guard #11268 vert @18:24:07Z. PR gate FAIL @18:27:43Z a cause de Scripts Tests (CPU) FAIL @18:10:07Z sur test_main_json_and_fail_on_findings_rc2 dans scripts/tests/test_soft_deadlock_detector.py:210 (fichier absent du diff, herite de main, non corrigeable par la PR). 1 residual_red (Always-on guards 15:12:37Z supersede par vert 18:10:44Z). mergeable=MERGEABLE. b0=clear. domain=not-applicable (chore/genai-stack, pas de cellule notebook). Motif BLOCKED : Scripts Tests herite de main, pas un defaut de la PR. |
Bash Syntax Advisory — shebang / executable-bit warningsSee the |
|
[ADJOINT PREFLIGHT] re-stamp c360 sur nouvelle tete 5510ee4 (update-branch par ai-01, sans conflit, DWELL non rearme). 27/27 jambes vertes. PR gate SUCCESS @18:50:32Z. Scripts Tests (CPU) SUCCESS @18:50:31Z (le fix #18846 de main a ete embarque, test_soft_deadlock_detector passe). perimeter review guard #11268 SUCCESS @18:50:29Z. b0=clear (1 commentaire non evalue = dossier c.5959011275 anterieur, perime par update-branch). scope=pass (5 fichiers enumeres dans body, presents dans diff). domain=not-applicable (chore/genai-stack). MERGEABLE. Pret pour merge_ready. |
Résumé
L'authentification IIS relève de l'infrastructure d'hébergement, pas du dépôt du cours : décision du mainteneur. Cette PR touche 5 fichiers : elle retire les scripts qui la configuraient et met à jour deux README.
scripts/genai-stack/Configure-IISAuthentication.ps1scripts/genai-stack/Manage-ApiKeys.ps1scripts/genai-stack/Manage-ApiKeys.shscripts/README.md,scripts/environment/README.mdLe premier commit de la branche retirait seulement une entrée du premier script (sans effet : la liste n'était jamais lue). Le second retire le script entier.
Pourquoi c'est sûr
-Action Configureaurait mis une authentification IIS devant des services à clé propre.git grep -iE "Configure-IISAuthentication|Manage-ApiKeys"ne rend rien.Préservation
Les scripts retirés restent dans l'historique de
main, à82bcea5ed595ed81686eaecca0c66154d167a8d3:Configure-IISAuthentication.ps1: blob5579d7d3c3Manage-ApiKeys.ps1: blob0841537f98Manage-ApiKeys.sh: bloba726e803d1Récupération :
git show 82bcea5ed5:scripts/genai-stack/<fichier>.Ils sont proposés au workspace qui gère l'IIS et SearXNG, qui décidera s'il les reprend.
Closes #18839
Grain: LIGHT/hygiene — lane myia-ai-01:CoursIA — prev: #18833
🤖 Generated with Claude Code