opencode-idea 0.0.6 → 0.0.7

package/CHANGELOG.md

@@ -1,5 +1,11 @@
 # Changelog
 
+## 0.0.7
+
+- **修复:0.0.6 因 skill schema 校验失败导致整个插件被禁用**。`idea-run-config` skill 的定义用了 `location` 字段,而 `Skill.Info` 要求的是 `path` (AbsolutePath);`ctx.skill.transform` 因此抛 `SchemaError(Missing key at ["path"])`。OpenCode 的策略是**任何 transform 失败都会禁用整个插件**,所以 0.0.6 里 `/open-in-idea` 命令和 IDE MCP 注册都没能生效 (日志:`disabled plugin after transform failure plugin.id=opencode-idea state=skill`)。现在改用 `path`,并在测试里钉死字段集合 (`content` / `description` / `id` / `name` / `path`)。
+- **skill 注册改为尽力而为**:`runConfigSkill()` 或 `ctx.skill.transform` 抛错时只放弃 skill,不再中断 `setup` —— 可选知识不该拖垮命令与 MCP 接线。
+- **新增回归测试**:skill transform 被拒时 `/open-in-idea` 仍然注册。
+
 ## 0.0.6
 
 - **修复:工具报错不再注销 IDE MCP**。旧实现在 `tool.execute.after` 里把**任何** `idea_*` 报错都当成「端点已死」并注销整个注册 —— 业务错误 (例如路径不存在返回 `File not found`) 和一次传输抖动都会触发;注销后紧跟的 `ctx.mcp.reload()` 还会掐断同一批正在执行的其他调用。现在插件**不注册**该钩子:连接状态以 OpenCode 自己维护的工具快照为准 (`context` 钩子收到的 `event.tools` 里有没有 `idea_*`),只有 `/open-in-idea` 会 (重新) 连接。

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "opencode-idea",
-  "version": "0.0.6",
+  "version": "0.0.7",
   "description": "OpenCode plugin that connects to JetBrains IDE MCP servers and prefers IDE tools when the current project is open in the IDE",
   "main": "src/index.js",
   "type": "module",

package/src/ide-run-config-skill.js

@@ -41,17 +41,21 @@ export function readRunConfigSkillContent() {
 /**
  * Build the skill definition registered through `ctx.skill.transform`.
  *
- * `location` points at the shipped `.md` — the real skill document — so any
- * path-shaped validation is satisfied without inventing a virtual filename.
+ * The schema (`Skill.Info` in `@opencode/schema/skill`) requires the field to be
+ * named `path` and to hold an absolute path. Point it at the shipped `.md` — the
+ * real skill document — rather than inventing a virtual filename.
  *
- * @returns {{ id: string, name: string, description: string, location: string, content: string }}
+ * @returns {{ id: string, name: string, description: string, path: string, content: string }}
  */
 export function runConfigSkill() {
   return {
     id: RUN_CONFIG_SKILL_ID,
     name: 'idea-run-config',
     description: RUN_CONFIG_SKILL_DESCRIPTION,
-    location: fileURLToPath(SKILL_FILE),
+    // Field names are schema-critical: `Skill.Info` requires `path`
+    // (AbsolutePath). A wrong key fails validation and takes the whole plugin
+    // down, not just the skill.
+    path: fileURLToPath(SKILL_FILE),
     content: readRunConfigSkillContent(),
   };
 }

package/src/idea-mcp.js

@@ -81,7 +81,7 @@ export async function probePort(port, options = {}) {
         params: {
           protocolVersion: PROTOCOL_VERSION,
           capabilities: {},
-          clientInfo: { name: 'opencode-idea', version: '0.0.6' },
+          clientInfo: { name: 'opencode-idea', version: '0.0.7' },
         },
       }),
       signal: controller.signal,
@@ -151,7 +151,7 @@ export async function callTool(port, projectPath, name, args = {}, options = {})
       params: {
         protocolVersion: PROTOCOL_VERSION,
         capabilities: {},
-        clientInfo: { name: 'opencode-idea', version: '0.0.6' },
+        clientInfo: { name: 'opencode-idea', version: '0.0.7' },
       },
     });
     const response = await post({

package/src/plugin.js

@@ -507,12 +507,20 @@ export default {
     // The run-configuration skill is static knowledge — where `.run/*.run.xml`
     // live, how to bootstrap a schema from a real example, and how to verify a
     // write. JetBrains projects only, and independent of `/open-in-idea`.
+    //
+    // Best effort on purpose: optional knowledge must never take the command and
+    // the MCP wiring down with it. A rejected definition fails the transform, and
+    // a transform failure disables this plugin entirely.
     if (isIdeaProject && typeof ctx.skill?.transform === 'function') {
-      // Read the document before the transform: callbacks must stay synchronous.
-      const runConfigSkillDefinition = runConfigSkill();
-      skillRegistration = await ctx.skill.transform((editor) => {
-        editor.add(runConfigSkillDefinition);
-      });
+      try {
+        // Read the document before the transform: callbacks must stay synchronous.
+        const runConfigSkillDefinition = runConfigSkill();
+        skillRegistration = await ctx.skill.transform((editor) => {
+          editor.add(runConfigSkillDefinition);
+        });
+      } catch {
+        skillRegistration = undefined;
+      }
     }
 
     // `/open-in-idea`: manual, re-entrant trigger that also loads the IDE