@dozimple/abap-adt 1.0.0 → 1.0.1

package/dist/tools/docs/docs.js

@@ -59,6 +59,14 @@ export function assertDocId(id, cfg) {
         catch {
             throw new ToolError("INPUT", `«${id}» no es un id de documento válido.`);
         }
+        // La forma se comprueba sobre el id YA decodificado: un «%2F» no puede colar una URL de otro host.
+        const shapes = [
+            /^sap-help-[A-Za-z0-9_-]+(~[A-Za-z0-9._-]+)?$/,
+            /^community-\d+$/,
+            /^community-url-https:\/\/community\.sap\.com\/[A-Za-z0-9._~\/?=&#%+-]*$/,
+        ];
+        if (!shapes.some((re) => re.test(decoded)))
+            throw new ToolError("INPUT", `«${id}» no tiene la forma de un documento de SAP Help o SAP Community.`);
         assertPublicQuery(decoded.replace(/[-/:.?=&#]/g, " "), cfg, sc.blockTerms);
         return;
     }

package/dist/tools/transport-risk/_service.js

@@ -3,7 +3,7 @@ import axios from "axios";
 import { z } from "zod";
 import { tlsOptions } from "../../core/connection.js";
 import { getPassword } from "../../core/credentials.js";
-import { ToolError } from "../../core/errors.js";
+import { sanitizeMessage, ToolError } from "../../core/errors.js";
 /**
  * DoZimple Transport Risk: analizador de riesgo de transportes instalado en el
  * sistema de desarrollo del cliente (nodo ICF /sap/bc/zdz_risk, clase
@@ -31,6 +31,11 @@ export async function callRiskService(system, params) {
         auth: { username: system.user, password: await getPassword(system) },
         httpsAgent: tlsOptions(system).httpsAgent ?? new https.Agent(),
         timeout: 180_000,
+        // Un servicio ICF de solo lectura no redirige: sin redirecciones, las credenciales Basic nunca viajan a otra URL
+        // (sin depender de que una librería transitiva las quite al cambiar de host).
+        maxRedirects: 0,
+        // Tope de tamaño de la respuesta antes de procesarla (un sistema averiado no agota la memoria del proceso).
+        maxContentLength: 50 * 1024 * 1024,
         // Los 4xx/5xx traen un JSON con la causa: se lee en vez de lanzar.
         validateStatus: () => true,
         responseType: "text",
@@ -56,7 +61,7 @@ export async function callRiskService(system, params) {
         return { text: `El servicio ${RISK_SERVICE_PATH} no existe en este sistema.\n${BACKEND_HINT}`, isError: true };
     if (response.status !== 200) {
         const detail = json && typeof json === "object" && "error" in json ? String(json.error) : body;
-        return { text: `El servicio de riesgo respondió HTTP ${response.status}: ${detail}`, isError: true };
+        return { text: `El servicio de riesgo respondió HTTP ${response.status}: ${sanitizeMessage(detail, 300)}`, isError: true };
     }
     return body;
 }

package/docs/THREAT_MODEL.md

@@ -30,8 +30,9 @@ flowchart LR
 |---|---|---|---|
 | **S**uplantación | Otro proceso lee la contraseña del llavero | Llavero del SO; `--strict` (opt-in) exige confirmación en cada lectura | Sin `--strict`, un proceso del mismo usuario que invoque `security` puede leerla. Solución de raíz prevista: certificado de cliente / SSO (roadmap) |
 | **S**uplantación | TLS interceptado | Verificación TLS; `caFile` | `allowSelfSigned` sigue existiendo como último recurso, marcado |
-| **T**ampering | Cambiar `systems.json` para habilitar escritura o redirigir a otro host | Rechazo si es modificable por otros o de otro usuario | Un proceso del propio usuario puede modificarlo |
-| **T**ampering | Retocar el registro de auditoría | Cadena de hash verificable | Un atacante con la cuenta del usuario puede reescribir la cadena entera; para evidencia fuerte, enviar el registro a un almacén externo (ver roadmap) |
+| **T**ampering | Cambiar `systems.json` para habilitar escritura, redirigir a otro host o cambiar el comando de un componente (ejecución de código) | Rechazo si es modificable por otros o de otro usuario: permisos en macOS/Linux, ACL por SID en Windows; si no se puede comprobar, no se arranca | Un proceso del propio usuario puede modificarlo |
+| **T**ampering | Retocar el registro de auditoría | Cadena de hash + firma HMAC con clave en el llavero del SO (fecha de creación incluida: quitar firmas se detecta); bloqueo entre procesos; `audit:verify` imprime el último hash para anclarlo fuera | Quien pueda leer el llavero del usuario (su propia sesión, sin `--strict`) podría recalcularlo; el anclaje externo del último hash cubre ese caso. En Windows, sin almacén, solo cadena |
+| **T**ampering | Una escritura aprobada se aplica sobre una versión distinta de la mostrada | La huella de lo mostrado viaja con la confirmación y se compara bajo el bloqueo del objeto; si cambió, no se escribe | — |
 | **R**epudio | «La IA lo cambió sola» o «no hay constancia de ese cambio» | Confirmación humana con vista previa + auditoría con forma de confirmación y huella del contenido. **Fail-closed**: la intención se registra antes de tocar SAP, y si no se puede registrar (disco lleno, permiso denegado, ruta inválida) la escritura no se ejecuta. Si lo que falla es solo el registro del resultado, la respuesta lo avisa explícitamente. Cubierto por tests que simulan ENOSPC y EACCES | El registro es local; un corte de energía entre intención y resultado deja una intención sin resultado, que se interpreta como «estado desconocido, verificar en SAP» |
 | **I**nformation disclosure | Leer hashes, almacén seguro, PSE, OAuth, dumps | Veto directo y a través de vistas y CDS | Tablas Z que copien esos datos no se detectan por nombre |
 | **I**nformation disclosure | Volcar datos personales al proveedor LLM | Clase de datos, enmascarado, tope de filas | Columnas personales con nombres no estándar (Z) no se enmascaran; el tope limita el volumen |
@@ -44,7 +45,7 @@ flowchart LR
 
 | Riesgo | Cómo aplica a un MCP de SAP | Control |
 |---|---|---|
-| LLM01 Prompt injection | Comentarios en fuente ABAP, textos, datos de tablas, documentación o posts de comunidad con instrucciones | Marca de «dato, nunca instrucciones» en toda respuesta; ninguna escritura sin confirmación humana; tools de escritura destructivas para el cliente |
+| LLM01 Prompt injection | Comentarios en fuente ABAP, textos, datos de tablas, documentación o posts de comunidad con instrucciones | Marca de «dato, nunca instrucciones» en toda respuesta (probabilística: viaja en el mismo canal que los datos y el contenido podría imitarla; el protocolo MCP aún no separa instrucciones de datos); marcado neutralizado tras decodificar HTML; **la barrera efectiva**: ninguna escritura sin confirmación humana, tools de escritura destructivas para el cliente, y no existen tools para liberar, importar ni borrar |
 | LLM02 Divulgación de información sensible | Datos personales y material de seguridad en respuestas que llegan al proveedor | Vetos, enmascarado, tope de filas, filtro de salida a internet |
 | LLM03 Cadena de suministro | Dependencias npm, MCP de terceros | Versiones exactas, firmas, SBOM, `ignore-scripts`, terceros aislados tras revisión |
 | LLM05 Manejo inadecuado de la salida | El agente usa una respuesta como si fuera verdad | Errores tipados que nunca parecen vacíos; «tope alcanzado» explícito; calidad de la evidencia en diffs |

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@dozimple/abap-adt",
-  "version": "1.0.0",
+  "version": "1.0.1",
   "description": "DoZimple MCP server for SAP ABAP: multi-system, secure by design — code review, ATC, diagnosis and documentation for ECC and S/4HANA",
   "keywords": [
     "sap",
@@ -43,11 +43,12 @@
     "test": "vitest run",
     "smoke": "node dist/scripts/smoke.js",
     "inspector": "npx @modelcontextprotocol/inspector node dist/index.js",
-    "security": "npm audit --omit=dev && npm audit signatures && node scripts/scan-sensitive.mjs",
+    "security": "npm audit --omit=dev && npm audit signatures && node scripts/scan-sensitive.mjs && node scripts/scan-sensitive.mjs --history",
     "check": "npm test && npm run build && npm run security",
     "docs": "npm run build && node dist/scripts/gen-docs.js",
     "audit:verify": "node dist/scripts/audit-verify.js",
-    "sbom": "npm sbom --omit=dev --sbom-format=cyclonedx --sbom-type=application > sbom.cdx.json"
+    "sbom": "npm sbom --omit=dev --sbom-format=cyclonedx --sbom-type=application > sbom.cdx.json",
+    "hooks": "git config core.hooksPath .githooks && echo \"hooks activos: pre-commit (código completo) y pre-push (historial)\""
   },
   "engines": {
     "node": ">=22"
@@ -60,6 +61,7 @@
   },
   "devDependencies": {
     "@types/node": "22.20.2",
+    "fast-check": "4.10.2",
     "typescript": "5.9.3",
     "vitest": "5.0.0"
   },