[PR #263] [MERGED] Revert "fix: increase max_tokens from 5 to 16 for some specific model… #270

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

📋 Pull Request Information

Original PR: https://github.com/langgenius/dify-plugin-sdks/pull/263
Author: @Mairuis
Created: 12/9/2025
Status: Merged
Merged: 12/9/2025
Merged by: @Mairuis

Base: mainHead: fix/revert_max_tokens


📝 Commits (1)

  • 97f9d2c Revert "fix: increase max_tokens from 5 to 16 for some specific models like gpt-5-codex in ping test, add PING_MAX_TOKENS configuration for model validation ping requests (#235)"

📊 Changes

2 files changed (+2 additions, -11 deletions)

View changed files

📝 python/dify_plugin/config/config.py (+0 -5)
📝 python/dify_plugin/interfaces/model/openai_compatible/llm.py (+2 -6)

📄 Description

This reverts commit ee8ea3de16.

Pull Request Checklist

Thank you for your contribution! Before submitting your PR, please make sure you have completed the following checks:

Compatibility Check

  • I have checked whether this change affects the backward compatibility of the plugin declared in README.md
  • I have checked whether this change affects the forward compatibility of the plugin declared in README.md
  • If this change introduces a breaking change, I have discussed it with the project maintainer and specified the release version in the README.md
  • I have described the compatibility impact and the corresponding version number in the PR description
  • I have checked whether the plugin version is updated in the README.md

Available Checks

  • Code has passed local tests
  • Relevant documentation has been updated (if necessary)

🔄 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-plugin-sdks/pull/263 **Author:** [@Mairuis](https://github.com/Mairuis) **Created:** 12/9/2025 **Status:** ✅ Merged **Merged:** 12/9/2025 **Merged by:** [@Mairuis](https://github.com/Mairuis) **Base:** `main` ← **Head:** `fix/revert_max_tokens` --- ### 📝 Commits (1) - [`97f9d2c`](https://github.com/langgenius/dify-plugin-sdks/commit/97f9d2c83f364ef1c8eb08716ca85fb804bd8f56) Revert "fix: increase max_tokens from 5 to 16 for some specific models like gpt-5-codex in ping test, add PING_MAX_TOKENS configuration for model validation ping requests (#235)" ### 📊 Changes **2 files changed** (+2 additions, -11 deletions) <details> <summary>View changed files</summary> 📝 `python/dify_plugin/config/config.py` (+0 -5) 📝 `python/dify_plugin/interfaces/model/openai_compatible/llm.py` (+2 -6) </details> ### 📄 Description This reverts commit ee8ea3de16b91d6bcde824da6460f78037e4d55d. # Pull Request Checklist Thank you for your contribution! Before submitting your PR, please make sure you have completed the following checks: ## Compatibility Check - [ ] I have checked whether this change affects the **backward compatibility** of the plugin declared in `README.md` - [ ] I have checked whether this change affects the **forward compatibility** of the plugin declared in `README.md` - [ ] If this change introduces a breaking change, I have discussed it with the project maintainer and specified the release version in the `README.md` - [ ] I have described the compatibility impact and the corresponding version number in the PR description - [ ] I have checked whether the plugin version is updated in the `README.md` ## Available Checks - [x] Code has passed local tests - [x] Relevant documentation has been updated (if necessary) --- <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-15 21:16:31 -05:00
yindo closed this issue 2026-02-15 21:16:31 -05:00
Sign in to join this conversation.
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: langgenius/dify-plugin-sdks#270