[codex] ROY-66: Add review guidelines to the review/testing agent

ADHD.ai review/testing passed; squash merging this PR.

Author: 0xroylee

skills/piv-review-test/SKILL.md

@@ -13,6 +13,13 @@ You are the review/testing agent in a separate Codex session.
 2. Run `bun test` to verify the workspace is workable. If `bun test` cannot be run, return `RESULT: FAIL` and explain the blocker in `SUMMARY`.
 3. Produce final status and bug list suitable for issue creation.
 
+## Review Guidelines
+
+1. Focus findings on concrete defects, regressions, broken behavior, or missing test coverage tied to changed code.
+2. Do not fail solely for style or minor advisory suggestions; include those only when they indicate real risk.
+3. When reporting `RESULT: FAIL`, include only actionable bugs in `BUGS_JSON` with specific technical details.
+4. When reporting `RESULT: PASS`, return an empty `BUGS_JSON` array.
+
 ## Output Contract
 
 Return the final section exactly with:

tests/prompts.test.ts

@@ -59,6 +59,18 @@ describe("buildReviewPrompt", () => {
 		expect(prompt).toContain("If `bun test` cannot be run");
 		expect(prompt).toContain("RESULT: FAIL");
 	});
+
+	it("includes review guidelines from the repo review skill", async () => {
+		const prompt = await buildReviewPrompt(
+			path.resolve(process.cwd(), "skills/piv-review-test/SKILL.md"),
+			issue,
+			pr,
+		);
+
+		expect(prompt).toContain("## Review Guidelines");
+		expect(prompt).toContain("Do not fail solely for style");
+		expect(prompt).toContain("When reporting `RESULT: PASS`");
+	});
 });
 
 describe("buildPlanPrompt", () => {