-
Notifications
You must be signed in to change notification settings - Fork 9
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 LLM-Based Recommendations for Broken Pipeline Runs #57
Open
Morgan-Summer-Davis
wants to merge
113
commits into
zenml-io:develop
Choose a base branch
from
sklarfox:feature/llm-fix-my-pipeline
base: develop
Could not load branches
Branch not found: {{ refName }}
Loading
Could not load tags
Nothing to show
Loading
Are you sure you want to change the base?
Some commits from the old base branch may be removed from the timeline,
and old review comments may become outdated.
+4,228
−381
Open
Changes from 107 commits
Commits
Show all changes
113 commits
Select commit
Hold shift + click to select a range
bb52c88
feat(secrets): add commands to register, update, and remove OpenAI AP…
sklarfox 5fc0003
feat(secrets): add function to retrieve secrets from VSCode
sklarfox a0de734
feat: remove unnecessary deleteOpenAIKey function
sklarfox a22ee0e
feat(genai): scaffold command to send OpenAI API request
sklarfox 97bb892
feat: add basic OpenAI API request with hardcoded prompt
sklarfox a64b86f
feat: send hardcoded message to OpenAI
sklarfox f3deec4
feat: Added basic custom context menu to DAG
Morgan-Summer-Davis 03d8e07
feat: Implemented hardcoded OpenAI calls from the DAG context menu
Morgan-Summer-Davis 62c7fdd
fix: Fixed error in sendOpenAIRequest from user error during merge
Morgan-Summer-Davis 6824174
feat: generalize LLM API Key registry command, support for Anthropic,…
sklarfox e1a363f
chore: remove todo comments
sklarfox 5035dc0
feat: Context menu now stays open and indicates it is loading when Su…
Morgan-Summer-Davis c193356
chore: address coderabbitai suggestions
sklarfox 29a93ee
feat: Suggest Fix only appears in context menu if the step resulted i…
Morgan-Summer-Davis fd94ec0
fix: DAG context menu now correctly retrieves component id
Morgan-Summer-Davis eb420c4
feat: Add initial fixMyPipelineRequest function to send code/error to…
sklarfox 948c2c7
feat: read log file from disk, send source code / log to OpenAI
sklarfox 0ca54d0
feat: added editStepFile function to allow gen AI features to make co…
Morgan-Summer-Davis 0d3acb6
feat: modify LLM prompts to highlight typos and include edited code i…
sklarfox 068ffc9
feat: suggest fix now opens a hard-coded file, finds where the step i…
Morgan-Summer-Davis 38d76b3
fix: adjusted virtual document rendering
Morgan-Summer-Davis 88fb58f
feat: add searchWorkspaceByFileContent function
sklarfox 04501cb
fix: create StepData type declaration
sklarfox bf8648c
fix: replace JsonObject type with StepData
sklarfox dec0d91
fix: remove console log
sklarfox 3777ce7
feat: refactored editStepFile to use vsCode's native difference manag…
Morgan-Summer-Davis fce9137
Merge branch 'exp/source-to-chatgpt' into feature/openai-api-requests
sklarfox 7838108
fix: adjusted the 'suggest fix' text document to correctly display ma…
Morgan-Summer-Davis 9ede811
refactor: add ArtifactData type and replace JsonObject types
sklarfox d1b5720
Merge branch 'stepdata-type' into feature/openai-api-requests
sklarfox 10c1ee8
feat: dag context menu will no longer extend off screen
Morgan-Summer-Davis f045b45
feat: create AIService singleton class
sklarfox 3c186ef
refactor: searchWorksapceByFileContent returns an array of 0 or more …
sklarfox aff9c40
refactor: split context menu logic into more encapsulated functions
Morgan-Summer-Davis 54b3e74
refactor: modify apikey secret name
sklarfox 8f0a4fe
refactor: change api key naming convention to `zenml.{provider}.key`
sklarfox d9fffce
Merge remote-tracking branch 'origin/feature/openai-api-key' into fea…
sklarfox a58e05c
feat: 'suggest fix' AI response now opens as markdown preview, rather…
Morgan-Summer-Davis f42ee33
feat: add FixMyPipeline interface
sklarfox 3749483
refactor: editStepFile accepts a vscode.Uri type directly instead of …
sklarfox bc63e1f
refactor: searchWOrkspaceByFileContent returns array of matches
sklarfox d274e4d
feat: aiService.fixMyPipelineRequest returns fixMyPipelineResponse ty…
sklarfox 00fb0d7
feat: migrate openai API call to structured response
sklarfox 2ade4dd
feat: implemented functionality to cycle between 'suggest fix' code …
Morgan-Summer-Davis d772e2b
refactor: removed some unnecessary comments / logic
Morgan-Summer-Davis 473dd16
feat: added command palette command to cycle through code recommendat…
Morgan-Summer-Davis ab965be
feat: Add OpenAI structured response format
sklarfox 9b245b1
Merge branch 'feature/openai-api-requests' into aiService-singleton
sklarfox 157c88b
feat: Added a command to the command bar for files edited by Suggest …
Morgan-Summer-Davis 7d1f6a3
Merge pull request #1 from sklarfox/aiService-singleton
sklarfox c176a82
fix: fixed logic in createCodeRecommendation
Morgan-Summer-Davis 2f7f9b8
fix: code recommendation should now always open in the proper viewColumn
Morgan-Summer-Davis 2a4fd27
feat: code recommendations for files no longer persist after being cl…
Morgan-Summer-Davis 515c575
fix: suggest fix files now open relative to the DAG that spawned them
Morgan-Summer-Davis 2b3b85a
fix: suggest fix no longer prompts the user to see the next code sugg…
Morgan-Summer-Davis 0ccbf10
fix: removed hardcoded values for ChatGPT response in suggest fix
Morgan-Summer-Davis 2192fdb
refactor: moved fixBrokenStep to AIStepFixer
Morgan-Summer-Davis cc1bb36
fix: removed unnecessary imports
Morgan-Summer-Davis fab0fc0
Merge branch 'feature/ui-improvements' into feature/openai-api-requests
Morgan-Summer-Davis 18ab0a5
fix: fixed several merge-related issues
Morgan-Summer-Davis 11632b6
feat: migrate to provider agnostic, replace openai with token.js
sklarfox 5f38d20
chore: remove comment
sklarfox 89b5842
chore: token.js ts-expect-error annotation
sklarfox 196a325
fix: add check for existing API key variable
sklarfox 9f17914
fix: implemented Suggest Fix with vscode.diff and a virtual document …
Morgan-Summer-Davis 23afa52
feat: implemented Suggest Fix with a virtual file system to allow liv…
Morgan-Summer-Davis 5c48e9a
fix: fixed updateCodeRecommendation functionality with new virtual fi…
Morgan-Summer-Davis 40d609a
fix: adjusted several uri comparisons to string comparisons
Morgan-Summer-Davis 0877593
feat: implemented git grep as a fallback in cases where manual file s…
Morgan-Summer-Davis 9f15f60
feat: searchWorkspaceByFileContent now returns both uri and the file'…
Morgan-Summer-Davis 090b5d8
feat: implemented an Accept Change option in suggest fix
Morgan-Summer-Davis 495b8d0
fix: suggest fix will now search for the location of the relevant fil…
Morgan-Summer-Davis be2ac96
feat: added notifications on unexpected outcomes when searching for f…
Morgan-Summer-Davis 09c64c4
fix: suggest fix will not attempt to create a code recommendation if …
Morgan-Summer-Davis cddc78e
feat: implemented UI for selecting default LLM for suggest fix
Morgan-Summer-Davis 0b5c677
feat: implemented frontend and dummy backend code for setting a defau…
Morgan-Summer-Davis 338c356
refactor: split secondary functionality in AIStepFixer.ts into modules
Morgan-Summer-Davis e85a3cb
fix: change order of conditionals when checking for an existing API k…
sklarfox c205c74
feat: migrate python snippets to non-regex method, map response
sklarfox 189fa44
test: add mockResponse
sklarfox 4b4d639
feat: expand configuration options to include Anthropic and Gemini mo…
sklarfox 79d1bc9
chore: remove old config file
sklarfox f23ff51
feat: add llm configuration options / getter methods for frontend UI
sklarfox f37d385
chore: fix linting curly warnings
sklarfox 0e61c98
fix: change order of conditionals when checking for an existing API k…
sklarfox 51cd458
feat: migrate python snippets to non-regex method, map response
sklarfox 2bbd7e2
test: add mockResponse
sklarfox 3292518
feat: expand configuration options to include Anthropic and Gemini mo…
sklarfox 31d2b1f
chore: remove old config file
sklarfox 596c1ae
feat: add llm configuration options / getter methods for frontend UI
sklarfox 63df37a
chore: fix linting curly warnings
sklarfox 6cb4c5b
Merge branch 'prompt-engineering' of https://github.com/sklarfox/vsco…
sklarfox 2daee5b
refactor: rename zenml config setting from llm-provider to llm-model
sklarfox 88a66bb
fix: SaveAIChangeEmitter import path
sklarfox 320c6c2
refactor(AIStepFixer): add Provider/Model typing from AIService, adju…
sklarfox 6c29ce7
chore: remove mock data and deprecated test command
sklarfox c69cdfe
Merge pull request #2 from sklarfox/prompt-engineering
sklarfox 5819e4f
fix: resolved bug that only allowed users to change model by resettin…
Morgan-Summer-Davis 02d8207
feat: added indicator to DAG context menu to show the current LLM
Morgan-Summer-Davis 40d9e9e
fix: fixed error which would incorrectly set API key to stringified u…
Morgan-Summer-Davis 62cf143
refactor: reestablished front-end refactor that was accidentally remo…
Morgan-Summer-Davis a45b6c7
fix: fixed secret storage so that storage of LLM names with periods w…
Morgan-Summer-Davis 2f2d092
feat: added message when user attempts to suggest a fix with no LLM s…
Morgan-Summer-Davis b353d33
feat: added error-handling when LLM sends back an error response
Morgan-Summer-Davis fcc1eab
fix: re-registering an LLM API key now updates the corresponding envi…
sklarfox ed4f7df
Merge branch 'feature/llm-fix-my-pipeline' into develop
Morgan-Summer-Davis 3601132
fix: addresses most of coderabbitai's suggestions
Morgan-Summer-Davis 3fa49bf
fix: code file will no longer open when there are no code fixes to su…
Morgan-Summer-Davis d920b32
refactor: remove process.env calls throughout AIService, make secretS…
sklarfox 754c0f1
refactor: removed console.log
Morgan-Summer-Davis 00a1f26
feat: added notice to top of markdown document
Morgan-Summer-Davis aa43052
fix: fixed broken 'Accept Changes' functionality
Morgan-Summer-Davis 5ea8113
fix: codeblocks that are indented in the response are now de-indented…
Morgan-Summer-Davis File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
Large diffs are not rendered by default.
Oops, something went wrong.
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Oops, something went wrong.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.