Skip to content

review skills: a comment convention for nominating guidance to the standard #82

Closed

Description

@mmaclay

Skill: pr-review, pr-fix, revise (scripts/record.py) · Plugin commit: 4495e1441dbd

Context.

  • Reviewers already mark comments informally as meant for agents to learn from. In the reference run, several comments said so in their own words.
  • Nothing in the harvest recognises it.
  • A related mechanism exists. A PR body's Already checked block may carry a Nominated: line (the corrections a person made at the plan and diff gates). record.py reads it into inner.nominated (scripts/record.py:24, :132), and prepare.md step 5 adds it to clusters. It covers only the author's own build session, not reviewers.

Change.

  • A documented convention any reviewer can type in a review comment or reply, for example a line starting Standard: or For agents:. record.py records it as a nominated item with its comment id and author.
  • The next revision must list every nominated item: as a proposal if it meets revise: a single general statement from a person never reaches the report #78's bar, held with a reason otherwise. It is never silently dropped.
  • pr-review and pr-fix preserve such lines when they draft or quote replies, and never write one themselves.
  • The README and quickstart say how to use it, in one paragraph.

Done when. A fixture pull request with a For agents: reply produces a nominated item in its record, and the revision lists it.

Related: #78, #87. Tracked in #89.

Activity

  1. mmaclay commented on Nov 7, 2026

    @mmaclay
    MemberAuthor

    Landed in #108. A reviewer's ask to keep a point, in any wording and with no prefix, is read as asks for guidance and named in the revision's report. #109 carries it across covered repositories, and #117 also reads it in a reply posted through pr-fix when the poster read the reply.

    🤖 AI-assisted comment, posted by @mmaclay. AI assistance level: 5/5 (not read by a person).

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Metadata

Assignees

No one assigned

    Labels

    skill-feedbackUser-submitted feedback about a skill

    Type

    No type

    Projects

    No projects

      Milestone

      No milestone

      Relationships

      None yet

      Development

      No branches or pull requests

      Issue actions