fix: improve telegram command guidance

Author: MackDing

README.md

@@ -132,6 +132,7 @@ General:
 - `/plan <task>` - ask Codex for a plan only, without direct file modification intent
 - `/model [name|reset]` - show or set the model override for the current chat
 - `/skill list` - show skill switches for the current chat
+- `/skill status` - alias of `/skill list`
 - `/skill on <name>` - enable a skill for the current chat
 - `/skill off <name>` - disable a skill for the current chat
 - `/sh <command>` - run a safe allowlisted Linux command in the current project (disabled by default)

src/bot/commandUtils.js

@@ -3,6 +3,49 @@ export function extractCommandPayload(rawText = "", commandName) {
   return String(rawText).replace(pattern, "").trim();
 }
 
+function levenshteinDistance(a, b) {
+  const left = String(a);
+  const right = String(b);
+  const dp = Array.from({ length: left.length + 1 }, () => new Array(right.length + 1).fill(0));
+
+  for (let i = 0; i <= left.length; i += 1) dp[i][0] = i;
+  for (let j = 0; j <= right.length; j += 1) dp[0][j] = j;
+
+  for (let i = 1; i <= left.length; i += 1) {
+    for (let j = 1; j <= right.length; j += 1) {
+      const cost = left[i - 1] === right[j - 1] ? 0 : 1;
+      dp[i][j] = Math.min(
+        dp[i - 1][j] + 1,
+        dp[i][j - 1] + 1,
+        dp[i - 1][j - 1] + cost
+      );
+    }
+  }
+
+  return dp[left.length][right.length];
+}
+
+export function suggestClosestWord(input, candidates, maxDistance = 2) {
+  const normalizedInput = String(input || "").trim().toLowerCase();
+  if (!normalizedInput) return "";
+
+  let best = "";
+  let bestDistance = Number.POSITIVE_INFINITY;
+
+  for (const candidate of candidates) {
+    const normalizedCandidate = String(candidate || "").trim().toLowerCase();
+    if (!normalizedCandidate) continue;
+
+    const distance = levenshteinDistance(normalizedInput, normalizedCandidate);
+    if (distance < bestDistance) {
+      best = normalizedCandidate;
+      bestDistance = distance;
+    }
+  }
+
+  return bestDistance <= maxDistance ? best : "";
+}
+
 export function buildPlanPrompt(task) {
   return [
     "Planning mode only.",

src/bot/handlers.js

@@ -89,6 +89,7 @@ export function registerHandlers({
         "/plan <task> - 仅生成执行计划，不直接修改代码",
         "/model [name|reset] - 查看或设置当前 chat 的模型",
         "/skill list - 查看当前 chat 的 skill 开关",
+        "/skill status - 同 /skill list",
         "/skill on <name> - 启用 skill",
         "/skill off <name> - 禁用 skill",
         "/sh <command> - 运行受限 Linux 命令 (默认关闭)",
@@ -244,7 +245,7 @@ export function registerHandlers({
 
   bot.command("skill", async (ctx) => {
     const payload = extractCommandPayload(ctx.message.text, "skill");
-    if (!payload || /^list$/i.test(payload)) {
+    if (!payload || /^(list|status)$/i.test(payload)) {
       await sendChunkedMarkdown(
         ctx,
         [
@@ -525,7 +526,18 @@ export function registerHandlers({
 
   bot.on("text", async (ctx) => {
     const text = ctx.message.text?.trim() || "";
-    if (!text || text.startsWith("/")) return;
+    if (!text) return;
+    if (/^\/\s+\S+/.test(text)) {
+      await sendChunkedMarkdown(
+        ctx,
+        [
+          "命令格式错误：`/` 后面不要加空格。",
+          `try: ${text.replace(/^\/\s+/, "/")}`
+        ].join("\n")
+      );
+      return;
+    }
+    if (text.startsWith("/")) return;
 
     try {
       const route = await router.routeMessage(text, {

src/orchestrator/skills/mcpSkill.js

@@ -1,3 +1,5 @@
+import { suggestClosestWord } from "../../bot/commandUtils.js";
+
 export class McpSkill {
   constructor({ mcpClient }) {
     this.mcpClient = mcpClient;
@@ -27,6 +29,7 @@ export class McpSkill {
 
   async handleCommand(rawText) {
     const stripped = rawText.replace(/^\/mcp(@\w+)?\s*/i, "").trim();
+    const supportedSubcommands = ["list", "status", "reconnect", "enable", "disable", "tools", "call"];
     if (!stripped) {
       return {
         text: [
@@ -168,6 +171,11 @@ export class McpSkill {
       };
     }
 
-    return { text: "未知 MCP 子命令。支持: tools, call。" };
+    const suggested = suggestClosestWord(subcommand, supportedSubcommands);
+    return {
+      text: suggested
+        ? `未知 MCP 子命令: ${subcommand}。你是不是想输入 \`/mcp ${suggested}\`?`
+        : `未知 MCP 子命令: ${subcommand}。支持: ${supportedSubcommands.join(", ")}。`
+    };
   }
 }

tests/commandUtils.test.js

@@ -1,6 +1,10 @@
 import test from "node:test";
 import assert from "node:assert/strict";
-import { buildPlanPrompt, extractCommandPayload } from "../src/bot/commandUtils.js";
+import {
+  buildPlanPrompt,
+  extractCommandPayload,
+  suggestClosestWord
+} from "../src/bot/commandUtils.js";
 
 test("extractCommandPayload removes telegram command prefix and bot suffix", () => {
   assert.equal(extractCommandPayload("/exec@ExampleBot run tests", "exec"), "run tests");
@@ -15,3 +19,8 @@ test("buildPlanPrompt forces planning-only behavior", () => {
   assert.match(prompt, /Do not modify files/);
   assert.match(prompt, /Task:\nrefactor src\/index\.js/);
 });
+
+test("suggestClosestWord returns the nearest supported command when the typo is small", () => {
+  assert.equal(suggestClosestWord("ststus", ["list", "status", "tools"]), "status");
+  assert.equal(suggestClosestWord("zzz", ["list", "status", "tools"]), "");
+});

tests/mcpSkill.test.js

@@ -0,0 +1,57 @@
+import test from "node:test";
+import assert from "node:assert/strict";
+import { McpSkill } from "../src/orchestrator/skills/mcpSkill.js";
+
+function createSkill() {
+  return new McpSkill({
+    mcpClient: {
+      hasServers: () => true,
+      listServers: () => [
+        {
+          name: "context7",
+          enabled: true,
+          connected: true,
+          command: "npx",
+          args: ["-y", "@upstash/context7-mcp"],
+          cwd: process.cwd()
+        }
+      ],
+      reconnectServer: async (name) => ({
+        name,
+        enabled: true,
+        connected: true
+      }),
+      enableServer: async (name) => ({
+        name,
+        enabled: true,
+        connected: true
+      }),
+      disableServer: async (name) => ({
+        name,
+        enabled: false,
+        connected: false
+      }),
+      listTools: async () => [],
+      callTool: async () => ""
+    }
+  });
+}
+
+test("mcp skill suggests the closest subcommand for small typos", async () => {
+  const skill = createSkill();
+  const result = await skill.execute({
+    text: "/mcp ststus"
+  });
+
+  assert.match(result.text, /\/mcp status/);
+});
+
+test("mcp skill returns expanded help text when no subcommand is provided", async () => {
+  const skill = createSkill();
+  const result = await skill.execute({
+    text: "/mcp"
+  });
+
+  assert.match(result.text, /\/mcp list/);
+  assert.match(result.text, /\/mcp status \[server\]/);
+});