Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add PR URL and repository name to GithubViewPRTool response #900

Draft
wants to merge 2 commits into
base: develop
Choose a base branch
from

Conversation

codegen-sh[bot]
Copy link
Contributor

@codegen-sh codegen-sh bot commented Mar 18, 2025

This PR addresses the issue where the GithubViewPRTool doesn't include the PR URL and repository name in its response.

Changes

  • Added pr_url and repo_name fields to the ViewPRObservation class
  • Updated the view_pr function to extract and include these values in the observation
  • Updated the string template to include the new fields

This will help users quickly identify which repository a PR belongs to and provide a direct link to the PR, improving the user experience when working with PRs across multiple repositories.

Fixes the issue where the agent was looking at the wrong repository when analyzing PRs.

@CLAassistant
Copy link

CLA assistant check
Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.


codegen-bot seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.
You have signed the CLA already but the status is still pending? Let us recheck it.

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.

1 participant