Skip to content

✨ feat(revu): add implement skill for applying review comments#1

Merged
kud merged 1 commit into
mainfrom
feat/revu-implement-skill
Jun 23, 2026
Merged

✨ feat(revu): add implement skill for applying review comments#1
kud merged 1 commit into
mainfrom
feat/revu-implement-skill

Conversation

@kud

@kud kud commented Jun 23, 2026

Copy link
Copy Markdown
Owner

Description

Adds the implement skill to the revu plugin β€” the execution-phase counterpart to the existing discuss skill. This completes the revu review loop:

  • discuss β†’ talks a review through interactively
  • implement β†’ reads the export, presents a checklist, confirms, then applies every comment exactly as written

The skill reads from .revu.json (preferred, structured) or revu-review.md (fallback annotated diff), groups comments by file into a numbered TODO checklist, gates behind an explicit yes/no confirmation, applies changes file-by-file without scope creep, then offers to delete the export and optionally run lint/tests.

It is self-contained and has no dependency on private agents β€” portable to any Claude Code setup that has the revu plugin.

This skill was previously living in the personal my repo as k-notes-implement, co-located with unrelated tooling. Moving it here puts it next to the review tooling it belongs to and makes it available to anyone using the plugin.

Screencast

No visual changes.

How to Validate

  1. Check out feat/revu-implement-skill and install the revu plugin.
  2. In a project directory, drop a .revu.json with a comments array (file, startLine, endLine, text fields).
  3. Run the implement skill β€” verify it displays the numbered checklist grouped by file and does not touch any code yet.
  4. Confirm with "yes" β€” verify it applies exactly the changes described, no more.
  5. Verify it offers to delete the export only after all comments are processed.
  6. Repeat with a revu-review.md to confirm the fallback path works.
  7. Verify that if neither file is present, the skill stops with a clear message.

Developer Checklist

  • Code is readable and maintainable
  • PR is atomic and focused on a single feature
  • Commits follow Conventional Commits
  • Tests included and passing (if applicable)

Add the `implement` skill to the revu plugin. It reads review data from
.revu.json or revu-review.md, presents them as a checklist, gets explicit
confirmation, then applies each comment exactly as written. This is the
execution-focused counterpart to the `discuss` skill.
@kud
kud marked this pull request as ready for review June 23, 2026 11:39
@kud
kud merged commit 31885ff into main Jun 23, 2026
1 check passed
@kud
kud deleted the feat/revu-implement-skill branch June 23, 2026 11:39
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant