feat: generate code suggestions via AI

[diivi]

In action (except the duplicate buttons):

I need to take care of some weird DOM-watching stuff; after that, this should be ready to merge.

The Generator button is disappearing randomly right now, maybe something to do with how the watcher is set up.

[diivi]

lol it's the same issue I guess.

[diivi]

This is 95% done, I am having just one issue though:

Sometimes the refactor button randomly disappears (it comes back after adding a new section or reloading).

Like in the second review here ^

[diivi]

Apparently, adding a delay to DOM manipulation did the trick. Maybe some small details can still be changed, but this should work overall (at least for single reviews).

[bdougie]

I won't be able to review for another hour l. Can you a gif of this working with code.

[diivi]

Here's a random sample generation (didn't verify if it's correct or not, but it does assist the reviewer):

[bdougie]

We can continue to develop this (in future prs). Doesn't have to be correct now, is the beauty of machine learning.

In a future PR, we can add a "was this helpful link" or similar 👍or👎buttons

[bdougie]

I have tried testing this a few times and have not been able to even get the button to show.

bdougie/bot-test-repo#480

[Anush008]

It worked for me.

[diivi]

I have tried testing this a few times and have not been able to even get the button to show.

Maybe there's something wrong with the selectors? Can you confirm the js-suggestion-button-placeholder class exists on the change suggestion button?

[bdougie]

Maybe there's something wrong with the selectors? Can you confirm the js-suggestion-button-placeholder class exists on the change suggestion button?

The class exists.

[diivi]

This is weird, what about inline-comments (on the comment box)?

[bdougie]

@open-sauced/triage team, want to check this out and see if it works for you?

[a0m0rajab]

export const ChangeSuggestorButton = (commentNode: HTMLElement) => {
  const changeSuggestorButton = createHtmlElement("a", {
    innerHTML: `
      <span id="ai-change-gen" class="toolbar-item btn-octicon">
        <img class="octicon octicon-heading" height="16px" width="16px" id="ai-description-button-logo" src=${chrome.runtime.getURL(openSaucedLogoIcon)}>
      </span>
      <tool-tip for="ai-change-gen" class="sr-only" role="tooltip">Get Refactor Suggestions</tool-tip>
    `, being returned.ified the onclick event by removing the async/await as it doesn't seem necessary.
- Formatted the code for better readability.
Description generated using [OpenSauced](https://opensauced.ai/).
- Simpl
    onclick: () => handleSubmit(commentNode),
  });
  return changeSuggestorButton;
};

• Added a return statement to the function to make it more clear what is

[a0m0rajab]

Worked on my PC.
I needed to hard refresh (CTRL+SHIFT+R) to get the OpenSauced icon.

Another example:

[diivi]

@bdougie does hard refresh work for you? I have started working on the endpoint for this feature.

[bdougie]

@bdougie does hard refresh work for you? I have started working on the endpoint for this feature.

No. I wonder if it is because I have another extension installed.

https://chrome.google.com/webstore/detail/gifs-for-github/dkgjnpbipbdaoaadbdhpiokaemhlphep

[diivi]

No. I wonder if it is because I have another extension installed.

Did you try disabling the GIF extension once? If we decide that's the issue, maybe I can look at the conflict (shouldn't be important in that case though).

[takanome-dev]

Not working for me even after turning off all other extensions. There is only a div > button containing the suggested changes btn from GitHub inside div.js-suggestion-button-placeholder

pull-request-.100.open-sauced_ai.mp4

[diivi]

Is there something relevant in the console? @takanome-dev

[takanome-dev]

Is there something relevant in the console? @takanome-dev

No, there is nothing in there 🤷‍♀️

[diivi]

Super weird, I'm confused about how to debug this🤔 Can someone who can reproduce volunteer? I'm open for a discord call too.

[a0m0rajab]

I tried the last updates, and it's not working right now, not sure how it worked before. @diivi I would love to have a call to debug it together.

[diivi]

@a0m0rajab Right now, does the button appear for you? If it does, then it's fine, the suggestion won't appear for some time because the API endpoint is not live, so you'd be getting a 404 error.

If even the button does not appear, we can debug anytime you're free in the next 4 hours.

[a0m0rajab]

Nope, even the button is not visible right now, I changed branches and it stopped working. (I could not make it appear even if I went back with commits)

[diivi]

@bdougie and @takanome-dev can you test if the button shows up now? The refactor might not work until we have the refactor endpoint in production on api.opensauced.pizza.

[bdougie]

We will probably see it get released a little later today. @brandonroberts let us know when it goes live.

https://beta.api.opensauced.pizza/#/Pull%20Requests%20service/generateCodeRefactor

update: https://github.com/open-sauced/api.opensauced.pizza/pull/172

[bdougie]

@bdougie and @takanome-dev can you test if the button shows up now? The refactor might not work until we have the refactor endpoint in production on api.opensauced.pizza.

The button now shows.

[brandonroberts]

API release is merged, waiting on deployment

[brandonroberts]

API is deployed https://api.opensauced.pizza/#/Pull%20Requests%20service/generateCodeRefactor

[diivi]

@bdougie does it work for you?

[bdougie]

Could you update the docs.opensauced.pizza to include steps on how to leverage this feature.

I think we are good to go.

[bdougie]

Look forward to shipping this live!

[github-actions]

🎉 This PR is included in version 1.2.0-beta.3 🎉

The release is available on GitHub release

Your semantic-release bot 📦🚀

[github-actions]

🎉 This PR is included in version 1.2.0 🎉

The release is available on GitHub release

Your semantic-release bot 📦🚀