@solidxai/core 0.1.13-beta.8 → 0.1.13-beta.9

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@solidxai/core",
-  "version": "0.1.13-beta.8",
+  "version": "0.1.13-beta.9",
   "description": "This module is a NestJS module containing all the required core providers required by a Solid application",
   "main": "dist/index.js",
   "types": "dist/index.d.ts",

package/src/entities/common.entity.ts

@@ -25,25 +25,29 @@ export abstract class CommonEntity {
     @Column({ name: 'published_at', default: null, nullable: true, transformer: LocalDateTimeTransformer })
     publishedAt: Date;
 
+    // Optional (not just nullable) because legacy tables (see LegacyCommonEntityWithExistingId)
+    // don't carry these versioning columns at all. CRUDService<T extends CommonEntity> is
+    // otherwise used for both plain and legacy entities, so these must stay structurally
+    // satisfiable by a type that never declares them.
     @Expose()
     @Column({ name: 'is_published', default: false })
     @Index()
-    isPublished: boolean;
+    isPublished?: boolean;
 
     @Expose()
     @Column({ name: 'is_latest', default: true })
     @Index()
-    isLatest: boolean;
+    isLatest?: boolean;
 
     @Expose()
     @Column({ type: "int", name: 'initial_entity_version_id', default: null, nullable: true })
     @Index()
-    initialEntityVersionId: number;
+    initialEntityVersionId?: number;
 
     @Expose()
     @Column({ name: 'published_tracker', default: "na" })
     @Index()
-    publishedTracker: string;
+    publishedTracker?: string;
 
     @Expose()
     @Column({ type: "varchar", name: 'locale_name', default: null })

package/src/services/settings/default-settings-provider.service.ts

@@ -10,6 +10,13 @@ import {
 export const DEFAULT_MEDIA_UPLOAD_DIR = "media-uploads";
 export const DEFAULT_MEDIA_FILE_STORAGE_DIR = "media-files-storage";
 
+// Seeded setting values are written once and never overwritten, so an unparseable
+// env var would persist. Fall back whenever it is missing or not a positive number.
+const timeoutFromEnv = (value: string | undefined, fallback: number): number => {
+  const parsed = Number(value);
+  return Number.isFinite(parsed) && parsed > 0 ? parsed : fallback;
+};
+
 // 1.
 const getSolidCoreSettings = (isProd: boolean) =>
   [
@@ -1477,7 +1484,7 @@ const getSolidCoreSettings = (isProd: boolean) =>
     {
       moduleName: "solid-core",
       key: "uiTestDefaultTimeoutMs",
-      value: 30000,
+      value: timeoutFromEnv(process.env.COMMON_UI_TEST_DEFAULT_TIMEOUT_MS, 30000),
       level: SettingLevel.SystemAdminEditable,
       label: "UI Test Default Timeout (ms)",
       group: "testing-settings",
@@ -1488,7 +1495,10 @@ const getSolidCoreSettings = (isProd: boolean) =>
     {
       moduleName: "solid-core",
       key: "uiTestNavigationTimeoutMs",
-      value: 30000,
+      value: timeoutFromEnv(
+        process.env.COMMON_UI_TEST_NAVIGATION_TIMEOUT_MS,
+        timeoutFromEnv(process.env.COMMON_UI_TEST_DEFAULT_TIMEOUT_MS, 30000),
+      ),
       level: SettingLevel.SystemAdminEditable,
       label: "UI Test Navigation Timeout (ms)",
       group: "testing-settings",

package/src/testing/README.md

@@ -104,6 +104,15 @@ Or set the installation-wide default under **Settings → Testing Settings** in
 (`uiTestDefaultTimeoutMs` / `uiTestNavigationTimeoutMs`). Those rows are created by `solid seed`;
 until then the built-in `30000` applies. Flags and env vars still win, so CI keeps per-run control.
 
+The seeded value of those two settings can be provisioned from the environment with
+`COMMON_UI_TEST_DEFAULT_TIMEOUT_MS` and `COMMON_UI_TEST_NAVIGATION_TIMEOUT_MS` (the latter falls
+back to the former). Seeding is insert-only, so these apply on first seed only — afterwards the
+stored value wins and the setting must be changed in the admin UI. Values that are missing,
+non-numeric, or not positive fall back to `30000`.
+
+Note these are distinct from the run-time `UI_TIMEOUT_MS` / `UI_NAVIGATION_TIMEOUT_MS` vars,
+which override the setting on a per-run basis and are read every run.
+
 Or bump only the one slow step, leaving the rest of the suite strict:
 ```json
 {