[PR #1494] [MERGED] feat(openai-api-compatible):remove thinking content from assistant messages #1923

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

📋 Pull Request Information

Original PR: https://github.com/langgenius/dify-official-plugins/pull/1494
Author: @QIN2DIM
Created: 8/12/2025
Status: Merged
Merged: 8/12/2025
Merged by: @crazywoola

Base: mainHead: oai-remove-thinking


📝 Commits (7)

  • b6f63ff feat(openai-api-compatible): add support for remote installation and improve model handling
  • 9dd3dce refactor(llm): remove thinking content from assistant messages
  • 60d8d2a Merge branch 'main' into oai-remove-thinking
  • 69f0549 build(manifest): update version to 0.1.0 in manifest.yaml
  • 30c7d53 ff
  • 6a19ede Update .gitignore
  • 1b76efd Create .difyignore

📊 Changes

14 files changed (+267 additions, -80 deletions)

View changed files

models/openai_api_compatible/.difyignore (+2 -0)
models/openai_api_compatible/.env.example (+3 -0)
models/openai_api_compatible/.gitignore (+2 -0)
models/openai_api_compatible/.python-version (+1 -0)
📝 models/openai_api_compatible/manifest.yaml (+1 -1)
📝 models/openai_api_compatible/models/common_openai.py (+11 -4)
📝 models/openai_api_compatible/models/llm/llm.py (+69 -30)
📝 models/openai_api_compatible/models/rerank/rerank.py (+10 -15)
📝 models/openai_api_compatible/models/speech2text/speech2text.py (+6 -7)
📝 models/openai_api_compatible/models/text_embedding/text_embedding.py (+7 -11)
📝 models/openai_api_compatible/models/tts/tts.py (+6 -8)
📝 models/openai_api_compatible/provider/openai_api_compatible.py (+1 -1)
models/openai_api_compatible/pyproject.toml (+21 -0)
📝 models/openai_api_compatible/requirements.txt (+127 -3)

📄 Description

Related Issues or Context

image image image

This PR contains Changes to Non-Plugin

  • Documentation
  • Other

This PR contains Changes to Non-LLM Models Plugin

  • I have Run Comprehensive Tests Relevant to My Changes

This PR contains Changes to LLM Models Plugin

  • My Changes Affect Message Flow Handling (System Messages and User→Assistant Turn-Taking)
  • My Changes Affect Tool Interaction Flow (Multi-Round Usage and Output Handling, for both Agent App and Agent Node)
  • My Changes Affect Multimodal Input Handling (Images, PDFs, Audio, Video, etc.)
  • My Changes Affect Multimodal Output Generation (Images, Audio, Video, etc.)
  • My Changes Affect Structured Output Format (JSON, XML, etc.)
  • My Changes Affect Token Consumption Metrics
  • My Changes Affect Other LLM Functionalities (Reasoning Process, Grounding, Prompt Caching, etc.)
  • Other Changes (Add New Models, Fix Model Parameters etc.)

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)

Dify Plugin SDK Version

  • I have Ensured dify_plugin>=0.3.0,<0.5.0 is in requirements.txt (SDK docs)

Environment Verification (If Any Code Changes)

Local Deployment Environment

  • Dify Version is: , I have Tested My Changes on Local Deployment Dify with a Clean Environment That Matches the Production Configuration.

SaaS Environment

  • I have Tested My Changes on cloud.dify.ai with a Clean Environment That Matches the Production Configuration

🔄 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/1494 **Author:** [@QIN2DIM](https://github.com/QIN2DIM) **Created:** 8/12/2025 **Status:** ✅ Merged **Merged:** 8/12/2025 **Merged by:** [@crazywoola](https://github.com/crazywoola) **Base:** `main` ← **Head:** `oai-remove-thinking` --- ### 📝 Commits (7) - [`b6f63ff`](https://github.com/langgenius/dify-official-plugins/commit/b6f63ff476d2e9cce24a22c9644dee0e3ec4d803) feat(openai-api-compatible): add support for remote installation and improve model handling - [`9dd3dce`](https://github.com/langgenius/dify-official-plugins/commit/9dd3dce826885395da7cf8f3e87480f8b135379d) refactor(llm): remove thinking content from assistant messages - [`60d8d2a`](https://github.com/langgenius/dify-official-plugins/commit/60d8d2a6cb91cea18000cd83d7e9963eca287f8b) Merge branch 'main' into oai-remove-thinking - [`69f0549`](https://github.com/langgenius/dify-official-plugins/commit/69f05499d65c3dbe0eb88cfbb6ef0705e1a36fb7) build(manifest): update version to 0.1.0 in manifest.yaml - [`30c7d53`](https://github.com/langgenius/dify-official-plugins/commit/30c7d53ace0bc2472170959a3b85a74afd51dae1) ff - [`6a19ede`](https://github.com/langgenius/dify-official-plugins/commit/6a19edef7c8022bb9ec7375d26932f05bb50d0e9) Update .gitignore - [`1b76efd`](https://github.com/langgenius/dify-official-plugins/commit/1b76efd2c4c87475b8bb1ad25087b2c1e5622fc6) Create .difyignore ### 📊 Changes **14 files changed** (+267 additions, -80 deletions) <details> <summary>View changed files</summary> ➕ `models/openai_api_compatible/.difyignore` (+2 -0) ➕ `models/openai_api_compatible/.env.example` (+3 -0) ➕ `models/openai_api_compatible/.gitignore` (+2 -0) ➕ `models/openai_api_compatible/.python-version` (+1 -0) 📝 `models/openai_api_compatible/manifest.yaml` (+1 -1) 📝 `models/openai_api_compatible/models/common_openai.py` (+11 -4) 📝 `models/openai_api_compatible/models/llm/llm.py` (+69 -30) 📝 `models/openai_api_compatible/models/rerank/rerank.py` (+10 -15) 📝 `models/openai_api_compatible/models/speech2text/speech2text.py` (+6 -7) 📝 `models/openai_api_compatible/models/text_embedding/text_embedding.py` (+7 -11) 📝 `models/openai_api_compatible/models/tts/tts.py` (+6 -8) 📝 `models/openai_api_compatible/provider/openai_api_compatible.py` (+1 -1) ➕ `models/openai_api_compatible/pyproject.toml` (+21 -0) 📝 `models/openai_api_compatible/requirements.txt` (+127 -3) </details> ### 📄 Description ## Related Issues or Context <!-- ⚠️ NOTE: This repository is for Dify Official Plugins only. For community contributions, please submit to https://github.com/langgenius/dify-plugins instead. - Link Related Issues if Applicable: #issue_number - Or Provide Context about Why this Change is Needed --> <img width="701" height="392" alt="image" src="https://github.com/user-attachments/assets/42e97bf4-b292-4edb-8e57-946fc0573d21" /> <img width="1461" height="489" alt="image" src="https://github.com/user-attachments/assets/c8de2979-3530-4680-8328-d83f80f9f601" /> <img width="1839" height="522" alt="image" src="https://github.com/user-attachments/assets/aaf50588-3225-4dff-b1e2-408664260178" /> ## This PR contains Changes to *Non-Plugin* <!-- Put an `x` in all the boxes that apply by replacing [ ] with [x] For example: - [x] Documentation --> - [ ] Documentation - [ ] Other ## This PR contains Changes to *Non-LLM Models Plugin* - [x] I have Run Comprehensive Tests Relevant to My Changes <!-- 📷 Include Screenshots/Videos Demonstrating the Fix, New Feature, or the Behavior Before/After Breaking Changes. --> ## This PR contains Changes to *LLM Models Plugin* <!-- LLM Models Test Example: --> <!-- https://github.com/langgenius/dify-official-plugins/blob/main/.assets/test-examples/llm-plugin-tests/llm_test_example.md --> - [x] My Changes Affect Message Flow Handling (System Messages and User→Assistant Turn-Taking) <!-- 📷 Include Screenshots/Videos Demonstrating the Fix, New Feature, or the Behavior Before/After Breaking Changes. --> - [ ] My Changes Affect Tool Interaction Flow (Multi-Round Usage and Output Handling, for both Agent App and Agent Node) <!-- 📷 Include Screenshots/Videos Demonstrating the Fix, New Feature, or the Behavior Before/After Breaking Changes. --> - [ ] My Changes Affect Multimodal Input Handling (Images, PDFs, Audio, Video, etc.) <!-- 📷 Include Screenshots/Videos Demonstrating the Fix, New Feature, or the Behavior Before/After Breaking Changes. --> - [ ] My Changes Affect Multimodal Output Generation (Images, Audio, Video, etc.) <!-- 📷 Include Screenshots/Videos Demonstrating the Fix, New Feature, or the Behavior Before/After Breaking Changes. --> - [ ] My Changes Affect Structured Output Format (JSON, XML, etc.) <!-- 📷 Include Screenshots/Videos Demonstrating the Fix, New Feature, or the Behavior Before/After Breaking Changes. --> - [ ] My Changes Affect Token Consumption Metrics <!-- 📷 Include Screenshots/Videos Demonstrating the Fix, New Feature, or the Behavior Before/After Breaking Changes. --> - [ ] My Changes Affect Other LLM Functionalities (Reasoning Process, Grounding, Prompt Caching, etc.) <!-- 📷 Include Screenshots/Videos Demonstrating the Fix, New Feature, or the Behavior Before/After Breaking Changes. --> - [ ] Other Changes (Add New Models, Fix Model Parameters etc.) <!-- 📷 Include Screenshots/Videos Demonstrating the Fix, New Feature, or the Behavior Before/After Breaking Changes. --> ## 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) <!-- ⚠️ NOTE: Version Format: MAJOR.MINOR.PATCH - MAJOR (0.x.x): Reserved for Significant architectural changes or incompatible API modifications - MINOR (x.0.x): For New feature additions while maintaining backward compatibility - PATCH (x.x.0): For Backward-compatible bug fixes and minor improvements - Note: Each Version Component (MAJOR, MINOR, PATCH) Can Be 2 Digits, e.g., 10.11.22 --> ## Dify Plugin SDK Version - [x] I have Ensured `dify_plugin>=0.3.0,<0.5.0` is in requirements.txt ([SDK docs](https://github.com/langgenius/dify-plugin-sdks/blob/main/python/README.md)) ## Environment Verification (If Any Code Changes) <!-- ⚠️ NOTE: At Least One Environment Must Be Tested. --> ### Local Deployment Environment - [x] Dify Version is: <!-- Specify Your Version (e.g., 1.2.0) -->, I have Tested My Changes on Local Deployment Dify with a Clean Environment That Matches the Production Configuration. <!-- - Python Virtual Env Matching Manifest.yaml & requirements.txt - No Breaking Changes in Dify That May Affect the Testing Result --> ### SaaS Environment - [x] I have Tested My Changes on cloud.dify.ai with a Clean Environment That Matches the Production Configuration <!-- - Python Virtual Env Matching Manifest.yaml & requirements.txt --> --- <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:15:14 -05:00
yindo closed this issue 2026-02-16 11:15:14 -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#1923