feat: AI PR review menu #174

Anush008 · 2023-06-11T10:33:19Z

What type of PR is this? (check all applicable)

Description

This PR includes several updates. Firstly, a new menu has been added to test, refactor and explain code. Secondly, the PR description config page has been refactored for better readability. Thirdly, the AICodeReview has been restructured for improved organisation. Lastly, a fix has been made to the discussions URL.

Generated using OpenSauced.

The feature is added with extensibility in mind. Adding a new feature item to the menu is simple and intuitive.

ai/src/content-scripts/components/AICodeReview/AICodeReviewButton.ts

Lines 36 to 43 in a9af46f

    
           const explainCode = AICodeReviewMenuItem( 
        
               "Explain Code", 
        
               "Generate an explanation for the code", 
        
               generateCodeExplanation, 
        
               commentNode, 
        
           ); 
        
           const menu = AICodeReviewMenu([refactorCode, testCode, explainCode]);

The button injection and selected code retrieval is preserved, reused from the previous version with no changes.

Additionally, a patch has been applied to the githubPullRequestPattern regex to resolve #172, #175. 0a19fbc, 28ea7f0.

Related Tickets & Documents

Resolves #135.
Resolves #136.
Resolves #172.
Resolves #175.

Mobile & Desktop Screenshots/Recordings

Screen.Recording.2023-06-12.at.7.07.40.PM.mp4

Added tests?

👍 yes
🙅 no, because they aren't needed
🙋 no, because I need help

Added to documentation?

bdougie · 2023-06-11T19:47:40Z

I like this approach a lot and it is a great addition to the user flow.

Anush008 · 2023-06-13T12:20:39Z

The API endpoints are now available for testing at https://beta.api.opensauced.pizza/#/Pull%20Requests%20service/generateCodeExplanation https://beta.api.opensauced.pizza/#/Pull%20Requests%20service/generateCodeTest.

src/content-scripts/components/AICodeReview/AICodeReviewButton.ts

diivi · 2023-06-13T12:57:00Z

Thanks for this important feature! I'll test it out later today.

Co-authored-by: Divyansh Singh <[email protected]>

bdougie · 2023-06-13T21:30:16Z

Testing out the feature, but a little lost on how much of this should be working. I see this PR adds all options, but I get nothing when trying to generate tests or explanations. Refactoring works.

Perhaps this still needs a production deploy from @brandonroberts? I was expecting to see the "generic message" similar to your video @Anush008.

Testing out the Preview Menu 🧐 - Watch Video

brandonroberts · 2023-06-13T21:42:45Z

Yes, the plan is to cut an API release today

Anush008 · 2023-06-14T01:26:55Z

Perhaps this still needs a production deploy from @brandonroberts? I was expecting to see the "generic message" similar to your video @Anush008.

I believe you used the production build which routed the API requests to api.opensauced.pizza.

The endpoints are available on beta so you can test out using npm run dev, the requests will be made to beta.api.opensauced.pizza.

brandonroberts · 2023-06-14T05:00:33Z

The API is deployed to production also with the new endpoints

bdougie

This works as expected now that the API is deployed. We will need the documentation written before cutting the release.

Anush008 · 2023-06-14T15:15:18Z

This works as expected now that the API is deployed. We will need the documentation written before cutting the release.

I am working on updating the usage docs at docs.opensauced.pizza.. My PC network connection seems to have a problem.

bdougie · 2023-06-15T07:35:06Z

Merging to have this work during a demo today. Thanks for the docs PR @Anush008

## [1.7.0-beta.2](v1.7.0-beta.1...v1.7.0-beta.2) (2023-06-15) ### 🍕 Features * AI PR review menu ([#174](#174)) ([ef52439](ef52439))

github-actions · 2023-06-15T07:36:59Z

🎉 This PR is included in version 1.7.0-beta.2 🎉

The release is available on GitHub release

Your semantic-release bot 📦🚀

## [1.7.0](v1.6.0...v1.7.0) (2023-06-15) ### 🐛 Bug Fixes * update email invite message ([c1008e5](c1008e5)) ### 🍕 Features * Added Contributed Repos to profile page ([#181](#181)) ([4901f7b](4901f7b)) * AI PR review menu ([#174](#174)) ([ef52439](ef52439))

github-actions · 2023-06-15T08:06:00Z

🎉 This PR is included in version 1.7.0 🎉

The release is available on GitHub release

Your semantic-release bot 📦🚀

## [1.7.0-beta.2](open-sauced/ai@v1.7.0-beta.1...v1.7.0-beta.2) (2023-06-15) ### 🍕 Features * AI PR review menu ([#174](open-sauced/ai#174)) ([ef52439](open-sauced/ai@ef52439))

## [1.7.0](open-sauced/ai@v1.6.0...v1.7.0) (2023-06-15) ### 🐛 Bug Fixes * update email invite message ([c1008e5](open-sauced/ai@c1008e5)) ### 🍕 Features * Added Contributed Repos to profile page ([#181](open-sauced/ai#181)) ([4901f7b](open-sauced/ai@4901f7b)) * AI PR review menu ([#174](open-sauced/ai#174)) ([ef52439](open-sauced/ai@ef52439))

Anush008 added 5 commits June 11, 2023 13:11

feat: code-test-refactor-explanation menu

de3922a

refactor: pr description config page

2761666

chore: restructure AICodeReview

eb4d7c9

fix: discussions URL

a9af46f

fix: dependency vulnerability

d554568

fix: multiple processing of PR page

0a19fbc

Anush008 mentioned this pull request Jun 12, 2023

Bug: Duplicate Icon and multiple PR description generating. #172

Closed

2 tasks

Anush008 added 2 commits June 13, 2023 17:39

refactor: Move AI utilities to ai-utils dir

71ed35e

chore: use new endpoints

8d580ce

chore: Added optional trailing slash PR page regex

28ea7f0

Anush008 marked this pull request as ready for review June 13, 2023 12:43

Anush008 requested review from a0m0rajab, diivi and bdougie and removed request for a0m0rajab, diivi and bdougie June 13, 2023 12:43

diivi reviewed Jun 13, 2023

View reviewed changes

src/content-scripts/components/AICodeReview/AICodeReviewButton.ts Outdated Show resolved Hide resolved

Update src/content-scripts/components/AICodeReview/AICodeReviewButton.ts

0bf3d15

Co-authored-by: Divyansh Singh <[email protected]>

bdougie approved these changes Jun 14, 2023

View reviewed changes

Anush008 mentioned this pull request Jun 15, 2023

docs: Added PR review menu docs open-sauced/docs#98

Merged

19 tasks

bdougie merged commit ef52439 into beta Jun 15, 2023

github-actions bot pushed a commit that referenced this pull request Jun 15, 2023

chore(minor): release 1.7.0-beta.2 on beta channel [skip ci]

d1a400b

## [1.7.0-beta.2](v1.7.0-beta.1...v1.7.0-beta.2) (2023-06-15) ### 🍕 Features * AI PR review menu ([#174](#174)) ([ef52439](ef52439))

github-actions bot added the released on @beta label Jun 15, 2023

github-actions bot added the released label Jun 15, 2023

Anush008 deleted the feat-pr-review-menu branch June 15, 2023 08:55

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

feat: AI PR review menu #174

feat: AI PR review menu #174

Anush008 commented Jun 11, 2023 •

edited

Loading

bdougie commented Jun 11, 2023

Anush008 commented Jun 13, 2023

diivi commented Jun 13, 2023

bdougie commented Jun 13, 2023 •

edited

Loading

brandonroberts commented Jun 13, 2023

Anush008 commented Jun 14, 2023 •

edited

Loading

brandonroberts commented Jun 14, 2023

bdougie left a comment

Anush008 commented Jun 14, 2023 •

edited

Loading

bdougie commented Jun 15, 2023

github-actions bot commented Jun 15, 2023

github-actions bot commented Jun 15, 2023

	const explainCode = AICodeReviewMenuItem(
	"Explain Code",
	"Generate an explanation for the code",
	generateCodeExplanation,
	commentNode,
	);

	const menu = AICodeReviewMenu([refactorCode, testCode, explainCode]);

feat: AI PR review menu #174

feat: AI PR review menu #174

Conversation

Anush008 commented Jun 11, 2023 • edited Loading

What type of PR is this? (check all applicable)

Description

Related Tickets & Documents

Mobile & Desktop Screenshots/Recordings

Added tests?

Added to documentation?

bdougie commented Jun 11, 2023

Anush008 commented Jun 13, 2023

diivi commented Jun 13, 2023

bdougie commented Jun 13, 2023 • edited Loading

brandonroberts commented Jun 13, 2023

Anush008 commented Jun 14, 2023 • edited Loading

brandonroberts commented Jun 14, 2023

bdougie left a comment

Choose a reason for hiding this comment

Anush008 commented Jun 14, 2023 • edited Loading

bdougie commented Jun 15, 2023

github-actions bot commented Jun 15, 2023

github-actions bot commented Jun 15, 2023

Anush008 commented Jun 11, 2023 •

edited

Loading

bdougie commented Jun 13, 2023 •

edited

Loading

Anush008 commented Jun 14, 2023 •

edited

Loading

Anush008 commented Jun 14, 2023 •

edited

Loading