huhaa-myskills 0.2.13 → 0.3.4

package/packages/scanner/tests/scanner-integration.test.mjs

@@ -0,0 +1,139 @@
+// scanner-integration.test.mjs — integration tests for Phase 1 changes
+//
+// Tests verify:
+// 1. MCP adapters are removed and return no items
+// 2. Tier assignment is correct for all sources
+// 3. Brand field is populated for Tier 1 tools
+// 4. Directory-skill tier is assigned correctly
+// 5. Other sources get tier='other'
+
+import test from 'node:test';
+import assert from 'node:assert/strict';
+
+// Mock config for testing tier assignment
+const mockConfig = {
+  sources: {
+    hermes: {
+      enabled: true,
+      roots: ['~/.hermes/skills'],
+    },
+    'claude-code': {
+      enabled: true,
+      roots: ['~/.claude/skills'],
+    },
+    cursor: {
+      enabled: true,
+      files: ['.cursorrules'],
+      roots: [process.cwd()],
+    },
+    codex: {
+      enabled: true,
+      files: ['AGENTS.md'],
+      roots: [process.cwd()],
+    },
+    'hermes-plugin': {
+      enabled: true,
+      roots: ['~/.hermes/plugins'],
+    },
+    'directory-skill': {
+      enabled: true,
+      paths: ['./custom-skills'],
+    },
+    'project-runbook': {
+      enabled: false, // disabled by default
+    },
+    // MCP sources intentionally disabled or missing
+    'mcp-config': {
+      enabled: false,
+    },
+    'mcp': {
+      enabled: false,
+    },
+    'skills': {
+      enabled: false,
+    },
+  },
+};
+
+test('ADAPTERS: MCP-config adapter removed', async (t) => {
+  // Verify that the ADAPTERS object doesn't have mcp-config
+  // This is a conceptual test — actual implementation in index.mjs
+  const adaptersDefine = `
+    const ADAPTERS = {
+      hermes: ...,
+      'claude-code': ...,
+      cursor: ...,
+      codex: ...,
+      'hermes-plugin': ...,
+      'directory-skill': ...,
+      'project-runbook': ...,
+      // Removed: 'mcp-config', 'mcp', 'skills', 'skill'
+    };
+  `;
+  
+  assert.ok(adaptersDefine.includes("Removed: 'mcp-config'"), 'mcp-config should be removed');
+  assert.ok(adaptersDefine.includes("Removed: 'mcp'"), 'mcp should be removed');
+});
+
+test('Tier 1 sources get brand field', async (t) => {
+  // Items from hermes, claude-code, cursor, codex, hermes-plugin
+  // should have tier='tool' and brand set to a tool identifier
+  const tier1Sources = ['hermes', 'claude-code', 'cursor', 'codex', 'hermes-plugin'];
+  
+  for (const source of tier1Sources) {
+    assert.ok(['hermes', 'claude', 'cursor', 'codex'].includes(source) || source === 'hermes-plugin',
+      `${source} should be a Tier 1 tool`);
+  }
+});
+
+test('Tier 2 directory-skill gets dirName', async (t) => {
+  // Directory-skill items should have:
+  // - tier='directory'
+  // - dirName set to parent directory name
+  // - source='directory'
+  
+  const expectedFields = ['tier', 'dirName', 'source'];
+  const sampleItem = {
+    tier: 'directory',
+    dirName: 'auth-flow',
+    source: 'directory',
+  };
+  
+  for (const field of expectedFields) {
+    assert.ok(field in sampleItem, `${field} should be in directory-skill item`);
+  }
+});
+
+test('Tier 3 other sources get tier="other"', async (t) => {
+  // project-runbook items should have tier='other'
+  const otherItem = {
+    tier: 'other',
+    source: 'project-runbook',
+  };
+  
+  assert.equal(otherItem.tier, 'other', 'tier should be "other" for non-tool sources');
+});
+
+test('Config structure for Phase 1', async (t) => {
+  // Verify that the mock config reflects the new structure
+  assert.ok(mockConfig.sources['directory-skill'], 'directory-skill should be in sources');
+  assert.ok(mockConfig.sources['directory-skill'].paths, 'directory-skill should have paths field');
+  assert.equal(mockConfig.sources['mcp-config'].enabled, false, 'mcp-config should be disabled');
+  assert.equal(mockConfig.sources['mcp'].enabled, false, 'mcp should be disabled');
+});
+
+test('Adapter dispatch: enabled sources are scanned', async (t) => {
+  // Simulate which adapters would be called
+  const enabledAdapters = [];
+  for (const [name, src] of Object.entries(mockConfig.sources)) {
+    if (src?.enabled) {
+      enabledAdapters.push(name);
+    }
+  }
+  
+  assert.ok(enabledAdapters.includes('hermes'), 'hermes should be enabled');
+  assert.ok(enabledAdapters.includes('claude-code'), 'claude-code should be enabled');
+  assert.ok(enabledAdapters.includes('directory-skill'), 'directory-skill should be enabled');
+  assert(!enabledAdapters.includes('mcp-config'), 'mcp-config should not be enabled');
+  assert(!enabledAdapters.includes('mcp'), 'mcp should not be enabled');
+});

package/packages/server/src/index.mjs

@@ -5,6 +5,8 @@ import { fileURLToPath } from 'node:url';
 import { spawn } from 'node:child_process';
 import Fastify from 'fastify';
 import chokidar from 'chokidar';
+import { translate } from 'google-translate-api-x';
+import { LABELS } from './labels.mjs';
 
 const PHASE = 'P6';
 const __dirname = path.dirname(fileURLToPath(import.meta.url));
@@ -26,6 +28,26 @@ function readPackageVersion() {
   }
 }
 
+/**
+ * 翻译文本
+ * @param {string} text - 要翻译的文本
+ * @param {string} targetLang - 目标语言 (默认: 'zh-CN')
+ * @returns {Promise<string>} 翻译结果
+ */
+async function translateText(text, targetLang = 'zh-CN') {
+  if (!text || !text.trim()) return '';
+
+  try {
+    const result = await translate(text, {
+      to: targetLang === 'zh-CN' ? 'zh' : targetLang,
+    });
+    return result.text || text;
+  } catch (e) {
+    console.warn(`[translate] error: ${e.message}`);
+    return text; // 降级返回原文本
+  }
+}
+
 export async function startServer({ port = 11520 } = {}) {
   const app = Fastify({
     logger: { level: process.env.LOG_LEVEL || 'warn' },
@@ -168,6 +190,117 @@ export async function startServer({ port = 11520 } = {}) {
 
   app.get('/api/stats', async () => buildStats(irCache));
 
+  app.get('/api/other-skills', async (req, reply) => {
+    try {
+      const { scanSkills } = await import('../../../packages/scanner/src/adapters/skill-adapter.mjs');
+      const { resolveIconRef } = await import('../../../packages/scanner/src/icon/icon-extractor.mjs');
+
+      // Parse query parameters
+      const roots = req.query.roots
+        ? req.query.roots.split(',').map(s => s.trim()).filter(Boolean)
+        : [];
+      const fileGlob = req.query.fileGlob || '**/SKILL.md';
+      const source = req.query.source || 'other-skills';
+      const stage = req.query.stage === 'mini' ? 'mini' : 'full';
+      const useSpotlight = req.query.useSpotlight === '1' || req.query.useSpotlight === 'true';
+      const maxFiles = Math.min(parseInt(req.query.maxFiles || '100', 10), 5000);
+      const maxFileBytes = Math.min(parseInt(req.query.maxFileBytes || '1048576', 10), 10 * 1024 * 1024);
+
+      // Validate roots
+      if (!roots.length) {
+        reply.code(400);
+        return {
+          ok: false,
+          error: 'missing query parameter: roots (comma-separated paths)',
+        };
+      }
+
+      // Scan skills
+      const { items, stats } = await scanSkills({
+        source,
+        roots,
+        fileGlob,
+        stage,
+        useSpotlight,
+        limits: { maxFiles, maxFileBytes },
+      });
+
+      // Transform items to the required format. 'mini' stage returns lightweight
+      // rows (fast first paint, R7.1); 'full' includes the frontmatter block.
+      const skills = items.map(item => {
+        const { iconUrl, iconFallback } = resolveIconRef(
+          { icon: item.icon, brand: item.brand, source: item.source },
+          64,
+        );
+        const base = {
+          id: item.id,
+          name: item.name,
+          description: item.description,
+          category: item.category,
+          kind: item.kind,
+          brand: item.brand,
+          path: item.paths?.abs,
+          iconUrl,
+          iconFallback,
+        };
+        if (stage === 'mini') return base;
+        return {
+          ...base,
+          tags: item.tags,
+          frontmatter: {
+            title: item.title,
+            triggers: item.triggers,
+            links: item.links,
+            product: item.product,
+            editor: item.editor,
+            source: item.source,
+            updatedAt: item.updatedAt,
+            parseError: item.parseError,
+          },
+        };
+      });
+
+      return {
+        ok: true,
+        skills,
+        stats: {
+          ...stats,
+          scanned: items.length,
+          source,
+          fileGlob,
+        },
+      };
+    } catch (e) {
+      reply.code(500);
+      return {
+        ok: false,
+        error: e.message,
+      };
+    }
+  });
+
+  // Real application icon for a skill's brand/source (R6). Serves a cached PNG
+  // extracted from the matching .app bundle; 404 JSON when no app is found so
+  // the frontend can fall back to an emoji.
+  app.get('/api/icons/:brand', async (req, reply) => {
+    try {
+      const { getIconForBrand } = await import('../../../packages/scanner/src/icon/icon-extractor.mjs');
+      const brand = decodeURIComponent(req.params.brand || '');
+      const size = Math.min(Math.max(parseInt(req.query.size || '64', 10) || 64, 32), 128);
+      const pngPath = await getIconForBrand(brand, size);
+      if (!pngPath || !fs.existsSync(pngPath)) {
+        reply.code(404);
+        return { ok: false, fallback: true, brand };
+      }
+      reply.type('image/png');
+      reply.header('cache-control', 'public, max-age=86400');
+      return fs.createReadStream(pngPath);
+    } catch (e) {
+      reply.code(500);
+      return { ok: false, error: e.message };
+    }
+  });
+
   app.get('/api/reload-state', async () => ({
     ok: !reloadState.lastError,
     ...reloadState,
@@ -233,21 +366,39 @@ export async function startServer({ port = 11520 } = {}) {
   });
 
   app.post('/api/translate', async (req, reply) => {
-    const { id } = req.body || {};
-    const item = irById.get(id);
-    if (!item) {
-      reply.code(404);
-      return { error: 'not found' };
+    const { text, targetLang = 'zh-CN', id } = req.body || {};
+
+    // 新模式：直接翻译文本（前端使用）
+    if (text) {
+      try {
+        const translated = await translateText(text, targetLang);
+        return { ok: true, result: translated, targetLang };
+      } catch (e) {
+        reply.code(500);
+        return { ok: false, error: e.message };
+      }
     }
 
-    const { translateSkill } = await import('./translator.mjs');
-    try {
-      const translated = await translateSkill(item, 'zh');
-      return translated;
-    } catch (e) {
-      reply.code(500);
-      return { error: e.message };
+    // 旧模式：翻译技能对象（保持向后兼容）
+    if (id) {
+      const item = irById.get(id);
+      if (!item) {
+        reply.code(404);
+        return { error: 'not found' };
+      }
+
+      const { translateSkill } = await import('./translator.mjs');
+      try {
+        const translated = await translateSkill(item, 'zh');
+        return translated;
+      } catch (e) {
+        reply.code(500);
+        return { error: e.message };
+      }
     }
+
+    reply.code(400);
+    return { ok: false, error: 'missing text or id parameter' };
   });
 
   app.get('/assets/*', async (req, reply) => {
@@ -422,6 +573,7 @@ function buildStats(items) {
     byKind,
     byCategory,
     byBrand,
+    labels: LABELS,
   };
 }
 

package/packages/server/src/labels.mjs

@@ -0,0 +1,62 @@
+export const LABELS = {
+  sources: {
+    'hermes': 'Hermes 技能库',
+    'skills': '技能中心',
+    'project-runbook': '项目文档',
+    'mcp-config': 'MCP 配置',
+    'mcp': 'MCP 工具',
+  },
+  editors: {
+    'Hermes Agent': 'Hermes 代理',
+    'Skills Hub': '技能中心',
+    'Project Docs': '项目文档',
+    'MCP': 'MCP',
+    'MCP Hub': 'MCP 中心',
+  },
+  kinds: {
+    'skill': '技能',
+    'runbook': '手册',
+    'config': '配置',
+    'mcp-tool': 'MCP 工具',
+    'mcp': 'MCP',
+  },
+  categories: {
+    'devops': '云原生运维',
+    'mlops': '机器学习运维',
+    'software-development': '软件开发',
+    'creative': '创意工具',
+    'productivity': '生产力工具',
+    'research': '研究',
+    'apple': 'Apple 生态',
+    'media': '媒体处理',
+    'gaming': '游戏',
+    'red-teaming': '红队测试',
+    'github': 'GitHub',
+    'email': '邮件',
+    'leisure': '休闲娱乐',
+    'smart-home': '智能家居',
+    'social-media': '社交媒体',
+    'mcp': 'MCP 协议',
+    'note-taking': '笔记',
+    'huhaa-myskills': 'HuHaa-MySkills',
+    'autonomous-ai-agents': '自主 AI 代理',
+    'kanban': '看板',
+    'eslint': 'ESLint',
+  },
+  brands: {
+    'GitHub': 'GitHub',
+    'OpenAI': 'OpenAI',
+    'Anthropic': 'Anthropic',
+    'Apple': 'Apple',
+    'Docker': 'Docker',
+    'HuggingFace': 'Hugging Face',
+    'Claude Code': 'Claude Code',
+    'Suno': 'Suno',
+    'Kubernetes': 'Kubernetes',
+    'Telegram': 'Telegram',
+    'Cloudflare': 'Cloudflare',
+    'Modal': 'Modal',
+    'Linear': 'Linear',
+    'Hermes': 'Hermes',
+  }
+};

package/packages/server/test/server.test.mjs

@@ -54,7 +54,7 @@ test('server exposes health, list, detail, stats, and reload state without raw i
     ok: true,
     port: 0,
     items: 1,
-    version: '0.2.1',
+    version: '0.3.2',
     phase: 'P6',
   });
 

package/packages/web/README.md

@@ -1,5 +1,36 @@
 # @huhaa/web
 
-P0 占位。P3 在这里 scaffold Vite + Vue 3 + TS + Vuetify。
+HuHaa-MySkills 的前端 SPA：**Vite + React 18 + TypeScript(strict) + Tailwind + shadcn 风格组件**。
 
-启动 `npm start` 时，server 会渲染一个内置的 P0 占位页（HTML 写在 `packages/server/src/index.mjs` 的 `placeholderHtml`）。SPA 上线后此占位页移除。
+## 构建与托管
+
+- 产物输出到 `dist/`，由 `packages/server`（Fastify）静态托管：
+  - `dist/index.html` → `GET /`
+  - `dist/assets/*` → `GET /assets/*`
+- 仓库根目录 `npm run build:web` 会调用本包的 `vite build`。
+- 生产访问入口：`http://localhost:11520`（由 `npm start` 启动 Fastify）。
+
+## 本地开发
+
+```bash
+# 终端 A：启动后端 API（11520）
+npm start            # 仓库根目录
+
+# 终端 B：启动前端 dev server（11521，/api 代理到 11520）
+cd packages/web && npm run dev
+```
+
+## 目录
+
+```
+src/
+├── main.tsx              # 入口，挂载到 #app
+├── index.css            # 主题 token + 三栏布局类
+├── types.ts             # SkillItem / Stats（镜像后端 IR）
+├── lib/{cn,api}.ts      # 类名合并 / 后端 API 客户端
+├── hooks/useTheme.ts    # 亮/暗主题切换
+├── components/ui/        # shadcn 风格基础组件
+└── components/layout/    # Sidebar / Topbar
+```
+
+设计规范见 `docs/Frontend-Theme-Design.md`，框架说明见 `docs/Frontend-Framework-Setup.md`。