@xeplr/base-apis 2.0.5 → 2.0.6

package/README.md

@@ -203,7 +203,7 @@ Responses use `@xeplr/utils`' shape: `{ code, message, error, dataArray, updated
 
 | error | status |
 |---|---|
-| `ValidationError` | 422 |
+| `ValidationError` | 422. jsonSchema's answers `Validation failed` with `fields`; one a model hook throws with its own `type` (e.g. a licence limit) answers its own `message` |
 | `UniqueViolationError`, `ForeignKeyViolationError` | 409 |
 | `NotNullViolationError` (`<column> is required`), `CheckViolationError`, `DataError` | 400 |
 | `NotFoundError` | 404 |

package/lib/genericController.js

@@ -7,7 +7,7 @@ var { generateId } = require('@xeplr/utils/lib/helpers');
  * of a blanket 500. Matched by `err.name` (not instanceof) so it survives two
  * copies of objection in a linked monorepo. Unknown errors still fall to 500.
  *
- *   ValidationError        → 422  { fields }   (jsonSchema: required/format/…)
+ *   ValidationError        → 422  { fields }   (jsonSchema: required/format/…); a hook's own type keeps its message
  *   UniqueViolationError   → 409  already exists
  *   NotNullViolationError  → 400  <column> is required
  *   ForeignKeyViolationError → 409 referenced record missing / in use
@@ -19,8 +19,12 @@ function sendError(req, res, err) {
 
   switch (err && err.name) {
     case 'ValidationError':
+      // A model's own check (a hook throwing ValidationError with a type of
+      // its own, e.g. a licence limit) carries a sentence for the person;
+      // jsonSchema's (ModelValidation) carries per-field detail instead.
       return respond(res, HTTP.VALIDATION_ERROR, STATUS.VALIDATION_ERROR, 'validation_error', {
-        message: 'Validation failed', fields: err.data || {}
+        message: err.type && err.type !== 'ModelValidation' && err.message ? err.message : 'Validation failed',
+        fields: err.data || {}
       });
     case 'UniqueViolationError':
       return respond(res, HTTP.CONFLICT, STATUS.CONFLICT, 'conflict', {
@@ -66,7 +70,6 @@ function sendError(req, res, err) {
  */
 async function save(req, res, hierarchy) {
   var changeSet = req.body;
-console.log('bruh: ', 1);
   if (!Array.isArray(changeSet) || changeSet.length === 0) {
     return respond(res, HTTP.BAD_REQUEST, STATUS.BAD_REQUEST, 'bad_request', {
       message: 'Expected a non-empty changeset array'
@@ -102,7 +105,6 @@ console.log('bruh: ', 1);
       var ids = await processEntry(changeSet[i], rootDef, trx, null, queryContext);
       updatedIds = updatedIds.concat(ids);
     }
-console.log('bruh: ', 2);
 
     await trx.commit();
 
@@ -114,7 +116,6 @@ console.log('bruh: ', 2);
         else console.error('[genericController] afterCommit callback failed:', err.message);
       }
     }
-console.log('bruh: ', 3);
 
     respond(res, HTTP.OK, STATUS.UPDATED, 'updated', { updatedIds: updatedIds });
   } catch (err) {
@@ -296,4 +297,4 @@ async function list(req, res, Model, graph) {
   }
 }
 
-module.exports = { save, deleteByIds, getById, list };
+module.exports = { save, deleteByIds, getById, list, sendError };

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@xeplr/base-apis",
-  "version": "2.0.5",
+  "version": "2.0.6",
   "description": "Base API bootstrapper — Express and plain HTTP server factories with middleware injection",
   "main": "index.js",
   "bin": {