@geekmidas/audit 0.0.5 → 0.0.7

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@geekmidas/audit",
-  "version": "0.0.5",
+  "version": "0.0.7",
   "type": "module",
   "exports": {
     ".": {

package/src/__tests__/KyselyAuditStorage.spec.ts

@@ -221,7 +221,7 @@ describe('KyselyAuditStorage', () => {
       expect(calledValues[0].id).toBeUndefined();
     });
 
-    it('should use provided ID when autoId is true', async () => {
+    it('should omit ID even if provided when autoId is true', async () => {
       const storageAutoId = new KyselyAuditStorage({
         db: mockDb.db as any,
         tableName: 'audit_logs',
@@ -230,7 +230,7 @@ describe('KyselyAuditStorage', () => {
 
       const records: AuditRecord[] = [
         {
-          id: 'explicit-id',
+          id: 'explicit-id', // This should be ignored when autoId is true
           type: 'user.created',
           operation: 'CUSTOM',
           timestamp: new Date(),
@@ -239,11 +239,9 @@ describe('KyselyAuditStorage', () => {
 
       await storageAutoId.write(records);
 
-      expect(mockDb.insertBuilder.values).toHaveBeenCalledWith([
-        expect.objectContaining({
-          id: 'explicit-id',
-        }),
-      ]);
+      // When autoId is true, ID should always be omitted (let database generate)
+      const calledValues = mockDb.insertBuilder.values.mock.calls[0][0];
+      expect(calledValues[0].id).toBeUndefined();
     });
   });
 

package/src/kysely.ts

@@ -327,9 +327,9 @@ export class KyselyAuditStorage<DB> implements AuditStorage {
   }
 
   private toRow(record: AuditRecord): AuditLogTable {
-    // If autoId is true, let database generate ID (omit if not provided)
-    // If autoId is false (default), generate with nanoid if not provided
-    const id = record.id || (this.autoId ? undefined : nanoid());
+    // If autoId is true, let database generate ID (ignore record.id)
+    // If autoId is false (default), use record.id or generate with nanoid
+    const id = this.autoId ? undefined : record.id || nanoid();
 
     return {
       ...(id && { id }),

package/src/storage.ts

@@ -1,4 +1,4 @@
-import type { AuditableAction, AuditRecord } from './types';
+import type { AuditRecord, AuditableAction } from './types';
 
 /**
  * Options for querying audit records.