History

fix-root-causes: flag verbose workaround comments (#162)

lauren committed 3 months ago

Commit c7a985e1 parent 352a4ea2 files · +2 −1Browse files

Unified · Split

2 files
  1. .cursor-plugin/plugin.json +1 −1
  2. skills/principle-fix-root-causes/SKILL.md +1 −0
.cursor-plugin/plugin.json+1−1
@@ -1,7 +1,7 @@
1{1{
2 "name": "pstack",2 "name": "pstack",
3 "displayName": "pstack",3 "displayName": "pstack",
4 "version": "0.11.3",4 "version": "0.11.4",
5 "description": "if you want to go fast, go deep first. pstack helps you write less, but higher quality code. rigorous agent workflows you can parallelize with confidence.",5 "description": "if you want to go fast, go deep first. pstack helps you write less, but higher quality code. rigorous agent workflows you can parallelize with confidence.",
6 "author": {6 "author": {
7 "name": "Lauren Tan"7 "name": "Lauren Tan"
skills/principle-fix-root-causes/SKILL.md+1−0
@@ -14,6 +14,7 @@ When debugging, do not paper over symptoms. Trace every problem to its root caus · 13 unmodified lines
14- Reproduce first (if you can't reproduce it, you can't verify your fix)14- Reproduce first (if you can't reproduce it, you can't verify your fix)
15- Ask "why" until you hit the root cause15- Ask "why" until you hit the root cause
16- Resist the urge to add guards (adding a nil check to silence a crash is a symptom fix)16- Resist the urge to add guards (adding a nil check to silence a crash is a symptom fix)
17- If a workaround needs a paragraph-long comment to justify it, the code is wrong (fix the code, not the comment)
17- Check for the pattern, not just the instance (grep for the same pattern, fix all instances)18- Check for the pattern, not just the instance (grep for the same pattern, fix all instances)
18- When stuck, instrument. Don't guess (add logging, read the actual error)19- When stuck, instrument. Don't guess (add logging, read the actual error)
19 20