@aeriajs/core 0.0.175 → 0.0.177

package/dist/collection/traverseDocument.js

@@ -128,7 +128,7 @@ const autoCast = (value, ctx) => {
             }
         }
         case 'object': {
-            if (!value || (0, common_1.isObjectId)(value)) {
+            if (!value || value instanceof mongodb_1.ObjectId) {
                 return value;
             }
             if (!('description' in ctx.options) || !ctx.options.recurseDeep) {

package/dist/collection/traverseDocument.mjs

@@ -1,6 +1,6 @@
 "use strict";
 import { Result, ACError, ValidationErrorCode, TraverseError } from "@aeriajs/types";
-import { throwIfError, pipe, isReference, getValueFromPath, isObjectId, isError } from "@aeriajs/common";
+import { throwIfError, pipe, isReference, getValueFromPath, isError } from "@aeriajs/common";
 import { makeValidationError, validateProperty, validateWholeness } from "@aeriajs/validation";
 import { getCollection } from "@aeriajs/entrypoint";
 import { ObjectId } from "mongodb";
@@ -96,7 +96,7 @@ const autoCast = (value, ctx) => {
       }
     }
     case "object": {
-      if (!value || isObjectId(value)) {
+      if (!value || value instanceof ObjectId) {
         return value;
       }
       if (!("description" in ctx.options) || !ctx.options.recurseDeep) {

package/dist/functions/getAll.d.ts

@@ -1,6 +1,6 @@
-import type { Context, SchemaWithId, GetAllPayload } from '@aeriajs/types';
-import { Result } from '@aeriajs/types';
+import type { Context, SchemaWithId, GetAllPayload, GetAllReturnType } from '@aeriajs/types';
 export type GetAllOptions = {
     bypassSecurity?: boolean;
+    noDefaultLimit?: boolean;
 };
-export declare const getAll: <TContext extends Context>(payload: GetAllPayload<SchemaWithId<TContext["description"]>> | undefined, context: TContext, options?: GetAllOptions) => Promise<Result.Either<any, any>>;
+export declare const getAll: <TContext extends Context>(payload: GetAllPayload<SchemaWithId<TContext["description"]>> | undefined, context: TContext, options?: GetAllOptions) => Promise<GetAllReturnType<SchemaWithId<TContext["description"]>>>;

package/dist/functions/getAll.js

@@ -6,7 +6,7 @@ const types_1 = require("@aeriajs/types");
 const common_1 = require("@aeriajs/common");
 const index_js_1 = require("../collection/index.js");
 const internalGetAll = async (payload, context) => {
-    const { limit = context.config.defaultPaginationLimit, sort, project, offset = 0, } = payload;
+    const { limit, sort, project, offset = 0, } = payload;
     const filters = payload.filters
         ? Object.assign({}, payload.filters)
         : {};
@@ -52,9 +52,11 @@ const internalGetAll = async (payload, context) => {
             $skip: offset,
         });
     }
-    pipeline.push({
-        $limit: limit,
-    });
+    if (limit) {
+        pipeline.push({
+            $limit: limit,
+        });
+    }
     if (project) {
         const projection = (0, index_js_1.normalizeProjection)(project, context.description);
         if (projection) {
@@ -97,7 +99,12 @@ const getAll = async (payload, context, options = {}) => {
     const security = (0, security_1.useSecurity)(context);
     const { error, result: securedPayload } = await security.secureReadPayload(payload);
     if (error) {
-        return types_1.Result.error(error);
+        return context.error(types_1.HTTPStatus.Forbidden, {
+            code: error,
+        });
+    }
+    if (!options.noDefaultLimit) {
+        securedPayload.limit ||= context.config.defaultPaginationLimit;
     }
     return (0, security_1.applyReadMiddlewares)(securedPayload, context, internalGetAll);
 };

package/dist/functions/getAll.mjs

@@ -1,6 +1,6 @@
 "use strict";
 import { useSecurity, applyReadMiddlewares } from "@aeriajs/security";
-import { Result } from "@aeriajs/types";
+import { HTTPStatus, Result } from "@aeriajs/types";
 import { throwIfError } from "@aeriajs/common";
 import {
   traverseDocument,
@@ -10,7 +10,7 @@ import {
 } from "../collection/index.mjs";
 const internalGetAll = async (payload, context) => {
   const {
-    limit = context.config.defaultPaginationLimit,
+    limit,
     sort,
     project,
     offset = 0
@@ -52,9 +52,11 @@ const internalGetAll = async (payload, context) => {
       $skip: offset
     });
   }
-  pipeline.push({
-    $limit: limit
-  });
+  if (limit) {
+    pipeline.push({
+      $limit: limit
+    });
+  }
   if (project) {
     const projection = normalizeProjection(project, context.description);
     if (projection) {
@@ -95,7 +97,12 @@ export const getAll = async (payload, context, options = {}) => {
   const security = useSecurity(context);
   const { error, result: securedPayload } = await security.secureReadPayload(payload);
   if (error) {
-    return Result.error(error);
+    return context.error(HTTPStatus.Forbidden, {
+      code: error
+    });
+  }
+  if (!options.noDefaultLimit) {
+    securedPayload.limit ||= context.config.defaultPaginationLimit;
   }
   return applyReadMiddlewares(securedPayload, context, internalGetAll);
 };

package/dist/functions/insert.d.ts

@@ -17,10 +17,10 @@ export declare const insertErrorSchema: () => {
             readonly required: readonly ["httpStatus", "code"];
             readonly properties: {
                 readonly httpStatus: {
-                    readonly enum: [HTTPStatus.Forbidden, HTTPStatus.NotFound, HTTPStatus.UnprocessableContent];
+                    readonly enum: [HTTPStatus.Forbidden, HTTPStatus.NotFound, HTTPStatus.UnprocessableContent, HTTPStatus.BadRequest];
                 };
                 readonly code: {
-                    readonly enum: [ACError.InsecureOperator, ACError.OwnershipError, ACError.ResourceNotFound, ACError.TargetImmutable, ValidationErrorCode.EmptyTarget, ValidationErrorCode.InvalidProperties, ValidationErrorCode.MissingProperties, TraverseError.InvalidDocumentId, TraverseError.InvalidTempfile];
+                    readonly enum: [ACError.InsecureOperator, ACError.OwnershipError, ACError.ResourceNotFound, ACError.TargetImmutable, ACError.MalformedInput, ValidationErrorCode.EmptyTarget, ValidationErrorCode.InvalidProperties, ValidationErrorCode.MissingProperties, TraverseError.InvalidDocumentId, TraverseError.InvalidTempfile];
                 };
                 readonly message: {
                     readonly type: "string";

package/dist/functions/insert.js

@@ -12,12 +12,14 @@ const insertErrorSchema = () => (0, common_1.endpointErrorSchema)({
         types_1.HTTPStatus.Forbidden,
         types_1.HTTPStatus.NotFound,
         types_1.HTTPStatus.UnprocessableContent,
+        types_1.HTTPStatus.BadRequest,
     ],
     code: [
         types_1.ACError.InsecureOperator,
         types_1.ACError.OwnershipError,
         types_1.ACError.ResourceNotFound,
         types_1.ACError.TargetImmutable,
+        types_1.ACError.MalformedInput,
         types_1.ValidationErrorCode.EmptyTarget,
         types_1.ValidationErrorCode.InvalidProperties,
         types_1.ValidationErrorCode.MissingProperties,
@@ -74,13 +76,16 @@ const internalInsert = async (payload, context) => {
         ...context,
         inherited: true,
     };
-    const newDocument = (0, common_1.throwIfError)(await (0, get_js_1.get)({
+    const { error: getError, result: newDocument } = await (0, get_js_1.get)({
         filters: {
             _id: newId,
         },
     }, inheritedContext, {
         bypassSecurity: true,
-    }));
+    });
+    if (getError) {
+        return types_1.Result.error(getError);
+    }
     return types_1.Result.result(newDocument);
 };
 const insert = async (payload, context, options = {}) => {

package/dist/functions/insert.mjs

@@ -2,20 +2,22 @@
 import { ObjectId } from "mongodb";
 import { Result, HTTPStatus, ACError, ValidationErrorCode, TraverseError } from "@aeriajs/types";
 import { useSecurity, applyWriteMiddlewares } from "@aeriajs/security";
-import { throwIfError, endpointErrorSchema } from "@aeriajs/common";
+import { endpointErrorSchema } from "@aeriajs/common";
 import { traverseDocument, prepareCreate, prepareUpdate } from "../collection/index.mjs";
 import { get } from "./get.mjs";
 export const insertErrorSchema = () => endpointErrorSchema({
   httpStatus: [
     HTTPStatus.Forbidden,
     HTTPStatus.NotFound,
-    HTTPStatus.UnprocessableContent
+    HTTPStatus.UnprocessableContent,
+    HTTPStatus.BadRequest
   ],
   code: [
     ACError.InsecureOperator,
     ACError.OwnershipError,
     ACError.ResourceNotFound,
     ACError.TargetImmutable,
+    ACError.MalformedInput,
     ValidationErrorCode.EmptyTarget,
     ValidationErrorCode.InvalidProperties,
     ValidationErrorCode.MissingProperties,
@@ -66,13 +68,16 @@ const internalInsert = async (payload, context) => {
     ...context,
     inherited: true
   };
-  const newDocument = throwIfError(await get({
+  const { error: getError, result: newDocument } = await get({
     filters: {
       _id: newId
     }
   }, inheritedContext, {
     bypassSecurity: true
-  }));
+  });
+  if (getError) {
+    return Result.error(getError);
+  }
   return Result.result(newDocument);
 };
 export const insert = async (payload, context, options = {}) => {

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@aeriajs/core",
-  "version": "0.0.175",
+  "version": "0.0.177",
   "description": "",
   "main": "dist/index.js",
   "aeriaMain": "tests/fixtures/aeriaMain.js",
@@ -42,13 +42,13 @@
     "mongodb-memory-server": "^9.2.0"
   },
   "peerDependencies": {
-    "@aeriajs/builtins": "^0.0.175",
-    "@aeriajs/common": "^0.0.105",
-    "@aeriajs/entrypoint": "^0.0.108",
-    "@aeriajs/http": "^0.0.119",
-    "@aeriajs/security": "^0.0.175",
-    "@aeriajs/types": "^0.0.88",
-    "@aeriajs/validation": "^0.0.108"
+    "@aeriajs/builtins": "^0.0.177",
+    "@aeriajs/common": "^0.0.107",
+    "@aeriajs/entrypoint": "^0.0.110",
+    "@aeriajs/http": "^0.0.121",
+    "@aeriajs/security": "^0.0.177",
+    "@aeriajs/types": "^0.0.90",
+    "@aeriajs/validation": "^0.0.110"
   },
   "dependencies": {
     "mongodb": "^6.5.0",