@futdevpro/nts-dynamo 1.15.37 → 1.15.38

package/src/_modules/server/errors/errors.data-service.ts

@@ -12,7 +12,8 @@ import {
   DyFM_Paged,
   DyFM_Time,
   DyFM_SearchQuery,
-  DyFM_SearchResult
+  DyFM_SearchResult,
+  DyFM_Errors_FixAttempt
 } from '@futdevpro/fsm-dynamo';
 
 import { DyNTS_Errors_ControlService } from './errors.control-service';
@@ -142,6 +143,20 @@ export class DyNTS_Errors_DataService<
           };
         }
 
+        // Regression detect — a duplicate-ag-on egy NEW occurrence érkezett.
+        // Ha mar volt fix-attempt erre az error-ra, es a jelenlegi version
+        // ujabb vagy egyenlo mint a fix-attempt verzioja, akkor a hipotezis
+        // nem zarta le a problemat → flag-eljuk a dashboardnak.
+        if (
+          duplicateError.lastFixedInVersion &&
+          DyNTS_Errors_DataService.isVersionAtLeast(errorVersion, duplicateError.lastFixedInVersion)
+        ) {
+          (duplicateRecordUpdate as any).$set = {
+            ...((duplicateRecordUpdate as any).$set ?? {}),
+            regressedAfterFix: true,
+          };
+        }
+
         await this.updateData({
           filterBy: { _id: duplicateError._id } as DyFM_DBFilter<T_ErrorsRecord>,
           update: duplicateRecordUpdate,
@@ -452,6 +467,99 @@ export class DyNTS_Errors_DataService<
     }
   }
 
+  /**
+   * Record a "we think this is fixed" claim on an existing error.
+   *
+   * Append-only: every call pushes a new `DyFM_Errors_FixAttempt` to
+   * `fixAttempts[]` and updates the `last*` aliases for quick reads.
+   * Also clears `regressedAfterFix` so the dashboard treats the error as
+   * freshly-claimed-fixed; if a new occurrence arrives with a version
+   * `>= version`, `recordError` will flip the flag back on.
+   */
+  async recordFixAttempt(
+    errorId: string,
+    version: string,
+    hypothesis: string,
+    issuer: string
+  ): Promise<DyFM_Errors_FixAttempt> {
+    try {
+      if (issuer && !this.issuer) {
+        this.issuer = issuer;
+      }
+
+      const existing = await this.findData({ _id: errorId } as DyFM_DBFilter<T_ErrorsRecord>);
+      if (!existing) {
+        throw new DyFM_Error({
+          ...this.getDefaultErrorSettings('recordFixAttempt', new Error(`error not found: ${errorId}`)),
+          errorCode: `${DyNTS_global_settings.systemShortCodeName}|DyNTS-EDS-RFA0`,
+        });
+      }
+
+      const attempt: DyFM_Errors_FixAttempt = {
+        at: new Date(),
+        by: issuer,
+        version: version,
+        hypothesis: hypothesis,
+      };
+
+      const nextAttempts: DyFM_Errors_FixAttempt[] = [
+        ...((existing as any).fixAttempts ?? []),
+        attempt,
+      ];
+
+      await this.updateData({
+        filterBy: { _id: errorId } as DyFM_DBFilter<T_ErrorsRecord>,
+        update: {
+          $set: {
+            fixAttempts: nextAttempts,
+            lastFixedAt: attempt.at,
+            lastFixedBy: attempt.by,
+            lastFixedInVersion: attempt.version,
+            lastFixHypothesis: attempt.hypothesis,
+            regressedAfterFix: false,
+          },
+        } as DyNTS_DBUpdate<T_ErrorsRecord>,
+      });
+
+      return attempt;
+    } catch (error) {
+      throw new DyFM_Error({
+        ...this.getDefaultErrorSettings('recordFixAttempt', error),
+        errorCode: `${DyNTS_global_settings.systemShortCodeName}|DyNTS-EDS-RFA1`,
+      });
+    }
+  }
+
+  /**
+   * Loose numeric compare of two version strings of shape `MAJOR.MINOR.PATCH`
+   * (`01.15.117`). Pads missing segments with 0. Non-numeric segments are
+   * treated as 0 — version strings carrying prefixes like `SERVER-` from
+   * `recordError` are caller-stripped before this is invoked.
+   *
+   * Returns true when `a >= b`.
+   */
+  static isVersionAtLeast(a: string, b: string): boolean {
+    if (!a || !b) {
+      return false;
+    }
+    const strip = (v: string): string => v.replace(/^[^0-9]+/, '');
+    const parse = (v: string): number[] => strip(v).split('.').map(s => Number(s) || 0);
+    const aParts = parse(a);
+    const bParts = parse(b);
+    const len = Math.max(aParts.length, bParts.length);
+    for (let i = 0; i < len; i++) {
+      const av = aParts[i] ?? 0;
+      const bv = bParts[i] ?? 0;
+      if (av > bv) {
+        return true;
+      }
+      if (av < bv) {
+        return false;
+      }
+    }
+    return true;
+  }
+
   async getErrorsInRange(range: DyFM_RelativeDate, issuer: string): Promise<DyFM_Paged<T_ErrorsRecord>> {
     try {
       if (issuer && !this.issuer) {

package/src/_modules/server/index.ts

@@ -1,30 +1,30 @@
-
-//
-// Server module
-//
-// This module contains solutions for basic server endpoints such as:
-//
-// - Errors; 
-//     saving and retrieving errors that are occuring on the server
-//
-// - Server Status; 
-//     retrieving the status of the server
-//
-// - Server Status Snapshot; 
-//     saving and retrieving the status of the server at a specific time
-//
-
-
-// ERRORS
-export * from './errors/errors.controller';
-export * from './errors/errors.control-service';
-export * from './errors/errors.data-service';
-
-// SERVER-STATUS
-export * from './server-status/server-status-snapshot.data-service';
-export * from './server-status/server-status-snapshot.control-service';
-export * from './server-status/server-status.control-service';
-export * from './server-status/server-status.controller';
-
-
-
+
+//
+// Server module
+//
+// This module contains solutions for basic server endpoints such as:
+//
+// - Errors; 
+//     saving and retrieving errors that are occuring on the server
+//
+// - Server Status; 
+//     retrieving the status of the server
+//
+// - Server Status Snapshot; 
+//     saving and retrieving the status of the server at a specific time
+//
+
+
+// ERRORS
+export * from './errors/errors.controller';
+export * from './errors/errors.control-service';
+export * from './errors/errors.data-service';
+
+// SERVER-STATUS
+export * from './server-status/server-status-snapshot.data-service';
+export * from './server-status/server-status-snapshot.control-service';
+export * from './server-status/server-status.control-service';
+export * from './server-status/server-status.controller';
+
+
+

package/src/_modules/server/server-status/server-status-snapshot.control-service.spec.ts

@@ -1,70 +1,70 @@
-
-import { DyNTS_ServerStatusSnapshot_ControlService } from './server-status-snapshot.control-service';
-import { DyFM_ServerStatus } from '@futdevpro/fsm-dynamo';
-
-class TestServerStatus extends DyFM_ServerStatus {
-  constructor(set?: Partial<TestServerStatus>) {
-    super();
-    Object.assign(this, set);
-  }
-}
-
-class TestServerStatusSnapshotControlService extends DyNTS_ServerStatusSnapshot_ControlService<TestServerStatus> {
-  async saveSnapshot(snapshot: TestServerStatus, issuer: string): Promise<TestServerStatus> {
-    return snapshot;
-  }
-
-  static getInstance(): TestServerStatusSnapshotControlService {
-    return (TestServerStatusSnapshotControlService as any).getSingletonInstance();
-  }
-}
-
-describe('| DyNTS_ServerStatusSnapshot_ControlService', () => {
-  let service: TestServerStatusSnapshotControlService;
-
-  beforeEach(() => {
-    service = new TestServerStatusSnapshotControlService();
-  });
-
-  describe('| constructor', () => {
-    it('| should create instance', () => {
-      expect(service).toBeDefined();
-      expect(service).toBeInstanceOf(TestServerStatusSnapshotControlService);
-    });
-  });
-
-  describe('| saveSnapshot', () => {
-    it('| should save snapshot', async () => {
-      const snapshot: TestServerStatus = new TestServerStatus({
-        status: 'ready',
-        systemName: 'Test System',
-      });
-
-      const result = await service.saveSnapshot(snapshot, 'test-issuer');
-
-      expect(result).toBeDefined();
-      expect(result.status).toBe('ready');
-      expect(result.systemName).toBe('Test System');
-    });
-
-    it('| should handle snapshot with all properties', async () => {
-      const snapshot: TestServerStatus = new TestServerStatus({
-        status: 'ready',
-        systemName: 'Test System',
-        systemShortCode: 'TEST',
-        uptime: 1000,
-        memoryUsage: process.memoryUsage(),
-        cpuUsage: process.cpuUsage(),
-      });
-
-      const result = await service.saveSnapshot(snapshot, 'test-issuer');
-
-      expect(result).toBeDefined();
-      expect(result.status).toBe('ready');
-      expect(result.systemName).toBe('Test System');
-      expect(result.systemShortCode).toBe('TEST');
-      expect(result.uptime).toBe(1000);
-    });
-  });
-});
-
+
+import { DyNTS_ServerStatusSnapshot_ControlService } from './server-status-snapshot.control-service';
+import { DyFM_ServerStatus } from '@futdevpro/fsm-dynamo';
+
+class TestServerStatus extends DyFM_ServerStatus {
+  constructor(set?: Partial<TestServerStatus>) {
+    super();
+    Object.assign(this, set);
+  }
+}
+
+class TestServerStatusSnapshotControlService extends DyNTS_ServerStatusSnapshot_ControlService<TestServerStatus> {
+  async saveSnapshot(snapshot: TestServerStatus, issuer: string): Promise<TestServerStatus> {
+    return snapshot;
+  }
+
+  static getInstance(): TestServerStatusSnapshotControlService {
+    return (TestServerStatusSnapshotControlService as any).getSingletonInstance();
+  }
+}
+
+describe('| DyNTS_ServerStatusSnapshot_ControlService', () => {
+  let service: TestServerStatusSnapshotControlService;
+
+  beforeEach(() => {
+    service = new TestServerStatusSnapshotControlService();
+  });
+
+  describe('| constructor', () => {
+    it('| should create instance', () => {
+      expect(service).toBeDefined();
+      expect(service).toBeInstanceOf(TestServerStatusSnapshotControlService);
+    });
+  });
+
+  describe('| saveSnapshot', () => {
+    it('| should save snapshot', async () => {
+      const snapshot: TestServerStatus = new TestServerStatus({
+        status: 'ready',
+        systemName: 'Test System',
+      });
+
+      const result = await service.saveSnapshot(snapshot, 'test-issuer');
+
+      expect(result).toBeDefined();
+      expect(result.status).toBe('ready');
+      expect(result.systemName).toBe('Test System');
+    });
+
+    it('| should handle snapshot with all properties', async () => {
+      const snapshot: TestServerStatus = new TestServerStatus({
+        status: 'ready',
+        systemName: 'Test System',
+        systemShortCode: 'TEST',
+        uptime: 1000,
+        memoryUsage: process.memoryUsage(),
+        cpuUsage: process.cpuUsage(),
+      });
+
+      const result = await service.saveSnapshot(snapshot, 'test-issuer');
+
+      expect(result).toBeDefined();
+      expect(result.status).toBe('ready');
+      expect(result.systemName).toBe('Test System');
+      expect(result.systemShortCode).toBe('TEST');
+      expect(result.uptime).toBe(1000);
+    });
+  });
+});
+

package/src/_modules/server/server-status/server-status-snapshot.control-service.ts

@@ -1,17 +1,17 @@
-
-
-
-
-/* import { ServerStatus_ControlService } from './server-status.control-service'; */
-
-import { DyFM_Error, DyFM_ServerStatus } from '@futdevpro/fsm-dynamo';
-import { DyNTS_SingletonService } from '../../../_services/base/singleton.service';
-
-export abstract class DyNTS_ServerStatusSnapshot_ControlService<
-  T_ServerStatusSnapshot extends DyFM_ServerStatus
-> {
-  abstract saveSnapshot(
-    snapshot: T_ServerStatusSnapshot, 
-    issuer: string
-  ): Promise<T_ServerStatusSnapshot>;
-}
+
+
+
+
+/* import { ServerStatus_ControlService } from './server-status.control-service'; */
+
+import { DyFM_Error, DyFM_ServerStatus } from '@futdevpro/fsm-dynamo';
+import { DyNTS_SingletonService } from '../../../_services/base/singleton.service';
+
+export abstract class DyNTS_ServerStatusSnapshot_ControlService<
+  T_ServerStatusSnapshot extends DyFM_ServerStatus
+> {
+  abstract saveSnapshot(
+    snapshot: T_ServerStatusSnapshot, 
+    issuer: string
+  ): Promise<T_ServerStatusSnapshot>;
+}

package/src/_modules/server/server-status/server-status-snapshot.data-service.spec.ts

@@ -1,77 +1,77 @@
-
-import { DyNTS_ServerStatusSnapshot_DataService } from './server-status-snapshot.data-service';
-import { DyFM_ServerStatus, DyFM_DataModel_Params } from '@futdevpro/fsm-dynamo';
-import { DyNTS_GlobalService } from '../../../_services/core/global.service';
-import { DyNTS_DBService } from '../../../_services/base/db.service';
-
-class TestServerStatus extends DyFM_ServerStatus {
-  testProperty?: string;
-}
-
-const testDataParams = new DyFM_DataModel_Params<TestServerStatus>({
-  dataName: 'test_server_status',
-  properties: {},
-});
-
-describe('| DyNTS_ServerStatusSnapshot_DataService', () => {
-  let service: DyNTS_ServerStatusSnapshot_DataService<TestServerStatus>;
-  let mockDBService: jasmine.SpyObj<DyNTS_DBService<TestServerStatus>>;
-
-  beforeEach(() => {
-    mockDBService = jasmine.createSpyObj('DyNTS_DBService', [
-      'getAll',
-      'getDataById',
-      'createData',
-      'modifyData',
-    ]);
-
-    spyOn(DyNTS_GlobalService, 'getDBService').and.returnValue(mockDBService);
-
-    const testData = new TestServerStatus({
-      status: 'ready',
-      systemName: 'test-system',
-    });
-
-    service = new DyNTS_ServerStatusSnapshot_DataService<TestServerStatus>(
-      testData,
-      testDataParams,
-      'test-issuer'
-    );
-  });
-
-  it('| should create service instance', () => {
-    expect(service).toBeDefined();
-    expect(service.data).toBeDefined();
-  });
-
-  describe('| saveSnapshot', () => {
-    it('| should save snapshot', async () => {
-      const snapshot = new TestServerStatus({
-        status: 'ready',
-        systemName: 'test-system',
-        uptime: 1000,
-      });
-      const savedSnapshot = { ...snapshot, _id: 'snapshot-id' };
-      spyOn(service, 'saveData').and.returnValue(Promise.resolve(savedSnapshot));
-
-      const result = await service.saveSnapshot(snapshot, 'test-issuer');
-
-      expect(result).toBe(savedSnapshot);
-      expect(service.issuer).toBe('test-issuer');
-      expect(service.saveData).toHaveBeenCalledWith(snapshot);
-    });
-
-    it('| should throw error when save fails', async () => {
-      const snapshot = new TestServerStatus({
-        status: 'ready',
-        systemName: 'test-system',
-      });
-      spyOn(service, 'saveData').and.returnValue(Promise.reject(new Error('Save failed')));
-
-      await expectAsync(
-        service.saveSnapshot(snapshot, 'test-issuer')
-      ).toBeRejected();
-    });
-  });
-});
-
+
+import { DyNTS_ServerStatusSnapshot_DataService } from './server-status-snapshot.data-service';
+import { DyFM_ServerStatus, DyFM_DataModel_Params } from '@futdevpro/fsm-dynamo';
+import { DyNTS_GlobalService } from '../../../_services/core/global.service';
+import { DyNTS_DBService } from '../../../_services/base/db.service';
+
+class TestServerStatus extends DyFM_ServerStatus {
+  testProperty?: string;
+}
+
+const testDataParams = new DyFM_DataModel_Params<TestServerStatus>({
+  dataName: 'test_server_status',
+  properties: {},
+});
+
+describe('| DyNTS_ServerStatusSnapshot_DataService', () => {
+  let service: DyNTS_ServerStatusSnapshot_DataService<TestServerStatus>;
+  let mockDBService: jasmine.SpyObj<DyNTS_DBService<TestServerStatus>>;
+
+  beforeEach(() => {
+    mockDBService = jasmine.createSpyObj('DyNTS_DBService', [
+      'getAll',
+      'getDataById',
+      'createData',
+      'modifyData',
+    ]);
+
+    spyOn(DyNTS_GlobalService, 'getDBService').and.returnValue(mockDBService);
+
+    const testData = new TestServerStatus({
+      status: 'ready',
+      systemName: 'test-system',
+    });
+
+    service = new DyNTS_ServerStatusSnapshot_DataService<TestServerStatus>(
+      testData,
+      testDataParams,
+      'test-issuer'
+    );
+  });
+
+  it('| should create service instance', () => {
+    expect(service).toBeDefined();
+    expect(service.data).toBeDefined();
+  });
+
+  describe('| saveSnapshot', () => {
+    it('| should save snapshot', async () => {
+      const snapshot = new TestServerStatus({
+        status: 'ready',
+        systemName: 'test-system',
+        uptime: 1000,
+      });
+      const savedSnapshot = { ...snapshot, _id: 'snapshot-id' };
+      spyOn(service, 'saveData').and.returnValue(Promise.resolve(savedSnapshot));
+
+      const result = await service.saveSnapshot(snapshot, 'test-issuer');
+
+      expect(result).toBe(savedSnapshot);
+      expect(service.issuer).toBe('test-issuer');
+      expect(service.saveData).toHaveBeenCalledWith(snapshot);
+    });
+
+    it('| should throw error when save fails', async () => {
+      const snapshot = new TestServerStatus({
+        status: 'ready',
+        systemName: 'test-system',
+      });
+      spyOn(service, 'saveData').and.returnValue(Promise.reject(new Error('Save failed')));
+
+      await expectAsync(
+        service.saveSnapshot(snapshot, 'test-issuer')
+      ).toBeRejected();
+    });
+  });
+});
+

package/src/_modules/server/server-status/server-status-snapshot.data-service.ts

@@ -1,37 +1,37 @@
-
-
-
-
-/* import { ServerStatus_ControlService } from './server-status.control-service'; */
-
-import { DyFM_Error, DyFM_ServerStatus } from '@futdevpro/fsm-dynamo';
-import { DyNTS_DataService } from '../../../_services/base/data.service';
-import { DyNTS_ServerStatusSnapshot_ControlService } from './server-status-snapshot.control-service';
-import { DyNTS_global_settings } from '../../../_collections/global-settings.const';
-
-export class DyNTS_ServerStatusSnapshot_DataService<
-  T_ServerStatusSnapshot extends DyFM_ServerStatus
-> extends DyNTS_DataService<
-  T_ServerStatusSnapshot
-> implements DyNTS_ServerStatusSnapshot_ControlService<
-  T_ServerStatusSnapshot
-> {
-
-  async saveSnapshot(
-    snapshot: T_ServerStatusSnapshot, 
-    issuer: string
-  ): Promise<T_ServerStatusSnapshot> {
-    try {
-      this.issuer = issuer;
-
-      return await this.saveData(snapshot);
-    } catch (error) {
-      throw new DyFM_Error({
-        ...this.getDefaultErrorSettings('saveSnapshot', error),
-
-        errorCode: `${DyNTS_global_settings.systemShortCodeName}|DyNTS-SSSDS-SS0`,
-      });
-    }
-  }
-
-}
+
+
+
+
+/* import { ServerStatus_ControlService } from './server-status.control-service'; */
+
+import { DyFM_Error, DyFM_ServerStatus } from '@futdevpro/fsm-dynamo';
+import { DyNTS_DataService } from '../../../_services/base/data.service';
+import { DyNTS_ServerStatusSnapshot_ControlService } from './server-status-snapshot.control-service';
+import { DyNTS_global_settings } from '../../../_collections/global-settings.const';
+
+export class DyNTS_ServerStatusSnapshot_DataService<
+  T_ServerStatusSnapshot extends DyFM_ServerStatus
+> extends DyNTS_DataService<
+  T_ServerStatusSnapshot
+> implements DyNTS_ServerStatusSnapshot_ControlService<
+  T_ServerStatusSnapshot
+> {
+
+  async saveSnapshot(
+    snapshot: T_ServerStatusSnapshot, 
+    issuer: string
+  ): Promise<T_ServerStatusSnapshot> {
+    try {
+      this.issuer = issuer;
+
+      return await this.saveData(snapshot);
+    } catch (error) {
+      throw new DyFM_Error({
+        ...this.getDefaultErrorSettings('saveSnapshot', error),
+
+        errorCode: `${DyNTS_global_settings.systemShortCodeName}|DyNTS-SSSDS-SS0`,
+      });
+    }
+  }
+
+}