@shetty4l/core 0.1.38 → 0.1.40

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@shetty4l/core",
-  "version": "0.1.38",
+  "version": "0.1.40",
   "description": "Shared infrastructure primitives for Bun/TypeScript services",
   "repository": {
     "type": "git",

package/src/db.ts

@@ -77,6 +77,7 @@ export function createDatabaseManager(opts: DatabaseOpts): DatabaseManager {
       // Enable WAL mode for better concurrent access (not applicable to :memory:)
       if (path !== ":memory:") {
         db.exec("PRAGMA journal_mode = WAL;");
+        db.exec("PRAGMA busy_timeout = 5000;");
       }
 
       // Execute schema SQL

package/src/state/decorators.ts

@@ -4,10 +4,8 @@
  * Uses explicit type specification for all fields to ensure correct
  * serialization without relying on runtime type inference.
  *
- * Note: Bun's TC39 decorator implementation differs from the spec:
- * - Field decorator context is the field name as a string (not an object)
- * - Class decorator context is undefined (not an object)
- * - Field decorators run before class decorator (allows global accumulation)
+ * Note: Field decorators run before class decorator, which allows
+ * global accumulation of field definitions.
  *
  * @example
  * ```ts
@@ -33,6 +31,19 @@ function toSnakeCase(str: string): string {
   );
 }
 
+/**
+ * Extract property name from TC39 decorator context.
+ *
+ * TC39 field decorator context is a ClassFieldDecoratorContext object with a `name` property.
+ * This handles both the standard TC39 context object and legacy string contexts.
+ */
+function extractPropertyName(context: unknown): string {
+  if (typeof context === "object" && context !== null && "name" in context) {
+    return String((context as { name: unknown }).name);
+  }
+  return String(context);
+}
+
 /** Options for the @Field decorator. */
 export interface FieldOptions {
   /** Custom column name. Defaults to snake_case of property name. */
@@ -99,10 +110,8 @@ export function Persisted(table: string) {
  * @param options - Optional field configuration (column name override)
  */
 export function Field(type: FieldType, options?: FieldOptions) {
-  // Bun's TC39 decorator: context is the field name as string, not ClassFieldDecoratorContext
   return function (_target: undefined, context: unknown): void {
-    // In Bun, context is the field name as a string
-    const property = typeof context === "string" ? context : String(context);
+    const property = extractPropertyName(context);
     const column = options?.column ?? toSnakeCase(property);
 
     // Accumulate field definitions

package/src/state/loader.ts

@@ -538,8 +538,9 @@ export class StateLoader {
   /**
    * Execute a function within a database transaction.
    *
-   * Uses BEGIN IMMEDIATE to acquire a write lock immediately, preventing
-   * other writers. If the function throws, the transaction is rolled back.
+   * Uses BEGIN DEFERRED to defer lock acquisition until the first write.
+   * This allows concurrent readers and writers with WAL mode.
+   * If the function throws, the transaction is rolled back.
    * Otherwise, it is committed.
    *
    * @param fn - The function to execute within the transaction
@@ -559,7 +560,7 @@ export class StateLoader {
    * ```
    */
   async transaction<T>(fn: () => T | Promise<T>): Promise<T> {
-    this.db.exec("BEGIN IMMEDIATE");
+    this.db.exec("BEGIN DEFERRED");
     try {
       const result = await fn();
       this.db.exec("COMMIT");