@onebun/drizzle 0.1.0 → 0.1.3

package/README.md

@@ -26,10 +26,11 @@ Choose the function that matches your database type configured in step 2.
 **For PostgreSQL:**
 ```typescript
 // src/schema/users.ts
-import { pgTable, serial, text, integer, timestamp } from 'drizzle-orm/pg-core';
+import { pgTable, text, integer, timestamp } from 'drizzle-orm/pg-core';
 
 export const users = pgTable('users', {
-  id: serial('id').primaryKey(),
+  // Use generatedAlwaysAsIdentity() for auto-increment primary key
+  id: integer('id').primaryKey().generatedAlwaysAsIdentity(),
   name: text('name').notNull(),
   email: text('email').notNull().unique(),
   age: integer('age'),
@@ -281,3 +282,7 @@ DB_MIGRATIONS_FOLDER=./drizzle
 DB_AUTO_MIGRATE=true
 DB_LOG_QUERIES=false
 ```
+
+## License
+
+[LGPL-3.0](../../LICENSE)

package/package.json

@@ -1,9 +1,19 @@
 {
   "name": "@onebun/drizzle",
-  "version": "0.1.0",
-  "description": "Drizzle ORM module for OneBun framework with SQLite and PostgreSQL support",
+  "version": "0.1.3",
+  "description": "Drizzle ORM module for OneBun framework - SQLite and PostgreSQL support",
   "license": "LGPL-3.0",
   "author": "RemRyahirev",
+  "keywords": [
+    "onebun",
+    "drizzle",
+    "orm",
+    "database",
+    "sqlite",
+    "postgresql",
+    "bun",
+    "typescript"
+  ],
   "repository": {
     "type": "git",
     "url": "git+https://github.com/RemRyahirev/onebun.git",
@@ -28,9 +38,9 @@
     "dev": "bun run --watch src/index.ts"
   },
   "dependencies": {
-    "@onebun/core": "workspace:*",
-    "@onebun/envs": "workspace:*",
-    "@onebun/logger": "workspace:*",
+    "@onebun/core": "workspace:^",
+    "@onebun/envs": "workspace:^",
+    "@onebun/logger": "workspace:^",
     "drizzle-orm": "^0.44.7",
     "drizzle-kit": "^0.31.6",
     "drizzle-arktype": "^0.1.3",

package/src/drizzle.service.ts

@@ -185,7 +185,41 @@ export class DrizzleService<TDbType extends DatabaseTypeLiteral = DatabaseTypeLi
 
   constructor(...args: unknown[]) {
     super(...args);
-    this.initPromise = this.autoInitialize();
+    // Only start auto-initialization if there's configuration to use
+    // Check synchronously to avoid unnecessary async work
+    if (this.shouldAutoInitialize()) {
+      this.initPromise = this.autoInitialize();
+    }
+  }
+
+  /**
+   * Check synchronously if auto-initialization should be attempted
+   * This avoids starting async work when there's no configuration
+   * Note: Only checks env vars here to avoid slow require() in constructor.
+   * Module options are checked in autoInitialize() which is async.
+   */
+  private shouldAutoInitialize(): boolean {
+    // Check if DB_URL env var is set
+    // Module options will be checked in autoInitialize() if this returns false
+    const dbUrl = process.env.DB_URL;
+    if (dbUrl && dbUrl.trim() !== '') {
+      return true;
+    }
+
+    // Also check if DrizzleModule has options set (static check without require)
+    // This is done by checking if the module was already loaded
+    try {
+      // Only check if module is already in cache (don't trigger new require)
+      const modulePath = require.resolve('./drizzle.module');
+      const cachedModule = require.cache[modulePath];
+      if (cachedModule?.exports?.DrizzleModule?.getOptions?.()?.connection) {
+        return true;
+      }
+    } catch {
+      // Module not resolved or not in cache
+    }
+
+    return false;
   }
 
   /**
@@ -448,7 +482,13 @@ export class DrizzleService<TDbType extends DatabaseTypeLiteral = DatabaseTypeLi
    * Close database connection
    */
   async close(): Promise<void> {
-    await this.waitForInit();
+    // Only wait for init if there are actual database clients to close
+    // This prevents hanging when close() is called on an uninitialized service
+    // Note: we check clients directly, not `initialized` flag, because autoInitialize()
+    // may still be running and `initialized` could be false while initPromise is pending
+    if (this.postgresClient || this.sqliteClient) {
+      await this.waitForInit();
+    }
 
     if (this.postgresClient) {
       // Bun.SQL uses close() instead of end()

package/src/migrations.ts

@@ -7,10 +7,10 @@ import { spawnSync } from 'bun';
  * using pgTable() or sqliteTable(). The schemas should be exported from schema files
  * specified in schemaPath.
  * 
- * @param options - Migration generation options
- * @param options.schemaPath - Path or paths to schema files containing pgTable/sqliteTable definitions
- * @param options.migrationsFolder - Output folder for migration files
- * @param options.dialect - Database dialect (sqlite or postgresql)
+ * @param options - Migration generation options:
+ *   - schemaPath: Path or paths to schema files containing pgTable/sqliteTable definitions
+ *   - migrationsFolder: Output folder for migration files
+ *   - dialect: Database dialect (sqlite or postgresql)
  * 
  * @example
  * ```typescript
@@ -73,10 +73,10 @@ export default {
  * This function applies schema changes directly to the database without creating
  * migration files. Useful for development, but not recommended for production.
  * 
- * @param options - Schema push options
- * @param options.schemaPath - Path(s) to schema files containing pgTable/sqliteTable definitions
- * @param options.dialect - Database dialect ('sqlite' or 'postgresql')
- * @param options.connectionString - Database connection string
+ * @param options - Schema push options:
+ *   - schemaPath: Path(s) to schema files containing pgTable/sqliteTable definitions
+ *   - dialect: Database dialect ('sqlite' or 'postgresql')
+ *   - connectionString: Database connection string
  * 
  * @example
  * ```typescript