feat: add ids in state machines

LIT-Protocol · Nov 27, 2024 · 40bbafe · 40bbafe
1 parent 327964e
commit 40bbafe
Show file tree

Hide file tree

Showing 5 changed files with 20 additions and 8 deletions.
diff --git a/packages/automation/src/lib/listeners/fetch.spec.ts b/packages/automation/src/lib/listeners/fetch.spec.ts
@@ -28,7 +28,9 @@ describe('FetchListener', () => {
 
   it('should fetch data and emit the correct value', async () => {
     let callbackCalled: () => void;
-    const callbackPromise = new Promise<void>(resolve => callbackCalled = resolve);
+    const callbackPromise = new Promise<void>(
+      (resolve) => (callbackCalled = resolve)
+    );
 
     const callback = jest.fn(async () => {
       callbackCalled();

diff --git a/packages/automation/src/lib/state-machine.spec.ts b/packages/automation/src/lib/state-machine.spec.ts
@@ -38,6 +38,13 @@ describe('StateMachine', () => {
     });
   });
 
+  it('should generate a unique id for each state machine instance', () => {
+    const anotherStateMachine = new StateMachine();
+    expect(stateMachine.id).toBeDefined();
+    expect(anotherStateMachine.id).toBeDefined();
+    expect(stateMachine.id).not.toEqual(anotherStateMachine.id);
+  });
+
   it('should add states and transitions correctly', () => {
     stateMachine.addTransition({
       fromState: 'A',

diff --git a/packages/automation/src/lib/state-machine.ts b/packages/automation/src/lib/state-machine.ts
@@ -18,14 +18,16 @@ type MachineStatus = 'running' | 'stopped';
  * A StateMachine class that manages states and transitions between them.
  */
 export class StateMachine {
-  private status: MachineStatus = 'stopped';
+  public id: string;
+  public status: MachineStatus = 'stopped';
   private states = new Map<string, State>();
   private transitions = new Map<string, Map<string, Transition>>();
   private currentState?: State;
   private onStopCallback?: () => Promise<void>;
   private debug = false;
 
   constructor(params: BaseStateMachineParams = {}) {
+    this.id = this.generateId();
     this.debug = params.debug ?? false;
   }
 
@@ -173,4 +175,8 @@ export class StateMachine {
       throw new Error(`Could not enter state ${stateKey}`);
     }
   }
+
+  private generateId(): string {
+    return Math.random().toString(36).substring(2);
+  }
 }
diff --git a/packages/automation/tsconfig.json b/packages/automation/tsconfig.json
@@ -1,16 +1,13 @@
 {
   "extends": "../../tsconfig.base.json",
   "compilerOptions": {
-    "module": "system",
+    "module": "commonjs",
     "forceConsistentCasingInFileNames": true,
     "strict": true,
     "noImplicitOverride": true,
     "noPropertyAccessFromIndexSignature": true,
     "noImplicitReturns": true,
-    "noFallthroughCasesInSwitch": true,
-    "allowJs": true,
-    "checkJs": false,
-    "resolveJsonModule": false
+    "noFallthroughCasesInSwitch": true
   },
   "files": [],
   "include": ["global.d.ts"],

diff --git a/packages/wrapped-keys-lit-actions/tsconfig.json b/packages/wrapped-keys-lit-actions/tsconfig.json
@@ -1,7 +1,7 @@
 {
   "extends": "../../tsconfig.base.json",
   "compilerOptions": {
-    "module": "system",
+    "module": "commonjs",
     "forceConsistentCasingInFileNames": true,
     "strict": true,
     "noImplicitOverride": true,