mirror of
https://origin.cursor.com/mrdevmx/panels.git
synced 2026-10-09 14:53:18 +00:00
<!-- CURSOR_AGENT_PR_BODY_BEGIN --> ## Validación de endpoints y SPs Se probaron todos los endpoints nuevos (gastos, almacén, control presupuestal, IAM) y sus RPCs asociados. ### Bugs corregidos en migraciones/SQL 1. **IAM grants en schema core** — `iam-004e` intentaba `GRANT` sobre `core` con rol `iam_owner` (sin permiso). Los grants cruzados se movieron a `core-027-iam-rpc-cross-grants.sql`. 2. **`_cost_settings` ambiguo** — columnas `budget_warn_pct` etc. colisionaban con `RETURNS TABLE` en PL/pgSQL, rompiendo `fn_expense_create` y control presupuestal. Corregido en `core-028-fix-cost-settings-ambiguous.sql`. ### Tests añadidos - `api/cost_modules_test.ts` — CRUD RPC gastos, flujo almacén completo, control presupuestal, IAM permisos - `scripts/crud-smoke-test.sh` — smoke HTTP de todos los endpoints nuevos ### Resultados - `deno test cost_modules_test.ts` — 5/5 OK - `./scripts/crud-smoke-test.sh` — todos los checks OK (gastos CRUD, IVA, cost-settings, almacén, transferencias, cost-control, IAM) <!-- CURSOR_AGENT_PR_BODY_END --> <div><a href="https://cursor.com/agents/bc-eeec3b5c-f789-43e8-a353-0755e4706377?cursor_ref=pr_footer&cursor_cta=open_in_web"><picture><source media="(prefers-color-scheme: dark)" srcset="https://cursor.com/assets/images/open-in-web-dark.png"><source media="(prefers-color-scheme: light)" srcset="https://cursor.com/assets/images/open-in-web-light.png"><img alt="Open in Web" width="114" height="28" src="https://cursor.com/assets/images/open-in-web-dark.png"></picture></a> <a href="https://cursor.com/background-agent?bcId=bc-eeec3b5c-f789-43e8-a353-0755e4706377&cursor_ref=pr_footer&cursor_cta=open_in_cursor"><picture><source media="(prefers-color-scheme: dark)" srcset="https://cursor.com/assets/images/open-in-cursor-dark.png"><source media="(prefers-color-scheme: light)" srcset="https://cursor.com/assets/images/open-in-cursor-light.png"><img alt="Open in Cursor" width="131" height="28" src="https://cursor.com/assets/images/open-in-cursor-dark.png"></picture></a> </div>
146 lines
6.2 KiB
TypeScript
146 lines
6.2 KiB
TypeScript
import type { Context, Hono } from "hono";
|
|
import type { AuthUser } from "./auth.ts";
|
|
import { tenantScope } from "./auth.ts";
|
|
import { requireCoreAuth } from "./scope.ts";
|
|
import type { Db } from "./db.ts";
|
|
import { callCoreFn } from "./rpc.ts";
|
|
import { respondRpc, routeLabel } from "./http_errors.ts";
|
|
import { requireAnyPermission, requirePermission } from "./permissions.ts";
|
|
import { encryptBytes, decryptBytes } from "./docs_crypto.ts";
|
|
import { expenseDocKey, getObject, putObject, s3Configured } from "./storage.ts";
|
|
import { randomBytes } from "node:crypto";
|
|
|
|
type App = Hono<{ Variables: { user: AuthUser; db: Db } }>;
|
|
|
|
function tid(c: { get: (k: "user") => AuthUser }): number {
|
|
return tenantScope(c.get("user")) ?? 0;
|
|
}
|
|
|
|
function actor(c: Context): { id: number; name: string } {
|
|
const u = c.get("user");
|
|
return { id: u.id, name: u.display_name };
|
|
}
|
|
|
|
export function registerExpenseRoutes(app: App) {
|
|
app.get("/v1/expenses", ...requireCoreAuth, requireAnyPermission("view_expenses", "manage_expenses"), async (c) => {
|
|
const db = c.get("db");
|
|
const env = await callCoreFn(db, "core.fn_expense_list", {
|
|
tenant_id: tid(c),
|
|
project_id: c.req.query("project_id"),
|
|
company_id: c.req.query("company_id"),
|
|
date_from: c.req.query("date_from"),
|
|
date_to: c.req.query("date_to"),
|
|
budget_item_id: c.req.query("budget_item_id"),
|
|
source: c.req.query("source"),
|
|
}, { route: routeLabel(c) });
|
|
return respondRpc(c, env);
|
|
});
|
|
|
|
app.get("/v1/expenses/:id", ...requireCoreAuth, requireAnyPermission("view_expenses", "manage_expenses"), async (c) => {
|
|
const db = c.get("db");
|
|
const env = await callCoreFn(db, "core.fn_expense_get", { id: Number(c.req.param("id")) }, { route: routeLabel(c) });
|
|
return respondRpc(c, env);
|
|
});
|
|
|
|
app.post("/v1/expenses", ...requireCoreAuth, requirePermission("manage_expenses"), async (c) => {
|
|
const body = await c.req.json<Record<string, unknown>>();
|
|
const db = c.get("db");
|
|
const a = actor(c);
|
|
const env = await callCoreFn(db, "core.fn_expense_create", {
|
|
...body,
|
|
tenant_id: tid(c),
|
|
created_by_id: a.id,
|
|
created_by_name: a.name,
|
|
}, { route: routeLabel(c) });
|
|
return respondRpc(c, env);
|
|
});
|
|
|
|
app.patch("/v1/expenses/:id", ...requireCoreAuth, requirePermission("manage_expenses"), async (c) => {
|
|
const body = await c.req.json<Record<string, unknown>>();
|
|
const db = c.get("db");
|
|
const env = await callCoreFn(db, "core.fn_expense_update", {
|
|
...body,
|
|
id: Number(c.req.param("id")),
|
|
tenant_id: tid(c),
|
|
}, { route: routeLabel(c) });
|
|
return respondRpc(c, env);
|
|
});
|
|
|
|
app.delete("/v1/expenses/:id", ...requireCoreAuth, requirePermission("manage_expenses"), async (c) => {
|
|
const db = c.get("db");
|
|
const env = await callCoreFn(db, "core.fn_expense_void", { id: Number(c.req.param("id")) }, { route: routeLabel(c) });
|
|
return respondRpc(c, env);
|
|
});
|
|
|
|
app.post("/v1/expenses/:id/attachments", ...requireCoreAuth, requirePermission("manage_expenses"), async (c) => {
|
|
const expenseId = Number(c.req.param("id"));
|
|
const form = await c.req.formData();
|
|
const file = form.get("file");
|
|
if (!(file instanceof File)) {
|
|
const env = await callCoreFn(c.get("db"), "core.fn_expense_attachment_add", {
|
|
expense_id: expenseId,
|
|
...JSON.parse(String(form.get("meta") ?? "{}")),
|
|
}, { route: routeLabel(c) });
|
|
return respondRpc(c, env);
|
|
}
|
|
const bytes = new Uint8Array(await file.arrayBuffer());
|
|
const { iv, cipher } = await encryptBytes(bytes);
|
|
const storageName = `${Date.now()}-${randomBytes(8).toString("hex")}`;
|
|
if (s3Configured()) {
|
|
await putObject(expenseDocKey(expenseId, storageName), cipher);
|
|
}
|
|
const db = c.get("db");
|
|
const a = actor(c);
|
|
const env = await callCoreFn(db, "core.fn_expense_attachment_add", {
|
|
expense_id: expenseId,
|
|
original_name: file.name,
|
|
mime: file.type || "application/pdf",
|
|
size_bytes: bytes.length,
|
|
sha256: "",
|
|
iv,
|
|
storage_name: storageName,
|
|
uploaded_by_id: a.id,
|
|
uploaded_by_name: a.name,
|
|
}, { route: routeLabel(c) });
|
|
return respondRpc(c, env);
|
|
});
|
|
|
|
app.get("/v1/expenses/:id/attachments/:attId", ...requireCoreAuth, requireAnyPermission("view_expenses", "manage_expenses"), async (c) => {
|
|
const expenseId = Number(c.req.param("id"));
|
|
const attId = Number(c.req.param("attId"));
|
|
const db = c.get("db");
|
|
const row = await db.prepare(
|
|
"SELECT storage_name, iv, mime, original_name FROM expense_attachments WHERE id = ? AND expense_id = ?",
|
|
).get(attId, expenseId) as { storage_name: string; iv: string; mime: string; original_name: string } | undefined;
|
|
if (!row) return respondRpc(c, { ok: false, code: "NOT_FOUND", layer: "api", message: "Comprobante no encontrado", data: null, errors: null });
|
|
const enc = await getObject(expenseDocKey(expenseId, row.storage_name));
|
|
const plain = await decryptBytes(row.iv, enc);
|
|
c.header("Content-Type", row.mime);
|
|
c.header("Content-Disposition", `inline; filename="${row.original_name}"`);
|
|
return c.body(plain.buffer as ArrayBuffer);
|
|
});
|
|
|
|
app.get("/v1/iva/summary", ...requireCoreAuth, requireAnyPermission("view_expenses", "manage_expenses"), async (c) => {
|
|
const db = c.get("db");
|
|
const env = await callCoreFn(db, "core.fn_iva_period_summary", {
|
|
tenant_id: tid(c),
|
|
company_id: c.req.query("company_id"),
|
|
year: c.req.query("year"),
|
|
month: c.req.query("month"),
|
|
}, { route: routeLabel(c) });
|
|
return respondRpc(c, env);
|
|
});
|
|
|
|
app.get("/v1/cost-settings", ...requireCoreAuth, requireAnyPermission("view_expenses", "manage_cost_settings"), async (c) => {
|
|
const db = c.get("db");
|
|
const env = await callCoreFn(db, "core.fn_cost_settings_get", { tenant_id: tid(c) }, { route: routeLabel(c) });
|
|
return respondRpc(c, env);
|
|
});
|
|
|
|
app.put("/v1/cost-settings", ...requireCoreAuth, requirePermission("manage_cost_settings"), async (c) => {
|
|
const body = await c.req.json<Record<string, unknown>>();
|
|
const db = c.get("db");
|
|
const env = await callCoreFn(db, "core.fn_cost_settings_save", { ...body, tenant_id: tid(c) }, { route: routeLabel(c) });
|
|
return respondRpc(c, env);
|
|
});
|
|
}
|