fragment-ts 1.0.34 → 1.0.36

package/src/testing/runner.ts

@@ -1,6 +1,8 @@
 import * as fs from "fs";
 import * as path from "path";
 import { glob } from "glob";
+import { EnvUtils } from "../shared/env.utils";
+import { TsConfigUtils } from "../shared/tsconfig.utils";
 
 /* ======================================================
  * Global augmentation
@@ -42,6 +44,11 @@ export class TestRunner {
     error: string;
     stack?: string;
   }> = [];
+  private verbose = false;
+
+  constructor(options: { verbose?: boolean } = {}) {
+    this.verbose = options.verbose ?? false;
+  }
 
   describe(name: string, fn: () => void): void {
     const suite: TestSuite = {
@@ -111,7 +118,18 @@ export class TestRunner {
           this.passed++;
         } catch (error: any) {
           console.log(`  ✗ ${test.name}`);
-          console.error(`    ${error?.message ?? error}`);
+
+          if (this.verbose || process.env.DEBUG === "true") {
+            console.error(`    ${error?.message ?? error}`);
+            if (error?.stack) {
+              console.error(
+                `    Stack: ${error.stack.split("\n").slice(1, 3).join("\n")}`,
+              );
+            }
+          } else {
+            console.error(`    ${error?.message ?? error}`);
+          }
+
           this.failed++;
           this.errors.push({
             suite: suite.name,
@@ -134,10 +152,10 @@ export class TestRunner {
         console.log(`  ${err.suite} > ${err.test}`);
         console.log(`    ${err.error}\n`);
       });
-      process.exit(1);
+      process.exitCode = 1;
     } else {
       console.log("✅ All tests passed!\n");
-      process.exit(0);
+      process.exitCode = 0;
     }
   }
 
@@ -168,6 +186,23 @@ export class TestRunner {
       }
     }
   }
+
+  /**
+   * Auto-detect test patterns based on environment and configuration
+   */
+  getAutoTestPattern(): string {
+    const isDevMode = EnvUtils.isDevelopmentMode();
+
+    if (isDevMode) {
+      // Development mode - use TypeScript
+      const srcDir = TsConfigUtils.getRootDir();
+      return `${path.relative(process.cwd(), srcDir)}/**/*.spec.ts`;
+    } else {
+      // Production mode - use JavaScript
+      const outDir = TsConfigUtils.getOutDir();
+      return `${path.relative(process.cwd(), outDir)}/**/*.spec.js`;
+    }
+  }
 }
 
 /* ======================================================
@@ -177,7 +212,13 @@ let runnerInstance: TestRunner | null = null;
 
 export function getTestRunner(): TestRunner {
   if (!runnerInstance) {
-    runnerInstance = new TestRunner();
+    // Check for verbose mode
+    const verbose =
+      process.env.TEST_VERBOSE === "true" ||
+      process.argv.includes("--verbose") ||
+      process.argv.includes("-v");
+
+    runnerInstance = new TestRunner({ verbose });
     G.__testRunner = runnerInstance;
   }
   return runnerInstance;
@@ -270,7 +311,7 @@ export function expect(actual: any) {
       }
     },
 
-    toThrow(expectedError?: string) {
+    toThrow(expectedError?: string | RegExp) {
       if (typeof actual !== "function") {
         throw new Error("toThrow expects a function");
       }
@@ -288,10 +329,20 @@ export function expect(actual: any) {
         throw new Error("Expected function to throw an error");
       }
 
-      if (expectedError && error?.message !== expectedError) {
-        throw new Error(
-          `Expected error message "${expectedError}" but got "${error?.message}"`,
-        );
+      if (expectedError) {
+        if (typeof expectedError === "string") {
+          if (error?.message !== expectedError) {
+            throw new Error(
+              `Expected error message "${expectedError}" but got "${error?.message}"`,
+            );
+          }
+        } else if (expectedError instanceof RegExp) {
+          if (!expectedError.test(error?.message)) {
+            throw new Error(
+              `Expected error message to match ${expectedError}, but got "${error?.message}"`,
+            );
+          }
+        }
       }
     },
 
@@ -355,11 +406,8 @@ if (require.main === module) {
     try {
       const runner = getTestRunner();
 
-      // Auto-detect TypeScript based on arguments
-      const isTsNode = process.argv.some(
-        (arg) => arg.includes("ts-node") || arg.includes("ts-node/register"),
-      );
-      const pattern = isTsNode ? "src/**/*.spec.ts" : "dist/**/*.spec.js";
+      // Use auto-detection instead of hardcoded patterns
+      const pattern = runner.getAutoTestPattern();
 
       await runner.loadTestFiles(pattern);
       await runner.run();

package/src/typeorm/typeorm-module.ts

@@ -70,6 +70,16 @@ type SupportedDatabaseConfig =
   | Partial<CapacitorConnectionOptions>
   | Partial<SpannerConnectionOptions>;
 
+// New interface for the fragment.json structure
+interface FragmentConfig {
+  development?: {
+    database?: SupportedDatabaseConfig;
+  };
+  production?: {
+    database?: SupportedDatabaseConfig;
+  };
+}
+
 export class TypeORMModule {
   private static dataSource: DataSource;
 
@@ -276,10 +286,16 @@ export class TypeORMModule {
       database: config.database ?? "database.sqlite",
       synchronize: config.synchronize ?? false,
       logging: config.logging ?? false,
-      entities: config.entities ?? ["dist/**/*.entity.js"],
-      migrations: config.migrations ?? ["dist/migrations/**/*.js"],
-      subscribers: config.subscribers ?? [],
-      ...config,
+      ...(config.entities !== undefined ? { entities: config.entities } : {}),
+      ...(config.migrations !== undefined
+        ? { migrations: config.migrations }
+        : {}),
+      ...(config.subscribers !== undefined
+        ? { subscribers: config.subscribers }
+        : {}),
+      ...(config.maxQueryExecutionTime !== undefined
+        ? { maxQueryExecutionTime: config.maxQueryExecutionTime }
+        : {}),
     };
   }
 
@@ -291,10 +307,13 @@ export class TypeORMModule {
       database: config.database ?? "database.sqlite",
       synchronize: config.synchronize ?? false,
       logging: config.logging ?? false,
-      entities: config.entities ?? ["dist/**/*.entity.js"],
-      migrations: config.migrations ?? ["dist/migrations/**/*.js"],
-      subscribers: config.subscribers ?? [],
-      ...config,
+      ...(config.entities !== undefined ? { entities: config.entities } : {}),
+      ...(config.migrations !== undefined
+        ? { migrations: config.migrations }
+        : {}),
+      ...(config.subscribers !== undefined
+        ? { subscribers: config.subscribers }
+        : {}),
     };
   }
 
@@ -310,10 +329,16 @@ export class TypeORMModule {
       database: config.database ?? "app",
       synchronize: config.synchronize ?? false,
       logging: config.logging ?? false,
-      entities: config.entities ?? ["dist/**/*.entity.js"],
-      migrations: config.migrations ?? ["dist/migrations/**/*.js"],
-      subscribers: config.subscribers ?? [],
-      ...config,
+      ...(config.entities !== undefined ? { entities: config.entities } : {}),
+      ...(config.migrations !== undefined
+        ? { migrations: config.migrations }
+        : {}),
+      ...(config.subscribers !== undefined
+        ? { subscribers: config.subscribers }
+        : {}),
+      ...(config.maxQueryExecutionTime !== undefined
+        ? { maxQueryExecutionTime: config.maxQueryExecutionTime }
+        : {}),
     };
   }
 
@@ -329,10 +354,16 @@ export class TypeORMModule {
       database: config.database ?? "app",
       synchronize: config.synchronize ?? false,
       logging: config.logging ?? false,
-      entities: config.entities ?? ["dist/**/*.entity.js"],
-      migrations: config.migrations ?? ["dist/migrations/**/*.js"],
-      subscribers: config.subscribers ?? [],
-      ...config,
+      ...(config.entities !== undefined ? { entities: config.entities } : {}),
+      ...(config.migrations !== undefined
+        ? { migrations: config.migrations }
+        : {}),
+      ...(config.subscribers !== undefined
+        ? { subscribers: config.subscribers }
+        : {}),
+      ...(config.maxQueryExecutionTime !== undefined
+        ? { maxQueryExecutionTime: config.maxQueryExecutionTime }
+        : {}),
     };
   }
 
@@ -348,11 +379,14 @@ export class TypeORMModule {
       database: config.database ?? "defaultdb",
       synchronize: config.synchronize ?? false,
       logging: config.logging ?? false,
-      entities: config.entities ?? ["dist/**/*.entity.js"],
-      migrations: config.migrations ?? ["dist/migrations/**/*.js"],
-      subscribers: config.subscribers ?? [],
       timeTravelQueries: config.timeTravelQueries ?? false,
-      ...config,
+      ...(config.entities !== undefined ? { entities: config.entities } : {}),
+      ...(config.migrations !== undefined
+        ? { migrations: config.migrations }
+        : {}),
+      ...(config.subscribers !== undefined
+        ? { subscribers: config.subscribers }
+        : {}),
     };
   }
 
@@ -368,11 +402,14 @@ export class TypeORMModule {
       database: config.database ?? "master",
       synchronize: config.synchronize ?? false,
       logging: config.logging ?? false,
-      entities: config.entities ?? ["dist/**/*.entity.js"],
-      migrations: config.migrations ?? ["dist/migrations/**/*.js"],
-      subscribers: config.subscribers ?? [],
-      options: config.options,
-      ...config,
+      ...(config.entities !== undefined ? { entities: config.entities } : {}),
+      ...(config.migrations !== undefined
+        ? { migrations: config.migrations }
+        : {}),
+      ...(config.subscribers !== undefined
+        ? { subscribers: config.subscribers }
+        : {}),
+      ...(config.options !== undefined ? { options: config.options } : {}),
     };
   }
 
@@ -388,10 +425,13 @@ export class TypeORMModule {
       sid: config.sid ?? "xe",
       synchronize: config.synchronize ?? false,
       logging: config.logging ?? false,
-      entities: config.entities ?? ["dist/**/*.entity.js"],
-      migrations: config.migrations ?? ["dist/migrations/**/*.js"],
-      subscribers: config.subscribers ?? [],
-      ...config,
+      ...(config.entities !== undefined ? { entities: config.entities } : {}),
+      ...(config.migrations !== undefined
+        ? { migrations: config.migrations }
+        : {}),
+      ...(config.subscribers !== undefined
+        ? { subscribers: config.subscribers }
+        : {}),
     };
   }
 
@@ -404,13 +444,16 @@ export class TypeORMModule {
       port: config.port ?? 30015,
       username: config.username ?? "SYSTEM",
       password: config.password ?? "",
-      schema: config.schema,
       synchronize: config.synchronize ?? false,
       logging: config.logging ?? false,
-      entities: config.entities ?? ["dist/**/*.entity.js"],
-      migrations: config.migrations ?? ["dist/migrations/**/*.js"],
-      subscribers: config.subscribers ?? [],
-      ...config,
+      ...(config.schema !== undefined ? { schema: config.schema } : {}),
+      ...(config.entities !== undefined ? { entities: config.entities } : {}),
+      ...(config.migrations !== undefined
+        ? { migrations: config.migrations }
+        : {}),
+      ...(config.subscribers !== undefined
+        ? { subscribers: config.subscribers }
+        : {}),
     };
   }
 
@@ -442,14 +485,24 @@ export class TypeORMModule {
       password: config.password ?? "",
       synchronize: config.synchronize ?? false,
       logging: config.logging ?? false,
-      entities: config.entities ?? ["dist/**/*.entity.js"],
-      migrations: config.migrations ?? ["dist/migrations/**/*.js"],
-      subscribers: config.subscribers ?? [],
-      driver: config.driver,
-      serviceConfigOptions: config.serviceConfigOptions,
-      formatOptions: config.formatOptions,
-      legacySpatialSupport: config.legacySpatialSupport,
-      ssl: config.ssl,
+      ...(config.entities !== undefined ? { entities: config.entities } : {}),
+      ...(config.migrations !== undefined
+        ? { migrations: config.migrations }
+        : {}),
+      ...(config.subscribers !== undefined
+        ? { subscribers: config.subscribers }
+        : {}),
+      ...(config.driver !== undefined ? { driver: config.driver } : {}),
+      ...(config.serviceConfigOptions !== undefined
+        ? { serviceConfigOptions: config.serviceConfigOptions }
+        : {}),
+      ...(config.formatOptions !== undefined
+        ? { formatOptions: config.formatOptions }
+        : {}),
+      ...(config.legacySpatialSupport !== undefined
+        ? { legacySpatialSupport: config.legacySpatialSupport }
+        : {}),
+      ...(config.ssl !== undefined ? { ssl: config.ssl } : {}),
     };
   }
 
@@ -476,14 +529,26 @@ export class TypeORMModule {
       database: config.database ?? "app",
       synchronize: config.synchronize ?? false,
       logging: config.logging ?? false,
-      entities: config.entities ?? ["dist/**/*.entity.js"],
-      migrations: config.migrations ?? ["dist/migrations/**/*.js"],
-      subscribers: config.subscribers ?? [],
-      driver: config.driver,
-      transformParameters: config.transformParameters,
-      poolErrorHandler: config.poolErrorHandler,
-      serviceConfigOptions: config.serviceConfigOptions,
-      formatOptions: config.formatOptions,
+      ...(config.entities !== undefined ? { entities: config.entities } : {}),
+      ...(config.migrations !== undefined
+        ? { migrations: config.migrations }
+        : {}),
+      ...(config.subscribers !== undefined
+        ? { subscribers: config.subscribers }
+        : {}),
+      ...(config.driver !== undefined ? { driver: config.driver } : {}),
+      ...(config.transformParameters !== undefined
+        ? { transformParameters: config.transformParameters }
+        : {}),
+      ...(config.poolErrorHandler !== undefined
+        ? { poolErrorHandler: config.poolErrorHandler }
+        : {}),
+      ...(config.serviceConfigOptions !== undefined
+        ? { serviceConfigOptions: config.serviceConfigOptions }
+        : {}),
+      ...(config.formatOptions !== undefined
+        ? { formatOptions: config.formatOptions }
+        : {}),
     };
   }
 
@@ -497,10 +562,13 @@ export class TypeORMModule {
       databaseId: config.databaseId ?? "",
       synchronize: config.synchronize ?? false,
       logging: config.logging ?? false,
-      entities: config.entities ?? ["dist/**/*.entity.js"],
-      migrations: config.migrations ?? ["dist/migrations/**/*.js"],
-      subscribers: config.subscribers ?? [],
-      ...config,
+      ...(config.entities !== undefined ? { entities: config.entities } : {}),
+      ...(config.migrations !== undefined
+        ? { migrations: config.migrations }
+        : {}),
+      ...(config.subscribers !== undefined
+        ? { subscribers: config.subscribers }
+        : {}),
     };
   }
 
@@ -517,8 +585,7 @@ export class TypeORMModule {
       database: config.database ?? "app",
       synchronize: config.synchronize ?? false,
       logging: config.logging ?? false,
-      entities: config.entities ?? ["dist/**/*.entity.js"],
-      ...config,
+      ...(config.entities !== undefined ? { entities: config.entities } : {}),
     };
   }
 
@@ -534,10 +601,13 @@ export class TypeORMModule {
       location: config.location ?? "default",
       synchronize: config.synchronize ?? false,
       logging: config.logging ?? false,
-      entities: config.entities ?? ["dist/**/*.entity.js"],
-      migrations: config.migrations ?? ["dist/migrations/**/*.js"],
-      subscribers: config.subscribers ?? [],
-      ...config,
+      ...(config.entities !== undefined ? { entities: config.entities } : {}),
+      ...(config.migrations !== undefined
+        ? { migrations: config.migrations }
+        : {}),
+      ...(config.subscribers !== undefined
+        ? { subscribers: config.subscribers }
+        : {}),
     };
   }
 
@@ -555,17 +625,22 @@ export class TypeORMModule {
       database: config.database ?? "app.db",
       driver: config.driver,
       readOnly: config.readOnly ?? false,
-      key: config.key,
       multithreading: config.multithreading ?? false,
       migrate: config.migrate ?? false,
-      iosFlags: config.iosFlags,
-      androidFlags: config.androidFlags,
       synchronize: config.synchronize ?? false,
       logging: config.logging ?? false,
-      entities: config.entities ?? ["dist/**/*.entity.js"],
-      migrations: config.migrations ?? ["dist/migrations/**/*.js"],
-      subscribers: config.subscribers ?? [],
-      ...config,
+      ...(config.key !== undefined ? { key: config.key } : {}),
+      ...(config.iosFlags !== undefined ? { iosFlags: config.iosFlags } : {}),
+      ...(config.androidFlags !== undefined
+        ? { androidFlags: config.androidFlags }
+        : {}),
+      ...(config.entities !== undefined ? { entities: config.entities } : {}),
+      ...(config.migrations !== undefined
+        ? { migrations: config.migrations }
+        : {}),
+      ...(config.subscribers !== undefined
+        ? { subscribers: config.subscribers }
+        : {}),
     };
   }
 
@@ -578,10 +653,13 @@ export class TypeORMModule {
       location: config.location ?? "default",
       synchronize: config.synchronize ?? false,
       logging: config.logging ?? false,
-      entities: config.entities ?? ["dist/**/*.entity.js"],
-      migrations: config.migrations ?? ["dist/migrations/**/*.js"],
-      subscribers: config.subscribers ?? [],
-      ...config,
+      ...(config.entities !== undefined ? { entities: config.entities } : {}),
+      ...(config.migrations !== undefined
+        ? { migrations: config.migrations }
+        : {}),
+      ...(config.subscribers !== undefined
+        ? { subscribers: config.subscribers }
+        : {}),
     };
   }
 
@@ -594,10 +672,13 @@ export class TypeORMModule {
       location: config.location ?? "browser",
       synchronize: config.synchronize ?? false,
       logging: config.logging ?? false,
-      entities: config.entities ?? ["dist/**/*.entity.js"],
-      migrations: config.migrations ?? ["dist/migrations/**/*.js"],
-      subscribers: config.subscribers ?? [],
-      ...config,
+      ...(config.entities !== undefined ? { entities: config.entities } : {}),
+      ...(config.migrations !== undefined
+        ? { migrations: config.migrations }
+        : {}),
+      ...(config.subscribers !== undefined
+        ? { subscribers: config.subscribers }
+        : {}),
     };
   }
 
@@ -616,10 +697,13 @@ export class TypeORMModule {
       driver: config.driver,
       synchronize: config.synchronize ?? false,
       logging: config.logging ?? false,
-      entities: config.entities ?? ["dist/**/*.entity.js"],
-      migrations: config.migrations ?? ["dist/migrations/**/*.js"],
-      subscribers: config.subscribers ?? [],
-      ...config,
+      ...(config.entities !== undefined ? { entities: config.entities } : {}),
+      ...(config.migrations !== undefined
+        ? { migrations: config.migrations }
+        : {}),
+      ...(config.subscribers !== undefined
+        ? { subscribers: config.subscribers }
+        : {}),
     };
   }
 
@@ -638,15 +722,18 @@ export class TypeORMModule {
       driver: config.driver,
       synchronize: config.synchronize ?? false,
       logging: config.logging ?? false,
-      entities: config.entities ?? ["dist/**/*.entity.js"],
-      migrations: config.migrations ?? ["dist/migrations/**/*.js"],
-      subscribers: config.subscribers ?? [],
-      ...config,
+      ...(config.entities !== undefined ? { entities: config.entities } : {}),
+      ...(config.migrations !== undefined
+        ? { migrations: config.migrations }
+        : {}),
+      ...(config.subscribers !== undefined
+        ? { subscribers: config.subscribers }
+        : {}),
     };
   }
 
   /* ======================================================
-   * fragment.json loader
+   * fragment.json loader - UPDATED FOR NEW STRUCTURE
    * ====================================================== */
   private static loadFileConfig(): SupportedDatabaseConfig {
     const configPath = path.join(process.cwd(), "fragment.json");
@@ -656,8 +743,25 @@ export class TypeORMModule {
     }
 
     try {
-      const raw = JSON.parse(fs.readFileSync(configPath, "utf-8"));
-      const dbConfig = raw.database ?? {};
+      const raw = JSON.parse(
+        fs.readFileSync(configPath, "utf-8"),
+      ) as FragmentConfig;
+
+      // Determine environment mode
+      const isDevMode = this.isDevelopmentMode();
+      console.log(`Detected ${isDevMode ? "development" : "production"} mode`);
+
+      // Get the appropriate config section
+      let dbConfig = {};
+      if (isDevMode && raw.development?.database) {
+        dbConfig = raw.development.database;
+      } else if (!isDevMode && raw.production?.database) {
+        dbConfig = raw.production.database;
+      } else if (raw.development?.database) {
+        // Fallback to development if production not found
+        dbConfig = raw.development.database;
+        console.warn("Production config not found, using development config");
+      }
 
       // Interpolate environment variables
       return this.interpolateEnvVars(dbConfig);
@@ -667,6 +771,25 @@ export class TypeORMModule {
     }
   }
 
+  /* ======================================================
+   * Environment detection
+   * ====================================================== */
+  private static isDevelopmentMode(): boolean {
+    // Check explicit environment variable
+    if (process.env.FRAGMENT_DEV_MODE !== undefined) {
+      return process.env.FRAGMENT_DEV_MODE === "true";
+    }
+
+    // Check NODE_ENV
+    if (process.env.NODE_ENV) {
+      return process.env.NODE_ENV === "development";
+    }
+
+    // Check if we're running TypeScript files directly (dev mode)
+    const mainFile = require.main?.filename || "";
+    return mainFile.includes(".ts") || mainFile.includes("ts-node");
+  }
+
   /* ======================================================
    * Utilities
    * ====================================================== */

package/src/web/application.ts

@@ -19,6 +19,7 @@ import {
   Interceptor,
   ExceptionFilter,
 } from "./interfaces";
+import { EnvUtils } from "../shared/env.utils";
 
 export class FragmentWebApplication {
   private app: Express;
@@ -93,8 +94,8 @@ export class FragmentWebApplication {
     console.log(`📁 dist/ exists: ${distExists}`);
     console.log(`📁 src/ exists: ${srcExists}`);
 
-    // Determine if we're running TypeScript directly (dev) or compiled JS (prod)
-    const isDevMode = this.isRunningTypeScript();
+    // Use EnvUtils for consistent environment detection
+    const isDevMode = EnvUtils.isDevelopmentMode();
     console.log(
       `💻 Running in ${isDevMode ? "development" : "production"} mode`,
     );
@@ -112,37 +113,7 @@ export class FragmentWebApplication {
     }
   }
 
-  private isRunningTypeScript(): boolean {
-    // Check if ts-node is in require.extensions
-    if (require.extensions[".ts"]) {
-      return true;
-    }
-
-    // Check if process is running with ts-node or tsx
-    const execPath = process.argv[0];
-    const scriptPath = process.argv[1] || "";
-
-    if (
-      execPath.includes("ts-node") ||
-      execPath.includes("tsx") ||
-      scriptPath.includes("ts-node") ||
-      scriptPath.includes("tsx")
-    ) {
-      return true;
-    }
-
-    // Check if main module has .ts extension
-    if (require.main?.filename.endsWith(".ts")) {
-      return true;
-    }
-
-    // Check NODE_ENV or explicit flag
-    if (process.env.FRAGMENT_DEV_MODE === "true") {
-      return true;
-    }
-
-    return false;
-  }
+  // Remove the old isRunningTypeScript method since we're using EnvUtils
 
   private discoverAndRegisterComponents(): void {
     const classes = this.metadataStorage.getAllClasses();