@webitel/api-services 26.8.7 → 26.8.8

package/package.json

@@ -1,6 +1,6 @@
 {
 	"name": "@webitel/api-services",
-	"version": "26.8.7",
+	"version": "26.8.8",
 	"private": false,
 	"repository": {
 		"type": "git",

package/src/api/clients/queues/__tests__/queueHooksSkillsEnabled.spec.ts

@@ -0,0 +1,84 @@
+import { beforeEach, describe, expect, it, vi } from 'vitest';
+
+const readQueueHook = vi.fn();
+const readQueueSkill = vi.fn();
+
+vi.mock('../../../../gen-wire', () => ({
+	getQueueHookService: () => ({
+		readQueueHook,
+	}),
+	getQueueSkillService: () => ({
+		readQueueSkill,
+	}),
+}));
+
+const { QueueHooksAPI } = await import('../queueHooks');
+const { QueueSkillsAPI } = await import('../queueSkills');
+
+/**
+ * Protobuf omits `enabled` when it is false, so a disabled record reads back
+ * without the field. The card store seeds anything missing from the schema,
+ * where `enabled` defaults to `true` for newly added records — so `get` has to
+ * spell the false out, or an untouched save would re-enable the record.
+ */
+describe('reading a disabled queue hook / skill', () => {
+	beforeEach(() => {
+		readQueueHook.mockReset();
+		readQueueSkill.mockReset();
+	});
+
+	it('spells out enabled: false for a hook that omits it', async () => {
+		readQueueHook.mockResolvedValue({
+			data: {
+				id: 40,
+				event: 'bridged',
+			},
+		});
+
+		await expect(
+			QueueHooksAPI.get({
+				parentId: '208',
+				itemId: '40',
+			}),
+		).resolves.toMatchObject({
+			enabled: false,
+		});
+	});
+
+	it('spells out enabled: false for a skill that omits it', async () => {
+		readQueueSkill.mockResolvedValue({
+			data: {
+				id: 7,
+				lvl: 10,
+			},
+		});
+
+		await expect(
+			QueueSkillsAPI.get({
+				parentId: '208',
+				itemId: '7',
+			}),
+		).resolves.toMatchObject({
+			enabled: false,
+		});
+	});
+
+	it('keeps enabled: true as sent', async () => {
+		readQueueHook.mockResolvedValue({
+			data: {
+				id: 41,
+				event: 'missed',
+				enabled: true,
+			},
+		});
+
+		await expect(
+			QueueHooksAPI.get({
+				parentId: '208',
+				itemId: '41',
+			}),
+		).resolves.toMatchObject({
+			enabled: true,
+		});
+	});
+});

package/src/api/clients/queues/queueHooks.ts

@@ -77,6 +77,9 @@ const getQueueHook = async ({ parentId, itemId: id }: NestedGetItemParams) => {
 		);
 		return applyTransform(response.data, [
 			snakeToCamel(),
+			merge({
+				enabled: false,
+			}),
 		]);
 	} catch (err) {
 		throw applyTransform(err, [

package/src/api/clients/queues/queueSkills.ts

@@ -76,6 +76,9 @@ const getQueueSkill = async ({ parentId, itemId: id }: NestedGetItemParams) => {
 		);
 		return applyTransform(response.data, [
 			snakeToCamel(),
+			merge({
+				enabled: false,
+			}),
 		]);
 	} catch (err) {
 		throw applyTransform(err, [

package/src/validations/queueHook/__tests__/queueHook.validations.spec.ts

@@ -0,0 +1,31 @@
+import { getDefaultsFromZodSchema } from '@webitel/api-services/gen/utils';
+import { describe, expect, it } from 'vitest';
+
+import { queueHookSchema } from '../queueHook.validations';
+
+describe('queueHookSchema', () => {
+	/**
+	 * The card store seeds a new item from the schema defaults, so a hook added
+	 * from the queue card must come out enabled — as it did before the pinia
+	 * migration replaced the vuex `resettableItemState`.
+	 */
+	it('defaults a new hook to enabled', () => {
+		expect(getDefaultsFromZodSchema(queueHookSchema, {})).toMatchObject({
+			enabled: true,
+		});
+	});
+
+	it('keeps a disabled hook disabled', () => {
+		const result = queueHookSchema.safeParse({
+			event: 'joined',
+			schema: {
+				id: '1',
+				name: 'flow',
+			},
+			enabled: false,
+		});
+
+		expect(result.success).toBe(true);
+		expect(result.data?.enabled).toBe(false);
+	});
+});

package/src/validations/queueHook/queueHook.validations.ts

@@ -6,5 +6,5 @@ export const queueHookSchema = z.object({
 	event: z.string().min(1),
 	schema: filledLookupSchema,
 	properties: z.array(z.string()).optional(),
-	enabled: z.boolean().optional(),
+	enabled: z.boolean().default(true),
 });

package/src/validations/queueSkill/__tests__/queueSkill.validations.spec.ts

@@ -0,0 +1,27 @@
+import { getDefaultsFromZodSchema } from '@webitel/api-services/gen/utils';
+import { describe, expect, it } from 'vitest';
+
+import { queueSkillSchema } from '../queueSkill.validations';
+
+describe('queueSkillSchema', () => {
+	/** see the hook schema spec — same pre-migration default */
+	it('defaults a new skill to enabled', () => {
+		expect(getDefaultsFromZodSchema(queueSkillSchema, {})).toMatchObject({
+			enabled: true,
+		});
+	});
+
+	it('keeps a disabled skill disabled', () => {
+		const result = queueSkillSchema.safeParse({
+			skill: {
+				id: '1',
+				name: 'skill',
+			},
+			lvl: 1,
+			enabled: false,
+		});
+
+		expect(result.success).toBe(true);
+		expect(result.data?.enabled).toBe(false);
+	});
+});

package/src/validations/queueSkill/queueSkill.validations.ts

@@ -11,7 +11,7 @@ export const queueSkillSchemaBase = z.object({
 	minCapacity: z.number().min(0).max(100).optional(),
 	maxCapacity: z.number().min(0).max(100).optional(),
 	buckets: z.array(flexibleLookupSchema).optional(),
-	enabled: z.boolean().optional(),
+	enabled: z.boolean().default(true),
 });
 
 /**