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 glhf.chat to bolt.diy #968

Open
wants to merge 3 commits into
base: main
Choose a base branch
from

Conversation

Techno-Trek
Copy link

I added the open model execution service, all tests have already been carried out.
Some models were selected but others may be added later.

What was done:
The service provider has been added to the directory:

bolt.diy\app\lib\modules\llm\providers\glhf.chat.ts

The new service provider configuration was added to the file:

bolt.diy\app\lib\modules\llm\registry.ts

The API settings and access link were added to the file:

bolt.diy.env.example

add glhf.chat service to providers
add glhf.chat service to providers
add glhf.chat to bolt.diy
@Techno-Trek Techno-Trek changed the title Add glhf.chat to bolt.diy feat: Add glhf.chat to bolt.diy Jan 2, 2025
@Techno-Trek Techno-Trek changed the title feat: Add glhf.chat to bolt.diy feat: add glhf.chat to bolt.diy Jan 2, 2025
): Promise<ModelInfo[]> {
// Retornamos apenas os modelos estáticos, evitando duplicação
return this.staticModels;
}
Copy link
Collaborator

@thecodacus thecodacus Jan 3, 2025

Choose a reason for hiding this comment

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

delete this function getDynamicModels if no dynamic models added


// Usa o primeiro modelo como padrão se nenhum for especificado
const modelToUse = model || this.staticModels[0].name;
return getOpenAILikeModel(effectiveBaseUrl, apiKey, modelToUse);
Copy link
Collaborator

Choose a reason for hiding this comment

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

const modelToUse = model || this.staticModels[0].name;
no need for a fallback model as this is already handled in the core framework.

Copy link
Collaborator

Choose a reason for hiding this comment

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

@Techno-Trek as I see this is not fixed already. Can you do? :)

throw error;
}
}
}
Copy link
Collaborator

Choose a reason for hiding this comment

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

whats the use of this function, its not being used anywhere

Copy link
Author

Choose a reason for hiding this comment

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

I was using this to test the API connection because the glhf service was undergoing modifications and I ended up not removing it, but I will make the suggested changes and also the tests.

@thecodacus thecodacus added this to the v0.0.6 milestone Jan 6, 2025
@thecodacus thecodacus modified the milestones: v0.0.6, 0.0.7 Jan 22, 2025
@leex279 leex279 self-requested a review January 23, 2025 16:36
Copy link
Collaborator

@leex279 leex279 left a comment

Choose a reason for hiding this comment

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

Changes mentioned by @thecodacus not done yet.

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.

3 participants