Skip to content

Conversation

@yshngg
Copy link
Contributor

@yshngg yshngg commented Jun 23, 2025

The changes:

  1. Expanded the skip condition to include additional AI providers (DEEPSEEK, GROQ, MISTRAL, MLX, OPENROUTER) beyond just TEST
  2. Maintained existing TEST provider skip behavior
  3. Added explicit comment explaining the skip logic

The why:
Prevents migration execution for unsupported AI providers to avoid potential runtime errors or data inconsistencies, ensuring migrations only run for properly handled configurations.

Fix #468

The changes:
1. Expanded the skip condition to include additional AI providers (DEEPSEEK, GROQ, MISTRAL, MLX, OPENROUTER) beyond just TEST
2. Maintained existing TEST provider skip behavior
3. Added explicit comment explaining the skip logic

The why:
Prevents migration execution for unsupported AI providers to avoid potential runtime errors or data inconsistencies, ensuring migrations only run for properly handled configurations.
Repository owner deleted a comment from wshao777 Jun 29, 2025
@di-sukharev di-sukharev merged commit fd22f71 into di-sukharev:master Jun 29, 2025
5 of 6 checks passed
@zxf4399
Copy link

zxf4399 commented Jul 26, 2025

Hi @di-sukharev Could you publish a new tag with this change.
I still encounter the issue, thanks

@di-sukharev
Copy link
Owner

@zxf4399 this is now in npm, please update

Repository owner deleted a comment from wshao777 Aug 2, 2025
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.

Failed to apply migration 00_use_single_api_key_and_url: Error: Migration failed

3 participants