ethan-skill 1.7.0 → 1.9.0

package/dist/skills/15-git-workflow.js

@@ -0,0 +1,288 @@
+"use strict";
+Object.defineProperty(exports, "__esModule", { value: true });
+exports.gitWorkflowSkill = void 0;
+exports.gitWorkflowSkill = {
+    id: 'git-workflow',
+    name: 'Git 工作流',
+    nameEn: 'git_workflow',
+    order: 15,
+    category: '执行侧',
+    description: '规范 Git 分支策略、提交规范、合并流程，建立团队一致的版本控制工作流',
+    descriptionEn: 'Establish consistent Git branching strategy, commit conventions, and merge workflow for teams',
+    detailDescription: `系统梳理 Git 工作流全流程，涵盖 GitFlow/Trunk-Based 分支策略选型、Conventional Commits 提交规范、
+rebase vs merge 决策、冲突解决流程和 PR/MR 最佳实践，帮助团队建立高效、可追溯的版本控制体系。`,
+    triggers: [
+        'Git 工作流',
+        'git workflow',
+        'git 规范',
+        '分支策略',
+        'branching strategy',
+        'commit 规范',
+        'commit convention',
+        '提交规范',
+        'PR 规范',
+        'rebase vs merge',
+        '冲突解决',
+        '@ethan git',
+        '@ethan git-workflow',
+    ],
+    steps: [
+        {
+            title: '1. 评估项目特征，选择分支策略',
+            content: `根据团队规模和发布节奏选择合适的分支策略：
+
+**GitFlow 适用场景**
+- 有明确版本号的产品（如 App、SDK、开源库）
+- 需要维护多个线上版本
+- 发布周期较长（周/月级别）
+
+\`\`\`
+main          ──●────────────────────●──  (生产稳定)
+hotfix/1.0.1    └──●──┘                  (紧急修复)
+release/1.1       └──●──┘               (预发布验证)
+develop       ──●──────●──────●──────●── (集成分支)
+feature/login    └──●──┘                 (功能开发)
+\`\`\`
+
+**Trunk-Based Development 适用场景**
+- 持续部署（CD）体系成熟
+- 有完善的 Feature Flag 机制
+- 团队规模适中（≤50 人），发布频率高（日/周）
+
+\`\`\`
+main  ──●──●──●──●──●──  (直接推送或短命分支 <2天)
+feat   └──●──┘           (短命功能分支，快速合并)
+\`\`\`
+
+**决策矩阵**
+
+| 维度 | GitFlow | Trunk-Based |
+|------|---------|-------------|
+| 发布频率 | 低（周/月） | 高（日/周） |
+| 团队规模 | 大 | 中小 |
+| 多版本维护 | 支持 | 不擅长 |
+| CI/CD 成熟度 | 低要求 | 高要求 |`,
+        },
+        {
+            title: '2. 制定提交信息规范（Conventional Commits）',
+            content: `采用 Conventional Commits 规范，格式：\`<type>(<scope>): <subject>\`
+
+**类型（type）定义**
+
+| type | 用途 | 版本影响 |
+|------|------|---------|
+| \`feat\` | 新功能 | MINOR |
+| \`fix\` | Bug 修复 | PATCH |
+| \`perf\` | 性能优化 | PATCH |
+| \`refactor\` | 重构（无功能变化） | — |
+| \`docs\` | 文档变更 | — |
+| \`test\` | 测试相关 | — |
+| \`chore\` | 构建/依赖/工具 | — |
+| \`ci\` | CI 配置变更 | — |
+| \`BREAKING CHANGE\` | 破坏性变更（Footer） | MAJOR |
+
+**示例**
+\`\`\`bash
+# 好的提交信息
+feat(auth): add OAuth2 login with Google provider
+fix(cart): prevent duplicate item addition on rapid click
+perf(query): add composite index on (user_id, created_at)
+refactor(api): extract pagination helper to shared utils
+docs(readme): update installation steps for Node 20
+
+# 破坏性变更写法
+feat(api)!: rename /users endpoint to /accounts
+
+BREAKING CHANGE: /users endpoint removed, use /accounts instead
+\`\`\`
+
+**工具链配置**
+\`\`\`bash
+# 安装 commitlint
+npm install -D @commitlint/cli @commitlint/config-conventional
+echo "module.exports = {extends: ['@commitlint/config-conventional']}" > commitlint.config.js
+
+# 配合 husky 在 commit-msg 钩子校验
+npx husky add .husky/commit-msg 'npx --no -- commitlint --edit $1'
+\`\`\``,
+        },
+        {
+            title: '3. Rebase vs Merge 决策与实践',
+            content: `**核心原则：黄金法则 — 不要 rebase 已推送的公共分支**
+
+**何时用 Merge**
+- 合并长期分支（feature → develop）
+- 需要保留完整历史记录（审计场景）
+- 多人协作的共享分支
+
+\`\`\`bash
+# 保留合并记录（推荐用于 PR/MR 合并）
+git merge --no-ff feature/login
+
+# 快进合并（适合独立小修改）
+git merge --ff-only hotfix/typo
+\`\`\`
+
+**何时用 Rebase**
+- 更新本地功能分支，与主干保持同步
+- 整理本地提交历史，推送 PR 前清理
+
+\`\`\`bash
+# 将功能分支变基到最新 main
+git checkout feature/login
+git rebase origin/main
+
+# 交互式 rebase：合并/重排/修改最近 3 个提交
+git rebase -i HEAD~3
+# 选项: pick / squash(s) / fixup(f) / reword(r) / drop(d)
+\`\`\`
+
+**Squash Merge**（GitHub/GitLab PR 推荐）
+\`\`\`bash
+# 将功能分支所有提交合并为一个干净提交
+git merge --squash feature/login
+git commit -m "feat(auth): add login page with form validation"
+\`\`\`
+
+**推荐工作流**
+1. 本地开发：随意提交，保持节奏
+2. 推送 PR 前：\`git rebase -i origin/main\` 整理提交
+3. PR 合并：使用 Squash Merge 保持主干干净`,
+        },
+        {
+            title: '4. 冲突解决流程',
+            content: `**结构化冲突解决步骤**
+
+\`\`\`bash
+# Step 1: 理解冲突来源
+git log --oneline --graph --all  # 查看分支关系
+git diff HEAD origin/main        # 对比差异
+
+# Step 2: 标记冲突文件分析
+git status  # 查看所有冲突文件
+# conflict markers: <<<<<<< HEAD ... ======= ... >>>>>>> branch
+
+# Step 3: 使用工具辅助解决
+git mergetool  # 调用配置的 merge tool（VSCode / IntelliJ）
+
+# 配置 VSCode 为默认 merge tool
+git config --global merge.tool vscode
+git config --global mergetool.vscode.cmd 'code --wait $MERGED'
+\`\`\`
+
+**三路合并理解（Three-way merge）**
+\`\`\`
+BASE（公共祖先）：const timeout = 5000;
+OURS（当前分支）：const timeout = 10000;  // 改为10s
+THEIRS（被合并）：const TIMEOUT = 5000;   // 改为大写常量名
+RESULT（手动）：  const TIMEOUT = 10000;  // 两个改动都要
+\`\`\`
+
+**预防冲突的最佳实践**
+- 功能分支生命周期控制在 1-3 天内
+- 每日同步主干：\`git pull --rebase origin main\`
+- 大文件/自动生成文件加入 \`.gitattributes\` 配置合并策略
+\`\`\`gitattributes
+# 始终使用 ours 策略合并 lock 文件（减少冲突）
+package-lock.json merge=ours
+yarn.lock merge=ours
+\`\`\``,
+        },
+        {
+            title: '5. Pull Request / Code Review 流程规范',
+            content: `**PR 模板设计**
+\`\`\`markdown
+## 变更说明
+[简洁描述本次变更做了什么、为什么]
+
+## 变更类型
+- [ ] 新功能 (feat)
+- [ ] Bug 修复 (fix)
+- [ ] 重构 (refactor)
+- [ ] 性能优化 (perf)
+
+## 测试验证
+- [ ] 单元测试通过
+- [ ] 手动测试场景: [描述]
+- [ ] 截图/录屏（UI 变更必填）
+
+## 影响范围
+[描述可能影响的模块或依赖方]
+
+## Checklist
+- [ ] 代码自查完毕
+- [ ] 无调试代码 (console.log/debugger)
+- [ ] 文档已更新（如需要）
+\`\`\`
+
+**PR 规模控制**
+- 理想 PR 大小：< 400 行（不含测试）
+- 超过 800 行：强制拆分为多个 PR
+- 可用 \`git diff --stat origin/main\` 提前检查
+
+**分支保护规则（GitHub/GitLab 配置）**
+\`\`\`
+main 分支保护：
+✅ Require pull request reviews (min: 1)
+✅ Require status checks to pass (CI/lint/test)
+✅ Require branches to be up to date
+✅ Restrict push access (仅管理员)
+✅ Require signed commits（高安全场景）
+\`\`\``,
+        },
+        {
+            title: '6. 输出工作流规范文档',
+            content: `整理为团队可直接使用的规范文档，格式如下：
+
+\`\`\`markdown
+## Git 工作流规范
+
+### 分支命名
+- feature/<ticket-id>-short-description  (如: feature/PROJ-123-user-login)
+- fix/<ticket-id>-short-description
+- hotfix/<version>-short-description     (如: hotfix/1.2.1-payment-crash)
+- release/<version>                      (如: release/1.3.0)
+
+### 提交规范
+格式: <type>(<scope>): <subject>
+示例: feat(auth): add JWT refresh token support
+
+### 禁止行为
+❌ 直接推送到 main/master
+❌ force push 到共享分支
+❌ rebase 已推送的公共分支
+❌ 超过 1000 行的单次 PR（紧急 hotfix 除外）
+
+### 分支生命周期
+- feature 分支: ≤ 5 个工作日
+- release 分支: ≤ 2 周
+- hotfix 分支: ≤ 24 小时
+\`\`\``,
+        },
+    ],
+    outputFormat: 'Markdown 工作流规范文档，含分支策略选型建议、提交规范示例、rebase/merge 决策指南、冲突解决 SOP 和 PR 规范模板',
+    examples: [
+        {
+            input: '我们团队5人，做 SaaS 产品，每周发版一次，应该用什么 Git 工作流？',
+            output: `## Git 工作流建议：GitHub Flow（简化 GitFlow）
+
+**推荐理由**：5人小团队 + 每周发版，GitHub Flow 复杂度适中，无需维护多个长期分支。
+
+**分支结构**
+- \`main\`：始终可部署，受保护
+- \`feature/xxx\`：功能开发，完成即 PR，合并即删除
+
+**提交规范**：Conventional Commits
+**合并策略**：Squash Merge（保持 main 历史干净）
+**发版标记**：\`git tag v1.x.0\` 打标签`,
+        },
+    ],
+    notes: [
+        '分支策略没有银弹，根据团队规模和发版频率选择最适合的',
+        'force push 操作必须在团队内公告，避免其他成员本地分支混乱',
+        '建议在 CI 中自动校验 commit message 格式，而非依赖人工审查',
+        '冲突解决后务必运行测试，确保合并结果功能正常',
+    ],
+    nextSkill: 'unit-testing',
+};
+//# sourceMappingURL=15-git-workflow.js.map

package/dist/skills/15-git-workflow.js.map

@@ -0,0 +1 @@
+{"version":3,"file":"15-git-workflow.js","sourceRoot":"","sources":["../../src/skills/15-git-workflow.ts"],"names":[],"mappings":";;;AAEa,QAAA,gBAAgB,GAAoB;IAC/C,EAAE,EAAE,cAAc;IAClB,IAAI,EAAE,SAAS;IACf,MAAM,EAAE,cAAc;IACtB,KAAK,EAAE,EAAE;IACT,QAAQ,EAAE,KAAK;IACf,WAAW,EAAE,sCAAsC;IACnD,aAAa,EAAE,+FAA+F;IAC9G,iBAAiB,EAAE;2DACsC;IACzD,QAAQ,EAAE;QACR,SAAS;QACT,cAAc;QACd,QAAQ;QACR,MAAM;QACN,oBAAoB;QACpB,WAAW;QACX,mBAAmB;QACnB,MAAM;QACN,OAAO;QACP,iBAAiB;QACjB,MAAM;QACN,YAAY;QACZ,qBAAqB;KACtB;IACD,KAAK,EAAE;QACL;YACE,KAAK,EAAE,kBAAkB;YACzB,OAAO,EAAE;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;0BAgCW;SACrB;QACD;YACE,KAAK,EAAE,mCAAmC;YAC1C,OAAO,EAAE;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;OAuCR;SACF;QACD;YACE,KAAK,EAAE,0BAA0B;YACjC,OAAO,EAAE;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;gCAuCiB;SAC3B;QACD;YACE,KAAK,EAAE,WAAW;YAClB,OAAO,EAAE;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;OAmCR;SACF;QACD;YACE,KAAK,EAAE,oCAAoC;YAC3C,OAAO,EAAE;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;OAsCR;SACF;QACD;YACE,KAAK,EAAE,cAAc;YACrB,OAAO,EAAE;;;;;;;;;;;;;;;;;;;;;;;;;OAyBR;SACF;KACF;IACD,YAAY,EACV,wEAAwE;IAC1E,QAAQ,EAAE;QACR;YACE,KAAK,EAAE,wCAAwC;YAC/C,MAAM,EAAE;;;;;;;;;;gCAUkB;SAC3B;KACF;IACD,KAAK,EAAE;QACL,4BAA4B;QAC5B,oCAAoC;QACpC,yCAAyC;QACzC,wBAAwB;KACzB;IACD,SAAS,EAAE,cAAc;CAC1B,CAAC"}

package/dist/skills/16-unit-testing.d.ts

@@ -0,0 +1,3 @@
+import type { SkillDefinition } from './types';
+export declare const unitTestingSkill: SkillDefinition;
+//# sourceMappingURL=16-unit-testing.d.ts.map

package/dist/skills/16-unit-testing.d.ts.map

@@ -0,0 +1 @@
+{"version":3,"file":"16-unit-testing.d.ts","sourceRoot":"","sources":["../../src/skills/16-unit-testing.ts"],"names":[],"mappings":"AAAA,OAAO,KAAK,EAAE,eAAe,EAAE,MAAM,SAAS,CAAC;AAE/C,eAAO,MAAM,gBAAgB,EAAE,eAsS9B,CAAC"}

package/dist/skills/16-unit-testing.js

@@ -0,0 +1,298 @@
+"use strict";
+Object.defineProperty(exports, "__esModule", { value: true });
+exports.unitTestingSkill = void 0;
+exports.unitTestingSkill = {
+    id: 'unit-testing',
+    name: '单元测试',
+    nameEn: 'unit_testing',
+    order: 16,
+    category: '质量侧',
+    description: '运用 AAA 模式和 TDD 工作流编写高质量单元测试，建立覆盖率目标和 Mock 策略',
+    descriptionEn: 'Write high-quality unit tests using AAA pattern, TDD workflow, mocking strategies, and coverage goals',
+    detailDescription: `系统指导单元测试的设计与实现，涵盖 AAA（Arrange-Act-Assert）模式、TDD 红绿重构循环、
+Mock/Stub/Spy 策略选择、覆盖率目标制定和测试命名规范，帮助团队建立可维护、有效的测试体系。`,
+    triggers: [
+        '单元测试',
+        'unit test',
+        '写测试',
+        'write tests',
+        'TDD',
+        '测试设计',
+        'test design',
+        'mock 策略',
+        'mocking',
+        '测试覆盖率',
+        'coverage',
+        '@ethan test',
+        '@ethan unit-testing',
+    ],
+    steps: [
+        {
+            title: '1. 明确测试目标与范围',
+            content: `在编写测试前，先明确测什么：
+
+**测试金字塔**
+\`\`\`
+        ┌───────────┐
+        │  E2E 测试  │  (少量，慢，高置信)
+       ┌┴───────────┴┐
+       │  集成测试    │  (适量，中速)
+      ┌┴─────────────┴┐
+      │  单元测试      │  (大量，快，低成本)
+      └───────────────┘
+\`\`\`
+
+**单元测试应该覆盖**
+- ✅ 纯函数的各种输入输出（含边界）
+- ✅ 类/模块的公共方法逻辑
+- ✅ 条件分支（if/switch/三元）
+- ✅ 错误处理路径（throw/catch）
+- ✅ 异步操作（Promise/async-await）
+
+**不应该单元测试**
+- ❌ 简单的 getter/setter（无逻辑）
+- ❌ 第三方库内部实现
+- ❌ 框架本身（如 React 渲染机制）
+- ❌ 私有方法（通过公共方法间接测试）`,
+        },
+        {
+            title: '2. AAA 模式编写测试用例',
+            content: `每个测试用例遵循 **Arrange → Act → Assert** 三段式结构：
+
+**基础示例（JavaScript/TypeScript with Vitest/Jest）**
+\`\`\`typescript
+describe('calculateDiscount', () => {
+  it('should apply 20% discount for premium users', () => {
+    // Arrange（准备：设置测试数据和依赖）
+    const user = { type: 'premium', cart: [{ price: 100 }, { price: 50 }] };
+    const expectedTotal = 120;  // 150 * 0.8
+
+    // Act（执行：调用被测函数）
+    const result = calculateDiscount(user);
+
+    // Assert（断言：验证结果）
+    expect(result.total).toBe(expectedTotal);
+    expect(result.discountRate).toBe(0.2);
+  });
+});
+\`\`\`
+
+**测试命名规范（Given-When-Then）**
+\`\`\`typescript
+// 格式: should <expected behavior> when <condition>
+it('should return null when user is not found')
+it('should throw AuthError when token is expired')
+it('should apply 20% discount when user has premium status')
+
+// 或使用 Given-When-Then 风格
+it('given empty cart, when checkout, then throws EmptyCartError')
+\`\`\`
+
+**边界条件测试清单**
+\`\`\`typescript
+describe('parseAge', () => {
+  // 正常值
+  it('should parse valid age 25')
+  // 边界值
+  it('should accept minimum age 0')
+  it('should accept maximum age 150')
+  // 非法值
+  it('should throw when age is negative')
+  it('should throw when age exceeds 150')
+  // 类型边界
+  it('should throw when age is not a number')
+  it('should throw when age is null or undefined')
+  it('should handle decimal by flooring to integer')
+});
+\`\`\``,
+        },
+        {
+            title: '3. TDD 工作流（红-绿-重构）',
+            content: `**TDD 循环步骤**
+
+\`\`\`
+🔴 Red   → 写一个失败的测试（先设计接口）
+🟢 Green → 写最少代码让测试通过（不过度设计）
+🔵 Refactor → 在测试保护下重构代码
+\`\`\`
+
+**实践示例：用 TDD 实现邮箱验证**
+
+\`\`\`typescript
+// Step 1 🔴 先写测试（此时 validateEmail 还不存在）
+describe('validateEmail', () => {
+  it('should return true for valid email', () => {
+    expect(validateEmail('user@example.com')).toBe(true);
+  });
+  it('should return false for missing @', () => {
+    expect(validateEmail('userexample.com')).toBe(false);
+  });
+  it('should return false for empty string', () => {
+    expect(validateEmail('')).toBe(false);
+  });
+});
+
+// Step 2 🟢 写最简实现让测试通过
+export function validateEmail(email: string): boolean {
+  return /^[^\s@]+@[^\s@]+\.[^\s@]+$/.test(email);
+}
+
+// Step 3 🔵 重构：提取正则为常量，添加类型注释
+const EMAIL_REGEX = /^[^\s@]+@[^\s@]+\.[^\s@]+$/;
+export function validateEmail(email: string): boolean {
+  if (!email) return false;
+  return EMAIL_REGEX.test(email);
+}
+\`\`\`
+
+**TDD 适用场景**
+- 明确需求的业务逻辑函数
+- 工具库/SDK 开发
+- Bug 修复（先写复现测试再修复）
+
+**不强制 TDD 的场景**
+- 探索性开发阶段
+- UI 组件（先实现再补测试）`,
+        },
+        {
+            title: '4. Mock / Stub / Spy 策略',
+            content: `**三种测试替身的区别**
+
+| 类型 | 用途 | 验证方式 |
+|------|------|---------|
+| **Stub** | 替换外部依赖，控制返回值 | 只验证输出 |
+| **Mock** | 验证函数是否被正确调用 | 验证调用行为 |
+| **Spy** | 监听真实函数的调用情况 | 包装真实实现 |
+
+**Vitest/Jest 实践**
+\`\`\`typescript
+import { vi, describe, it, expect, beforeEach } from 'vitest';
+
+// Stub: 控制外部 API 返回值
+vi.mock('../api/user', () => ({
+  fetchUser: vi.fn().mockResolvedValue({ id: 1, name: 'Alice' }),
+}));
+
+// Mock: 验证函数被调用
+it('should call sendEmail when user registers', async () => {
+  const sendEmail = vi.fn();
+  await registerUser({ email: 'test@test.com' }, { sendEmail });
+  expect(sendEmail).toHaveBeenCalledOnce();
+  expect(sendEmail).toHaveBeenCalledWith('test@test.com', expect.objectContaining({ subject: 'Welcome' }));
+});
+
+// Spy: 包装真实函数监听
+it('should log error when fetch fails', async () => {
+  const consoleSpy = vi.spyOn(console, 'error').mockImplementation(() => {});
+  vi.mocked(fetchUser).mockRejectedValue(new Error('Network Error'));
+  await loadUserProfile(1);
+  expect(consoleSpy).toHaveBeenCalledWith(expect.stringContaining('Network Error'));
+  consoleSpy.mockRestore();
+});
+\`\`\`
+
+**Mock 黄金法则**
+- 只 Mock 跨边界的依赖（网络、数据库、文件系统、时间）
+- 不要 Mock 被测单元的内部实现
+- 每次测试后还原 Mock（使用 \`beforeEach(() => vi.clearAllMocks())\`）`,
+        },
+        {
+            title: '5. 覆盖率目标与质量保障',
+            content: `**覆盖率类型与目标**
+
+| 覆盖率类型 | 说明 | 建议目标 |
+|----------|------|---------|
+| 语句覆盖（Statements） | 执行的语句比例 | ≥ 80% |
+| 分支覆盖（Branches） | if/else 分支比例 | ≥ 75% |
+| 函数覆盖（Functions） | 调用的函数比例 | ≥ 80% |
+| 行覆盖（Lines） | 执行的代码行比例 | ≥ 80% |
+
+**Vitest 覆盖率配置**
+\`\`\`typescript
+// vitest.config.ts
+export default defineConfig({
+  test: {
+    coverage: {
+      provider: 'v8',  // 或 'istanbul'
+      reporter: ['text', 'html', 'lcov'],
+      thresholds: {
+        statements: 80,
+        branches: 75,
+        functions: 80,
+        lines: 80,
+      },
+      exclude: [
+        'node_modules/',
+        'src/types/',
+        '**/*.config.*',
+        '**/*.d.ts',
+      ],
+    },
+  },
+});
+\`\`\`
+
+**覆盖率反模式（要避免）**
+\`\`\`typescript
+// ❌ 为了覆盖率写无意义断言
+it('does something', () => {
+  expect(doSomething()).toBeDefined();  // 没有验证具体行为
+});
+
+// ✅ 验证真实业务逻辑
+it('should return correct discounted price', () => {
+  expect(calculatePrice(100, 0.1)).toBe(90);
+});
+\`\`\`
+
+**CI 集成**
+\`\`\`yaml
+# .github/workflows/test.yml
+- name: Run tests with coverage
+  run: npm run test -- --coverage
+
+- name: Comment coverage on PR
+  uses: MishaKav/jest-coverage-comment@main
+  with:
+    coverage-summary-path: ./coverage/coverage-summary.json
+\`\`\``,
+        },
+    ],
+    outputFormat: 'Markdown 测试方案文档，含测试用例设计（AAA 格式）、Mock 策略说明、覆盖率目标和 CI 配置示例',
+    examples: [
+        {
+            input: '帮我为这个 calculateTax 函数写单元测试',
+            output: `## 单元测试方案：calculateTax
+
+\`\`\`typescript
+describe('calculateTax', () => {
+  describe('standard rate (20%)', () => {
+    it('should apply 20% tax to positive amount', () => {
+      expect(calculateTax(100, 'standard')).toBe(20);
+    });
+    it('should return 0 tax for zero amount', () => {
+      expect(calculateTax(0, 'standard')).toBe(0);
+    });
+  });
+
+  describe('error cases', () => {
+    it('should throw when amount is negative', () => {
+      expect(() => calculateTax(-1, 'standard')).toThrow('Amount must be positive');
+    });
+    it('should throw for unknown tax category', () => {
+      expect(() => calculateTax(100, 'unknown' as any)).toThrow('Unknown tax category');
+    });
+  });
+});
+\`\`\``,
+        },
+    ],
+    notes: [
+        '测试应该是自文档化的，好的测试名称比注释更有价值',
+        '避免测试实现细节，测试行为而非内部结构，有助于重构时测试不频繁失败',
+        '不要追求 100% 覆盖率，关注核心业务逻辑的质量覆盖',
+        '测试代码同样需要维护，避免过度复杂的测试辅助函数',
+    ],
+    nextSkill: 'system-design',
+};
+//# sourceMappingURL=16-unit-testing.js.map

package/dist/skills/16-unit-testing.js.map

@@ -0,0 +1 @@
+{"version":3,"file":"16-unit-testing.js","sourceRoot":"","sources":["../../src/skills/16-unit-testing.ts"],"names":[],"mappings":";;;AAEa,QAAA,gBAAgB,GAAoB;IAC/C,EAAE,EAAE,cAAc;IAClB,IAAI,EAAE,MAAM;IACZ,MAAM,EAAE,cAAc;IACtB,KAAK,EAAE,EAAE;IACT,QAAQ,EAAE,KAAK;IACf,WAAW,EAAE,8CAA8C;IAC3D,aAAa,EAAE,uGAAuG;IACtH,iBAAiB,EAAE;qDACgC;IACnD,QAAQ,EAAE;QACR,MAAM;QACN,WAAW;QACX,KAAK;QACL,aAAa;QACb,KAAK;QACL,MAAM;QACN,aAAa;QACb,SAAS;QACT,SAAS;QACT,OAAO;QACP,UAAU;QACV,aAAa;QACb,qBAAqB;KACtB;IACD,KAAK,EAAE;QACL;YACE,KAAK,EAAE,cAAc;YACrB,OAAO,EAAE;;;;;;;;;;;;;;;;;;;;;;;;qBAwBM;SAChB;QACD;YACE,KAAK,EAAE,iBAAiB;YACxB,OAAO,EAAE;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;OA+CR;SACF;QACD;YACE,KAAK,EAAE,oBAAoB;YAC3B,OAAO,EAAE;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;iBA4CE;SACZ;QACD;YACE,KAAK,EAAE,yBAAyB;YAChC,OAAO,EAAE;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;4DAsC6C;SACvD;QACD;YACE,KAAK,EAAE,eAAe;YACtB,OAAO,EAAE;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;OAyDR;SACF;KACF;IACD,YAAY,EACV,0DAA0D;IAC5D,QAAQ,EAAE;QACR;YACE,KAAK,EAAE,4BAA4B;YACnC,MAAM,EAAE;;;;;;;;;;;;;;;;;;;;;;OAsBP;SACF;KACF;IACD,KAAK,EAAE;QACL,0BAA0B;QAC1B,mCAAmC;QACnC,6BAA6B;QAC7B,0BAA0B;KAC3B;IACD,SAAS,EAAE,eAAe;CAC3B,CAAC"}

package/dist/skills/17-system-design.d.ts

@@ -0,0 +1,3 @@
+import type { SkillDefinition } from './types';
+export declare const systemDesignSkill: SkillDefinition;
+//# sourceMappingURL=17-system-design.d.ts.map

package/dist/skills/17-system-design.d.ts.map

@@ -0,0 +1 @@
+{"version":3,"file":"17-system-design.d.ts","sourceRoot":"","sources":["../../src/skills/17-system-design.ts"],"names":[],"mappings":"AAAA,OAAO,KAAK,EAAE,eAAe,EAAE,MAAM,SAAS,CAAC;AAE/C,eAAO,MAAM,iBAAiB,EAAE,eAkS/B,CAAC"}