coding-tool-x 3.6.7 → 3.6.9

package/src/server/api/plugins.js

@@ -7,9 +7,10 @@
 const express = require('express');
 const { PluginsService } = require('../services/plugins-service');
 const { maskToken } = require('../services/oauth-utils');
+const { sendApiError } = require('./validation-errors');
 
 const router = express.Router();
-const SUPPORTED_PLATFORMS = ['claude', 'opencode'];
+const SUPPORTED_PLATFORMS = ['claude', 'codex', 'gemini', 'opencode'];
 const pluginServices = new Map();
 
 function resolvePlatform(rawPlatform) {
@@ -41,7 +42,8 @@ function extractRepoPayload(source = {}) {
     projectPath: repo.projectPath || source.projectPath || '',
     localPath: repo.localPath || source.localPath || '',
     repoUrl: repo.repoUrl || repo.url || source.repoUrl || source.url || '',
-    token: repo.token || source.token || ''
+    token: repo.token || source.token || '',
+    marketplace: repo.marketplace || source.marketplace || ''
   };
 }
 
@@ -63,6 +65,28 @@ function sanitizeRepos(service, repos = []) {
   return (Array.isArray(repos) ? repos : []).map(sanitizeRepo);
 }
 
+/**
+ * 获取平台插件能力
+ * GET /api/plugins/capabilities
+ */
+router.get('/capabilities', (req, res) => {
+  try {
+    const { platform, service } = getPluginsService(req);
+    const capabilities = typeof service.getCapabilities === 'function'
+      ? service.getCapabilities()
+      : { platform, supportsPlugins: true };
+
+    res.json({
+      success: true,
+      platform,
+      capabilities
+    });
+  } catch (err) {
+    console.error('[Plugins API] Get capabilities error:', err);
+    sendApiError(res, err);
+  }
+});
+
 /**
  * 获取插件列表
  * GET /api/plugins
@@ -79,10 +103,7 @@ router.get('/', (req, res) => {
     });
   } catch (err) {
     console.error('[Plugins API] List plugins error:', err);
-    res.status(500).json({
-      success: false,
-      message: err.message
-    });
+    sendApiError(res, err);
   }
 });
 
@@ -107,10 +128,7 @@ router.get('/market', async (req, res) => {
     });
   } catch (err) {
     console.error('[Plugins API] Get market plugins error:', err);
-    res.status(500).json({
-      success: false,
-      message: err.message
-    });
+    sendApiError(res, err);
   }
 });
 
@@ -123,12 +141,13 @@ router.post('/install', async (req, res) => {
   try {
     const { platform, service } = getPluginsService(req);
     const { directory, repo, gitUrl, source } = req.body;
+    const hasDirectoryField = Object.prototype.hasOwnProperty.call(req.body, 'directory');
 
     // Support both new format (directory + repo) and legacy format (gitUrl)
     let installUrl;
     if (source) {
       installUrl = source;
-    } else if (directory && repo) {
+    } else if (repo && hasDirectoryField) {
       installUrl = '';
     } else if (gitUrl) {
       installUrl = gitUrl;
@@ -141,10 +160,10 @@ router.post('/install', async (req, res) => {
 
     const result = await service.installPlugin(
       installUrl,
-      directory && repo
+      repo && hasDirectoryField
         ? {
             ...extractRepoPayload({ repo }),
-            directory
+            directory: directory || ''
           }
         : null
     );
@@ -164,10 +183,7 @@ router.post('/install', async (req, res) => {
     });
   } catch (err) {
     console.error('[Plugins API] Install plugin error:', err);
-    res.status(500).json({
-      success: false,
-      message: err.message
-    });
+    sendApiError(res, err);
   }
 });
 
@@ -188,10 +204,7 @@ router.get('/repos', (req, res) => {
     });
   } catch (err) {
     console.error('[Plugins API] Get repos error:', err);
-    res.status(500).json({
-      success: false,
-      message: err.message
-    });
+    sendApiError(res, err);
   }
 });
 
@@ -223,10 +236,7 @@ router.post('/repos', (req, res) => {
     });
   } catch (err) {
     console.error('[Plugins API] Add repo error:', err);
-    res.status(500).json({
-      success: false,
-      message: err.message
-    });
+    sendApiError(res, err);
   }
 });
 
@@ -243,10 +253,7 @@ router.delete('/repos', (req, res) => {
     });
   } catch (err) {
     console.error('[Plugins API] Remove repo error:', err);
-    res.status(500).json({
-      success: false,
-      message: err.message
-    });
+    sendApiError(res, err);
   }
 });
 
@@ -271,10 +278,7 @@ router.put('/repos/toggle', (req, res) => {
     });
   } catch (err) {
     console.error('[Plugins API] Toggle repo error:', err);
-    res.status(500).json({
-      success: false,
-      message: err.message
-    });
+    sendApiError(res, err);
   }
 });
 
@@ -298,10 +302,7 @@ router.delete('/repos/:owner/:name', (req, res) => {
     });
   } catch (err) {
     console.error('[Plugins API] Remove repo error:', err);
-    res.status(500).json({
-      success: false,
-      message: err.message
-    });
+    sendApiError(res, err);
   }
 });
 
@@ -333,10 +334,7 @@ router.put('/repos/:owner/:name/toggle', (req, res) => {
     });
   } catch (err) {
     console.error('[Plugins API] Toggle repo error:', err);
-    res.status(500).json({
-      success: false,
-      message: err.message
-    });
+    sendApiError(res, err);
   }
 });
 
@@ -367,10 +365,7 @@ router.put('/repos/auth', (req, res) => {
     });
   } catch (err) {
     console.error('[Plugins API] Update repo auth error:', err);
-    res.status(500).json({
-      success: false,
-      message: err.message
-    });
+    sendApiError(res, err);
   }
 });
 
@@ -391,10 +386,7 @@ router.post('/repos/sync', async (req, res) => {
     });
   } catch (err) {
     console.error('[Plugins API] Sync repos error:', err);
-    res.status(500).json({
-      success: false,
-      message: err.message
-    });
+    sendApiError(res, err);
   }
 });
 
@@ -415,10 +407,7 @@ router.post('/sync', async (req, res) => {
     });
   } catch (err) {
     console.error('[Plugins API] Sync plugins error:', err);
-    res.status(500).json({
-      success: false,
-      message: err.message
-    });
+    sendApiError(res, err);
   }
 });
 
@@ -504,10 +493,7 @@ router.get('/:name', (req, res) => {
     });
   } catch (err) {
     console.error('[Plugins API] Get plugin error:', err);
-    res.status(500).json({
-      success: false,
-      message: err.message
-    });
+    sendApiError(res, err);
   }
 });
 
@@ -536,10 +522,7 @@ router.delete('/:name', (req, res) => {
     });
   } catch (err) {
     console.error('[Plugins API] Uninstall plugin error:', err);
-    res.status(500).json({
-      success: false,
-      message: err.message
-    });
+    sendApiError(res, err);
   }
 });
 
@@ -571,10 +554,7 @@ router.put('/:name/toggle', (req, res) => {
     });
   } catch (err) {
     console.error('[Plugins API] Toggle plugin error:', err);
-    res.status(500).json({
-      success: false,
-      message: err.message
-    });
+    sendApiError(res, err);
   }
 });
 
@@ -605,10 +585,7 @@ router.put('/:name/config', (req, res) => {
     });
   } catch (err) {
     console.error('[Plugins API] Update plugin config error:', err);
-    res.status(500).json({
-      success: false,
-      message: err.message
-    });
+    sendApiError(res, err);
   }
 });
 

package/src/server/api/skills.js

@@ -5,6 +5,7 @@
 const express = require('express');
 const { SkillService } = require('../services/skill-service');
 const { maskToken } = require('../services/oauth-utils');
+const { sendApiError } = require('./validation-errors');
 
 const router = express.Router();
 const SUPPORTED_PLATFORMS = ['claude', 'codex', 'gemini', 'opencode'];
@@ -84,10 +85,7 @@ router.get('/', async (req, res) => {
     });
   } catch (err) {
     console.error('[Skills API] List skills error:', err);
-    res.status(500).json({
-      success: false,
-      message: err.message
-    });
+    sendApiError(res, err);
   }
 });
 
@@ -116,10 +114,7 @@ router.get('/detail/*', async (req, res) => {
     });
   } catch (err) {
     console.error('[Skills API] Get skill detail error:', err);
-    res.status(500).json({
-      success: false,
-      message: err.message
-    });
+    sendApiError(res, err);
   }
 });
 
@@ -138,10 +133,7 @@ router.get('/installed', (req, res) => {
     });
   } catch (err) {
     console.error('[Skills API] Get installed skills error:', err);
-    res.status(500).json({
-      success: false,
-      message: err.message
-    });
+    sendApiError(res, err);
   }
 });
 
@@ -184,10 +176,7 @@ router.post('/install', async (req, res) => {
     });
   } catch (err) {
     console.error('[Skills API] Install skill error:', err);
-    res.status(500).json({
-      success: false,
-      message: err.message
-    });
+    sendApiError(res, err);
   }
 });
 
@@ -209,7 +198,7 @@ router.post('/install-local', (req, res) => {
     res.json({ success: true, platform, ...result });
   } catch (err) {
     console.error('[Skills API] Install local skill error:', err);
-    res.status(500).json({ success: false, message: err.message });
+    sendApiError(res, err);
   }
 });
 
@@ -259,10 +248,7 @@ router.post('/create', (req, res) => {
     });
   } catch (err) {
     console.error('[Skills API] Create skill error:', err);
-    res.status(500).json({
-      success: false,
-      message: err.message
-    });
+    sendApiError(res, err);
   }
 });
 
@@ -292,10 +278,7 @@ router.post('/uninstall', (req, res) => {
     });
   } catch (err) {
     console.error('[Skills API] Uninstall skill error:', err);
-    res.status(500).json({
-      success: false,
-      message: err.message
-    });
+    sendApiError(res, err);
   }
 });
 
@@ -314,10 +297,7 @@ router.get('/repos', (req, res) => {
     });
   } catch (err) {
     console.error('[Skills API] Get repos error:', err);
-    res.status(500).json({
-      success: false,
-      message: err.message
-    });
+    sendApiError(res, err);
   }
 });
 
@@ -349,10 +329,7 @@ router.post('/repos', (req, res) => {
     });
   } catch (err) {
     console.error('[Skills API] Add repo error:', err);
-    res.status(500).json({
-      success: false,
-      message: err.message
-    });
+    sendApiError(res, err);
   }
 });
 
@@ -369,10 +346,7 @@ router.delete('/repos', (req, res) => {
     });
   } catch (err) {
     console.error('[Skills API] Remove repo error:', err);
-    res.status(500).json({
-      success: false,
-      message: err.message
-    });
+    sendApiError(res, err);
   }
 });
 
@@ -390,10 +364,7 @@ router.put('/repos/toggle', (req, res) => {
     });
   } catch (err) {
     console.error('[Skills API] Toggle repo error:', err);
-    res.status(500).json({
-      success: false,
-      message: err.message
-    });
+    sendApiError(res, err);
   }
 });
 
@@ -425,10 +396,7 @@ router.put('/repos/auth', (req, res) => {
     });
   } catch (err) {
     console.error('[Skills API] Update repo auth error:', err);
-    res.status(500).json({
-      success: false,
-      message: err.message
-    });
+    sendApiError(res, err);
   }
 });
 
@@ -451,10 +419,7 @@ router.delete('/repos/:owner/:name', (req, res) => {
     });
   } catch (err) {
     console.error('[Skills API] Remove repo error:', err);
-    res.status(500).json({
-      success: false,
-      message: err.message
-    });
+    sendApiError(res, err);
   }
 });
 
@@ -479,10 +444,7 @@ router.put('/repos/:owner/:name/toggle', (req, res) => {
     });
   } catch (err) {
     console.error('[Skills API] Toggle repo error:', err);
-    res.status(500).json({
-      success: false,
-      message: err.message
-    });
+    sendApiError(res, err);
   }
 });
 
@@ -529,10 +491,7 @@ router.post('/create-with-files', (req, res) => {
     });
   } catch (err) {
     console.error('[Skills API] Create skill with files error:', err);
-    res.status(500).json({
-      success: false,
-      message: err.message
-    });
+    sendApiError(res, err);
   }
 });
 
@@ -554,10 +513,7 @@ router.get('/:directory/files', (req, res) => {
     });
   } catch (err) {
     console.error('[Skills API] Get skill files error:', err);
-    res.status(500).json({
-      success: false,
-      message: err.message
-    });
+    sendApiError(res, err);
   }
 });
 
@@ -588,10 +544,7 @@ router.get('/:directory/file/*', (req, res) => {
     });
   } catch (err) {
     console.error('[Skills API] Get skill file content error:', err);
-    res.status(500).json({
-      success: false,
-      message: err.message
-    });
+    sendApiError(res, err);
   }
 });
 
@@ -622,10 +575,7 @@ router.post('/:directory/files', (req, res) => {
     });
   } catch (err) {
     console.error('[Skills API] Add skill files error:', err);
-    res.status(500).json({
-      success: false,
-      message: err.message
-    });
+    sendApiError(res, err);
   }
 });
 
@@ -655,10 +605,7 @@ router.delete('/:directory/file/*', (req, res) => {
     });
   } catch (err) {
     console.error('[Skills API] Delete skill file error:', err);
-    res.status(500).json({
-      success: false,
-      message: err.message
-    });
+    sendApiError(res, err);
   }
 });
 
@@ -697,10 +644,7 @@ router.put('/:directory/file/*', (req, res) => {
     });
   } catch (err) {
     console.error('[Skills API] Update skill file error:', err);
-    res.status(500).json({
-      success: false,
-      message: err.message
-    });
+    sendApiError(res, err);
   }
 });
 

package/src/server/api/validation-errors.js

@@ -0,0 +1,34 @@
+const VALIDATION_ERROR_PATTERNS = [
+  /^Invalid\b/i,
+  /不能为空/,
+  /不合法/,
+  /只能包含/,
+  /不能包含/,
+  /必须/,
+  /不能/,
+  /Missing\b/i,
+  /requires?/i,
+  /required/i,
+  /escapes/i,
+  /not support/i,
+  /不支持/
+];
+
+function isValidationError(error) {
+  const message = String(error?.message || error || '');
+  return VALIDATION_ERROR_PATTERNS.some(pattern => pattern.test(message));
+}
+
+function sendApiError(res, error, fallbackStatus = 500, extra = {}) {
+  const status = isValidationError(error) ? 400 : fallbackStatus;
+  return res.status(status).json({
+    success: false,
+    message: error?.message || String(error),
+    ...extra
+  });
+}
+
+module.exports = {
+  isValidationError,
+  sendApiError
+};