@neyugn/agent-kits 0.5.0 → 0.5.3

package/kits/coder/skills/testing-patterns/SKILL.md

@@ -7,7 +7,16 @@ version: 2.0
 
 # Testing Patterns - Principles & Best Practices
 
-> **Philosophy:** Tests are documentation. Write tests that explain behavior, not implementation.
+## ⚡ Quick Reference
+
+- **Pyramid**: Unit (fast/many) > Integration (medium) > E2E (slow/few) · 70/20/10 ratio
+- **AAA pattern**: Arrange (setup) → Act (execute) → Assert (verify) · 1 assertion per concern
+- **Naming**: `describe('UserService')` / `it('should throw when email invalid')` · behavior not implementation
+- **Mocking**: Mock external deps (DB/API/time) · Don't mock internals · Use MSW for HTTP
+- **Coverage**: 80%+ meaningful coverage · 100% for critical paths · Coverage ≠ quality
+- **Avoid**: `expect(true).toBe(true)` · Tests that test mocks · Snapshot tests for logic
+
+---
 
 ---
 
@@ -85,13 +94,11 @@ describe("UserService", () => {
 
 ### What to Unit Test
 
-| ✅ Test        | ❌ Don't Test            |
-| -------------- | ------------------------ |
-| Business logic | Framework code           |
-| Edge cases     | Third-party libraries    |
-| Error handling | Simple getters/setters   |
-| Calculations   | Constructor-only classes |
-| Validations    | Private methods directly |
+- Business logic: Framework code
+- Edge cases: Third-party libraries
+- Error handling: Simple getters/setters
+- Calculations: Constructor-only classes
+- Validations: Private methods directly
 
 ---
 
@@ -139,13 +146,11 @@ describe("UserAPI", () => {
 
 ### When to Mock
 
-| ✅ Mock                  | ❌ Don't Mock       |
-| ------------------------ | ------------------- |
-| External APIs            | Code under test     |
-| Database (in unit tests) | Simple dependencies |
-| Time/Date                | Pure functions      |
-| Network calls            | In-memory stores    |
-| Third-party services     | Internal modules    |
+- External APIs: Code under test
+- Database (in unit tests): Simple dependencies
+- Time/Date: Pure functions
+- Network calls: In-memory stores
+- Third-party services: Internal modules
 
 ### Mock Types
 
@@ -213,12 +218,10 @@ const user = createUser({ role: "admin" });
 
 ## Test Naming Conventions
 
-| Pattern                           | Example                                  |
-| --------------------------------- | ---------------------------------------- |
-| **should_behavior**               | `should returnErrorWhenUserNotFound`     |
-| **when_condition**                | `whenUserIsNull_throwsError`             |
-| **given_when_then**               | `givenValidInput_whenSubmit_thenSuccess` |
-| **methodName_condition_expected** | `getUser_withInvalidId_returnsNull`      |
+- **should_behavior**: `should returnErrorWhenUserNotFound`
+- **when_condition**: `whenUserIsNull_throwsError`
+- **given_when_then**: `givenValidInput_whenSubmit_thenSuccess`
+- **methodName_condition_expected**: `getUser_withInvalidId_returnsNull`
 
 **Rule:** Test names should describe behavior, not implementation.
 
@@ -226,34 +229,28 @@ const user = createUser({ role: "admin" });
 
 ## Coverage Guidelines
 
-| Level    | Meaning                             |
-| -------- | ----------------------------------- |
-| **80%+** | Good coverage for most projects     |
-| **90%+** | High coverage, critical systems     |
-| **100%** | Often impractical, may waste effort |
+- **80%+**: Good coverage for most projects
+- **90%+**: High coverage, critical systems
+- **100%**: Often impractical, may waste effort
 
 ### What to Cover
 
-| Priority   | Area                         |
-| ---------- | ---------------------------- |
-| **High**   | Business logic, calculations |
-| **Medium** | Error paths, edge cases      |
-| **Low**    | Simple getters, boilerplate  |
+- **High**: Business logic, calculations
+- **Medium**: Error paths, edge cases
+- **Low**: Simple getters, boilerplate
 
 ---
 
 ## Anti-Patterns
 
-| ❌ Don't                    | ✅ Do                      |
-| --------------------------- | -------------------------- |
-| Test implementation details | Test behavior/outcomes     |
-| Duplicate test code         | Use factories/helpers      |
-| Complex test setup          | Simplify or split tests    |
-| Ignore flaky tests          | Fix root cause immediately |
-| Skip cleanup                | Reset state every test     |
-| Test multiple things        | One assertion per test     |
-| Hard-code test data         | Use factories              |
-| Rely on test order          | Make tests independent     |
+- Test implementation details: Test behavior/outcomes
+- Duplicate test code: Use factories/helpers
+- Complex test setup: Simplify or split tests
+- Ignore flaky tests: Fix root cause immediately
+- Skip cleanup: Reset state every test
+- Test multiple things: One assertion per test
+- Hard-code test data: Use factories
+- Rely on test order: Make tests independent
 
 ---
 
@@ -273,12 +270,10 @@ Before pushing code:
 
 ## Related Skills
 
-| Need                | Skill                   |
-| ------------------- | ----------------------- |
-| Clean code          | `clean-code`            |
-| API testing         | `api-patterns`          |
-| E2E testing         | `webapp-testing`        |
-| Performance testing | `performance-profiling` |
+- Clean code: `clean-code`
+- API testing: `api-patterns`
+- E2E testing: `webapp-testing`
+- Performance testing: `performance-profiling`
 
 ---
 

package/kits/coder/skills/typescript-patterns/SKILL.md

@@ -6,17 +6,25 @@ allowed-tools: Read, Write, Edit, Glob, Grep
 
 # TypeScript Patterns
 
-> **Philosophy:** Types are documentation that never lies. Strong types catch bugs at compile time, not runtime.
+## ⚡ Quick Reference
+
+- **Strict mode**: Always `"strict": true` · `noUncheckedIndexedAccess` · `exactOptionalPropertyTypes`
+- **No `any`**: Use `unknown` for unknowns · type narrowing with guards · `as` only as last resort
+- **Utilities**: `Partial<T>` · `Required<T>` · `Pick<T,K>` · `Omit<T,K>` · `Record<K,V>` · `ReturnType<F>`
+- **Type vs Interface**: `interface` for objects/classes (extendable) · `type` for unions/intersections
+- **Async**: Always type return `Promise<T>` · use `try/catch` with typed errors · avoid `void` for important returns
+- **Inference**: Let TS infer when obvious · explicit types for API boundaries and function params
+
+---
+
 
 ---
 
 ## 📑 Content Map
 
-| File                            | When to Read            |
-| ------------------------------- | ----------------------- |
-| `references/advanced-types.md`  | Complex type gymnastics |
-| `references/error-patterns.md`  | Debugging type errors   |
-| `references/migration-guide.md` | JS to TS migration      |
+- `references/advanced-types.md`: Complex type gymnastics
+- `references/error-patterns.md`: Debugging type errors
+- `references/migration-guide.md`: JS to TS migration
 
 ---
 
@@ -144,12 +152,10 @@ type GetUserParams = Parameters<typeof getUser>;
 }
 ```
 
-| Flag                         | What It Does                                 |
-| ---------------------------- | -------------------------------------------- |
-| `strict`                     | Enables all strict type checking             |
-| `noUncheckedIndexedAccess`   | Array/object access returns `T \| undefined` |
-| `noImplicitOverride`         | Require `override` keyword                   |
-| `exactOptionalPropertyTypes` | Distinguish `undefined` vs missing           |
+- `strict`: Enables all strict type checking
+- `noUncheckedIndexedAccess`: Array/object access returns `T \
+- `noImplicitOverride`: Require `override` keyword
+- `exactOptionalPropertyTypes`: Distinguish `undefined` vs missing
 
 ---
 
@@ -162,12 +168,10 @@ type GetUserParams = Parameters<typeof getUser>;
 npx tsc --extendedDiagnostics --incremental false | grep -E "Check time|Files:|Lines:|Nodes:"
 ```
 
-| Problem                       | Solution                    |
-| ----------------------------- | --------------------------- |
-| Slow type checking            | Enable `skipLibCheck: true` |
-| Rebuilding everything         | Enable `incremental: true`  |
-| Large codebase                | Use Project References      |
-| "Type instantiation too deep" | See fixes below             |
+- Slow type checking: Enable `skipLibCheck: true`
+- Rebuilding everything: Enable `incremental: true`
+- Large codebase: Use Project References
+- "Type instantiation too deep": See fixes below
 
 ### Fixing "Type instantiation is excessively deep"
 
@@ -194,20 +198,16 @@ type Prev = [never, 0, 1, 2, 3, 4];
 
 ### "The inferred type of X cannot be named"
 
-| Cause               | Fix                                  |
-| ------------------- | ------------------------------------ |
-| Missing type export | Export the required type explicitly  |
-| Circular dependency | Use type-only imports: `import type` |
-| Complex inference   | Use `ReturnType<typeof fn>`          |
+- Missing type export: Export the required type explicitly
+- Circular dependency: Use type-only imports: `import type`
+- Complex inference: Use `ReturnType<typeof fn>`
 
 ### "Cannot find module"
 
-| Cause                  | Fix                                       |
-| ---------------------- | ----------------------------------------- |
-| Wrong moduleResolution | Match your bundler (node, bundler)        |
-| Missing baseUrl        | Check tsconfig paths alignment            |
-| Monorepo setup         | Use workspace protocol (workspace:\*)     |
-| Cache issues           | `rm -rf node_modules/.cache .tsbuildinfo` |
+- Wrong moduleResolution: Match your bundler (node, bundler)
+- Missing baseUrl: Check tsconfig paths alignment
+- Monorepo setup: Use workspace protocol (workspace:\*)
+- Cache issues: `rm -rf node_modules/.cache .tsbuildinfo`
 
 ### Missing Type Declarations
 
@@ -231,11 +231,9 @@ declare module "cjs-package" {
 
 ### ESM-First Approach
 
-| Configuration       | Value                     |
-| ------------------- | ------------------------- |
-| package.json `type` | `"module"`                |
-| moduleResolution    | `"bundler"` or `"node16"` |
-| module              | `"esnext"` or `"node16"`  |
+- package.json `type`: `"module"`
+- moduleResolution: `"bundler"` or `"node16"`
+- module: `"esnext"` or `"node16"`
 
 ### CJS Interop
 
@@ -269,10 +267,8 @@ const defaultExport = pkg.default || pkg;
 
 ### Nx vs Turborepo Decision
 
-| Choose        | When                                  |
-| ------------- | ------------------------------------- |
-| **Turborepo** | Simple structure, speed, <20 packages |
-| **Nx**        | Complex deps, visualization, plugins  |
+- **Turborepo**: Simple structure, speed, <20 packages
+- **Nx**: Complex deps, visualization, plugins
 
 ---
 
@@ -280,10 +276,8 @@ const defaultExport = pkg.default || pkg;
 
 ### Biome vs ESLint
 
-| Choose     | When                                     |
-| ---------- | ---------------------------------------- |
-| **Biome**  | Speed critical, single tool, TS-first    |
-| **ESLint** | Need specific rules, Vue/Angular support |
+- **Biome**: Speed critical, single tool, TS-first
+- **ESLint**: Need specific rules, Vue/Angular support
 
 ### Type Testing
 
@@ -357,15 +351,13 @@ function handleStatus(status: Status) {
 
 ## 10. Anti-Patterns
 
-| ❌ Don't                          | ✅ Do                            |
-| --------------------------------- | -------------------------------- |
-| `any` everywhere                  | `unknown` with type guards       |
-| Excessive type assertions (`as`)  | Proper type inference            |
-| Ignoring strict mode              | Enable all strict flags          |
-| Complex mapped types in hot paths | Pre-compute, cache types         |
-| Index as object key without check | `noUncheckedIndexedAccess: true` |
-| Type in separate file from impl   | Co-locate types with code        |
-| Over-complicated generics         | Simpler solution first           |
+- `any` everywhere: `unknown` with type guards
+- Excessive type assertions (`as`): Proper type inference
+- Ignoring strict mode: Enable all strict flags
+- Complex mapped types in hot paths: Pre-compute, cache types
+- Index as object key without check: `noUncheckedIndexedAccess: true`
+- Type in separate file from impl: Co-locate types with code
+- Over-complicated generics: Simpler solution first
 
 ---
 
@@ -373,20 +365,16 @@ function handleStatus(status: Status) {
 
 ### JavaScript to TypeScript
 
-| Phase | Action                                  |
-| ----- | --------------------------------------- |
-| 1     | Enable `allowJs`, `checkJs` in tsconfig |
-| 2     | Rename files `.js` → `.ts` gradually    |
-| 3     | Add types file by file                  |
-| 4     | Enable strict mode progressively        |
+- 1: Enable `allowJs`, `checkJs` in tsconfig
+- 2: Rename files `.js` → `.ts` gradually
+- 3: Add types file by file
+- 4: Enable strict mode progressively
 
 ### Useful Tools
 
-| Tool         | Purpose                         |
-| ------------ | ------------------------------- |
-| `ts-migrate` | Automated migration from Airbnb |
-| `typesync`   | Install missing @types          |
-| `TypeStat`   | Auto-fix TypeScript types       |
+- `ts-migrate`: Automated migration from Airbnb
+- `typesync`: Install missing @types
+- `TypeStat`: Auto-fix TypeScript types
 
 ---
 
@@ -405,12 +393,10 @@ function handleStatus(status: Status) {
 
 ## Related Skills
 
-| Need               | Skill                   |
-| ------------------ | ----------------------- |
-| React + TypeScript | `react-patterns`        |
-| API design         | `api-patterns`          |
-| Testing            | `testing-patterns`      |
-| Performance        | `performance-profiling` |
+- React + TypeScript: `react-patterns`
+- API design: `api-patterns`
+- Testing: `testing-patterns`
+- Performance: `performance-profiling`
 
 ---