Skip to content

feat(gastown): add REVIEW.md support to refinery code review prompts#3608

Open
IamCoder18 wants to merge 1 commit into
Kilo-Org:mainfrom
IamCoder18:feat/refinery-review-md-support
Open

feat(gastown): add REVIEW.md support to refinery code review prompts#3608
IamCoder18 wants to merge 1 commit into
Kilo-Org:mainfrom
IamCoder18:feat/refinery-review-md-support

Conversation

@IamCoder18
Copy link
Copy Markdown

@IamCoder18 IamCoder18 commented May 30, 2026

Summary

Added REVIEW.md support to Gas Town Refinery agents, allowing them to read and follow repository-level review guidance from the workspace root. This mirrors the web implementation (PR #3183) but leverages the fact that Refineries already have the repository cloned and checked out on the feature branch, eliminating the need for API fetching.

Closes #3609

Verification

I have not manually verified.

Visual Changes

N/A

Reviewer Notes

N/A

Comment thread services/gastown/src/prompts/refinery-system.prompt.ts Outdated
Comment thread services/gastown/src/prompts/refinery-system.prompt.ts Outdated
Comment thread services/gastown/src/prompts/refinery-system.prompt.ts Outdated
@kilo-code-bot
Copy link
Copy Markdown
Contributor

kilo-code-bot Bot commented May 30, 2026

Code Review Summary

Status: No Issues Found | Recommendation: Merge

Executive Summary

All three previous WARNING issues are resolved: the REVIEW.md conditional logic is now clear and unambiguous in both buildBranchReviewPrompt and buildPRReviewPrompt, with explicit if-present/if-absent branching replacing the former sequential numbered steps.

Previously Flagged Issues — Now Resolved
File Previous Issue Status
services/gastown/src/prompts/refinery-system.prompt.ts Default review bullets visually grouped under REVIEW.md step, risking misapplication when REVIEW.md is present Resolved — now guarded by explicit If REVIEW.md is absent, use these default review rules: condition
services/gastown/src/prompts/refinery-system.prompt.ts Sequential numbering in Step 2 could cause agent to execute all sub-steps unconditionally Resolved — numbered list replaced with if/if-absent bullet structure
services/gastown/src/prompts/refinery-system.prompt.ts Same ambiguity in buildPRReviewPrompt Resolved — symmetric fix applied
Files Reviewed (1 file)
  • services/gastown/src/prompts/refinery-system.prompt.ts

Reviewed by claude-sonnet-4.6 · 349,361 tokens

Review guidance: REVIEW.md from base branch main

@IamCoder18 IamCoder18 force-pushed the feat/refinery-review-md-support branch from 9c37387 to 41b999a Compare May 30, 2026 22:26
@IamCoder18 IamCoder18 force-pushed the feat/refinery-review-md-support branch from 41b999a to 401b1da Compare May 30, 2026 22:34
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.

Refinery agents do not follow REVIEW.md from reviewed branches

1 participant