[PR #2473] [MERGED] fix: models/tongyi has global side effect on base_url #2506

Closed
opened 2026-02-16 11:17:11 -05:00 by yindo · 0 comments
Owner

📋 Pull Request Information

Original PR: https://github.com/langgenius/dify-official-plugins/pull/2473
Author: @Stream29
Created: 1/22/2026
Status: Merged
Merged: 1/23/2026
Merged by: @Stream29

Base: mainHead: fix/models/tongyi


📝 Commits (5)

  • c77d1d8 fix: global side effect
  • dfe71be chore: bump version
  • 3ff9278 Merge branch 'main' into fix/models/tongyi
  • 603ad67 Merge branch 'main' into fix/models/tongyi
  • 2f89774 chore: add .venv to .difyignore

📊 Changes

9 files changed (+108 additions, -41 deletions)

View changed files

models/tongyi/.difyignore (+1 -0)
📝 models/tongyi/manifest.yaml (+1 -1)
📝 models/tongyi/models/_common.py (+19 -0)
📝 models/tongyi/models/llm/llm.py (+2 -1)
📝 models/tongyi/models/rerank/rerank.py (+10 -4)
📝 models/tongyi/models/speech2text/speech2text.py (+8 -5)
📝 models/tongyi/models/text_embedding/text_embedding.py (+41 -13)
📝 models/tongyi/models/tts/tts.py (+16 -7)
📝 models/tongyi/uv.lock (+10 -10)

📄 Description

Related Issues or Context

Fixes #30611 .

Previously, models/tongyi plugin may produce global side effect that modifies base_url.
This PR fixes this behavior.

This PR contains Changes to LLM Models Plugin

Updated plugin was packaged and tested on cloud.dify.ai.

Version Control (Any Changes to the Plugin Will Require Bumping the Version)

  • I have Bumped Up the Version in Manifest.yaml (Top-Level Version Field, Not in Meta Section)

🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.

## 📋 Pull Request Information **Original PR:** https://github.com/langgenius/dify-official-plugins/pull/2473 **Author:** [@Stream29](https://github.com/Stream29) **Created:** 1/22/2026 **Status:** ✅ Merged **Merged:** 1/23/2026 **Merged by:** [@Stream29](https://github.com/Stream29) **Base:** `main` ← **Head:** `fix/models/tongyi` --- ### 📝 Commits (5) - [`c77d1d8`](https://github.com/langgenius/dify-official-plugins/commit/c77d1d89deb1249ee14595d41b148f21a7faed1f) fix: global side effect - [`dfe71be`](https://github.com/langgenius/dify-official-plugins/commit/dfe71be7fd76cc239e45f81c6a5b3c6163960022) chore: bump version - [`3ff9278`](https://github.com/langgenius/dify-official-plugins/commit/3ff9278620cb3bf7bc35cdf535bf9aed31c48b65) Merge branch 'main' into fix/models/tongyi - [`603ad67`](https://github.com/langgenius/dify-official-plugins/commit/603ad67b98a58d011ea7de74128cf5b1b794b606) Merge branch 'main' into fix/models/tongyi - [`2f89774`](https://github.com/langgenius/dify-official-plugins/commit/2f8977432373f1797e672a5acba0e70be01b9655) chore: add .venv to .difyignore ### 📊 Changes **9 files changed** (+108 additions, -41 deletions) <details> <summary>View changed files</summary> ➕ `models/tongyi/.difyignore` (+1 -0) 📝 `models/tongyi/manifest.yaml` (+1 -1) 📝 `models/tongyi/models/_common.py` (+19 -0) 📝 `models/tongyi/models/llm/llm.py` (+2 -1) 📝 `models/tongyi/models/rerank/rerank.py` (+10 -4) 📝 `models/tongyi/models/speech2text/speech2text.py` (+8 -5) 📝 `models/tongyi/models/text_embedding/text_embedding.py` (+41 -13) 📝 `models/tongyi/models/tts/tts.py` (+16 -7) 📝 `models/tongyi/uv.lock` (+10 -10) </details> ### 📄 Description ## Related Issues or Context Fixes #30611 . Previously, `models/tongyi` plugin may produce global side effect that modifies `base_url`. This PR fixes this behavior. ## This PR contains Changes to *LLM Models Plugin* Updated plugin was packaged and tested on `cloud.dify.ai`. ## Version Control (Any Changes to the Plugin Will Require Bumping the Version) - [x] I have Bumped Up the Version in Manifest.yaml (Top-Level `Version` Field, Not in Meta Section) --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>
yindo added the pull-request label 2026-02-16 11:17:11 -05:00
yindo closed this issue 2026-02-16 11:17:11 -05:00
Sign in to join this conversation.
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: langgenius/dify-official-plugins#2506