If no statement of work (SoW) was provided, stop and ask, do not proceed.
MANDATORY: Your task is to report only, you do not make any chode change.
Standard Operating Procedure
- •Determine changes to analyze:
- •If a PR was provided, run
gh pr view <number>andgh pr diff <number> - •If revisions were provided, run either of:
- •Jujutsu revset:
jj diff --git -r <revset> - •Git commits:
git diff <id1..id2>
- •Jujutsu revset:
- •If a PR was provided, run
- •Determine Statement of Work.
- •This can come from:
- •a Jira ticket, use the jira-ticket skill to retrieve it.
- •a revision's description
- •If none of the above are provided, search for a ticket in the commit messages:
- •Jujutsu revset:
jj log -r <revset> -T description --no-graph - •Git commits:
git log <revision-range>
- •Jujutsu revset:
- •If you cannot determine the SoW: STOP and ASK.
- •This can come from:
- •Perform a rigorous, constraint-based comparison against the Statement of Work:
- •EXPLICITLY cross-reference the list of modified files in the diff against the "Files to modify" list in the SoW. Flag any unlisted files as out-of-scope deviations.
- •EXPLICITLY verify that the exact code requested in the "Plan" section was implemented. Do not accept functionally equivalent alternatives if a specific implementation was mandated.
- •Confirm all acceptance criteria are met.
- •Confirm expected work product provided.
- •Analyze the changes and provide a thorough code review that includes:
- •Overview of what the changes do
- •Analysis of code quality and style
- •Specific suggestions for improvements
- •Any potential issues or risks
- •The detailed SoW comparison results from step 3.
- •Write the review to file: ,reviews/review-<ticket>-<date>.md. CRITICAL: the comma in front of 'reviews' is required. MANDATORY: if a review already exists, update it, do not create a new file.
- •If a ticket was provided, add as comment:
jira update <ticket> --comment <review>MANDATORY: format the comment using the Jira markup language.