langgenius/dify-plugin-sdks
1
0
Fork 0
mirror of https://github.com/langgenius/dify-plugin-sdks.git synced 2026-07-21 18:05:30 -04:00
Code Issues 58 Packages Projects Releases 10 Wiki Activity

[PR #274] [CLOSED] Make validate_credentials max_tokens configurable and bump SDK version #278

New Issue
Closed
opened 2026-02-15 21:16:32 -05:00 by yindo · 0 comments
yindo commented 2026-02-15 21:16:32 -05:00
Owner
Copy Link
Copy Source

📋 Pull Request Information

Original PR: https://github.com/langgenius/dify-plugin-sdks/pull/274
Author: @Yeuoly
Created: 12/27/2025
Status: ❌ Closed

Base: main ← Head: codex/make-max_tokens-configurable-xjc583

📝 Commits (1)

  • 6365baf Apply ruff auto-fix adjustments

📊 Changes

2 files changed (+10 additions, -3 deletions)

View changed files

📝 python/dify_plugin/interfaces/model/openai_compatible/llm.py (+9 -2)
📝 python/pyproject.toml (+1 -1)

📄 Description

Motivation

  • Allow tuning the ping max_tokens used during credentials validation so providers with different behaviors can be supported.
  • Preserve the streaming validation behavior that historically used max_tokens=10 to ensure a token chunk is emitted while still permitting explicit overrides.
  • Keep project metadata up to date by bumping the SDK package version.

Description

  • Read validate_credentials_max_tokens from credentials and use it as the ping max_tokens for non-stream validation in python/dify_plugin/interfaces/model/openai_compatible/llm.py.
  • For stream-mode validation, keep the default at 10 but allow overriding when validate_credentials_max_tokens is explicitly provided, and add an inline comment explaining the rationale.
  • Bump SDK version to 0.7.1 in python/pyproject.toml and update python/pdm.lock accordingly, and apply small lint/format adjustments in llm.py.

Testing

  • Ran ./scripts/fix.sh which executed pdm run ruff format and pdm run ruff check --fix and completed successfully with one auto-fix applied.
  • No additional automated test suite was executed for this change.

Codex Task

🔄 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/274 **Author:** [@Yeuoly](https://github.com/Yeuoly) **Created:** 12/27/2025 **Status:** ❌ Closed **Base:** `main` ← **Head:** `codex/make-max_tokens-configurable-xjc583` --- ### 📝 Commits (1) - [`6365baf`](https://github.com/langgenius/dify-plugin-sdks/commit/6365baf9afb557bb157959e3f408186cf7404015) Apply ruff auto-fix adjustments ### 📊 Changes **2 files changed** (+10 additions, -3 deletions) <details> <summary>View changed files</summary> 📝 `python/dify_plugin/interfaces/model/openai_compatible/llm.py` (+9 -2) 📝 `python/pyproject.toml` (+1 -1) </details> ### 📄 Description ### Motivation - Allow tuning the ping `max_tokens` used during credentials validation so providers with different behaviors can be supported. - Preserve the streaming validation behavior that historically used `max_tokens=10` to ensure a token chunk is emitted while still permitting explicit overrides. - Keep project metadata up to date by bumping the SDK package version. ### Description - Read `validate_credentials_max_tokens` from `credentials` and use it as the ping `max_tokens` for non-stream validation in `python/dify_plugin/interfaces/model/openai_compatible/llm.py`. - For stream-mode validation, keep the default at `10` but allow overriding when `validate_credentials_max_tokens` is explicitly provided, and add an inline comment explaining the rationale. - Bump SDK version to `0.7.1` in `python/pyproject.toml` and update `python/pdm.lock` accordingly, and apply small lint/format adjustments in `llm.py`. ### Testing - Ran `./scripts/fix.sh` which executed `pdm run ruff format` and `pdm run ruff check --fix` and completed successfully with one auto-fix applied. - No additional automated test suite was executed for this change. ------ [Codex Task](https://chatgpt.com/codex/tasks/task_b_69491f6bf5648326a7880f45f886800d) --- <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:32 -05:00
yindo closed this issue 2026-02-15 21:16:32 -05:00
Sign in to join this conversation.
No Branch/Tag Specified
Branches Tags
Labels
Clear labels
bug
documentation
enhancement
pull-request
Mirrored from GitHub Pull Request
 question
No Label pull-request
Milestone
No items
No Milestone
Projects
Clear projects
No project
Assignees
Clear assignees
yindo
No Assignees
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: langgenius/dify-plugin-sdks#278
Delete Branch "%!s()"

Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?