-
Notifications
You must be signed in to change notification settings - Fork 41
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
feat: CodeEditor AI driven auto completion #484
Conversation
📊 Bundle Size AnalysisTotal Bundle Size
DiffNo significant changes in bundle size. View Dependencies
What is this?This is an automated bundle size report generated during the build process. |
@seveibar idk the backend structure for the api key, so I will leave it to you. |
📊 Bundle Size AnalysisTotal Bundle Size
DiffNo significant changes in bundle size. View Dependencies
What is this?This is an automated bundle size report generated during the build process. |
📊 Bundle Size AnalysisTotal Bundle Size
DiffNo significant changes in bundle size. View Dependencies
What is this?This is an automated bundle size report generated during the build process. |
src/components/CodeEditor.tsx
Outdated
@@ -180,6 +182,20 @@ export const CodeEditor = ({ | |||
: javascript({ typescript: true, jsx: true }), | |||
keymap.of([indentWithTab]), | |||
EditorState.readOnly.of(readOnly), | |||
copilotPlugin({ | |||
apiKey: codeCompletionApi.apiKey, |
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.
can you enable only if the codeCompletionApi.apiKey is defined?
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.
ok
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.
awesome work, one minor change before it goes to production
} | ||
return import.meta.env.VITE_USE_DIRECT_AI_REQUESTS === "true" | ||
? { | ||
apiKey: import.meta.env.VITE_CODE_COMPLETION_API_KEY, |
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.
what service?
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.
we should make the API_KEY indicate the service being used? Is it anthropic?
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.
@seveibar it should be codium api key?
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.
ok then we should name the api key VITE_CODIUM_API_KEY
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.
how did you test this? I'm not sure codium offers an api
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.
it looks like the user would have to provide a codium key? I'm a bit confused how we use this in practice
📊 Bundle Size AnalysisTotal Bundle Size
DiffNo significant changes in bundle size. View Dependencies
What is this?This is an automated bundle size report generated during the build process. |
📊 Bundle Size AnalysisTotal Bundle Size
DiffNo significant changes in bundle size. View Dependencies
What is this?This is an automated bundle size report generated during the build process. |
No description provided.