@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 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 offer to render it.
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
- The skill never auto-launches anything it teaches the agent to write the
221
- artifact and then *ask* before opening the review. Add `--force` to overwrite or
222
- remove a hand-edited skill file. User and `--local` scopes are independent.
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 offer to render it with review-intent so the reviewer adjudicates decisions instead of skimming lines.
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
- Offer to render never auto-launch:
185
+ Render itdon'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
- > I've written the review intent to \`.review/intent.json\`. Want me to open the
188
- > side-by-side review? (\`review-intent\`)
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 user accepts, run \`review-intent\` via Bash from the repo root. It diffs
191
- the current branch against main and opens the rendered page in the browser.
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.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",