Skip to content

Conversation

@seanmcguire12
Copy link
Member

why

  • selectors changed

what changed

  • updated with new selectors

@seanmcguire12 seanmcguire12 requested a review from kamath May 29, 2025 01:04
@changeset-bot
Copy link

changeset-bot bot commented May 29, 2025

⚠️ No Changeset found

Latest commit: 6753785

Merging this PR will not cause a version bump for any packages. If these changes should not result in a new version, you're good to go. If these changes should result in a version bump, you need to add a changeset.

Click here to learn what changesets are, and how to add one.

Click here if you're a maintainer who wants to add a changeset to this PR

Copy link
Contributor

@greptile-apps greptile-apps bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

PR Summary

Updates selectors in evals/tasks/observe_github.ts to handle GitHub's UI changes, particularly for locating the repository file tree element.

  • Added 10 new selectors in possibleLocators array to target both file tree container and parent elements
  • Updated selectors to handle React-based UI structure with new class names like prc-PageLayout-PageLayoutRoot-1zlEO
  • Included direct ID selector #repos-file-tree alongside more specific nested selectors
  • Added comprehensive DOM traversal paths to improve element location reliability

1 file(s) reviewed, no comment(s)
Edit PR Review Bot Settings | Greptile

@seanmcguire12 seanmcguire12 merged commit e887aa2 into main May 29, 2025
14 checks passed
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.

3 participants