feat(server): infer commit and PR title style from repo history#1112
feat(server): infer commit and PR title style from repo history#1112binbandit wants to merge 3 commits intopingdotgg:mainfrom
Conversation
- add recent commit subject and PR title discovery for repo style guidance - default generated commit and PR titles to conventional commits when history is unavailable - cover the new prompt guidance and git/github lookups with tests
|
Important Review skippedAuto reviews are disabled on this repository. Please check the settings in the CodeRabbit UI or the ⚙️ Run configurationConfiguration used: Repository UI Review profile: CHILL Plan: Pro Run ID: You can disable this status message by setting the Use the checkbox below for a quick retry:
✨ Finishing Touches🧪 Generate unit tests (beta)
📝 Coding Plan
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment Tip CodeRabbit can use OpenGrep to find security vulnerabilities and bugs across 17+ programming languages.OpenGrep is compatible with Semgrep configurations. Add an |
UtkarshUsername
left a comment
There was a problem hiding this comment.
I think it should be fork-aware, and check the titles of PRs made to the original repo
| ), | ||
| Effect.map((pullRequests) => pullRequests.map(normalizePullRequestSummary)), | ||
| ), | ||
| listRecentPullRequestTitles: (input) => |
There was a problem hiding this comment.
listRecentPullRequestTitles() uses gh pr list against the current checkout repo, right? So in a fork clone it gets PR titles from the fork, not the upstream repo
I agree, but there is a different PR for that. Ill do it as part of that |
Summary
Testing
Note
Infer commit and PR title style from repo history in text generation prompts
readRecentCommitSubjectstoGitCoreShapeandlistRecentPullRequestTitlestoGitHubCliShape, backed bygit logandgh pr listrespectively.generateCommitMessageandgeneratePrContentinCodexTextGenerationLivenow fetch recent commit subjects and PR titles concurrently and inject style guidance into Codex prompts.Macroscope summarized 564deeb.