@opensas/xml-rest-adapter 0.1.0 → 0.2.1

package/README.md

@@ -2,3 +2,9 @@
 
 Proyecto _de uso interno_ que expone mediante una API REST las bases de datos que utilizan el
 formato propio de `xmlEntidad_Accion` para encapsular la lógica de negocio.
+
+npm: https://www.npmjs.com/package/@opensas/xml-rest-adapter
+
+```bash
+pnpm add @opensas/xml-rest-adapter
+```

package/dist/create-adapter.d.ts

@@ -1,8 +1,7 @@
-import type { RequestEvent } from '@sveltejs/kit';
 import type { AppError } from './errors.js';
 import type { Result } from './result.js';
 import type { Aplicacion, Usuario } from './types.js';
 export declare function createXmlRestAdapter(app: Aplicacion): {
     resolveUser: (identity: string | Request) => Promise<Result<Usuario, AppError>>;
-    handle: (event: RequestEvent, usuario: Usuario) => Promise<Response>;
+    handle: (request: Request, path: string, usuario: Usuario) => Promise<Response>;
 };

package/dist/create-adapter.js

@@ -1,7 +1,19 @@
-import { json } from '@sveltejs/kit';
+// createXmlRestAdapter(app) — la función pública principal de la librería. Valida la Aplicacion
+// configurada y devuelve un adapter con dos métodos: `resolveUser(domainUser)` (identidad +
+// permisos, ver auth.ts) y `handle(request, path, usuario)`, que atiende cualquier request
+// GET/POST/PUT/DELETE contra esa app — juntos reemplazan `authUser` (hooks.server.ts) y los 4
+// handlers que hoy viven repartidos en dos `+server.ts` distintos en convenios_web
+// (routes/api/[aplicacion]/[objeto]/+server.ts y .../[objeto]/[id]/+server.ts). `usuario` se pasa
+// separado, no vía `event.locals`: no le impone al consumidor que cablee nada en su propio
+// `hooks.server.ts` (ver docs/proximos-pasos.md). `request`/`path` sueltos, no un objeto tipo
+// `RequestEvent` — `handle` nunca necesitó más que estos dos datos, y un objeto `{params,request}`
+// es la forma que le da SvelteKit a su evento, no un estándar: `Request` (Fetch API) sí lo es,
+// `path` es un `string` plano porque nunca hay más de un segmento que leer (ver
+// docs/proximos-pasos.md, "agnóstico de framework").
 import { resolveUser } from './auth.js';
 import { execXmlConsulta, execXmlSentencia } from './db.js';
 import { appError, jsonError } from './errors.js';
+import { json } from './json.js';
 import { parseRequest } from './parse-request.js';
 function validarAplicacion(app) {
     if (!app.name)
@@ -55,12 +67,9 @@ export function createXmlRestAdapter(app) {
     function resolveUserFn(identity) {
         return resolveUser(app, identity);
     }
-    async function handle(event, usuario) {
+    async function handle(request, path, usuario) {
         try {
-            // Genérico a propósito: cada consumidor define su propia carpeta `[...path]`, así que
-            // el nombre real del param lo tipa `$types` del lado del consumidor, no acá.
-            const path = event.params.path ?? '';
-            const parsed = await parseRequest(app, event.request, path, usuario);
+            const parsed = await parseRequest(app, request, path, usuario);
             if (!parsed.ok)
                 return jsonError(parsed.error);
             const { objeto, id, sentencia, archivos, archivoParams } = parsed.data;

package/dist/errors.js

@@ -1,8 +1,8 @@
 // Error de negocio estructurado y homogéneo para toda la librería. Se arma con `appError(...)`
 // (título + código + detalle opcional) y se devuelve al cliente con `jsonError(...)`, que lo
 // envuelve en un `Response` JSON con el status HTTP que corresponde al código.
-import { json } from '@sveltejs/kit';
 import { BAD_REQUEST, FORBIDDEN, INTERNAL_ERROR, NOT_FOUND, UNAUTHORIZED, UNPROCESSABLE_ENTITY, } from './http-status.js';
+import { json } from './json.js';
 export const APP_ERROR_CODES = [
     'VALIDATION_ERROR',
     'BAD_REQUEST',

package/dist/index.d.ts

@@ -4,5 +4,5 @@ export { getDomainUser } from './auth.js';
 export { appError, jsonError, type AppError, type AppErrorCode } from './errors.js';
 export { closeAllPools } from './db.js';
 export type { Result } from './result.js';
-export type { Accion, Aplicacion, Authentication, Authorization, Endpoint, Permisos, Usuario, } from './types.js';
+export type { Accion, Aplicacion, Authentication, Authorization, Endpoint, HandleEvent, Permisos, Usuario, } from './types.js';
 export type { DbInfo } from './db.js';

package/dist/json.d.ts

@@ -0,0 +1 @@
+export declare function json(data: unknown, init?: ResponseInit): Response;

package/dist/json.js

@@ -0,0 +1,19 @@
+// Reimplementación mínima de `json()` de `@sveltejs/kit` — evita que la librería dependa de
+// `@sveltejs/kit` en runtime (ver docs/proximos-pasos.md, "peerDependencies real vs. de tipos"):
+// antes, `import { json } from '@sveltejs/kit'` en errors.ts/create-adapter.ts obligaba a instalar
+// el paquete completo (y, por peerDependency, arrastraba `svelte`) aunque el consumidor no use
+// SvelteKit para nada — el propio `RequestEvent` que sí necesita la librería es solo un tipo,
+// borrado en tiempo de compilación. Comportamiento idéntico al original (mismo orden de headers,
+// mismo default de `content-type`/`content-length` solo si el caller no los puso ya).
+const textEncoder = new TextEncoder();
+export function json(data, init) {
+    const body = JSON.stringify(data);
+    const headers = new Headers(init?.headers);
+    if (!headers.has('content-length')) {
+        headers.set('content-length', textEncoder.encode(body).byteLength.toString());
+    }
+    if (!headers.has('content-type')) {
+        headers.set('content-type', 'application/json');
+    }
+    return new Response(body, { ...init, headers });
+}

package/dist/types.d.ts

@@ -1,4 +1,8 @@
 import type { DbInfo } from './db.js';
+export type HandleEvent = {
+    params: Record<string, string | undefined>;
+    request: Request;
+};
 export type Accion = 'consulta' | 'alta' | 'baja' | 'modificacion';
 export type Permisos = Record<string, Accion[]>;
 export type Usuario = {

package/dist/valid.js

@@ -23,7 +23,25 @@ function toPascalCase(str) {
     })
         .join('');
 }
-const getEndpoint = (app, objeto) => app.endpoints.find((ep) => ep.name === toPascalCase(objeto));
+// Match exacto primero. Si falla y el segmento tiene un separador `_` explícito, reintenta
+// case-insensitive: un `Endpoint.name` con un acrónimo en mayúsculas (`ActividadCIIU`,
+// `CategoriaCIIU`) nunca coincide con lo que arma `toPascalCase` a partir del segmento de URL
+// (`actividad_ciiu` → `ActividadCiiu`, con "Ciiu" no "CIIU") — ningún algoritmo genérico de
+// snake_case→PascalCase puede adivinar qué partes de un nombre compuesto son sigla.
+// convenios_web nunca tuvo este problema porque su comparación (`isValidObjeto`) ya era
+// case-insensitive — portado acá, pero acotado al caso con separador: sin `_` en el segmento
+// mantenemos el exacto a propósito (`tipodato`, sin límite de palabra explícito, sigue sin
+// matchear `TipoDato` — ver custom-url-casing.fixture.test.ts, cualquier coincidencia ahí sería
+// casualidad, no una reconstrucción real). Encontrado migrando `CctDigitalizacion` en
+// convenios_web (ver docs/todos/rest-baseline-pre-migracion.md).
+const getEndpoint = (app, objeto) => {
+    const pascal = toPascalCase(objeto);
+    const exact = app.endpoints.find((ep) => ep.name === pascal);
+    if (exact || !objeto.includes('_'))
+        return exact;
+    const normalized = pascal.toLowerCase();
+    return app.endpoints.find((ep) => ep.name.toLowerCase() === normalized);
+};
 export function validParams(app, path, withId = 'noId') {
     const [objetoRaw, idRaw] = path.split('/');
     if (!objetoRaw)

package/package.json

@@ -1,6 +1,6 @@
 {
 	"name": "@opensas/xml-rest-adapter",
-	"version": "0.1.0",
+	"version": "0.2.1",
 	"publishConfig": {
 		"access": "public"
 	},
@@ -33,7 +33,7 @@
 		"fixture:purge": "docker compose -f docker/fixture/docker-compose.yaml down -v",
 		"fixture:start": "docker compose -f docker/fixture/docker-compose.yaml up -d",
 		"fixture:stop": "docker compose -f docker/fixture/docker-compose.yaml stop",
-		"fixture:status": "docker ps --filter name=fixture-db --format 'table {{.Names}}\\t{{.Status}}'",
+		"fixture:status": "docker ps --filter name=xml-rest-adapter-fixture-db --format 'table {{.Names}}\\t{{.Status}}'",
 		"gitlab-runner:start": "sudo gitlab-runner start",
 		"gitlab-runner:stop": "sudo gitlab-runner stop",
 		"gitlab-runner:status": "sudo gitlab-runner status",
@@ -47,20 +47,14 @@
 	"sideEffects": [
 		"**/*.css"
 	],
-	"svelte": "./dist/index.js",
 	"types": "./dist/index.d.ts",
 	"type": "module",
 	"exports": {
 		".": {
 			"types": "./dist/index.d.ts",
-			"svelte": "./dist/index.js",
 			"default": "./dist/index.js"
 		}
 	},
-	"peerDependencies": {
-		"@sveltejs/kit": "^2.0.0",
-		"svelte": "^5.0.0"
-	},
 	"devDependencies": {
 		"@eslint/js": "^10.0.1",
 		"@playwright/test": "^1.62.1",