@morehao/dsh-github-trending 0.7.2 → 0.7.3

package/CHANGELOG.md

@@ -1,5 +1,26 @@
 # Changelog
 
+## [v0.7.3] - 2026-11-01
+
+> 订阅页三处修复：待首次分析的仓库不再点出必败请求、添加订阅改弹窗、变更预览抽屉改紧凑 md 预览。
+
+### 修复
+
+- **单仓「触发分析」必败请求**：新添加、尚未跑过发版检查的仓库（`status=baseline`）在列表里也可点击「触发分析」，
+  但单仓分析走 `--stages releases` 不合并 pending，核心会报 `repo ... is not in the subscription list` 后立即失败。
+  现在 `baseline` 行的「触发分析」禁用并提示「待首次分析后才能触发」；分析失败时把核心「未纳入订阅 / 无发版记录」
+  类错误翻成一句人话，错误条稳定可见，不再一闪而过。
+
+### 变更
+
+- **添加订阅改为居中模态弹窗**：工具条内联输入框改为弹窗（输入 owner/repo + 实时校验 + 回车提交 + Esc/遮罩关闭）；
+  添加失败就地显示错误。
+
+### 改进
+
+- **变更预览抽屉**：正文包一层纸面卡（不再直接贴页面底色），移植原型紧凑 markdown 排版（h1 15 / h2 13 带强调左边框），
+  不再复用日报的大号样式；抽屉遮罩加深 + 模糊、阴影加强，观感与设计原型一致。
+
 ## [v0.7.2] - 2026-11-01
 
 > 依赖升级：`@morehao/dsh-plugin-kit` 0.1.0 → **0.2.1**。

package/client.js

@@ -472,6 +472,8 @@ var DICT = {
     "subs.meta.thisWeek": "本周 {from} ~ {to} · {changed} 个仓库有变化",
     "subs.meta.lastCheck": "上次发版检查 {time}",
     "subs.add": "添加",
+    "subs.add.title": "添加订阅仓库",
+    "subs.add.exists": "{repo} 已在订阅列表中",
     "subs.add.placeholder": "denoland/deno",
     "subs.add.confirm": "添加",
     "subs.adding": "添加中…",
@@ -485,6 +487,7 @@ var DICT = {
     "subs.row.changesRepo": "查看 {repo} 的变更",
     "subs.row.trigger": "触发分析",
     "subs.row.triggerRepo": "只对 {repo} 生成变更分析",
+    "subs.row.triggerFresh": "待首次分析后才能触发",
     "subs.search.placeholder": "搜索仓库",
     "subs.search.label": "搜索仓库",
     "subs.toolbar.add": "＋ 添加仓库",
@@ -521,7 +524,7 @@ var DICT = {
     "subs.confirm.cancel": "取消",
     "subs.confirm.ok": "确认移除",
     "subs.empty": "还没有订阅任何仓库",
-    "subs.empty.hint": "在上面的输入框里输入 owner/repo，回车添加。",
+    "subs.empty.hint": "点「＋ 添加仓库」输入 owner/repo 添加。",
     "subs.empty.match": "没有匹配的仓库",
     "subs.empty.match.hint": "按 Esc 清空搜索，或换一个 owner / 名字。",
     "subs.empty.clear": "清除搜索",
@@ -994,6 +997,8 @@ var DICT = {
     "subs.meta.thisWeek": "This week {from} ~ {to} · {changed} repos changed",
     "subs.meta.lastCheck": "Last release check {time}",
     "subs.add": "Add",
+    "subs.add.title": "Add a repository",
+    "subs.add.exists": "{repo} is already subscribed",
     "subs.add.placeholder": "denoland/deno",
     "subs.add.confirm": "Add",
     "subs.adding": "Adding…",
@@ -1007,6 +1012,7 @@ var DICT = {
     "subs.row.changesRepo": "Show changes for {repo}",
     "subs.row.trigger": "Analyze",
     "subs.row.triggerRepo": "Generate change analysis for {repo} only",
+    "subs.row.triggerFresh": "Available after the first analysis",
     "subs.search.placeholder": "Search repositories",
     "subs.search.label": "Search repositories",
     "subs.toolbar.add": "＋ Add repository",
@@ -1043,7 +1049,7 @@ var DICT = {
     "subs.confirm.cancel": "Cancel",
     "subs.confirm.ok": "Remove",
     "subs.empty": "No repos subscribed yet",
-    "subs.empty.hint": "Type owner/repo in the field above and press Enter to add one.",
+    "subs.empty.hint": 'Click "＋ Add repository" and enter owner/repo to add one.',
     "subs.empty.match": "No matching repos",
     "subs.empty.match.hint": "Press Esc to clear the search, or try another owner / name.",
     "subs.empty.clear": "Clear search",
@@ -1462,9 +1468,6 @@ var CSS = `
 .dshgt-subs-search input:focus{outline:2px solid color-mix(in srgb,var(--gt-accent) 30%,transparent);outline-offset:1px;border-color:color-mix(in srgb,var(--gt-accent) 45%,var(--gt-border));}
 .dshgt-subs-search input::placeholder{color:var(--gt-muted);}
 .dshgt-subs-toolbar-spacer{flex:1 1 auto;min-width:0;}
-.dshgt-subs-addform{display:flex;align-items:center;gap:6px;}
-.dshgt-subs-addform input{height:32px;box-sizing:border-box;width:200px;padding:0 9px;border:1px solid var(--gt-border);border-radius:8px;background:var(--gt-bg);color:var(--gt-fg);font:inherit;font-size:12.5px;font-family:ui-monospace,SFMono-Regular,Menlo,monospace;}
-.dshgt-subs-addform input:focus{outline:2px solid color-mix(in srgb,var(--gt-accent) 30%,transparent);outline-offset:1px;}
 .dshgt-subs-act.gen{color:var(--gt-muted);}
 .dshgt-subs-act.gen:hover{color:var(--gt-accent);background:color-mix(in srgb,var(--gt-accent) 10%,transparent);}
 .dshgt-subs-act.gen:disabled{opacity:.5;cursor:not-allowed;}
@@ -1528,8 +1531,8 @@ var CSS = `
 .dshgt-subs-linkbtn{appearance:none;border:0;background:none;font:inherit;font-size:12.5px;color:var(--gt-accent);cursor:pointer;padding:2px 4px;border-radius:5px;}
 .dshgt-subs-linkbtn:hover{text-decoration:underline;}
 .dshgt-subs-confirm-list{list-style:none;margin:0;padding:0;display:flex;flex-direction:column;gap:2px;font-family:ui-monospace,SFMono-Regular,Menlo,monospace;font-size:12px;color:var(--gt-muted);}
-.dshgt-subs-drawer-mask{position:fixed;inset:0;z-index:2147483000;display:flex;justify-content:flex-end;background:color-mix(in srgb,var(--gt-fg) 26%,transparent);}
-.dshgt-subs-drawer{width:460px;max-width:100%;height:100%;display:flex;flex-direction:column;background:var(--gt-bg);border-left:1px solid var(--gt-border);box-shadow:-24px 0 60px color-mix(in srgb,var(--gt-fg) 16%,transparent);}
+.dshgt-subs-drawer-mask{position:fixed;inset:0;z-index:2147483000;display:flex;justify-content:flex-end;background:color-mix(in srgb,var(--gt-fg) 34%,transparent);backdrop-filter:blur(1.5px);}
+.dshgt-subs-drawer{width:460px;max-width:100%;height:100%;display:flex;flex-direction:column;background:var(--gt-bg);border-left:1px solid var(--gt-border);box-shadow:-30px 0 70px rgba(15,23,42,.3);}
 .dshgt-subs-drawer-head{flex:none;display:flex;align-items:flex-start;gap:10px;padding:16px 20px 12px;border-bottom:1px solid var(--gt-border);}
 .dshgt-subs-drawer-main{flex:1;min-width:0;}
 .dshgt-subs-drawer-repo{font-family:ui-monospace,SFMono-Regular,Menlo,monospace;font-size:13px;font-weight:600;word-break:break-all;}
@@ -1541,6 +1544,26 @@ var CSS = `
 .dshgt-subs-vol .label{font-family:ui-monospace,SFMono-Regular,Menlo,monospace;}
 .dshgt-subs-drawer-note{margin-top:auto;padding-top:12px;border-top:1px solid var(--gt-border);font-size:11.5px;color:var(--gt-muted);line-height:1.65;word-break:break-word;}
 .dshgt-subs-drawer-raw{margin:0;padding:12px 14px;border-radius:8px;background:color-mix(in srgb,var(--gt-fg) 5%,var(--gt-card));font-family:ui-monospace,SFMono-Regular,Menlo,monospace;font-size:12px;line-height:1.65;white-space:pre-wrap;overflow-wrap:break-word;}
+/* 抽屉正文纸面：md 铺在卡面上（与日报预览 .dshgt-digest-sheet 同一观感），不再直接贴页面底色。 */
+.dshgt-subs-drawer-doc{border:1px solid var(--gt-border);border-radius:10px;background:var(--gt-card);padding:14px 16px;}
+.dshgt-subs-drawer-doc .dshgt-subs-drawer-raw{background:transparent;padding:0;}
+/* 抽屉专用紧凑 md 排版：覆盖日报的大号 .dshgt-md（h1 24 / h2 18），保留结构其余样式。 */
+.dshgt-subs-drawer-doc .dshgt-md{font-size:12.5px;line-height:1.7;}
+.dshgt-subs-drawer-doc .dshgt-md-h1{font-size:15px;margin:0 0 4px;}
+.dshgt-subs-drawer-doc .dshgt-md-h2{font-size:13px;margin:14px 0 6px;padding-left:8px;border-left:3px solid var(--gt-accent);}
+.dshgt-subs-drawer-doc .dshgt-md-h3{font-size:12.5px;}
+.dshgt-subs-drawer-doc .dshgt-md-h4,.dshgt-subs-drawer-doc .dshgt-md-h5,.dshgt-subs-drawer-doc .dshgt-md-h6{font-size:12px;}
+.dshgt-subs-drawer-doc .dshgt-md-p{font-size:12.5px;margin:6px 0;}
+.dshgt-subs-drawer-doc .dshgt-md-list{font-size:12.5px;padding-left:18px;}
+.dshgt-subs-drawer-doc .dshgt-md-list li{margin:3px 0;}
+.dshgt-subs-drawer-doc .dshgt-md-quote{font-size:12.5px;}
+.dshgt-subs-drawer-doc .dshgt-md-table{min-width:0;font-size:12px;}
+.dshgt-subs-drawer-doc .dshgt-md-pre{font-size:11.5px;padding:10px 12px;}
+.dshgt-subs-drawer-doc .dshgt-md-code{font-size:11.5px;}
+/* 添加订阅居中小弹窗 */
+.dshgt-addmodal{max-width:420px;}
+.dshgt-addmodal-input{height:34px;width:100%;}
+.dshgt-addmodal-err{color:var(--gt-err);}
 @media (max-width:1040px){
   .dshgt-subs-thead,.dshgt-subs-row{grid-template-columns:240px 120px 110px minmax(0,1fr);}
   .dshgt-subs-table.is-selecting .dshgt-subs-thead,.dshgt-subs-table.is-selecting .dshgt-subs-row{grid-template-columns:22px 240px 120px 110px minmax(0,1fr);}
@@ -4471,6 +4494,9 @@ var DATE_RE2 = /^\d{4}-\d{2}-\d{2}$/;
 function errorText(failure) {
   return failure instanceof Error ? failure.message : String(failure);
 }
+function isNotReadyMessage(message) {
+  return /is not in the subscription list|has no release document|no release record|not subscribed/i.test(message);
+}
 function monthDay(value) {
   if (value === void 0 || value === "") return "—";
   return DATE_RE2.test(value) ? value.slice(5) : value;
@@ -4597,7 +4623,7 @@ function DocDrawer(props) {
     { kind: "err", title: t("subs.drawer.title", { repo }) },
     h("div", null, error),
     h("div", { style: { marginTop: 8 } }, h(Btn, { small: true, onClick: () => setAttempt((current2) => current2 + 1) }, t("subs.drawer.retry")))
-  ) : parseFailed || nodes === null ? h("pre", { className: "dshgt-subs-drawer-raw" }, doc?.markdown ?? "") : h("div", { className: "dshgt-md" }, nodes);
+  ) : parseFailed || nodes === null ? h("div", { className: "dshgt-subs-drawer-doc" }, h("pre", { className: "dshgt-subs-drawer-raw" }, doc?.markdown ?? "")) : h("div", { className: "dshgt-subs-drawer-doc" }, h("div", { className: "dshgt-md" }, nodes));
   return h(
     "div",
     {
@@ -4688,6 +4714,7 @@ function SubscriptionsView(props) {
   const [openRepo, setOpenRepo] = import_react.default.useState(null);
   const [addOpen, setAddOpen] = import_react.default.useState(false);
   const [addValue, setAddValue] = import_react.default.useState("");
+  const [addError, setAddError] = import_react.default.useState(null);
   const [analyzing, setAnalyzing] = import_react.default.useState(null);
   const [result, setResult] = import_react.default.useState(null);
   const load = import_react.default.useCallback((signal) => {
@@ -4717,12 +4744,13 @@ function SubscriptionsView(props) {
     setBusy("add");
     setError(null);
     setNote(null);
+    setAddError(null);
     postSubscriptions("add", [repo]).then((next) => {
       setList(next);
       setAddValue("");
       setAddOpen(false);
       setNote(t("subs.add.queued", { repo }));
-    }).catch((failure) => setError(errorText(failure))).finally(() => setBusy(null));
+    }).catch((failure) => setAddError(errorText(failure))).finally(() => setBusy(null));
   };
   const removeOne = (repo) => {
     setBusy(`remove:${repo}`);
@@ -4757,11 +4785,13 @@ function SubscriptionsView(props) {
     postReleasesRun(repo).then((record) => {
       setResult({ scope: repo === void 0 ? "all" : "one", ...repo === void 0 ? {} : { repo }, ok: record.status !== "failed", changed: record.changed });
       if (record.status === "failed" && record.errorMessage !== void 0) {
-        setError(t("subs.analyze.failed", { message: record.errorMessage }));
+        const notReady = isNotReadyMessage(record.errorMessage);
+        setError(notReady ? t("subs.row.triggerFresh") : t("subs.analyze.failed", { message: record.errorMessage }));
       }
       load();
     }).catch((failure) => {
-      setError(errorText(failure));
+      const message = errorText(failure);
+      setError(isNotReadyMessage(message) ? t("subs.row.triggerFresh") : message);
       setResult({ scope: repo === void 0 ? "all" : "one", ...repo === void 0 ? {} : { repo }, ok: false, changed: 0 });
     }).finally(() => setAnalyzing(null));
   };
@@ -4798,6 +4828,7 @@ function SubscriptionsView(props) {
     const item = row.item;
     const tone = toneOf(item?.status);
     const rowAnalyzing = analyzing !== null && analyzing.scope === "one" && analyzing.repo === row.name;
+    const fresh = item?.status === "baseline";
     const checkbox = selecting ? h(
       "span",
       { key: "cb", role: "cell", className: "dshgt-subs-cbwrap" },
@@ -4843,12 +4874,12 @@ function SubscriptionsView(props) {
           {
             type: "button",
             className: "dshgt-subs-act gen",
-            "aria-label": t("subs.row.triggerRepo", { repo: row.name }),
-            title: t("subs.row.triggerRepo", { repo: row.name }),
-            disabled: globalBusy || busy !== null,
+            "aria-label": fresh ? t("subs.row.triggerFresh") : t("subs.row.triggerRepo", { repo: row.name }),
+            title: fresh ? t("subs.row.triggerFresh") : t("subs.row.triggerRepo", { repo: row.name }),
+            disabled: globalBusy || busy !== null || fresh,
             onClick: (event) => {
               event.stopPropagation();
-              runAnalyze(row.name);
+              if (!fresh) runAnalyze(row.name);
             }
           },
           t("subs.row.trigger")
@@ -4936,33 +4967,7 @@ function SubscriptionsView(props) {
         })
       ),
       h("div", { className: "dshgt-subs-toolbar-spacer" }),
-      addOpen ? h(
-        "div",
-        { className: "dshgt-subs-addform" },
-        h("input", {
-          type: "text",
-          value: addValue,
-          placeholder: t("subs.add.placeholder"),
-          "aria-label": t("subs.add.placeholder"),
-          autoComplete: "off",
-          spellCheck: false,
-          autoFocus: true,
-          onInput: (event) => setAddValue(event.target.value),
-          onKeyDown: (event) => {
-            if (event.isComposing === true) return;
-            if (event.key === "Enter" && canAdd && !alreadySubscribed) addRepo(addValue.trim());
-            if (event.key === "Escape") {
-              setAddOpen(false);
-              setAddValue("");
-            }
-          }
-        }),
-        h(
-          Btn,
-          { small: true, variant: "primary", disabled: busy !== null || !canAdd || alreadySubscribed, onClick: () => canAdd && !alreadySubscribed && addRepo(addValue.trim()) },
-          t("subs.add.confirm")
-        )
-      ) : h(
+      h(
         Btn,
         {
           small: true,
@@ -4970,6 +4975,7 @@ function SubscriptionsView(props) {
           onClick: () => {
             setAddOpen(true);
             setAddValue("");
+            setAddError(null);
           }
         },
         t("subs.toolbar.add")
@@ -5065,6 +5071,76 @@ function SubscriptionsView(props) {
         )
       )
     ),
+    addOpen ? h(
+      "div",
+      {
+        className: "dshgt-modal-mask",
+        onClick: (event) => {
+          if (event.target === event.currentTarget) {
+            setAddOpen(false);
+            setAddValue("");
+            setAddError(null);
+          }
+        }
+      },
+      h(
+        "div",
+        { className: "dshgt-modal dshgt-addmodal", role: "dialog", "aria-modal": "true", "aria-label": t("subs.add.title") },
+        h("h3", { style: { margin: 0, fontSize: 14 } }, t("subs.add.title")),
+        h("input", {
+          type: "text",
+          className: "dshgt-input dshgt-addmodal-input",
+          value: addValue,
+          placeholder: t("subs.add.placeholder"),
+          "aria-label": t("subs.add.placeholder"),
+          autoComplete: "off",
+          spellCheck: false,
+          autoFocus: true,
+          onInput: (event) => {
+            setAddValue(event.target.value);
+            if (addError !== null) setAddError(null);
+          },
+          onKeyDown: (event) => {
+            if (event.isComposing === true) return;
+            if (event.key === "Enter" && canAdd && !alreadySubscribed && busy === null) addRepo(addValue.trim());
+            if (event.key === "Escape") {
+              setAddOpen(false);
+              setAddValue("");
+              setAddError(null);
+            }
+          }
+        }),
+        addValue.trim() !== "" && !canAdd ? h("p", { className: "dshgt-hint dshgt-addmodal-err", style: { margin: 0 } }, t("subs.input.invalid")) : null,
+        alreadySubscribed ? h("p", { className: "dshgt-hint", style: { margin: 0 } }, t("subs.add.exists", { repo: addValue.trim() })) : null,
+        addError === null ? null : h("p", { className: "dshgt-hint dshgt-addmodal-err", style: { margin: 0 } }, addError),
+        h(
+          "div",
+          { className: "dshgt-footer", style: { justifyContent: "flex-end" } },
+          h(
+            Btn,
+            {
+              small: true,
+              onClick: () => {
+                setAddOpen(false);
+                setAddValue("");
+                setAddError(null);
+              }
+            },
+            t("subs.cancel")
+          ),
+          h(
+            Btn,
+            {
+              small: true,
+              variant: "primary",
+              disabled: busy !== null || !canAdd || alreadySubscribed,
+              onClick: () => canAdd && !alreadySubscribed && addRepo(addValue.trim())
+            },
+            busy === "add" ? t("subs.adding") : t("subs.add.confirm")
+          )
+        )
+      )
+    ) : null,
     openRepo === null ? null : h(DocDrawer, {
       key: openRepo,
       t,

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@morehao/dsh-github-trending",
-  "version": "0.7.2",
+  "version": "0.7.3",
   "description": "GitHub 趋势日报插件：每天定时 spawn 本地 trendradar 核心，抓取 Trending、生成中文简介与总结，提交到你的仓库并自动合并 PR。",
   "type": "module",
   "main": "lib/index.js",