Skip to content

chore: use zstd for plugin model provider cache#38382

Merged
hjlarry merged 2 commits into
langgenius:mainfrom
hjlarry:codex/zstd-plugin-model-cache
Jul 3, 2026
Merged

chore: use zstd for plugin model provider cache#38382
hjlarry merged 2 commits into
langgenius:mainfrom
hjlarry:codex/zstd-plugin-model-cache

Conversation

@hjlarry

@hjlarry hjlarry commented Jul 3, 2026

Copy link
Copy Markdown
Contributor

Important

  1. Make sure you have read our contribution guidelines
  2. Ensure there is an associated issue and you have been assigned to it
  3. Use the correct syntax to link this PR: Fixes #<issue number>.

Summary

the follow of #38374

Screenshots

Before After
... ...

Checklist

  • This change requires a documentation update, included: Dify Document
  • I understand that this PR may be closed in case there was no previous discussion or issues. (This doesn't apply to typos!)
  • I've added a test for each change that was introduced, and I tried as much as possible to make a single atomic change.
  • I've updated the documentation accordingly.
  • I ran make lint && make type-check (backend) and cd web && pnpm exec vp staged (frontend) to appease the lint gods

@hjlarry hjlarry requested a review from a team July 3, 2026 08:18
@dosubot dosubot Bot added size:S This PR changes 10-29 lines, ignoring generated files. dependencies Pull requests that update a dependency file labels Jul 3, 2026
@hjlarry hjlarry enabled auto-merge July 3, 2026 08:19
@github-actions

github-actions Bot commented Jul 3, 2026 •
edited
Loading

Copy link
Copy Markdown
Contributor

Pyrefly Type Coverage

Metric Base PR Delta
Type coverage 52.30% 52.30% 0.00%
Strict coverage 51.81% 51.81% 0.00%
Typed symbols 32,016 32,016 0
Untyped symbols 29,477 29,477 0
Modules 2970 2970 0

crazywoola
crazywoola previously approved these changes Jul 3, 2026
View reviewed changes
@dosubot dosubot Bot added the lgtm This PR has been approved by a maintainer label Jul 3, 2026
41tair
41tair previously approved these changes Jul 3, 2026
View reviewed changes
@hjlarry hjlarry added this pull request to the merge queue Jul 3, 2026
QuantumGhost
QuantumGhost reviewed Jul 3, 2026
View reviewed changes
Comment thread api/pyproject.toml Outdated
@hjlarry hjlarry removed this pull request from the merge queue due to a manual request Jul 3, 2026
@hjlarry hjlarry dismissed stale reviews from 41tair and crazywoola via 5a9a88c July 3, 2026 08:41
@hjlarry hjlarry enabled auto-merge July 3, 2026 08:42
@hjlarry hjlarry added this pull request to the merge queue Jul 3, 2026
Merged via the queue into langgenius:main with commit 91cc503 Jul 3, 2026
34 checks passed
@hjlarry hjlarry deleted the codex/zstd-plugin-model-cache branch July 3, 2026 09:13
hjlarry added a commit to hjlarry/dify that referenced this pull request Jul 3, 2026
@hjlarry
chore: use zstd for plugin model provider cache (langgenius#38382)
357706e 
(cherry picked from commit 91cc503)
hjlarry added a commit that referenced this pull request Jul 3, 2026
@hjlarry
chore: use zstd for plugin model provider cache (#38382)
6c4ba71 
(cherry picked from commit 91cc503)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@QuantumGhost QuantumGhost QuantumGhost approved these changes
@41tair 41tair 41tair left review comments
@crazywoola crazywoola crazywoola left review comments
@WH-2099 WH-2099 Awaiting requested review from WH-2099 WH-2099 is a code owner
@laipz8200 laipz8200 Awaiting requested review from laipz8200 laipz8200 is a code owner

Assignees

No one assigned

Labels

dependencies Pull requests that update a dependency file in-linear lgtm This PR has been approved by a maintainer size:S This PR changes 10-29 lines, ignoring generated files.

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

4 participants

@hjlarry @QuantumGhost @41tair @crazywoola