@webitel/api-services 26.8.81 → 26.8.82

package/package.json

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

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

@@ -4,6 +4,8 @@ const searchMemberInQueue = vi.fn();
 const resetMembersCount = vi.fn();
 const resetMembers = vi.fn();
 const searchMemberCommunication = vi.fn();
+const createMember = vi.fn();
+const updateMember = vi.fn();
 
 vi.mock('../../../../gen-wire', () => ({
 	getMemberService: () => ({
@@ -11,6 +13,8 @@ vi.mock('../../../../gen-wire', () => ({
 		resetMembersCount,
 		resetMembers,
 		searchMemberCommunication,
+		createMember,
+		updateMember,
 	}),
 }));
 
@@ -240,3 +244,62 @@ describe('QueueMembersAPI.getCommunications', () => {
 		});
 	});
 });
+
+/**
+ * [Claude] The card store keeps the add/update response as the new item, so it
+ * has to come back in the same shape `get` returns — variables as pairs.
+ * [WTEL-10587](https://webitel.atlassian.net/browse/WTEL-10587)
+ */
+describe('QueueMembersAPI add/update response', () => {
+	const wireMember = {
+		id: '5',
+		variables: {
+			someKey: 'value',
+		},
+	};
+
+	beforeEach(() => {
+		createMember.mockReset().mockResolvedValue({
+			data: wireMember,
+		});
+		updateMember.mockReset().mockResolvedValue({
+			data: wireMember,
+		});
+	});
+
+	const itemInstance = {
+		variables: [
+			{
+				key: 'someKey',
+				value: 'value',
+			},
+		],
+	};
+
+	it('add returns variables as key/value pairs', async () => {
+		const item = await QueueMembersAPI.add({
+			parentId: '7',
+			itemInstance,
+		});
+		expect(item.variables).toEqual([
+			{
+				key: 'someKey',
+				value: 'value',
+			},
+		]);
+	});
+
+	it('update returns variables as key/value pairs', async () => {
+		const item = await QueueMembersAPI.update({
+			parentId: '7',
+			itemId: '5',
+			itemInstance,
+		});
+		expect(item.variables).toEqual([
+			{
+				key: 'someKey',
+				value: 'value',
+			},
+		]);
+	});
+});

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

@@ -85,6 +85,30 @@ const preRequestHandler = (item: ApiParams) => {
 	};
 };
 
+/**
+ * [Claude] Back to the card's shape: `variables` as key/value pairs and
+ * communications filled with defaults. Applied to add/update responses too —
+ * the card store takes those as the new item, so a raw map there renders the
+ * variables tab empty after save.
+ * [WTEL-10587](https://webitel.atlassian.net/browse/WTEL-10587)
+ */
+const itemResponseHandler = (response: ApiParams) => {
+	const copy = deepCopy(response);
+	let variables: ApiParams[] = [];
+	if (copy.variables) {
+		variables = Object.keys(copy.variables).map((key) => ({
+			key,
+			value: copy.variables[key],
+		}));
+	}
+	const communications = mapDefaultCommunications(copy);
+	return {
+		...response,
+		variables,
+		communications,
+	};
+};
+
 const getMembersList = async (params: ApiParams) => {
 	const listHandler = (items: ApiParams[]) => {
 		const copy = deepCopy(items);
@@ -305,23 +329,6 @@ const exportMembers = async ({
 };
 
 const getMember = async ({ parentId, itemId: id }: NestedGetItemParams) => {
-	const responseHandler = (response: ApiParams) => {
-		const copy = deepCopy(response);
-		let variables: ApiParams[] = [];
-		if (copy.variables) {
-			variables = Object.keys(copy.variables).map((key) => ({
-				key,
-				value: copy.variables[key],
-			}));
-		}
-		const communications = mapDefaultCommunications(copy);
-		return {
-			...response,
-			variables,
-			communications,
-		};
-	};
-
 	try {
 		const response = await getMemberService().readMember(
 			String(parentId),
@@ -329,7 +336,7 @@ const getMember = async ({ parentId, itemId: id }: NestedGetItemParams) => {
 		);
 		return applyTransform(response.data, [
 			snakeToCamel(doNotConvertKeys),
-			responseHandler,
+			itemResponseHandler,
 		]);
 	} catch (err) {
 		throw applyTransform(err, [
@@ -351,6 +358,7 @@ const addMember = async ({ parentId, itemInstance }: NestedAddItemParams) => {
 		);
 		return applyTransform(response.data, [
 			snakeToCamel(doNotConvertKeys),
+			itemResponseHandler,
 		]);
 	} catch (err) {
 		throw applyTransform(err, [
@@ -377,6 +385,7 @@ const updateMember = async ({
 		);
 		return applyTransform(response.data, [
 			snakeToCamel(doNotConvertKeys),
+			itemResponseHandler,
 		]);
 	} catch (err) {
 		throw applyTransform(err, [