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.

Sign up for GitHub

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

Jump to bottom

feat: display AI Fix ⚡️ [IDE-580] #596

Merged

acke merged 28 commits into master from feat/IDE-580-ai-fix

Sep 20, 2024

Contributor

cat2608 commented Sep 4, 2024 •

edited by acke

Loading

Description

Wire-up CTAs (call-to-action):

Generate fix using Snyk DeepCode AI
Apply fix
Retry generating AI fixes

This change needs: snyk/snyk-ls#647

Tickets:

IDE-579
IDE-580

Checklist

Tests added and all succeed
Linted
CHANGELOG.md updated
README.md updated, if user-facing

Screenshots / GIFs

Visuals that may help the reviewer. Please add screenshots for any UI change. GIFs are most welcome!

Catalina Oyaneder and others added 6 commits

September 3, 2024 12:04


          wip: enable AI Fix

1ea7c69


          feat: wip - bridge button action with LS wrapper

73b5081


          feat: get Code Diff from LS

5563b32


          debug: test Language Server response

b156653


          fix: wip parsing of command

575fa56


          feat: display AI suggestions ⚡️

7421d80

cat2608 mentioned this pull request

feat: AI Fix for IntelliJ ⚡️ [IDE-580] snyk/snyk-ls#647

Merged

4 tasks

Catalina Oyaneder and others added 11 commits

September 4, 2024 14:56


          chore: update Apply fix button style

ff1400b


          chore: group similar JS logic

8a4b271


          fix: responsive UI during AI Fix call with LS async request

e03954c


          feat: wip - apply fix

6c0b5fc


          fix: added support for navigating between AI Fixes

281670a


          feat: show section for error, if ai fix reqest fails to generate fixes

c987b11


          fix: add the same spacing for error section as for show fixes section

ddea6a7


          chore: wip - apply fix

dbaacd1


          chore: wip add test to handle patches

d0937ba


          fix: fix flow for regenerating ai fixes after error

8a387b6


          tidy: replace hardcoded colors with JBUI theme colors

e0fc1d2

bastiandoetsch reviewed

View reviewed changes

src/main/kotlin/io/snyk/plugin/ui/jcef/ApplyFixHandler.kt Outdated Show resolved Hide resolved

bastiandoetsch reviewed

View reviewed changes

src/main/kotlin/io/snyk/plugin/ui/jcef/ApplyFixHandler.kt Outdated Show resolved Hide resolved

bastiandoetsch reviewed

View reviewed changes

src/main/kotlin/io/snyk/plugin/ui/jcef/ApplyFixHandler.kt Outdated Show resolved Hide resolved

bastiandoetsch reviewed

View reviewed changes

src/main/kotlin/io/snyk/plugin/ui/jcef/ApplyFixHandler.kt Outdated Show resolved Hide resolved

bastiandoetsch reviewed

View reviewed changes

src/main/kotlin/io/snyk/plugin/ui/jcef/ApplyFixHandler.kt Outdated Show resolved Hide resolved

bastiandoetsch reviewed

View reviewed changes

src/main/kotlin/io/snyk/plugin/ui/jcef/ApplyFixHandler.kt Outdated Show resolved Hide resolved

bastiandoetsch reviewed

View reviewed changes

src/main/kotlin/io/snyk/plugin/ui/jcef/ApplyFixHandler.kt Outdated Show resolved Hide resolved

bastiandoetsch reviewed

View reviewed changes

src/main/kotlin/io/snyk/plugin/ui/jcef/ApplyFixHandler.kt Outdated Show resolved Hide resolved

bastiandoetsch reviewed

View reviewed changes

src/main/kotlin/io/snyk/plugin/ui/jcef/ApplyFixHandler.kt Outdated Show resolved Hide resolved

bastiandoetsch reviewed

View reviewed changes

src/main/kotlin/io/snyk/plugin/ui/jcef/GenerateAIFixHandler.kt Outdated Show resolved Hide resolved

bastiandoetsch reviewed

View reviewed changes

src/main/kotlin/io/snyk/plugin/ui/jcef/GenerateAIFixHandler.kt Outdated Show resolved Hide resolved

bastiandoetsch reviewed

View reviewed changes

src/main/kotlin/io/snyk/plugin/ui/jcef/GenerateAIFixHandler.kt Outdated Show resolved Hide resolved


          fix: update unit test and remove duplicate code

a174e45

acke force-pushed the feat/IDE-580-ai-fix branch from f4e88fd to a174e45 Compare

September 16, 2024 14:23

acke marked this pull request as ready for review

September 16, 2024 14:24

acke requested a review from a team as a code owner

September 16, 2024 14:24

acke added 3 commits

September 16, 2024 17:02


          fix: improve error handling and added balloon notifier for failing to…

ee0b923

… apply patch


          tidy: refactor functions to a new patcher

280a979


          tidy: remove unused parameters

f155b48

ShawkyZ reviewed

View reviewed changes

src/main/kotlin/snyk/common/lsp/LanguageServerWrapper.kt Outdated Show resolved Hide resolved

ShawkyZ reviewed

View reviewed changes

src/main/kotlin/snyk/common/lsp/LanguageServerWrapper.kt Show resolved Hide resolved

ShawkyZ reviewed

View reviewed changes

src/main/kotlin/io/snyk/plugin/ui/jcef/ApplyFixHandler.kt Outdated Show resolved Hide resolved

ShawkyZ approved these changes

View reviewed changes

Contributor

ShawkyZ left a comment

Looks great !

Please check my comments.
Nothing blocking

bastiandoetsch reviewed

View reviewed changes

src/main/kotlin/io/snyk/plugin/DiffPatcher.kt Show resolved Hide resolved

bastiandoetsch reviewed

View reviewed changes

src/main/kotlin/io/snyk/plugin/ui/jcef/ApplyFixHandler.kt Outdated Show resolved Hide resolved

bastiandoetsch reviewed

View reviewed changes

src/main/kotlin/io/snyk/plugin/ui/jcef/ApplyFixHandler.kt Outdated Show resolved Hide resolved

bastiandoetsch reviewed

View reviewed changes

src/main/kotlin/io/snyk/plugin/ui/jcef/ApplyFixHandler.kt Outdated Show resolved Hide resolved

bastiandoetsch reviewed

View reviewed changes

src/main/kotlin/io/snyk/plugin/ui/jcef/ApplyFixHandler.kt Outdated Show resolved Hide resolved

bastiandoetsch reviewed

View reviewed changes

src/main/kotlin/io/snyk/plugin/ui/jcef/ApplyFixHandler.kt Outdated Show resolved Hide resolved

bastiandoetsch reviewed

View reviewed changes

src/main/kotlin/io/snyk/plugin/ui/jcef/ApplyFixHandler.kt Outdated Show resolved Hide resolved

bastiandoetsch reviewed

View reviewed changes

src/main/kotlin/io/snyk/plugin/ui/jcef/ApplyFixHandler.kt Outdated Show resolved Hide resolved

bastiandoetsch reviewed

View reviewed changes

src/main/kotlin/io/snyk/plugin/ui/jcef/ThemeBasedStylingGenerator.kt Show resolved Hide resolved

bastiandoetsch reviewed

View reviewed changes

src/main/kotlin/io/snyk/plugin/ui/jcef/ThemeBasedStylingGenerator.kt Show resolved Hide resolved

bastiandoetsch reviewed

View reviewed changes

src/main/kotlin/io/snyk/plugin/ui/jcef/ThemeBasedStylingGenerator.kt Show resolved Hide resolved


          fix: updating logging and refactor types

69e7c1e

acke force-pushed the feat/IDE-580-ai-fix branch from 6ae96a3 to 69e7c1e Compare

September 19, 2024 09:50

bastiandoetsch reviewed

View reviewed changes

src/main/kotlin/io/snyk/plugin/ui/jcef/ApplyFixHandler.kt Show resolved Hide resolved


          fix: add balloon notifier for file not found

2d00d29

acke enabled auto-merge (squash)

September 20, 2024 09:21

acke disabled auto-merge

September 20, 2024 09:22

acke merged commit 2c09695 into master

9 checks passed

acke deleted the feat/IDE-580-ai-fix branch

September 20, 2024 09:34

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet