@dozimple/abap-adt 1.0.1 → 1.2.0

package/CHANGELOG.md

@@ -4,6 +4,82 @@ Formato basado en [Keep a Changelog](https://keepachangelog.com/es-ES/1.1.0/). V
 
 ## [Unreleased]
 
+## [1.2.0] - 2026-09-23
+
+Cuatro tools para diagnóstico y soporte: dumps agrupados por periodo, búsqueda de texto en el código, estado de notas SAP y ampliaciones/BAdI. Todas verificadas en vivo en un NW 7.50.
+
+### Añadido
+- **`enhancements`: ampliaciones y BAdI.** Tres preguntas en una tool, por SQL de diccionario (7.50 y S/4):
+  implementaciones de una BAdI, nuevas y clásicas, activas o no, con las propias (Z/Y) primero; ampliaciones de código
+  implementadas dentro de un programa, clase o grupo de funciones, avisando de las que **sustituyen** código estándar
+  (overwrite); y todas las implementaciones de un espacio de nombres por tipo, con el total real. Lo que cambia el
+  comportamiento de un estándar sin tocar su código, que es lo primero a mirar en un incidente. Verificado en vivo en
+  un 7.50: 1.070 implementaciones Z (566 de código, 501 de BAdI).
+- **`sap_notes`: estado de notas SAP en el sistema, como en SNOTE**, sin portal ni S-user: si la nota está descargada,
+  estado de implementación y de tratamiento, versión, componente y título (español si existe). Por notas concretas
+  («¿está la 2198647 en PRD?») o filtrando por estado y/o prefijo de componente, con el total real. Distingue «no
+  descargada» de «no implementada». Los códigos de estado no son valores fijos del diccionario: su significado sale de
+  las constantes de SAP (`IF_SCWN_NA_CONSTANTS`), verificadas en un 7.50. En sistemas con datos productivos no se
+  muestra quién trató la nota. Sustituto seguro del MCP de notas de SAP, que usa APIs privadas y el S-user.
+- **`source_search`: buscar texto en el código de un paquete, una orden o una lista de objetos.** Texto literal o
+  expresión regular (máx. 200 caracteres), sin distinguir mayúsculas; devuelve objeto, include, línea y la línea
+  encontrada, con salida estructurada. Lee cada fuente (programas, clases con sus includes locales, interfaces,
+  módulos e includes propios de los grupos de funciones, CDS) con 4 lecturas en paralelo, progreso y cancelación;
+  exige un alcance, tiene tope de objetos (150 por defecto, ~0,8 s por objeto medido en 7.50) y lista aparte los
+  objetos que no pudo leer. Existe porque ADT no tiene búsqueda de texto en NW 7.50 (`textsearch` da 404 y no está
+  en el discovery), y llega donde `where_used` no: verificado en vivo, `where_used` no encontró ningún uso de una
+  clase de excepción y `source_search` encontró 49 en 6 objetos del mismo paquete.
+- **`dumps(group_by, days)`: los errores más frecuentes de un periodo.** Cuenta los dumps de los últimos N días (hasta
+  90) agrupados por error, programa, error+programa, usuario o día, con primer y último caso, usuarios distintos
+  afectados y dónde terminó el más reciente (include y línea); salida estructurada. Los grupos salen de la cabecera
+  de ST22 (`SNAP_BEG.FLIST`, solo metadatos: `SNAP`, que guarda valores de variables, sigue vetada), porque el feed
+  ADT solo trae los dumps más recientes: en un 7.50 real devolvió 5 con miles en ST22, y su parámetro `from` no
+  amplía la ventana. En sistemas con datos productivos no se agrupa por usuario (columna personal); el número de
+  usuarios afectados sí se da. Cierra el hueco anotado el 11-09.
+
+### Corregido
+- `package_contents` listaba objetos **borrados** que TADIR conserva con `DELFLAG = 'X'`; ahora no aparecen.
+
+## [1.1.0] - 2026-09-22
+
+Tool nueva y contrato MCP ampliado (salida estructurada, progreso, cancelación) más resiliencia de conexión: el sprint 1 y parte del 2 del plan de mejoras.
+
+### Añadido
+- **Progreso y cancelación** (sprint 2 del plan): las tools largas informan de su avance (`transport_diff` objeto a
+  objeto; `run_atc`, `where_used` y `run_unit_tests` por fase) y el cliente lo recibe como notificaciones de progreso
+  MCP si las pidió. La cancelación del cliente se respeta **entre pasos**: el paso en curso termina (una llamada ADT
+  no se puede abortar) y el siguiente ya no empieza. Una petición cancelada termina como `CANCELLED`, un tipo propio
+  que no cuenta como fallo del servidor ni abre el circuito.
+- **Salida estructurada** (`outputSchema` / `structuredContent` de MCP), sprint 2 del plan: una tool puede declarar
+  `output` y devolver los mismos datos del texto de forma tipada, para que el cliente no tenga que interpretar la
+  prosa. El registro exige que toda respuesta no errónea de esas tools la traiga (si falta es error del servidor,
+  no un éxito a medias) y el SDK la valida contra el esquema antes de responder. Primeras tools: `sql_query` (filas,
+  columnas, valores hasta 500, `truncated`, avisos) y `syntax_check` (errores, avisos y mensajes con línea y
+  severidad). `docs/TOOLS.md` documenta el esquema de salida de cada una.
+- **`revert_source`: volver a una versión anterior con confirmación.** Tras un `write_source` cuya activación falló,
+  el objeto queda con un borrador inactivo encima de la versión activa; la tool vuelve a escribir la versión elegida
+  (`active`: la última activa; `previous`: la anterior a la activa; `N`: una del historial de `object_versions`)
+  pasando por la misma vista previa, huella, bloqueo y orden que cualquier escritura. Nunca revierte por su cuenta:
+  un rollback automático que pisara una versión sin preguntar sería peor que dejar el objeto inactivo.
+- **Resiliencia de conexión** (sprint 1 del plan de mejoras):
+  - **Sesión caducada renovada en las lecturas.** Si una tool de lectura recibe un token CSRF rechazado o un 401 en
+    una sesión que ya había entrado, el servidor descarta el cliente, vuelve a entrar y repite la lectura una vez;
+    la respuesta lo anota. Nunca en escrituras ni ejecuciones: el bloqueo se perdió con la sesión y un reintento
+    podría escribir dos veces. Una contraseña rechazada sigue fallando a la primera y sigue olvidándose.
+  - **Circuit breaker por sistema.** Tres fallos de red en un minuto abren el circuito de ESE sistema durante un
+    minuto: toda tool responde al instante «no se vuelve a intentar durante N s» en vez de esperar 120 s por llamada.
+    Los demás sistemas no se ven afectados; `sap_systems(check=true)` lo cierra y reintenta. Solo cuentan los fallos
+    de red reales de la librería, no los tiempos agotados propios.
+  - **Tiempo máximo por tool** (`timeoutMs`, por defecto 60 s; ATC y diff de orden 180 s, where-used y ABAP Unit
+    120 s). Al vencer, error `NETWORK` con el tiempo y la sugerencia de acotar; no se reintenta.
+
+### Cambiado
+- Insignia de **OpenSSF Best Practices (Passing)** en el README: el proyecto cumple los 67 criterios del nivel
+  Passing, incluidas las sugerencias, con la ficha pública en https://www.bestpractices.dev/projects/14759.
+- El job de publicación usa Node 24, que ya trae npm >= 11.5.1: se quita la instalación global de npm, que no se
+  puede fijar por hash. Un paso comprueba la versión y falla antes de publicar si no la cumple. El build sigue en
+  Node 22, la versión mínima que soporta el servidor.
+
 ## [1.0.1] - 2026-09-22
 
 Versión de mantenimiento: seguimiento de la auditoría de seguridad, procedimiento de commit y pruebas por propiedades.

package/README.es.md

@@ -4,7 +4,7 @@
 
 [English](README.md) · **Español**
 
-[![CI](../../actions/workflows/ci.yml/badge.svg)](../../actions/workflows/ci.yml) ![Node 22+](https://img.shields.io/badge/node-22%2B-2f7d32) ![MCP](https://img.shields.io/badge/MCP-stdio-5a4fcf) ![SAP](https://img.shields.io/badge/SAP-ECC%20%7C%20S%2F4HANA-0a6ed1) ![License](https://img.shields.io/badge/license-Apache--2.0-blue)
+[![CI](../../actions/workflows/ci.yml/badge.svg)](../../actions/workflows/ci.yml) [![OpenSSF Best Practices](https://www.bestpractices.dev/projects/14759/badge)](https://www.bestpractices.dev/projects/14759) ![Node 22+](https://img.shields.io/badge/node-22%2B-2f7d32) ![MCP](https://img.shields.io/badge/MCP-stdio-5a4fcf) ![SAP](https://img.shields.io/badge/SAP-ECC%20%7C%20S%2F4HANA-0a6ed1) ![License](https://img.shields.io/badge/license-Apache--2.0-blue)
 
 **IA que trabaja en tu SAP con las reglas de un consultor senior.**
 
@@ -56,12 +56,12 @@ El agente elige las tools, las encadena y responde con evidencia. Cada respuesta
 | Grupo | Para qué | Tools |
 |---|---|---|
 | [Revisión de código y pases](#g-revision) | Saber qué cambia de verdad una orden y qué puede romper, antes de liberarla. | 5 |
-| [Calidad, ATC y remediación](#g-calidad) | Encontrar, entender y corregir hallazgos con la sintaxis y las correcciones reales de SAP. | 5 |
-| [Exploración del repositorio](#g-exploracion) | Leer y entender cualquier objeto ABAP y sus relaciones, en ECC y en S/4HANA. | 9 |
+| [Calidad, ATC y remediación](#g-calidad) | Encontrar, entender y corregir hallazgos con la sintaxis y las correcciones reales de SAP. | 6 |
+| [Exploración del repositorio](#g-exploracion) | Leer y entender cualquier objeto ABAP y sus relaciones, en ECC y en S/4HANA. | 11 |
 | [Consulta de datos](#g-datos) | Preguntar a las tablas con ABAP SQL, de solo lectura y sin tocar material de credenciales. | 2 |
 | [Diagnóstico de incidentes](#g-diagnostico) | Reunir en una conversación lo que antes exigía ST22, SM37, SLG1 y /IWFND/ERROR_LOG. | 4 |
 | [Documentación SAP](#g-documentacion) | Responder con la documentación oficial y comprobar qué sintaxis existe en cada release. | 7 |
-| [Escritura controlada](#g-escritura) | Guardar cambios solo en desarrollo, en la orden correcta y con la sintaxis verificada antes. | 4 |
+| [Escritura controlada](#g-escritura) | Guardar cambios solo en desarrollo, en la orden correcta y con la sintaxis verificada antes. | 5 |
 | [DoZimple Transport Risk](#g-transport-risk) | Decidir si un pase entero puede ir a calidad o productivo, con el porqué en lenguaje de negocio. | 7 |
 | [Operación y crecimiento](#g-operacion) | Ver qué funciona en cada sistema y decidir con datos cuál es la siguiente tool. | 4 |
 <!-- groups:end -->
@@ -122,6 +122,7 @@ la **[referencia completa](docs/TOOLS.md)**.
 | [`run_atc`](docs/TOOLS.md#calidad) | **Ejecutar ATC.** Ejecuta el ATC sobre un objeto o una orden de transporte y lista los hallazgos numerados (prioridad, línea, check, mensaje), con los totales P1/P2/P3 que da SAP. | lectura |
 | [`atc_quickfix`](docs/TOOLS.md#calidad) | **Correcciones propuestas por SAP.** Correcciones que SAP ofrece (las mismas de Ctrl+1 en Eclipse) para un hallazgo ATC o una línea: crear símbolo de texto, extraer constante, etc. | lectura |
 | [`api_release_state`](docs/TOOLS.md#calidad) | **¿Está liberada esta API? ¿Cuál es su sucesor?** Estado de liberación de un objeto SAP (clase, FM/BAPI, tabla, CDS…) por contrato C0–C4 y su sucesor liberado, leído del propio sistema. | lectura |
+| [`sap_notes`](docs/TOOLS.md#calidad) | **Notas SAP en el sistema (SNOTE).** Estado de notas SAP en este sistema, como en SNOTE: si está descargada, estado de implementación (completa, incompleta, versión anterior, se puede implementar, obsoleta, no se puede implementar), estado de tratamiento, versión, componente y título. | lectura |
 | [`syntax_check`](docs/TOOLS.md#calidad) | **Chequeo de sintaxis SAP.** Chequeo de sintaxis real de SAP (no abaplint). | lectura |
 | [`run_unit_tests`](docs/TOOLS.md#calidad) | **Ejecutar ABAP Unit.** Ejecuta los tests ABAP Unit de una clase o programa y devuelve el resultado por método, con el detalle de cada fallo. | ejecuta (DEV) |
 
@@ -135,12 +136,14 @@ la **[referencia completa](docs/TOOLS.md)**.
 | [`search_objects`](docs/TOOLS.md#exploracion) | **Buscar objetos ABAP.** Busca objetos del repositorio por nombre (admite * como comodín). | lectura |
 | [`get_source`](docs/TOOLS.md#exploracion) | **Leer fuente ABAP.** Lee la fuente de cualquier objeto: programa, include, clase (y sus includes), interfaz, módulo de función (sin necesidad de saber el grupo), CDS, tabla/estructura, etc. | lectura |
 | [`where_used`](docs/TOOLS.md#exploracion) | **Dónde se usa.** Lista de uso (where-used) de un objeto: quién lo referencia, con paquete y responsable. | lectura |
+| [`source_search`](docs/TOOLS.md#exploracion) | **Buscar texto en el código.** Busca un texto (o una expresión regular) en el código fuente de un paquete (con subpaquetes), de una orden de transporte o de una lista de objetos, y devuelve objeto, include, línea y la línea encontrada. | lectura |
 | [`object_versions`](docs/TOOLS.md#exploracion) | **Versiones de un objeto.** Historial de versiones de un objeto (fecha, autor, orden). | lectura |
 | [`package_contents`](docs/TOOLS.md#exploracion) | **Contenido de un paquete.** Objetos de un paquete de desarrollo agrupados por tipo, con sus subpaquetes (TADIR/TDEVC, cualquier release). | lectura |
 | [`ddic_type_info`](docs/TOOLS.md#exploracion) | **Elemento de datos, dominio o tipo tabla.** Definición de un tipo DDIC: elemento de datos (dominio, tipo, longitud, textos), dominio (tipo, longitud, valores fijos, tabla de valores) o tipo tabla (tipo de línea, clave). | lectura |
 | [`transaction_info`](docs/TOOLS.md#exploracion) | **Qué ejecuta una transacción.** Programa, dynpro y parámetros de una transacción (TSTC/TSTCP), con su texto. | lectura |
 | [`function_modules`](docs/TOOLS.md#exploracion) | **Módulos de función de un grupo.** Lista los módulos de función de un grupo de funciones, con su texto y si son RFC o de actualización. | lectura |
 | [`text_elements`](docs/TOOLS.md#exploracion) | **Símbolos de texto y textos de selección.** Lee los símbolos de texto (TEXT-001…), textos de selección o encabezados de un programa, clase o grupo de funciones. | lectura |
+| [`enhancements`](docs/TOOLS.md#exploracion) | **Ampliaciones y BAdI.** Qué ampliaciones hay y dónde. badi=NOMBRE: implementaciones de esa BAdI, nuevas y clásicas, activas o no. program + program_type (PROG, CLAS, FUGR): ampliaciones de código (puntos y secciones implícitos o explícitos) implementadas dentro de ese programa, clase o grupo — lo que cambia el comportamiento de un estándar sin tocar su código. prefix (p. ej. Z): implementaciones de ampliación de ese espacio de nombres, por tipo. | lectura |
 
 <a id="g-datos"></a>
 ### Consulta de datos
@@ -187,6 +190,7 @@ la **[referencia completa](docs/TOOLS.md)**.
 | Tool | Qué hace | Acceso |
 |---|---|---|
 | [`write_source`](docs/TOOLS.md#escritura) | **Guardar fuente en SAP.** Sustituye la fuente COMPLETA de un objeto existente (o de un include de clase), en la orden indicada. | escribe (DEV autorizado) |
+| [`revert_source`](docs/TOOLS.md#escritura) | **Volver a una versión anterior.** Deshace un cambio escribiendo de nuevo una versión anterior del objeto: la última activa (para limpiar un borrador inactivo tras un write_source cuya activación falló), la anterior a la activa, o una concreta del historial de object_versions. | escribe (DEV autorizado) |
 | [`activate`](docs/TOOLS.md#escritura) | **Activar objeto.** Activa un objeto y devuelve los mensajes de SAP tal cual (errores con línea, avisos, objetos que quedan inactivos). | escribe (DEV autorizado) |
 | [`write_text_elements`](docs/TOOLS.md#escritura) | **Crear o cambiar símbolos de texto.** Añade o modifica símbolos de texto (o textos de selección) de un programa/clase/grupo, fusionando con los existentes: no borra los que no se mencionan. | escribe (DEV autorizado) |
 | [`create_transport`](docs/TOOLS.md#escritura) | **Crear orden de transporte.** Crea una orden workbench para el paquete de un objeto, ANTES de la primera edición, para que el cambio caiga en la orden del ticket y no en una tarea reutilizada. | escribe (DEV autorizado) |
@@ -291,7 +295,7 @@ Modelo de amenazas con STRIDE y OWASP Top 10 para aplicaciones LLM: **[docs/THRE
 Con la versión exacta y sin scripts de instalación, igual que las dependencias del propio proyecto:
 
 ```sh
-npm install -g --ignore-scripts @dozimple/abap-adt@1.0.1
+npm install -g --ignore-scripts @dozimple/abap-adt@1.2.0
 PKG="$(npm root -g)/@dozimple/abap-adt"
 mkdir -p ~/.config/abap-adt-dozimple && chmod 700 ~/.config/abap-adt-dozimple
 cp "$PKG/config/systems.example.json" ~/.config/abap-adt-dozimple/systems.json   # sistemas, roles y permisos
@@ -307,9 +311,9 @@ Registro en el cliente MCP:
 ```
 
 Cada versión se publica desde el CI con [procedencia de npm](https://docs.npmjs.com/generating-provenance-statements):
-`npm view @dozimple/abap-adt@1.0.1 dist.attestations` muestra la atestación, y la release de GitHub incluye el
+`npm view @dozimple/abap-adt@1.2.0 dist.attestations` muestra la atestación, y la release de GitHub incluye el
 paquete, su bundle de Sigstore (`.sigstore.json`), el mismo bundle como procedencia in-toto (`.intoto.jsonl`)
-y el SBOM. Para comprobarlo: `gh attestation verify dozimple-abap-adt-1.0.1.tgz --repo <owner>/abap-adt-dozimple`.
+y el SBOM. Para comprobarlo: `gh attestation verify dozimple-abap-adt-1.2.0.tgz --repo <owner>/abap-adt-dozimple`.
 
 ### Desde el código fuente
 
@@ -343,7 +347,7 @@ abap-adt-doZimple se construye sobre el trabajo de otros, y lo reconoce: cada to
 <!-- credits:start -->
 | Proyecto | Autor / titular | Licencia | Tipo | Usado en |
 |---|---|---|---|---|
-| [abap-adt-api](https://github.com/marcellourbani/abap-adt-api) | Marcello Urbani | MIT | dependencia | todas (núcleo), `transport_diff`, `transport_contents`, `inactive_objects`, `edit_preflight`, `run_atc` y 22 más |
+| [abap-adt-api](https://github.com/marcellourbani/abap-adt-api) | Marcello Urbani | MIT | dependencia | todas (núcleo), `transport_diff`, `transport_contents`, `inactive_objects`, `edit_preflight`, `run_atc` y 26 más |
 | [Model Context Protocol TypeScript SDK](https://github.com/modelcontextprotocol/typescript-sdk) | Model Context Protocol | MIT | dependencia | todas (núcleo) |
 | [mcp-sap-docs](https://github.com/marianfoo/mcp-sap-docs) | Marian Zeis (marianfoo) | Apache-2.0 | dependencia | `abap_feature_matrix`, `docs_search`, `docs_fetch`, `clean_core_objects`, `clean_core_object`, `abap_lint` y 1 más |
 | [abaplint](https://github.com/abaplint/abaplint) | Lars Hvam y contribuidores | MIT | dependencia | `abap_lint` |
@@ -358,7 +362,7 @@ abap-adt-doZimple se construye sobre el trabajo de otros, y lo reconoce: cada to
 | [ABAP Remote FS (vscode_abap_remote_fs)](https://github.com/marcellourbani/vscode_abap_remote_fs) | Marcello Urbani | MIT | idea | `syntax_check` |
 | [mcp-abap-adt](https://github.com/mario-andreschak/mcp-abap-adt) | mario-andreschak | MIT | idea | `search_objects`, `get_source`, `package_contents`, `ddic_type_info`, `transaction_info`, `table_contents` |
 | [ARC-1](https://github.com/arc-mcp/arc-1) | arc-mcp (Marian Zeis y contribuidores) | MIT | idea | `transport_diff`, `atc_quickfix`, `gateway_errors` |
-| [vibing-steampunk](https://github.com/oisee/vibing-steampunk) | oisee y contribuidores | MIT | idea | `co_change`, `api_release_state`, `jobs`, `application_log` |
+| [vibing-steampunk](https://github.com/oisee/vibing-steampunk) | oisee y contribuidores | MIT | idea | `co_change`, `api_release_state`, `source_search`, `enhancements`, `jobs`, `application_log` |
 | [ABAP Accelerator for Amazon Q Developer](https://github.com/aws-solutions-library-samples/guidance-for-deploying-sap-abap-accelerator-for-amazon-q-developer) | AWS Solutions Library Samples | MIT-0 | idea | todas (núcleo), `usage_stats` |
 | [An O(ND) Difference Algorithm and Its Variations (1986)](https://doi.org/10.1007/BF01840446) | Eugene W. Myers | algoritmo publicado | algoritmo | `transport_diff`, `atc_quickfix` |
 <!-- credits:end -->

package/README.md

@@ -4,7 +4,7 @@
 
 **English** · [Español](README.es.md)
 
-[![CI](../../actions/workflows/ci.yml/badge.svg)](../../actions/workflows/ci.yml) ![Node 22+](https://img.shields.io/badge/node-22%2B-2f7d32) ![MCP](https://img.shields.io/badge/MCP-stdio-5a4fcf) ![SAP](https://img.shields.io/badge/SAP-ECC%20%7C%20S%2F4HANA-0a6ed1) ![License](https://img.shields.io/badge/license-Apache--2.0-blue)
+[![CI](../../actions/workflows/ci.yml/badge.svg)](../../actions/workflows/ci.yml) [![OpenSSF Best Practices](https://www.bestpractices.dev/projects/14759/badge)](https://www.bestpractices.dev/projects/14759) ![Node 22+](https://img.shields.io/badge/node-22%2B-2f7d32) ![MCP](https://img.shields.io/badge/MCP-stdio-5a4fcf) ![SAP](https://img.shields.io/badge/SAP-ECC%20%7C%20S%2F4HANA-0a6ed1) ![License](https://img.shields.io/badge/license-Apache--2.0-blue)
 
 **AI that works on your SAP system with the rules of a senior consultant.**
 
@@ -57,12 +57,12 @@ and **a failure is never presented as an empty result or as success**.
 | Group | What for | Tools |
 |---|---|---|
 | [Code review and transports](#g-revision) | Know what a transport really changes and what it may break, before releasing it. | 5 |
-| [Quality, ATC and remediation](#g-calidad) | Find, understand and fix findings with SAP's real syntax check and quick fixes. | 5 |
-| [Repository exploration](#g-exploracion) | Read and understand any ABAP object and its relations, on ECC and S/4HANA. | 9 |
+| [Quality, ATC and remediation](#g-calidad) | Find, understand and fix findings with SAP's real syntax check and quick fixes. | 6 |
+| [Repository exploration](#g-exploracion) | Read and understand any ABAP object and its relations, on ECC and S/4HANA. | 11 |
 | [Data queries](#g-datos) | Query tables with ABAP SQL, read-only, with sensitive and personal data protected. | 2 |
 | [Incident diagnosis](#g-diagnostico) | One conversation for what used to take ST22, SM37, SLG1 and /IWFND/ERROR_LOG. | 4 |
 | [SAP documentation](#g-documentacion) | Answer from official documentation and check which syntax exists in each release. | 7 |
-| [Controlled writes](#g-escritura) | Save changes only in development, in the right transport, previewed and confirmed by a human. | 4 |
+| [Controlled writes](#g-escritura) | Save changes only in development, in the right transport, previewed and confirmed by a human. | 5 |
 | [DoZimple Transport Risk](#g-transport-risk) | Decide whether a whole release can go to QA or production, with the why in business terms. | 7 |
 | [Operations and growth](#g-operacion) | See what works on each system and decide the next tool with data. | 4 |
 <!-- groups:end -->
@@ -123,6 +123,7 @@ Summary per group; each tool's details — parameters, types, defaults, requirem
 | [`run_atc`](docs/TOOLS.md#calidad) | **Run ATC.** Runs the ABAP Test Cockpit on an object or a transport and lists numbered findings (priority, line, check, message) with SAP's P1/P2/P3 totals. | read |
 | [`atc_quickfix`](docs/TOOLS.md#calidad) | **SAP-proposed fixes.** The fixes SAP offers (the same as Ctrl+1 in Eclipse) for an ATC finding or a line: create text symbol, extract constant, etc. | read |
 | [`api_release_state`](docs/TOOLS.md#calidad) | **Is this API released? What is its successor?** Release state of an SAP object (class, function module/BAPI, table, CDS…) by contract C0–C4 and its released successor, read from the system itself. | read |
+| [`sap_notes`](docs/TOOLS.md#calidad) | **SAP notes in the system (SNOTE).** Whether a note is downloaded in this system, its implementation and processing status, version, component and title, per note or filtered by status/component; no SAP portal or S-user needed. | read |
 | [`syntax_check`](docs/TOOLS.md#calidad) | **SAP syntax check.** SAP's real syntax check (not abaplint), also on code not saved yet. | read |
 | [`run_unit_tests`](docs/TOOLS.md#calidad) | **Run ABAP Unit.** Runs the ABAP Unit tests of a class or program (harmless and short only) and returns the result per method, with each failure in detail. | executes (DEV) |
 
@@ -136,12 +137,14 @@ Summary per group; each tool's details — parameters, types, defaults, requirem
 | [`search_objects`](docs/TOOLS.md#exploracion) | **Search ABAP objects.** Searches repository objects by name (supports * wildcards). | read |
 | [`get_source`](docs/TOOLS.md#exploracion) | **Read ABAP source.** Reads the source of any object: program, include, class (and its includes), interface, function module (without knowing its group), CDS, table/structure, etc. | read |
 | [`where_used`](docs/TOOLS.md#exploracion) | **Where used.** Where-used list of an object: who references it, with package and owner. | read |
+| [`source_search`](docs/TOOLS.md#exploracion) | **Search text in source code.** Searches a text or regex in the source of a package (with subpackages), a transport or a list of objects, and returns object, include, line and the matching line; reaches what where-used misses (dynamic calls, literals, Z code outside the reference index). | read |
 | [`object_versions`](docs/TOOLS.md#exploracion) | **Object versions.** Version history of an object (date, author, transport). | read |
 | [`package_contents`](docs/TOOLS.md#exploracion) | **Package contents.** Objects of a development package grouped by type, with subpackages (TADIR/TDEVC, any release). | read |
 | [`ddic_type_info`](docs/TOOLS.md#exploracion) | **Data element, domain or table type.** Definition of a DDIC type: data element (domain, type, length, texts), domain (type, length, fixed values, value table) or table type (line type, key). | read |
 | [`transaction_info`](docs/TOOLS.md#exploracion) | **What a transaction runs.** Program, screen and parameters of a transaction (TSTC/TSTCP), with its text. | read |
 | [`function_modules`](docs/TOOLS.md#exploracion) | **Function modules of a group.** Lists the function modules of a function group with their text and whether they are RFC or update modules; given a module, finds its group and siblings. Works with /XXX/ namespaces. | read |
 | [`text_elements`](docs/TOOLS.md#exploracion) | **Text symbols and selection texts.** Reads the text symbols (TEXT-001…), selection texts or headings of a program, class or function group. | read |
+| [`enhancements`](docs/TOOLS.md#exploracion) | **Enhancements and BAdIs.** Implementations of a BAdI (new and classic, active or not), source code enhancements implemented inside a program, class or function group (flagging those that overwrite standard code), or all enhancement implementations of a namespace by type. | read |
 
 <a id="g-datos"></a>
 ### Data queries
@@ -188,6 +191,7 @@ Summary per group; each tool's details — parameters, types, defaults, requirem
 | Tool | What it does | Access |
 |---|---|---|
 | [`write_source`](docs/TOOLS.md#escritura) | **Save source to SAP.** Replaces the FULL source of an existing object (or class include) in the given transport, after a preview with syntax check and diff and a human confirmation. | writes (authorized DEV) |
+| [`revert_source`](docs/TOOLS.md#escritura) | **Revert to an earlier version.** Writes back an earlier version of an object (the last active one, the one before it, or a numbered one from object_versions) through the same preview, fingerprint, lock and transport as write_source; never reverts on its own. | writes (authorized DEV) |
 | [`activate`](docs/TOOLS.md#escritura) | **Activate object.** Activates an object and returns SAP's messages as they are (errors with line, warnings, objects left inactive). | writes (authorized DEV) |
 | [`write_text_elements`](docs/TOOLS.md#escritura) | **Create or change text symbols.** Adds or changes text symbols (or selection texts) of a program/class/group, merging with the existing ones: nothing not mentioned is deleted. | writes (authorized DEV) |
 | [`create_transport`](docs/TOOLS.md#escritura) | **Create transport request.** Creates a workbench request for an object's package BEFORE the first edit, so the change lands in the ticket's transport and not in a reused task. | writes (authorized DEV) |
@@ -291,7 +295,7 @@ Designed to pass a Security and Basis review without exceptions. Details: **[SEC
 Pinned to an exact version, without install scripts, like the project's own dependencies:
 
 ```sh
-npm install -g --ignore-scripts @dozimple/abap-adt@1.0.1
+npm install -g --ignore-scripts @dozimple/abap-adt@1.2.0
 PKG="$(npm root -g)/@dozimple/abap-adt"
 mkdir -p ~/.config/abap-adt-dozimple && chmod 700 ~/.config/abap-adt-dozimple
 cp "$PKG/config/systems.example.json" ~/.config/abap-adt-dozimple/systems.json   # systems, roles and permissions
@@ -307,9 +311,9 @@ MCP client registration:
 ```
 
 Every release is published from CI with [npm provenance](https://docs.npmjs.com/generating-provenance-statements):
-`npm view @dozimple/abap-adt@1.0.1 dist.attestations` shows the attestation, and the GitHub release carries the
+`npm view @dozimple/abap-adt@1.2.0 dist.attestations` shows the attestation, and the GitHub release carries the
 tarball, its Sigstore bundle (`.sigstore.json`), the same bundle as in-toto provenance (`.intoto.jsonl`) and the
-SBOM. To check it: `gh attestation verify dozimple-abap-adt-1.0.1.tgz --repo <owner>/abap-adt-dozimple`.
+SBOM. To check it: `gh attestation verify dozimple-abap-adt-1.2.0.tgz --repo <owner>/abap-adt-dozimple`.
 
 ### From source
 
@@ -344,7 +348,7 @@ abap-adt-doZimple is built on other people's work, and says so: each tool lists
 <!-- credits:start -->
 | Project | Author / holder | License | Type | Used in |
 |---|---|---|---|---|
-| [abap-adt-api](https://github.com/marcellourbani/abap-adt-api) | Marcello Urbani | MIT | dependency | all (core), `transport_diff`, `transport_contents`, `inactive_objects`, `edit_preflight`, `run_atc` and 22 more |
+| [abap-adt-api](https://github.com/marcellourbani/abap-adt-api) | Marcello Urbani | MIT | dependency | all (core), `transport_diff`, `transport_contents`, `inactive_objects`, `edit_preflight`, `run_atc` and 26 more |
 | [Model Context Protocol TypeScript SDK](https://github.com/modelcontextprotocol/typescript-sdk) | Model Context Protocol | MIT | dependency | all (core) |
 | [mcp-sap-docs](https://github.com/marianfoo/mcp-sap-docs) | Marian Zeis (marianfoo) | Apache-2.0 | dependency | `abap_feature_matrix`, `docs_search`, `docs_fetch`, `clean_core_objects`, `clean_core_object`, `abap_lint` and 1 more |
 | [abaplint](https://github.com/abaplint/abaplint) | Lars Hvam and contributors | MIT | dependency | `abap_lint` |
@@ -359,7 +363,7 @@ abap-adt-doZimple is built on other people's work, and says so: each tool lists
 | [ABAP Remote FS (vscode_abap_remote_fs)](https://github.com/marcellourbani/vscode_abap_remote_fs) | Marcello Urbani | MIT | idea | `syntax_check` |
 | [mcp-abap-adt](https://github.com/mario-andreschak/mcp-abap-adt) | mario-andreschak | MIT | idea | `search_objects`, `get_source`, `package_contents`, `ddic_type_info`, `transaction_info`, `table_contents` |
 | [ARC-1](https://github.com/arc-mcp/arc-1) | arc-mcp (Marian Zeis and contributors) | MIT | idea | `transport_diff`, `atc_quickfix`, `gateway_errors` |
-| [vibing-steampunk](https://github.com/oisee/vibing-steampunk) | oisee and contributors | MIT | idea | `co_change`, `api_release_state`, `jobs`, `application_log` |
+| [vibing-steampunk](https://github.com/oisee/vibing-steampunk) | oisee and contributors | MIT | idea | `co_change`, `api_release_state`, `source_search`, `enhancements`, `jobs`, `application_log` |
 | [ABAP Accelerator for Amazon Q Developer](https://github.com/aws-solutions-library-samples/guidance-for-deploying-sap-abap-accelerator-for-amazon-q-developer) | AWS Solutions Library Samples | MIT-0 | idea | all (core), `usage_stats` |
 | [An O(ND) Difference Algorithm and Its Variations (1986)](https://doi.org/10.1007/BF01840446) | Eugene W. Myers | published algorithm | algorithm | `transport_diff`, `atc_quickfix` |
 <!-- credits:end -->

package/dist/core/catalog.en.js

@@ -23,11 +23,14 @@ export const TOOLS_EN = {
     run_atc: "**Run ATC.** Runs the ABAP Test Cockpit on an object or a transport and lists numbered findings (priority, line, check, message) with SAP's P1/P2/P3 totals.",
     atc_quickfix: "**SAP-proposed fixes.** The fixes SAP offers (the same as Ctrl+1 in Eclipse) for an ATC finding or a line: create text symbol, extract constant, etc.",
     api_release_state: "**Is this API released? What is its successor?** Release state of an SAP object (class, function module/BAPI, table, CDS…) by contract C0–C4 and its released successor, read from the system itself.",
+    sap_notes: "**SAP notes in the system (SNOTE).** Whether a note is downloaded in this system, its implementation and processing status, version, component and title, per note or filtered by status/component; no SAP portal or S-user needed.",
     syntax_check: "**SAP syntax check.** SAP's real syntax check (not abaplint), also on code not saved yet.",
     run_unit_tests: "**Run ABAP Unit.** Runs the ABAP Unit tests of a class or program (harmless and short only) and returns the result per method, with each failure in detail.",
     search_objects: "**Search ABAP objects.** Searches repository objects by name (supports * wildcards).",
     get_source: "**Read ABAP source.** Reads the source of any object: program, include, class (and its includes), interface, function module (without knowing its group), CDS, table/structure, etc.",
     where_used: "**Where used.** Where-used list of an object: who references it, with package and owner.",
+    source_search: "**Search text in source code.** Searches a text or regex in the source of a package (with subpackages), a transport or a list of objects, and returns object, include, line and the matching line; reaches what where-used misses (dynamic calls, literals, Z code outside the reference index).",
+    enhancements: "**Enhancements and BAdIs.** Implementations of a BAdI (new and classic, active or not), source code enhancements implemented inside a program, class or function group (flagging those that overwrite standard code), or all enhancement implementations of a namespace by type.",
     object_versions: "**Object versions.** Version history of an object (date, author, transport).",
     package_contents: "**Package contents.** Objects of a development package grouped by type, with subpackages (TADIR/TDEVC, any release).",
     ddic_type_info: "**Data element, domain or table type.** Definition of a DDIC type: data element (domain, type, length, texts), domain (type, length, fixed values, value table) or table type (line type, key).",
@@ -48,6 +51,7 @@ export const TOOLS_EN = {
     abap_lint: "**abaplint on a snippet.** Runs abaplint locally (code never leaves the machine) on an ABAP snippet or source.",
     docs_community_search: "**Search SAP Community.** Searches SAP Community (blogs and questions) by error message, class or concept.",
     write_source: "**Save source to SAP.** Replaces the FULL source of an existing object (or class include) in the given transport, after a preview with syntax check and diff and a human confirmation.",
+    revert_source: "**Revert to an earlier version.** Writes back an earlier version of an object (the last active one, the one before it, or a numbered one from object_versions) through the same preview, fingerprint, lock and transport as write_source; never reverts on its own.",
     activate: "**Activate object.** Activates an object and returns SAP's messages as they are (errors with line, warnings, objects left inactive).",
     write_text_elements: "**Create or change text symbols.** Adds or changes text symbols (or selection texts) of a program/class/group, merging with the existing ones: nothing not mentioned is deleted.",
     create_transport: "**Create transport request.** Creates a workbench request for an object's package BEFORE the first edit, so the change lands in the ticket's transport and not in a reused task.",

package/dist/core/catalog.js

@@ -40,6 +40,7 @@ export const GROUPS = [
             { name: "run_atc", credits: ADT },
             { name: "atc_quickfix", credits: ["abapAdtApi", "arc1", "myers"] },
             { name: "api_release_state", credits: ["abapAdtApi", "vsp"] },
+            { name: "sap_notes", credits: ADT },
             { name: "syntax_check", credits: ["abapAdtApi", "abapFs"] },
             { name: "run_unit_tests", credits: ADT },
         ],
@@ -52,12 +53,14 @@ export const GROUPS = [
             { name: "search_objects", credits: ["abapAdtApi", "marioAdt"] },
             { name: "get_source", credits: ["abapAdtApi", "marioAdt"] },
             { name: "where_used", credits: ADT },
+            { name: "source_search", credits: ["abapAdtApi", "vsp"] },
             { name: "object_versions", credits: ADT },
             { name: "package_contents", credits: ["abapAdtApi", "marioAdt"] },
             { name: "ddic_type_info", credits: ["abapAdtApi", "marioAdt"] },
             { name: "transaction_info", credits: ["abapAdtApi", "marioAdt"] },
             { name: "function_modules", credits: ADT },
             { name: "text_elements", credits: ADT },
+            { name: "enhancements", credits: ["abapAdtApi", "vsp"] },
         ],
     },
     {
@@ -100,6 +103,7 @@ export const GROUPS = [
         pitch: "Guardar cambios solo en desarrollo, en la orden correcta y con la sintaxis verificada antes.",
         tools: [
             { name: "write_source", credits: ADT },
+            { name: "revert_source", credits: ADT },
             { name: "activate", credits: ADT },
             { name: "write_text_elements", credits: ADT },
             { name: "create_transport", credits: ADT },

package/dist/core/concurrency.js

@@ -0,0 +1,13 @@
+/** Limita cuántas promesas corren a la vez (no saturar SAP). Conserva el orden de los resultados. */
+export async function pool(items, n, fn) {
+    const out = new Array(items.length);
+    let next = 0;
+    await Promise.all(Array.from({ length: Math.min(n, items.length) }, async () => {
+        while (next < items.length) {
+            const i = next++;
+            out[i] = await fn(items[i]);
+        }
+    }));
+    return out;
+}
+//# sourceMappingURL=concurrency.js.map

package/dist/core/connection.js

@@ -8,6 +8,10 @@ import { normalizeValue, wrapSql } from "./sql.js";
 const CACHE_DIR = join(homedir(), ".cache", "abap-adt-dozimple");
 const DISCOVERY_TTL_MS = 7 * 24 * 3600 * 1000;
 const REQUEST_TIMEOUT_MS = 120_000;
+/** Circuit breaker: N fallos de red en la ventana abren el circuito durante CIRCUIT_OPEN_MS. */
+const CIRCUIT_FAILURES = 3;
+const CIRCUIT_WINDOW_MS = 60_000;
+const CIRCUIT_OPEN_MS = 60_000;
 /**
  * TLS: con caFile se verifica contra ese certificado (lo correcto para un
  * sistema con certificado propio); allowSelfSigned desactiva la verificación
@@ -30,9 +34,44 @@ export class SapConnection {
     reader;
     caps;
     basisRelease;
+    networkFailures = [];
+    openUntil = 0;
     constructor(system) {
         this.system = system;
     }
+    /** Olvida el cliente de lectura (sesión caducada): la siguiente llamada vuelve a entrar. */
+    resetReader() {
+        const old = this.reader;
+        this.reader = undefined;
+        old?.then((c) => c.logout()).catch(() => undefined);
+    }
+    /**
+     * Tres fallos de red en un minuto abren el circuito: durante un minuto ninguna tool intenta nada contra este
+     * sistema y responde al instante. Un sistema caído (VPN, host) dejaba de responder 120 s por cada llamada.
+     */
+    noteNetworkFailure(now = Date.now()) {
+        this.networkFailures = this.networkFailures.filter((t) => now - t < CIRCUIT_WINDOW_MS);
+        this.networkFailures.push(now);
+        if (this.networkFailures.length >= CIRCUIT_FAILURES) {
+            this.openUntil = now + CIRCUIT_OPEN_MS;
+            this.networkFailures = [];
+        }
+    }
+    /** Segundos que faltan para volver a intentar, o 0 si el circuito está cerrado. */
+    circuitOpenFor(now = Date.now()) {
+        return this.openUntil > now ? Math.ceil((this.openUntil - now) / 1000) : 0;
+    }
+    /** Cierra el circuito a mano (sap_systems con check): se vuelve a intentar ya. */
+    resetCircuit() {
+        this.openUntil = 0;
+        this.networkFailures = [];
+    }
+    assertCircuitClosed() {
+        const s = this.circuitOpenFor();
+        if (s) {
+            throw new ToolError("NETWORK", `${this.system.id} no respondió ${CIRCUIT_FAILURES} veces seguidas: no se vuelve a intentar durante ${s} s.`, "sap_systems(check=true) lo reintenta ahora mismo.");
+        }
+    }
     async newClient() {
         const s = this.system;
         const password = await getPassword(s);
@@ -53,6 +92,7 @@ export class SapConnection {
     }
     /** Cliente de lectura, reutilizado entre llamadas. */
     async adt() {
+        this.assertCircuitClosed();
         if (!this.reader) {
             this.reader = this.newClient().then((c) => this.login(c));
             this.reader.catch(() => (this.reader = undefined));
@@ -61,6 +101,7 @@ export class SapConnection {
     }
     /** Ejecuta fn con una sesión stateful propia y la cierra siempre. */
     async stateful(fn) {
+        this.assertCircuitClosed();
         const c = await this.login(await this.newClient());
         c.stateful = session_types.stateful;
         try {

package/dist/core/errors.js

@@ -19,6 +19,9 @@ export function normalizeError(e, systemId) {
     if (e instanceof ToolError)
         return e;
     const any = e;
+    // signal.throwIfAborted() entre pasos de una tool larga: cancelación del cliente, no fallo del servidor.
+    if (any?.name === "AbortError")
+        return new ToolError("CANCELLED", "Cancelado por el cliente antes de terminar.");
     const code = errCode(any);
     if (code && NETWORK_CODES.has(code)) {
         return new ToolError("NETWORK", `No se llega a ${systemId ?? "SAP"} (${code}).`, "¿Está levantada la VPN de ese cliente?");
@@ -50,6 +53,16 @@ export function normalizeError(e, systemId) {
     const message = sanitizeMessage(any?.message ? String(any.message) : String(e));
     return new ToolError("INTERNAL", message || "Error sin mensaje");
 }
+/**
+ * Sesión caducada a mitad de una lectura: token CSRF rechazado, o 401 en una sesión que ya había entrado. El registro
+ * la renueva y repite la lectura UNA vez; nunca una escritura (el bloqueo se perdió con la sesión).
+ */
+export function isSessionExpired(e) {
+    if (isCsrfError(e))
+        return true;
+    const st = e;
+    return (isHttpError(e) && st?.status === 401) || (isAdtError(e) && st?.err === 401);
+}
 export function renderError(te) {
     const labels = {
         NETWORK: "Sin conexión",
@@ -60,6 +73,7 @@ export function renderError(te) {
         CAPABILITY: "No disponible en este sistema",
         MODULE: "Módulo no habilitado",
         INPUT: "Parámetros inválidos",
+        CANCELLED: "Cancelado",
         INTERNAL: "Error interno",
     };
     return `${labels[te.kind]}: ${te.message}${te.hint ? `\n${te.hint}` : ""}\n` +

package/dist/core/packages.js

@@ -0,0 +1,26 @@
+import { sqlLiteral } from "./objects.js";
+/** Paquete y, si se pide, sus subpaquetes (hasta 5 niveles). `subs` son los hijos encontrados (directos si no es recursivo). */
+export async function packageTree(sap, root, recursive) {
+    const packages = [root];
+    const subs = [];
+    let frontier = [root];
+    for (let level = 0; level < (recursive ? 5 : 1) && frontier.length; level++) {
+        const r = await sap.query(`SELECT devclass FROM tdevc WHERE parentcl IN ( ${frontier.map(sqlLiteral).join(", ")} )`, 2000);
+        frontier = r.values.map((v) => v.DEVCLASS);
+        subs.push(...frontier);
+        if (recursive)
+            packages.push(...frontier);
+    }
+    return { packages, subs };
+}
+/**
+ * Objetos R3TR de esos paquetes (sin DEVC), ordenados por tipo y nombre. Sin los marcados como borrados
+ * (DELFLAG = 'X'): TADIR los conserva, pero ya no existen en el repositorio.
+ */
+export async function packageObjects(sap, packages, max, objectType) {
+    const typeFilter = objectType ? ` AND object = ${sqlLiteral(objectType.toUpperCase())}` : "";
+    const r = await sap.query(`SELECT devclass, pgmid, object, obj_name FROM tadir WHERE pgmid = 'R3TR' AND devclass IN ( ${packages.map(sqlLiteral).join(", ")} )` +
+        `${typeFilter} AND object <> 'DEVC' AND delflag = ' ' ORDER BY object, obj_name`, max);
+    return r.values;
+}
+//# sourceMappingURL=packages.js.map

package/dist/core/registry.js

@@ -3,11 +3,11 @@ import { join } from "node:path";
 import { pathToFileURL } from "node:url";
 import { z } from "zod";
 import { canWrite, resolveSystem } from "./config.js";
-import { normalizeError, renderError, ToolError } from "./errors.js";
+import { isSessionExpired, normalizeError, renderError, ToolError } from "./errors.js";
 import { budget } from "./output.js";
 import { appendAudit, auditArgs } from "./audit.js";
 import { consumeTokenWithState, issueToken } from "./confirm.js";
-import { renderNotes, withNotes } from "./notes.js";
+import { addNote, renderNotes, withNotes } from "./notes.js";
 import { assertAccess } from "./policy.js";
 import { recordUsage } from "./telemetry.js";
 /**
@@ -182,6 +182,15 @@ export async function invoke(def, rawArgs, env) {
                     throw new ToolError("INTERNAL", "No hay componentes auxiliares en este servidor.");
                 return env.sidecars.get(name);
             },
+            signal: env.signal,
+            progress(message, current, total) {
+                try {
+                    env.progress?.(message, current, total);
+                }
+                catch {
+                    /* el progreso es cortesía: un fallo al notificar nunca rompe la tool */
+                }
+            },
         };
         const head = resolved ? `Sistema: ${resolved.system.id} (${resolved.source})\n${SAP_DATA_NOTE}\n\n` : "";
         if (resolved && (def.access === "write" || def.access === "exec")) {
@@ -218,14 +227,23 @@ export async function invoke(def, rawArgs, env) {
                     throw new ToolError("INTERNAL", `No se ejecutó: no se pudo escribir el registro de auditoría (${e.message}).`);
                 }
             }
-            const { result: res, notes } = await withNotes(() => def.run(args, ctx));
+            const { result: res, notes } = await withNotes(() => runGuarded(def, args, ctx, sapConn, systemId));
             const text = renderNotes(notes) + (typeof res === "string" ? res : res.text);
             const isError = typeof res === "string" ? false : !!res.isError;
-            outcome = { text: head + budget(text), isError, kind: isError ? "RESULT" : undefined, system: systemId };
+            const structured = typeof res === "string" ? undefined : res.structured;
+            // Una tool con esquema de salida que responde bien sin datos estructurados es un bug: mejor verlo aquí que
+            // como error críptico del SDK en el cliente.
+            if (def.output && !isError && !structured)
+                throw new ToolError("INTERNAL", `${def.name} declara salida estructurada y no la devolvió.`);
+            outcome = { text: head + budget(text), isError, kind: isError ? "RESULT" : undefined, system: systemId, structured };
         }
     }
     catch (e) {
         let te = normalizeError(e, systemId);
+        // Solo los fallos de red REALES (de la librería) cuentan para el circuito: ni un timeout nuestro ni el propio
+        // aviso de circuito abierto.
+        if (te.kind === "NETWORK" && !(e instanceof ToolError) && sapConn)
+            sapConn.noteNetworkFailure();
         // Un 404 de SAP (no un «objeto no existe» nuestro) sobre un endpoint que el
         // discovery tampoco lista: ahora sí hay evidencia de que falta la función.
         if (te.kind === "NOT_FOUND" && !(e instanceof ToolError) && missing.length && sapConn) {
@@ -251,6 +269,50 @@ export async function invoke(def, rawArgs, env) {
     });
     return outcome;
 }
+const DEFAULT_TIMEOUT_MS = 60_000;
+function withTimeout(p, ms, what, signal) {
+    let timer;
+    let onAbort;
+    const limit = new Promise((_, reject) => {
+        if (signal) {
+            onAbort = () => reject(new ToolError("CANCELLED", `${what}: cancelado por el cliente.`));
+            if (signal.aborted)
+                onAbort();
+            else
+                signal.addEventListener("abort", onAbort, { once: true });
+        }
+        timer = setTimeout(() => reject(new ToolError("NETWORK", `${what}: tiempo agotado (${ms >= 1000 ? `${Math.round(ms / 1000)} s` : `${ms} ms`}).`, "Acota la petición (menos objetos, más filtro) o repite más tarde.")), ms);
+    });
+    return Promise.race([p, limit]).finally(() => {
+        clearTimeout(timer);
+        if (signal && onAbort)
+            signal.removeEventListener("abort", onAbort);
+    });
+}
+/**
+ * Ejecuta la tool con su tiempo máximo y, si la sesión de lectura había caducado (CSRF o 401 tras haber entrado),
+ * la renueva y repite UNA vez. Nunca repite escrituras ni ejecuciones: el bloqueo se perdió con la sesión y un
+ * reintento podría escribir dos veces.
+ */
+async function runGuarded(def, args, ctx, sapConn, systemId) {
+    const ms = def.timeoutMs ?? DEFAULT_TIMEOUT_MS;
+    const attempt = () => {
+        if (ctx.signal?.aborted)
+            throw new ToolError("CANCELLED", `${def.name}: cancelado por el cliente.`);
+        return withTimeout(def.run(args, ctx), ms, def.name, ctx.signal);
+    };
+    try {
+        return await attempt();
+    }
+    catch (e) {
+        if (def.access === "read" && sapConn && isSessionExpired(e)) {
+            sapConn.resetReader();
+            addNote(`La sesión con ${systemId ?? "SAP"} había caducado: se renovó y la lectura se repitió.`);
+            return await attempt();
+        }
+        throw e;
+    }
+}
 /** Elicitación de formulario, si el cliente la anuncia (ABAP_DZ_CONFIRM=token la desactiva). */
 function elicitFor(server) {
     const caps = server.server.getClientCapabilities()?.elicitation;
@@ -313,10 +375,21 @@ export function registerAll(server, defs, config, pool, sidecars) {
             title: def.title,
             description: describe(def, config),
             inputSchema,
+            ...(def.output ? { outputSchema: z.object(def.output) } : {}),
             annotations: annotationsFor(def),
-        }, (async (args) => {
-            const r = await invoke(def, args ?? {}, { config, pool, tools: defs, sidecars, elicit: elicitFor(server) });
-            return { content: [{ type: "text", text: r.text }], isError: r.isError };
+        }, (async (args, extra) => {
+            const token = extra?._meta?.progressToken;
+            const progress = token !== undefined && extra?.sendNotification
+                ? (message, current, total) => {
+                    void extra.sendNotification({ method: "notifications/progress", params: { progressToken: token, progress: current ?? 0, total, message } }).catch(() => undefined);
+                }
+                : undefined;
+            const r = await invoke(def, args ?? {}, { config, pool, tools: defs, sidecars, elicit: elicitFor(server), signal: extra?.signal, progress });
+            return {
+                content: [{ type: "text", text: r.text }],
+                isError: r.isError,
+                ...(r.structured ? { structuredContent: r.structured } : {}),
+            };
         }));
         published.push(def.name);
     }