@webitel/ui-datalist 26.8.29 → 26.10.2

package/package.json

@@ -1,6 +1,6 @@
 {
 	"name": "@webitel/ui-datalist",
-	"version": "26.8.29",
+	"version": "26.10.2",
 	"description": "Toolkit for building data lists in webitel ui system",
 	"scripts": {
 		"build:types": "vue-tsc -b tsconfig.build.json",
@@ -39,9 +39,9 @@
 		"@vuelidate/core": "^2.0.3",
 		"@vuelidate/validators": "^2.0.4",
 		"@vueuse/core": "^14.0",
-		"@webitel/api-services": "~26.8",
-		"@webitel/styleguide": "~26.8",
-		"@webitel/ui-sdk": "~26.8",
+		"@webitel/api-services": "~26.10",
+		"@webitel/styleguide": "~26.10",
+		"@webitel/ui-sdk": "~26.10",
 		"axios": "^1.15.2",
 		"date-fns": "^4",
 		"lodash-es": "^4.17.21",

package/src/modules/card/composables/__tests__/useCardValidation.spec.ts

@@ -6,6 +6,7 @@ import {
 	caseCloseFieldsChecks,
 	caseSchema,
 	queueSchema,
+	resourceSchema,
 } from '@webitel/api-services/validations';
 import { describe, expect, it } from 'vitest';
 import { effectScope, nextTick, ref } from 'vue';
@@ -134,3 +135,34 @@ describe('card validation of the close reason on a new final case', () => {
 		expect(r$.$fields.closeResult.$error).toBe(true);
 	});
 });
+
+/**
+ * [Claude] A required lookup cleared by the select becomes `{}`. regle builds a
+ * nested status for it, so the required issue has to land on `id` — otherwise
+ * the field shows no error and save stays enabled.
+ *
+ * [WTEL-10632](https://webitel.atlassian.net/browse/WTEL-10632)
+ */
+describe('card validation of a cleared required lookup', () => {
+	it('reports a cleared resource gateway on the field', async () => {
+		const state = ref({
+			...getDefaultsFromZodSchema(resourceSchema, {}),
+			name: 'a resource',
+			gateway: {},
+		});
+		const scope = effectScope(true);
+		// biome-ignore lint/suspicious/noExplicitAny: regle's inferred state type
+		let r$: any;
+
+		scope.run(() => {
+			({ r$ } = useRegleSchema(state, resourceSchema as never));
+		});
+
+		const result = await r$.$validate();
+
+		expect(result.valid).toBe(false);
+		expect(r$.$error).toBe(true);
+		expect(r$.$fields.gateway.$error).toBe(true);
+		expect(r$.$fields.gateway.$errors.id).toHaveLength(1);
+	});
+});

package/src/modules/permissions-page/scripts/PermissionsApiModule.ts

@@ -1,3 +1,4 @@
+import { AccessRuleName } from '@webitel/ui-sdk/modules/ObjectPermissions/enums';
 import type { TableApiModule } from '../../types/tableStore.types';
 import type {
 	PermissionEntity,
@@ -13,6 +14,25 @@ import type {
 export const PermissionsApiModule = (
 	rawApiModule: RawPermissionsApiModule,
 ): TableApiModule<PermissionEntity> => {
+	/**
+	 * [Claude] The rule columns are keyed by their letter (`r`, `w`, `d`), so
+	 * each sorts on its own, but the endpoint has no such attributes in `fields`:
+	 * it returns the three rules merged into one `granted` string, asked for once.
+	 */
+	const toAclFields = (fields?: string[]) => {
+		if (!fields) return fields;
+		const ruleFields = Object.values(AccessRuleName).filter(
+			(rule) => rule !== AccessRuleName.X,
+		);
+		return [
+			...new Set(
+				fields
+					.filter((field) => field !== 'id')
+					.map((field) => (ruleFields.includes(field) ? 'granted' : field)),
+			),
+		];
+	};
+
 	/**
 	 * The table store always asks for `id` (every registry needs it to open or
 	 * delete a row), but an access rule has no such attribute and the endpoint
@@ -27,7 +47,7 @@ export const PermissionsApiModule = (
 	} & Record<string, unknown>) =>
 		rawApiModule.getPermissionsList?.({
 			...params,
-			fields: fields?.filter((field) => field !== 'id'),
+			fields: toAclFields(fields),
 		});
 
 	return {

package/src/modules/permissions-page/scripts/__tests__/PermissionsApiModule.spec.ts

@@ -0,0 +1,31 @@
+import { describe, expect, it, vi } from 'vitest';
+
+import { PermissionsApiModule } from '../PermissionsApiModule';
+
+describe('PermissionsApiModule', () => {
+	it('asks for one `granted` in place of the rule columns, and never for `id`', async () => {
+		const getPermissionsList = vi.fn();
+		const { getList } = PermissionsApiModule({
+			getPermissionsList,
+		} as never);
+
+		await getList({
+			fields: [
+				'id',
+				'grantee',
+				'r',
+				'w',
+				'd',
+			],
+			sort: '+w',
+		} as never);
+
+		expect(getPermissionsList).toHaveBeenCalledWith({
+			fields: [
+				'grantee',
+				'granted',
+			],
+			sort: '+w',
+		});
+	});
+});