bmad-plus 0.19.0 → 0.21.0

package/CHANGELOG.md

@@ -5,6 +5,41 @@ All notable changes to BMAD+ will be documented in this file.
 The format is based on [Keep a Changelog](https://keepachangelog.com/en/1.1.0/),
 and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0.html).
 
+## [0.21.0] - 2026-09-28
+
+### Added
+
+- **Review run record.** `coverage.json` can carry `run {stop, detail?, passes, attempts[], tokens?, durationMs?}`. Stop reasons are `completed`, `budget`, `time-limit`, `failure-streak` or `interrupted`; any stop other than `completed` needs a detail, and a review that stopped early is always `incomplete`. Usage is recorded only when the host reports it, never estimated. After three consecutive failed attempts a unit is abandoned and a fourth attempt is refused.
+- **`review gate --emit-check <file>`** writes the verdict as `bmad-plus/review-check/1` for CI, atomically and without a timestamp, so the same evidence always gives the same bytes. It lists open findings by severity with their anchored lines and a `github` block shaped for the check-runs API. Exit codes are unchanged. The gate's reasons are redacted once, so the terminal, `--json` and the check carry the same text.
+- **`review continue <id>`** resumes an interrupted review on the same sealed scope and writes `continue.json`: units and files still owed, abandoned files, findings to requote, passes left. It refuses when the head, the rules or a selected file changed, and points to a new scope and `review compare`.
+- **Rule groups.** A rule can name a `group`; units list their groups so parallel reviewers can split a unit by rule family, and a unit is complete as a whole or when every group is done. Built-in groups are `code`, `data` and `delivery`; the group is part of the rule-set hash.
+- **Review bench** (maintainer tool, not in the npm package): seven small repositories in seven languages with 12 planted defects and 8 decoys, a published matching rule (path, category and a located quote that points at one plant; never a line number), and a scorer reporting recall, precision, decoy hits, unlocated quotes and reported usage. Three reference runs are replayed by a blocking CI step. No model call.
+- **CI review recipes** (`docs/recipes/ci-review/`): copy-ready workflows that seal the scope and gate a pull request with a host agent on a self-hosted runner. They are read-only, read no secret, pin every action, trust nothing the agent can rewrite (a step output holds a digest of the sealed scope), and fail fork pull requests instead of letting a required check pass unreviewed.
+- The open-code-review study states how the bench decides whether an optional `ocr` bridge is ever built.
+
+### Security
+
+- **SEO fetcher and crawler: SSRF protection and connection pinning.** Each host is resolved by the scripts, any non-public address is refused, and the connection goes to the exact address that was validated, on the first request and on every redirect hop, which defeats DNS rebinding. The guard sits in the connection pool every `requests` release goes through (verified on 2.34, 2.31 and 2.28). URL credentials, invalid ports, CGNAT and site-local ranges and IPv4-mapped or NAT64 forms of internal addresses are refused; proxies and `~/.netrc` are not used; custom CA bundles are still honoured.
+- **MCP knowledge ingestion is confined to each source checkout.** Paths resolving outside the checkout are refused; symlinks and `.git` are never followed.
+
+### Changed
+
+- **MCP server stack modernized** on Python 3.12: official `mcp` SDK, chromadb 1.5, sentence-transformers 6.1, CPU-only torch. `RAG_EMBEDDING_MODEL` selects the embedding model and Chroma telemetry is off. Existing stores open in place; re-run `ingest.py` once after upgrading. Chunks are keyed by their path in the checkout and stale chunks are pruned after each pass that indexed something. An offline smoke (`mcp-server/ci/smoke.py`) ingests a fixture, lists tools, queries and merges PDFs over an authenticated MCP session.
+
+## [0.20.0] - 2026-09-28
+
+### Added
+
+- **Review rules by path.** `review scope` writes `checklist.md`: each rule that applies to the changed paths, once, with the files it covers. Eight built-in rules written for BMAD+ (general, JavaScript/TypeScript, Python, SQL and migrations, shell, CI workflows, containers, configuration); a project adds, replaces or disables rules in `_bmad/review-rules.yaml`, whose documents are confined to `_bmad/` and size-capped. The rule set is part of the scope hash. A finding may name its `rule`; one that does not apply to its path is refused. `review rules [path]` shows the effective rules.
+- **Effort.** `review scope --effort low|medium|high` seals the depth in the scope and prints a plan: passes, refutation, survey first on large changes, parallel units when there are several.
+- **`review compare <id> --since <earlier-id>`** sorts findings into new, persisting, refuted, resolved and not reviewed. A finding is matched by path, category and quote, never by line; renamed files are followed. A finding gone from a file the later review did not complete is not reviewed, never resolved.
+- **Redaction floor.** Credential-like values (key blocks, URL credentials, authorization headers, credential-named assignments, well-known token shapes) become `[REDACTED]` in anchored review findings and in acceptance-run notes before `uat serve` or `uat import` writes them; references such as `process.env.X` or `${{ secrets.X }}` are kept. Each record counts its `redactions`.
+
+### Fixed
+
+- Every command gives the terminal back. In an interactive terminal the CLI kept its standard input open after the command had finished, so the shell prompt only returned after Ctrl+C. A test now runs the CLI under a pseudo-terminal whose input stays open.
+- The release workflow tolerates a distribution remote left by an earlier run on a self-hosted runner, and always removes the distribution credential from the workspace at the end of the job.
+
 ## [0.19.0] - 2026-09-27
 
 ### Added

package/README.md

@@ -1,8 +1,8 @@
 # BMAD+
 
-[![Version](https://img.shields.io/badge/version-0.19.0-blue)](https://www.npmjs.com/package/bmad-plus)
+[![Version](https://img.shields.io/badge/version-0.21.0-blue)](https://www.npmjs.com/package/bmad-plus)
 
-**Version 0.19.0** · Node.js `>=20.0.0` · MIT
+**Version 0.21.0** · Node.js `>=20.0.0` · MIT
 
 Project-local AI development workflows with clear roles, shared context, safe updates and coding-tool adapters
 
@@ -35,7 +35,7 @@ Run the installer. Select the adapters for the AI tools you use and the packs yo
 **In your project terminal:**
 
 ```sh
-npx bmad-plus@0.19.0 install
+npx bmad-plus@0.21.0 install
 ```
 
 The installer creates the agent instructions, shared project spine and selected tool adapters. Optional packs may need additional runtimes or API access.
@@ -71,7 +71,7 @@ Ask Sentinel to check the change against its acceptance criteria. Ask for test e
 **In your AI assistant:**
 
 ```text
-Sentinel, review this change against the acceptance criteria. Check the main success and failure paths, report remaining issues and summarize what the next session needs to know. If the BMAD+ CLI is available, seal the scope with bmad-plus review scope, quote the code of every finding, account for every selected file, then report what bmad-plus review gate answers.
+Sentinel, review this change against the acceptance criteria. Check the main success and failure paths, report remaining issues and summarize what the next session needs to know. If the BMAD+ CLI is available, seal the scope with bmad-plus review scope, apply the rules in its checklist, quote the code of every finding, account for every selected file, then report what bmad-plus review gate answers.
 ```
 
 Read the changes and test results. The assistant follows the permissions and capabilities of its host tool.
@@ -171,18 +171,18 @@ Read AGENTS.md and the available project memory. Summarize the last verified sta
 
 Expected result: a short, evidence-based restart. Project notes are useful context and still need to be checked against the current work.
 
-## What’s new in 0.19.0
+## What’s new in 0.21.0
 
-Code reviews you can check
+Reviews that stop honestly, resume, and report to CI
 
-bmad-plus review seals what a review must cover, locates every finding by the code it quotes, and derives the verdict from coverage instead of the number of findings. Review triage now keeps a finding unless a quoted ground refutes it. Nothing changes in how you install or update.
+A review now records why it stopped and what it used, writes a check a CI step can read, and resumes on the scope it sealed. The SEO fetcher connects only to addresses it has validated, and the MCP server moves to the official SDK on Python 3.12. Nothing changes in how you install or update.
 
-- bmad-plus review scope lists the files changed since the base branch or in your working tree, names why every other changed file is left out (secret, binary, deleted, generated), splits the work into ordered units and seals it with a hash. Secret files are excluded before any include pattern and never come back.
-- Findings go in findings.json and quote the code they are about, word for word. bmad-plus review anchor finds that code in the file and reports a quote it cannot find, or finds twice, instead of trusting a line number. Unknown categories or severities are refused, not guessed.
-- bmad-plus review gate answers incomplete, findings or clean. A review is incomplete while any selected file is missing from coverage.json or an open finding is not anchored; clean covers the reviewed scope only.
-- Sentinel and the Dev Studio code-review workflow keep a finding by default and refute it only on a quoted ground: the code is absent, the code contradicts it, or it duplicates another finding. Security, data-loss and money findings without such a quote stay unresolved.
-- The maintainer README now states what BMAD+ deliberately does not do: no model runtime, no background process, no review engine that repositions findings, no automatic fixes from a review.
-- Release tooling accepts the one-element array npm now returns for integrity checks. Continuous integration checks that every third-party GitHub Action is pinned to a full commit.
+- coverage.json can record how a review ran: why it stopped (completed, budget, time limit, three failed attempts in a row, interrupted), the passes run and the usage the host reported. A review that stopped early is always incomplete, and a unit is abandoned after three consecutive failed attempts.
+- bmad-plus review gate --emit-check followed by a file name writes the verdict as a check document for CI: disposition, reasons, open findings by severity with their lines, and a block shaped for GitHub check runs. The same evidence always gives the same file.
+- bmad-plus review continue resumes an interrupted review on the same sealed scope and lists what is still owed. It refuses when the code, the rules or a selected file changed, and points to a new scope and review compare.
+- Review rules can belong to a group (code, data, delivery or your own), so parallel reviewers can split a unit by rule family.
+- For maintainers, a review bench scores recorded review answers against planted defects and decoys with a published matching rule, and copy-ready CI recipes gate pull requests with a host agent on a self-hosted runner without trusting anything the agent can rewrite.
+- The SEO fetcher and crawler resolve each host themselves, refuse internal addresses and connect to the exact address they validated, on every redirect too, which defeats DNS rebinding. The MCP server uses the official SDK on Python 3.12 with current chromadb and sentence-transformers; re-run ingest.py once after upgrading.
 
 ## Version History
 
@@ -190,9 +190,9 @@ These dates identify reviewed CHANGELOG notes, not npm publication dates. The we
 
 | Version | Release-notes date | Reviewed summary |
 | --- | --- | --- |
+| 0.21.0 | 2026-09-28 | A review now records why it stopped and what it used, writes a check a CI step can read, and resumes on the scope it sealed. The SEO fetcher connects only to addresses it has validated, and the MCP server moves to the official SDK on Python 3.12. Nothing changes in how you install or update. |
+| 0.20.0 | 2026-09-28 | bmad-plus review now hands the reviewer the rules that apply to the changed files, fixes the depth of a review before it starts, compares a review with an earlier one without calling an unchecked finding fixed, and masks credentials in what it writes. Nothing changes in how you install or update. |
 | 0.19.0 | 2026-09-27 | bmad-plus review seals what a review must cover, locates every finding by the code it quotes, and derives the verdict from coverage instead of the number of findings. Review triage now keeps a finding unless a quoted ground refutes it. Nothing changes in how you install or update. |
-| 0.18.0 | 2026-09-25 | The acceptance page now verifies every save, restores your answers on reload before anything else, keeps runs from other revisions and other tabs apart, and names sixteen guarantees the build refuses to lose. Nothing changes in how you install or update; rebuild your recipe pages to get the new page. |
-| 0.17.1 | 2026-09-24 | Installing over a different version now stops and points to update, re-installs keep your settings, and updates remove files a version no longer ships while keeping a restorable backup. Some install behaviors changed; review your scripts before rerunning them. |
 
 [Release history and update guide](https://bmad-plus.rochetta.fr/docs/#changelog) · [All published npm versions](https://www.npmjs.com/package/bmad-plus?activeTab=versions)
 

package/package.json

@@ -1,7 +1,7 @@
 {
   "$schema": "https://json.schemastore.org/package.json",
   "name": "bmad-plus",
-  "version": "0.19.0",
+  "version": "0.21.0",
   "description": "Project-local AI development workflows with clear roles, shared context, safe updates and coding-tool adapters",
   "homepage": "https://bmad-plus.rochetta.fr",
   "keywords": [

package/readme-international/README.de.md

@@ -4,9 +4,9 @@
   <a href="../README.md">English</a> | <a href="README.fr.md">Français</a> | <a href="README.es.md">Español</a> | 🌐 <b>Deutsch</b>
 </div>
 
-[![Version](https://img.shields.io/badge/version-0.19.0-blue)](https://www.npmjs.com/package/bmad-plus)
+[![Version](https://img.shields.io/badge/version-0.21.0-blue)](https://www.npmjs.com/package/bmad-plus)
 
-**Version 0.19.0** · Node.js `>=20.0.0` · MIT
+**Version 0.21.0** · Node.js `>=20.0.0` · MIT
 
 Projektlokale KI-Entwicklungsworkflows mit klaren Rollen, gemeinsamem Kontext, sicheren Updates und Adaptern für Coding-Tools.
 
@@ -39,7 +39,7 @@ Starte das Installationsprogramm. Wähle die Adapter deiner KI-Werkzeuge und die
 **Im Terminal deines Projekts:**
 
 ```sh
-npx bmad-plus@0.19.0 install
+npx bmad-plus@0.21.0 install
 ```
 
 Die Installation erstellt Agentenanweisungen, die gemeinsame Projektbasis und die gewählten Adapter. Optionale Packs können weitere Laufzeitumgebungen oder API-Zugänge benötigen.
@@ -75,7 +75,7 @@ Bitte Sentinel, die Änderung anhand der Akzeptanzkriterien zu prüfen. Fordere
 **In deinem KI-Assistenten:**
 
 ```text
-Sentinel, prüfe diese Änderung anhand der Akzeptanzkriterien. Überprüfe die wichtigsten Erfolgs- und Fehlerfälle, nenne offene Probleme und fasse zusammen, was die nächste Sitzung wissen muss. Wenn das BMAD+ CLI verfügbar ist, versiegle den Umfang mit bmad-plus review scope, zitiere den Code jedes Befunds, belege jede ausgewählte Datei und berichte, was bmad-plus review gate antwortet.
+Sentinel, prüfe diese Änderung anhand der Akzeptanzkriterien. Überprüfe die wichtigsten Erfolgs- und Fehlerfälle, nenne offene Probleme und fasse zusammen, was die nächste Sitzung wissen muss. Wenn das BMAD+ CLI verfügbar ist, versiegle den Umfang mit bmad-plus review scope, wende die Regeln seiner Checkliste an, zitiere den Code jedes Befunds, belege jede ausgewählte Datei und berichte, was bmad-plus review gate antwortet.
 ```
 
 Lies die Änderungen und Testergebnisse. Der Assistent arbeitet innerhalb der Rechte und Fähigkeiten seines Host-Werkzeugs.
@@ -175,18 +175,18 @@ Lies AGENTS.md und die verfügbare Projekterinnerung. Fasse den letzten geprüft
 
 Erwartetes Ergebnis: ein kurzer, belegter Wiedereinstieg. Projektnotizen liefern Kontext und müssen mit dem aktuellen Stand abgeglichen werden.
 
-## Neu in 0.19.0
+## Neu in 0.21.0
 
-Code-Reviews, die sich prüfen lassen
+Reviews, die ehrlich anhalten, fortgesetzt werden und an die CI berichten
 
-bmad-plus review versiegelt, was ein Review abdecken muss, findet jeden Befund über den Code, den er zitiert, und leitet das Urteil aus der Abdeckung ab statt aus der Zahl der Befunde. Die Review-Sichtung behält einen Befund jetzt, solange kein zitierter Grund ihn widerlegt. An Installation und Update ändert sich nichts.
+Ein Review hält jetzt fest, warum es angehalten hat und was es verbraucht hat, schreibt eine Prüfung, die ein CI-Schritt lesen kann, und wird auf dem versiegelten Umfang fortgesetzt. Der SEO-Abruf verbindet sich nur mit Adressen, die er geprüft hat, und der MCP-Server wechselt zum offiziellen SDK unter Python 3.12. An Installation und Update ändert sich nichts.
 
-- bmad-plus review scope listet die seit dem Basis-Branch oder im Arbeitsverzeichnis geänderten Dateien, nennt für jede andere geänderte Datei den Grund des Ausschlusses (Geheimnis, binär, gelöscht, generiert), teilt die Arbeit in geordnete Einheiten und versiegelt alles mit einem Hash. Geheime Dateien werden vor jedem Einschlussmuster ausgeschlossen und kommen nie zurück.
-- Befunde stehen in findings.json und zitieren den betroffenen Code wörtlich. bmad-plus review anchor findet diesen Code in der Datei und meldet ein Zitat, das es nicht oder doppelt findet, statt einer Zeilennummer zu vertrauen. Eine unbekannte Kategorie oder Schwere wird abgelehnt, nie erraten.
-- bmad-plus review gate antwortet incomplete, findings oder clean. Ein Review bleibt unvollständig, solange eine ausgewählte Datei in coverage.json fehlt oder ein offener Befund nicht verankert ist; clean gilt nur für den geprüften Umfang.
-- Sentinel und der Dev-Studio-Workflow code-review behalten einen Befund standardmäßig und widerlegen ihn nur mit einem zitierten Grund: Der Code fehlt, der Code widerspricht ihm, oder er doppelt einen anderen Befund. Befunde zu Sicherheit, Datenverlust oder Geld ohne ein solches Zitat bleiben ungeklärt.
-- Das Maintainer-README nennt jetzt, was BMAD+ bewusst nicht tut: keine Modell-Laufzeit, keine Hintergrundprozesse, keine Review-Engine, die Befunde verschiebt, keine automatischen Korrekturen aus einem Review.
-- Die Veröffentlichungswerkzeuge akzeptieren das Ein-Element-Array, das npm jetzt bei Integritätsprüfungen liefert. Die Continuous Integration prüft, dass jede GitHub-Action von Dritten auf einen vollständigen Commit festgelegt ist.
+- coverage.json kann festhalten, wie ein Review verlief: warum es anhielt (abgeschlossen, Budget, Zeitlimit, drei Fehlversuche in Folge, unterbrochen), die ausgeführten Durchgänge und die vom Host gemeldete Nutzung. Ein vorzeitig angehaltenes Review ist immer unvollständig, und eine Einheit wird nach drei Fehlversuchen in Folge aufgegeben.
+- bmad-plus review gate --emit-check mit einem Dateinamen schreibt das Urteil als Prüfdokument für die CI: Ergebnis, Gründe, offene Befunde nach Schwere mit ihren Zeilen und einen Block im Format der GitHub-Check-Runs. Dieselben Belege ergeben immer dieselbe Datei.
+- bmad-plus review continue setzt ein unterbrochenes Review auf demselben versiegelten Umfang fort und listet, was noch aussteht. Es verweigert, wenn sich Code, Regeln oder eine ausgewählte Datei geändert haben, und verweist auf einen neuen Umfang und review compare.
+- Review-Regeln können einer Gruppe angehören (code, data, delivery oder eine eigene), damit parallele Prüfende eine Einheit nach Regelfamilie aufteilen.
+- Für Maintainer bewertet ein Review-Prüfstand aufgezeichnete Review-Antworten gegen eingebaute Fehler und Köder mit einer veröffentlichten Zuordnungsregel, und kopierfertige CI-Vorlagen lassen einen Pull Request von einem Host-Agenten auf einem selbst gehosteten Runner prüfen, ohne etwas zu vertrauen, das der Agent umschreiben kann.
+- SEO-Abruf und Crawler lösen jeden Host selbst auf, lehnen interne Adressen ab und verbinden sich genau mit der geprüften Adresse, auch bei jeder Weiterleitung, was DNS-Rebinding verhindert. Der MCP-Server nutzt das offizielle SDK unter Python 3.12 mit aktuellem chromadb und sentence-transformers; führe ingest.py nach dem Update einmal erneut aus.
 
 ## Versionsverlauf
 
@@ -194,9 +194,9 @@ Diese Daten bezeichnen geprüfte CHANGELOG-Einträge, nicht die Veröffentlichun
 
 | Version | Datum der Notizen | Geprüfte Zusammenfassung |
 | --- | --- | --- |
+| 0.21.0 | 2026-09-28 | Ein Review hält jetzt fest, warum es angehalten hat und was es verbraucht hat, schreibt eine Prüfung, die ein CI-Schritt lesen kann, und wird auf dem versiegelten Umfang fortgesetzt. Der SEO-Abruf verbindet sich nur mit Adressen, die er geprüft hat, und der MCP-Server wechselt zum offiziellen SDK unter Python 3.12. An Installation und Update ändert sich nichts. |
+| 0.20.0 | 2026-09-28 | bmad-plus review gibt der prüfenden Person jetzt die Regeln, die für die geänderten Dateien gelten, legt die Tiefe eines Reviews vor Beginn fest, vergleicht ein Review mit einem früheren, ohne einen ungeprüften Befund für behoben zu erklären, und schwärzt Zugangsdaten in dem, was es schreibt. An Installation und Update ändert sich nichts. |
 | 0.19.0 | 2026-09-27 | bmad-plus review versiegelt, was ein Review abdecken muss, findet jeden Befund über den Code, den er zitiert, und leitet das Urteil aus der Abdeckung ab statt aus der Zahl der Befunde. Die Review-Sichtung behält einen Befund jetzt, solange kein zitierter Grund ihn widerlegt. An Installation und Update ändert sich nichts. |
-| 0.18.0 | 2026-09-25 | Die Abnahmeseite prüft jetzt jedes Speichern, stellt deine Antworten beim Neuladen vor allem anderen wieder her, hält Durchläufe anderer Fassungen und anderer Tabs auseinander und benennt sechzehn Garantien, die der Build nicht verlieren darf. An Installation und Update ändert sich nichts; baue deine Rezeptseiten neu, um die neue Seite zu erhalten. |
-| 0.17.1 | 2026-09-24 | Eine Installation über eine andere Version bricht jetzt ab und verweist auf update, eine erneute Installation behält deine Einstellungen, und Updates entfernen Dateien, die eine Version nicht mehr mitliefert, mit wiederherstellbarer Sicherung. Einige Installationsabläufe ändern sich: Prüfe deine Skripte, bevor du sie erneut ausführst. |
 
 [Versionsverlauf und Update-Anleitung](https://bmad-plus.rochetta.fr/de/docs/#changelog) · [Alle veröffentlichten Versionen auf npm](https://www.npmjs.com/package/bmad-plus?activeTab=versions)
 

package/readme-international/README.es.md

@@ -4,9 +4,9 @@
   <a href="../README.md">English</a> | <a href="README.fr.md">Français</a> | 🌐 <b>Español</b> | <a href="README.de.md">Deutsch</a>
 </div>
 
-[![Version](https://img.shields.io/badge/version-0.19.0-blue)](https://www.npmjs.com/package/bmad-plus)
+[![Version](https://img.shields.io/badge/version-0.21.0-blue)](https://www.npmjs.com/package/bmad-plus)
 
-**Versión 0.19.0** · Node.js `>=20.0.0` · MIT
+**Versión 0.21.0** · Node.js `>=20.0.0` · MIT
 
 Flujos de desarrollo con IA propios de cada proyecto, con roles claros, contexto compartido, actualizaciones seguras y adaptadores para herramientas de programación.
 
@@ -39,7 +39,7 @@ Ejecuta el instalador. Selecciona los adaptadores de tus herramientas de IA y lo
 **En la terminal del proyecto:**
 
 ```sh
-npx bmad-plus@0.19.0 install
+npx bmad-plus@0.21.0 install
 ```
 
 El instalador crea las instrucciones de los agentes, la base compartida y los adaptadores seleccionados. Algunos packs opcionales necesitan otros entornos o acceso a API.
@@ -75,7 +75,7 @@ Pide a Sentinel que compruebe el cambio según los criterios de aceptación. Sol
 **En tu asistente de IA:**
 
 ```text
-Sentinel, revisa este cambio según los criterios de aceptación. Comprueba los principales casos de éxito y fallo, informa de los problemas pendientes y resume lo que necesita saber la próxima sesión. Si el CLI de BMAD+ está disponible, sella el alcance con bmad-plus review scope, cita el código de cada hallazgo, da cuenta de cada archivo seleccionado y comunica lo que responde bmad-plus review gate.
+Sentinel, revisa este cambio según los criterios de aceptación. Comprueba los principales casos de éxito y fallo, informa de los problemas pendientes y resume lo que necesita saber la próxima sesión. Si el CLI de BMAD+ está disponible, sella el alcance con bmad-plus review scope, aplica las reglas de su lista de revisión, cita el código de cada hallazgo, da cuenta de cada archivo seleccionado y comunica lo que responde bmad-plus review gate.
 ```
 
 Lee los cambios y los resultados de las pruebas. El asistente sigue los permisos y las capacidades de su herramienta anfitriona.
@@ -175,18 +175,18 @@ Lee AGENTS.md y la memoria de proyecto disponible. Resume el último estado veri
 
 Resultado esperado: una reanudación breve y fundamentada. Las notas aportan contexto que debe contrastarse con el trabajo actual.
 
-## Novedades de la 0.19.0
+## Novedades de la 0.21.0
 
-Revisiones de código verificables
+Revisiones que se detienen con honestidad, se reanudan e informan a la CI
 
-bmad-plus review sella lo que una revisión debe cubrir, localiza cada hallazgo por el código que cita y deduce el veredicto de la cobertura en lugar del número de hallazgos. La clasificación de las revisiones conserva ahora un hallazgo mientras ningún motivo citado lo refute. Nada cambia en la instalación ni en la actualización.
+Una revisión registra ahora por qué se detuvo y qué consumió, escribe un control que un paso de CI puede leer y se reanuda sobre el alcance que selló. El descargador SEO solo se conecta a direcciones que ha validado, y el servidor MCP pasa al SDK oficial con Python 3.12. Nada cambia en la instalación ni en la actualización.
 
-- bmad-plus review scope enumera los archivos modificados desde la rama base o en tu árbol de trabajo, indica por qué se descarta cada otro archivo modificado (secreto, binario, eliminado, generado), divide el trabajo en lotes ordenados y lo sella con una huella. Los archivos secretos se descartan antes de cualquier patrón de inclusión y nunca vuelven.
-- Los hallazgos van en findings.json y citan literalmente el código al que se refieren. bmad-plus review anchor encuentra ese código en el archivo e informa de una cita que no encuentra, o que encuentra dos veces, en lugar de fiarse de un número de línea. Una categoría o gravedad desconocida se rechaza, nunca se adivina.
-- bmad-plus review gate responde incomplete, findings o clean. Una revisión sigue incompleta mientras falte un archivo seleccionado en coverage.json o un hallazgo abierto no esté anclado; clean solo cubre el alcance revisado.
-- Sentinel y el flujo code-review de Dev Studio conservan un hallazgo por defecto y solo lo refutan con un motivo citado: el código no está, el código lo contradice o duplica otro hallazgo. Un hallazgo de seguridad, pérdida de datos o dinero sin esa cita queda sin resolver.
-- El README de mantenimiento indica ahora lo que BMAD+ no hace a propósito: sin motor de modelos, sin procesos en segundo plano, sin revisión que reubique los hallazgos, sin correcciones automáticas a partir de una revisión.
-- Las herramientas de publicación aceptan la matriz de un elemento que npm devuelve ahora en las comprobaciones de integridad. La integración continua comprueba que cada acción de GitHub de terceros esté fijada a un commit completo.
+- coverage.json puede registrar cómo transcurrió una revisión: por qué se detuvo (completada, presupuesto, límite de tiempo, tres intentos fallidos seguidos, interrumpida), las pasadas realizadas y el uso que informó el host. Una revisión detenida antes de tiempo siempre está incompleta, y una unidad se abandona tras tres intentos fallidos consecutivos.
+- bmad-plus review gate --emit-check seguido de un nombre de archivo escribe el veredicto como un documento de control para la CI: disposición, motivos, hallazgos abiertos por gravedad con sus líneas y un bloque con el formato de los check runs de GitHub. Las mismas pruebas dan siempre el mismo archivo.
+- bmad-plus review continue reanuda una revisión interrumpida sobre el mismo alcance sellado y enumera lo que queda pendiente. Se niega cuando cambiaron el código, las reglas o un archivo seleccionado, y remite a un nuevo alcance y a review compare.
+- Las reglas de revisión pueden pertenecer a un grupo (code, data, delivery o uno propio), para que revisores en paralelo se repartan una unidad por familia de reglas.
+- Para mantenedores, un banco de pruebas puntúa respuestas de revisión grabadas frente a defectos sembrados y señuelos con una regla de correspondencia publicada, y recetas de CI listas para copiar hacen pasar una pull request por un agente en un runner autoalojado sin fiarse de nada que el agente pueda reescribir.
+- El descargador y el rastreador SEO resuelven cada host por sí mismos, rechazan direcciones internas y se conectan exactamente a la dirección validada, también en cada redirección, lo que neutraliza el DNS rebinding. El servidor MCP usa el SDK oficial con Python 3.12 y chromadb y sentence-transformers actuales; vuelve a ejecutar ingest.py una vez tras actualizar.
 
 ## Historial de versiones
 
@@ -194,9 +194,9 @@ Estas fechas identifican las notas revisadas del CHANGELOG, no las fechas de pub
 
 | Versión | Fecha de las notas | Resumen revisado |
 | --- | --- | --- |
+| 0.21.0 | 2026-09-28 | Una revisión registra ahora por qué se detuvo y qué consumió, escribe un control que un paso de CI puede leer y se reanuda sobre el alcance que selló. El descargador SEO solo se conecta a direcciones que ha validado, y el servidor MCP pasa al SDK oficial con Python 3.12. Nada cambia en la instalación ni en la actualización. |
+| 0.20.0 | 2026-09-28 | bmad-plus review ahora entrega a quien revisa las reglas que se aplican a los archivos modificados, fija la profundidad de una revisión antes de empezar, compara una revisión con otra anterior sin dar por corregido un hallazgo no comprobado y oculta las credenciales en lo que escribe. Nada cambia en la instalación ni en la actualización. |
 | 0.19.0 | 2026-09-27 | bmad-plus review sella lo que una revisión debe cubrir, localiza cada hallazgo por el código que cita y deduce el veredicto de la cobertura en lugar del número de hallazgos. La clasificación de las revisiones conserva ahora un hallazgo mientras ningún motivo citado lo refute. Nada cambia en la instalación ni en la actualización. |
-| 0.18.0 | 2026-09-25 | La página de aceptación ahora verifica cada guardado, restaura tus respuestas al recargar antes que nada, mantiene separadas las pruebas de otras versiones y otras pestañas, y nombra dieciséis garantías que la construcción se niega a perder. Nada cambia en la instalación ni en la actualización; reconstruye tus páginas de receta para obtener la nueva página. |
-| 0.17.1 | 2026-09-24 | Instalar sobre otra versión ahora se detiene y remite a update, una reinstalación conserva tus ajustes y las actualizaciones eliminan los archivos que una versión ya no incluye, con una copia restaurable. Algunos comportamientos de instalación cambian: revisa tus scripts antes de volver a ejecutarlos. |
 
 [Historial de versiones y guía de actualización](https://bmad-plus.rochetta.fr/es/docs/#changelog) · [Todas las versiones publicadas en npm](https://www.npmjs.com/package/bmad-plus?activeTab=versions)
 

package/readme-international/README.fr.md

@@ -4,9 +4,9 @@
   <a href="../README.md">English</a> | 🌐 <b>Français</b> | <a href="README.es.md">Español</a> | <a href="README.de.md">Deutsch</a>
 </div>
 
-[![Version](https://img.shields.io/badge/version-0.19.0-blue)](https://www.npmjs.com/package/bmad-plus)
+[![Version](https://img.shields.io/badge/version-0.21.0-blue)](https://www.npmjs.com/package/bmad-plus)
 
-**Version 0.19.0** · Node.js `>=20.0.0` · MIT
+**Version 0.21.0** · Node.js `>=20.0.0` · MIT
 
 Des workflows de développement IA propres à chaque projet, avec des rôles clairs, un contexte partagé, des mises à jour sûres et des adaptateurs pour vos outils de code.
 
@@ -39,7 +39,7 @@ Lancez l’installateur. Sélectionnez les adaptateurs de vos outils IA et les p
 **Dans le terminal du projet :**
 
 ```sh
-npx bmad-plus@0.19.0 install
+npx bmad-plus@0.21.0 install
 ```
 
 L’installateur crée les instructions des agents, le socle partagé et les adaptateurs sélectionnés. Certains packs facultatifs nécessitent d’autres environnements ou des accès API.
@@ -75,7 +75,7 @@ Demandez à Sentinel de contrôler le changement selon les critères d’accepta
 **Dans votre assistant IA :**
 
 ```text
-Sentinel, relis ce changement selon les critères d’acceptation. Vérifie les principaux cas de réussite et d’échec, signale les problèmes restants et résume ce que la prochaine session doit savoir. Si le CLI BMAD+ est disponible, scelle le périmètre avec bmad-plus review scope, cite le code de chaque remarque, rends compte de chaque fichier retenu, puis rapporte ce que répond bmad-plus review gate.
+Sentinel, relis ce changement selon les critères d’acceptation. Vérifie les principaux cas de réussite et d’échec, signale les problèmes restants et résume ce que la prochaine session doit savoir. Si le CLI BMAD+ est disponible, scelle le périmètre avec bmad-plus review scope, applique les règles de sa checklist, cite le code de chaque remarque, rends compte de chaque fichier retenu, puis rapporte ce que répond bmad-plus review gate.
 ```
 
 Relisez les changements et les résultats des tests. L’assistant suit les autorisations et les capacités de son outil hôte.
@@ -175,18 +175,18 @@ Lis AGENTS.md et la mémoire de projet disponible. Résume le dernier état vér
 
 Résultat attendu : une reprise courte et fondée sur des éléments vérifiés. Les notes apportent du contexte, à confronter au travail actuel.
 
-## Nouveautés de la 0.19.0
+## Nouveautés de la 0.21.0
 
-Des revues de code vérifiables
+Des revues qui s’arrêtent honnêtement, reprennent et parlent à la CI
 
-bmad-plus review scelle ce qu’une revue doit couvrir, retrouve chaque remarque par le code qu’elle cite, et déduit le verdict de la couverture plutôt que du nombre de remarques. Le tri des revues garde désormais une remarque tant qu’un motif cité ne la réfute pas. Rien ne change dans l’installation ou la mise à jour.
+Une revue enregistre désormais pourquoi elle s’est arrêtée et ce qu’elle a consommé, écrit un contrôle lisible par une étape de CI, et reprend sur le périmètre qu’elle a scellé. Le récupérateur SEO ne se connecte qu’aux adresses qu’il a validées, et le serveur MCP passe au SDK officiel sous Python 3.12. Rien ne change dans l’installation ou la mise à jour.
 
-- bmad-plus review scope liste les fichiers modifiés depuis la branche de base ou dans votre arbre de travail, dit pourquoi chaque autre fichier modifié est écarté (secret, binaire, supprimé, généré), découpe le travail en lots ordonnés et scelle le tout par une empreinte. Les fichiers secrets sont écartés avant tout motif d’inclusion et ne reviennent jamais.
-- Les remarques vont dans findings.json et citent mot pour mot le code qu’elles visent. bmad-plus review anchor retrouve ce code dans le fichier et signale une citation introuvable, ou trouvée deux fois, au lieu de se fier à un numéro de ligne. Une catégorie ou une sévérité inconnue est refusée, jamais devinée.
-- bmad-plus review gate répond incomplete, findings ou clean. Une revue reste incomplète tant qu’un fichier retenu manque dans coverage.json ou qu’une remarque ouverte n’est pas ancrée ; clean ne couvre que le périmètre revu.
-- Sentinel et le workflow code-review de Dev Studio gardent une remarque par défaut et ne la réfutent que sur un motif cité : le code est absent, le code la contredit, ou elle en double une autre. Une remarque sur la sécurité, la perte de données ou l’argent sans cette citation reste non résolue.
-- Le README mainteneur dit désormais ce que BMAD+ ne fait pas, volontairement : pas de moteur de modèle, pas de processus d’arrière-plan, pas de revue qui repositionne les remarques, pas de correctif automatique issu d’une revue.
-- Les outils de publication acceptent le tableau à un élément que npm renvoie désormais pour les contrôles d’intégrité. L’intégration continue vérifie que chaque action GitHub tierce est épinglée sur un commit complet.
+- coverage.json peut enregistrer le déroulé d’une revue : pourquoi elle s’est arrêtée (terminée, budget, limite de temps, trois échecs de suite, interrompue), les passes faites et l’usage rapporté par l’hôte. Une revue arrêtée avant la fin est toujours incomplète, et une unité est abandonnée après trois échecs consécutifs.
+- bmad-plus review gate --emit-check suivi d’un nom de fichier écrit le verdict sous forme de contrôle pour la CI : disposition, raisons, remarques ouvertes par gravité avec leurs lignes, et un bloc au format des check runs GitHub. Les mêmes preuves donnent toujours le même fichier.
+- bmad-plus review continue reprend une revue interrompue sur le même périmètre scellé et liste ce qui reste dû. Il refuse quand le code, les règles ou un fichier retenu ont changé, et renvoie vers un nouveau périmètre et review compare.
+- Les règles de revue peuvent appartenir à un groupe (code, data, delivery ou le vôtre), pour que des relecteurs parallèles se partagent une unité par famille de règles.
+- Pour les mainteneurs, un banc d’essai note des réponses de revue enregistrées face à des défauts plantés et des leurres, avec une règle de correspondance publiée, et des modèles de CI prêts à copier font passer une pull request par un agent hôte sur un runner auto-hébergé sans rien croire de ce que l’agent peut réécrire.
+- Le récupérateur et le crawler SEO résolvent eux-mêmes chaque hôte, refusent les adresses internes et se connectent exactement à l’adresse validée, redirections comprises, ce qui neutralise le DNS rebinding. Le serveur MCP utilise le SDK officiel sous Python 3.12 avec chromadb et sentence-transformers à jour ; relancez ingest.py une fois après la mise à jour.
 
 ## Historique des versions
 
@@ -194,9 +194,9 @@ Ces dates identifient les notes relues du CHANGELOG, pas les dates de publicatio
 
 | Version | Date des notes | Résumé relu |
 | --- | --- | --- |
+| 0.21.0 | 2026-09-28 | Une revue enregistre désormais pourquoi elle s’est arrêtée et ce qu’elle a consommé, écrit un contrôle lisible par une étape de CI, et reprend sur le périmètre qu’elle a scellé. Le récupérateur SEO ne se connecte qu’aux adresses qu’il a validées, et le serveur MCP passe au SDK officiel sous Python 3.12. Rien ne change dans l’installation ou la mise à jour. |
+| 0.20.0 | 2026-09-28 | bmad-plus review donne désormais au relecteur les règles qui s’appliquent aux fichiers modifiés, fixe la profondeur d’une revue avant qu’elle commence, compare une revue avec une précédente sans déclarer corrigée une remarque non vérifiée, et masque les identifiants dans ce qu’il écrit. Rien ne change dans l’installation ou la mise à jour. |
 | 0.19.0 | 2026-09-27 | bmad-plus review scelle ce qu’une revue doit couvrir, retrouve chaque remarque par le code qu’elle cite, et déduit le verdict de la couverture plutôt que du nombre de remarques. Le tri des revues garde désormais une remarque tant qu’un motif cité ne la réfute pas. Rien ne change dans l’installation ou la mise à jour. |
-| 0.18.0 | 2026-09-25 | La page de recette vérifie désormais chaque enregistrement, restaure vos réponses au rechargement avant toute autre chose, distingue les passages d’autres révisions et d’autres onglets, et nomme seize garanties que la construction refuse de perdre. Rien ne change dans l’installation ou la mise à jour ; reconstruisez vos pages de recette pour obtenir la nouvelle page. |
-| 0.17.1 | 2026-09-24 | Installer par-dessus une autre version s’arrête désormais et renvoie vers update, une réinstallation conserve vos réglages, et les mises à jour retirent les fichiers qu’une version ne livre plus en gardant une sauvegarde restaurable. Certains comportements d’installation changent : vérifiez vos scripts avant de les relancer. |
 
 [Historique des versions et guide de mise à jour](https://bmad-plus.rochetta.fr/fr/docs/#changelog) · [Toutes les versions publiées sur npm](https://www.npmjs.com/package/bmad-plus?activeTab=versions)
 

package/src/bmad-plus/agents/agent-quality/SKILL.md

@@ -62,7 +62,7 @@ When auto-activating a role, **announce it**: "💡 I'm switching to [Role] mode
 3. Verify each suspected issue before grouping findings. Trace its trigger through the actual caller and guards, inspect counterevidence, and reproduce it when practical. A failure in an unreachable state is not an established defect. Check every source, including a reviewer that describes its own findings as pre-verified.
 4. Keep a compact finding record with identity, source, location, claimed consequence, evidence or refutation, disposition, relevant input identity and next action. The default disposition is keep: refute only on a quoted ground — the described construct is absent from the file, the code contradicts the claim (guard, caller or test quoted), or another entry already carries the same cause — and write the analysis before the disposition. A security, data-loss or money finding without such a quote stays unresolved. Use confirmed, refuted or unresolved; rank confirmed defects by impact and likelihood. Preserve previous entries when their disposition changes. Group confirmed findings only when the same cause explains them, retaining each source record. Do not merge away a refutation or impose a finding quota.
 5. Inspect verification at the actual consumer. Read assertions and check how tests are selected before judging their coverage. Search relevant symbols and imports before asserting that coverage is absent. For a verification gap, name the consumer and a concrete regression or missed adoption that the current checks would fail to catch. Skipped tests, mocks that bypass the changed path and helper-only success do not prove that consumer works.
-6. When the BMAD+ CLI is available, back a code review with its evidence: `bmad-plus review scope <id>` before reading, then `findings.json` with every finding quoting its code verbatim and `coverage.json` accounting for every selected file, then `bmad-plus review anchor <id>` and `bmad-plus review gate <id>`. Report the gate's disposition (incomplete, findings or clean) without upgrading it.
+6. When the BMAD+ CLI is available, back a code review with its evidence: `bmad-plus review scope <id>` before reading (its `checklist.md` holds the review rules that apply to the changed paths, and `--effort` sets the passes), then `findings.json` with every finding quoting its code verbatim and `coverage.json` accounting for every selected file, then `bmad-plus review anchor <id>` and `bmad-plus review gate <id>`. Record in `coverage.json` how the session ended (`run.stop`), the passes actually run and every unit attempt; stop retrying a unit or rule group after three failures in a row, and never estimate tokens or time. Report the gate's disposition (incomplete, findings or clean) without upgrading it; a stopped review is never clean. To resume an interrupted review, `bmad-plus review continue <id>`: it refuses when the code or the rules moved. On a second review of the same work, `bmad-plus review compare <id> --since <earlier-id>`: a finding gone from a file this review did not complete is not reviewed, never fixed.
 7. Reconcile acceptance using executed commands or direct observations against the current artifact set. Record passed, failed, skipped and unavailable checks separately. A missing required reviewer or untested criterion prevents a completed-acceptance claim, even when the remaining checks pass. A same-author review must not be described as independent verification.
 
 ## Repair and continuation

package/src/bmad-plus/packs/pack-dev-studio/categories/implementation/code-review.md

@@ -21,7 +21,17 @@ arbitrary branch. Read the intended behavior and applicable project constraints.
    `bmad-plus review scope <id> --base <ref>` (or `--workspace` for uncommitted
    work) writes `_bmad-output/review/<id>/scope.json` with every selected file,
    every excluded file and its reason (secret, binary, deleted, generated), and
-   ordered review units. Never read or quote an excluded secret file. Inventory
+   ordered review units. Never read or quote an excluded secret file. Choose the
+   depth with `--effort low|medium|high` (one pass without refutation; two passes
+   with refutation, the default; three passes) and follow the `plan` the scope
+   prints: survey the whole change before reading when it says `planFirst`, and
+   review units in parallel when it says `parallelUnits`. Read `checklist.md`
+   beside the scope: it holds, once each, the review rules that apply to the
+   selected paths — built-in rules by language and file kind, plus the project's
+   own from `_bmad/review-rules.yaml`, which can add, replace or disable rules.
+   `bmad-plus review rules <path>` shows which rules a file gets. Each unit
+   lists its rule groups (`code`, `data`, `delivery` or the project's own):
+   parallel reviewers may split a unit by group. Inventory
    related callers, tests and requirements. Preserve existing edits and record
    unavailable context. A supplied diff may omit the surrounding behavior needed
    to evaluate it.
@@ -63,7 +73,25 @@ arbitrary branch. Read the intended behavior and applicable project constraints.
    and requote any finding it reports ambiguous or unlocated; then
    `bmad-plus review gate <id>`. Report its disposition as it is: `incomplete`
    when a selected file is unaccounted for or a finding is not anchored, `findings`
-   or `clean` otherwise. `clean` covers the reviewed scope only.
+   or `clean` otherwise. `clean` covers the reviewed scope only. A finding that
+   applies a checklist rule names it in `rule`; the CLI refuses a rule that does
+   not apply to that path. The anchored record replaces credential-like values
+   quoted in a finding with `[REDACTED]` and counts them. Record the session in
+   `coverage.json` under `run`: `stop` (`completed`, `budget`, `time-limit`,
+   `failure-streak` or `interrupted`, with a `detail` unless completed), the
+   `passes` actually run, every unit attempt in order (`unit`, the rule `group`
+   when reviewers split by group, `completed` or `failed` with a reason), and
+   tokens or duration only when the host reports them — never estimate them.
+   After three failed attempts in a row on the same unit or group, stop
+   retrying it: mark its files failed. A stopped review is never clean. In CI,
+   `bmad-plus review gate <id> --emit-check <file.json>` also writes the verdict
+   as a check result (`bmad-plus/review-check/1`, with a GitHub check-runs
+   payload); the exit code stays the verdict.
+9. On a second review of the same work, compare it with the earlier one:
+   `bmad-plus review compare <id> --since <earlier-id>` writes `compare.json`
+   with each finding new, persisting, refuted, resolved or not reviewed. An
+   earlier finding counts as resolved only when this review completed its file;
+   otherwise it is not reviewed, never fixed. Renamed files are followed.
 
 ## Output and acceptance
 
@@ -79,7 +107,14 @@ or an untested requirement.
 
 ## Continue
 
-Compare the current diff and input hashes with the reviewed snapshot. Preserve
-prior findings and check their resolution against actual changes. Re-review
+With a sealed scope, run `bmad-plus review continue <id>` first. When the code,
+its head ref or the review rules moved, it refuses: seal a new scope and compare
+it with this one (step 9) instead of continuing. Otherwise `continue.json` lists
+the units, files and rule groups still owed, the files abandoned after three
+failures (they stay failed), the findings to requote and the passes left; add to
+the same `coverage.json` and `findings.json` and record the new `run.stop`.
+
+Without a sealed scope, compare the current diff and input hashes with the
+reviewed snapshot. Preserve prior findings and check their resolution against actual changes. Re-review
 affected behavior and invalidate conclusions that relied on changed inputs;
 do not rerun unchanged checks merely to refresh the report date.

package/src/bmad-plus/packs/pack-seo/scripts/seo_crawl.py

@@ -32,13 +32,14 @@ except ImportError:
     )
     sys.exit(1)
 
-# Reuse the hardened SSRF guard from seo_fetch (same package/directory).
+# Reuse the hardened SSRF guard and pinned transport from seo_fetch
+# (same package/directory).
 sys.path.insert(0, os.path.dirname(os.path.abspath(__file__)))
 try:
-    from seo_fetch import is_safe_url
+    from seo_fetch import ALLOWED_SCHEMES, UnsafeURLError, create_session
 except ImportError:
     print(
-        "Error: seo_fetch.is_safe_url is required (SSRF protection). "
+        "Error: seo_fetch.create_session is required (SSRF protection). "
         "Ensure seo_fetch.py is present alongside seo_crawl.py.",
         file=sys.stderr,
     )
@@ -65,6 +66,8 @@ USER_AGENT = (
     "(KHTML, like Gecko) Chrome/131.0.0.0 Safari/537.36 BMADSEOEngine/2.0"
 )
 
+MAX_REDIRECTS = 5
+
 
 class SEOCrawler:
     """Recursive mini-crawler for SEO site structure analysis."""
@@ -82,6 +85,7 @@ class SEOCrawler:
         self.sitemap_urls: list = []
         self.robots_txt: Optional[str] = None
         self.errors: list = []
+        self.session = create_session()
 
     def normalize_url(self, url: str) -> str:
         """Normalize URL for deduplication."""
@@ -96,42 +100,50 @@ class SEOCrawler:
     def _safe_get(self, url: str, error_prefix: str = "Blocked"):
         """GET a URL, following redirects manually with per-hop SSRF revalidation.
 
-        Redirects are followed with allow_redirects=False so is_safe_url() runs on
-        EVERY hop: a public URL that 302s to an internal/metadata endpoint
-        (redirect-based SSRF) is refused. Returns the final requests.Response, or
-        None (with an entry appended to self.errors) when a hop is unsafe or too
-        many redirects occur. Used by fetch(), fetch_robots_txt() and
-        parse_sitemap() so every network path shares the same guard.
+        Every request goes through the pinned session from seo_fetch, which
+        validates the target and connects to the address it validated (no DNS
+        rebinding between check and connect). Redirects are followed with
+        allow_redirects=False so each hop is validated the same way: a public URL
+        that 302s to an internal/metadata endpoint (redirect-based SSRF) is
+        refused. Returns the final requests.Response, or None (with an entry
+        appended to self.errors) when a hop is unsafe or too many redirects
+        occur. Used by fetch(), fetch_robots_txt() and parse_sitemap() so every
+        network path shares the same guard.
         """
-        if not is_safe_url(url):
-            self.errors.append(
-                {"url": url, "error": f"{error_prefix}: private/internal URL (SSRF protection)"}
-            )
-            return None
         current_url = url
         hops = 0
         while True:
-            response = requests.get(
-                current_url,
-                headers={"User-Agent": USER_AGENT},
-                timeout=self.timeout,
-                allow_redirects=False,
-            )
+            try:
+                response = self.session.get(
+                    current_url,
+                    headers={"User-Agent": USER_AGENT},
+                    timeout=self.timeout,
+                    allow_redirects=False,
+                )
+            except UnsafeURLError as e:
+                prefix = error_prefix if current_url == url else f"{error_prefix} redirect"
+                self.errors.append(
+                    {"url": current_url, "error": f"{prefix}: {e} (SSRF protection)"}
+                )
+                return None
             if not response.is_redirect:
                 return response
             location = response.headers.get("Location")
             if not location:
                 return response
+            response.close()
             next_url = urljoin(current_url, location)
-            if urlparse(next_url).scheme not in ("http", "https") or not is_safe_url(next_url):
+            if urlparse(next_url).scheme not in ALLOWED_SCHEMES:
                 self.errors.append(
                     {"url": next_url,
-                     "error": f"{error_prefix} redirect: non-HTTP(S) or private/internal URL (SSRF protection)"}
+                     "error": f"{error_prefix} redirect: non-HTTP(S) URL (SSRF protection)"}
                 )
                 return None
             hops += 1
-            if hops > 5:
-                self.errors.append({"url": current_url, "error": "Too many redirects (max 5)"})
+            if hops > MAX_REDIRECTS:
+                self.errors.append(
+                    {"url": current_url, "error": f"Too many redirects (max {MAX_REDIRECTS})"}
+                )
                 return None
             current_url = next_url