Skip to content
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: add AI assistant to dokploy #1079

Open
wants to merge 2 commits into
base: canary
Choose a base branch
from

Conversation

kucherenko
Copy link
Contributor

Description:

This pull request introduces new AI-related features to enhance the functionality of the Dokploy open-source project. Details of the changes are as follows:

Features Added:

  1. AI Assistant for LLM-Generated Projects:

    • Implemented an AI Assistant feature that enables users to create projects using Large Language Model (LLM) generated content. This addition aims to streamline project initiation and enhance user productivity.
  2. AI Settings:

    • Added a dedicated settings section for configuring AI-related options to tailor the assistant's behavior and optimize the use of generated data for specific project needs.

Known Issues:

  1. OpenAI API Exclusivity:

    • Currently, the AI Assistant functionality only supports the OpenAI API. Future updates may include support for additional APIs to broaden usability.
  2. Dialogue UI Limitations:

    • The user interface for the dialogue feature does not currently allow users to view all generated data at once. This limitation may impact the user experience while interacting with project suggestions and outputs.

screenshot-localhost_3000-2025_01_08-16_55_36
screenshot-localhost_3000-2025_01_08-17_04_23
screenshot-localhost_3000-2025_01_08-17_03_46
screenshot-localhost_3000-2025_01_09-18_44_04
screenshot-localhost_3000-2025_01_08-16_59_51
screenshot-localhost_3000-2025_01_08-17_01_25
screenshot-localhost_3000-2025_01_08-17_01_42
screenshot-localhost_3000-2025_01_08-17_02_37
screenshot-localhost_3000-2025_01_08-17_02_58
screenshot-localhost_3000-2025_01_09-18_39_25

Comment on lines 43 to 57
<MonacoEditor
height="200px"
language="yaml"
theme="vs-dark"
value={templateInfo.details.dockerCompose}
options={{
minimap: { enabled: false },
scrollBeyondLastLine: false,
fontSize: 14,
lineNumbers: "on",
readOnly: true,
wordWrap: "on",
automaticLayout: true,
}}
/>
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Use the component, instead of monaco editor

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

do you have a code highlighter component?

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@kucherenko Sorry, this is the component

<CodeEditor
  lineWrapping
  value={data || "Empty"}
  disabled
  className="font-mono"
/>

"Sendgrid service opensource analogue",
];

export function StepOne({ nextStep, setTemplateInfo, templateInfo }: any) {
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

We already have a package installed for stepps called @stepperize/react we already use in the welcome-suscription.tsx component you can take a look how is implemented to have an idea

Also can you use arrow functions instead of regular functions?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Hi @Siumauricio,

I've addressed most of the comments from the review. Specifically, I removed Monaco and converted the functions to arrow functions. However, I wasn't able to work on the stepper due to time constraints as my Christmas holidays have ended.

Would it be possible to merge the changes without the stepper for now? I will add it to the technical debt list and prioritize fixing it as soon as I have the time.

Thank you for your understanding!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants