xiaozhi-client: Skill for Claude Code

我是 CI 检查验证和质量保障技能，专门确保 xiaozhi-client 项目的代码质量符合标准，同时遵循务实开发理念。

技能使用原则

• 保证质量，但避免过度工程化：维持代码质量，但不追求理论上的完美
• 实用功能优先，理论完美次之：解决实际问题比预防各种可能更重要
• 简单解决方案优于复杂方案：优先选择直接有效的解决方案
• 务实开发指导：检查代码是否符合"如无必要勿增实体"的原则

技能能力

1. 完整代码质量检查流程

核心能力：自动执行项目要求的所有代码质量检查，确保代码符合 CI 标准。

检查流程

# 1. 执行完整检查（xiaozhi-client 项目）
pnpm check:all

# 2. 运行测试套件
pnpm test

# 3. （可选）生成覆盖率报告
pnpm test:coverage

检查内容详解

• pnpm check:all 包含：

  • pnpm lint - Biome 代码规范和格式检查
  • pnpm typecheck - TypeScript 严格类型检查
  • pnpm spellcheck - 拼写检查
  • pnpm check:cpd - 重复代码检查

• pnpm test 包含：

  • Vitest 单元测试执行
  • 集成测试验证
  • 功能测试覆盖

2. 智能问题诊断与修复建议

当检查失败时，自动分析失败原因并提供针对性修复建议。

类型检查失败诊断

// 常见问题1：any 类型使用
// ❌ 错误示例
function processData(data: any): any {
  return data.value;
}

// ✅ 修复建议
function processData<T extends Record<string, unknown>>(data: T): T[keyof T] {
  return data.value as T[keyof T];
}

// 常见问题2：类型定义缺失
// ❌ 错误示例
const config = {
  apiEndpoint: "https://api.home-assistant.local",
  timeout: 5000,
};

// ✅ 修复建议
interface Config {
  apiEndpoint: string;
  timeout: number;
}

const config: Config = {
  apiEndpoint: "https://api.home-assistant.local",
  timeout: 5000,
};

代码规范失败诊断

# 常见问题及修复命令

# 问题：Biome 检查失败
# 解决方案：运行自动修复
pnpm lint

# 问题：导入路径不规范
# 解决方案：使用路径别名系统
import { UnifiedMCPServer } from "@core/unified-server"; // ✅
import { StartCommand } from "@cli/commands/start";       // ✅
import { UnifiedMCPServer } from "./core/unified-server"; // ❌

# 问题：未使用的导入
# 解决方案：移除未使用的导入
pnpm lint  # 会自动清理

拼写检查失败诊断

# 常见问题：技术术语被标记为拼写错误
# 解决方案1：确认确实是错误，修正拼写
# 解决方案2：如果是专业术语，添加到项目词典
echo "homeassistant" >> .cspell.json

# 解决方案3：忽略特定文件或目录
# 在 .cspell.json 中配置 ignorePaths

测试失败诊断

// 常见问题1：测试覆盖率不足
// 解决方案：补充测试用例，覆盖未测试的代码路径

// 常见问题2：异步测试处理不当
// ❌ 错误示例
test("should fetch data", () => {
  const result = await fetchData();
  expect(result).toBeDefined();
});

// ✅ 修复建议
test("should fetch data", async () => {
  const result = await fetchData();
  expect(result).toBeDefined();
});

// 常见问题3：Mock 设置不完整
// ❌ 错误示例
vi.mock("@/services/light-service");

// ✅ 修复建议
vi.mock("@/services/light-service", () => ({
  LightService: {
    turnOn: vi.fn(),
    turnOff: vi.fn(),
  },
}));