omgkit 2.25.1 → 2.26.1
This diff represents the content of publicly available package versions that have been released to one of the supported registries. The information contained in this diff is provided for informational purposes only and reflects changes between package versions as they appear in their respective public registries.
- package/README.md +9 -1
- package/lib/cli.js +1 -1
- package/package.json +1 -1
- package/plugin/agents/fullstack-developer.md +44 -4
- package/plugin/agents/sprint-master.md +25 -1
- package/plugin/agents/tester.md +45 -4
- package/plugin/commands/dev/feature-tested.md +33 -2
- package/plugin/commands/dev/feature.md +80 -3
- package/plugin/commands/dev/fix-fast.md +64 -3
- package/plugin/commands/dev/fix-hard.md +83 -11
- package/plugin/commands/dev/fix.md +72 -5
- package/plugin/commands/dev/tdd.md +55 -2
- package/plugin/commands/dev/test-write.md +37 -1
- package/plugin/commands/dev/test.md +73 -2
- package/plugin/commands/quality/optimize.md +70 -7
- package/plugin/commands/quality/refactor.md +64 -5
- package/plugin/commands/sprint/team-run.md +135 -18
- package/plugin/registry.yaml +2 -2
- package/plugin/skills/methodology/test-driven-development/SKILL.md +45 -4
- package/plugin/skills/methodology/test-enforcement/SKILL.md +33 -0
- package/plugin/skills/methodology/test-task-generation/SKILL.md +33 -0
- package/plugin/workflows/development/bug-fix.md +53 -6
- package/plugin/workflows/development/feature.md +52 -4
- package/plugin/workflows/testing/automated-testing.md +26 -1
- package/plugin/workflows/testing/test-driven-development.md +52 -3
package/README.md
CHANGED
|
@@ -278,13 +278,21 @@ Commands are slash-prefixed actions organized by namespace.
|
|
|
278
278
|
```bash
|
|
279
279
|
/dev:feature <desc> # Full feature development
|
|
280
280
|
/dev:fix <error> # Debug and fix bugs
|
|
281
|
-
/dev:fix-fast <error> # Quick bug fix
|
|
281
|
+
/dev:fix-fast <error> # Quick bug fix (tests optional)
|
|
282
282
|
/dev:fix-hard <error> # Complex bug (deep analysis)
|
|
283
283
|
/dev:test <scope> # Generate tests
|
|
284
284
|
/dev:tdd <feature> # Test-driven development
|
|
285
285
|
/dev:review [file] # Code review
|
|
286
286
|
```
|
|
287
287
|
|
|
288
|
+
**Testing Options** (available on most dev commands):
|
|
289
|
+
```bash
|
|
290
|
+
/dev:feature "login" --no-test # Skip test enforcement
|
|
291
|
+
/dev:fix "bug" --test-level strict # Override enforcement level
|
|
292
|
+
/dev:feature-tested "auth" --coverage 90 # Custom coverage target
|
|
293
|
+
/dev:fix-fast "typo" --with-test # Opt-in to testing
|
|
294
|
+
```
|
|
295
|
+
|
|
288
296
|
### Planning (`/planning:*`)
|
|
289
297
|
|
|
290
298
|
```bash
|
package/lib/cli.js
CHANGED
|
@@ -51,7 +51,7 @@ ${COLORS.magenta}╔════════════════════
|
|
|
51
51
|
║ ║
|
|
52
52
|
║ 🔮 OMGKIT - Omega-Level Development Kit ║
|
|
53
53
|
║ ║
|
|
54
|
-
║ 41 Agents •
|
|
54
|
+
║ 41 Agents • 160 Commands • 161 Skills • 10 Modes ║
|
|
55
55
|
║ "Think Omega. Build Omega. Be Omega." ║
|
|
56
56
|
║ ║
|
|
57
57
|
╚════════════════════════════════════════════════════════════╝${COLORS.reset}
|
package/package.json
CHANGED
|
@@ -7,14 +7,18 @@ skills:
|
|
|
7
7
|
- methodology/executing-plans
|
|
8
8
|
- methodology/verification-before-completion
|
|
9
9
|
- methodology/test-driven-development
|
|
10
|
+
- methodology/test-enforcement
|
|
10
11
|
- languages/typescript
|
|
11
12
|
- languages/javascript
|
|
12
13
|
commands:
|
|
13
14
|
- /dev:feature
|
|
15
|
+
- /dev:feature-tested
|
|
14
16
|
- /dev:fix
|
|
15
17
|
- /dev:fix-fast
|
|
16
18
|
- /dev:fix-hard
|
|
17
19
|
- /dev:tdd
|
|
20
|
+
- /quality:refactor
|
|
21
|
+
- /quality:optimize
|
|
18
22
|
---
|
|
19
23
|
|
|
20
24
|
# ⚡ Fullstack Developer Agent
|
|
@@ -432,9 +436,45 @@ Before marking task complete:
|
|
|
432
436
|
|
|
433
437
|
---
|
|
434
438
|
|
|
439
|
+
## Test Enforcement
|
|
440
|
+
|
|
441
|
+
### Configuration
|
|
442
|
+
|
|
443
|
+
Testing is enforced based on `.omgkit/workflow.yaml`:
|
|
444
|
+
|
|
445
|
+
```yaml
|
|
446
|
+
testing:
|
|
447
|
+
enabled: true
|
|
448
|
+
enforcement:
|
|
449
|
+
level: standard # soft | standard | strict
|
|
450
|
+
```
|
|
451
|
+
|
|
452
|
+
### Command Options
|
|
453
|
+
|
|
454
|
+
| Option | Description | Example |
|
|
455
|
+
|--------|-------------|---------|
|
|
456
|
+
| `--no-test` | Skip test enforcement | `/dev:fix "typo" --no-test` |
|
|
457
|
+
| `--test-level <level>` | Override enforcement level | `/dev:feature "auth" --test-level strict` |
|
|
458
|
+
| `--coverage <percent>` | Override coverage minimum | `/dev:feature "api" --coverage 95` |
|
|
459
|
+
|
|
460
|
+
### Behavior by Command
|
|
461
|
+
|
|
462
|
+
| Command | Default | Testing Behavior |
|
|
463
|
+
|---------|---------|------------------|
|
|
464
|
+
| `/dev:feature` | Tests enabled | Regression + unit tests |
|
|
465
|
+
| `/dev:feature-tested` | Tests enforced | Full test suite auto-generated |
|
|
466
|
+
| `/dev:fix` | Tests enabled | Regression test required |
|
|
467
|
+
| `/dev:fix-fast` | Tests disabled | Optional with `--with-test` |
|
|
468
|
+
| `/dev:fix-hard` | Tests enabled | Comprehensive testing |
|
|
469
|
+
| `/dev:tdd` | Tests enforced | Test-first development |
|
|
470
|
+
|
|
435
471
|
## Commands
|
|
436
472
|
|
|
437
|
-
- `/feature [description]` - Implement a feature
|
|
438
|
-
- `/
|
|
439
|
-
- `/
|
|
440
|
-
- `/
|
|
473
|
+
- `/dev:feature [description]` - Implement a feature
|
|
474
|
+
- `/dev:feature-tested [desc]` - Feature with auto-generated tests
|
|
475
|
+
- `/dev:fix [issue]` - Fix a bug with regression test
|
|
476
|
+
- `/dev:fix-fast [issue]` - Quick fix (tests optional)
|
|
477
|
+
- `/dev:fix-hard [issue]` - Deep investigation with tests
|
|
478
|
+
- `/dev:tdd [feature]` - Test-driven development
|
|
479
|
+
- `/quality:refactor [target]` - Refactor with test verification
|
|
480
|
+
- `/quality:optimize [target]` - Optimize with test verification
|
|
@@ -259,11 +259,14 @@ Generates:
|
|
|
259
259
|
|
|
260
260
|
### Configuration Loading
|
|
261
261
|
|
|
262
|
-
At sprint start, read `.omgkit/workflow.yaml` for testing configuration
|
|
262
|
+
At sprint start, read `.omgkit/workflow.yaml` for testing configuration.
|
|
263
|
+
|
|
264
|
+
**Via workflow.yaml:**
|
|
263
265
|
|
|
264
266
|
```yaml
|
|
265
267
|
# .omgkit/workflow.yaml
|
|
266
268
|
testing:
|
|
269
|
+
enabled: true
|
|
267
270
|
enforcement:
|
|
268
271
|
level: standard # soft | standard | strict
|
|
269
272
|
auto_generate_tasks: true
|
|
@@ -282,6 +285,27 @@ testing:
|
|
|
282
285
|
on_coverage_below_minimum: true
|
|
283
286
|
```
|
|
284
287
|
|
|
288
|
+
**Via CLI:**
|
|
289
|
+
|
|
290
|
+
```bash
|
|
291
|
+
# Set enforcement level
|
|
292
|
+
omgkit config set testing.enforcement.level strict
|
|
293
|
+
|
|
294
|
+
# Enable/disable auto-generation
|
|
295
|
+
omgkit config set testing.auto_generate_tasks true
|
|
296
|
+
|
|
297
|
+
# View testing config
|
|
298
|
+
omgkit config list testing
|
|
299
|
+
```
|
|
300
|
+
|
|
301
|
+
**Via Command Options:**
|
|
302
|
+
|
|
303
|
+
| Option | Description | Example |
|
|
304
|
+
|--------|-------------|---------|
|
|
305
|
+
| `--no-test` | Skip test enforcement | `/sprint:team-run --no-test` |
|
|
306
|
+
| `--test-level <level>` | Override enforcement level | `/sprint:team-run --test-level strict` |
|
|
307
|
+
| `--mode <mode>` | Autonomy mode | `/sprint:team-run --mode full-auto` |
|
|
308
|
+
|
|
285
309
|
### Auto Test Task Generation
|
|
286
310
|
|
|
287
311
|
When `auto_generate_tasks: true`, automatically create test tasks:
|
package/plugin/agents/tester.md
CHANGED
|
@@ -5,14 +5,20 @@ tools: Read, Write, Bash, Glob, Grep, Task
|
|
|
5
5
|
model: inherit
|
|
6
6
|
skills:
|
|
7
7
|
- methodology/test-driven-development
|
|
8
|
+
- methodology/test-enforcement
|
|
9
|
+
- methodology/test-task-generation
|
|
8
10
|
- methodology/testing-anti-patterns
|
|
9
11
|
- testing/vitest
|
|
10
12
|
- testing/playwright
|
|
11
13
|
- testing/pytest
|
|
12
14
|
commands:
|
|
13
15
|
- /dev:test
|
|
16
|
+
- /dev:test-write
|
|
14
17
|
- /dev:tdd
|
|
15
18
|
- /dev:fix-test
|
|
19
|
+
- /dev:feature-tested
|
|
20
|
+
- /quality:verify-done
|
|
21
|
+
- /quality:coverage-check
|
|
16
22
|
---
|
|
17
23
|
|
|
18
24
|
# 🧪 Tester Agent
|
|
@@ -577,9 +583,44 @@ Before marking testing complete:
|
|
|
577
583
|
|
|
578
584
|
---
|
|
579
585
|
|
|
586
|
+
## Test Enforcement Integration
|
|
587
|
+
|
|
588
|
+
### Configuration
|
|
589
|
+
|
|
590
|
+
Read testing configuration from `.omgkit/workflow.yaml`:
|
|
591
|
+
|
|
592
|
+
```yaml
|
|
593
|
+
testing:
|
|
594
|
+
enabled: true
|
|
595
|
+
enforcement:
|
|
596
|
+
level: standard # soft | standard | strict
|
|
597
|
+
coverage_gates:
|
|
598
|
+
unit:
|
|
599
|
+
minimum: 80
|
|
600
|
+
target: 90
|
|
601
|
+
```
|
|
602
|
+
|
|
603
|
+
### Command Options
|
|
604
|
+
|
|
605
|
+
| Option | Description | Example |
|
|
606
|
+
|--------|-------------|---------|
|
|
607
|
+
| `--coverage <percent>` | Override coverage minimum | `/dev:test "src/" --coverage 90` |
|
|
608
|
+
| `--test-types <types>` | Specify test types | `/dev:test "api/" --test-types unit,integration` |
|
|
609
|
+
| `--watch` | Run in watch mode | `/dev:test "src/" --watch` |
|
|
610
|
+
| `--fail-under <percent>` | Fail if coverage below | `/dev:test "lib/" --fail-under 80` |
|
|
611
|
+
|
|
612
|
+
### Enforcement Behavior
|
|
613
|
+
|
|
614
|
+
When enforcement is enabled:
|
|
615
|
+
- Block task completion if tests fail
|
|
616
|
+
- Block if coverage below minimum
|
|
617
|
+
- Warn about missing test types
|
|
618
|
+
|
|
580
619
|
## Commands
|
|
581
620
|
|
|
582
|
-
- `/test [target]` - Run tests for target
|
|
583
|
-
- `/test
|
|
584
|
-
- `/
|
|
585
|
-
- `/
|
|
621
|
+
- `/dev:test [target]` - Run tests for target
|
|
622
|
+
- `/dev:test-write [file]` - Write comprehensive tests
|
|
623
|
+
- `/dev:tdd [feature]` - Test-driven development workflow
|
|
624
|
+
- `/dev:feature-tested [desc]` - Feature with auto-generated tests
|
|
625
|
+
- `/quality:verify-done` - Verify test requirements
|
|
626
|
+
- `/quality:coverage-check` - Check coverage gates
|
|
@@ -1,6 +1,6 @@
|
|
|
1
1
|
---
|
|
2
2
|
name: Feature Tested
|
|
3
|
-
description: Create a feature with automatically generated test tasks
|
|
3
|
+
description: Create a feature with automatically generated test tasks and strict test enforcement
|
|
4
4
|
category: dev
|
|
5
5
|
related_skills:
|
|
6
6
|
- methodology/test-task-generation
|
|
@@ -12,6 +12,10 @@ related_commands:
|
|
|
12
12
|
- /dev:feature
|
|
13
13
|
- /dev:test
|
|
14
14
|
allowed-tools: Task, Read, Write, Bash, Grep, Glob
|
|
15
|
+
argument-hint: <feature-description> [--coverage PERCENT] [--test-types TYPES] [--tdd] [--strict]
|
|
16
|
+
testing:
|
|
17
|
+
default: true
|
|
18
|
+
configurable: false
|
|
15
19
|
---
|
|
16
20
|
|
|
17
21
|
# /dev:feature-tested
|
|
@@ -26,7 +30,11 @@ Build a feature with automatically generated test tasks. This command ensures co
|
|
|
26
30
|
/dev:feature-tested "Payment processing" --test-types unit,integration,e2e
|
|
27
31
|
```
|
|
28
32
|
|
|
29
|
-
## Options
|
|
33
|
+
## Testing Options
|
|
34
|
+
|
|
35
|
+
This command **always enforces testing** (testing cannot be disabled). This is a strict version of `/dev:feature`.
|
|
36
|
+
|
|
37
|
+
### Options
|
|
30
38
|
|
|
31
39
|
| Option | Description | Default |
|
|
32
40
|
|--------|-------------|---------|
|
|
@@ -35,6 +43,29 @@ Build a feature with automatically generated test tasks. This command ensures co
|
|
|
35
43
|
| `--tdd` | Use TDD approach (tests first) | false |
|
|
36
44
|
| `--strict` | Strict enforcement (no overrides) | false |
|
|
37
45
|
|
|
46
|
+
### Enforcement Behavior
|
|
47
|
+
|
|
48
|
+
Unlike `/dev:feature`, this command:
|
|
49
|
+
- **Cannot skip tests** (no `--no-test` option)
|
|
50
|
+
- **Blocks completion** until all test tasks pass
|
|
51
|
+
- **Auto-generates test tasks** for every implementation task
|
|
52
|
+
|
|
53
|
+
### Configuration
|
|
54
|
+
|
|
55
|
+
Configure via `.omgkit/workflow.yaml`:
|
|
56
|
+
|
|
57
|
+
```yaml
|
|
58
|
+
testing:
|
|
59
|
+
enabled: true
|
|
60
|
+
enforcement:
|
|
61
|
+
level: strict # Recommended for feature-tested
|
|
62
|
+
auto_generate_tasks: true
|
|
63
|
+
coverage_gates:
|
|
64
|
+
unit:
|
|
65
|
+
minimum: 80
|
|
66
|
+
target: 90
|
|
67
|
+
```
|
|
68
|
+
|
|
38
69
|
## How It Works
|
|
39
70
|
|
|
40
71
|
### 1. Feature Analysis
|
|
@@ -1,25 +1,102 @@
|
|
|
1
1
|
---
|
|
2
2
|
description: Full feature development with planning, testing, and review
|
|
3
3
|
allowed-tools: Task, Read, Write, Bash, Grep, Glob
|
|
4
|
-
argument-hint: <feature description>
|
|
4
|
+
argument-hint: <feature description> [--no-test] [--test-level LEVEL]
|
|
5
|
+
related_skills:
|
|
6
|
+
- methodology/writing-plans
|
|
7
|
+
- methodology/test-enforcement
|
|
8
|
+
- methodology/test-task-generation
|
|
9
|
+
related_commands:
|
|
10
|
+
- /dev:feature-tested
|
|
11
|
+
- /quality:verify-done
|
|
12
|
+
- /dev:tdd
|
|
13
|
+
testing:
|
|
14
|
+
default: true
|
|
15
|
+
configurable: true
|
|
5
16
|
---
|
|
6
17
|
|
|
7
|
-
#
|
|
18
|
+
# Feature: $ARGUMENTS
|
|
8
19
|
|
|
9
20
|
Build feature: **$ARGUMENTS**
|
|
10
21
|
|
|
11
22
|
## Workflow
|
|
23
|
+
|
|
12
24
|
1. **Plan** (planner) - Create implementation plan
|
|
13
25
|
2. **Implement** (fullstack-developer) - Write code
|
|
14
26
|
3. **Test** (tester) - Write and run tests
|
|
15
27
|
4. **Review** (code-reviewer) - Code review
|
|
16
28
|
5. **Commit** (git-manager) - Create commit
|
|
17
29
|
|
|
30
|
+
## Testing Options
|
|
31
|
+
|
|
32
|
+
This command respects project testing configuration from `.omgkit/workflow.yaml`.
|
|
33
|
+
|
|
34
|
+
### Default Behavior
|
|
35
|
+
|
|
36
|
+
- **Testing**: Enabled by default
|
|
37
|
+
- **Enforcement Level**: Read from `testing.enforcement.level` (default: standard)
|
|
38
|
+
- **Auto-generate Tests**: Read from `testing.auto_generate_tasks` (default: true)
|
|
39
|
+
|
|
40
|
+
### Options
|
|
41
|
+
|
|
42
|
+
| Option | Description | Example |
|
|
43
|
+
|--------|-------------|---------|
|
|
44
|
+
| `--no-test` | Skip test enforcement | `/dev:feature "add login" --no-test` |
|
|
45
|
+
| `--test-level <level>` | Override enforcement level | `/dev:feature "add login" --test-level strict` |
|
|
46
|
+
| `--coverage <percent>` | Override minimum coverage | `/dev:feature "add login" --coverage 90` |
|
|
47
|
+
|
|
48
|
+
### Enforcement Levels
|
|
49
|
+
|
|
50
|
+
| Level | Test Failure | Coverage Below Min | Missing Tests |
|
|
51
|
+
|-------|--------------|-------------------|---------------|
|
|
52
|
+
| `soft` | Warning | Warning | Warning |
|
|
53
|
+
| `standard` | Block | Block | Warning |
|
|
54
|
+
| `strict` | Block | Block | Block |
|
|
55
|
+
|
|
56
|
+
## Configuration
|
|
57
|
+
|
|
58
|
+
Configure via `.omgkit/workflow.yaml`:
|
|
59
|
+
|
|
60
|
+
```yaml
|
|
61
|
+
testing:
|
|
62
|
+
enabled: true
|
|
63
|
+
enforcement:
|
|
64
|
+
level: standard # soft | standard | strict
|
|
65
|
+
auto_generate_tasks: true
|
|
66
|
+
coverage_gates:
|
|
67
|
+
unit:
|
|
68
|
+
minimum: 80
|
|
69
|
+
```
|
|
70
|
+
|
|
71
|
+
Or via CLI:
|
|
72
|
+
|
|
73
|
+
```bash
|
|
74
|
+
omgkit config set testing.enforcement.level strict
|
|
75
|
+
```
|
|
76
|
+
|
|
18
77
|
## Progress
|
|
78
|
+
|
|
19
79
|
- [ ] Planning complete
|
|
20
80
|
- [ ] Implementation complete
|
|
21
|
-
- [ ] Tests passing
|
|
81
|
+
- [ ] Tests passing (when enabled)
|
|
82
|
+
- [ ] Coverage meets threshold (when enabled)
|
|
22
83
|
- [ ] Review approved
|
|
23
84
|
- [ ] Committed
|
|
24
85
|
|
|
86
|
+
## Examples
|
|
87
|
+
|
|
88
|
+
```bash
|
|
89
|
+
# Default: with test enforcement
|
|
90
|
+
/dev:feature "add user authentication"
|
|
91
|
+
|
|
92
|
+
# With strict enforcement
|
|
93
|
+
/dev:feature "add payment processing" --test-level strict
|
|
94
|
+
|
|
95
|
+
# Skip testing (requires soft level or justification)
|
|
96
|
+
/dev:feature "add logging" --no-test
|
|
97
|
+
|
|
98
|
+
# High coverage requirement
|
|
99
|
+
/dev:feature "add API endpoint" --coverage 95
|
|
100
|
+
```
|
|
101
|
+
|
|
25
102
|
Show progress after each step.
|
|
@@ -1,19 +1,80 @@
|
|
|
1
1
|
---
|
|
2
|
-
description: Quick bug fix with minimal investigation
|
|
2
|
+
description: Quick bug fix with minimal investigation (testing optional)
|
|
3
3
|
allowed-tools: Read, Write, Bash, Grep
|
|
4
|
-
argument-hint: <bug>
|
|
4
|
+
argument-hint: <bug> [--with-test]
|
|
5
|
+
related_skills:
|
|
6
|
+
- methodology/systematic-debugging
|
|
7
|
+
related_commands:
|
|
8
|
+
- /dev:fix
|
|
9
|
+
- /dev:fix-hard
|
|
10
|
+
testing:
|
|
11
|
+
default: false
|
|
12
|
+
configurable: true
|
|
5
13
|
---
|
|
6
14
|
|
|
7
|
-
#
|
|
15
|
+
# Fix Fast: $ARGUMENTS
|
|
8
16
|
|
|
9
17
|
Quick fix for: **$ARGUMENTS**
|
|
10
18
|
|
|
11
19
|
Skip deep analysis. Apply obvious fix immediately.
|
|
12
20
|
|
|
13
21
|
## Process
|
|
22
|
+
|
|
14
23
|
1. Locate the issue
|
|
15
24
|
2. Apply fix
|
|
16
25
|
3. Verify it works
|
|
17
26
|
4. Done
|
|
18
27
|
|
|
19
28
|
Use for obvious bugs with clear solutions.
|
|
29
|
+
|
|
30
|
+
## Testing Options
|
|
31
|
+
|
|
32
|
+
This command has **testing disabled by default** for speed. Use `--with-test` to enable.
|
|
33
|
+
|
|
34
|
+
### Default Behavior
|
|
35
|
+
|
|
36
|
+
- **Testing**: Disabled by default (quick fixes)
|
|
37
|
+
- **Use For**: Typos, obvious one-line fixes, config changes
|
|
38
|
+
|
|
39
|
+
### Options
|
|
40
|
+
|
|
41
|
+
| Option | Description | Example |
|
|
42
|
+
|--------|-------------|---------|
|
|
43
|
+
| `--with-test` | Enable test enforcement | `/dev:fix-fast "typo" --with-test` |
|
|
44
|
+
| `--test-level <level>` | Set enforcement level | `/dev:fix-fast "bug" --with-test --test-level standard` |
|
|
45
|
+
|
|
46
|
+
## When to Use
|
|
47
|
+
|
|
48
|
+
| Use `/dev:fix-fast` | Use `/dev:fix` instead |
|
|
49
|
+
|---------------------|------------------------|
|
|
50
|
+
| Typo in string/message | Logic errors |
|
|
51
|
+
| Obvious one-liner | Multiple files affected |
|
|
52
|
+
| Config value change | Root cause unclear |
|
|
53
|
+
| Import/export fix | Could have side effects |
|
|
54
|
+
|
|
55
|
+
## Configuration
|
|
56
|
+
|
|
57
|
+
Even when `--with-test` is used, respects project config:
|
|
58
|
+
|
|
59
|
+
```yaml
|
|
60
|
+
testing:
|
|
61
|
+
enforcement:
|
|
62
|
+
level: standard
|
|
63
|
+
```
|
|
64
|
+
|
|
65
|
+
## Examples
|
|
66
|
+
|
|
67
|
+
```bash
|
|
68
|
+
# Quick fix without testing (default)
|
|
69
|
+
/dev:fix-fast "typo in error message"
|
|
70
|
+
|
|
71
|
+
# Quick fix but still run tests
|
|
72
|
+
/dev:fix-fast "wrong import path" --with-test
|
|
73
|
+
|
|
74
|
+
# Quick fix with strict testing
|
|
75
|
+
/dev:fix-fast "config value" --with-test --test-level strict
|
|
76
|
+
```
|
|
77
|
+
|
|
78
|
+
## Warning
|
|
79
|
+
|
|
80
|
+
For complex bugs or bugs with unclear root cause, use `/dev:fix` or `/dev:fix-hard` instead.
|
|
@@ -1,25 +1,97 @@
|
|
|
1
1
|
---
|
|
2
|
-
description: Complex bug fix with deep investigation
|
|
2
|
+
description: Complex bug fix with deep investigation and comprehensive testing
|
|
3
3
|
allowed-tools: Task, Read, Write, Bash, Grep, Glob, WebSearch
|
|
4
|
-
argument-hint: <complex bug>
|
|
4
|
+
argument-hint: <complex bug> [--no-test] [--test-level LEVEL]
|
|
5
|
+
related_skills:
|
|
6
|
+
- methodology/systematic-debugging
|
|
7
|
+
- methodology/test-enforcement
|
|
8
|
+
- omega/omega-thinking
|
|
9
|
+
related_commands:
|
|
10
|
+
- /dev:fix
|
|
11
|
+
- /dev:fix-fast
|
|
12
|
+
- /quality:verify-done
|
|
13
|
+
testing:
|
|
14
|
+
default: true
|
|
15
|
+
configurable: true
|
|
5
16
|
---
|
|
6
17
|
|
|
7
|
-
#
|
|
18
|
+
# Fix Hard: $ARGUMENTS
|
|
8
19
|
|
|
9
20
|
Deep investigation for: **$ARGUMENTS**
|
|
10
21
|
|
|
11
22
|
## Workflow
|
|
23
|
+
|
|
12
24
|
1. **Analyze** (oracle) - Apply 7 thinking modes
|
|
13
25
|
2. **Investigate** (debugger) - Multiple hypotheses
|
|
14
26
|
3. **Fix** (fullstack-developer) - Implement solution
|
|
15
27
|
4. **Test** (tester) - Comprehensive testing
|
|
16
28
|
|
|
29
|
+
## Testing Options
|
|
30
|
+
|
|
31
|
+
This command respects project testing configuration from `.omgkit/workflow.yaml`.
|
|
32
|
+
|
|
33
|
+
### Default Behavior
|
|
34
|
+
|
|
35
|
+
- **Testing**: Enabled by default (comprehensive tests for complex bugs)
|
|
36
|
+
- **Enforcement Level**: Read from `testing.enforcement.level` (default: standard)
|
|
37
|
+
- **Test Types**: Unit + Integration + Regression
|
|
38
|
+
|
|
39
|
+
### Options
|
|
40
|
+
|
|
41
|
+
| Option | Description | Example |
|
|
42
|
+
|--------|-------------|---------|
|
|
43
|
+
| `--no-test` | Skip test enforcement | `/dev:fix-hard "race condition" --no-test` |
|
|
44
|
+
| `--test-level <level>` | Override enforcement level | `/dev:fix-hard "crash" --test-level strict` |
|
|
45
|
+
|
|
46
|
+
### Enforcement Levels
|
|
47
|
+
|
|
48
|
+
| Level | Test Failure | Coverage Below Min | Missing Tests |
|
|
49
|
+
|-------|--------------|-------------------|---------------|
|
|
50
|
+
| `soft` | Warning | Warning | Warning |
|
|
51
|
+
| `standard` | Block | Block | Warning |
|
|
52
|
+
| `strict` | Block | Block | Block |
|
|
53
|
+
|
|
17
54
|
## Oracle Analysis
|
|
18
|
-
|
|
19
|
-
|
|
20
|
-
|
|
21
|
-
-
|
|
22
|
-
-
|
|
23
|
-
-
|
|
24
|
-
-
|
|
25
|
-
-
|
|
55
|
+
|
|
56
|
+
Apply all 7 thinking modes:
|
|
57
|
+
|
|
58
|
+
- **Telescopic** - Bigger context
|
|
59
|
+
- **Microscopic** - First principles
|
|
60
|
+
- **Lateral** - Other approaches
|
|
61
|
+
- **Inversion** - What causes failure
|
|
62
|
+
- **Temporal** - When did it start
|
|
63
|
+
- **Systemic** - System effects
|
|
64
|
+
- **Quantum** - All possibilities
|
|
65
|
+
|
|
66
|
+
## Configuration
|
|
67
|
+
|
|
68
|
+
Configure via `.omgkit/workflow.yaml`:
|
|
69
|
+
|
|
70
|
+
```yaml
|
|
71
|
+
testing:
|
|
72
|
+
enabled: true
|
|
73
|
+
enforcement:
|
|
74
|
+
level: standard
|
|
75
|
+
coverage_gates:
|
|
76
|
+
unit:
|
|
77
|
+
minimum: 80
|
|
78
|
+
```
|
|
79
|
+
|
|
80
|
+
Or via CLI:
|
|
81
|
+
|
|
82
|
+
```bash
|
|
83
|
+
omgkit config set testing.enforcement.level strict
|
|
84
|
+
```
|
|
85
|
+
|
|
86
|
+
## Examples
|
|
87
|
+
|
|
88
|
+
```bash
|
|
89
|
+
# Default: comprehensive testing for complex bug
|
|
90
|
+
/dev:fix-hard "intermittent race condition in auth"
|
|
91
|
+
|
|
92
|
+
# Strict mode for critical systems
|
|
93
|
+
/dev:fix-hard "data corruption in payment" --test-level strict
|
|
94
|
+
|
|
95
|
+
# Skip testing (not recommended for complex bugs)
|
|
96
|
+
/dev:fix-hard "UI glitch" --no-test
|
|
97
|
+
```
|
|
@@ -1,22 +1,89 @@
|
|
|
1
1
|
---
|
|
2
|
-
description: Debug and fix bugs
|
|
2
|
+
description: Debug and fix bugs with regression test enforcement
|
|
3
3
|
allowed-tools: Task, Read, Write, Bash, Grep, Glob
|
|
4
|
-
argument-hint: <error or issue>
|
|
4
|
+
argument-hint: <error or issue> [--no-test] [--test-level LEVEL]
|
|
5
|
+
related_skills:
|
|
6
|
+
- methodology/systematic-debugging
|
|
7
|
+
- methodology/test-enforcement
|
|
8
|
+
related_commands:
|
|
9
|
+
- /dev:fix-fast
|
|
10
|
+
- /dev:fix-hard
|
|
11
|
+
- /quality:verify-done
|
|
12
|
+
testing:
|
|
13
|
+
default: true
|
|
14
|
+
configurable: true
|
|
5
15
|
---
|
|
6
16
|
|
|
7
|
-
#
|
|
17
|
+
# Fix: $ARGUMENTS
|
|
8
18
|
|
|
9
19
|
Fix issue: **$ARGUMENTS**
|
|
10
20
|
|
|
11
21
|
## Workflow
|
|
22
|
+
|
|
12
23
|
1. **Investigate** (debugger) - Find root cause
|
|
13
24
|
2. **Fix** (fullstack-developer) - Implement fix
|
|
14
|
-
3. **Test** (tester) - Verify fix
|
|
25
|
+
3. **Test** (tester) - Verify fix + add regression test
|
|
15
26
|
4. **Commit** (git-manager) - Commit fix
|
|
16
27
|
|
|
28
|
+
## Testing Options
|
|
29
|
+
|
|
30
|
+
This command respects project testing configuration from `.omgkit/workflow.yaml`.
|
|
31
|
+
|
|
32
|
+
### Default Behavior
|
|
33
|
+
|
|
34
|
+
- **Testing**: Enabled by default (regression tests)
|
|
35
|
+
- **Enforcement Level**: Read from `testing.enforcement.level` (default: standard)
|
|
36
|
+
- **Regression Test**: Always recommended for bug fixes
|
|
37
|
+
|
|
38
|
+
### Options
|
|
39
|
+
|
|
40
|
+
| Option | Description | Example |
|
|
41
|
+
|--------|-------------|---------|
|
|
42
|
+
| `--no-test` | Skip test enforcement | `/dev:fix "login error" --no-test` |
|
|
43
|
+
| `--test-level <level>` | Override enforcement level | `/dev:fix "crash" --test-level strict` |
|
|
44
|
+
|
|
45
|
+
### Enforcement Levels
|
|
46
|
+
|
|
47
|
+
| Level | Test Failure | Missing Regression Test |
|
|
48
|
+
|-------|--------------|------------------------|
|
|
49
|
+
| `soft` | Warning | Warning |
|
|
50
|
+
| `standard` | Block | Warning |
|
|
51
|
+
| `strict` | Block | Block |
|
|
52
|
+
|
|
17
53
|
## Debug Process
|
|
54
|
+
|
|
18
55
|
1. Reproduce the issue
|
|
19
56
|
2. Form hypotheses
|
|
20
57
|
3. Test each hypothesis
|
|
21
58
|
4. Implement minimal fix
|
|
22
|
-
5. Add regression test
|
|
59
|
+
5. Add regression test (prevents recurrence)
|
|
60
|
+
|
|
61
|
+
## Configuration
|
|
62
|
+
|
|
63
|
+
Configure via `.omgkit/workflow.yaml`:
|
|
64
|
+
|
|
65
|
+
```yaml
|
|
66
|
+
testing:
|
|
67
|
+
enabled: true
|
|
68
|
+
enforcement:
|
|
69
|
+
level: standard
|
|
70
|
+
```
|
|
71
|
+
|
|
72
|
+
Or via CLI:
|
|
73
|
+
|
|
74
|
+
```bash
|
|
75
|
+
omgkit config set testing.enforcement.level strict
|
|
76
|
+
```
|
|
77
|
+
|
|
78
|
+
## Examples
|
|
79
|
+
|
|
80
|
+
```bash
|
|
81
|
+
# Default: with regression test enforcement
|
|
82
|
+
/dev:fix "login returns 500 error"
|
|
83
|
+
|
|
84
|
+
# Strict mode: requires regression test
|
|
85
|
+
/dev:fix "data corruption bug" --test-level strict
|
|
86
|
+
|
|
87
|
+
# Skip testing for trivial fixes
|
|
88
|
+
/dev:fix "typo in error message" --no-test
|
|
89
|
+
```
|