@getstrata/core 0.5.90 → 0.5.97

package/CHANGELOG.md

@@ -1,5 +1,18 @@
 # @getstrata/core changelog
 
+## 0.5.97
+
+- `mapDatabaseError()` recognizes HTTP errors by `status` + `message`, not only `instanceof HttpError`. Subpath builds duplicate the class, so a thrown `ForbiddenError` was remapped to `400 Bad Request`.
+
+## 0.5.92
+
+- Re-export `resolveMembershipLookup` and `runWithMembershipContext` from the public barrel. Shared subpath shims re-export that barrel, so published `@getstrata/bootstrap` can import `@getstrata/core/auth/membershipContext` without a missing-export boot failure.
+
+## 0.5.91
+
+- `Schedule.command()` accepts any expression `Bun.cron.parse` understands. `dueTasks()` uses the next fire time in the current minute instead of a `*/N` whitelist.
+- `Factory.create()` persists `make()` through subclass `persist()` and strips a placeholder `id` of `0`. No states, sequences, or relationships.
+
 ## 0.5.90
 
 - **Breaking identity defaults:** `appKeyPrefix()` is `strata` and `appDisplayName()` is `Strata` when `APP_KEY_PREFIX` / `APP_NAME` are unset (were `workhub` / `WorkHub`). WorkHub pins those env vars.

package/README.md

@@ -82,7 +82,7 @@ import type { Migration } from "@getstrata/core/database/migrations/types";
 Package name: **`@getstrata/core`** (npm org [`@getstrata`](https://www.npmjs.com/org/getstrata)).
 
 1. Add `NPM_TOKEN` to GitHub repository secrets.
-2. Tag a release: `git tag v0.5.95 && git push origin v0.5.95`
+2. Tag a release: `git tag v0.5.96 && git push origin v0.5.96`
 3. [Release workflow](../../.github/workflows/release.yml) builds and runs `npm publish --access public`.
 
 Previously published as `@eyk-workhub/framework@0.1.0`, deprecated in favor of this package.

package/dist/core/database/errors.d.ts

@@ -1,4 +1,4 @@
-import { HttpError } from "@getstrata/core/errors/http";
+import { type HttpError } from "@getstrata/core/errors/http";
 interface PostgresErrorLike {
     code?: string;
     errno?: string | number;

package/dist/core/database/factory.d.ts

@@ -2,5 +2,8 @@ declare class Factory<TRecord extends object> {
     constructor();
     protected definition(): TRecord;
     make(overrides?: Partial<TRecord>): TRecord;
+    create(overrides?: Partial<TRecord>): Promise<TRecord>;
+    protected insertable(record: TRecord): Partial<TRecord>;
+    protected persist(_values: Partial<TRecord>): Promise<TRecord>;
 }
 export { Factory };

package/dist/core/errors/http.d.ts

@@ -30,4 +30,11 @@ declare class PayloadTooLargeError extends HttpError {
 declare class PreconditionFailedError extends HttpError {
     constructor(message?: string, details?: unknown);
 }
-export { BadRequestError, ConflictError, ForbiddenError, HttpError, NotFoundError, PayloadTooLargeError, PreconditionFailedError, UnauthorizedError, UnprocessableEntityError, ValidationError, };
+interface HttpErrorLike {
+    status: number;
+    message: string;
+    details?: unknown;
+}
+declare function isHttpErrorLike(error: unknown): error is HttpErrorLike;
+declare function toHttpError(error: unknown): HttpError | null;
+export { BadRequestError, ConflictError, ForbiddenError, HttpError, isHttpErrorLike, NotFoundError, PayloadTooLargeError, PreconditionFailedError, toHttpError, UnauthorizedError, UnprocessableEntityError, ValidationError, };

package/dist/core/scheduler/osCron.d.ts

@@ -7,8 +7,10 @@ interface InProcessCronJob {
     unref(): InProcessCronJob;
 }
 declare function parseScheduleExpression(expression: string, from?: Date): Date | null;
+declare function isScheduleExpressionDue(expression: string, now?: Date): boolean;
+declare function assertScheduleExpression(expression: string): void;
 declare function registerInProcessScheduleRunner(run: () => void | Promise<void>, expression?: string): InProcessCronJob;
 declare function installOsScheduleRunner(workerScriptPath: string, expression?: string, title?: string): Promise<void>;
 declare function uninstallOsScheduleRunner(title?: string): Promise<void>;
 export type { InProcessCronJob };
-export { DEFAULT_SCHEDULE_RUN_EXPRESSION, installOsScheduleRunner, OS_CRON_JOB_TITLE, parseScheduleExpression, registerInProcessScheduleRunner, uninstallOsScheduleRunner, };
+export { assertScheduleExpression, DEFAULT_SCHEDULE_RUN_EXPRESSION, installOsScheduleRunner, isScheduleExpressionDue, OS_CRON_JOB_TITLE, parseScheduleExpression, registerInProcessScheduleRunner, uninstallOsScheduleRunner, };

package/dist/entries/database/errors.js

@@ -3,7 +3,7 @@
 import {
   BadRequestError,
   ConflictError,
-  HttpError,
+  toHttpError,
   UnprocessableEntityError
 } from "@getstrata/core/errors/http";
 function isPostgresError(error) {
@@ -22,8 +22,9 @@ function getPostgresSqlState(error) {
   return;
 }
 function mapDatabaseError(error) {
-  if (error instanceof HttpError) {
-    return error;
+  const httpError = toHttpError(error);
+  if (httpError) {
+    return httpError;
   }
   if (!isPostgresError(error)) {
     const message = error instanceof Error ? error.message : "Database operation failed.";

package/dist/entries/database/factory.js

@@ -11,6 +11,19 @@ class Factory {
       ...overrides
     };
   }
+  async create(overrides = {}) {
+    return this.persist(this.insertable(this.make(overrides)));
+  }
+  insertable(record) {
+    const values = { ...record };
+    if (values.id === 0 || values.id === undefined || values.id === null) {
+      delete values.id;
+    }
+    return values;
+  }
+  persist(_values) {
+    throw new Error("Factory.persist() must be implemented to use create().");
+  }
 }
 export {
   Factory

package/dist/entries/http/response.js

@@ -1,12 +1,12 @@
 // @bun
 // ../../src/core/http/response.ts
-import { HttpError as HttpError3 } from "@getstrata/core/errors/http";
+import { toHttpError as toHttpError3 } from "@getstrata/core/errors/http";
 
 // ../../src/core/database/errors.ts
 import {
   BadRequestError,
   ConflictError,
-  HttpError,
+  toHttpError,
   UnprocessableEntityError
 } from "@getstrata/core/errors/http";
 function isPostgresError(error) {
@@ -25,8 +25,9 @@ function getPostgresSqlState(error) {
   return;
 }
 function mapDatabaseError(error) {
-  if (error instanceof HttpError) {
-    return error;
+  const httpError = toHttpError(error);
+  if (httpError) {
+    return httpError;
   }
   if (!isPostgresError(error)) {
     const message = error instanceof Error ? error.message : "Database operation failed.";
@@ -66,7 +67,7 @@ async function withDatabaseErrorHandling(operation) {
 }
 
 // ../../src/core/http/webErrorResponse.ts
-import { HttpError as HttpError2, UnauthorizedError, ValidationError } from "@getstrata/core/errors/http";
+import { toHttpError as toHttpError2, ValidationError } from "@getstrata/core/errors/http";
 
 // ../../src/core/runtime/frontendMode.ts
 function readFrontendMode() {
@@ -301,11 +302,11 @@ async function webErrorResponse(error, request) {
   if (!request || !isViewsEnabled() || requestPrefersJson(request)) {
     return null;
   }
-  const mappedError = error instanceof HttpError2 ? error : mapDatabaseError(error);
-  if (mappedError instanceof UnauthorizedError) {
+  const mappedError = toHttpError2(error) ?? mapDatabaseError(error);
+  if (mappedError.status === 401) {
     return Response.redirect(loginRedirectLocation(request), 302);
   }
-  const errors = mappedError instanceof ValidationError ? normalizeFieldErrors(mappedError.details) : undefined;
+  const errors = mappedError instanceof ValidationError || mappedError.name === "ValidationError" ? normalizeFieldErrors(mappedError.details) : undefined;
   return htmlErrorResponse({
     status: mappedError.status,
     title: errorPageTitle(mappedError.status, mappedError.message),
@@ -329,7 +330,7 @@ function noContentResponse() {
   return new Response(null, { status: 204 });
 }
 function errorResponse(error) {
-  const mappedError = error instanceof HttpError3 ? error : mapDatabaseError(error);
+  const mappedError = toHttpError3(error) ?? mapDatabaseError(error);
   return Response.json({
     error: mappedError.message,
     ...mappedError.details === undefined ? {} : { details: mappedError.details }

package/dist/entries/http/webErrorResponse.js

@@ -1,12 +1,12 @@
 // @bun
 // ../../src/core/http/webErrorResponse.ts
-import { HttpError as HttpError2, UnauthorizedError, ValidationError } from "@getstrata/core/errors/http";
+import { toHttpError as toHttpError2, ValidationError } from "@getstrata/core/errors/http";
 
 // ../../src/core/database/errors.ts
 import {
   BadRequestError,
   ConflictError,
-  HttpError,
+  toHttpError,
   UnprocessableEntityError
 } from "@getstrata/core/errors/http";
 function isPostgresError(error) {
@@ -25,8 +25,9 @@ function getPostgresSqlState(error) {
   return;
 }
 function mapDatabaseError(error) {
-  if (error instanceof HttpError) {
-    return error;
+  const httpError = toHttpError(error);
+  if (httpError) {
+    return httpError;
   }
   if (!isPostgresError(error)) {
     const message = error instanceof Error ? error.message : "Database operation failed.";
@@ -298,11 +299,11 @@ async function webErrorResponse(error, request) {
   if (!request || !isViewsEnabled() || requestPrefersJson(request)) {
     return null;
   }
-  const mappedError = error instanceof HttpError2 ? error : mapDatabaseError(error);
-  if (mappedError instanceof UnauthorizedError) {
+  const mappedError = toHttpError2(error) ?? mapDatabaseError(error);
+  if (mappedError.status === 401) {
     return Response.redirect(loginRedirectLocation(request), 302);
   }
-  const errors = mappedError instanceof ValidationError ? normalizeFieldErrors(mappedError.details) : undefined;
+  const errors = mappedError instanceof ValidationError || mappedError.name === "ValidationError" ? normalizeFieldErrors(mappedError.details) : undefined;
   return htmlErrorResponse({
     status: mappedError.status,
     title: errorPageTitle(mappedError.status, mappedError.message),

package/dist/entries/scheduler/osCron.js

@@ -5,6 +5,30 @@ var OS_CRON_JOB_TITLE = "getstrata-schedule-run";
 function parseScheduleExpression(expression, from = new Date) {
   return Bun.cron.parse(expression, from);
 }
+function minuteWindow(now) {
+  const start = new Date(now);
+  start.setSeconds(0, 0);
+  return { start, end: new Date(start.getTime() + 60000) };
+}
+function isScheduleExpressionDue(expression, now = new Date) {
+  const { start, end } = minuteWindow(now);
+  const next = parseScheduleExpression(expression, new Date(start.getTime() - 1));
+  if (!next) {
+    return false;
+  }
+  const timestamp = next.getTime();
+  return timestamp >= start.getTime() && timestamp < end.getTime();
+}
+function assertScheduleExpression(expression) {
+  try {
+    const next = parseScheduleExpression(expression);
+    if (!(next instanceof Date) || Number.isNaN(next.getTime())) {
+      throw new Error("unparsable");
+    }
+  } catch {
+    throw new Error(`Unsupported schedule expression "${expression}".`);
+  }
+}
 function registerInProcessScheduleRunner(run, expression = DEFAULT_SCHEDULE_RUN_EXPRESSION) {
   const register = Bun.cron;
   return register(expression, async () => {
@@ -20,7 +44,9 @@ async function uninstallOsScheduleRunner(title = OS_CRON_JOB_TITLE) {
 export {
   DEFAULT_SCHEDULE_RUN_EXPRESSION,
   OS_CRON_JOB_TITLE,
+  assertScheduleExpression,
   installOsScheduleRunner,
+  isScheduleExpressionDue,
   parseScheduleExpression,
   registerInProcessScheduleRunner,
   uninstallOsScheduleRunner

package/dist/entries/scheduler/schedule.js

@@ -1,34 +1,57 @@
 // @bun
-// ../../src/core/scheduler/schedule.ts
-function isSupportedScheduleExpression(expression) {
-  if (expression === "* * * * *") {
-    return true;
+// ../../src/core/scheduler/osCron.ts
+var DEFAULT_SCHEDULE_RUN_EXPRESSION = "* * * * *";
+var OS_CRON_JOB_TITLE = "getstrata-schedule-run";
+function parseScheduleExpression(expression, from = new Date) {
+  return Bun.cron.parse(expression, from);
+}
+function minuteWindow(now) {
+  const start = new Date(now);
+  start.setSeconds(0, 0);
+  return { start, end: new Date(start.getTime() + 60000) };
+}
+function isScheduleExpressionDue(expression, now = new Date) {
+  const { start, end } = minuteWindow(now);
+  const next = parseScheduleExpression(expression, new Date(start.getTime() - 1));
+  if (!next) {
+    return false;
+  }
+  const timestamp = next.getTime();
+  return timestamp >= start.getTime() && timestamp < end.getTime();
+}
+function assertScheduleExpression(expression) {
+  try {
+    const next = parseScheduleExpression(expression);
+    if (!(next instanceof Date) || Number.isNaN(next.getTime())) {
+      throw new Error("unparsable");
+    }
+  } catch {
+    throw new Error(`Unsupported schedule expression "${expression}".`);
   }
-  return /^(\*\/\d+)( \*){4}$/.test(expression);
+}
+function registerInProcessScheduleRunner(run, expression = DEFAULT_SCHEDULE_RUN_EXPRESSION) {
+  const register = Bun.cron;
+  return register(expression, async () => {
+    await run();
+  }).unref();
+}
+async function installOsScheduleRunner(workerScriptPath, expression = DEFAULT_SCHEDULE_RUN_EXPRESSION, title = OS_CRON_JOB_TITLE) {
+  await Bun.cron(workerScriptPath, expression, title);
+}
+async function uninstallOsScheduleRunner(title = OS_CRON_JOB_TITLE) {
+  await Bun.cron.remove(title);
 }
 
+// ../../src/core/scheduler/schedule.ts
 class Schedule {
   tasks = [];
   command(expression, name, run) {
-    if (!isSupportedScheduleExpression(expression)) {
-      throw new Error(`Unsupported schedule expression "${expression}". Only "* * * * *" and "*/N * * * *" are implemented.`);
-    }
+    assertScheduleExpression(expression);
     this.tasks.push({ expression, name, run });
     return this;
   }
   dueTasks(now = new Date) {
-    const minute = now.getMinutes();
-    return this.tasks.filter((task) => {
-      if (task.expression === "* * * * *") {
-        return true;
-      }
-      const intervalMatch = task.expression.match(/^\*\/(\d+)(?: \*){4}$/);
-      if (intervalMatch) {
-        const interval = Number.parseInt(intervalMatch[1] ?? "", 10);
-        return Number.isInteger(interval) && interval > 0 && minute % interval === 0;
-      }
-      return false;
-    });
+    return this.tasks.filter((task) => isScheduleExpressionDue(task.expression, now));
   }
   tasksList() {
     return [...this.tasks];

package/dist/framework/public-api.d.ts

@@ -10,7 +10,7 @@ export type { AuthUser } from "../core/auth/authContext.ts";
 export { authContext, currentAuthUser, runWithAuthUser } from "../core/auth/authContext.ts";
 export type { AuthGuard } from "../core/auth/guard.ts";
 export { ApiTokenGuard, AuthManager, CompositeGuard, DatabaseTokenGuard, GuestGuard, } from "../core/auth/guard.ts";
-export { configureMembershipLookup, currentOrganizationIds, currentOrgRole, hasMinimumOrgRole, hasOrgMembership, } from "../core/auth/membershipContext.ts";
+export { configureMembershipLookup, currentOrganizationIds, currentOrgRole, hasMinimumOrgRole, hasOrgMembership, resolveMembershipLookup, runWithMembershipContext, } from "../core/auth/membershipContext.ts";
 export { createMembershipMiddleware } from "../core/auth/membershipMiddleware.ts";
 export { appendOrganizationScope, appendProjectScope, assertOrganizationReadable, assertResourceInCurrentTenant, emptyPaginateResult, resolveOrganizationScope, scopedOrganizationIds, } from "../core/auth/membershipScope.ts";
 export { default as MembershipService, resolveMembershipService, } from "../core/auth/membershipService.ts";
@@ -48,7 +48,7 @@ export { runInTransaction } from "../core/database/transaction.ts";
 export type { QueryJoin, QueryJoinOn, QueryOptions, QueryOrder, QuerySelectItem, QueryWhere, } from "../core/database/types.ts";
 export type { WhereNode } from "../core/database/whereBuilder.ts";
 export { WhereBuilder } from "../core/database/whereBuilder.ts";
-export { BadRequestError, ConflictError, ForbiddenError, HttpError, NotFoundError, PayloadTooLargeError, PreconditionFailedError, UnauthorizedError, UnprocessableEntityError, ValidationError, } from "../core/errors/http.ts";
+export { BadRequestError, ConflictError, ForbiddenError, HttpError, isHttpErrorLike, NotFoundError, PayloadTooLargeError, PreconditionFailedError, toHttpError, UnauthorizedError, UnprocessableEntityError, ValidationError, } from "../core/errors/http.ts";
 export type { EventListener } from "../core/events/eventBus.ts";
 export { EventBus, eventBus } from "../core/events/eventBus.ts";
 export { modelEventName } from "../core/events/index.ts";

package/dist/index.js

@@ -120,6 +120,22 @@ class PreconditionFailedError extends HttpError {
     super(412, message, details);
   }
 }
+function isHttpErrorLike(error) {
+  if (typeof error !== "object" || error === null) {
+    return false;
+  }
+  const candidate = error;
+  return typeof candidate.status === "number" && Number.isInteger(candidate.status) && candidate.status >= 400 && candidate.status < 600 && typeof candidate.message === "string";
+}
+function toHttpError(error) {
+  if (error instanceof HttpError) {
+    return error;
+  }
+  if (!isHttpErrorLike(error)) {
+    return null;
+  }
+  return new HttpError(error.status, error.message, error.details);
+}
 
 // ../../src/core/runtime/asyncContextStore.ts
 import { AsyncLocalStorage } from "async_hooks";
@@ -1575,8 +1591,9 @@ function getPostgresSqlState(error) {
   return;
 }
 function mapDatabaseError(error) {
-  if (error instanceof HttpError) {
-    return error;
+  const httpError = toHttpError(error);
+  if (httpError) {
+    return httpError;
   }
   if (!isPostgresError(error)) {
     const message = error instanceof Error ? error.message : "Database operation failed.";
@@ -5298,11 +5315,11 @@ async function webErrorResponse(error, request) {
   if (!request || !isViewsEnabled() || requestPrefersJson(request)) {
     return null;
   }
-  const mappedError = error instanceof HttpError ? error : mapDatabaseError(error);
-  if (mappedError instanceof UnauthorizedError) {
+  const mappedError = toHttpError(error) ?? mapDatabaseError(error);
+  if (mappedError.status === 401) {
     return Response.redirect(loginRedirectLocation(request), 302);
   }
-  const errors = mappedError instanceof ValidationError ? normalizeFieldErrors(mappedError.details) : undefined;
+  const errors = mappedError instanceof ValidationError || mappedError.name === "ValidationError" ? normalizeFieldErrors(mappedError.details) : undefined;
   return htmlErrorResponse({
     status: mappedError.status,
     title: errorPageTitle(mappedError.status, mappedError.message),
@@ -5326,7 +5343,7 @@ function noContentResponse() {
   return new Response(null, { status: 204 });
 }
 function errorResponse(error) {
-  const mappedError = error instanceof HttpError ? error : mapDatabaseError(error);
+  const mappedError = toHttpError(error) ?? mapDatabaseError(error);
   return Response.json({
     error: mappedError.message,
     ...mappedError.details === undefined ? {} : { details: mappedError.details }
@@ -6556,36 +6573,45 @@ async function collectQueueMetrics() {
     failedCount
   };
 }
-// ../../src/core/scheduler/schedule.ts
-function isSupportedScheduleExpression(expression) {
-  if (expression === "* * * * *") {
-    return true;
+// ../../src/core/scheduler/osCron.ts
+function parseScheduleExpression(expression, from = new Date) {
+  return Bun.cron.parse(expression, from);
+}
+function minuteWindow(now) {
+  const start = new Date(now);
+  start.setSeconds(0, 0);
+  return { start, end: new Date(start.getTime() + 60000) };
+}
+function isScheduleExpressionDue(expression, now = new Date) {
+  const { start, end } = minuteWindow(now);
+  const next = parseScheduleExpression(expression, new Date(start.getTime() - 1));
+  if (!next) {
+    return false;
+  }
+  const timestamp = next.getTime();
+  return timestamp >= start.getTime() && timestamp < end.getTime();
+}
+function assertScheduleExpression(expression) {
+  try {
+    const next = parseScheduleExpression(expression);
+    if (!(next instanceof Date) || Number.isNaN(next.getTime())) {
+      throw new Error("unparsable");
+    }
+  } catch {
+    throw new Error(`Unsupported schedule expression "${expression}".`);
   }
-  return /^(\*\/\d+)( \*){4}$/.test(expression);
 }
 
+// ../../src/core/scheduler/schedule.ts
 class Schedule {
   tasks = [];
   command(expression, name, run) {
-    if (!isSupportedScheduleExpression(expression)) {
-      throw new Error(`Unsupported schedule expression "${expression}". Only "* * * * *" and "*/N * * * *" are implemented.`);
-    }
+    assertScheduleExpression(expression);
     this.tasks.push({ expression, name, run });
     return this;
   }
   dueTasks(now = new Date) {
-    const minute = now.getMinutes();
-    return this.tasks.filter((task) => {
-      if (task.expression === "* * * * *") {
-        return true;
-      }
-      const intervalMatch = task.expression.match(/^\*\/(\d+)(?: \*){4}$/);
-      if (intervalMatch) {
-        const interval = Number.parseInt(intervalMatch[1] ?? "", 10);
-        return Number.isInteger(interval) && interval > 0 && minute % interval === 0;
-      }
-      return false;
-    });
+    return this.tasks.filter((task) => isScheduleExpressionDue(task.expression, now));
   }
   tasksList() {
     return [...this.tasks];
@@ -6690,8 +6716,9 @@ function createTenantMiddleware() {
         });
       });
     } catch (error) {
-      if (error instanceof HttpError) {
-        return Response.json({ error: error.message }, { status: error.status });
+      const httpError = toHttpError(error);
+      if (httpError) {
+        return Response.json({ error: httpError.message }, { status: httpError.status });
       }
       throw error;
     }
@@ -7300,6 +7327,7 @@ export {
   isEtagEnabled,
   isGlobalAdmin,
   isHtmxRequest,
+  isHttpErrorLike,
   isInsideTenantDatabaseScope,
   isPublicReadsEnabled,
   isTenancyEnabled,
@@ -7362,6 +7390,7 @@ export {
   resolveCsrfTokenForRequest,
   resolveDatabaseDriver,
   resolveHtmlContentSecurityPolicy,
+  resolveMembershipLookup,
   resolveMembershipService,
   resolveOrganizationScope,
   resolveRepositoryConnection,
@@ -7378,6 +7407,7 @@ export {
   runSeedersFromDirectory,
   runWithAuthUser,
   runWithDatabaseConnection,
+  runWithMembershipContext,
   runWithRequestMeta,
   runWithTenant,
   runWithTenantDatabase,
@@ -7400,6 +7430,7 @@ export {
   stripMarkdown,
   temporarySignedUrl,
   textResponse,
+  toHttpError,
   toPaginatedResourceCollection,
   toResourceCollection,
   trustForwardedFor,

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@getstrata/core",
-  "version": "0.5.90",
+  "version": "0.5.97",
   "description": "Strata — Laravel-inspired Bun framework public API",
   "type": "module",
   "license": "MIT",