@quark-fw/entity 0.1.15 → 0.1.17

package/dist/client/createEntityContext.js

@@ -77,7 +77,7 @@ export function createEntityContext(entityKey) {
                 createdAt: now,
                 ...data,
             };
-            const old = completeList;
+            shiftCount(1);
             setList((prev) => (prev ? [...prev, item] : [item]));
             return withMutation(async () => {
                 try {
@@ -85,7 +85,10 @@ export function createEntityContext(entityKey) {
                 }
                 catch (error) {
                     reportError(error, entityKey);
-                    setList(old ?? []);
+                    // откат точечный, а не снимком списка: снимок, снятый до
+                    // мутации, стёр бы параллельные create/delete
+                    setList((prev) => (prev ?? []).filter((e) => e.id !== item.id));
+                    shiftCount(-1);
                     return null;
                 }
             });
@@ -121,7 +124,11 @@ export function createEntityContext(entityKey) {
             });
         };
         const deleteById = async (id) => {
-            const old = completeList;
+            const current = completeList ?? fresh;
+            const index = current.findIndex((e) => e.id === id);
+            const removed = index === -1 ? undefined : current[index];
+            if (removed)
+                shiftCount(-1);
             setList((prev) => (prev ?? []).filter((e) => e.id !== id));
             await withMutation(async () => {
                 try {
@@ -129,7 +136,18 @@ export function createEntityContext(entityKey) {
                 }
                 catch (error) {
                     reportError(error, entityKey);
-                    setList(old ?? []);
+                    if (!removed)
+                        return;
+                    // возвращаем только удалённую запись и на её место —
+                    // снимок списка стёр бы параллельные мутации
+                    setList((prev) => {
+                        const arr = [...(prev ?? [])];
+                        if (arr.some((e) => e.id === id))
+                            return arr;
+                        arr.splice(Math.min(index, arr.length), 0, removed);
+                        return arr;
+                    });
+                    shiftCount(1);
                 }
             });
         };
@@ -167,7 +185,26 @@ export function createEntityContext(entityKey) {
         // их состав: иначе эффект ниже перезапускается на каждый рендер
         const serverArgs = useMemo(() => JSON.parse(argsKey), [argsKey]);
         const fresh = tinyCaller.list.mutate(serverArgs);
-        const count = tinyCaller.count.mutate(filter ? { where: filter } : {});
+        const countArgs = filter ? { where: filter } : {};
+        const count = tinyCaller.count.mutate(countArgs);
+        /**
+         * Сдвиг закэшированного count вслед за оптимистичной мутацией: иначе
+         * total застывал на значении первой загрузки. Вне рендера mutate
+         * отдаёт значение из кэша; если его там нет (ещё грузится) — бросает,
+         * и сдвигать нечего.
+         */
+        const shiftCount = (delta) => {
+            let current;
+            try {
+                current = tinyCaller.count.mutate(countArgs);
+            }
+            catch {
+                return;
+            }
+            if (typeof current === "number") {
+                tinyCaller.count.setCache(countArgs)(current + delta);
+            }
+        };
         const loadMore = async () => {
             const nextPage = loadedPages.current + 1;
             const rows = await withMutation(() => trpcCaller.list.mutate({ ...serverArgs, page: nextPage }));
@@ -183,6 +220,14 @@ export function createEntityContext(entityKey) {
             });
             return rows.length;
         };
+        // Другой фильтр/сортировка/лимит — другой список, снова с первой
+        // страницы. Без сброса догруженные страницы старого запроса навсегда
+        // выключали синхронизацию. Эффект объявлен до эффекта синхронизации:
+        // в одном коммите они выполняются по порядку.
+        const baseKey = JSON.stringify([filter, sort, limit]);
+        useEffect(() => {
+            loadedPages.current = 1;
+        }, [baseKey]);
         // мутации обновляют кэш store по тем же аргументам, что и чтение
         useEffect(() => {
             // ref читаем до подписки: к моменту очистки .current уже может
@@ -263,13 +308,17 @@ export function createEntityContext(entityKey) {
         const entities = useEntities();
         const controller = new Proxy({}, {
             get(_target, method) {
-                return (data) => trpcCaller.controller.mutate({
-                    id,
-                    // ключ прокси может быть символом, а по сети уходит
-                    // имя метода — берём только строковые
-                    method: String(method),
-                    data,
-                });
+                // Не методы контроллера: символы и служебные имена, которые
+                // трогают await (then), JSON.stringify (toJSON) и React/
+                // devtools ($$typeof) — иначе любое такое обращение, например
+                // `await controller`, уходило бы сетевым вызовом.
+                if (typeof method !== "string" ||
+                    method === "then" ||
+                    method === "toJSON" ||
+                    method.startsWith("$$")) {
+                    return undefined;
+                }
+                return (data) => trpcCaller.controller.mutate({ id, method, data });
             },
         });
         const value = {

package/dist/client/types.d.ts

@@ -36,6 +36,9 @@ export type EntityTinyCaller<Row> = {
         mutate(args: {
             where?: Record<string, unknown>;
         }): number;
+        setCache(args: {
+            where?: Record<string, unknown>;
+        }): (n: number) => void;
     };
     getById: {
         mutate(id: string): Row | null;

package/dist/createEntity.js

@@ -143,7 +143,16 @@ export function createEntity(args) {
         const proc = args.procedure ?? publicProcedure;
         // TRPCUser — пустой интерфейс-шов, его поля объявляет плагин
         // авторизации, поэтому id читаем структурно
-        const ownerId = (ctx) => ctx.user?.id;
+        //
+        // Без пользователя — UNAUTHORIZED, а не undefined: в Prisma ключ
+        // where со значением undefined фильтром не считается, и скоуп
+        // { userId: undefined } на publicProcedure отдал бы все строки.
+        const ownerId = (ctx) => {
+            const id = ctx.user?.id;
+            if (!id)
+                throw new TRPCError({ code: "UNAUTHORIZED" });
+            return id;
+        };
         // where-скоуп владельца: {} если authField не задан
         const scope = (ctx) => args.authField ? { [args.authField]: ownerId(ctx) } : {};
         /**
@@ -188,6 +197,31 @@ export function createEntity(args) {
                 }
             }
         };
+        /**
+         * Обновление по запросу клиента: отсев защищённых полей, проверка
+         * родителей и скоуп во where. Общее для процедуры update и
+         * controller.update — иначе через контроллер клиент обходил бы отсев
+         * и переписывал владельца записи.
+         */
+        const updateFromClient = async (id, input, ctx) => {
+            const data = stripProtected(input, ["id"]);
+            await checkRelations(data, ctx);
+            // скоуп во where: чужую запись обновить нельзя
+            const where = { id, ...scope(ctx) };
+            // после отсева могло не остаться ни одного поля —
+            // отвечаем самой записью, а не ложным NOT_FOUND
+            if (!Object.keys(data).length) {
+                const row = await methods.getOne(where);
+                if (!row) {
+                    throw new TRPCError({
+                        code: "NOT_FOUND",
+                        message: `${args.model} not found`,
+                    });
+                }
+                return row;
+            }
+            return methods.updateWhere(where, data);
+        };
         const custom = args.extendMethods?.({ model: getModel() }) ?? {};
         const customProcedures = Object.fromEntries(Object.entries(custom).map(([name, m]) => [
             name,
@@ -220,25 +254,7 @@ export function createEntity(args) {
                 id: z.string(),
                 data: z.record(z.string(), z.any()),
             }))
-                .mutation(async ({ input, ctx }) => {
-                const data = stripProtected(input.data, ["id"]);
-                await checkRelations(data, ctx);
-                // скоуп во where: чужую запись обновить нельзя
-                const where = { id: input.id, ...scope(ctx) };
-                // после отсева могло не остаться ни одного поля —
-                // отвечаем самой записью, а не ложным NOT_FOUND
-                if (!Object.keys(data).length) {
-                    const row = await methods.getOne(where);
-                    if (!row) {
-                        throw new TRPCError({
-                            code: "NOT_FOUND",
-                            message: `${args.model} not found`,
-                        });
-                    }
-                    return row;
-                }
-                return methods.updateWhere(where, data);
-            }),
+                .mutation(({ input, ctx }) => updateFromClient(input.id, input.data, ctx)),
             delete: proc
                 .input(z.object({ id: z.string() }))
                 .mutation(({ input, ctx }) => methods.deleteWhere({ id: input.id, ...scope(ctx) })),
@@ -262,7 +278,18 @@ export function createEntity(args) {
                 }).first());
                 if (!row)
                     throw new TRPCError({ code: "NOT_FOUND" });
-                const controller = buildController(row);
+                // встроенный update подменяется клиентской версией;
+                // кастомные методы extendController по-прежнему главнее
+                const controller = {
+                    ...baseController(row),
+                    update: {
+                        input: z.record(z.string(), z.any()),
+                        handler: (data) => updateFromClient(row.id, 
+                        // input выше гарантирует объект
+                        data, ctx),
+                    },
+                    ...(args.extendController?.({ entity: row }) ?? {}),
+                };
                 // Только собственные методы контроллера: без этой проверки
                 // имя вроде "constructor" или "toString" дотягивается до
                 // прототипа объекта и вызывается как метод сущности.

package/package.json

@@ -1,6 +1,6 @@
 {
 	"name": "@quark-fw/entity",
-	"version": "0.1.15",
+	"version": "0.1.17",
 	"description": "Entity CRUD layer over tRPC and Prisma for the Quark framework",
 	"type": "module",
 	"main": "dist/index.js",
@@ -31,13 +31,13 @@
 		"access": "public"
 	},
 	"dependencies": {
-		"@quark-fw/plugin-trpc": "0.1.15",
-		"@quark-fw/store": "0.1.15",
+		"@quark-fw/plugin-trpc": "0.1.17",
+		"@quark-fw/store": "0.1.17",
 		"@trpc/server": "^11.0.0",
 		"zod": "^4.0.0"
 	},
 	"peerDependencies": {
-		"@quark-fw/core": "0.1.15",
+		"@quark-fw/core": "0.1.17",
 		"react": "^19.2.4"
 	},
 	"devDependencies": {