@n8n/frontend-module-sdk 0.4.4 → 0.5.0

package/package.json

@@ -1,7 +1,7 @@
 {
   "type": "module",
   "name": "@n8n/frontend-module-sdk",
-  "version": "0.4.4",
+  "version": "0.5.0",
   "main": "src/index.ts",
   "import": "src/index.ts",
   "exports": {
@@ -10,8 +10,8 @@
   "dependencies": {
     "vue": "^3.5.13",
     "vue-router": "^4.5.0",
-    "@n8n/api-types": "1.34.2",
-    "@n8n/design-system": "2.33.4"
+    "@n8n/api-types": "1.35.0",
+    "@n8n/design-system": "2.34.0"
   },
   "devDependencies": {
     "typescript": "6.0.2",
@@ -19,9 +19,9 @@
     "vite": "^8.0.2",
     "vitest": "^4.1.9",
     "vue-tsc": "^2.2.8",
-    "@n8n/eslint-config": "0.0.1",
     "@n8n/typescript-config": "1.9.0",
-    "@n8n/vitest-config": "1.19.0"
+    "@n8n/vitest-config": "1.20.0",
+    "@n8n/eslint-config": "0.0.1"
   },
   "license": "SEE LICENSE IN LICENSE.md",
   "homepage": "https://n8n.io",

package/src/__tests__/setup.ts

@@ -1,4 +1 @@
-// Test setup entry required by the shared frontend vitest config
-// (`@n8n/vitest-config/frontend` points `setupFiles` here). The registry suites
-// need no global setup, so this file intentionally stays empty.
-export {};
+import '@n8n/vitest-config/setup/frontend';

package/src/registries/modalRegistry.test.ts

@@ -28,9 +28,7 @@ describe('modalRegistry', () => {
 	};
 
 	beforeEach(() => {
-		// Clear all modals before each test
-		const keys = modalRegistry.getKeys();
-		keys.forEach((key) => modalRegistry.unregister(key));
+		modalRegistry.clear();
 	});
 
 	describe('register', () => {
@@ -112,6 +110,53 @@ describe('modalRegistry', () => {
 			expect(listener).toHaveBeenCalledWith(expect.any(Map));
 			expect(listener).toHaveBeenCalledTimes(1);
 		});
+
+		it('should allow a key to be registered again after it was unregistered', () => {
+			modalRegistry.register(mockModal1);
+			modalRegistry.unregister('test-modal-1');
+
+			const replacement: ModalDefinition = {
+				key: 'test-modal-1',
+				component: mockComponent2,
+				initialState: { open: true },
+			};
+			modalRegistry.register(replacement);
+
+			expect(modalRegistry.get('test-modal-1')).toEqual(replacement);
+		});
+	});
+
+	describe('clear', () => {
+		it('should remove all registered modals', () => {
+			modalRegistry.register(mockModal1);
+			modalRegistry.register(mockModal2);
+
+			modalRegistry.clear();
+
+			expect(modalRegistry.getKeys()).toEqual([]);
+			expect(modalRegistry.has('test-modal-1')).toBe(false);
+			expect(modalRegistry.getAll().size).toBe(0);
+		});
+
+		it('should notify listeners so renderers drop the cleared modals', () => {
+			modalRegistry.register(mockModal1);
+			const listener = vi.fn<(modals: Map<string, ModalDefinition>) => void>();
+			modalRegistry.subscribe(listener);
+
+			modalRegistry.clear();
+
+			expect(listener).toHaveBeenCalledTimes(1);
+			expect(listener.mock.calls[0][0].size).toBe(0);
+		});
+
+		it('should allow registration again after clearing', () => {
+			modalRegistry.register(mockModal1);
+			modalRegistry.clear();
+
+			modalRegistry.register(mockModal1);
+
+			expect(modalRegistry.get('test-modal-1')).toEqual(mockModal1);
+		});
 	});
 
 	describe('get', () => {

package/src/registries/modalRegistry.ts

@@ -45,3 +45,11 @@ export function subscribe(listener: (modals: Map<string, ModalDefinition>) => vo
 		listeners.delete(listener);
 	};
 }
+
+/**
+ * Remove all registered modals. Primarily for test isolation.
+ */
+export function clear(): void {
+	modals.clear();
+	notifyListeners();
+}