@addozhang/dsh-discord 0.2.0 → 0.2.1

package/README.md

@@ -98,7 +98,6 @@ The settings card exposes the three high-frequency fields (guild allowlist, auto
 | `/steer`, `/stop` | session thread | steer or cancel the running turn (owner only) |
 | `/guild forget` | any channel | operator-only removal of adapter records |
 
-`/model` remains registered ahead of its interactive provider → model → thinking/reasoning cascade (next milestone); `/preset`, `/skill`, and `/host` are deregistered until the router wires them — see Known limitations.
 
 ## Design notes
 

package/README.zh.md

@@ -98,7 +98,6 @@ dsh-discord:
 | `/steer`, `/stop` | 会话线程 | 插话或取消运行中的 Turn（仅属主） |
 | `/guild forget` | 任意频道 | 仅操作员：移除适配器记录 |
 
-`/model` 保持注册，等待其交互式 provider → 模型 → thinking/reasoning 级联（下个里程碑）；`/preset`、`/skill`、`/host` 已取消注册，待路由接线后回归——见已知限制。
 
 ## 设计说明
 

package/lib/features/ask-wiring.d.ts

@@ -55,6 +55,15 @@ export interface AskWiringDeps {
         reason?: string;
         messageId?: string;
     }>;
+    /**
+     * Edit an already-posted control message (PATCH). Retirement greys the
+     * controls out on the ORIGINAL message — a POST would only add a new,
+     * empty message while the live buttons stayed behind (16.41).
+     */
+    editMessage(channelId: string, messageId: string, payload: unknown): Promise<{
+        stored: boolean;
+        reason?: string;
+    }>;
 }
 export declare function createAskWiring(deps: AskWiringDeps): {
     /** Retire a rendered control (clear components) once the ask is settled. */

package/lib/features/ask-wiring.js

@@ -14,8 +14,16 @@ import { renderQuestionControls } from './question-view.js';
 import { abandonUnrenderableQuestion } from './question-expiry.js';
 import { handleRemoteResolution } from './question-routing.js';
 export function createAskWiring(deps) {
-    /** Discord message refs for rendered controls, keyed by ask id. */
+    /** Discord message refs + rendered rows for controls, keyed by ask id. */
     const controlMessages = new Map();
+    function disabledComponents(components) {
+        if (!Array.isArray(components))
+            return [];
+        return components.map(row => ({
+            ...row,
+            components: (row.components ?? []).map(control => ({ ...control, disabled: true })),
+        }));
+    }
     const disableControl = async (key) => {
         const target = controlMessages.get(key);
         if (target === undefined)
@@ -24,9 +32,11 @@ export function createAskWiring(deps) {
         // Retirement is best-effort: a failed disable must never fail the ask's
         // settled outcome — the registry TTL keeps stray clicks answerable.
         try {
-            const posted = await deps.postMessage(target.channelId, { components: [] });
-            if (!posted.stored)
-                deps.log('discord_control_disable_failed', { key });
+            const patched = await deps.editMessage(target.channelId, target.messageId, {
+                components: disabledComponents(target.components),
+            });
+            if (!patched.stored)
+                deps.log('discord_control_disable_failed', { key, reason: patched.reason });
         }
         catch (cause) {
             deps.log('discord_control_disable_failed', { key, cause: String(cause) });
@@ -77,7 +87,7 @@ export function createAskWiring(deps) {
             void deps.postMessage(input.threadId, payload)
                 .then(sent => {
                 if (sent.stored && sent.messageId !== undefined) {
-                    controlMessages.set(input.approvalId, { channelId: input.threadId, messageId: sent.messageId });
+                    controlMessages.set(input.approvalId, { channelId: input.threadId, messageId: sent.messageId, components: payload.components });
                 }
             })
                 .catch((cause) => { deps.log('discord_approval_render_failed', String(cause)); });
@@ -136,7 +146,7 @@ export function createAskWiring(deps) {
                     abandonQuestion(sent.reason ?? 'post rejected');
                     return;
                 }
-                controlMessages.set(input.rpcId, { channelId: input.threadId, messageId: sent.messageId });
+                controlMessages.set(input.rpcId, { channelId: input.threadId, messageId: sent.messageId, components: payload.components });
             })
                 .catch((cause) => { abandonQuestion(String(cause)); });
         },

package/lib/index.js

@@ -225,6 +225,21 @@ export function apply(ctx, config = DEFAULT_DISCORD_SETTINGS) {
                     return { stored: false, reason: `HTTP ${String(sent.status)}: response missing message id` };
                 return { stored: true, messageId: sent.body.id };
             },
+            editMessage: async (channelId, messageId, payload) => {
+                const rest = await sharedRest();
+                if (rest === undefined)
+                    return { stored: false, reason: 'rest unavailable' };
+                const patched = await rest.request('PATCH', `/channels/${channelId}/messages/${messageId}`, payload);
+                if (patched.outcome !== 'completed') {
+                    return {
+                        stored: false,
+                        reason: patched.outcome === 'rejected'
+                            ? `HTTP ${String(patched.status)} ${patched.error.message}`
+                            : patched.reason,
+                    };
+                }
+                return { stored: true };
+            },
         });
         const componentFollowUp = async (_interactionId, interactionToken, content) => {
             const rest = await sharedRest();

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@addozhang/dsh-discord",
-  "version": "0.2.0",
+  "version": "0.2.1",
   "description": "Discord-first adapter for DeepSeek Harness",
   "license": "MIT",
   "type": "module",