aigroup-workflow 2.2.1 → 2.2.2

package/skills/backend/debugging-wizard/SKILL.md

@@ -1,105 +1,105 @@
----
-name: debugging-wizard
-description: Parses error messages, traces execution flow through stack traces, correlates log entries to identify failure points, and applies systematic hypothesis-driven methodology to isolate and resolve bugs. Use when investigating errors, analyzing stack traces, finding root causes of unexpected behavior, troubleshooting crashes, or performing log analysis, error investigation, or root cause analysis.
-license: MIT
-metadata:
-  author: https://github.com/Jeffallan
-  version: "1.1.0"
-  domain: quality
-  triggers: debug, error, bug, exception, traceback, stack trace, troubleshoot, not working, crash, fix issue
-  role: specialist
-  scope: analysis
-  output-format: analysis
-  related-skills: test-master, fullstack-guardian, monitoring-expert
----
-
-# Debugging Wizard
-
-Expert debugger applying systematic methodology to isolate and resolve issues in any codebase.
-
-## Core Workflow
-
-1. **Reproduce** - Establish consistent reproduction steps
-2. **Isolate** - Narrow down to smallest failing case
-3. **Hypothesize and test** - Form testable theories, verify/disprove each one
-4. **Fix** - Implement and verify solution
-5. **Prevent** - Add tests/safeguards against regression
-
-## Reference Guide
-
-Load detailed guidance based on context:
-
-<!-- Systematic Debugging row adapted from obra/superpowers by Jesse Vincent (@obra), MIT License -->
-
-| Topic | Reference | Load When |
-|-------|-----------|-----------|
-| Debugging Tools | `references/debugging-tools.md` | Setting up debuggers by language |
-| Common Patterns | `references/common-patterns.md` | Recognizing bug patterns |
-| Strategies | `references/strategies.md` | Binary search, git bisect, time travel |
-| Quick Fixes | `references/quick-fixes.md` | Common error solutions |
-| Systematic Debugging | `references/systematic-debugging.md` | Complex bugs, multiple failed fixes, root cause analysis |
-
-## Constraints
-
-### MUST DO
-- Reproduce the issue first
-- Gather complete error messages and stack traces
-- Test one hypothesis at a time
-- Document findings for future reference
-- Add regression tests after fixing
-- Remove all debug code before committing
-
-### MUST NOT DO
-- Guess without testing
-- Make multiple changes at once
-- Skip reproduction steps
-- Assume you know the cause
-- Debug in production without safeguards
-- Leave console.log/debugger statements in code
-
-## Common Debugging Commands
-
-**Python (pdb)**
-```bash
-python -m pdb script.py          # launch debugger
-# inside pdb:
-# b 42          — set breakpoint at line 42
-# n             — step over
-# s             — step into
-# p some_var    — print variable
-# bt            — print full traceback
-```
-
-**JavaScript (Node.js)**
-```bash
-node --inspect-brk script.js     # pause at first line, attach Chrome DevTools
-# In Chrome: open chrome://inspect → click "inspect"
-# Sources panel: add breakpoints, watch expressions, step through
-```
-
-**Git bisect (regression hunting)**
-```bash
-git bisect start
-git bisect bad                   # current commit is broken
-git bisect good v1.2.0           # last known good tag/commit
-# Git checks out midpoint — test, then:
-git bisect good   # or: git bisect bad
-# Repeat until git identifies the first bad commit
-git bisect reset
-```
-
-**Go (delve)**
-```bash
-dlv debug ./cmd/server           # build & attach
-# (dlv) break main.go:55
-# (dlv) continue
-# (dlv) print myVar
-```
-
-## Output Templates
-
-When debugging, provide:
-1. **Root Cause**: What specifically caused the issue
-2. **Evidence**: Stack trace, logs, or test that proves it
-3. **Fix**: Code change that resolves it
-4. **Prevention**: Test or safeguard to prevent recurrence
+---
+name: debugging-wizard
+description: Parses error messages, traces execution flow through stack traces, correlates log entries to identify failure points, and applies systematic hypothesis-driven methodology to isolate and resolve bugs. Use when investigating errors, analyzing stack traces, finding root causes of unexpected behavior, troubleshooting crashes, or performing log analysis, error investigation, or root cause analysis.
+license: MIT
+metadata:
+  author: https://github.com/Jeffallan
+  version: "1.1.0"
+  domain: quality
+  triggers: debug, error, bug, exception, traceback, stack trace, troubleshoot, not working, crash, fix issue
+  role: specialist
+  scope: analysis
+  output-format: analysis
+  related-skills: test-master, fullstack-guardian, monitoring-expert
+---
+
+# Debugging Wizard
+
+Expert debugger applying systematic methodology to isolate and resolve issues in any codebase.
+
+## Core Workflow
+
+1. **Reproduce** - Establish consistent reproduction steps
+2. **Isolate** - Narrow down to smallest failing case
+3. **Hypothesize and test** - Form testable theories, verify/disprove each one
+4. **Fix** - Implement and verify solution
+5. **Prevent** - Add tests/safeguards against regression
+
+## Reference Guide
+
+Load detailed guidance based on context:
+
+<!-- Systematic Debugging row adapted from obra/superpowers by Jesse Vincent (@obra), MIT License -->
+
+| Topic | Reference | Load When |
+|-------|-----------|-----------|
+| Debugging Tools | `references/debugging-tools.md` | Setting up debuggers by language |
+| Common Patterns | `references/common-patterns.md` | Recognizing bug patterns |
+| Strategies | `references/strategies.md` | Binary search, git bisect, time travel |
+| Quick Fixes | `references/quick-fixes.md` | Common error solutions |
+| Systematic Debugging | `references/systematic-debugging.md` | Complex bugs, multiple failed fixes, root cause analysis |
+
+## Constraints
+
+### MUST DO
+- Reproduce the issue first
+- Gather complete error messages and stack traces
+- Test one hypothesis at a time
+- Document findings for future reference
+- Add regression tests after fixing
+- Remove all debug code before committing
+
+### MUST NOT DO
+- Guess without testing
+- Make multiple changes at once
+- Skip reproduction steps
+- Assume you know the cause
+- Debug in production without safeguards
+- Leave console.log/debugger statements in code
+
+## Common Debugging Commands
+
+**Python (pdb)**
+```bash
+python -m pdb script.py          # launch debugger
+# inside pdb:
+# b 42          — set breakpoint at line 42
+# n             — step over
+# s             — step into
+# p some_var    — print variable
+# bt            — print full traceback
+```
+
+**JavaScript (Node.js)**
+```bash
+node --inspect-brk script.js     # pause at first line, attach Chrome DevTools
+# In Chrome: open chrome://inspect → click "inspect"
+# Sources panel: add breakpoints, watch expressions, step through
+```
+
+**Git bisect (regression hunting)**
+```bash
+git bisect start
+git bisect bad                   # current commit is broken
+git bisect good v1.2.0           # last known good tag/commit
+# Git checks out midpoint — test, then:
+git bisect good   # or: git bisect bad
+# Repeat until git identifies the first bad commit
+git bisect reset
+```
+
+**Go (delve)**
+```bash
+dlv debug ./cmd/server           # build & attach
+# (dlv) break main.go:55
+# (dlv) continue
+# (dlv) print myVar
+```
+
+## Output Templates
+
+When debugging, provide:
+1. **Root Cause**: What specifically caused the issue
+2. **Evidence**: Stack trace, logs, or test that proves it
+3. **Fix**: Code change that resolves it
+4. **Prevention**: Test or safeguard to prevent recurrence

package/skills/backend/debugging-wizard/references/common-patterns.md

@@ -1,132 +1,132 @@
-# Common Bug Patterns
-
-## Pattern Recognition
-
-| Pattern | Symptom | Likely Cause |
-|---------|---------|--------------|
-| Race condition | Intermittent failures | Missing await, async timing |
-| Off-by-one | Missing first/last item | `<` vs `<=`, array bounds |
-| Null reference | "undefined is not..." | Missing null check |
-| Memory leak | Growing memory | Uncleaned listeners/intervals |
-| N+1 queries | Slow with more data | Fetching in loop |
-| Type coercion | Unexpected behavior | `==` instead of `===` |
-| Closure issue | Wrong variable value | Loop variable capture |
-| Stale state | Old value used | React state closure |
-
-## Race Condition
-
-```typescript
-// BUG: Race condition
-let data;
-fetchData().then(result => { data = result; });
-console.log(data); // undefined!
-
-// FIX: Await the result
-const data = await fetchData();
-console.log(data);
-```
-
-## Off-by-One
-
-```typescript
-// BUG: Skips last element
-for (let i = 0; i < array.length - 1; i++) { }
-
-// FIX: Include last element
-for (let i = 0; i < array.length; i++) { }
-
-// BUG: Array index out of bounds
-const last = array[array.length]; // undefined
-
-// FIX: Correct index
-const last = array[array.length - 1];
-```
-
-## Null Reference
-
-```typescript
-// BUG: Crashes if user is null
-const name = user.profile.name;
-
-// FIX: Optional chaining
-const name = user?.profile?.name ?? 'Unknown';
-
-// FIX: Guard clause
-if (!user?.profile) {
-  return 'Unknown';
-}
-return user.profile.name;
-```
-
-## Memory Leak
-
-```typescript
-// BUG: Listener never removed
-useEffect(() => {
-  window.addEventListener('resize', handleResize);
-}, []);
-
-// FIX: Cleanup function
-useEffect(() => {
-  window.addEventListener('resize', handleResize);
-  return () => window.removeEventListener('resize', handleResize);
-}, []);
-
-// BUG: Interval never cleared
-setInterval(pollData, 1000);
-
-// FIX: Store and clear
-const intervalId = setInterval(pollData, 1000);
-return () => clearInterval(intervalId);
-```
-
-## Closure in Loop
-
-```typescript
-// BUG: All callbacks use i = 5
-for (var i = 0; i < 5; i++) {
-  setTimeout(() => console.log(i), 100);
-}
-
-// FIX: Use let (block scoped)
-for (let i = 0; i < 5; i++) {
-  setTimeout(() => console.log(i), 100);
-}
-
-// FIX: Capture in closure
-for (var i = 0; i < 5; i++) {
-  ((j) => setTimeout(() => console.log(j), 100))(i);
-}
-```
-
-## React Stale State
-
-```typescript
-// BUG: count is stale in closure
-const [count, setCount] = useState(0);
-useEffect(() => {
-  setInterval(() => {
-    setCount(count + 1); // Always uses initial count
-  }, 1000);
-}, []);
-
-// FIX: Use functional update
-setCount(prev => prev + 1);
-
-// FIX: Include in dependency array with cleanup
-useEffect(() => {
-  const id = setInterval(() => setCount(c => c + 1), 1000);
-  return () => clearInterval(id);
-}, []);
-```
-
-## Quick Reference
-
-| Symptom | First Check |
-|---------|-------------|
-| "undefined is not..." | Null check missing |
-| Works sometimes | Race condition |
-| Wrong value in callback | Closure/stale state |
-| Gets slower over time | Memory leak, N+1 |
-| Off by one item | Loop bounds, array index |
-| Type mismatch | `==` vs `===`, coercion |
+# Common Bug Patterns
+
+## Pattern Recognition
+
+| Pattern | Symptom | Likely Cause |
+|---------|---------|--------------|
+| Race condition | Intermittent failures | Missing await, async timing |
+| Off-by-one | Missing first/last item | `<` vs `<=`, array bounds |
+| Null reference | "undefined is not..." | Missing null check |
+| Memory leak | Growing memory | Uncleaned listeners/intervals |
+| N+1 queries | Slow with more data | Fetching in loop |
+| Type coercion | Unexpected behavior | `==` instead of `===` |
+| Closure issue | Wrong variable value | Loop variable capture |
+| Stale state | Old value used | React state closure |
+
+## Race Condition
+
+```typescript
+// BUG: Race condition
+let data;
+fetchData().then(result => { data = result; });
+console.log(data); // undefined!
+
+// FIX: Await the result
+const data = await fetchData();
+console.log(data);
+```
+
+## Off-by-One
+
+```typescript
+// BUG: Skips last element
+for (let i = 0; i < array.length - 1; i++) { }
+
+// FIX: Include last element
+for (let i = 0; i < array.length; i++) { }
+
+// BUG: Array index out of bounds
+const last = array[array.length]; // undefined
+
+// FIX: Correct index
+const last = array[array.length - 1];
+```
+
+## Null Reference
+
+```typescript
+// BUG: Crashes if user is null
+const name = user.profile.name;
+
+// FIX: Optional chaining
+const name = user?.profile?.name ?? 'Unknown';
+
+// FIX: Guard clause
+if (!user?.profile) {
+  return 'Unknown';
+}
+return user.profile.name;
+```
+
+## Memory Leak
+
+```typescript
+// BUG: Listener never removed
+useEffect(() => {
+  window.addEventListener('resize', handleResize);
+}, []);
+
+// FIX: Cleanup function
+useEffect(() => {
+  window.addEventListener('resize', handleResize);
+  return () => window.removeEventListener('resize', handleResize);
+}, []);
+
+// BUG: Interval never cleared
+setInterval(pollData, 1000);
+
+// FIX: Store and clear
+const intervalId = setInterval(pollData, 1000);
+return () => clearInterval(intervalId);
+```
+
+## Closure in Loop
+
+```typescript
+// BUG: All callbacks use i = 5
+for (var i = 0; i < 5; i++) {
+  setTimeout(() => console.log(i), 100);
+}
+
+// FIX: Use let (block scoped)
+for (let i = 0; i < 5; i++) {
+  setTimeout(() => console.log(i), 100);
+}
+
+// FIX: Capture in closure
+for (var i = 0; i < 5; i++) {
+  ((j) => setTimeout(() => console.log(j), 100))(i);
+}
+```
+
+## React Stale State
+
+```typescript
+// BUG: count is stale in closure
+const [count, setCount] = useState(0);
+useEffect(() => {
+  setInterval(() => {
+    setCount(count + 1); // Always uses initial count
+  }, 1000);
+}, []);
+
+// FIX: Use functional update
+setCount(prev => prev + 1);
+
+// FIX: Include in dependency array with cleanup
+useEffect(() => {
+  const id = setInterval(() => setCount(c => c + 1), 1000);
+  return () => clearInterval(id);
+}, []);
+```
+
+## Quick Reference
+
+| Symptom | First Check |
+|---------|-------------|
+| "undefined is not..." | Null check missing |
+| Works sometimes | Race condition |
+| Wrong value in callback | Closure/stale state |
+| Gets slower over time | Memory leak, N+1 |
+| Off by one item | Loop bounds, array index |
+| Type mismatch | `==` vs `===`, coercion |