dsh-connect-minimaxcode 0.6.7 → 0.6.8

package/README.en.md

@@ -384,6 +384,38 @@ The OAuth token is valid for about an hour and is refreshed automatically by the
 
 ## Changelog
 
+### 0.6.8 (2026-11-13)
+
+
+- **The plan tier moved into the Plan box**: the tier and the day it ends used to be a row of their own below the two boxes, stretched into a band of empty space. The tier is now the Plan box's headline line, wearing **the balance's own big-number size and height** (20px), and the day it ends stays the small text it was, on the right and on the tier's baseline; the box itself is named "Plan" rather than "Plan allowance".
+- **The grid has no hole any more**: the tier row no longer occupies a grid cell, so the check-in and the expiring grants now sit side by side in the second row and both columns are filled — the empty band under the left column and the hole at the bottom right are gone together. The Plan box did not grow by a row either: the tier line takes the place of the invisible number that used to hold the row height open.
+- The tier is still **the value the upstream sends** (`token_plan_tier`), not a fixed label: another plan shows its own name, and only an answer without that field falls back to "M Plan subscription".
+
+
+- **Fixed "Could not send it to the conversation": the session method was never bound.** Before this release the composer's **Generate an image** entry failed *every* time, reporting "Could not send it to the conversation; generate on the plugin page and download it instead." The plugin took `session.prompt` off the session object and then called it as a standalone function:
+
+  ```ts
+  const send = binding?.session?.prompt
+  return typeof send === 'function' ? { prompt: send } : undefined
+  ```
+
+  The host's own `Session.prompt` reaches the gateway **through `this.remote`**:
+
+  ```ts
+  result = await this.remote.session.prompt({ ... })
+  ```
+
+  Once the method is taken off the object, `this` no longer points at the session but at the `{ prompt: send }` wrapper, so `this.remote` is `undefined` and the call throws `undefined is not an object (evaluating 'this.remote.session')`. The method is now bound back to its session (`beginSubmission` for the same reason), and the picture actually goes out.
+
+  Worth noting: the draft channel in the same file has always written `create.bind(conversation)` — only the send path missed it. And because the send path is tried first, the draft path never runs, so that correct binding could not rescue it.
+
+- **A sent picture now shows up in the conversation.** The plugin used to invent its own `requestId` (`minimax-h3-${Date.now()}`) and hand it to `prompt()`: the model received the picture, but the **conversation showed no such message** — the host wants a local submission echo registered with `beginSubmission()` first, and the prompt to carry the identity that registration mints. The dock now does what the host's own composer does, and the picture arrives as part of that turn.
+
+- **A refusal is no longer read as a success.** A refused prompt resolves with `{ ok: false }` rather than throwing, and the plugin used to treat that as success and clear the note, leaving nothing on screen. The outcome is now read, and a refusal says so.
+
+- **Three new regression cases.** The mock in `tests/generate-dock.spec.ts` used to be an **arrow function that never reads `this`**, so whether the method was bound could not be observed — while the real host reads `this.remote`, and that is exactly the difference. The new cases make the mock read `this.remote` the way the host does, and pin three things: the call keeps the session as its receiver, the prompt carries the identity `beginSubmission` minted, and a refusal is reported. **Revert the fix and they fail at once.**
+
+
 ### 0.6.7 (2026-11-12)
 
 - **Merged PR [#5](https://github.com/JeffTsuiCUG/dsh-connect-minimaxcode/pull/5) by [azsora](https://github.com/azsora)**:
@@ -643,6 +675,8 @@ Fixing it yourself is even better — just open the PR. The format does not matt
 - **[azsora](https://github.com/azsora)** — found the plan's rate-limit endpoint in the desktop app's own traffic and wired the 5-hour and weekly usage windows into the credit pill and the plugin page ([#4](https://github.com/JeffTsuiCUG/dsh-connect-minimaxcode/pull/4)): the window length is derived from the response's own edges rather than hard-coded. It became 0.5.5.
 - **[qzz19970504](https://github.com/qzz19970504)** — found and fixed the wiring gap that made every image upload fail ([#3](https://github.com/JeffTsuiCUG/dsh-connect-minimaxcode/pull/3)): the catalog declared the `image` modality, but the adapter never connected the host's durable attachment service, so pi-ai refused image requests before they were sent. The fix became 0.5.3.
 - **[azsora](https://github.com/azsora)** — found that the official quota endpoint renamed its window field and inverted its meaning (the share used became the share remaining), closed the gap where a 200 with an unreadable shape never reached the older path, and reworked the pill detail and the page card into rounded sub-cards that say why there are no windows ([#5](https://github.com/JeffTsuiCUG/dsh-connect-minimaxcode/pull/5)). It became 0.6.7.
+- **[lulunick](https://github.com/lulunick)** — found the real reason the "Generate an image" entry failed **every** time: the plugin took `session.prompt` off the session object and called it as a standalone function, so `this` pointed at the wrapper instead of at the session — and the host's own `Session.prompt` reaches the gateway through `this.remote`, which made the call throw `undefined is not an object`. With the methods bound back to their session the picture actually goes out; he also added the local submission echo the host asks for (`beginSubmission()` — the plugin used to invent its own `requestId`, so the model received the picture and the conversation showed nothing) and reports a refusal honestly (the host resolves with `{ ok: false }` rather than throwing). The test's mock now reads `this.remote` the way the host does, which is what makes all three observable ([#7](https://github.com/JeffTsuiCUG/dsh-connect-minimaxcode/pull/7)). It became 0.6.8.
+- **[azsora](https://github.com/azsora)** — moved the plan tier into the Plan box as its headline line, at the balance box's own big-number size, named the box "Plan", and left no hole in the grid ([#6](https://github.com/JeffTsuiCUG/dsh-connect-minimaxcode/pull/6)). It became 0.6.8.
 - Plugin author: **[JeffTsuiCUG](https://github.com/JeffTsuiCUG)**
 
 ## Acknowledgements

package/README.md

@@ -395,6 +395,38 @@ OAuth 令牌有效期约 1 小时，由客户端自动续期。插件**只在自
 
 ## 更新记录
 
+### 0.6.8（2026-11-13）
+
+
+- **套餐档位移进 Plan 框**：档位与到期日原本是卡片网格里独立的一行，横在两个框下面，被拉伸成一片空白。现在档位成为 Plan 框的标题行，**字号与高度和余额框的大数字完全一致**（20px），到期日仍是原来的小字、靠右基线对齐；Plan 框的名字由「Plan 额度」简化为「Plan」。
+- **网格不再留洞**：档位行不再占一个网格单元，「每日签到」与「到期批次」因此并排落到第二行，两列都填满 —— 之前左列下方那条空白与右下角的空格子一并消失。Plan 框内的高度也一行不多：档位行顶替的正是原先用来对齐行高的那个隐藏占位。
+- 档位本身仍是**上游给的值**（`token_plan_tier`），不是写死的标签：别的套餐会显示它自己的名字；只有上游没返回该字段时，才回落到「M Plan 订阅」。
+
+
+- **修复「无法发送到对话」：会话方法没有绑定 `this`**。这一版之前，输入框里的「生成图片」**必然**失败，提示「无法发送到对话，请到插件页生成后下载」。原因在插件把 `session.prompt` 从会话对象上摘下来之后，以独立函数的方式调用：
+
+  ```ts
+  const send = binding?.session?.prompt
+  return typeof send === 'function' ? { prompt: send } : undefined
+  ```
+
+  宿主自己的 `Session.prompt` 是**通过 `this.remote` 走到网关**的：
+
+  ```ts
+  result = await this.remote.session.prompt({ ... })
+  ```
+
+  方法一旦被摘下来，`this` 就不再指向会话，而是指向外面那个 `{ prompt: send }` 包装对象，于是 `this.remote` 是 `undefined`，调用抛出 `undefined is not an object (evaluating 'this.remote.session')`。现在方法被绑回会话对象（`beginSubmission` 同理），图片就能真正发出去了。
+
+  顺带一提：同一个文件里的草稿通道**一直**写着 `create.bind(conversation)`，只是发送通道漏了 —— 而发送通道优先，草稿通道根本走不到，所以那个正确的绑定救不了场。
+
+- **发图现在会出现在对话里**。此前插件自己编了一个 `requestId`（`minimax-h3-${Date.now()}`）交给 `prompt()`：模型能收到图片，但**对话里不显示这条消息** —— 宿主要求先用 `beginSubmission()` 注册一次本地回执、再用它给出的身份发送。现在照宿主输入框的做法走，图片作为这一轮消息的一部分出现。
+
+- **不再把失败当成功**。`prompt()` 被拒绝时返回的是 `{ ok: false }` 而不是抛异常，插件此前会把它当成功、把提示条清空，于是屏幕上什么都不剩。现在会读出结果，拒绝时如实说「无法发送到对话」。
+
+- **新增回归测试（3 个用例）**。此前 `tests/generate-dock.spec.ts` 的 mock 是个**不读 `this`** 的箭头函数，因此方法有没有绑定根本测不出来 —— 真实宿主要读 `this.remote`，差别正在于此。这次的新用例让 mock 像宿主一样读 `this.remote`，三条断言分别钉住：调用带着会话当接收者、发送的是 `beginSubmission` 给出的身份、宿主拒绝时如实提示。**把这三条用例里的修复撤掉，它们会立刻失败。**
+
+
 ### 0.6.7（2026-11-12）
 
 - **并入 PR [#5](https://github.com/JeffTsuiCUG/dsh-connect-minimaxcode/pull/5)（作者 [azsora](https://github.com/azsora)）**：
@@ -656,6 +688,8 @@ dsh plugin --profile desktop add dsh-qoder-connect
 - **[azsora](https://github.com/azsora)** —— 从桌面 App 的实际调用里找出了套餐的**限额窗口接口**，并把 5 小时与每周两个窗口的用量接进了积分胶囊与插件页卡片（[#4](https://github.com/JeffTsuiCUG/dsh-connect-minimaxcode/pull/4)）：窗口时长从响应自身的起止字段推导，不写死。该功能成为 0.5.5。
 - **[qzz19970504](https://github.com/qzz19970504)** —— 定位并修复了「发图必然失败」的接线缺口（[#3](https://github.com/JeffTsuiCUG/dsh-connect-minimaxcode/pull/3)）：模型目录声明了 `image` 模态，但适配器没有把宿主的持久化附件服务接上，导致带图请求被 pi-ai 在发出前拒掉。该修复成为 0.5.3。
 - **[azsora](https://github.com/azsora)** —— 找出官方额度接口改名并反转语义（「已用百分比」→「剩余百分比」）这件事，补上了「200 但形状读不懂时也要回落旧路径」的缺口，并把积分胶囊与插件页卡片重排为圆角分栏、让「为什么没有窗口」逐项有说法（[#5](https://github.com/JeffTsuiCUG/dsh-connect-minimaxcode/pull/5)）。该合并成为 0.6.7。
+- **[lulunick](https://github.com/lulunick)** —— 找到了「生成图片**必然**失败」的真正原因：插件把 `session.prompt` 从会话对象上摘下来后以独立函数调用，`this` 因此指向了包装对象而不是会话，宿主自己的 `Session.prompt` 正是通过 `this.remote` 走到网关的，于是抛 `undefined is not an object`。方法被绑回会话后图片才真正发得出去；顺带补上了宿主要求的 `beginSubmission()` 本地回执（此前插件自己编 `requestId`，模型收到了图片但对话里不显示），以及被拒绝时（返回 `{ ok: false }` 而非抛异常）的如实提示。他同时把测试里的 mock 改成像宿主一样读 `this.remote`，让这三点钉得住（[#7](https://github.com/JeffTsuiCUG/dsh-connect-minimaxcode/pull/7)）。该修复成为 0.6.8。
+- **[azsora](https://github.com/azsora)** —— 把套餐档位移进 Plan 框当标题行，字号与余额框的大数字一致，Plan 框改名为「Plan」，网格也不再留洞（[#6](https://github.com/JeffTsuiCUG/dsh-connect-minimaxcode/pull/6)）。该重排成为 0.6.8。
 - 插件作者：**[JeffTsuiCUG](https://github.com/JeffTsuiCUG)**
 
 ## 致谢

package/lib/client.js

@@ -50,7 +50,7 @@
         expiringAmount: '{amount} 积分',
         nextExpiry: '最近到期日',
         creditsBox: '积分',
-        planTitle: 'Plan 额度',
+        planTitle: 'Plan',
         usageTitle: '本机用量',
         usageTotal: '累计',
         lots: '到期批次',
@@ -136,7 +136,7 @@
         expiringAmount: '{amount} credits',
         nextExpiry: 'Next expiry',
         creditsBox: 'Credits',
-        planTitle: 'Plan allowance',
+        planTitle: 'Plan',
         usageTitle: 'Local usage',
         usageTotal: 'Total',
         lots: 'Expiring grants',
@@ -440,6 +440,11 @@
              * it nothing: the rolling windows it does not have are already
              * explained by the notice beside them. What this adds is for the
              * accounts that do hold one — the tier's name and the day it ends.
+             *
+             * It heads the plan box rather than standing on its own below it: the
+             * tier is the plan's headline fact, so it wears the balance's own big
+             * number size. The day it ends is a detail beside the tier, and keeps
+             * the row size.
              */
             function planLine(t, plan) {
                 if (plan === undefined || !plan.active)
@@ -448,7 +453,7 @@
                 const expiry = plan.expiresAtMs === undefined
                     ? null
                     : t('planExpires', { date: day(plan.expiresAtMs) });
-                return h('div', { style: rowStyle, 'data-minimax-plan': true }, h('span', null, tier), h('span', null, expiry ?? ''));
+                return h('div', { style: planHeadStyle, 'data-minimax-plan': true }, h('span', null, tier), h('span', { style: planExpiryStyle }, expiry ?? ''));
             }
             /** The pill itself; the gate decides whether it is drawn at all. */
             function CreditPill(props) {
@@ -612,14 +617,32 @@
                  * `binding(id).session.prompt(content, mode, signal, requestId)` is
                  * what the composer calls when the person presses send, so a picture
                  * sent this way is a normal turn.
+                 *
+                 * The method is taken off the session object, so it has to be bound to
+                 * that object before it is called. `prompt` reaches the gateway
+                 * through `this.remote`, and an unbound call runs with `this` pointing
+                 * at the wrapper below instead of at the session — which throws
+                 * `undefined is not an object (evaluating 'this.remote.session')` and
+                 * leaves the dock reporting that it cannot send the picture.
                  */
                 const sessionFace = () => {
                     const id = sessionId();
                     if (id === undefined)
                         return undefined;
                     const binding = clientContext?.sessions?.binding?.(id);
-                    const send = binding?.session?.prompt;
-                    return typeof send === 'function' ? { prompt: send } : undefined;
+                    const session = binding?.session;
+                    const send = session?.prompt;
+                    if (typeof send !== 'function')
+                        return undefined;
+                    // `beginSubmission` is taken off the session for the same reason: it
+                    // is a method of the session and must keep its receiver.
+                    const begin = session?.beginSubmission;
+                    return {
+                        prompt: send.bind(session),
+                        ...(typeof begin === 'function'
+                            ? { beginSubmission: begin.bind(session) }
+                            : {}),
+                    };
                 };
                 /**
                  * The conversation's own draft channel.
@@ -692,7 +715,24 @@
                                     : t('generateNoBinding'));
                         }
                         if (face !== undefined) {
-                            await face.prompt([part, { type: 'text', text }], 'queue', undefined, `minimax-h3-${Date.now()}`);
+                            // The host registers a local submission echo before it prompts,
+                            // and the identity that registration mints is the one the prompt
+                            // must carry — that is how the sent picture shows up as this
+                            // turn's message. A made-up identity reaches the model but leaves
+                            // no echo, so the conversation would not show what was sent.
+                            const submission = face.beginSubmission?.({
+                                mode: 'queue',
+                                text,
+                                attachments: [],
+                            });
+                            const result = await face.prompt([part, { type: 'text', text }], 'queue', undefined, submission?.requestId);
+                            // A refused prompt resolves with `{ ok: false }` rather than
+                            // throwing, so the outcome has to be read: either way the picture
+                            // did not arrive.
+                            if (result === undefined || result.ok !== true) {
+                                setNote(t('generateSendFailed'));
+                                return;
+                            }
                             setPrompt('');
                             setOpen(false);
                             // The picture's own arrival is the feedback: it is in the
@@ -1020,6 +1060,9 @@
                 // account's usage history across both.
                 const lots = [...(credits_.upcoming ?? [])].sort((a, b) => a.expireAtMs - b.expireAtMs);
                 const windows = rateLimitRows(t, status?.rateLimit);
+                // The tier and the day it ends, for an account that holds one. It
+                // heads the plan box rather than standing on its own below it.
+                const planHead = planLine(t, status?.plan);
                 const profileLines = usageProfileRows(t, status?.usageProfile);
                 const updated = credits_.updatedAtMs !== undefined
                     ? t('updated', { time: new Date(credits_.updatedAtMs).toLocaleTimeString() })
@@ -1027,18 +1070,18 @@
                 const loginLine = login?.state === 'signed-in'
                     ? (login.source === 'plugin' ? t('loginPlugin') : t('loginDesktop'))
                     : undefined;
-                return h('div', { style: pageStyle, 'data-minimax-page': true }, h('div', { style: cardStyle, 'data-minimax-credits-card': true }, h('div', { style: cardHeadStyle }, h('span', { style: cardTitleStyle }, t('title')), h('span', { style: mutedStyle }, [loginLine, updated].filter(line => line !== undefined).join(' · '))), h('div', { style: gridStyle }, h('div', { style: boxStyle }, h('div', { style: boxTitleStyle }, t('creditsBox')), h('div', { style: bigNumberStyle }, credits(credits_.total, 2)), row(t('free'), credits(credits_.free, 3)), row(t('purchased'), credits(credits_.purchased, 3)), ...expiryRows(t, credits_).slice(0, 1)), h('div', { style: boxStyle, 'data-minimax-rate-limit': true }, h('div', { style: boxTitleStyle }, t('planTitle')), ...(windows.length > 0
-                    ? [
-                        // The invisible number keeps the windows lined up with
-                        // the balance box's rows, which sit below its own big
-                        // number. With no windows to align, it is only noise.
-                        h('div', { style: { ...bigNumberStyle, visibility: 'hidden' }, 'aria-hidden': 'true' }, '—'),
-                        ...windows,
-                    ]
-                    : [rateLimitFallback(t, status?.rateLimit)])), 
-                // The tier and the day it ends, for an account that holds one.
-                // An account without a plan has nothing to say here.
-                planLine(t, status?.plan), signinBlock(t, signin, refresh), h('div', { style: boxStyle }, h('div', { style: boxTitleStyle }, t('lots')), 
+                return h('div', { style: pageStyle, 'data-minimax-page': true }, h('div', { style: cardStyle, 'data-minimax-credits-card': true }, h('div', { style: cardHeadStyle }, h('span', { style: cardTitleStyle }, t('title')), h('span', { style: mutedStyle }, [loginLine, updated].filter(line => line !== undefined).join(' · '))), h('div', { style: gridStyle }, h('div', { style: boxStyle }, h('div', { style: boxTitleStyle }, t('creditsBox')), h('div', { style: bigNumberStyle }, credits(credits_.total, 2)), row(t('free'), credits(credits_.free, 3)), row(t('purchased'), credits(credits_.purchased, 3)), ...expiryRows(t, credits_).slice(0, 1)), h('div', { style: boxStyle, 'data-minimax-rate-limit': true }, h('div', { style: boxTitleStyle }, t('planTitle')), 
+                // The tier heads the box, wearing the balance's own big number
+                // size. An account without a plan has no headline to show, and
+                // the invisible number keeps the windows lined up with the
+                // balance box's rows instead.
+                ...(planHead !== null
+                    ? [planHead]
+                    : windows.length > 0
+                        ? [h('div', { style: { ...bigNumberStyle, visibility: 'hidden' }, 'aria-hidden': 'true' }, '—')]
+                        : []), ...(windows.length > 0
+                    ? windows
+                    : [rateLimitFallback(t, status?.rateLimit)])), signinBlock(t, signin, refresh), h('div', { style: boxStyle }, h('div', { style: boxTitleStyle }, t('lots')), 
                 // The upstream already caps the list at the five soonest lots
                 // with credits left; the sort only keeps that promise honest.
                 ...(lots.length === 0
@@ -1177,6 +1220,17 @@
             };
             const cardTitleStyle = { fontWeight: 600 };
             const bigNumberStyle = { fontSize: '20px', fontWeight: 600, marginBottom: '2px' };
+            // The plan's headline line: the tier wears the balance's own big number
+            // size, so the two boxes' headline lines are the same height, and the
+            // day it ends stays at the row size beside it.
+            const planHeadStyle = {
+                ...bigNumberStyle,
+                display: 'flex',
+                justifyContent: 'space-between',
+                alignItems: 'baseline',
+                gap: '16px',
+            };
+            const planExpiryStyle = { fontSize: '13px', fontWeight: 400, lineHeight: '1.6' };
             const mutedStyle = { opacity: 0.7, fontSize: '12px' };
             const actionButtonStyle = {
                 appearance: 'none',

package/package.json

@@ -2,20 +2,20 @@
   "name": "dsh-connect-minimaxcode",
   "displayName": "DSH Connect MiniMax Code",
   "description": "把本机已登录的 MiniMax Code 桌面 App 模型接入 DeepSeek Harness：零配置、Anthropic Messages 兼容、动态模型目录，并显示积分余额与到期 — bring MiniMax Code desktop app models into DSH with a dynamic model catalog and a credit balance with expiry details.",
-  "version": "0.6.7",
+  "version": "0.6.8",
   "author": "JeffTsuiCUG",
   "contributors": [
     {
       "name": "JeffTsuiCUG",
       "url": "https://github.com/JeffTsuiCUG"
     },
-    {
-      "name": "eghrhegpe",
-      "url": "https://github.com/eghrhegpe"
+    {
+      "name": "eghrhegpe",
+      "url": "https://github.com/eghrhegpe"
     },
-    {
-      "name": "qzz19970504",
-      "url": "https://github.com/qzz19970504"
+    {
+      "name": "qzz19970504",
+      "url": "https://github.com/qzz19970504"
     },
     {
       "name": "azsora",