feat: implement enabledSkills support in settings.json

Author: qorzj

docs/configuration.md

@@ -36,6 +36,7 @@ Deep Code 使用 `settings.json` 设置文件进行持久化配置，支持两
 | `webSearchTool`      | string    | 自定义联网搜索脚本的完整路径                                         |
 | `mcpServers`         | object    | MCP 服务器配置（键为服务名，值为 McpServerConfig 对象）              |
 | `temperature`        | number    | 模型采样温度，范围 `0` 到 `2`                           |
+| `enabledSkills`      | object    | 按 skill 名称启用或禁用 skill 的配置                                 |
 
 #### `env` 子字段
 
@@ -101,6 +102,23 @@ Deep Code 内置免费可用的 Web Search 工具。如果需要自定义搜索
 
 脚本接收一个搜索查询参数，输出 JSON 格式的结果供 AI 使用。
 
+#### `enabledSkills` — Skill 启用配置
+
+控制 skill 扫描时是否包含指定 skill。键是解析后的 skill 名称，值必须是布尔值：
+
+```json
+{
+  "enabledSkills": {
+    "skill-writer": false,
+    "code-review": true
+  }
+}
+```
+
+- 未配置的 skill 默认启用。
+- 将某个 skill 设置为 `false` 后，所有项目级和用户级目录中解析名称相同的 skill 都会被隐藏。
+- 项目设置会按 skill 覆盖用户设置。如果项目设置没有配置某个 skill，则使用用户设置。
+
 #### `mcpServers` — MCP 服务器
 
 MCP（Model Context Protocol）服务器配置。值是键值对，键为服务名称，值为服务器配置对象。

docs/configuration_en.md

@@ -36,6 +36,7 @@ The following are all the top-level fields supported in `settings.json`, along w
 | `webSearchTool`    | string  | Full path to a custom web search script                                     |
 | `mcpServers`       | object  | MCP server configurations (keys are service names, values are McpServerConfig objects) |
 | `temperature`      | number  | Sampling temperature for LLM, from `0` to `2`                 |
+| `enabledSkills`    | object  | Per-skill enable/disable map, keyed by skill name                           |
 
 #### `env` Sub-fields
 
@@ -101,6 +102,23 @@ Deep Code has a built-in, free-to-use Web Search tool. If you need custom search
 
 The script receives a search query as an argument and outputs results in JSON format for the AI.
 
+#### `enabledSkills` — Skill Enablement
+
+Controls whether skills are included during skill scanning. Keys are resolved skill names, and values must be booleans:
+
+```json
+{
+  "enabledSkills": {
+    "skill-writer": false,
+    "code-review": true
+  }
+}
+```
+
+- Missing entries are enabled by default.
+- Setting a skill to `false` hides every skill with that resolved `name`, across project and user skill roots.
+- Project settings override user settings per skill. If the project setting omits a skill, the user setting is used.
+
 #### `mcpServers` — MCP Servers
 
 Configuration for MCP (Model Context Protocol) servers. The value is a key-value pair, where the key is the service name and the value is a server configuration object.

src/session.ts

@@ -298,6 +298,7 @@ type SessionManagerOptions = {
     webSearchTool?: string;
     mcpServers?: Record<string, McpServerConfig>;
     permissions?: Required<PermissionSettings>;
+    enabledSkills?: Record<string, boolean>;
   };
   renderMarkdown: (text: string) => string;
   onAssistantMessage: (message: SessionMessage, shouldConnect: boolean) => void;
@@ -324,6 +325,7 @@ export class SessionManager {
     webSearchTool?: string;
     mcpServers?: Record<string, McpServerConfig>;
     permissions?: Required<PermissionSettings>;
+    enabledSkills?: Record<string, boolean>;
   };
   private readonly onAssistantMessage: (message: SessionMessage, shouldConnect: boolean) => void;
   private readonly onSessionEntryUpdated?: (entry: SessionEntry) => void;
@@ -808,6 +810,7 @@ ${agentInstructions}
 
   async listSkills(sessionId?: string): Promise<SkillInfo[]> {
     const skillRoots = this.getSkillScanRoots();
+    const enabledSkills = this.getResolvedSettings().enabledSkills ?? {};
     const skillsByName = new Map<string, SkillInfo>();
 
     const collectSkills = (root: string, displayRoot: string): SkillInfo[] => {
@@ -839,7 +842,11 @@ ${agentInstructions}
         } catch {
           continue;
         }
-        results.push(this.readSkillInfo(skillPath, `${displayRoot}/${skillName}/SKILL.md`, skillName));
+        const skill = this.readSkillInfo(skillPath, `${displayRoot}/${skillName}/SKILL.md`, skillName);
+        if (enabledSkills[skill.name] === false) {
+          continue;
+        }
+        results.push(skill);
       }
       return results;
     };

src/settings.ts

@@ -43,6 +43,8 @@ export type PermissionSettings = {
   defaultMode?: PermissionDefaultMode;
 };
 
+export type EnabledSkillsSettings = Record<string, boolean>;
+
 export type DeepcodingSettings = {
   env?: DeepcodingEnv;
   model?: string;
@@ -55,6 +57,7 @@ export type DeepcodingSettings = {
   webSearchTool?: string;
   mcpServers?: Record<string, McpServerConfig>;
   permissions?: PermissionSettings;
+  enabledSkills?: EnabledSkillsSettings;
 };
 
 export type ResolvedDeepcodingSettings = {
@@ -71,6 +74,7 @@ export type ResolvedDeepcodingSettings = {
   webSearchTool?: string;
   mcpServers?: Record<string, McpServerConfig>;
   permissions: Required<PermissionSettings>;
+  enabledSkills: EnabledSkillsSettings;
 };
 
 export type ModelConfigSelection = {
@@ -188,6 +192,30 @@ function mergePermissions(
   };
 }
 
+function normalizeEnabledSkills(value: unknown): EnabledSkillsSettings {
+  if (!value || typeof value !== "object" || Array.isArray(value)) {
+    return {};
+  }
+  const result: EnabledSkillsSettings = {};
+  for (const [name, enabled] of Object.entries(value)) {
+    if (!name || typeof enabled !== "boolean") {
+      continue;
+    }
+    result[name] = enabled;
+  }
+  return result;
+}
+
+function mergeEnabledSkills(
+  userSettings: DeepcodingSettings | null | undefined,
+  projectSettings: DeepcodingSettings | null | undefined
+): EnabledSkillsSettings {
+  return {
+    ...normalizeEnabledSkills(userSettings?.enabledSkills),
+    ...normalizeEnabledSkills(projectSettings?.enabledSkills),
+  };
+}
+
 function normalizeEnv(env: DeepcodingSettings["env"]): Record<string, string> {
   const result: Record<string, string> = {};
   if (!env) {
@@ -364,6 +392,7 @@ export function resolveSettingsSources(
     webSearchTool: webSearchTool || undefined,
     mcpServers: mergeMcpServers(userSettings, projectSettings, userEnv, projectEnv, systemEnv),
     permissions: mergePermissions(userSettings, projectSettings),
+    enabledSkills: mergeEnabledSkills(userSettings, projectSettings),
   };
 }
 

src/tests/session.test.ts

@@ -471,6 +471,60 @@ test("SessionManager lists skills from Deep Code and .agents roots by priority",
   assert.equal(sharedSkill?.description, "Project .deepcode skill");
 });
 
+test("SessionManager excludes disabled skills by resolved skill name", async () => {
+  const workspace = createTempDir("deepcode-disabled-skills-workspace-");
+  const home = createTempDir("deepcode-disabled-skills-home-");
+  setHomeDir(home);
+
+  const writeSkill = (root: string, dirName: string, skillName: string): void => {
+    const skillDir = path.join(root, dirName);
+    fs.mkdirSync(skillDir, { recursive: true });
+    fs.writeFileSync(
+      path.join(skillDir, "SKILL.md"),
+      `---\nname: ${skillName}\ndescription: ${skillName} description\n---\n# ${skillName}\n`,
+      "utf8"
+    );
+  };
+
+  for (const root of [
+    path.join(workspace, ".deepcode", "skills"),
+    path.join(workspace, ".agents", "skills"),
+    path.join(home, ".deepcode", "skills"),
+    path.join(home, ".agents", "skills"),
+  ]) {
+    writeSkill(root, "skill-writer", "skill-writer");
+  }
+  writeSkill(path.join(workspace, ".deepcode", "skills"), "frontmatter-disabled", "renamed-disabled");
+  writeSkill(path.join(workspace, ".deepcode", "skills"), "enabled-skill", "enabled-skill");
+
+  const manager = new SessionManager({
+    projectRoot: workspace,
+    createOpenAIClient: () => ({
+      client: null,
+      model: "test-model",
+      baseURL: "https://api.deepseek.com",
+      thinkingEnabled: false,
+      machineId: "machine-id-disabled-skills",
+    }),
+    getResolvedSettings: () => ({
+      model: "test-model",
+      enabledSkills: {
+        "skill-writer": false,
+        "renamed-disabled": false,
+        "enabled-skill": true,
+      },
+    }),
+    renderMarkdown: (text) => text,
+    onAssistantMessage: () => {},
+  });
+
+  const skills = await manager.listSkills();
+  const skillNames = skills.map((skill) => skill.name);
+
+  assert.deepEqual(skillNames, ["enabled-skill"]);
+  assert.equal(skills[0]?.path, "./.deepcode/skills/enabled-skill/SKILL.md");
+});
+
 test("SessionManager dispose disconnects MCP servers", async () => {
   const workspace = createTempDir("deepcode-mcp-dispose-workspace-");
   const serverPath = path.join(workspace, "mcp-server.cjs");

src/tests/settings-and-notify.test.ts

@@ -220,6 +220,39 @@ test("resolveSettingsSources merges permission settings", () => {
   assert.equal(resolved.permissions.defaultMode, "allowAll");
 });
 
+test("resolveSettingsSources merges enabledSkills with project precedence", () => {
+  const resolved = resolveSettingsSources(
+    {
+      enabledSkills: {
+        inherited: false,
+        "project-enabled": false,
+        "project-disabled": true,
+        invalid: "false" as never,
+      },
+    },
+    {
+      enabledSkills: {
+        "project-enabled": true,
+        "project-disabled": false,
+        projectOnly: true,
+        ignored: null as never,
+      },
+    },
+    {
+      model: "default-model",
+      baseURL: "https://default.example.com",
+    },
+    TEST_PROCESS_ENV
+  );
+
+  assert.deepEqual(resolved.enabledSkills, {
+    inherited: false,
+    "project-enabled": true,
+    "project-disabled": false,
+    projectOnly: true,
+  });
+});
+
 test("resolveSettingsSources merges MCP env with documented priority", () => {
   const resolved = resolveSettingsSources(
     {