@christianmorup/review-intent 0.1.1 → 0.1.3
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 +5 -4
- package/dist/skill.js +12 -6
- package/package.json +1 -1
package/README.md
CHANGED
|
@@ -208,7 +208,7 @@ whether the intent is genuine reasoning or post-hoc rationalization is the whole
|
|
|
208
208
|
ballgame. `review-intent` ships a Claude Code skill that teaches the agent to
|
|
209
209
|
author the artifact **honestly** (real rejected alternatives, stated
|
|
210
210
|
assumptions, incidental changes marked as incidental) when it finishes a change
|
|
211
|
-
set, then
|
|
211
|
+
set, then render it.
|
|
212
212
|
|
|
213
213
|
```sh
|
|
214
214
|
review-intent skill install # ~/.claude/skills/review-intent-authoring (all repos)
|
|
@@ -217,9 +217,10 @@ review-intent skill uninstall # remove user-scoped skill
|
|
|
217
217
|
review-intent skill uninstall --local # remove repo-scoped skill
|
|
218
218
|
```
|
|
219
219
|
|
|
220
|
-
|
|
221
|
-
|
|
222
|
-
remove a hand-edited skill file.
|
|
220
|
+
Once the artifact is written the skill tells the agent to *run* `review-intent`
|
|
221
|
+
and open the review — not to ask first, since invoking the skill already means a
|
|
222
|
+
review is wanted. Add `--force` to overwrite or remove a hand-edited skill file.
|
|
223
|
+
User and `--local` scopes are independent.
|
|
223
224
|
|
|
224
225
|
The honesty contract is the point: a fluent rationalization is worse than
|
|
225
226
|
nothing because it lowers the reviewer's guard while adding no signal. The skill
|
package/dist/skill.js
CHANGED
|
@@ -8,7 +8,7 @@ const SKILL_NAME = "review-intent-authoring";
|
|
|
8
8
|
// against it (line-ending-normalized) to decide whether the file is still ours.
|
|
9
9
|
export const SKILL_CONTENT = `---
|
|
10
10
|
name: review-intent-authoring
|
|
11
|
-
description: Use when you have just finished a set of code changes on a branch and the user (or another reviewer) is about to review the diff. Author a .review/intent.json that captures the genuine intent behind the changes — why, what you rejected, what it rests on — keyed to files and hunks, plus mermaid class and sequence diagrams. Then
|
|
11
|
+
description: Use when you have just finished a set of code changes on a branch and the user (or another reviewer) is about to review the diff. Author a .review/intent.json that captures the genuine intent behind the changes — why, what you rejected, what it rests on — keyed to files and hunks, plus mermaid class and sequence diagrams. Then render it with review-intent (run it, don't ask) so the reviewer adjudicates decisions instead of skimming lines.
|
|
12
12
|
---
|
|
13
13
|
|
|
14
14
|
# Authoring an honest intent artifact
|
|
@@ -182,13 +182,19 @@ don't draw a trivial two-box diagram to fill the slot.
|
|
|
182
182
|
|
|
183
183
|
## After writing it
|
|
184
184
|
|
|
185
|
-
|
|
185
|
+
Render it — don't ask first. You're inside this skill because a review is
|
|
186
|
+
wanted; writing the artifact and *then* asking "should I open it?" just adds a
|
|
187
|
+
round-trip the user did not want. Run \`review-intent\` via Bash from the repo
|
|
188
|
+
root — it diffs the current branch against main, writes \`review.html\`, and opens
|
|
189
|
+
it in the browser. Then tell the user you've opened it (and where the file is);
|
|
190
|
+
don't ask permission to.
|
|
186
191
|
|
|
187
|
-
|
|
188
|
-
|
|
192
|
+
The only times you don't run it: the change was trivial/mechanical so you
|
|
193
|
+
skipped the artifact entirely (say so in chat), or the user has explicitly
|
|
194
|
+
declined review-intent for this change set.
|
|
189
195
|
|
|
190
|
-
If the
|
|
191
|
-
|
|
196
|
+
If the completeness gate refuses to render because intent is missing, fix the
|
|
197
|
+
gaps and run again — do **not** reach for \`--allow-gaps\` to get past it.
|
|
192
198
|
|
|
193
199
|
## Why this exists
|
|
194
200
|
|
package/package.json
CHANGED
|
@@ -1,6 +1,6 @@
|
|
|
1
1
|
{
|
|
2
2
|
"name": "@christianmorup/review-intent",
|
|
3
|
-
"version": "0.1.
|
|
3
|
+
"version": "0.1.3",
|
|
4
4
|
"description": "Render the diff between the current branch and main as an intent-annotated HTML review page with mermaid class & sequence diagrams.",
|
|
5
5
|
"type": "module",
|
|
6
6
|
"license": "MIT",
|