dsh-stall-guard 0.1.3 → 0.1.4

package/CHANGELOG.md

@@ -5,6 +5,20 @@
 格式参考 [Keep a Changelog](https://keepachangelog.com/zh-CN/1.1.0/)，
 版本号遵循 [语义化版本](https://semver.org/lang/zh-CN/)。
 
+## [0.1.4] - 2026-10-21
+
+### 修复
+
+- **切到「浮动」后图标消失**：0.1.3 把浮动态拆到 overlay 渲染时，标题栏座位的守卫
+  `if (supervisorOff || floating) return null` 同时也把 overlay 座位挡掉了 ——
+  因为 overlay 座位收到的 `floating` 恒为 true。结果是图标彻底不画。
+  现在区分「这个座位要画的形态（forcedFloat）」与「用户选择的形态（wantsFloat）」：
+  标题栏在 `wantsFloat && !forcedFloat` 时让位，overlay 座位永不让位。
+- **两个座位可能同时画图标**：overlay 座位现在只在 `place === 'float'` 时渲染。
+- 外观状态改为共享 `useAppearance()` 钩子（`useSyncExternalStore`），
+  三个消费点（标题栏、浮动态、面板）对同一份记录同步重绘，
+  修掉面板改尺寸时图标不实时变化的问题。
+
 ## [0.1.3] - 2026-10-21
 
 ### 修复

package/client.js

@@ -1188,6 +1188,19 @@ window.__ModuleLoader__.load({
       }
     }
 
+    /**
+     * Subscribe to the appearance record.
+     *
+     * The record lives outside the polled snapshot, so it needs its own subscription. A
+     * shared hook keeps every consumer honest: an appearance change re-renders all of them.
+     */
+    function useAppearance() {
+      return useSyncExternalStore(
+        (listener) => store.subscribeAppearance(listener),
+        () => store.getAppearance(),
+      );
+    }
+
     function useTranslate(props) {
       return props.t !== undefined && typeof props.t === 'function' ? props.t : (boundT || ((key) => key));
     }
@@ -1208,11 +1221,10 @@ window.__ModuleLoader__.load({
       const snapshot = useStore();
       const orbRef = useRef(null);
       const [bubbleBelow, setBubbleBelow] = useState(false);
-      const [appearance, setAppearance] = useState(readAppearance);
+      const appearance = useAppearance();
       const [dragging, setDragging] = useState(false);
 
       useEffect(() => store.start(), []);
-      useEffect(() => store.subscribeAppearance(setAppearance), []);
 
       const config = snapshot.config;
       const supervisorOff = config !== null && config.supervisor.show === false;
@@ -1238,8 +1250,16 @@ window.__ModuleLoader__.load({
 
       /* Float placement: drag moves the control, and the position is clamped into the
        * viewport on release so a window resize cannot strand it off-screen. */
-      // The overlay seat forces the floating posture; the header seat reads the setting.
-      const floating = props.forcePlace === 'float' || appearance.place === 'float';
+      /**
+       * Which posture this render draws.
+       *
+       * The overlay seat forces floating; the header seat reads the user's setting. The two
+       * must be distinguished, because the header seat has to bail out when the chosen
+       * posture is floating — while the overlay seat must not bail out for the same reason.
+       */
+      const forcedFloat = props.forcePlace === 'float';
+      const wantsFloat = appearance.place === 'float';
+      const floating = forcedFloat || wantsFloat;
       const startDrag = useCallback((event) => {
         if (!floating) return;
         if (event.button !== undefined && event.button !== 0) return;
@@ -1270,10 +1290,11 @@ window.__ModuleLoader__.load({
         event.preventDefault();
       }, [floating]);
 
-      // Hidden, or drawn by the overlay because it floats: the header renders nothing in
-      // both cases. A floating figure inside the header's flow would push the header row
-      // around as it moved, which is exactly the bug this split fixes.
-      if (supervisorOff || floating) return null;
+      // The header seat renders nothing when the figure is hidden OR when the overlay owns
+      // it — a floating figure inside the header's flow would push that row around as it
+      // moved. The overlay seat asked for this render, so it never bails out here.
+      if (supervisorOff) return null;
+      if (wantsFloat && !forcedFloat) return null;
 
       const label = t('supervisor') + ' — ' + t(MODE_KEY[mode]);
       const size = appearance.size;
@@ -1354,9 +1375,13 @@ window.__ModuleLoader__.load({
     function FloatingSeat(props) {
       const t = useTranslate(props);
       const snapshot = useStore();
+      /* The overlay draws the figure only in the floating posture. Without this check both
+       * seats would render it, and the header would show a second copy. */
+      const appearance = useAppearance();
       useEffect(() => store.start(), []);
       if (snapshot.config === null) return null;
       if (snapshot.config.supervisor.show === false) return null;
+      if (appearance.place !== 'float') return null;
       return h(SupervisorSeat, { t, forcePlace: 'float' });
     }
 
@@ -1401,8 +1426,7 @@ window.__ModuleLoader__.load({
       const presetLocale = hostLocale() === 'en' ? 'en' : 'zh';
       /* Appearance is browser-local, so the panel mirrors it separately from the
        * Host draft; saving the config must not overwrite it and vice versa. */
-      const [localAppearance, setLocalAppearance] = useState(() => store.getAppearance());
-      useEffect(() => store.subscribeAppearance(setLocalAppearance), []);
+      const localAppearance = useAppearance();
       const [dirty, setDirty] = useState(false);
       const [status, setStatus] = useState({ kind: 'idle', text: '' });
 

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "dsh-stall-guard",
-  "version": "0.1.3",
+  "version": "0.1.4",
   "description": "DSH 卡顿守卫：下载类命令（git clone / pnpm install / pip / curl）超时后自动中止，并把「换个下载源重试」的提示写进工具结果给智能体看，解决 GitHub 在国内网络不稳导致智能体一直干等的问题。另可按工具类型自定义超时秒数，并附一个只在卡顿时唤醒的 DeepSeek 监督员。",
   "keywords": [
     "dsh",