@oondemand/oon-core-back 0.3.53 → 0.3.54

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@oondemand/oon-core-back",
-  "version": "0.3.53",
+  "version": "0.3.54",
   "repository": {
     "type": "git",
     "url": "git+https://github.com/oondemand/oon-platform.git",

package/src/boot/database.js

@@ -1,4 +1,5 @@
 const mongoose = require("mongoose");
+const registry = require("../core/registry");
 
 /**
  * Conexão com o MongoDB a partir das envs padrão da Central.
@@ -51,9 +52,42 @@ function databaseLabel(uri) {
   }
 }
 
+function shouldSyncDeclaredIndexes() {
+  return String(process.env.OON_SYNC_INDEXES ?? "true").toLowerCase() !== "false";
+}
+
+/**
+ * Mantém os índices físicos do MongoDB alinhados às models registradas.
+ *
+ * `createIndexes()` apenas cria índices ausentes e deixa índices removidos do
+ * domínio ativos indefinidamente. `syncIndexes()` também elimina esses índices
+ * obsoletos, evitando restrições antigas como um `codigo_1` único continuar
+ * bloqueando novos registros depois que o campo saiu do manifesto.
+ */
+async function syncDeclaredIndexes(entries = registry.listModels()) {
+  if (!shouldSyncDeclaredIndexes()) return [];
+
+  const results = [];
+  for (const entry of entries) {
+    const model = entry?.mongooseModel;
+    if (!model || typeof model.syncIndexes !== "function") continue;
+
+    const dropped = await model.syncIndexes();
+    const normalized = Array.isArray(dropped) ? dropped : [];
+    results.push({ model: entry.name, dropped: normalized });
+
+    if (normalized.length) {
+      console.log(`[OonCore] Índices obsoletos removidos de ${entry.name}: ${normalized.join(", ")}`);
+    }
+  }
+
+  return results;
+}
+
 async function connectDB() {
   const uri = buildMongoUri();
   await mongoose.connect(uri, connectionOptions());
+  await syncDeclaredIndexes();
   console.log(`Conectado ao MongoDB (${databaseLabel(uri)})`);
   return mongoose.connection;
 }
@@ -63,4 +97,6 @@ module.exports = {
   buildMongoUri,
   connectionOptions,
   databaseLabel,
+  shouldSyncDeclaredIndexes,
+  syncDeclaredIndexes,
 };

package/src/core/middlewares/error.js

@@ -1,27 +1,94 @@
 const helpers = require("../utils/helpers");
 const GenericError = require("../errors/GenericError");
 
+function firstDuplicateField(error) {
+  const source = error?.keyPattern || error?.keyValue;
+  const direct = source && typeof source === "object" ? Object.keys(source)[0] : undefined;
+  if (direct) return direct;
+
+  const index = String(error?.message || "").match(/index:\s+([^\s]+)\s+dup key/i)?.[1];
+  return index?.replace(/_\d+$/, "");
+}
+
+function duplicateIndexName(error) {
+  return String(error?.message || "").match(/index:\s+([^\s]+)\s+dup key/i)?.[1];
+}
+
+function fieldLabel(field) {
+  if (!field) return "o campo informado";
+  return field
+    .replace(/Id$/, "")
+    .replace(/([a-z0-9])([A-Z])/g, "$1 $2")
+    .replace(/[_-]+/g, " ")
+    .replace(/^./, (letter) => letter.toUpperCase());
+}
+
+function duplicateKeyDetails(error, requestId) {
+  const field = firstDuplicateField(error);
+  const index = duplicateIndexName(error);
+  const value = field && error?.keyValue && typeof error.keyValue === "object"
+    ? error.keyValue[field]
+    : undefined;
+  const staleNullIndex = field && value == null;
+  const message = staleNullIndex
+    ? `A estrutura do banco ainda possui uma restrição antiga para ${fieldLabel(field)}. Os índices precisam ser sincronizados antes de tentar novamente.`
+    : field
+      ? `Já existe um registro com o mesmo valor em ${fieldLabel(field)}.`
+      : "Já existe um registro com os mesmos dados únicos.";
+
+  return {
+    code: staleNullIndex ? "STALE_UNIQUE_INDEX" : "DUPLICATE_KEY",
+    message,
+    ...(field ? { field } : {}),
+    ...(index ? { index } : {}),
+    ...(requestId ? { requestId } : {}),
+  };
+}
+
 /**
- * Error handler central. Trata GenericError (domínio), erros de validação
- * do Mongoose e cai num 500 genérico para o resto.
+ * Error handler central. Trata GenericError (domínio), erros conhecidos do
+ * Mongo/Mongoose e cai num 500 genérico para o restante.
  */
-const errorMiddleware = (error, _req, res, next) => {
+const errorMiddleware = (error, req, res, next) => {
   if (!error) return next();
 
+  const requestId = req?.correlationId;
+
   if (error instanceof GenericError) {
+    const details = error.details && typeof error.details === "object" && !Array.isArray(error.details)
+      ? { ...error.details, ...(requestId ? { requestId } : {}) }
+      : error.details;
     return helpers.sendErrorResponse({
       res,
       statusCode: error.statusCode,
-      error: error.details,
+      error: details,
       message: error.message,
     });
   }
 
+  if (error?.code === 11000 || error?.name === "MongoServerError" && error?.code === 11000) {
+    const details = duplicateKeyDetails(error, requestId);
+    return helpers.sendErrorResponse({
+      res,
+      statusCode: 409,
+      error: details,
+      message: details.message,
+    });
+  }
+
   if (error.name === "ValidationError") {
     return helpers.sendErrorResponse({
       res,
       statusCode: 422,
-      error: Object.values(error.errors || {}).map((e) => e.message),
+      error: {
+        code: "VALIDATION_ERROR",
+        message: "Erro de validação.",
+        details: Object.entries(error.errors || {}).map(([field, item]) => ({
+          field,
+          message: item.message,
+        })),
+        ...(requestId ? { requestId } : {}),
+      },
       message: "Erro de validação.",
     });
   }
@@ -30,17 +97,33 @@ const errorMiddleware = (error, _req, res, next) => {
     return helpers.sendErrorResponse({
       res,
       statusCode: 400,
+      error: {
+        code: "INVALID_IDENTIFIER",
+        message: "Identificador inválido.",
+        ...(error.path ? { field: error.path } : {}),
+        ...(requestId ? { requestId } : {}),
+      },
       message: "Identificador inválido.",
     });
   }
 
   console.log("🆘 [UNHANDLED ERROR]:", error?.message || error);
+  const technicalMessage = process.env.NODE_ENV === "production"
+    ? undefined
+    : String(error?.message || error || "");
   return helpers.sendErrorResponse({
     res,
     statusCode: 500,
-    error: error.message,
+    error: {
+      code: "INTERNAL_ERROR",
+      message: "Houve um erro inesperado!",
+      ...(technicalMessage ? { technicalMessage } : {}),
+      ...(requestId ? { requestId } : {}),
+    },
     message: "Houve um erro inesperado!",
   });
 };
 
 module.exports = errorMiddleware;
+module.exports.firstDuplicateField = firstDuplicateField;
+module.exports.duplicateKeyDetails = duplicateKeyDetails;