260318:1635 Fix UUID #10

backend/src/modules/document-numbering/document-numbering.service.spec.ts

@@ -144,7 +144,7 @@ describe('DocumentNumberingService', () => {
     it('voidAndReplace should verify audit log exists', async () => {
       const auditRepo = module.get(getRepositoryToken(DocumentNumberAudit));
       (auditRepo.findOne as jest.Mock).mockResolvedValue({
-        generatedNumber: 'DOC-001',
+        documentNumber: 'DOC-001',
         counterKey: JSON.stringify({ projectId: 1, correspondenceTypeId: 1 }),
         templateUsed: 'test',
       });
@@ -162,7 +162,7 @@ describe('DocumentNumberingService', () => {
     it('cancelNumber should log cancellation', async () => {
       const auditRepo = module.get(getRepositoryToken(DocumentNumberAudit));
       (auditRepo.findOne as jest.Mock).mockResolvedValue({
-        generatedNumber: 'DOC-002',
+        documentNumber: 'DOC-002',
         counterKey: {},
       });
       (auditRepo.save as jest.Mock).mockResolvedValue({ id: 3 });

backend/src/modules/document-numbering/entities/document-number-audit.entity.ts

@@ -12,7 +12,7 @@ import {
 @Index(['documentId'])
 @Index(['status'])
 @Index(['operation'])
-@Index(['generatedNumber'])
+@Index(['documentNumber'])
 @Index(['reservationToken'])
 export class DocumentNumberAudit {
   @PrimaryGeneratedColumn()
@@ -21,8 +21,8 @@ export class DocumentNumberAudit {
   @Column({ name: 'document_id', nullable: true })
   documentId?: number;
 
-  @Column({ name: 'generated_number', length: 100 })
-  generatedNumber!: string;
+  @Column({ name: 'document_number', length: 100 })
+  documentNumber!: string;
 
   @Column({ name: 'counter_key', type: 'json' })
   counterKey!: any;

backend/src/modules/document-numbering/services/document-numbering.service.ts

@@ -146,7 +146,7 @@ export class DocumentNumberingService {
       const sequence = await this.counterService.incrementCounter(key);
 
       // 4. Format Number
-      const generatedNumber = await this.formatService.format({
+      const documentNumber = await this.formatService.format({
         projectId: ctx.projectId,
         correspondenceTypeId: ctx.typeId,
         subTypeId: ctx.subTypeId,
@@ -161,7 +161,7 @@ export class DocumentNumberingService {
 
       // 5. Audit Log
       const audit = await this.logAudit({
-        generatedNumber,
+        documentNumber,
         counterKey: JSON.stringify(key),
         templateUsed: 'DELEGATED_TO_FORMAT_SERVICE',
         context: ctx,
@@ -175,7 +175,7 @@ export class DocumentNumberingService {
         type_id: String(ctx.typeId),
       });
 
-      return { number: generatedNumber, auditId: audit.id };
+      return { number: documentNumber, auditId: audit.id };
     } catch (error: any) {
       await this.logError(error, ctx, 'GENERATE');
       throw error;
@@ -322,7 +322,7 @@ export class DocumentNumberingService {
   }) {
     // 1. Find the audit log for this number to get context
     const lastAudit = await this.auditRepo.findOne({
-      where: { generatedNumber: dto.documentNumber },
+      where: { documentNumber: dto.documentNumber },
       order: { createdAt: 'DESC' },
     });
 
@@ -334,7 +334,7 @@ export class DocumentNumberingService {
       );
       // Create a void audit anyway if possible?
       await this.logAudit({
-        generatedNumber: dto.documentNumber,
+        documentNumber: dto.documentNumber,
         counterKey: {}, // Unknown
         templateUsed: 'VOID_UNKNOWN',
         context: { userId: 0, ipAddress: '0.0.0.0' }, // System
@@ -349,7 +349,7 @@ export class DocumentNumberingService {
 
     // 2. Log VOID
     await this.logAudit({
-      generatedNumber: dto.documentNumber,
+      documentNumber: dto.documentNumber,
       counterKey: lastAudit.counterKey,
       templateUsed: lastAudit.templateUsed,
       context: { userId: 0, ipAddress: '0.0.0.0' }, // TODO: Pass userId from controller
@@ -409,14 +409,14 @@ export class DocumentNumberingService {
   }) {
     // Similar to VOID but status CANCELLED
     const lastAudit = await this.auditRepo.findOne({
-      where: { generatedNumber: dto.documentNumber },
+      where: { documentNumber: dto.documentNumber },
       order: { createdAt: 'DESC' },
     });
 
     const contextKey = lastAudit?.counterKey;
 
     await this.logAudit({
-      generatedNumber: dto.documentNumber,
+      documentNumber: dto.documentNumber,
       counterKey: contextKey || {},
       templateUsed: lastAudit?.templateUsed || 'CANCEL',
       context: {

backend/src/modules/document-numbering/services/manual-override.service.spec.ts

@@ -56,7 +56,7 @@ describe('ManualOverrideService', () => {
     expect(counterService.forceUpdateCounter).toHaveBeenCalledWith(dto, 999);
     expect(auditService.log).toHaveBeenCalledWith(
       expect.objectContaining({
-        generatedNumber: 'OVERRIDE-TO-999',
+        documentNumber: 'OVERRIDE-TO-999',
         operation: 'MANUAL_OVERRIDE',
         status: 'MANUAL',
         userId: userId,

backend/src/modules/document-numbering/services/manual-override.service.ts

@@ -23,7 +23,7 @@ export class ManualOverrideService {
     // 2. Log Audit
     await this.auditService.log({
       documentId: undefined, // No specific document
-      generatedNumber: `OVERRIDE-TO-${dto.newLastNumber}`,
+      documentNumber: `OVERRIDE-TO-${dto.newLastNumber}`,
       operation: 'MANUAL_OVERRIDE',
       status: 'MANUAL',
       counterKey: dto, // CounterKeyDto part of ManualOverrideDto