reviewer-harness 1.4.0 → 1.4.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/package.json +1 -1
- package/skills/safe-editing.md +3 -1
package/package.json
CHANGED
package/skills/safe-editing.md
CHANGED
|
@@ -1,6 +1,6 @@
|
|
|
1
1
|
---
|
|
2
2
|
name: safe-editing
|
|
3
|
-
description: Surgical editing — the smallest possible diff, no collateral damage
|
|
3
|
+
description: Surgical editing — the smallest possible diff, no collateral damage, always seasoned senior-developer quality and well structured.
|
|
4
4
|
---
|
|
5
5
|
|
|
6
6
|
## Skill: safe-editing
|
|
@@ -10,5 +10,7 @@ When to use: every change to existing code.
|
|
|
10
10
|
- Read the whole file (or the relevant part) before changing anything.
|
|
11
11
|
- Make the **smallest diff** that solves the task — never include changes that were not asked for.
|
|
12
12
|
- Do not reformat, reorder, or rename unrelated lines.
|
|
13
|
+
- Code as a seasoned senior developer with many years of experience shipping real products — websites and mobile apps deployed to production: clear names, small focused functions, no duplication, no clever hacks, edge cases handled, production-ready and maintainable — every change must be easy to understand at a glance.
|
|
14
|
+
- Keep the code well structured: separate concerns into files and modules with clear organization and placement. Never leave spaghetti code behind — if the change touches messy code, refactor that part (and only that part) so the result stays clean.
|
|
13
15
|
- If another file truly must change too, say why before doing it.
|
|
14
16
|
- Afterwards, show the diff and describe in one sentence what changed.
|