merge main and preserve CI test gates

Author: kense-lab

.github/workflows/ci.yml

@@ -90,6 +90,14 @@ jobs:
         run: pnpm i18n:check
       - name: Skill Market tests
         run: pnpm --filter @dmwork/skillmarket test
+      - name: Unified import/export unit tests
+        run: >-
+          pnpm --filter @octo/docs exec vitest run
+          src/editor/EditorShell.test.tsx
+          src/editor/fontFamilies.verify.test.ts
+          src/editor/importFlow.markdown.test.ts
+          src/pages/docsApi.export.test.ts
+          src/pages/docsApi.import.test.ts
       - name: Build
         run: pnpm build
       - name: Lint

apps/web/e2e-kit/screenshots/chromium/C1-loop-empty-workspace.spec.ts/loop-empty-workspace.png

@@ -14,7 +14,7 @@
   flex: 0 0 auto;
   align-items: center;
   gap: var(--wk-sp-2);
-  margin: var(--wk-sp-1) var(--wk-sp-4) var(--wk-sp-1);
+  margin: var(--wk-sp-3) var(--wk-sp-4) var(--wk-sp-2);
   padding: 0 var(--wk-sp-3);
   border: 1px solid var(--wk-border-subtle);
   border-radius: var(--wk-r-sm);

apps/web/e2e-kit/screenshots/chromium/C3-loop-workspace-default-view.spec.ts/loop-workspace-default-view.png

@@ -242,8 +242,8 @@
 
 .wk-group-management-remove {
   display: inline-flex;
-  width: var(--wk-sp-7);
-  height: var(--wk-sp-7);
+  width: var(--wk-sp-6);
+  height: var(--wk-sp-6);
   flex: 0 0 auto;
   align-items: center;
   justify-content: center;
@@ -253,10 +253,35 @@
   background: transparent;
   color: var(--wk-text-tertiary);
   cursor: pointer;
+  line-height: 0;
   transition: background-color var(--wk-dur-fast) var(--wk-ease),
     color var(--wk-dur-fast) var(--wk-ease);
 }
 
+.wk-group-management-remove-icon {
+  position: relative;
+  display: inline-flex;
+  width: calc(var(--wk-sp-3) + var(--wk-sp-0-5));
+  height: calc(var(--wk-sp-3) + var(--wk-sp-0-5));
+  align-items: center;
+  justify-content: center;
+  border: 1px solid currentColor;
+  border-radius: var(--wk-r-full);
+  box-sizing: border-box;
+}
+
+.wk-group-management-remove-icon::after {
+  position: absolute;
+  top: 50%;
+  left: 50%;
+  width: var(--wk-sp-2);
+  height: 1px;
+  border-radius: var(--wk-r-full);
+  background: currentColor;
+  content: "";
+  transform: translate(-50%, -50%);
+}
+
 .wk-group-management-remove:hover,
 .wk-group-management-remove:focus-visible {
   background: var(--wk-color-error-bg);

packages/dmworkbase/src/Components/GroupManagement/MemberPicker.css

@@ -1,6 +1,5 @@
 import React from "react";
 import { Switch } from "@douyinfe/semi-ui";
-import { Trash2 } from "lucide-react";
 import WKButton from "../../../Components/WKButton";
 import "./index.css";
 
@@ -103,7 +102,7 @@ function GroupManagementMemberRow({
           aria-label={`${labels.removeMember} ${labelText}`}
           title={labels.removeMember}
         >
-          <Trash2 size={15} aria-hidden="true" />
+          <span className="wk-group-management-remove-icon" aria-hidden="true" />
         </button>
       )}
     </li>

packages/dmworkbase/src/ui/channelSetting/GroupManagementView/index.css

@@ -12,6 +12,7 @@ import type {
   McpQuickStart,
   UpdateMcpParams,
 } from "../types/mcp";
+import { toWireParams } from "./mcpWireParams";
 import {
   MCP_CATEGORY_LABELS,
   MCP_CATEGORY_ORDER,
@@ -237,6 +238,7 @@ async function updateMcpMock(
   next.createdByType = prev.createdByType;
   next.createdByBotUid = prev.createdByBotUid;
   next.createdByBotName = prev.createdByBotName;
+  next.visibility = prev.visibility;
   MOCK_MCP_DETAILS[idx] = next;
   const listIdx = MOCK_MCP_LIST.findIndex((it) => it.id === id);
   if (listIdx !== -1) MOCK_MCP_LIST[listIdx] = projectListItem(next);
@@ -632,30 +634,6 @@ function mapDetail(raw: McpDetailWire): McpDetail {
   };
 }
 
-function toWireParams(params: CreateMcpParams | UpdateMcpParams) {
-  return {
-    name: params.name,
-    slug: params.slug,
-    slogan: params.slogan,
-    category: params.category,
-    icon: params.icon,
-    tags: params.tags,
-    transport: params.transport,
-    url: params.url,
-    command: params.command,
-    args: params.args,
-    env: params.env,
-    env_user_supplied: params.envUserSupplied,
-    headers: params.headers,
-    headers_user_supplied: params.headersUserSupplied,
-    tools: params.tools,
-    usage_examples: params.usageExamples,
-    faqs: params.faqs,
-    notes: params.notes,
-    visibility: params.visibility,
-  };
-}
-
 async function fetchMcpListReal(
   params: ListMcpParams
 ): Promise<ListMcpResponse> {

packages/dmworkbase/src/ui/channelSetting/GroupManagementView/index.tsx

@@ -0,0 +1,31 @@
+import { describe, expect, it } from "vitest";
+import { toWireParams } from "./mcpWireParams";
+import type { CreateMcpParams } from "../types/mcp";
+
+const form: CreateMcpParams = {
+  name: "GitHub MCP",
+  slug: "github-mcp",
+  category: "dev",
+  icon: "",
+  tags: [],
+  slogan: "Issues and pull requests",
+  transport: "streamable-http",
+  url: "https://mcp.example.com/github",
+  tools: [],
+  usageExamples: [],
+  faqs: [],
+  notes: [],
+};
+
+describe("toWireParams visibility", () => {
+  it("omits visibility from create and edit payloads", () => {
+    const payload = toWireParams(form);
+
+    expect(payload).not.toHaveProperty("visibility");
+    expect(payload).toMatchObject({
+      name: "GitHub MCP",
+      slug: "github-mcp",
+      transport: "streamable-http",
+    });
+  });
+});

packages/dmworkmcp/src/api/mcpService.ts

@@ -0,0 +1,24 @@
+import type { CreateMcpParams, UpdateMcpParams } from "../types/mcp";
+
+export function toWireParams(params: CreateMcpParams | UpdateMcpParams) {
+  return {
+    name: params.name,
+    slug: params.slug,
+    slogan: params.slogan,
+    category: params.category,
+    icon: params.icon,
+    tags: params.tags,
+    transport: params.transport,
+    url: params.url,
+    command: params.command,
+    args: params.args,
+    env: params.env,
+    env_user_supplied: params.envUserSupplied,
+    headers: params.headers,
+    headers_user_supplied: params.headersUserSupplied,
+    tools: params.tools,
+    usage_examples: params.usageExamples,
+    faqs: params.faqs,
+    notes: params.notes,
+  };
+}

packages/dmworkmcp/src/api/mcpService.visibility.test.ts

@@ -26,7 +26,6 @@ import type {
   McpFaq,
   McpProbeRequest,
   McpTransport,
-  McpVisibility,
 } from "../types/mcp";
 import { isImageIcon } from "../utils/icon";
 
@@ -97,7 +96,6 @@ const EMPTY: CreateMcpParams = {
   usageExamples: [],
   faqs: [],
   notes: [],
-  visibility: "public",
 };
 
 const TRANSPORT_OPTIONS: McpTransport[] = ["stdio", "streamable-http", "sse"];
@@ -194,7 +192,6 @@ function detailToForm(detail: McpDetail): CreateMcpParams {
     usageExamples: detail.usageExamples,
     faqs: detail.faqs,
     notes: detail.notes,
-    visibility: detail.visibility ?? "public",
   };
 }
 
@@ -991,11 +988,6 @@ const McpCreateModal: React.FC<McpCreateModalProps> = ({
     label: t(`mcp.create.transport.${tr}`),
   }));
 
-  const visibilitySegments = [
-    { value: "public" as McpVisibility, label: t("mcp.create.visPublic") },
-    { value: "private" as McpVisibility, label: t("mcp.create.visPrivate") },
-  ];
-
   // Preview src: the freshly-picked file's object URL wins; otherwise the
   // stored icon (a persisted storage URL, or a legacy base64 icon on edit).
   const iconSrc = iconPreview || form.icon;
@@ -1684,15 +1676,6 @@ const McpCreateModal: React.FC<McpCreateModalProps> = ({
               )}
             </Section>
 
-            {/* 7. 可见范围 */}
-            <Section full title={t("mcp.create.sectionVisibility")}>
-              <Segments
-                full
-                value={form.visibility}
-                options={visibilitySegments}
-                onChange={(v) => update("visibility", v)}
-              />
-            </Section>
           </>
         )}
         </>