@simplysm/sd-cli 13.0.74 → 13.0.76

package/templates/init/packages/client-admin/src/views/home/base/role-permission/RoleDetail.tsx.hbs

@@ -33,7 +33,6 @@ export function RoleDetail(props: { itemId?: number }) {
     return appService.orm.connectWithoutTransaction(async (db) => {
       const role = await db
         .role()
-        // eslint-disable-next-line solid/reactivity -- 비동기 콜백 내부
         .where((c) => [expr.eq(c.id, props.itemId)])
         .select((c) => ({
           name: c.name,
@@ -61,7 +60,6 @@ export function RoleDetail(props: { itemId?: number }) {
     await appService.orm.connect(async (db) => {
       const count = await db
         .role()
-        // eslint-disable-next-line solid/reactivity -- 비동기 콜백 내부
         .where((c) => [
           expr.eq(c.name, data.name ?? ""),
 
@@ -78,7 +76,6 @@ export function RoleDetail(props: { itemId?: number }) {
     });
 
     // 저장
-    // eslint-disable-next-line solid/reactivity -- 비동기 콜백 내부
     const savedId = await appService.orm.connect(async (db) => {
       if (props.itemId == null) {
         const [inserted] = await db.role().insert(
@@ -95,7 +92,6 @@ export function RoleDetail(props: { itemId?: number }) {
       } else {
         await db
           .role()
-          // eslint-disable-next-line solid/reactivity -- 비동기 콜백 내부
           .where((_c) => [expr.eq(_c.id, props.itemId)])
           .update((_c) => ({
             name: data.name ?? "",
@@ -112,7 +108,6 @@ export function RoleDetail(props: { itemId?: number }) {
     await appService.orm.connect(async (db) => {
       await db
         .role()
-        // eslint-disable-next-line solid/reactivity -- 비동기 콜백 내부
         .where((_c) => [expr.eq(_c.id, props.itemId!)])
         .update((_c) => ({
           isDeleted: del,
@@ -133,19 +128,16 @@ export function RoleDetail(props: { itemId?: number }) {
       {(ctx) => (
         <div class="p-2">
           <FormTable>
-            <tbody>
-            <tr>
-              <th>이름</th>
-              <td>
+            <FormTable.Row>
+              <FormTable.Item label="이름">
                 <TextInput
                   required
                   disabled={!perms().edit}
                   value={ctx.data.name ?? ""}
                   onValueChange={(v) => ctx.setData("name", v)}
                 />
-              </td>
-            </tr>
-            </tbody>
+              </FormTable.Item>
+            </FormTable.Row>
           </FormTable>
         </div>
       )}

package/templates/init/packages/client-admin/src/views/home/base/role-permission/RolePermissionDetail.tsx.hbs

@@ -57,11 +57,9 @@ export function RolePermissionDetail(props: { roleId: number }) {
   }
 
   async function handleSubmit(data: Record<string, boolean>) {
-    // eslint-disable-next-line solid/reactivity -- 비동기 콜백 내부
     await appService.orm.connect(async (db) => {
       await db
         .rolePermission()
-        // eslint-disable-next-line solid/reactivity -- 비동기 콜백 내부
         .where((c) => [expr.eq(c.roleId, props.roleId)])
         .delete();
 

package/templates/init/packages/client-admin/src/views/home/base/role-permission/RolePermissionView.tsx

@@ -24,7 +24,7 @@ export function RolePermissionView() {
               variant={"ghost"}
               theme={"primary"}
               size={"xs"}
-              onClick={() => dialog.show(() => <RoleSheet />, { header: "권한그룹", closeOnBackdrop: true })}
+              onClick={() => dialog.show(() => <RoleSheet />, { header: "권한그룹", closeOnInteractOutside: true })}
             >
               <Icon icon={IconExternalLink} />
             </Button>

package/templates/init/packages/client-admin/src/views/home/my-info/MyInfoDetail.tsx.hbs

@@ -148,9 +148,8 @@ export function MyInfoDetail() {
           <section>
             <h3 class="mb-4 border-l-4 border-base-500 pl-3 font-bold text-base-500">인증정보</h3>
             <FormTable>
-              <tr>
-                <th>이메일</th>
-                <td>
+              <FormTable.Row>
+                <FormTable.Item label="이메일">
                   <TextInput
                     type="email"
                     required
@@ -158,11 +157,10 @@ export function MyInfoDetail() {
                     onValueChange={(v) => ctx.setData("email", v)}
                     autocomplete="off"
                   />
-                </td>
-              </tr>
-              <tr>
-                <th>현재 비밀번호</th>
-                <td>
+                </FormTable.Item>
+              </FormTable.Row>
+              <FormTable.Row>
+                <FormTable.Item label="현재 비밀번호">
                   <TextInput
                     required={Boolean(ctx.data.newPassword || ctx.data.confirmPassword)}
                     class="w-56"
@@ -172,11 +170,10 @@ export function MyInfoDetail() {
                     onValueChange={(v) => ctx.setData("currentPassword", v)}
                     placeholder="비밀번호 변경 시에만 입력"
                   />
-                </td>
-              </tr>
-              <tr>
-                <th>새 비밀번호</th>
-                <td>
+                </FormTable.Item>
+              </FormTable.Row>
+              <FormTable.Row>
+                <FormTable.Item label="새 비밀번호">
                   <TextInput
                     required={Boolean(ctx.data.currentPassword || ctx.data.confirmPassword)}
                     minLength={8}
@@ -187,11 +184,10 @@ export function MyInfoDetail() {
                     onValueChange={(v) => ctx.setData("newPassword", v)}
                     placeholder="비밀번호 변경 시에만 입력"
                   />
-                </td>
-              </tr>
-              <tr>
-                <th>새 비밀번호 확인</th>
-                <td>
+                </FormTable.Item>
+              </FormTable.Row>
+              <FormTable.Row>
+                <FormTable.Item label="새 비밀번호 확인">
                   <TextInput
                     required={Boolean(ctx.data.currentPassword || ctx.data.newPassword)}
                     minLength={8}
@@ -207,38 +203,35 @@ export function MyInfoDetail() {
                     onValueChange={(v) => ctx.setData("confirmPassword", v)}
                     placeholder="비밀번호 변경 시에만 입력"
                   />
-                </td>
-              </tr>
+                </FormTable.Item>
+              </FormTable.Row>
             </FormTable>
           </section>
 
           <section>
             <h3 class="mb-4 border-l-4 border-base-500 pl-3 font-bold text-base-500">시스템설정</h3>
             <FormTable>
-              <tbody>
-                <tr>
-                  <th>로그인 시 화면</th>
-                  <td>
-                    <Select
-                      class="w-full"
-                      value={ctx.data.firstRouterLink}
-                      onValueChange={(v) => ctx.setData("firstRouterLink", v)}
-                      items={routeOptions()}
-                      renderValue={(v) => <>{routeMap().get(v) ?? v}</>}
-                      placeholder="선택 안함"
+              <FormTable.Row>
+                <FormTable.Item label="로그인 시 화면">
+                  <Select
+                    class="w-full"
+                    value={ctx.data.firstRouterLink}
+                    onValueChange={(v) => ctx.setData("firstRouterLink", v)}
+                    items={routeOptions()}
+                    renderValue={(v) => <>{routeMap().get(v) ?? v}</>}
+                    placeholder="선택 안함"
+                  >
+                    <Select.ItemTemplate>
+                      {(href: string) => <>{routeMap().get(href) ?? href}</>}
+                    </Select.ItemTemplate>
+                    <Select.Action
+                      onClick={() => ctx.setData("firstRouterLink", undefined as any)}
                     >
-                      <Select.ItemTemplate>
-                        {(href: string) => <>{routeMap().get(href) ?? href}</>}
-                      </Select.ItemTemplate>
-                      <Select.Action
-                        onClick={() => ctx.setData("firstRouterLink", undefined as any)}
-                      >
-                        <Icon icon={IconX} class={"text-danger-500"} />
-                      </Select.Action>
-                    </Select>
-                  </td>
-                </tr>
-              </tbody>
+                      <Icon icon={IconX} class={"text-danger-500"} />
+                    </Select.Action>
+                  </Select>
+                </FormTable.Item>
+              </FormTable.Row>
             </FormTable>
           </section>
         </div>

package/templates/init/packages/db-main/package.json.hbs

@@ -7,7 +7,7 @@
     ".": "./src/index.ts"
   },
   "dependencies": {
-    "@simplysm/core-common": "~13.0.74",
-    "@simplysm/orm-common": "~13.0.74"
+    "@simplysm/core-common": "~13.0.76",
+    "@simplysm/orm-common": "~13.0.76"
   }
 }

package/templates/init/packages/server/package.json.hbs

@@ -5,11 +5,11 @@
   "private": true,
   "dependencies": {
     "@{{projectName}}/db-main": "workspace:*",
-    "@simplysm/core-common": "~13.0.74",
+    "@simplysm/core-common": "~13.0.76",
     "@simplysm/excel": "^13.0.71",
-    "@simplysm/orm-common": "~13.0.74",
-    "@simplysm/orm-node": "~13.0.74",
-    "@simplysm/service-server": "~13.0.74",
+    "@simplysm/orm-common": "~13.0.76",
+    "@simplysm/orm-node": "~13.0.76",
+    "@simplysm/service-server": "~13.0.76",
     "bcrypt": "^6.0.0",
     "pg": "^8.19.0",
     "pg-copy-streams": "^7.0.0"

package/templates/init/tests/e2e/package.json.hbs

@@ -6,7 +6,7 @@
   "private": true,
   "dependencies": {
     "@{{projectName}}/db-main": "workspace:*",
-    "@simplysm/orm-node": "~13.0.74",
+    "@simplysm/orm-node": "~13.0.76",
     "bcrypt": "^6.0.0",
     "playwright": "^1.58.2"
   },

package/tests/get-compiler-options-for-package.spec.ts

@@ -81,21 +81,6 @@ describe("getCompilerOptionsForPackage", () => {
     expect(result.types).toContain("lodash");
   });
 
-  it("node target: removes duplicate node types", async () => {
-    const packageDir = "/project/packages/core-node";
-    vi.mocked(fsExists).mockResolvedValue(true);
-    vi.mocked(fsReadJson).mockResolvedValue({
-      devDependencies: {
-        "@types/node": "^20.0.0",
-      },
-    });
-
-    const result = await getCompilerOptionsForPackage(baseOptions, "node", packageDir);
-
-    // node type is included only once without duplicates
-    expect(result.types?.filter((t) => t === "node")).toHaveLength(1);
-  });
-
   it("handles missing package.json with empty types", async () => {
     const packageDir = "/project/packages/unknown";
     vi.mocked(fsExists).mockResolvedValue(false);
@@ -106,34 +91,4 @@ describe("getCompilerOptionsForPackage", () => {
     expect(result.types).toEqual(["node"]);
   });
 
-  it("handles undefined lib correctly", async () => {
-    const optionsWithoutLib: ts.CompilerOptions = {
-      strict: true,
-    };
-    const packageDir = "/project/packages/core-node";
-    vi.mocked(fsExists).mockResolvedValue(false);
-
-    const result = await getCompilerOptionsForPackage(optionsWithoutLib, "node", packageDir);
-
-    // handles undefined lib without error
-    expect(result.lib).toBeUndefined();
-    expect(result.types).toEqual(["node"]);
-  });
-
-  it("does not mutate original baseOptions (immutability)", async () => {
-    const originalOptions: ts.CompilerOptions = {
-      lib: ["ES2024", "DOM"],
-      types: ["original"],
-      strict: true,
-    };
-    const packageDir = "/project/packages/core-node";
-    vi.mocked(fsExists).mockResolvedValue(false);
-
-    await getCompilerOptionsForPackage(originalOptions, "node", packageDir);
-
-    // original options are not changed
-    expect(originalOptions.lib).toEqual(["ES2024", "DOM"]);
-    expect(originalOptions.types).toEqual(["original"]);
-    expect(originalOptions.noEmit).toBeUndefined();
-  });
 });

package/tests/get-package-source-files.spec.ts

@@ -4,8 +4,6 @@ import type ts from "typescript";
 import { getPackageSourceFiles, getPackageFiles } from "../src/utils/tsconfig";
 
 describe("getPackageSourceFiles", () => {
-  const sep = path.sep;
-
   it("filters files within package src directory only", () => {
     const pkgDir = `/project/packages/core-common`;
     const parsedConfig = {
@@ -60,35 +58,6 @@ describe("getPackageSourceFiles", () => {
     ]);
   });
 
-  it("returns empty array if no files", () => {
-    const pkgDir = `/project/packages/empty`;
-    const parsedConfig = {
-      fileNames: [
-        `/project/packages/core/src/index.ts`,
-        `/project/packages/core-common/src/index.ts`,
-      ],
-    } as ts.ParsedCommandLine;
-
-    const result = getPackageSourceFiles(pkgDir, parsedConfig);
-
-    expect(result).toEqual([]);
-  });
-
-  it("handles path separators correctly", () => {
-    // use path.sep for platform-independent test
-    const pkgDir = `${sep}project${sep}packages${sep}core`;
-    const parsedConfig = {
-      fileNames: [
-        `${sep}project${sep}packages${sep}core${sep}src${sep}index.ts`,
-        `${sep}project${sep}packages${sep}core-common${sep}src${sep}index.ts`,
-      ],
-    } as ts.ParsedCommandLine;
-
-    const result = getPackageSourceFiles(pkgDir, parsedConfig);
-
-    expect(result).toEqual([`${sep}project${sep}packages${sep}core${sep}src${sep}index.ts`]);
-  });
-
   it("handles forward slash paths from TypeScript API correctly (Windows compatible)", () => {
     // TypeScript API returns forward slash paths even on Windows
     // pkgDir is created with path.join using OS-native separator
@@ -152,17 +121,6 @@ describe("getPackageFiles", () => {
     ]);
   });
 
-  it("returns empty array if no files", () => {
-    const pkgDir = `/project/packages/empty`;
-    const parsedConfig = {
-      fileNames: [`/project/packages/core/src/index.ts`],
-    } as ts.ParsedCommandLine;
-
-    const result = getPackageFiles(pkgDir, parsedConfig);
-
-    expect(result).toEqual([]);
-  });
-
   it("handles forward slash paths from TypeScript API correctly (Windows compatible)", () => {
     const pkgDir = path.resolve("/project/packages/core-common");
     const parsedConfig = {

package/tests/get-types-from-package-json.spec.ts

@@ -68,25 +68,6 @@ describe("getTypesFromPackageJson", () => {
     expect(result).toEqual([]);
   });
 
-  it("filters out dependencies that are not @types/*", async () => {
-    const packageDir = "/project/packages/core-common";
-    const mockFsExists = vi.mocked(fsExists);
-    const mockFsReadJson = vi.mocked(fsReadJson);
-
-    mockFsExists.mockResolvedValue(true);
-    mockFsReadJson.mockResolvedValue({
-      devDependencies: {
-        typescript: "^5.0.0",
-        vitest: "^1.0.0",
-        eslint: "^9.0.0",
-      },
-    });
-
-    const result = await getTypesFromPackageJson(packageDir);
-
-    expect(result).toEqual([]);
-  });
-
   it("handles scoped @types packages correctly", async () => {
     const packageDir = "/project/packages/core-common";
     const mockFsExists = vi.mocked(fsExists);

package/tests/infra/ResultCollector.spec.ts

@@ -27,13 +27,4 @@ describe("ResultCollector", () => {
     expect(collector.toMap().size).toBe(1);
   });
 
-  it("returns internal Map", () => {
-    const collector = new ResultCollector();
-    collector.add({ name: "pkg1", target: "node", type: "build", status: "success" });
-
-    const map = collector.toMap();
-
-    expect(map).toBeInstanceOf(Map);
-    expect(map.size).toBe(1);
-  });
 });

package/tests/infra/WorkerManager.spec.ts

@@ -26,12 +26,6 @@ describe("WorkerManager", () => {
     expect(manager.get("test-worker")).toBe(worker);
   });
 
-  it("returns undefined when retrieving a non-existent Worker", () => {
-    const manager = new WorkerManager();
-
-    expect(manager.get("nonexistent")).toBeUndefined();
-  });
-
   it("terminates all Workers", async () => {
     const manager = new WorkerManager();
     const worker1 = manager.create("worker1", "/path/to/worker.ts");
@@ -57,34 +51,6 @@ describe("WorkerManager", () => {
     expect(manager.get("worker2")).toBe(worker2);
   });
 
-  it("handles terminating a non-existent Worker without error", async () => {
-    const manager = new WorkerManager();
-
-    await expect(manager.terminate("nonexistent")).resolves.toBeUndefined();
-  });
-
-  it("retrieves the count of managed Workers", () => {
-    const manager = new WorkerManager();
-
-    expect(manager.size).toBe(0);
-
-    manager.create("worker1", "/path/to/worker.ts");
-    expect(manager.size).toBe(1);
-
-    manager.create("worker2", "/path/to/worker.ts");
-    expect(manager.size).toBe(2);
-  });
-
-  it("retrieves list of all Worker IDs", () => {
-    const manager = new WorkerManager();
-    manager.create("worker1", "/path/to/worker.ts");
-    manager.create("worker2", "/path/to/worker.ts");
-
-    const ids = manager.ids;
-
-    expect(ids).toEqual(["worker1", "worker2"]);
-  });
-
   it("overwrites existing Worker when creating with the same ID", () => {
     const manager = new WorkerManager();
     const worker1 = manager.create("same-id", "/path/to/worker1.ts");

package/tests/load-ignore-patterns.spec.ts

@@ -156,33 +156,4 @@ describe("loadIgnorePatterns", () => {
     expect(mockJitiImportFn).toHaveBeenCalledWith(expect.stringContaining("eslint.config.mts"));
   });
 
-  it("returns empty pattern array if empty array exported", async () => {
-    const cwd = "/project";
-    const mockExists = vi.mocked(fsExists);
-
-    mockExists.mockImplementation((filePath: string) => {
-      return Promise.resolve(filePath === path.join(cwd, "eslint.config.ts"));
-    });
-
-    mockJitiImportFn.mockResolvedValue({
-      default: [],
-    });
-
-    const patterns = await loadIgnorePatterns(cwd);
-
-    expect(patterns).toEqual([]);
-  });
-
-  it("propagates error if jiti import fails", async () => {
-    const cwd = "/project";
-    const mockExists = vi.mocked(fsExists);
-
-    mockExists.mockImplementation((filePath: string) => {
-      return Promise.resolve(filePath === path.join(cwd, "eslint.config.ts"));
-    });
-
-    mockJitiImportFn.mockRejectedValue(new Error("Syntax error in config file"));
-
-    await expect(loadIgnorePatterns(cwd)).rejects.toThrow("Syntax error in config file");
-  });
 });

package/tests/load-sd-config.spec.ts

@@ -65,28 +65,6 @@ describe("loadSdConfig", () => {
     );
   });
 
-  it("throws error if return value is wrong format (packages is array)", async () => {
-    vi.mocked(fsExists).mockResolvedValue(true);
-    mockJitiImport.mockResolvedValue({
-      default: () => ({ packages: [] }), // packages is array
-    });
-
-    await expect(loadSdConfig({ cwd: "/project", dev: false, opt: [] })).rejects.toThrow(
-      /sd\.config\.ts return value is not in .* correct format/,
-    );
-  });
-
-  it("throws error if return value is wrong format (packages is null)", async () => {
-    vi.mocked(fsExists).mockResolvedValue(true);
-    mockJitiImport.mockResolvedValue({
-      default: () => ({ packages: null }),
-    });
-
-    await expect(loadSdConfig({ cwd: "/project", dev: false, opt: [] })).rejects.toThrow(
-      /sd\.config\.ts return value is not in .* correct format/,
-    );
-  });
-
   it("returns correct config", async () => {
     vi.mocked(fsExists).mockResolvedValue(true);
     mockJitiImport.mockResolvedValue({
@@ -117,21 +95,4 @@ describe("loadSdConfig", () => {
     expect(config.packages).toEqual({});
   });
 
-  it("handles async function default export correctly", async () => {
-    vi.mocked(fsExists).mockResolvedValue(true);
-    mockJitiImport.mockResolvedValue({
-      // eslint-disable-next-line @typescript-eslint/require-await
-      default: async () => ({
-        packages: {
-          "core-common": { target: "neutral" },
-        },
-      }),
-    });
-
-    const config = await loadSdConfig({ cwd: "/project", dev: false, opt: [] });
-
-    expect(config.packages).toEqual({
-      "core-common": { target: "neutral" },
-    });
-  });
 });

package/tests/run-lint.spec.ts

@@ -255,25 +255,6 @@ describe("runLint", () => {
     expect(mockState.lintedFiles).toHaveLength(0);
   });
 
-  it("does not set exitCode when only warnings exist", async () => {
-    const cwd = "/project";
-    vi.mocked(fsExists).mockImplementation((filePath: string) => {
-      return Promise.resolve(filePath === path.join(cwd, "eslint.config.ts"));
-    });
-
-    mockJitiImportFn.mockResolvedValue({
-      default: [{ ignores: ["node_modules/**"] }],
-    });
-
-    vi.mocked(fsGlob).mockResolvedValue(["/project/src/index.ts"]);
-
-    mockState.lintResults = [{ errorCount: 0, warningCount: 3 }];
-
-    await runLint({ targets: [], fix: false, timing: false });
-
-    expect(process.exitCode).toBeUndefined();
-  });
-
   it("sets TIMING environment variable when timing option is enabled", async () => {
     const cwd = "/project";
     vi.mocked(fsExists).mockImplementation((filePath: string) => {
@@ -326,20 +307,6 @@ describe("runLint", () => {
     expect(process.exitCode).toBeUndefined();
   });
 
-  it("propagates error when glob fails", async () => {
-    const cwd = "/project";
-    vi.mocked(fsExists).mockImplementation((filePath: string) => {
-      return Promise.resolve(filePath === path.join(cwd, "eslint.config.ts"));
-    });
-
-    mockJitiImportFn.mockResolvedValue({
-      default: [{ ignores: ["node_modules/**"] }],
-    });
-
-    vi.mocked(fsGlob).mockRejectedValue(new Error("Glob error"));
-
-    await expect(runLint({ targets: [], fix: false, timing: false })).rejects.toThrow("Glob error");
-  });
 });
 
 describe("executeLint", () => {
@@ -392,24 +359,4 @@ describe("executeLint", () => {
     expect(result.warningCount).toBe(1);
   });
 
-  it("includes formatter output in formattedOutput", async () => {
-    mockState.lintResults = [{ errorCount: 1, warningCount: 0 }];
-    vi.mocked(fsGlob).mockResolvedValue(["/project/packages/core-common/src/index.ts"]);
-
-    const result = await executeLint({ targets: [], fix: false, timing: false });
-
-    // MockESLint's formatter returns empty string, so formattedOutput is also empty string
-    expect(result.formattedOutput).toBeDefined();
-    expect(typeof result.formattedOutput).toBe("string");
-  });
-
-  it("returns success result when no files exist", async () => {
-    vi.mocked(fsGlob).mockResolvedValue([]);
-
-    const result = await executeLint({ targets: [], fix: false, timing: false });
-
-    expect(result.success).toBe(true);
-    expect(result.errorCount).toBe(0);
-    expect(result.warningCount).toBe(0);
-  });
 });