befly 3.10.0 → 3.10.2

package/tests/syncApi-type-compat.test.ts

@@ -1,72 +0,0 @@
-import { describe, expect, test } from "bun:test";
-
-import { syncApi } from "../sync/syncApi.js";
-
-describe("syncApi - type compatibility", () => {
-    test("缺少 type 时应视为 api；非 api type 应被跳过", async () => {
-        const existingRecords = [
-            { id: 1, routePath: "/api/app/keep", name: "Keep", addonName: "", state: 0 },
-            { id: 2, routePath: "/api/app/skip", name: "Skip", addonName: "", state: 0 }
-        ];
-
-        const calls = {
-            delForceBatch: [] as any[],
-            getAllArgs: null as any
-        };
-
-        const dbHelper = {
-            tableExists: async () => true,
-            updBatch: async () => 0,
-            insBatch: async () => [],
-            getAll: async (options: any) => {
-                calls.getAllArgs = options;
-                return { lists: existingRecords };
-            },
-            delForceBatch: async (_table: any, ids: any[]) => {
-                calls.delForceBatch.push(ids);
-                return ids.length;
-            }
-        } as any;
-
-        const ctx = {
-            db: dbHelper,
-            addons: [],
-            cache: {
-                cacheApis: async () => {},
-                rebuildRoleApiPermissions: async () => {}
-            }
-        } as any;
-
-        const apiItems = [
-            // 不带 type：应按 api 处理，保留 keep
-            {
-                source: "app",
-                sourceName: "项目",
-                filePath: "DUMMY",
-                relativePath: "keep",
-                fileName: "keep",
-                moduleName: "app_keep",
-                name: "Keep",
-                routePath: "/api/app/keep",
-                addonName: "",
-                fileBaseName: "keep.ts",
-                fileDir: "DUMMY",
-                content: { name: "Keep", handler: async () => {} }
-            },
-            // 带非 api type：应被跳过，因此 DB 中的 skip 会被当作“配置不存在”而删除
-            {
-                type: "menu",
-                name: "Skip",
-                routePath: "/api/app/skip",
-                addonName: ""
-            }
-        ] as any;
-
-        await syncApi(ctx, apiItems);
-
-        expect(calls.getAllArgs?.fields).toEqual(["id", "routePath", "name", "addonName", "state"]);
-
-        expect(calls.delForceBatch).toHaveLength(1);
-        expect(calls.delForceBatch[0]).toEqual([2]);
-    });
-});

package/tests/syncDev-permissions.test.ts

@@ -1,81 +0,0 @@
-import { describe, expect, test } from "bun:test";
-
-import { syncDev } from "../sync/syncDev.js";
-
-describe("syncDev - dev role permissions", () => {
-    test("dev 角色应拥有所有菜单和接口（state>=0）", async () => {
-        const calls = {
-            getAll: [],
-            insData: [],
-            updData: [],
-            rebuildRoleApiPermissionsCount: 0
-        };
-
-        let nextId = 100;
-
-        const ctx = {
-            db: {
-                tableExists: async (table) => {
-                    return table === "addon_admin_admin" || table === "addon_admin_role" || table === "addon_admin_menu" || table === "addon_admin_api";
-                },
-                getAll: async (options) => {
-                    calls.getAll.push(options);
-
-                    if (options?.table === "addon_admin_menu") {
-                        return {
-                            lists: [
-                                { path: "/dashboard", state: 0 },
-                                { path: "/permission/role", state: 0 }
-                            ]
-                        };
-                    }
-                    if (options?.table === "addon_admin_api") {
-                        return {
-                            lists: [
-                                { routePath: "/api/health", state: 0 },
-                                { routePath: "/api/addon/addonAdmin/auth/login", state: 0 }
-                            ]
-                        };
-                    }
-
-                    return { lists: [] };
-                },
-                getOne: async (_options) => {
-                    // 让所有角色/管理员都走插入逻辑，便于断言插入数据
-                    return null;
-                },
-                insData: async (options) => {
-                    calls.insData.push(options);
-                    nextId += 1;
-                    return nextId;
-                },
-                updData: async (options) => {
-                    calls.updData.push(options);
-                    return 1;
-                }
-            },
-            cache: {
-                rebuildRoleApiPermissions: async () => {
-                    calls.rebuildRoleApiPermissionsCount += 1;
-                }
-            }
-        };
-
-        await syncDev(ctx, { devEmail: "dev@qq.com", devPassword: "dev-password" });
-
-        // 断言读取菜单/接口时按 state>=0 查询
-        const menuGetAll = calls.getAll.find((c) => c?.table === "addon_admin_menu");
-        expect(menuGetAll?.where?.state$gte).toBe(0);
-
-        const apiGetAll = calls.getAll.find((c) => c?.table === "addon_admin_api");
-        expect(apiGetAll?.where?.state$gte).toBe(0);
-
-        // 断言 dev 角色写入时包含“所有路径”（按查询结果写入；统一为 pathname，不包含 method）
-        const devRoleInsert = calls.insData.find((c) => c?.table === "addon_admin_role" && c?.data?.code === "dev");
-        expect(devRoleInsert).toBeTruthy();
-        expect(devRoleInsert.data.menus).toEqual(["/dashboard", "/permission/role"]);
-        expect(devRoleInsert.data.apis).toEqual(["/api/health", "/api/addon/addonAdmin/auth/login"]);
-
-        expect(calls.rebuildRoleApiPermissionsCount).toBe(0);
-    });
-});

package/tests/syncMenu-disableMenus-hard-delete.test.ts

@@ -1,88 +0,0 @@
-import { describe, expect, test } from "bun:test";
-import { mkdirSync, rmSync, writeFileSync } from "node:fs";
-import { join } from "node:path";
-
-import { checkMenu } from "../checks/checkMenu.js";
-import { syncMenu } from "../sync/syncMenu.js";
-
-describe("syncMenu - disableMenus hard delete", () => {
-    test("命中 disableMenus 的菜单应被强制删除（不分 state）", async () => {
-        const originalCwd = process.cwd();
-        const projectDir = join(originalCwd, "temp", `syncMenu-disableMenus-hard-delete-${Date.now()}-${Math.random().toString(16).slice(2)}`);
-        const menusJsonPath = join(projectDir, "menus.json");
-
-        // /addon/admin/403 为禁用项，即使 state=-1 也应该被硬删除
-        const existingMenus = [
-            { id: 1, path: "/addon/admin/403", parentPath: "", name: "403", sort: 1, state: -1 },
-            { id: 2, path: "/keep", parentPath: "", name: "Keep", sort: 2, state: 0 }
-        ];
-
-        const calls = {
-            delForceBatch: [] as Array<{ table: string; ids: number[] }>
-        };
-
-        const dbHelper = {
-            tableExists: async () => true,
-            trans: async (callback: any) => {
-                return await callback(dbHelper);
-            },
-            getAll: async (options: any) => {
-                // syncMenu 会调用一次“全量不带 where”，一次 state>=0 的逻辑已在内存过滤
-                if (options?.table === "addon_admin_menu") {
-                    return { lists: existingMenus };
-                }
-                return { lists: [] };
-            },
-            updBatch: async () => 0,
-            insBatch: async () => [],
-            delForceBatch: async (table: string, ids: number[]) => {
-                calls.delForceBatch.push({ table: table, ids: ids });
-                return ids.length;
-            }
-        } as any;
-
-        const ctx = {
-            db: dbHelper,
-            addons: [],
-            config: {
-                disableMenus: ["**/403"]
-            },
-            cache: {
-                cacheMenus: async () => {}
-            }
-        } as any;
-
-        try {
-            mkdirSync(projectDir, { recursive: true });
-            process.chdir(projectDir);
-
-            // 配置中仅保留 /keep
-            writeFileSync(
-                menusJsonPath,
-                JSON.stringify(
-                    [
-                        {
-                            name: "Keep",
-                            path: "/keep",
-                            sort: 2
-                        }
-                    ],
-                    null,
-                    4
-                ),
-                { encoding: "utf8" }
-            );
-
-            const menus = await checkMenu(ctx.addons, { disableMenus: ctx.config.disableMenus });
-            await syncMenu(ctx, menus);
-        } finally {
-            process.chdir(originalCwd);
-            rmSync(projectDir, { recursive: true, force: true });
-        }
-
-        expect(calls.delForceBatch).toHaveLength(1);
-        expect(calls.delForceBatch[0].table).toBe("addon_admin_menu");
-        // 应包含禁用菜单 id=1；/keep 不应被删
-        expect(calls.delForceBatch[0].ids).toEqual([1]);
-    });
-});

package/tests/syncMenu-duplicate-path.test.ts

@@ -1,122 +0,0 @@
-import { describe, expect, test, beforeEach, afterEach, mock } from "bun:test";
-import { mkdirSync, rmSync, writeFileSync } from "node:fs";
-import { join } from "node:path";
-
-import { checkMenu } from "../checks/checkMenu.js";
-import { setMockLogger } from "../lib/logger.js";
-import { syncMenu } from "../sync/syncMenu.js";
-
-// Mock pino logger
-const mockPino = {
-    info: mock(() => {}),
-    warn: mock(() => {}),
-    error: mock(() => {}),
-    debug: mock(() => {}),
-    fatal: mock(() => {}),
-    trace: mock(() => {}),
-    silent: mock(() => {}),
-    child: mock(() => mockPino),
-    level: "info"
-};
-
-describe("syncMenu - duplicate path records", () => {
-    beforeEach(() => {
-        setMockLogger(mockPino as any);
-        mockPino.warn.mockClear();
-    });
-
-    afterEach(() => {
-        setMockLogger(null);
-    });
-
-    test("应检测重复 path 并删除多余记录（保留 id 最大的一条）", async () => {
-        const originalCwd = process.cwd();
-        const projectDir = join(originalCwd, "temp", `syncMenu-duplicate-path-${Date.now()}-${Math.random().toString(16).slice(2)}`);
-        const menusJsonPath = join(projectDir, "menus.json");
-
-        const existingMenus = [
-            { id: 10, path: "/keep", parentPath: "", name: "Keep", sort: 999, state: 0 },
-            { id: 20, path: "/keep", parentPath: "", name: "Keep", sort: 999, state: 0 }
-        ];
-
-        const calls = {
-            delForceBatch: [] as any[],
-            updBatchCount: 0,
-            insBatchCount: 0
-        };
-
-        const dbHelper = {
-            tableExists: async () => true,
-            trans: async (callback: any) => {
-                return await callback(dbHelper);
-            },
-            getAll: async (options: any) => {
-                const stateGte = options?.where?.state$gte;
-                if (typeof stateGte === "number") {
-                    return { lists: existingMenus.filter((m) => typeof m.state === "number" && m.state >= stateGte) };
-                }
-                return { lists: existingMenus };
-            },
-            updBatch: async () => {
-                calls.updBatchCount += 1;
-                return 0;
-            },
-            insBatch: async () => {
-                calls.insBatchCount += 1;
-                return [];
-            },
-            delForceBatch: async (table: string, ids: number[]) => {
-                calls.delForceBatch.push({ table: table, ids: ids });
-                return ids.length;
-            }
-        } as any;
-
-        const ctx = {
-            db: dbHelper,
-            addons: [],
-            config: {
-                disableMenus: []
-            },
-            cache: {
-                cacheMenus: async () => {}
-            }
-        } as any;
-
-        try {
-            mkdirSync(projectDir, { recursive: true });
-            process.chdir(projectDir);
-
-            writeFileSync(
-                menusJsonPath,
-                JSON.stringify(
-                    [
-                        {
-                            name: "Keep",
-                            path: "/keep",
-                            sort: 999
-                        }
-                    ],
-                    null,
-                    4
-                ),
-                { encoding: "utf8" }
-            );
-
-            const menus = await checkMenu(ctx.addons);
-            await syncMenu(ctx, menus);
-        } finally {
-            process.chdir(originalCwd);
-            rmSync(projectDir, { recursive: true, force: true });
-        }
-
-        expect(mockPino.warn).toHaveBeenCalledTimes(1);
-        expect(calls.updBatchCount).toBe(0);
-        expect(calls.insBatchCount).toBe(0);
-
-        expect(calls.delForceBatch).toHaveLength(1);
-        expect(calls.delForceBatch[0].table).toBe("addon_admin_menu");
-
-        // 只应删除较小 id 的重复记录，保留 id=20
-        expect(calls.delForceBatch[0].ids).toEqual([10]);
-    });
-});

package/tests/syncMenu-obsolete-records.test.ts

@@ -1,161 +0,0 @@
-import { describe, expect, test } from "bun:test";
-import { mkdirSync, rmSync, writeFileSync } from "node:fs";
-import { join } from "node:path";
-
-import { checkMenu } from "../checks/checkMenu.js";
-import { syncMenu } from "../sync/syncMenu.js";
-
-describe("syncMenu - delete obsolete records", () => {
-    test("应删除不在配置中的菜单记录（仅 state>=0）", async () => {
-        const originalCwd = process.cwd();
-        const projectDir = join(originalCwd, "temp", `syncMenu-obsolete-records-${Date.now()}-${Math.random().toString(16).slice(2)}`);
-        const menusJsonPath = join(projectDir, "menus.json");
-
-        const existingMenus = [
-            { id: 1, path: "/a", parentPath: "", state: 0 },
-            { id: 2, path: "/b", parentPath: "", state: -1 },
-            { id: 3, path: "", parentPath: "", state: 0 }
-        ];
-
-        const calls = {
-            getAllCount: 0,
-            delForceBatch: [] as any[]
-        };
-
-        const dbHelper = {
-            tableExists: async () => true,
-            trans: async (callback: any) => {
-                return await callback(dbHelper);
-            },
-            getAll: async (options: any) => {
-                calls.getAllCount += 1;
-                const stateGte = options?.where?.state$gte;
-                if (typeof stateGte === "number") {
-                    return { lists: existingMenus.filter((m) => typeof m.state === "number" && m.state >= stateGte) };
-                }
-                return { lists: existingMenus };
-            },
-            updBatch: async () => {
-                return 0;
-            },
-            insBatch: async () => {
-                return [];
-            },
-            delForceBatch: async (table: string, ids: number[]) => {
-                calls.delForceBatch.push({ table: table, ids: ids });
-                return ids.length;
-            }
-        } as any;
-
-        const ctx = {
-            db: dbHelper,
-            addons: [],
-            config: {
-                disableMenus: []
-            },
-            cache: {
-                cacheMenus: async () => {}
-            }
-        } as any;
-
-        try {
-            mkdirSync(projectDir, { recursive: true });
-            process.chdir(projectDir);
-
-            writeFileSync(menusJsonPath, "[]", { encoding: "utf8" });
-            const menus = await checkMenu(ctx.addons);
-            await syncMenu(ctx, menus);
-        } finally {
-            process.chdir(originalCwd);
-            rmSync(projectDir, { recursive: true, force: true });
-        }
-
-        expect(calls.getAllCount).toBe(1);
-        expect(calls.delForceBatch).toHaveLength(1);
-        expect(calls.delForceBatch[0].table).toBe("addon_admin_menu");
-        expect(calls.delForceBatch[0].ids).toEqual([1]);
-    });
-
-    test("不应删除仍在配置中的菜单记录", async () => {
-        const originalCwd = process.cwd();
-        const projectDir = join(originalCwd, "temp", `syncMenu-obsolete-records-${Date.now()}-${Math.random().toString(16).slice(2)}`);
-        const menusJsonPath = join(projectDir, "menus.json");
-
-        const existingMenus = [
-            { id: 1, path: "/keep", parentPath: "", name: "Keep", sort: 999, state: 0 },
-            { id: 2, path: "/remove", parentPath: "", name: "Remove", sort: 999, state: 0 }
-        ];
-
-        const calls = {
-            delForceBatch: [] as any[],
-            updBatchCount: 0
-        };
-
-        const dbHelper = {
-            tableExists: async () => true,
-            trans: async (callback: any) => {
-                return await callback(dbHelper);
-            },
-            getAll: async (options: any) => {
-                const stateGte = options?.where?.state$gte;
-                if (typeof stateGte === "number") {
-                    return { lists: existingMenus.filter((m) => typeof m.state === "number" && m.state >= stateGte) };
-                }
-                return { lists: existingMenus };
-            },
-            updBatch: async () => {
-                calls.updBatchCount += 1;
-                return 0;
-            },
-            insBatch: async () => {
-                return [];
-            },
-            delForceBatch: async (table: string, ids: number[]) => {
-                calls.delForceBatch.push({ table: table, ids: ids });
-                return ids.length;
-            }
-        } as any;
-
-        const ctx = {
-            db: dbHelper,
-            addons: [],
-            config: {
-                disableMenus: []
-            },
-            cache: {
-                cacheMenus: async () => {}
-            }
-        } as any;
-
-        try {
-            mkdirSync(projectDir, { recursive: true });
-            process.chdir(projectDir);
-
-            writeFileSync(
-                menusJsonPath,
-                JSON.stringify(
-                    [
-                        {
-                            name: "Keep",
-                            path: "/keep",
-                            sort: 999
-                        }
-                    ],
-                    null,
-                    4
-                ),
-                { encoding: "utf8" }
-            );
-            const menus = await checkMenu(ctx.addons);
-            await syncMenu(ctx, menus);
-        } finally {
-            process.chdir(originalCwd);
-            rmSync(projectDir, { recursive: true, force: true });
-        }
-
-        expect(calls.delForceBatch).toHaveLength(1);
-        expect(calls.delForceBatch[0].table).toBe("addon_admin_menu");
-        expect(calls.delForceBatch[0].ids).toEqual([2]);
-        expect(calls.updBatchCount).toBe(0);
-    });
-});

package/tests/syncMenu-parentPath-from-tree.test.ts

@@ -1,75 +0,0 @@
-import { describe, expect, test } from "bun:test";
-
-import { __test__ } from "../sync/syncMenu.js";
-
-describe("syncMenu - parentPath derived from tree", () => {
-    test("根级菜单不应强制按 URL path 推导 parentPath（避免把同级菜单挂到首页下面）", () => {
-        const mergedMenus: any[] = [
-            {
-                name: "首页",
-                path: "/addon/admin",
-                sort: 1
-            },
-            {
-                name: "日志管理",
-                path: "/addon/admin/log",
-                sort: 40
-            },
-            {
-                name: "配置管理",
-                path: "/addon/admin/config",
-                sort: 30
-            }
-        ];
-
-        const map = __test__.flattenMenusToDefMap(mergedMenus as any);
-
-        expect(map.get("/addon/admin")?.parentPath).toBe("");
-        expect(map.get("/addon/admin/log")?.parentPath).toBe("");
-        expect(map.get("/addon/admin/config")?.parentPath).toBe("");
-    });
-
-    test("子菜单应跟随 children 嵌套关系设置 parentPath", () => {
-        const mergedMenus: any[] = [
-            {
-                name: "日志管理",
-                path: "/addon/admin/log",
-                sort: 40,
-                children: [
-                    {
-                        name: "登录日志",
-                        path: "/addon/admin/log/login",
-                        sort: 1
-                    }
-                ]
-            }
-        ];
-
-        const map = __test__.flattenMenusToDefMap(mergedMenus as any);
-
-        expect(map.get("/addon/admin/log")?.parentPath).toBe("");
-        expect(map.get("/addon/admin/log/login")?.parentPath).toBe("/addon/admin/log");
-    });
-
-    test("显式 parentPath（包括空字符串）优先生效", () => {
-        const mergedMenus: any[] = [
-            {
-                name: "自定义根",
-                path: "/x",
-                parentPath: "/custom",
-                sort: 1
-            },
-            {
-                name: "显式根",
-                path: "/y",
-                parentPath: "",
-                sort: 2
-            }
-        ];
-
-        const map = __test__.flattenMenusToDefMap(mergedMenus as any);
-
-        expect(map.get("/x")?.parentPath).toBe("/custom");
-        expect(map.get("/y")?.parentPath).toBe("");
-    });
-});

package/tests/syncMenu-paths.test.ts

@@ -1,59 +0,0 @@
-import { describe, test, expect, beforeAll, afterAll } from "bun:test";
-import { existsSync, mkdirSync, rmSync, writeFileSync } from "node:fs";
-import { join } from "node:path";
-
-import { __test__ } from "../sync/syncMenu.js";
-
-const testRootDir = join(process.cwd(), "temp", "test-sync-menu-views");
-const viewsDir = join(testRootDir, "views");
-
-function writeVueIndex(dir: string, title: string, order?: number): void {
-    if (!existsSync(dir)) {
-        mkdirSync(dir, { recursive: true });
-    }
-
-    const metaOrder = typeof order === "number" ? `, order: ${order}` : "";
-
-    const content = `<script setup>\ndefinePage({ meta: { title: "${title}"${metaOrder} } });\n</script>\n\n<template>\n    <div />\n</template>\n`;
-    writeFileSync(join(dir, "index.vue"), content, { encoding: "utf8" });
-}
-
-beforeAll(() => {
-    if (existsSync(testRootDir)) {
-        rmSync(testRootDir, { recursive: true, force: true });
-    }
-    mkdirSync(viewsDir, { recursive: true });
-
-    // 根级 index：应映射为 prefix 本身（不带尾随 /）
-    writeVueIndex(join(viewsDir, "index"), "Root", 2);
-
-    // 带数字后缀目录：应清理后缀
-    writeVueIndex(join(viewsDir, "user_1"), "User", 1);
-});
-
-afterAll(() => {
-    if (existsSync(testRootDir)) {
-        rmSync(testRootDir, { recursive: true, force: true });
-    }
-});
-
-describe("syncMenu - scanViewsDir paths", () => {
-    test("根级 index 不应生成尾随斜杠", async () => {
-        const prefix = "/addon/addonAdmin";
-        const menus = await __test__.scanViewsDir(viewsDir, prefix);
-
-        const root = menus.find((m) => m.path === prefix);
-        expect(root).toBeDefined();
-        expect(root?.path).toBe(prefix);
-    });
-
-    test("目录名 _数字 后缀应被清理", async () => {
-        const prefix = "/addon/addonAdmin";
-        const menus = await __test__.scanViewsDir(viewsDir, prefix);
-
-        const user = menus.find((m) => m.path === "/addon/addonAdmin/user");
-        expect(user).toBeDefined();
-        expect(user?.name).toBe("User");
-        expect(user?.sort).toBe(1);
-    });
-});