@michengai/dsh-skills-manager 0.1.6 → 0.1.7

package/lib/client.js

@@ -84,6 +84,12 @@ window.__ModuleLoader__.load({
         "error.import.invalidName": "无法生成合法 kebab-case 名称（原始名: {name}）",
         "error.import.duplicateName": "批量来源中存在多个同名插件: {name}",
         "error.import.failed": "导入失败",
+        "error.proto.forbidden": "禁止的修改请求（缺少客户端标记）",
+        "error.proto.contentType": "请求体必须是 application/json",
+        "error.proto.method": "不支持的请求方法",
+        "error.proto.unknownAction": "未知操作",
+        "error.proto.bodyTooLarge": "请求体过大",
+        "error.proto.invalidJson": "请求体不是合法 JSON",
         "action.enable": "启用",
         "action.disable": "停用",
         "action.delete": "删除",
@@ -160,6 +166,12 @@ window.__ModuleLoader__.load({
         "error.import.invalidName": "Cannot generate a valid kebab-case name (original name: {name})",
         "error.import.duplicateName": "Batch source contains multiple plugins with the same name: {name}",
         "error.import.failed": "Import failed",
+        "error.proto.forbidden": "Forbidden mutation request (missing client marker)",
+        "error.proto.contentType": "Content type must be application/json",
+        "error.proto.method": "Method not allowed",
+        "error.proto.unknownAction": "Unknown action",
+        "error.proto.bodyTooLarge": "Request body too large",
+        "error.proto.invalidJson": "Invalid JSON request body",
         "action.enable": "enable",
         "action.disable": "disable",
         "action.delete": "deleting",
@@ -296,12 +308,12 @@ window.__ModuleLoader__.load({
       function selectFile(file) {
         if (!file) return;
         if (file.name.toLowerCase() !== "skill.md") {
-          setImportResult({ error: t("select.file.invalid") });
+          setImportResult({ code: "select.file.invalid" });
           return;
         }
         if (!file.path) {
           setSelected(null);
-          setImportResult({ error: t("select.path.missing") });
+          setImportResult({ code: "select.path.missing" });
           return;
         }
         setSelected({ name: file.name, source: file.path });
@@ -310,7 +322,7 @@ window.__ModuleLoader__.load({
 
       function selectDirectory() {
         if (!pickDirectory) {
-          setImportResult({ error: t("dir.service.missing") });
+          setImportResult({ code: "dir.service.missing" });
           return;
         }
         setImportResult(null);
@@ -318,12 +330,12 @@ window.__ModuleLoader__.load({
         pickDirectory().then(function (path) {
           setBusy(null);
           if (path) {
-            setSelected({ name: t("dir.selected"), source: path });
+            setSelected({ nameKey: "dir.selected", source: path });
             setImportResult(null);
           }
         }).catch(function (error) {
           setBusy(null);
-          setImportResult({ error: t("dir.pick.failed", { error: String(error && error.message || error) }) });
+          setImportResult({ code: "dir.pick.failed", params: { error: String(error && error.message || error) } });
         });
       }
 
@@ -403,13 +415,15 @@ window.__ModuleLoader__.load({
       if (importResult) {
         var nameSeparator = t("sep.names");
         var errorSeparator = t("sep.errors");
+        // 错误形态：业务/协议错误带 code，或 host 失败带 error（Error 对象）；结果形态只有 imported/failed/skipped。
+        var isImportError = importResult.code !== undefined || importResult.error !== undefined;
         var importedNames = (importResult.imported || []).map(function (item) { return item.name; });
         // catch 场景把 core 的失败明细挂在 Error.failed 上，这里一并取用。
         var failedItems = importResult.failed || (importResult.error && importResult.error.failed) || [];
         var skippedNames = (importResult.skipped || []).map(function (item) { return item.name; });
         var failedText = failedItems.map(function (item) { return translateError(t, item); }).join(errorSeparator);
         var resultText;
-        if (importResult.error) {
+        if (isImportError) {
           // 顶层聚合错误优先展示已翻译的失败明细；无明细时由 translateError 兜底（含 Error 对象）。
           resultText = t("result.failed", { error: failedItems.length ? failedText : translateError(t, importResult) });
         } else if (failedItems.length) {
@@ -421,16 +435,16 @@ window.__ModuleLoader__.load({
         } else {
           resultText = t("result.none");
         }
-        nodes.push(h("div", { key: "import-result", className: importResult.error || failedItems.length ? "dssm-error" : "dssm-note", role: "status" }, resultText));
+        nodes.push(h("div", { key: "import-result", className: isImportError || failedItems.length ? "dssm-error" : "dssm-note", role: "status" }, resultText));
       }
       if (uploadOpen) {
         nodes.push(h("div", { key: "upload", className: "dssm-mask" }, h("div", { className: "dssm-modal", role: "dialog", "aria-modal": "true", "aria-labelledby": "dssm-upload-title" },
           h("div", { className: "dssm-modal-head" }, h("h3", { id: "dssm-upload-title", className: "dssm-modal-title" }, t("upload.title")), h("button", { className: "dssm-icon-btn", "aria-label": t("upload.close"), onClick: function () { setUploadOpen(false); } }, "×")),
           h("input", { ref: inputRef, className: "dssm-hidden-input", type: "file", accept: ".md", onChange: function (event) { selectFile(event.target.files && event.target.files[0]); event.target.value = ""; } }),
           h("div", { className: "dssm-dropzone", tabIndex: 0, onClick: openNativePicker, onKeyDown: function (event) { if (event.key === "Enter" || event.key === " ") { event.preventDefault(); openNativePicker(); } }, onDragOver: function (event) { event.preventDefault(); }, onDrop: function (event) { event.preventDefault(); selectFile(event.dataTransfer.files && event.dataTransfer.files[0]); } }, h("span", { className: "dssm-dropzone-title" }, t("upload.drop.title")), h("span", { className: "dssm-dropzone-copy" }, t("upload.drop.copy"))),
-          selected ? h("div", { className: "dssm-file", title: selected.source }, h("span", { className: "dssm-file-name" }, selected.name), h("span", { className: "dssm-tag dssm-tag-on" }, t("status.selected"))) : h("p", { className: "dssm-desc" }, t("upload.hint")),
+          selected ? h("div", { className: "dssm-file", title: selected.source }, h("span", { className: "dssm-file-name" }, selected.nameKey ? t(selected.nameKey) : selected.name), h("span", { className: "dssm-tag dssm-tag-on" }, t("status.selected"))) : h("p", { className: "dssm-desc" }, t("upload.hint")),
           busy === "pick-directory" ? h("div", { className: "dssm-note", role: "status" }, t("upload.picking.dir")) : null,
-          importResult && importResult.error ? h("div", { className: "dssm-error", role: "alert" }, t("select.failed", { error: translateError(t, importResult) })) : null,
+          importResult && (importResult.code !== undefined || importResult.error !== undefined) ? h("div", { className: "dssm-error", role: "alert" }, t("select.failed", { error: translateError(t, importResult) })) : null,
           h("div", { className: "dssm-modal-actions" }, h("button", { className: "dssm-btn dssm-btn-secondary", disabled: !!busy, onClick: openNativePicker }, t("btn.file.pick")), pickDirectory ? h("button", { className: "dssm-btn dssm-btn-secondary", disabled: !!busy, onClick: selectDirectory }, t("btn.dir.pick")) : null, h("button", { className: "dssm-btn", disabled: !selected || !!busy, onClick: installSelected }, busy === "import" || busy === "import-check" ? t("btn.uploading") : t("btn.upload.dsh"))))));
       }
       if (confirmImport) {

package/lib/core.js

@@ -29,9 +29,9 @@ function codedError(message, code, params) {
   return error;
 }
 
-/** 把业务 Error 的 code/params 附加到失败明细；系统异常（无 code）保持原文。 */
+/** 把业务 Error 的 code/params 附加到失败明细；系统异常（ENOENT 等，非 error.* 前缀）保持原文。 */
 function attachCode(item, error) {
-  if (error && typeof error.code === "string") item.code = error.code;
+  if (error && typeof error.code === "string" && /^error\./.test(error.code)) item.code = error.code;
   if (error && error.params) item.params = error.params;
   return item;
 }

package/lib/index.js

@@ -47,6 +47,7 @@ function readBody(req, limit = 1 << 20) {
       if (size > limit) {
         const error = new Error("body too large");
         error.statusCode = 413;
+        error.code = "error.proto.bodyTooLarge";
         fail(error);
         req.resume();
         return;
@@ -63,6 +64,7 @@ function readBody(req, limit = 1 << 20) {
       } catch (e) {
         const error = new Error(`invalid JSON body: ${e.message}`);
         error.statusCode = 400;
+        error.code = "error.proto.invalidJson";
         fail(error);
       }
     });
@@ -72,9 +74,9 @@ function readBody(req, limit = 1 << 20) {
 
 /** 自定义请求头使跨站 fetch 必须预检；本地接口不提供 CORS 响应。 */
 function validateMutationRequest(req) {
-  if (req.headers[CLIENT_MARKER_HEADER] !== "1") return { statusCode: 403, error: "forbidden mutation request" };
+  if (req.headers[CLIENT_MARKER_HEADER] !== "1") return { statusCode: 403, code: "error.proto.forbidden", error: "forbidden mutation request" };
   const contentType = String(req.headers["content-type"] || "").split(";", 1)[0].trim().toLowerCase();
-  if (contentType !== "application/json") return { statusCode: 415, error: "content-type must be application/json" };
+  if (contentType !== "application/json") return { statusCode: 415, code: "error.proto.contentType", error: "content-type must be application/json" };
   return null;
 }
 
@@ -134,7 +136,7 @@ function apply(ctx) {
           return;
         }
         if (req.method !== "POST") {
-          json(res, 405, { ok: false, error: `method not allowed: ${req.method}` });
+          json(res, 405, { ok: false, code: "error.proto.method", error: `method not allowed: ${req.method}` });
           return;
         }
         const requestError = validateMutationRequest(req);
@@ -163,10 +165,14 @@ function apply(ctx) {
             }), body.dryRun === true ? undefined : invalidateSkills);
             return;
           default:
-            json(res, 404, { ok: false, error: `unknown action: ${path}` });
+            json(res, 404, { ok: false, code: "error.proto.unknownAction", error: `unknown action: ${path}` });
         }
       } catch (e) {
-        json(res, Number.isInteger(e && e.statusCode) ? e.statusCode : 500, { ok: false, error: String(e && e.message ? e.message : e) });
+        json(res, Number.isInteger(e && e.statusCode) ? e.statusCode : 500, {
+          ok: false,
+          ...(e && e.code ? { code: e.code } : {}),
+          error: String(e && e.message ? e.message : e),
+        });
       }
     },
   });

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@michengai/dsh-skills-manager",
-  "version": "0.1.6",
+  "version": "0.1.7",
   "description": "NPM-installable DSH Web plugin for managing local skills and viewing shared Agent skills safely.",
   "license": "Apache-2.0",
   "repository": {